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

Use previewct get-credentials in werft job #14091

Merged
merged 2 commits into from Oct 24, 2022
Merged

Conversation

vulkoingim
Copy link
Contributor

@vulkoingim vulkoingim commented Oct 21, 2022

Description

Use previewct get-credentials to populate a single kubeconfig and target contexts explicitly in tf.

Related Issue(s)

Fixes #

How to test

https://werft.gitpod-dev.com/job/gitpod-custom-aa-use-previewctl-get-creds.6/results

Release Notes

NONE

Documentation

Werft options:

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

@vulkoingim vulkoingim changed the title Aa/use previewctl get creds Use previewct get-credentials in werft job Oct 24, 2022
@gitpod-io gitpod-io deleted a comment from github-actions bot Oct 24, 2022
@vulkoingim vulkoingim marked this pull request as ready for review October 24, 2022 09:59
@vulkoingim vulkoingim requested a review from a team October 24, 2022 09:59
Copy link
Collaborator

@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, looks good to me. I added the hold label just to verify that the the successful job (job 6) was triggered through the CLI rather than a commit - is that the case? ☺️ If it was, feel free to remove the hold label. If not, please use the CLI so we don't get tricked by job protection ☺️

@vulkoingim
Copy link
Contributor Author

vulkoingim commented Oct 24, 2022

Awesome, looks good to me. I added the hold label just to verify that the the successful job (job 6) was triggered through the CLI rather than a commit - is that the case? ☺️ If it was, feel free to remove the hold label. If not, please use the CLI so we don't get tricked by job protection ☺️

Yeah, it's triggered form the CLI.

@roboquat roboquat merged commit b34bc3f into main Oct 24, 2022
@roboquat roboquat deleted the aa/use-previewctl-get-creds branch October 24, 2022 11:25
@vulkoingim
Copy link
Contributor Author

vulkoingim commented Oct 24, 2022

Hahah forgot to squash the last commit again 😅

image

@vulkoingim vulkoingim mentioned this pull request Oct 26, 2022
4 tasks
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.

None yet

3 participants