Bump up devfile registry checks to use go 1.24 #68063

JslYoon · 2025-08-12T14:53:48Z

Bumping up go version of registry and registry-operator checks to 1.24 per some CI tests are failing:
devfile/registry#626

openshift-ci · 2025-08-12T14:56:05Z

Hi @JslYoon. Thanks for your PR.

I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

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.

gabemontero · 2025-08-12T15:16:29Z

/ok-to-test

michael-valdron · 2025-08-12T15:55:50Z

/pj-rehearse auto-ack

openshift-ci-robot · 2025-08-12T15:55:52Z

@michael-valdron: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

michael-valdron

Hmm looks like there is no golang-1.24 tag on the CI image 🤔

Let me investigate if the image or tag has changed.

Signed-off-by: Lucas Yoon <lyoon@lyoon-thinkpadp1gen7.boston.csb>

JslYoon · 2025-08-15T15:46:19Z

/pj-rehearse auto-ack

openshift-ci-robot · 2025-08-15T15:46:21Z

@JslYoon: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

JslYoon · 2025-08-15T18:26:13Z

Created issue devfile/api/#1748

JslYoon · 2025-08-15T18:39:29Z

/test all

Signed-off-by: Lucas Yoon <lyoon@lyoon-thinkpadp1gen7.boston.csb>

JslYoon · 2025-08-18T20:05:57Z

/pj-rehearse

openshift-ci-robot · 2025-08-18T20:06:00Z

@JslYoon: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

JslYoon · 2025-08-19T20:22:08Z

/pj-rehearse

openshift-ci-robot · 2025-08-19T20:22:11Z

@JslYoon: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

Signed-off-by: Lucas Yoon <lyoon@lyoon-thinkpadp1gen7.boston.csb>

JslYoon · 2025-08-19T20:35:27Z

/pj-rehearse auto-ack

openshift-ci-robot · 2025-08-19T20:35:29Z

@JslYoon: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

openshift-ci-robot · 2025-08-19T20:38:01Z

[REHEARSALNOTIFIER]
@JslYoon: the pj-rehearse plugin accommodates running rehearsal tests for the changes in this PR. Expand 'Interacting with pj-rehearse' for usage details. The following rehearsable tests have been affected by this change:

Test name	Repo	Type	Reason
pull-ci-devfile-registry-operator-main-v4.18-images	devfile/registry-operator	presubmit	Ci-operator config changed
pull-ci-devfile-registry-operator-main-v4.18-registry-operator-integration-test	devfile/registry-operator	presubmit	Ci-operator config changed
pull-ci-devfile-registry-main-v4.16-images	devfile/registry	presubmit	Ci-operator config changed
pull-ci-devfile-registry-main-v4.16-registry-test	devfile/registry	presubmit	Ci-operator config changed

Interacting with pj-rehearse

Comment: /pj-rehearse to run up to 5 rehearsals
Comment: /pj-rehearse skip to opt-out of rehearsals
Comment: /pj-rehearse {test-name}, with each test separated by a space, to run one or more specific rehearsals
Comment: /pj-rehearse more to run up to 10 rehearsals
Comment: /pj-rehearse max to run up to 25 rehearsals
Comment: /pj-rehearse auto-ack to run up to 5 rehearsals, and add the rehearsals-ack label on success
Comment: /pj-rehearse list to get an up-to-date list of affected jobs
Comment: /pj-rehearse abort to abort all active rehearsals
Comment: /pj-rehearse network-access-allowed to allow rehearsals of tests that have the restrict_network_access field set to false. This must be executed by an openshift org member who is not the PR author

Once you are satisfied with the results of the rehearsals, comment: /pj-rehearse ack to unblock merge. When the rehearsals-ack label is present on your PR, merge will no longer be blocked by rehearsals.
If you would like the rehearsals-ack label removed, comment: /pj-rehearse reject to re-block merging.

michael-valdron · 2025-08-19T21:16:57Z

/pj-rehearse ack

openshift-ci-robot · 2025-08-19T21:16:59Z

@michael-valdron: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

michael-valdron

/lgtm

openshift-ci · 2025-08-19T21:20:28Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: JslYoon, michael-valdron

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

The pull request process is described here

Details

Needs approval from an approver in each of these files:

~~ci-operator/config/devfile/registry/OWNERS~~ [michael-valdron]
~~ci-operator/config/devfile/registry-operator/OWNERS~~ [michael-valdron]

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

openshift-ci · 2025-08-19T21:33:33Z

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

Test name	Commit	Details	Required	Rerun command
ci/rehearse/devfile/registry/main/v4.16-images	`d113c17`	link	unknown	`/pj-rehearse pull-ci-devfile-registry-main-v4.16-images`
ci/rehearse/devfile/registry/main/v4.16-registry-test	`d113c17`	link	unknown	`/pj-rehearse pull-ci-devfile-registry-main-v4.16-registry-test`

