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: Remove non-x86 restriction #25422

Merged
merged 1 commit into from May 23, 2023

Conversation

jrajahalme
Copy link
Member

Remove logic from Documentation/Makefile that skips building 'update-helm-values' on non-x86 platforms. This limitation is no longer needed as we use the helm toolbox image, which is available for multiple architectures.

Note to backporter: This change depends on #20236, which needs to be backported first.

Fixes: #20236

Updating documentation helm values now works also on arm64.

Remove logic from Documentation/Makefile that skips building
'update-helm-values' on non-x86 platforms. This limitation is no longer
needed as we use the helm toolbox image, which is available for multiple
architectures.

Fixes: cilium#20236
Signed-off-by: Jarno Rajahalme <jarno@isovalent.com>
@jrajahalme jrajahalme added area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. release-note/minor This PR changes functionality that users may find relevant to operating Cilium. needs-backport/1.12 needs-backport/1.13 This PR / issue needs backporting to the v1.13 branch labels May 12, 2023
@jrajahalme jrajahalme requested a review from a team as a code owner May 12, 2023 14:40
@jrajahalme jrajahalme requested a review from qmonnet May 12, 2023 14:40
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from main in 1.12.10 May 12, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from main in 1.13.3 May 12, 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.

Thanks!

@thorn3r thorn3r added this to Needs backport from main in 1.12.11 May 16, 2023
@thorn3r thorn3r removed this from Needs backport from main in 1.12.10 May 16, 2023
@thorn3r thorn3r added this to Needs backport from main in 1.13.4 May 17, 2023
@thorn3r thorn3r removed this from Needs backport from main in 1.13.3 May 17, 2023
@jrajahalme jrajahalme merged commit bfa4656 into cilium:main May 23, 2023
37 checks passed
@sayboras sayboras mentioned this pull request May 28, 2023
10 tasks
@sayboras sayboras added the backport-pending/1.13 The backport for Cilium 1.13.x for this PR is in progress. label May 28, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from main in 1.12.10 May 28, 2023
@sayboras sayboras mentioned this pull request May 28, 2023
5 tasks
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from main to Backport pending to v1.12 in 1.12.10 May 28, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from main to Backport pending to v1.12 in 1.12.10 May 28, 2023
@sayboras sayboras added backport-done/1.13 The backport for Cilium 1.13.x for this PR is done. and removed backport-pending/1.13 The backport for Cilium 1.13.x for this PR is in progress. needs-backport/1.13 This PR / issue needs backporting to the v1.13 branch labels Jun 2, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Backport done to v1.13 in 1.13.3 Jun 2, 2023
@qmonnet qmonnet moved this from Needs backport from main to Backport done to v1.13 in 1.13.4 Jun 9, 2023
@michi-covalent michi-covalent added this to Backport pending to v1.12 in 1.12.12 Jun 12, 2023
@michi-covalent michi-covalent removed this from Needs backport from main in 1.12.11 Jun 12, 2023
@julianwiedmann julianwiedmann added backport-done/1.12 The backport for Cilium 1.12.x for this PR is done. and removed backport-pending/1.12 labels Jul 10, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.12 to Backport done to v1.12 in 1.12.10 Jul 10, 2023
@gentoo-root gentoo-root moved this from Backport pending to v1.12 to Backport done to v1.12 in 1.12.12 Jul 26, 2023
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. backport-done/1.12 The backport for Cilium 1.12.x for this PR is done. backport-done/1.13 The backport for Cilium 1.13.x for this PR is done. release-note/minor This PR changes functionality that users may find relevant to operating Cilium.
Projects
No open projects
1.12.10
Backport done to v1.12
1.12.12
Backport done to v1.12
1.13.3
Backport done to v1.13
1.13.4
Backport done to v1.13
Development

Successfully merging this pull request may close these issues.

None yet

4 participants