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

Makefile: fix 'fast' make targets #28380

Merged
merged 2 commits into from Oct 5, 2023
Merged

Conversation

aanm
Copy link
Member

@aanm aanm commented Oct 3, 2023

These targets failed to select the nodes of a kind cluster. This change selects the right variables that are used to check for the cluster nodes.

Fixes 5f88bbf ("docs: Add Makefile and documentation for "fast" development targets")

@aanm aanm requested a review from a team as a code owner October 3, 2023 18:32
@aanm aanm requested a review from jibi October 3, 2023 18:32
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Oct 3, 2023
@aanm aanm added the release-note/misc This PR makes changes that have no direct user impact. label Oct 3, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Oct 3, 2023
These targets failed to select the nodes of a kind cluster. This change
selects the right variables that are used to check for the cluster
nodes.

Fixes: 5f88bbf ("docs: Add Makefile and documentation for "fast" development targets")
Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm added needs-backport/1.14 This PR / issue needs backporting to the v1.14 branch area/build Anything to do with the build, more general then area/CI labels Oct 3, 2023
@aanm
Copy link
Member Author

aanm commented Oct 3, 2023

/test

@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from main in 1.14.3 Oct 3, 2023
The kind-install-cilium-fast should install Cilium on all available
kind clusters.

Fixes: 5f88bbf ("docs: Add Makefile and documentation for "fast" development targets")
Signed-off-by: André Martins <andre@cilium.io>
@aanm
Copy link
Member Author

aanm commented Oct 3, 2023

/test

@aanm aanm merged commit 6ea3ed5 into cilium:main Oct 5, 2023
58 of 61 checks passed
@sayboras sayboras mentioned this pull request Oct 6, 2023
9 tasks
@sayboras sayboras added backport-pending/1.14 The backport for Cilium 1.14.x for this PR is in progress. and removed needs-backport/1.14 This PR / issue needs backporting to the v1.14 branch labels Oct 6, 2023
@github-actions github-actions bot added backport-done/1.14 The backport for Cilium 1.14.x for this PR is done. and removed backport-pending/1.14 The backport for Cilium 1.14.x for this PR is in progress. labels Oct 9, 2023
@jrajahalme jrajahalme moved this from Needs backport from main to Backport done to v1.14 in 1.14.3 Oct 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/build Anything to do with the build, more general then area/CI backport-done/1.14 The backport for Cilium 1.14.x for this PR is done. release-note/misc This PR makes changes that have no direct user impact.
Projects
No open projects
1.14.3
Backport done to v1.14
Development

Successfully merging this pull request may close these issues.

None yet

3 participants