Skip to content

Conversation

stephen-soltesz
Copy link
Contributor

@stephen-soltesz stephen-soltesz commented Dec 19, 2019

This change adds a basic Dockerfile and Cloud Build configuration for applying the stctl configuration to our common projects.

To support using a single cloud build file across multiple projects, this change also includes a new flag to stctl that makes execution conditional on the specified -allowed-projects -- if the command runs in another project, then it will exit cleanly without applying the config.

After merging this change, we will need to setup the Cloud Build triggers in each project.


This change is Reviewable

@coveralls
Copy link

Pull Request Test Coverage Report for Build 24

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 97.581%

Totals Coverage Status
Change from base Build 18: 0.0%
Covered Lines: 121
Relevant Lines: 124

💛 - Coveralls

Copy link
Contributor

@pboothe pboothe left a comment

Choose a reason for hiding this comment

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

Reviewed 3 of 3 files at r1.
Reviewable status: :shipit: complete! 1 of 1 approvals obtained (waiting on @pboothe)

Copy link
Contributor

@pboothe pboothe left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewable status: :shipit: complete! 1 of 1 approvals obtained (waiting on @pboothe)

@pboothe
Copy link
Contributor

pboothe commented Dec 19, 2019

Please add two issues: one to call for building a generic "only run jobs on the following projects" entrypoint to gcp-config, and another to switch away from substring searching and to move to exact string matching in a string array.

Copy link
Contributor Author

@stephen-soltesz stephen-soltesz left a comment

Choose a reason for hiding this comment

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

I've created two issues:

#7
#8

Reviewable status: :shipit: complete! 1 of 1 approvals obtained

@stephen-soltesz stephen-soltesz merged commit aaa2b84 into master Dec 19, 2019
@stephen-soltesz stephen-soltesz deleted the sandbox-soltesz-clouddbuild branch August 12, 2022 16:44
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.

3 participants