-
Notifications
You must be signed in to change notification settings - Fork 95
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
GCP: introducing provisioning code and tests #2290
base: main
Are you sure you want to change the base?
GCP: introducing provisioning code and tests #2290
Conversation
@stevenhorsman @wainersm Better late than never, clearing an old debt here. I'll continue working on this next week, but if you have any insight into why this commit is returning an error here, that would be helpful. Since no one has reported this issue and CI is still running fine, it makes me think the problem is on my side, but it's Friday night and my brain has shut down. |
b8d78d7
to
fa936ba
Compare
d443b92
to
a9f14fe
Compare
Yes, it was on my side! kubectl version was not supporting the syntax used here. Solved already. |
f727e9e
to
1badc87
Compare
Currently, this field is ignored and using 'default' network instead. Signed-off-by: Beraldo Leal <bleal@redhat.com>
Moving essential vars above. Signed-off-by: Beraldo Leal <bleal@redhat.com>
Common methods used by gcp provider. Signed-off-by: Beraldo Leal <bleal@redhat.com>
1badc87
to
0f61fdb
Compare
0f61fdb
to
4225d53
Compare
This will create a GKE cluster and vpc within GCP. Signed-off-by: Beraldo Leal <bleal@redhat.com>
For now, those are the basic tests we are adding. Signed-off-by: Beraldo Leal <bleal@redhat.com>
Making CI happy Signed-off-by: Beraldo Leal <bleal@redhat.com>
4225d53
to
f3b040b
Compare
This adds the initial GCP provisioning code along with some basic E2E tests. Still a work in progress, so marking as a draft while things get sorted out.