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

Revert "Merge pull request #3223 from xmudrii/fix-release-comparison" #3254

Merged
merged 1 commit into from
Sep 7, 2023

Conversation

xmudrii
Copy link
Member

@xmudrii xmudrii commented Sep 7, 2023

What type of PR is this?

/kind bug
/kind regression

What this PR does / why we need it:

I unintentionally made a regression in #3223 that's now causing kubernetes/sig-release#2333

Let's revert this change and I'll get back to it with a proper fix.

Which issue(s) this PR fixes:

Fixes kubernetes/sig-release#2333

Does this PR introduce a user-facing change?

NONE

/assign @cpanato @saschagrunert @Verolop
cc @kubernetes/release-engineering

…omparison"

This reverts commit 7288ca7, reversing
changes made to 0e27c38.
@k8s-ci-robot k8s-ci-robot added the release-note-none Denotes a PR that doesn't merit a release note. label Sep 7, 2023
@k8s-ci-robot k8s-ci-robot added kind/bug Categorizes issue or PR as related to a bug. kind/regression Categorizes issue or PR as related to a regression from a prior release. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. needs-priority area/release-eng Issues or PRs related to the Release Engineering subproject sig/release Categorizes an issue or PR as relevant to SIG Release. labels Sep 7, 2023
@k8s-ci-robot k8s-ci-robot added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Sep 7, 2023
Copy link
Contributor

@Verolop Verolop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/approve
/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Sep 7, 2023
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: saschagrunert, Verolop, xmudrii

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

@k8s-ci-robot k8s-ci-robot merged commit 0623236 into kubernetes:master Sep 7, 2023
4 checks passed
@xmudrii xmudrii deleted the revert-7288ca7 branch September 7, 2023 09:41
saschagrunert added a commit to saschagrunert/release that referenced this pull request May 14, 2024
Fixing the very specific use case of the failing semver comparison
mentioned in kubernetes/kubernetes#117115

Follow-up on kubernetes#3223 and
kubernetes#3254

Signed-off-by: Sascha Grunert <[email protected]>
saschagrunert added a commit to saschagrunert/release that referenced this pull request May 28, 2024
Fixing the very specific use case of the failing semver comparison
mentioned in kubernetes/kubernetes#117115

Follow-up on kubernetes#3223 and
kubernetes#3254

Signed-off-by: Sascha Grunert <[email protected]>
saschagrunert added a commit to saschagrunert/release that referenced this pull request May 29, 2024
Fixing the very specific use case of the failing semver comparison
mentioned in kubernetes/kubernetes#117115

Follow-up on kubernetes#3223 and
kubernetes#3254

Signed-off-by: Sascha Grunert <[email protected]>
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. area/release-eng Issues or PRs related to the Release Engineering subproject cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/bug Categorizes issue or PR as related to a bug. kind/regression Categorizes issue or PR as related to a regression from a prior release. lgtm "Looks good to me", indicates that a PR is ready to be merged. needs-priority release-note-none Denotes a PR that doesn't merit a release note. sig/release Categorizes an issue or PR as relevant to SIG Release. 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.

https://storage.googleapis.com/k8s-release-dev/ci/latest.txt seems to return invalid value
5 participants