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

Change google project for preview DNS's #10385

Merged
merged 2 commits into from
Jun 4, 2022
Merged

Conversation

ArthurSens
Copy link
Contributor

@ArthurSens ArthurSens commented May 31, 2022

Description

gitpod-dev is one of our oldest google projects that was used to manage all kinds of cloud resources in the past.

We're working towards more fine-grained billing cost allocations and one of the changes is that DNS records from preview environments will now be handled in the gitpod-core-dev project.

This particular PR does only one of the many steps necessary for the DNS migration and should be merged only when @meysholdt and I perform the migration. (Probably next Saturday 4/06)

Related Issue(s)

One of the steps for https://github.com/gitpod-io/ops/issues/2533

How to test

When performing the migration, a werft job should be run from this PR to make sure it is configured correctly. Make sure to use core-dev during the test, and not our default harvester previews

Release Notes

NONE

Signed-off-by: ArthurSens <arthursens2005@gmail.com>
@ArthurSens ArthurSens force-pushed the as/migrate-preview-dns branch from c430850 to 0de0390 Compare June 4, 2022 12:15
@meysholdt meysholdt self-requested a review June 4, 2022 12:35
Signed-off-by: ArthurSens <arthursens2005@gmail.com>
@ArthurSens
Copy link
Contributor Author

We cannot forget to clean up gitpod-dev ClusterIssuers from core-dev!

@ArthurSens ArthurSens marked this pull request as ready for review June 4, 2022 13:11
@roboquat roboquat merged commit e5e5ff8 into main Jun 4, 2022
@roboquat roboquat deleted the as/migrate-preview-dns branch June 4, 2022 13:12
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.

3 participants