Skip to content

[preview] Add config option for cert-issuer #14282

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

Merged
merged 1 commit into from
Oct 31, 2022
Merged

Conversation

vulkoingim
Copy link
Contributor

@vulkoingim vulkoingim commented Oct 31, 2022

Description

Add annotations:

  • cert-issuer - ability to choose the issuer for the cert of the preview env. [zerossl, letsencrypt]
  • recreate-preview - nuke the whole preview infra and recreate from scratch (including deps). Should be used sparingly, does not replace with-clean-slate-deployment
  • recreate-vm == with-clean-slate-deployment, but shorter

Fix a small issue with passing with-clean-slate-deployment - where the VM would get recreated twice, since the flag is set for both applies.

Related Issue(s)

Fixes https://gitpod.slack.com/archives/C032A46PWR0/p1667198847703189

How to test

Release Notes

NONE

Documentation

Werft options:

  • /werft with-local-preview
    If enabled this will build install/preview
  • /werft with-preview
  • /werft with-large-vm
  • /werft with-integration-tests=all
    Valid options are all, workspace, webapp, ide

Copy link
Contributor

@mads-hartmann mads-hartmann left a comment

Choose a reason for hiding this comment

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

Awesome ✅ Looking forward to deprecate "with-clean-slate-deployment" - the new names makes more sense and are shorter 🥳

@roboquat roboquat merged commit cc61e54 into main Oct 31, 2022
@roboquat roboquat deleted the aa/cfg-cert-issuer branch October 31, 2022 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants