Skip to content
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

chore: changes from org move #90

Merged
merged 1 commit into from
Feb 7, 2024
Merged

Conversation

richardcase
Copy link
Collaborator

Changes as a result of moving the repo to the k3s org. This includes:

  • Package names/imports
  • Image names

This includes various changes as a result of moving the repo to the k3s
org.

Signed-off-by: Richard Case <[email protected]>
@zawachte
Copy link
Collaborator

zawachte commented Feb 7, 2024

moving quick :)

@@ -110,7 +110,7 @@ $(RELEASE_DIR):
$(RELEASE_NOTES_DIR):
mkdir -p $(RELEASE_NOTES_DIR)/

REGISTRY ?= ghcr.io/cluster-api-provider-k3s/cluster-api-k3s
REGISTRY ?= ghcr.io/k3s-io/cluster-api-k3s
Copy link
Collaborator

Choose a reason for hiding this comment

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

unsure how the perms on this work for this org. Do you have any clue?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think we should be good as the workflow specifies write packages.

I have update the docker file to label it from this new repo. Also, i tested that the old images are still available. So this worked:

docker pull ghcr.io/cluster-api-provider-k3s/cluster-api-k3s/bootstrap-controller:v0.1.9

@zawachte
Copy link
Collaborator

zawachte commented Feb 7, 2024

Going to merge this and we can figure out the container registry/release details after.

@zawachte zawachte merged commit b894015 into k3s-io:main Feb 7, 2024
5 checks passed
@richardcase richardcase deleted the org_move branch February 8, 2024 09:40
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