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

Add GCP image uploads to the pipeline #219

Merged
merged 2 commits into from
Apr 7, 2020

Conversation

dustymabe
Copy link
Member

No description provided.

@dustymabe
Copy link
Member Author

Currently WIP, but this should be close. Doing some testing.

Will add kola tests in a followup.

HACKING.md Outdated
create a credentials config as a secret within OpenShift.

First grab a json file that represents the credentials for your service account
from GCP. Then create the secret in OpenShift:
Copy link
Member

Choose a reason for hiding this comment

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

Can we put more details here on where to get the JSON file from/what format it is?

Copy link
Member Author

Choose a reason for hiding this comment

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

fixed

optional: true
# This secret is used for running GCP kola tests
- name: gcp-image-upload-config
- name: gcp-kola-tests-config
Copy link
Member

Choose a reason for hiding this comment

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

Two - name: here.

Copy link
Member Author

Choose a reason for hiding this comment

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

fixed I think

Copy link
Member Author

Choose a reason for hiding this comment

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

fixed I think

- name: GCP_KOLA_TESTS_CONFIG
value: /.gcp-kola-tests-config
- name: gcp-image-upload-config
- name: gcp-kola-tests-config
Copy link
Member

Choose a reason for hiding this comment

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

Two - name: here.

Copy link
Member Author

Choose a reason for hiding this comment

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

fixed I think

Comment on lines 27 to 30
- name: GCP_IMAGE_UPLOAD_CONFIG
value: /.gcp-image-upload-config
- name: GCP_KOLA_TESTS_CONFIG
value: /.gcp-kola-tests-config
Copy link
Member

Choose a reason for hiding this comment

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

Yeah... we should really look at constraining each container run to the minimal set of credentials it needs. E.g. no reason the AWS kola test needs GCP creds mounted in.

Copy link
Member Author

Choose a reason for hiding this comment

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

I assume you mean something for us to do in the future?

Copy link
Member

Choose a reason for hiding this comment

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

Yup, definitely!

Jenkinsfile Outdated Show resolved Hide resolved
Jenkinsfile Show resolved Hide resolved
@dustymabe dustymabe force-pushed the dusty-add-gcp branch 2 times, most recently from 5a66e2e to 347fdd0 Compare April 6, 2020 21:45
Copy link
Member

@jlebon jlebon left a comment

Choose a reason for hiding this comment

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

LGTM!

@dustymabe
Copy link
Member Author

pushed up a few small fixes for this. Now seems to be working in the pipeline!

@dustymabe
Copy link
Member Author

depends on coreos/coreos-assembler#1319

@dustymabe
Copy link
Member Author

ok added a commit that removes the need to specify the project in various places. One final review before merge?

Copy link
Member

@jlebon jlebon left a comment

Choose a reason for hiding this comment

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

I'd squash those two commits together, but OK as is too.

We seem to be in a transition from GCE to GCP terminology. Let's
update it here too.
@dustymabe
Copy link
Member Author

coreos/coreos-assembler#1319 merged.

@dustymabe dustymabe marked this pull request as ready for review April 7, 2020 21:08
@dustymabe dustymabe merged commit 3b13f26 into coreos:master Apr 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants