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

[installer] Disable definitely-gp by default #8740

Closed
3 tasks done
corneliusludmann opened this issue Mar 11, 2022 · 0 comments · Fixed by #9094
Closed
3 tasks done

[installer] Disable definitely-gp by default #8740

corneliusludmann opened this issue Mar 11, 2022 · 0 comments · Fixed by #9094
Assignees
Labels
component: install Terraform installation scripts, helm charts, installer images team: delivery Issue belongs to the self-hosted team

Comments

@corneliusludmann
Copy link
Contributor

corneliusludmann commented Mar 11, 2022

The installer has a config value definitelyGpDisabled that is currently false by default. Since we deprecate definitely-gp soon, we should set it to true by default.

Things to consider:

@corneliusludmann corneliusludmann added component: install Terraform installation scripts, helm charts, installer images team: delivery Issue belongs to the self-hosted team labels Mar 11, 2022
@corneliusludmann corneliusludmann added this to the release/2022.03 milestone Mar 11, 2022
@Pothulapati Pothulapati self-assigned this Apr 1, 2022
@Pothulapati Pothulapati moved this from 🤝Proposed to ⚒In Progress in 🚚 Security, Infrastructure, and Delivery Team (SID) Apr 1, 2022
Pothulapati added a commit that referenced this issue Apr 4, 2022
Fixes #8740

`definitely-gp` is being deprecated soon, and hence
it is important to be disabled by default. Once it
is fully deprecated, even this option has to be removed.

For now, The following changes have been made:

- Update the `definitelyGpDisabled` option to be true, Thus
  disabling it by default
- Update the airgap docs to remove a mention of this

Signed-off-by: Tarun Pothulapati <[email protected]>
Pothulapati added a commit that referenced this issue Apr 4, 2022
Fixes #8740

`definitely-gp` is being deprecated soon, and hence
it is important to be disabled by default. Once it
is fully deprecated, even this option has to be removed.

For now, The following changes have been made:

- Update the `definitelyGpDisabled` option to be true, Thus
  disabling it by default
- Update the airgap docs to remove a mention of this

Signed-off-by: Tarun Pothulapati <[email protected]>
Pothulapati added a commit that referenced this issue Apr 5, 2022
Fixes #8740

`definitely-gp` is being deprecated soon, and hence
it is important to be disabled by default. Once it
is fully deprecated, even this option has to be removed.

For now, The following changes have been made:

- Update the `definitelyGpDisabled` option to be true, Thus
  disabling it by default
- Update the airgap docs to remove a mention of this

Signed-off-by: Tarun Pothulapati <[email protected]>
Pothulapati added a commit that referenced this issue Apr 8, 2022
Fixes #8740

`definitely-gp` is being deprecated soon, and hence
it is important to be disabled by default. Once it
is fully deprecated, even this option has to be removed.

For now, The following changes have been made:

- Update the `definitelyGpDisabled` option to be true, Thus
  disabling it by default both in the `installer` and `chart`.
- Update the airgap docs to remove a mention of this.

Signed-off-by: Tarun Pothulapati <[email protected]>
roboquat pushed a commit that referenced this issue Apr 13, 2022
Fixes #8740

`definitely-gp` is being deprecated soon, and hence
it is important to be disabled by default. Once it
is fully deprecated, even this option has to be removed.

For now, The following changes have been made:

- Update the `definitelyGpDisabled` option to be true, Thus
  disabling it by default both in the `installer` and `chart`.
- Update the airgap docs to remove a mention of this.

Signed-off-by: Tarun Pothulapati <[email protected]>
Repository owner moved this from ⚒In Progress to ✨Done in 🚚 Security, Infrastructure, and Delivery Team (SID) Apr 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: install Terraform installation scripts, helm charts, installer images team: delivery Issue belongs to the self-hosted team
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

3 participants