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

docs: add instructions to build kindest-node image #29079

Merged
merged 1 commit into from Nov 10, 2023

Conversation

aanm
Copy link
Member

@aanm aanm commented Nov 9, 2023

Add instructions on how to update kindest-node image so that others can also perform this tasks if necessary.

@aanm aanm added area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. sig/k8s Impacts the kubernetes API, or kubernetes -> cilium internals translation layers. release-note/misc This PR makes changes that have no direct user impact. labels Nov 9, 2023
@aanm aanm requested review from a team as code owners November 9, 2023 11:26
@aanm aanm requested review from bimmlerd and qmonnet November 9, 2023 11:26
@aanm aanm force-pushed the pr/add-nodes-for-kind-upgrade branch from c7eddff to c308c99 Compare November 9, 2023 11:28
@aanm aanm mentioned this pull request Nov 9, 2023
Copy link
Member

@qmonnet qmonnet left a comment

Choose a reason for hiding this comment

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

Looks good overall, but I've got various minor suggestions. They're not blocking and you are free to ignore them.

Documentation/contributing/development/dev_setup.rst Outdated Show resolved Hide resolved
Documentation/contributing/development/dev_setup.rst Outdated Show resolved Hide resolved
Documentation/contributing/development/dev_setup.rst Outdated Show resolved Hide resolved
Documentation/contributing/development/dev_setup.rst Outdated Show resolved Hide resolved

make -C images/base/ quick

Note the resulting image tag for that command, it should be last tag built for
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Note the resulting image tag for that command, it should be last tag built for
Note the resulting image tag for that command, it should be the last tag built for

How do you check the tags for the repository? Would it make sense to turn it into a URL (https://gcr.io/k8s-staging-kind/base)?

Copy link
Member Author

Choose a reason for hiding this comment

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

I've changed it to:

Take note of the resulting image tag for that command, it should be the last
tag built for the ``gcr.io/k8s-staging-kind/base`` repository in ``docker ps -a``.

Documentation/contributing/development/dev_setup.rst Outdated Show resolved Hide resolved
Documentation/contributing/development/dev_setup.rst Outdated Show resolved Hide resolved
Documentation/contributing/development/dev_setup.rst Outdated Show resolved Hide resolved
Documentation/contributing/development/dev_setup.rst Outdated Show resolved Hide resolved
Add instructions on how to update kindest-node image so that others can
also perform this tasks if necessary.

Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm force-pushed the pr/add-nodes-for-kind-upgrade branch from c308c99 to 46984e2 Compare November 9, 2023 17:09
@aanm aanm requested a review from qmonnet November 9, 2023 17:10
@aanm
Copy link
Member Author

aanm commented Nov 9, 2023

/test

Copy link
Member

@qmonnet qmonnet left a comment

Choose a reason for hiding this comment

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

Thanks!

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Nov 10, 2023
@squeed squeed merged commit ca17420 into cilium:main Nov 10, 2023
58 checks passed
@aanm aanm deleted the pr/add-nodes-for-kind-upgrade branch November 10, 2023 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact. sig/k8s Impacts the kubernetes API, or kubernetes -> cilium internals translation layers.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants