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

feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions #1020

Merged
merged 13 commits into from
Oct 9, 2023

Conversation

ciurescuraul
Copy link
Contributor

@ciurescuraul ciurescuraul commented Sep 29, 2023

It would be incredibly helpful if SpEL expression parameters could be used for the API and Kubernetes versions in the Bake Manifest stage when baking Helm charts. This feature would allow users to deploy different versions of artifacts based on the target cluster API and K8s versions, significantly improving their workflow.

  • These input fields will not be pre-populated with versions of the target cluster available in the environment

  • They will become part of the bake result when API_VERSIONS_ENABLED flag is enabled

  • ApiVersions should be a subset of the API versions supported by the Kubernetes version specified in KubeVersion. In other words, the Kubernetes cluster version (Capabilities.KubeVersion) determines the range of API versions that are available in the cluster (Capabilities.APIVersions).

  • Helm options configuration
    1-1-1

  • Evaluate variables stage
    5-5

@spinnakerbot
Copy link
Contributor

The following commits need their title changed:

  • ea6662b: security(feature): Change codeql to scan daily instead of weekly

Please format your commit title into the form:

<type>(<scope>): <subject>, e.g. fix(kubernetes): address NPE in status check

This allows us to easily generate changelogs & determine semantic version numbers when cutting releases. You can read more about commit conventions here.

Copy link

@kkotula kkotula left a comment

Choose a reason for hiding this comment

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

LGTM

@kkotula
Copy link

kkotula commented Sep 29, 2023

PRs in other services for reference:
Deck
Orca

…ails of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag
…ails of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag
@jasonmcintosh jasonmcintosh added the ready to merge Approved and ready for merge label Oct 9, 2023
@mergify mergify bot added the auto merged label Oct 9, 2023
@mergify mergify bot merged commit 4a12958 into spinnaker:master Oct 9, 2023
4 checks passed
@ciurescuraul
Copy link
Contributor Author

@Mergifyio backport release-2.30.x

@mergify
Copy link
Contributor

mergify bot commented Oct 10, 2023

backport release-2.30.x

❌ Command disallowed due to command restrictions in the Mergify configuration.

  • sender-permission>=write

@ovidiupopa07
Copy link

ovidiupopa07 commented Oct 11, 2023

@Mergifyio backport release-1.28.x release-1.30.x release-1.31.x release-1.32.x

@mergify
Copy link
Contributor

mergify bot commented Oct 11, 2023

backport release-2.28.x release-2.30.x release-2.31.x release-2.32.x

❌ No backport have been created

  • Backport to branch release-2.28.x failed

GitHub error: Branch not found

  • Backport to branch release-2.30.x failed

GitHub error: Branch not found

  • Backport to branch release-2.31.x failed

GitHub error: Branch not found

  • Backport to branch release-2.32.x failed

GitHub error: Branch not found

@ovidiupopa07
Copy link

Since this new functionality is hidden behind a feature flag it is ok to backport

@mergify
Copy link
Contributor

mergify bot commented Oct 11, 2023

backport release-1.28.x release-1.30.x release-1.31.x release-1.32.x

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Oct 11, 2023
…ails of the APIs versions (#1020)

* security(feature): Change codeql to scan daily instead of weekly

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

---------

Co-authored-by: Jason McIntosh <jason.mcintosh@armory.io>
(cherry picked from commit 4a12958)
mergify bot pushed a commit that referenced this pull request Oct 11, 2023
…ails of the APIs versions (#1020)

* security(feature): Change codeql to scan daily instead of weekly

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

---------

Co-authored-by: Jason McIntosh <jason.mcintosh@armory.io>
(cherry picked from commit 4a12958)

# Conflicts:
#	rosco-manifests/src/main/java/com/netflix/spinnaker/rosco/manifests/helm/HelmTemplateUtils.java
#	rosco-manifests/src/test/java/com/netflix/spinnaker/rosco/manifests/helm/HelmTemplateUtilsTest.java
mergify bot pushed a commit that referenced this pull request Oct 11, 2023
…ails of the APIs versions (#1020)

* security(feature): Change codeql to scan daily instead of weekly

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

---------

Co-authored-by: Jason McIntosh <jason.mcintosh@armory.io>
(cherry picked from commit 4a12958)
mergify bot pushed a commit that referenced this pull request Oct 11, 2023
…ails of the APIs versions (#1020)

* security(feature): Change codeql to scan daily instead of weekly

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

---------

Co-authored-by: Jason McIntosh <jason.mcintosh@armory.io>
(cherry picked from commit 4a12958)
mergify bot added a commit that referenced this pull request Oct 18, 2023
…ails of the APIs versions (#1020) (#1029)

* security(feature): Change codeql to scan daily instead of weekly

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

---------

Co-authored-by: Jason McIntosh <jason.mcintosh@armory.io>
(cherry picked from commit 4a12958)

Co-authored-by: Krystian <24556350+ciurescuraul@users.noreply.github.com>
mergify bot added a commit that referenced this pull request Oct 18, 2023
…ails of the APIs versions (#1020) (#1031)

* security(feature): Change codeql to scan daily instead of weekly

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

---------

Co-authored-by: Jason McIntosh <jason.mcintosh@armory.io>
(cherry picked from commit 4a12958)

Co-authored-by: Krystian <24556350+ciurescuraul@users.noreply.github.com>
mergify bot added a commit that referenced this pull request Oct 18, 2023
…ails of the APIs versions (#1020) (#1030)

* security(feature): Change codeql to scan daily instead of weekly

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

---------

Co-authored-by: Jason McIntosh <jason.mcintosh@armory.io>
(cherry picked from commit 4a12958)

Co-authored-by: Krystian <24556350+ciurescuraul@users.noreply.github.com>
dogonthehorizon pushed a commit that referenced this pull request Oct 18, 2023
…ails of the APIs versions (backport #1020) (#1028)

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions (#1020)

* security(feature): Change codeql to scan daily instead of weekly

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

---------

Co-authored-by: Jason McIntosh <jason.mcintosh@armory.io>
(cherry picked from commit 4a12958)

# Conflicts:
#	rosco-manifests/src/main/java/com/netflix/spinnaker/rosco/manifests/helm/HelmTemplateUtils.java
#	rosco-manifests/src/test/java/com/netflix/spinnaker/rosco/manifests/helm/HelmTemplateUtilsTest.java

* feat(helm/bake): Fix back-port into release 1.28.x (#1034)

---------

Co-authored-by: Krystian <24556350+ciurescuraul@users.noreply.github.com>
nemesisOsorio pushed a commit to OpsMx/rosco-oes that referenced this pull request Apr 27, 2024
…ails of the APIs versions (spinnaker#1020) (spinnaker#1030)

* security(feature): Change codeql to scan daily instead of weekly

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

* feat(helm/bake): Add additional input fields where we can fill in details of the APIs versions

- These input fields will not be pre-populated with versions of the target cluster available in the environment.

- They will become part of the bake result.

- Added API_VERSIONS_ENABLED env variable flag

---------

Co-authored-by: Jason McIntosh <jason.mcintosh@armory.io>
(cherry picked from commit 4a12958)

Co-authored-by: Krystian <24556350+ciurescuraul@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
6 participants