Skip to content

ci:validate conductor #3874

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

acpana
Copy link
Collaborator

@acpana acpana commented Mar 6, 2025

Let's start adding some simple CI for conductor!

  • as we all merge in yaml changes, let's have some validations

Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign justinsb for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@acpana acpana force-pushed the acpana/validate-conductor branch 2 times, most recently from 5074641 to 8953bfb Compare May 20, 2025 22:50
@acpana
Copy link
Collaborator Author

acpana commented May 20, 2025

Run ./experiments/conductor/scripts/verify-branches.sh
mkdir bin
GOWORK=off go build -o bin/conductor
go: downloading go1.24.2 (linux/amd[6](https://github.com/GoogleCloudPlatform/k8s-config-connector/actions/runs/15149419393/job/42592578713?pr=3874#step:4:7)4)
go: downloading github.com/spf13/cobra v1.8.1
go: downloading gopkg.in/yaml.v3 v3.0.1
go: downloading github.com/spf13/pflag v1.0.5
validating branches-all.yaml
2025/05/20 22:50:52 Running conductor runner with branch config: branches-all.yaml
2025/05/20 22:50:52 Starting Runner
2025/05/20 22:50:52 COMMAND: /usr/bin/ls -alh
2025/05/20 22:50:52 ls-alh: total [7](https://github.com/GoogleCloudPlatform/k8s-config-connector/actions/runs/15149419393/job/42592578713?pr=3874#step:4:8)96K
drwxr-xr-x 5 runner docker 4.0K May 20 22:50 .
drwxr-xr-x 7 runner docker 4.0K May 20 22:50 ..
-rw-r--r-- 1 runner docker  [8](https://github.com/GoogleCloudPlatform/k8s-config-connector/actions/runs/15149419393/job/42592578713?pr=3874#step:4:9)58 May 20 22:50 Makefile
-rw-r--r-- 1 runner docker 1.5K May 20 22:50 README.md
drwxr-xr-x 2 runner docker 4.0K May 20 22:50 bin
-rw-r--r-- 1 runner docker 757K May 20 22:50 branches-all.yaml
-rw-r--r-- 1 runner docker 2.3K May 20 22:50 branches.yaml
drwxr-xr-x 3 runner docker 4.0K May 20 22:50 cmd
-rw-r--r-- 1 runner docker 1.6K May 20 22:50 main.go
drwxr-xr-x 2 runner docker 4.0K May 20 22:50 scripts
2025/05/20 22:50:52 Uniqueness constrains violated for file branches-all.yaml

acpana added 4 commits May 20, 2025 22:55
Signed-off-by: Alex Pana <[email protected]>
Signed-off-by: Alex Pana <[email protected]>
Signed-off-by: Alex Pana <[email protected]>
@acpana acpana force-pushed the acpana/validate-conductor branch from 8953bfb to 3fbd289 Compare May 20, 2025 22:56
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