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
[dynamic-plugin-sdk] Reducer - convert extensions from static to dynamic #7854
[dynamic-plugin-sdk] Reducer - convert extensions from static to dynamic #7854
Conversation
@vojtechszocs PTAL
|
619f50d
to
62ee649
Compare
@christianvogt @vojtechszocs please review |
7036f09
to
5477625
Compare
f4bca91
to
c501016
Compare
/override ci/prow/ceph-storage-plugin The ceph-storage-plugin job should only have run on PRs with changes to |
@spadgett: Overrode contexts on behalf of spadgett: ci/prow/ceph-storage-plugin In response to this:
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/test-infra repository. |
c501016
to
721c9c8
Compare
/retest Please review the full test history for this PR and help us cut down flakes. |
/hold |
f81593b
to
65e3dd3
Compare
The failing test:
Ran on my local machine And it ss passing locally on my machine. any thoughts @spadgett @jeff-phillips-18 @christianvogt ? |
f189285
to
603107d
Compare
/retest |
@vojtechszocs I applied your patch. still fails on the same test
pretty stuck here @christianvogt @vojtechszocs |
@glekner The KubeVirt Redux reducer extension should be gated by KubeVirt feature flag, just like it was gated in |
603107d
to
f3c545a
Compare
tests passing. please re-lgtm @spadgett @christianvogt |
/lgtm |
/hold cancel |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: christianvogt, glekner, rawagner, vojtechszocs 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 |
/retest |
1 similar comment
/retest |
/skip |
/retest |
@glekner: The following test failed, say
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/test-infra repository. I understand the commands that are listed here. |
/retest Please review the full test history for this PR and help us cut down flakes. |
1 similar comment
/retest Please review the full test history for this PR and help us cut down flakes. |
No description provided.