project-infra icon indicating copy to clipboard operation
project-infra copied to clipboard

bazel: move to vendoring go dependencies

Open dhiller opened this issue 1 year ago • 21 comments

What this PR does / why we need it:

Replaces the bazel dependency management for go repositories with pure go mod vendoring.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged): Fixes #

Special notes for your reviewer:

/cc @brianmcarey

Checklist

This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR. Approvers are expected to review this list.

Release note:


dhiller avatar Jul 11 '24 11:07 dhiller

/cc @xpivarc

FYI: since kubernetes-sigs/test-infra and kubernetes-sigs/prow moved away from using bazel, maintaining it has become more of a burden - so we think it's best to remove it and work with go tools only.

dhiller avatar Jul 11 '24 12:07 dhiller

/test pull-project-infra-test-robots

dhiller avatar Jul 11 '24 12:07 dhiller

/rehearse ?

dhiller avatar Jul 11 '24 12:07 dhiller

Rehearsal is available for the following jobs in this PR:

pull-project-infra-test-robots

You can trigger rehearsal for all jobs by commenting either /rehearse or /rehearse all on this PR.

For a specific PR you can comment /rehearse {job-name}.

For a list of jobs that you can rehearse you can comment /rehearse ?.

kubevirt-bot avatar Jul 11 '24 12:07 kubevirt-bot

/rehearse pull-project-infra-test-robots

dhiller avatar Jul 11 '24 12:07 dhiller

Rehearsal jobs created for this PR:

rehearsal-pull-project-infra-test-robots

You can trigger rehearsal for all jobs by commenting either /rehearse or /rehearse all on this PR.

For a specific PR you can comment /rehearse {job-name}.

For a list of jobs that you can rehearse you can comment /rehearse ?.

kubevirt-bot avatar Jul 11 '24 12:07 kubevirt-bot

/test pull-project-infra-test-robots

dhiller avatar Jul 11 '24 13:07 dhiller

/cc @xpivarc

FYI: since kubernetes-sigs/test-infra and kubernetes-sigs/prow moved away from using bazel, maintaining it has become more of a burden - so we think it's best to remove it and work with go tools only.

I like vendoring :+1:

xpivarc avatar Jul 11 '24 13:07 xpivarc

/rehearse

dhiller avatar Jul 11 '24 13:07 dhiller

Rehearsal jobs created for this PR:


You can trigger rehearsal for all jobs by commenting either /rehearse or /rehearse all on this PR.

For a specific PR you can comment /rehearse {job-name}.

For a list of jobs that you can rehearse you can comment /rehearse ?.

kubevirt-bot avatar Jul 11 '24 13:07 kubevirt-bot

/rehearse pull-project-infra-test-robots

dhiller avatar Jul 11 '24 13:07 dhiller

Rehearsal jobs created for this PR:


You can trigger rehearsal for all jobs by commenting either /rehearse or /rehearse all on this PR.

For a specific PR you can comment /rehearse {job-name}.

For a list of jobs that you can rehearse you can comment /rehearse ?.

kubevirt-bot avatar Jul 11 '24 13:07 kubevirt-bot

/test pull-project-infra-test-robots

dhiller avatar Jul 11 '24 13:07 dhiller

/test pull-project-infra-test-bazel-config

dhiller avatar Jul 11 '24 14:07 dhiller

/rehearse pull-project-infra-test-bazel-config

dhiller avatar Jul 11 '24 14:07 dhiller

Rehearsal jobs created for this PR:


You can trigger rehearsal for all jobs by commenting either /rehearse or /rehearse all on this PR.

For a specific PR you can comment /rehearse {job-name}.

For a list of jobs that you can rehearse you can comment /rehearse ?.

kubevirt-bot avatar Jul 11 '24 14:07 kubevirt-bot

/test all

dhiller avatar Jul 12 '24 07:07 dhiller

/test all

dhiller avatar Jul 12 '24 14:07 dhiller

/test all

dhiller avatar Jul 25 '24 08:07 dhiller

Issues go stale after 90d of inactivity. Mark the issue as fresh with /remove-lifecycle stale. Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

/lifecycle stale

kubevirt-bot avatar Oct 23 '24 08:10 kubevirt-bot

/remove-lifecycle stale

@xpivarc this is the effort to move towards vendoring, which I talked about in today's sig-ci meeting

dhiller avatar Nov 04 '24 09:11 dhiller

/test ?

dhiller avatar Dec 18 '24 12:12 dhiller

@dhiller: The following commands are available to trigger required jobs:

  • /test build-autoowners-image
  • /test build-bootstrap-image
  • /test build-fedora-coreos-kubevirt-image
  • /test build-kubekins-e2e-image
  • /test build-kubevirt-infra-bootstrap-image
  • /test build-kubevirt-kubevirt.github.io-image
  • /test build-kubevirt-user-guide-image
  • /test build-pr-creator-image
  • /test build-prow-deploy-image
  • /test build-release-tool-image
  • /test build-shared-images-controller-image
  • /test build-test-label-analyzer-image
  • /test build-vm-image-builder-image
  • /test check-prow-config
  • /test pull-kubevirt-org-github-config-updater
  • /test pull-project-infra-check-testgrid-config
  • /test pull-project-infra-ci-search-deploy-test
  • /test pull-project-infra-grafana-deploy-test
  • /test pull-project-infra-job-config-validator
  • /test pull-project-infra-prow-deploy-test
  • /test pull-project-infra-test-bazel-config
  • /test pull-project-infra-test-external-plugins
  • /test pull-project-infra-test-github-ci-services
  • /test pull-project-infra-test-releng
  • /test pull-project-infra-test-robots
  • /test pull-prow-kubevirt-labels-update-precheck
  • /test pull-prow-nmstate-labels-update-precheck

Use /test all to run the following jobs that were automatically triggered:

  • build-release-tool-image
  • build-shared-images-controller-image
  • build-test-label-analyzer-image
  • check-prow-config
  • pull-project-infra-check-testgrid-config
  • pull-project-infra-ci-search-deploy-test
  • pull-project-infra-grafana-deploy-test
  • pull-project-infra-job-config-validator
  • pull-project-infra-prow-deploy-test
  • pull-project-infra-test-bazel-config
  • pull-project-infra-test-external-plugins
  • pull-project-infra-test-github-ci-services
  • pull-project-infra-test-releng
  • pull-project-infra-test-robots

In response to this:

/test ?

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

kubevirt-bot avatar Dec 18 '24 12:12 kubevirt-bot

/test pull-project-infra-ci-search-deploy-test

dhiller avatar Dec 18 '24 12:12 dhiller

/test pull-project-infra-test-bazel-config

dhiller avatar Dec 18 '24 13:12 dhiller

/test pull-project-infra-test-bazel-config

dhiller avatar Dec 18 '24 14:12 dhiller

/test pull-project-infra-ci-search-deploy-test

dhiller avatar Dec 18 '24 14:12 dhiller

/test pull-project-infra-test-bazel-config

dhiller avatar Dec 27 '24 09:12 dhiller

/test pull-project-infra-test-external-plugins /test pull-project-infra-test-releng /test pull-project-infra-test-robots

dhiller avatar Dec 27 '24 09:12 dhiller

@dhiller: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
rehearsal-pull-project-infra-test-robots 588fdeedd9a0448e346a1821da9ba6f4c46aac91 link unknown /test pull-project-infra-test-robots

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

kubevirt-bot avatar Jan 02 '25 15:01 kubevirt-bot