Skip to content

Conversation

@kinarashah
Copy link
Member

No description provided.

echo "IMAGE=${STAGE_REGISTRY_ENDPOINT}/rke-extended-life/hyperkube" >> "$GITHUB_ENV"
- name: Logout from Docker registry
run: docker logout ${{ env.REGISTRY_ENDPOINT }} || true
run: docker logout ${{ env.STAGE_REGISTRY_ENDPOINT }} || true

Choose a reason for hiding this comment

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

why do we need to logout?
the runners are supposed to be single use, so there should be no need for this afaik..

Choose a reason for hiding this comment

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

also we have https://github.com/docker/login-action?tab=readme-ov-file#inputs logout option with that in case we need to do that..

Copy link
Member Author

Choose a reason for hiding this comment

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

Added it earlier to fix issues with login based on EIO team's suggestion. Will remove, no longer required

Copy link

@mitulshah-suse mitulshah-suse left a comment

Choose a reason for hiding this comment

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

LGTM

@kinarashah kinarashah merged commit 9498038 into rancher:v1.32 Aug 6, 2025
3 checks passed
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.

2 participants