Skip to content
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

[buoy] adding support for disconnected release versions and go package version families #2923

Merged
merged 1 commit into from Oct 26, 2021

Conversation

n3wscott
Copy link
Contributor

@n3wscott n3wscott commented Oct 22, 2021

Signed-off-by: Scott Nichols n3wscott@chainguard.dev

What this PR does, why we need it:

Supporting knative 1.0.0 release madness.

User-visible changes in this PR:

Adds --module-release flags to exists, float, and check.

Demo

The following demo shows using a module release of 0.22 or 0.23 and a release of 1.22 to float k8s.io/apimachinery.

[10:30:56] n3wscott@book ~/go/src/github.com/knative/test-infra (buoy-suppport-versions)
(╯°□°)╯︵  buoy float --domain k8s.io --module-release 0.22 --release 1.22 go.mod
k8s.io/apimachinery@v0.22.2
[10:32:52] n3wscott@book ~/go/src/github.com/knative/test-infra (buoy-suppport-versions)
(╯°□°)╯︵  buoy float --domain k8s.io --module-release 0.23 --release 1.22 go.mod
k8s.io/apimachinery@release-1.22

@google-cla
Copy link

google-cla bot commented Oct 22, 2021

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@google-cla google-cla bot added the cla: no Indicates the PR's author has not signed the CLA. label Oct 22, 2021
@knative-prow-robot knative-prow-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Oct 22, 2021
@knative-prow-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: n3wscott

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

The pull request process is described 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

@knative-prow-robot knative-prow-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 22, 2021
@google-cla
Copy link

google-cla bot commented Oct 22, 2021

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@google-cla
Copy link

google-cla bot commented Oct 22, 2021

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@google-cla
Copy link

google-cla bot commented Oct 22, 2021

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@salaboy
Copy link
Member

salaboy commented Oct 25, 2021

@n3wscott I was trying this version of buoy and float seems to be working, but when I try check it always fails .. for example:
RELEASE=1.0
RELEASE_MODULE=0.27

buoy check go.mod --domain knative.dev --release ${RELEASE} --module-release ${RELEASE_MODULE} --verbose 
knative.dev/pkg
✘  knative.dev/hack

I've tried with pkg and test-infra both fails on hack, while float works and doesn't complain about hack

@n3wscott
Copy link
Contributor Author

I don't think check will work on these repos, try it on releasable repos like serving or a net-, I might have broken something, I can check

@salaboy
Copy link
Member

salaboy commented Oct 25, 2021

@n3wscott I am checking on the releasable.. but I do expect that they will fail in the same way complaining about hack and pkg.. i will come back with my findings.

@n3wscott
Copy link
Contributor Author

Oh, hack does not have a release-1.0 branch.

@n3wscott
Copy link
Contributor Author

This working as expected:

(╯°□°)╯︵  go run ./buoy check ../hack/go.mod --domain knative.dev --release ${RELEASE} --module-release ${RELEASE_MODULE} --verbose
knative.dev/hack
[08:01:25] n3wscott@book ~/go/src/github.com/knative/test-infra (buoy-suppport-versions)
(╯°□°)╯︵  go run ./buoy check ../pkg/go.mod --domain knative.dev --release ${RELEASE} --module-release ${RELEASE_MODULE} --verbose
knative.dev/pkg
✘  knative.dev/hack
knative.dev/pkg failed because of the following dependencies [knative.dev/hack]
exit status 1
[08:01:34] n3wscott@book ~/go/src/github.com/knative/test-infra (buoy-suppport-versions)
¯\(°_o)/¯

note: (╯°□°)╯︵ means returned successful.

@n3wscott
Copy link
Contributor Author

Here is another run but it works:

[08:13:12] n3wscott@book ~/go/src/github.com/knative/test-infra (buoy-suppport-versions)
(╯°□°)╯︵  go run ./buoy check ../pkg/go.mod --domain knative.dev --release 0.25 --module-release 0.26 --verbose
knative.dev/pkg
✔  knative.dev/hack@release-0.25

the release is 0.25 (last release) and mod release is this release, it finds that pkg has a release-0.25 release branch.

@salaboy
Copy link
Member

salaboy commented Oct 25, 2021

@n3wscott thanks! I will try that

@n3wscott
Copy link
Contributor Author

@googlebot I signed it!

@google-cla
Copy link

google-cla bot commented Oct 25, 2021

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@n3wscott
Copy link
Contributor Author

@googlebot I signed it!

@google-cla
Copy link

google-cla bot commented Oct 25, 2021

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

…on families

Signed-off-by: Scott Nichols <n3wscott@chainguard.dev>
@google-cla
Copy link

google-cla bot commented Oct 25, 2021

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@knative-metrics-robot
Copy link

The following is the coverage report on the affected files.
Say /test pull-knative-test-infra-go-coverage to re-run this coverage report

File Old Coverage New Coverage Delta
pkg/gomod/check.go 100.0% 97.0% -3.0
pkg/gomod/float.go 100.0% 94.1% -5.9
pkg/gomod/release_meta.go 100.0% 97.2% -2.8

@n3wscott
Copy link
Contributor Author

@googlebot I signed it!

I think I need an adult.

@google-cla
Copy link

google-cla bot commented Oct 25, 2021

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@n3wscott
Copy link
Contributor Author

@googlebot I signed it!

@google-cla
Copy link

google-cla bot commented Oct 25, 2021

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@thisisnotapril thisisnotapril added cla: yes Indicates the PR's author has signed the CLA. and removed cla: no Indicates the PR's author has not signed the CLA. labels Oct 25, 2021
@salaboy
Copy link
Member

salaboy commented Oct 26, 2021

/lgtm

@knative-prow-robot knative-prow-robot added the lgtm Indicates that a PR is ready to be merged. label Oct 26, 2021
@knative-prow-robot knative-prow-robot merged commit eba0d79 into knative:main Oct 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cla: yes Indicates the PR's author has signed the CLA. lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants