Docker login before compose pull #832

Closed
opened 2026-03-22 16:54:41 -05:00 by GiteaMirror · 3 comments
Owner

Originally created by @febalist on GitHub (Aug 20, 2024).

During the deploy stack process, I encounter an “access forbidden” error at the compose pull stage:
CleanShot 2024-08-20 at 09 51 34@2x

It looks like docker login isn’t happening before compose pull, even though config.registry_account is set up.

In the code, I noticed that in /bin/periphery/src/compose.rs, docker login happens after compose pull. It might need to be swapped to ensure login happens first.

Core 1.13.3
Periphery 1.13.3

Originally created by @febalist on GitHub (Aug 20, 2024). During the deploy stack process, I encounter an “access forbidden” error at the compose pull stage: ![CleanShot 2024-08-20 at 09 51 34@2x](https://github.com/user-attachments/assets/3d245360-5573-4195-ab3a-674a2345ae7b) It looks like docker login isn’t happening before compose pull, even though config.registry_account is set up. In the code, I noticed that in /bin/periphery/src/compose.rs, docker login happens after compose pull. It might need to be swapped to ensure login happens first. Core 1.13.3 Periphery 1.13.3
GiteaMirror added the donebug labels 2026-03-22 16:54:41 -05:00
Author
Owner

@mbecker20 commented on GitHub (Aug 20, 2024):

You are correct, thanks for the Report! Fixed in this commit: https://github.com/mbecker20/monitor/commit/a2758ce6f4c557b52ec8acb49e929648c4d48c91

I'm building 1.13.4 now to address this issue, will release shortly. Thanks for your interest in the project!

@mbecker20 commented on GitHub (Aug 20, 2024): You are correct, thanks for the Report! Fixed in this commit: [https://github.com/mbecker20/monitor/commit/a2758ce6f4c557b52ec8acb49e929648c4d48c91](https://github.com/mbecker20/monitor/commit/a2758ce6f4c557b52ec8acb49e929648c4d48c91) I'm building 1.13.4 now to address this issue, will release shortly. Thanks for your interest in the project!
Author
Owner

@mbecker20 commented on GitHub (Aug 20, 2024):

Hey, the lastest release is out and should fix the issue.

https://github.com/mbecker20/monitor/releases/tag/v1.13.4

@mbecker20 commented on GitHub (Aug 20, 2024): Hey, the lastest release is out and should fix the issue. [https://github.com/mbecker20/monitor/releases/tag/v1.13.4](https://github.com/mbecker20/monitor/releases/tag/v1.13.4)
Author
Owner

@febalist commented on GitHub (Aug 20, 2024):

Thank you for the quick fix! The project looks incredibly useful and convenient.

@febalist commented on GitHub (Aug 20, 2024): Thank you for the quick fix! The project looks incredibly useful and convenient.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/komodo#832