Skip to content

Skip docker registry login by default, login only when pulp is deployed #891

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Mar 6, 2024

Conversation

MoteHue
Copy link
Contributor

@MoteHue MoteHue commented Jan 19, 2024

No description provided.

@MoteHue MoteHue requested a review from a team as a code owner January 19, 2024 14:35
@MoteHue MoteHue mentioned this pull request Jan 19, 2024
@MoteHue MoteHue force-pushed the pulp-post-deploy-login branch from 97f5e34 to 740e90e Compare January 24, 2024 09:59
@MoteHue MoteHue force-pushed the pulp-post-deploy-login branch from 740e90e to ac809cb Compare January 24, 2024 10:36
Copy link
Contributor

@markgoddard markgoddard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but I assume it's untested?

@MoteHue
Copy link
Contributor Author

MoteHue commented Jan 24, 2024

LGTM, but I assume it's untested?

Yes it's untested

@MoteHue
Copy link
Contributor Author

MoteHue commented Mar 6, 2024

LGTM, but I assume it's untested?

Yes it's untested

Have now tested, it works.

@MoteHue MoteHue requested a review from markgoddard March 6, 2024 10:14
@markgoddard markgoddard merged commit 65a3d38 into stackhpc/yoga Mar 6, 2024
@markgoddard markgoddard deleted the pulp-post-deploy-login branch March 6, 2024 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants