Skip to content

Conversation

@sayan-biswas
Copy link
Contributor

@sayan-biswas sayan-biswas commented Nov 16, 2025

Changes:

  • Update go version to 1.25.3
  • Update dependencies to mitigate CVEs
  • Fix tests to work with broken APIs after update
  • Remove/Replace deprecated methods
  • Update Git command to ignore pre-commit hooks
  • Update Go builder image to 1.25 in docker files and Openshift CI
  • Update tekton pipeline task bundles to have updated golang image

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 16, 2025
@sayan-biswas sayan-biswas removed the request for review from divyansh42 November 16, 2025 20:40
@sayan-biswas sayan-biswas force-pushed the update-deps branch 3 times, most recently from 1c3bf8a to e7f558c Compare November 16, 2025 21:45
@yselkowitz
Copy link
Contributor

A memory link in github.com/ulikunitz/xz before 0.5.14 has been reported at https://bugzilla.redhat.com/show_bug.cgi?id=2391585. Can we get this updated too?

@sayan-biswas sayan-biswas force-pushed the update-deps branch 8 times, most recently from abc4680 to e8a88e6 Compare November 24, 2025 07:56
@sayan-biswas sayan-biswas requested a review from avinal November 24, 2025 08:10
Changes:
- Update go version to 1.25.3
- Update dependencies to mitigate CVEs
- Fix tests to work with broken APIs after update
- Remove/Replace deprecated methods
- Update Git command to ignore pre-commit hooks
- Update Go builder image to 1.25 in docker files and Openshift CI
- Update tekton pipeline task bundles to have updated golang image

Signed-off-by: Sayan Biswas <sayan-biswas@live.com>
@sayan-biswas
Copy link
Contributor Author

/override verify unit security integration
golang 1.25 builder image is not available in openshift ci yet.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 24, 2025

@sayan-biswas: /override requires failed status contexts, check run or a prowjob name to operate on.
The following unknown contexts/checkruns were given:

  • integration
  • security
  • unit
  • verify

Only the following failed contexts/checkruns were expected:

  • ci/prow/integration
  • ci/prow/security
  • ci/prow/unit
  • ci/prow/verify
  • pull-ci-openshift-source-to-image-master-integration
  • pull-ci-openshift-source-to-image-master-security
  • pull-ci-openshift-source-to-image-master-unit
  • pull-ci-openshift-source-to-image-master-verify
  • tide

If you are trying to override a checkrun that has a space in it, you must put a double quote on the context.

In response to this:

/override verify unit security integration
golang 1.25 builder image is not available in openshift ci yet.

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.

@sayan-biswas
Copy link
Contributor Author

/override ci/prow/verify ci/prow/unit ci/prow/security ci/prow/integration

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 24, 2025

@sayan-biswas: Overrode contexts on behalf of sayan-biswas: ci/prow/integration, ci/prow/security, ci/prow/unit, ci/prow/verify

In response to this:

/override ci/prow/verify ci/prow/unit ci/prow/security ci/prow/integration

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.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 24, 2025

@sayan-biswas: all tests passed!

Full PR test history. Your PR dashboard.

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.

Copy link
Member

@avinal avinal 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

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 24, 2025
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 24, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: avinal, sayan-biswas

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

@openshift-merge-bot openshift-merge-bot bot merged commit 133a13b into openshift:master Nov 24, 2025
9 checks passed
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. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants