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 #4546

Merged
merged 6 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

…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
Copy link
Contributor

@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
Contributor

kkotula commented Sep 29, 2023

PRs in other services for reference:
Deck
Rosco

…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 Merged automatically by a bot label Oct 9, 2023
@mergify mergify bot merged commit cd9781f into spinnaker:master Oct 9, 2023
4 checks passed
@ovidiupopa07
Copy link
Contributor

https://github.com/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

@ovidiupopa07
Copy link
Contributor

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

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

* 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: ovidiupopa07 <105648914+ovidiupopa07@users.noreply.github.com>
(cherry picked from commit cd9781f)
mergify bot pushed a commit that referenced this pull request Oct 11, 2023
…ails of the APIs versions (#4546)

* 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: ovidiupopa07 <105648914+ovidiupopa07@users.noreply.github.com>
(cherry picked from commit cd9781f)
mergify bot pushed a commit that referenced this pull request Oct 11, 2023
…ails of the APIs versions (#4546)

* 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: ovidiupopa07 <105648914+ovidiupopa07@users.noreply.github.com>
(cherry picked from commit cd9781f)
mergify bot pushed a commit that referenced this pull request Oct 11, 2023
…ails of the APIs versions (#4546)

* 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: ovidiupopa07 <105648914+ovidiupopa07@users.noreply.github.com>
(cherry picked from commit cd9781f)
mergify bot added a commit that referenced this pull request Oct 18, 2023
…ails of the APIs versions (#4546) (#4562)

* 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: ovidiupopa07 <105648914+ovidiupopa07@users.noreply.github.com>
(cherry picked from commit cd9781f)

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 (#4546) (#4563)

* 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: ovidiupopa07 <105648914+ovidiupopa07@users.noreply.github.com>
(cherry picked from commit cd9781f)

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 (#4546) (#4564)

* 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: ovidiupopa07 <105648914+ovidiupopa07@users.noreply.github.com>
(cherry picked from commit cd9781f)

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 (#4546) (#4565)

* 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: ovidiupopa07 <105648914+ovidiupopa07@users.noreply.github.com>
(cherry picked from commit cd9781f)

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
auto merged Merged automatically by a bot ready to merge Approved and ready for merge target-release/1.33
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants