FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

ci(gcb): adds flags to generate yaml triggers by devjgm · Pull Request #6125 · googleapis/google-cloud-cpp · GitHub

ci(gcb): adds flags to generate yaml triggers - #6125

Merged
devjgm merged 1 commit into
googleapis:masterfrom
devjgm:gcb-trigger-generator
Mar 28, 2021
Merged

ci(gcb): adds flags to generate yaml triggers#6125
devjgm merged 1 commit into
googleapis:masterfrom
devjgm:gcb-trigger-generator

Conversation

devjgm commented Mar 28, 2021
edited by coryan
Loading

Copy link
Copy Markdown
Contributor

Adds flags to output the YAML configs for a named build. The generated
YAML config can then be manually tweaked (if needed), and checked in.
Generating the config doesn't actually submit the file to GCB. To do
that, you must then use trigger.sh --import <new-yaml-file>.

I also used these flags to generate configs for a few builds that were
missing triggers and therefore weren't running (like integration and
xsan).

I also simplified the yaml configs:

  • I removed the created time and trigger IDs, which are output-only
    fields
  • I removed the trigger tags, which don't really do anything useful, and
    are difficult to generate automatically

This change is 

Adds flags to output the YAML configs for a named build. The generated
YAML config can then be manually tweaked (if needed), and checked in.
Generating the config doesn't actually submit the file to GCB. To do
that, you must then use `trigger.sh --import <new-yaml-file>`.

I also used these flags to generate configs for a few builds that were
missing triggers and therefore weren't running (like integration and
xsan).

I also simplified the yaml configs:
- I removed the created time and trigger IDs, which are output-only
fields
- I removed the trigger tags, which don't really do anything useful, and
are difficult to generate automatically
devjgm requested a review from a team March 28, 2021 18:45
googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 28, 2021

codecov Bot commented Mar 28, 2021
edited
Loading

Copy link
Copy Markdown

Codecov Report

Merging #6125 (537ea70) into master (53663dd) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #6125   +/-   ##
=======================================
  Coverage   95.67%   95.67%           
=======================================
  Files        1173     1173           
  Lines      104827   104827           
=======================================
  Hits       100296   100296           
  Misses       4531     4531           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 53663dd...537ea70. Read the comment docs.

devjgm merged commit 1eb35ba into googleapis:master Mar 28, 2021
devjgm deleted the gcb-trigger-generator branch March 28, 2021 21:11
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL