Skip to content

Update & release prep#188

Merged
kcp-ci-bot merged 6 commits intokcp-dev:mainfrom
mjudeikis:bump.kcp.image
Apr 2, 2026
Merged

Update & release prep#188
kcp-ci-bot merged 6 commits intokcp-dev:mainfrom
mjudeikis:bump.kcp.image

Conversation

@mjudeikis
Copy link
Copy Markdown
Contributor

@mjudeikis mjudeikis commented Apr 1, 2026

Summary

What Type of PR Is This?

Update kcp image
/kind chore

Related Issue(s)

Fixes #

Release Notes

NONE

@kcp-ci-bot kcp-ci-bot added release-note-none Denotes a PR that doesn't merit a release note. kind/chore Categorizes issue or PR as related to maintenance and other usually non-code changes. dco-signoff: yes Indicates the PR's author has signed the DCO. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Apr 1, 2026
ntnn
ntnn reviewed Apr 1, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@ntnn ntnn left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@kcp-ci-bot kcp-ci-bot added the lgtm Indicates that a PR is ready to be merged. label Apr 1, 2026
@kcp-ci-bot
Copy link
Copy Markdown
Contributor

kcp-ci-bot commented Apr 1, 2026

LGTM label has been added.

DetailsGit tree hash: 5d1fe77a124d44100bcb13786214fa380bd784e3

@mjudeikis
Copy link
Copy Markdown
Contributor Author

mjudeikis commented Apr 1, 2026

/retest

@ntnn ntnn added this to tbd Apr 1, 2026
@ntnn ntnn moved this to Reviewing in tbd Apr 1, 2026
@kcp-ci-bot kcp-ci-bot removed the lgtm Indicates that a PR is ready to be merged. label Apr 1, 2026
@kcp-ci-bot kcp-ci-bot requested a review from ntnn April 1, 2026 11:56
@kcp-ci-bot kcp-ci-bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Apr 1, 2026
@mjudeikis
Copy link
Copy Markdown
Contributor Author

mjudeikis commented Apr 1, 2026

/retest

xrstf
xrstf reviewed Apr 1, 2026
View reviewed changes
.prow.yaml
cpu: 4

- name: pull-kcp-operator-test-e2e-0.28
- name: pull-kcp-operator-test-e2e-0.29
Copy link
Copy Markdown
Contributor

@xrstf xrstf Apr 1, 2026

Choose a reason for hiding this comment

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

Why are we dropping 0.28 tests?

The explicit 0.30 tests are now redundant, because the e2e-default is already on 0.30.

Copy link
Copy Markdown
Contributor Author

@mjudeikis mjudeikis Apr 1, 2026

Choose a reason for hiding this comment

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

0.28 is failing either way, as the cache server is not backported in there. And as code is using "current operator" codebase and just swaps images, it fails with "unknown flag"

@mjudeikis-bot create a follow-up issue so that canary tests would use the operator release image, not only swap image. it should test full operator upgrade, not just image

Copy link
Copy Markdown

@mjudeikis-bot mjudeikis-bot Apr 2, 2026

Choose a reason for hiding this comment

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

Created follow-up issue: #189

@mjudeikis
Copy link
Copy Markdown
Contributor Author

mjudeikis commented Apr 1, 2026

/retest

2 similar comments
@mjudeikis
Copy link
Copy Markdown
Contributor Author

mjudeikis commented Apr 1, 2026

/retest

@mjudeikis
Copy link
Copy Markdown
Contributor Author

mjudeikis commented Apr 1, 2026

/retest

@mjudeikis
Copy link
Copy Markdown
Contributor Author

mjudeikis commented Apr 2, 2026

/retest

@mjudeikis mjudeikis changed the title Bump kcp image Update & release prep Apr 2, 2026
@mjudeikis
Copy link
Copy Markdown
Contributor Author

mjudeikis commented Apr 2, 2026

/retest
ci cluster issus. again.

@mjudeikis-bot mjudeikis-bot mentioned this pull request Apr 2, 2026
Open
@mjudeikis
Copy link
Copy Markdown
Contributor Author

mjudeikis commented Apr 2, 2026

/retest
oracle! Stop rotating my CI nodes! 3rd time in an hour!

@mjudeikis
Copy link
Copy Markdown
Contributor Author

mjudeikis commented Apr 2, 2026

/retest

@kcp-ci-bot
Copy link
Copy Markdown
Contributor

kcp-ci-bot commented Apr 2, 2026 •
edited
Loading

@mjudeikis: 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
pull-kcp-operator-test-e2e-0.28 7bb054e link true /test pull-kcp-operator-test-e2e-0.28

Full PR test history

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.

@mjudeikis mjudeikis added the tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges. label Apr 2, 2026
@kcp-ci-bot kcp-ci-bot added the lgtm Indicates that a PR is ready to be merged. label Apr 2, 2026
@kcp-ci-bot
Copy link
Copy Markdown
Contributor

kcp-ci-bot commented Apr 2, 2026

LGTM label has been added.

DetailsGit tree hash: 050386d24884509f8c76f4a8508382623a7a5644

@ntnn
Copy link
Copy Markdown
Member

ntnn commented Apr 2, 2026

/lgtm
/approve

@mjudeikis mjudeikis added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 2, 2026
@kcp-ci-bot
Copy link
Copy Markdown
Contributor

kcp-ci-bot commented Apr 2, 2026

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by: ntnn

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:

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

@kcp-ci-bot kcp-ci-bot merged commit a5d1ade into kcp-dev:main Apr 2, 2026
12 checks passed
@github-project-automation github-project-automation bot moved this from Reviewing to Done in tbd Apr 2, 2026
@ntnn ntnn moved this from Done to Reviewing in tbd Apr 2, 2026
@mjudeikis
Copy link
Copy Markdown
Contributor Author

mjudeikis commented Apr 2, 2026

/cherrypick release-0.5
/cherrypick release-0.4

@kcp-ci-bot
Copy link
Copy Markdown
Contributor

kcp-ci-bot commented Apr 2, 2026

@mjudeikis: #188 failed to apply on top of branch "release-0.4":

Applying: bump kcp image
Using index info to reconstruct a base tree...
M	internal/resources/resources.go
Falling back to patching base and 3-way merge...
Auto-merging internal/resources/resources.go
CONFLICT (content): Merge conflict in internal/resources/resources.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0001 bump kcp image
Details

In response to this:

/cherrypick release-0.5
/cherrypick release-0.4

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.

@mjudeikis
Copy link
Copy Markdown
Contributor Author

mjudeikis commented Apr 2, 2026

/cherry-pick release-0.5
/cherry-pick release-0.4

@kcp-ci-bot
Copy link
Copy Markdown
Contributor

kcp-ci-bot commented Apr 2, 2026

@mjudeikis: #188 failed to apply on top of branch "release-0.4":

Applying: bump kcp image
Using index info to reconstruct a base tree...
M	internal/resources/resources.go
Falling back to patching base and 3-way merge...
Auto-merging internal/resources/resources.go
CONFLICT (content): Merge conflict in internal/resources/resources.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0001 bump kcp image
Details

In response to this:

/cherry-pick release-0.5
/cherry-pick release-0.4

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.

mjudeikis added a commit to mjudeikis/kcp-operator that referenced this pull request Apr 2, 2026
@mjudeikis
Update & release prep (kcp-dev#188)
43c9e47 
* bump kcp image
* Add cache server cert support
* drop 0.28
* add missing mount into shards
* bump image
* make PROTOKOL not fail CI
kcp-ci-bot pushed a commit that referenced this pull request Apr 2, 2026
@mjudeikis
Update & release prep (#188) (#191)
810a774 
* bump kcp image
* Add cache server cert support
* drop 0.28
* add missing mount into shards
* bump image
* make PROTOKOL not fail CI
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xrstf xrstf xrstf left review comments

@ntnn ntnn ntnn approved these changes

+1 more reviewer

@mjudeikis-bot mjudeikis-bot mjudeikis-bot left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@ntnn ntnn

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates the PR's author has signed the DCO. kind/chore Categorizes issue or PR as related to maintenance and other usually non-code changes. lgtm Indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesn't merit a release note. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@mjudeikis @kcp-ci-bot @ntnn @xrstf @mjudeikis-bot