continuous-integration icon indicating copy to clipboard operation
continuous-integration copied to clipboard

Add pipelines for bazel-lib and rules_license to TF config

Open fweikert opened this issue 3 years ago • 3 comments

Progress towards #1309, #1313

fweikert avatar Feb 04 '22 14:02 fweikert

Hey, sorry I had to run terraform apply for a different change and noticed that these two pipelines were still missing, so I just added them and pushed already: https://github.com/bazelbuild/continuous-integration/commit/7bf04992182577a3d7aefea249c1aa23dd8e00e1

I verified that we did the same, so I basically just "merged" your PR already manually. 😊

FYI, to prevent the issue with settings being lost due to null not being handled correctly by the provider, I explicitly set all the provider_settings to the correct value instead of relying on a default. I sorted them in the same order as they appear in the UI.

philwo avatar Feb 08 '22 16:02 philwo

Is this still needed?

meteorcloudy avatar Feb 16 '22 15:02 meteorcloudy

Could we get this PR going again. I am trying to set up the CI runs for rules_license, but my tests are failing completely in a way I don't understand. https://buildkite.com/bazel/rules-license/builds/3#01826f31-5cb1-40d6-8b54-e35eabf585f0

The error is.

2022-08-05 18:08:14 FATAL  Config file is empty

  | 🚨 Error: The command exited with status 1

which I don't understand at all, nor can I find the message in the code here, so I don't know where to begin.

aiuto avatar Aug 05 '22 18:08 aiuto

Hi. Friendly ping. I need to get CI running for rules_license now that bazel depends on it.

aiuto avatar Aug 16 '22 16:08 aiuto

This PR is a no-op since Philipp already added the pipelines in https://github.com/bazelbuild/continuous-integration/commit/7bf04992182577a3d7aefea249c1aa23dd8e00e1

fweikert avatar Aug 17 '22 19:08 fweikert

Re https://buildkite.com/bazel/rules-license/builds/3#01826f31-5cb1-40d6-8b54-e35eabf585f0: your yaml config is invalid

fweikert avatar Aug 17 '22 19:08 fweikert