Full PR test history. Your PR dashboard.

Details

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.

* Bump up devfile registry checks to use go 1.24 Signed-off-by: Lucas Yoon <lyoon@lyoon-thinkpadp1gen7.boston.csb> * trying out external mirrored image Signed-off-by: Lucas Yoon <lyoon@lyoon-thinkpadp1gen7.boston.csb> * ocp 4.20 test Signed-off-by: Lucas Yoon <lyoon@lyoon-thinkpadp1gen7.boston.csb> * trying out new image Signed-off-by: Lucas Yoon <lyoon@lyoon-thinkpadp1gen7.boston.csb> --------- Signed-off-by: Lucas Yoon <lyoon@lyoon-thinkpadp1gen7.boston.csb> Co-authored-by: Lucas Yoon <lyoon@lyoon-thinkpadp1gen7.boston.csb>

openshift-ci bot added the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Aug 12, 2025

openshift-ci bot requested review from Jdubrick and michael-valdron August 12, 2025 14:56

openshift-ci bot added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Aug 12, 2025

michael-valdron mentioned this pull request Aug 12, 2025

Updating go version to 1.24 devfile/registry-operator#109

Merged

1 task

michael-valdron reviewed Aug 12, 2025

View reviewed changes

Bump up devfile registry checks to use go 1.24

9317882

Signed-off-by: Lucas Yoon <lyoon@lyoon-thinkpadp1gen7.boston.csb>

JslYoon force-pushed the JslYoon-update-go branch from dcae0db to 9317882 Compare August 12, 2025 19:26

trying out external mirrored image

87b0b1c

Signed-off-by: Lucas Yoon <lyoon@lyoon-thinkpadp1gen7.boston.csb>

ocp 4.20 test

1391abb

Signed-off-by: Lucas Yoon <lyoon@lyoon-thinkpadp1gen7.boston.csb>

trying out new image

d113c17

Signed-off-by: Lucas Yoon <lyoon@lyoon-thinkpadp1gen7.boston.csb>

JslYoon force-pushed the JslYoon-update-go branch from 88bcd01 to d113c17 Compare August 19, 2025 20:35

openshift-ci-robot added the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Aug 19, 2025

michael-valdron approved these changes Aug 19, 2025

View reviewed changes

openshift-ci bot assigned michael-valdron Aug 19, 2025

openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Aug 19, 2025

openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 19, 2025

This was referenced Aug 19, 2025

Jsl yoon/update go1.23 devfile/registry-support#290

Closed

Revert "Updating Go modules from 1.21 to 1.24" devfile/registry-support#291

Closed

openshift-merge-bot bot merged commit 51f18d4 into openshift:master Aug 19, 2025
14 of 17 checks passed

michael-valdron mentioned this pull request Aug 19, 2025

Devfile Registry support service modules should be updated to support Go 1.24 devfile/api#1710

Closed

12 tasks

JslYoon deleted the JslYoon-update-go branch August 20, 2025 13:55

Bump up devfile registry checks to use go 1.24 #68063

Bump up devfile registry checks to use go 1.24 #68063

Uh oh!

Conversation

JslYoon commented Aug 12, 2025

Uh oh!

openshift-ci bot commented Aug 12, 2025

Uh oh!

gabemontero commented Aug 12, 2025

Uh oh!

michael-valdron commented Aug 12, 2025

Uh oh!

openshift-ci-robot commented Aug 12, 2025

Uh oh!

michael-valdron left a comment

Choose a reason for hiding this comment

Uh oh!

JslYoon commented Aug 15, 2025

Uh oh!

openshift-ci-robot commented Aug 15, 2025

Uh oh!

JslYoon commented Aug 15, 2025

Uh oh!

JslYoon commented Aug 15, 2025

Uh oh!

JslYoon commented Aug 18, 2025

Uh oh!

openshift-ci-robot commented Aug 18, 2025

Uh oh!

JslYoon commented Aug 19, 2025

Uh oh!

openshift-ci-robot commented Aug 19, 2025

Uh oh!

JslYoon commented Aug 19, 2025

Uh oh!

openshift-ci-robot commented Aug 19, 2025

Uh oh!

openshift-ci-robot commented Aug 19, 2025

Uh oh!

michael-valdron commented Aug 19, 2025

Uh oh!

openshift-ci-robot commented Aug 19, 2025

Uh oh!

michael-valdron left a comment

Choose a reason for hiding this comment

Uh oh!

openshift-ci bot commented Aug 19, 2025

Uh oh!

Uh oh!

openshift-ci bot commented Aug 19, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants