feat: Retrieves auth token from image #585

Merged
decentral1se merged 1 commits from p4u1/abra:private-registry into main 2025-08-12 05:01:42 +00:00
Member

This allows using a private registry for an image.
To use it, you have to run docker login on your local machine before
running abra deploy.

This allows using a private registry for an image. To use it, you have to run docker login on your local machine before running abra deploy.
p4u1 added 1 commit 2025-08-11 09:10:04 +00:00
feat: Retrieves auth token from image
Some checks failed
continuous-integration/drone/pr Build is failing
4a0f677e40
This allows using a private registry for an image.
To use it, you have to run docker login on your local machine before
running abra deploy.
p4u1 force-pushed private-registry from 4a0f677e40 to ce3cd85271 2025-08-11 09:30:42 +00:00 Compare
decentral1se reviewed 2025-08-11 12:45:20 +00:00
decentral1se left a comment
Owner

Great stuff, thanks! We wanted this since ages ago: toolshed/organising#121

Would you be able to add some docs for this also? A small entry in https://docs.coopcloud.tech/operators/handbook/.

Few small comments.

Great stuff, thanks! We wanted this since ages ago: https://git.coopcloud.tech/toolshed/organising/issues/121 Would you be able to add some docs for this also? A small entry in https://docs.coopcloud.tech/operators/handbook/. Few small comments.
@ -449,0 +455,4 @@
if sendAuth {
dockerCLI, err := command.NewDockerCli()
if err != nil {
log.Errorf("failed create docker cli: %s", err)
Owner

An error like this will be very confusing without context, maybe we need to add more? E.g.

log.Errorf("retrieving docker auth token: failed create docker cli: %s", err)

An error like this will be very confusing without context, maybe we need to add more? E.g. `log.Errorf("retrieving docker auth token: failed create docker cli: %s", err)`
Author
Member

I updated the error message

I updated the error message
decentral1se marked this conversation as resolved
@ -588,3 +605,3 @@
)
if err := os.MkdirAll(filepath.Join(config.LOGS_DIR, opts.ServerName), 0764); err != nil {
if err := os.MkdirAll(filepath.Join(config.LOGS_DIR, opts.ServerName), 0o764); err != nil {
Owner

This seems unrelated, are you sure this won't break things?

This seems unrelated, are you sure this won't break things?
Author
Member

this is a change by the formatter, and is semantically the same

this is a change by the formatter, and is semantically the same
decentral1se marked this conversation as resolved
decentral1se added this to the Abra v0.11.x project 2025-08-11 12:46:15 +00:00
p4u1 force-pushed private-registry from ce3cd85271 to 7a24bf9cd5 2025-08-11 13:53:27 +00:00 Compare
decentral1se approved these changes 2025-08-12 05:01:35 +00:00
decentral1se merged commit 157d131b37 into main 2025-08-12 05:01:42 +00:00
Owner
https://git.coopcloud.tech/toolshed/docs.coopcloud.tech/commit/4dea3ac3ca0d1e8899fc80d1770e1a9029f3d644
decentral1se moved this to Done in Abra v0.11.x on 2025-08-12 05:21:33 +00:00
Sign in to join this conversation.
No description provided.