Skip to content

Rollout plugin support Draft PR - For discussion #4580

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.

Sign up for GitHub

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

Jump to bottom
Draft
Hariharasuthan99 wants to merge 15 commits into
base: master
Choose a base branch
from
Draft

Rollout plugin support Draft PR - For discussion #4580

Hariharasuthan99 wants to merge 15 commits into from
+15,078 −878

Conversation

@Hariharasuthan99
Copy link
Contributor

@Hariharasuthan99 Hariharasuthan99 commented Jan 6, 2026

Checklist:

  • Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this is a chore.
  • The title of the PR is (a) conventional with a list of types and scopes found here, (b) states what changed, and (c) suffixes the related issues number. E.g. "fix(controller): Updates such and such. Fixes #1234".
  • I've signed my commits with DCO
  • I have written unit and/or e2e tests for my change. PRs without these are unlikely to be merged.
  • My builds are green. Try syncing with master if they are not.
  • My organization is added to USERS.md.

@hariharasuthan1a
Resource Plugin Proposal
3c07aa4 
Signed-off-by: Hariharasuthan Raajeshwar <[email protected]>
Signed-off-by: phraajes <[email protected]>
@hariharasuthan1a
Updated resource plugin proposal
0122edf 
Signed-off-by: phraajes <[email protected]>
@hariharasuthan1a
Update proposal - remove redundancy, mention about controller-runtime…
811ea10 
…, setCanaryScale

Signed-off-by: phraajes <[email protected]>
@hariharasuthan1a
RolloutPlugin - 1
bc02dd2 
Signed-off-by: Hariharasuthan Raajeshwar <[email protected]>
@hariharasuthan1a
RolloutPlugin - 2
547d300 
Signed-off-by: Hariharasuthan Raajeshwar  <[email protected]>
@hariharasuthan1a
RolloutPlugin - 3
0183c26 
Signed-off-by: Hariharasuthan Raajeshwar <[email protected]>
@hariharasuthan1a
RolloutPlugin - 4 - test-sts-manifest
ad9eb8f 
Signed-off-by: Hariharasuthan Raajeshwar <[email protected]>
@hariharasuthan1a
RolloutPlugin - 4 - AnalysisRun - 1
3ef4926 
Signed-off-by: Hariharasuthan Raajeshwar <[email protected]>
@hariharasuthan1a
RolloutPlugin - 5 - Abort,Pause,Restart,PromoteFull,Non-RPC Statefuls…
7657f14 
…et plugin, new CRD fields, initial e2e tests

Signed-off-by: Hariharasuthan Raajeshwar <[email protected]>
@hariharasuthan1a
RolloutPlugin - 6 - Logrus logging
761c280 
Signed-off-by: Hariharasuthan Raajeshwar <[email protected]>
@hariharasuthan1a
RolloutPlugin - 7 - Add validations and e2e tests
3cb0912 
Signed-off-by: Hariharasuthan Raajeshwar <[email protected]>
@hariharasuthan1a
RolloutPlugin - 8 - Simpler Plugin implementation
d740228 
Signed-off-by: Hariharasuthan Raajeshwar <[email protected]>
@hariharasuthan1a
RolloutPlugin - 9 - Redesign restart, simpler pluginmanager, fixed e2…
2aa13e4 
…e tests

Signed-off-by: Hariharasuthan Raajeshwar <[email protected]>
@Hariharasuthan99 Hariharasuthan99 changed the title Rollout plugin support Rollout plugin support Draft PR - For discussion Jan 6, 2026
github-advanced-security[bot]
github-advanced-security bot found potential problems Jan 13, 2026
View reviewed changes
test/e2e/rolloutplugin/invalid-spec-missing-strategy.yaml
spec:
terminationGracePeriodSeconds: 1
containers:
- name: busybox

Check warning

Code scanning / SonarCloud

Service account permissions should be restricted Medium test

Bind this resource's automounted service account to RBAC or disable automounting. See more on SonarQube Cloud
test/e2e/rolloutplugin/invalid-spec-missing-strategy.yaml
terminationGracePeriodSeconds: 1
containers:
- name: busybox
image: quay.io/prometheus/busybox:latest

Check warning

Code scanning / SonarCloud

Memory limits should be enforced Medium test

Specify a memory limit for this container. See more on SonarQube Cloud
test/e2e/rolloutplugin/invalid-spec-plugin-notfound.yaml
spec:
terminationGracePeriodSeconds: 1
containers:
- name: busybox

Check warning

Code scanning / SonarCloud

Service account permissions should be restricted Medium test

Bind this resource's automounted service account to RBAC or disable automounting. See more on SonarQube Cloud
test/e2e/rolloutplugin/invalid-spec-plugin-notfound.yaml
terminationGracePeriodSeconds: 1
containers:
- name: busybox
image: quay.io/prometheus/busybox:latest

Check warning

Code scanning / SonarCloud

Memory limits should be enforced Medium test

Specify a memory limit for this container. See more on SonarQube Cloud
test/e2e/rolloutplugin/statefulset-canary-analysis.yaml
spec:
terminationGracePeriodSeconds: 1
containers:
- name: busybox

Check warning

Code scanning / SonarCloud

Service account permissions should be restricted Medium test

Bind this resource's automounted service account to RBAC or disable automounting. See more on SonarQube Cloud
test/e2e/rolloutplugin/statefulset-canary.yaml
image: quay.io/prometheus/busybox:latest
command: ["sleep", "3600"]
ports:
- containerPort: 80

Check warning

Code scanning / SonarCloud

Memory limits should be enforced Medium test

Specify a memory limit for this container. See more on SonarQube Cloud
test/e2e/rolloutplugin/valid-spec-fix.yaml
spec:
terminationGracePeriodSeconds: 1
containers:
- name: busybox

Check warning

Code scanning / SonarCloud

Service account permissions should be restricted Medium test

Bind this resource's automounted service account to RBAC or disable automounting. See more on SonarQube Cloud
test/e2e/rolloutplugin/valid-spec-fix.yaml
terminationGracePeriodSeconds: 1
containers:
- name: busybox
image: quay.io/prometheus/busybox:latest

Check warning

Code scanning / SonarCloud

Memory limits should be enforced Medium test

Specify a memory limit for this container. See more on SonarQube Cloud
test/rolloutplugin/test-sts-manifests/test-statefulset.yaml
app: test-sts
version: v1
spec:
containers:

Check warning

Code scanning / SonarCloud

Service account permissions should be restricted Medium test

Bind this resource's automounted service account to RBAC or disable automounting. See more on SonarQube Cloud
test/rolloutplugin/test-sts-manifests/test-statefulset.yaml
version: v1
spec:
containers:
- name: busybox

Check warning

Code scanning / SonarCloud

Memory limits should be enforced Medium test

Specify a memory limit for this container. See more on SonarQube Cloud
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 18, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
C Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

github-advanced-security[bot]
github-advanced-security bot found potential problems Jan 18, 2026
View reviewed changes
test/e2e/rolloutplugin/statefulset-canary-bg-analysis-fail.yaml
app: bg-analysis-fail
spec:
terminationGracePeriodSeconds: 1
containers:

Check warning

Code scanning / SonarCloud

Service account permissions should be restricted Medium test

Bind this resource's automounted service account to RBAC or disable automounting. See more on SonarQube Cloud
test/e2e/rolloutplugin/statefulset-canary-bg-analysis-fail.yaml
spec:
terminationGracePeriodSeconds: 1
containers:
- name: busybox

Check warning

Code scanning / SonarCloud

Memory limits should be enforced Medium test

Specify a memory limit for this container. See more on SonarQube Cloud
test/e2e/rolloutplugin/statefulset-canary-bg-analysis-success.yaml
app: bg-analysis-success
spec:
terminationGracePeriodSeconds: 1
containers:

Check warning

Code scanning / SonarCloud

Service account permissions should be restricted Medium test

Bind this resource's automounted service account to RBAC or disable automounting. See more on SonarQube Cloud
test/e2e/rolloutplugin/statefulset-canary-bg-analysis-success.yaml
spec:
terminationGracePeriodSeconds: 1
containers:
- name: busybox

Check warning

Code scanning / SonarCloud

Memory limits should be enforced Medium test

Specify a memory limit for this container. See more on SonarQube Cloud
test/e2e/rolloutplugin/statefulset-canary-inline-analysis-fail.yaml
app: inline-analysis-fail
spec:
terminationGracePeriodSeconds: 1
containers:

Check warning

Code scanning / SonarCloud

Service account permissions should be restricted Medium test

Bind this resource's automounted service account to RBAC or disable automounting. See more on SonarQube Cloud
test/e2e/rolloutplugin/statefulset-canary-inline-analysis-fail.yaml
spec:
terminationGracePeriodSeconds: 1
containers:
- name: busybox

Check warning

Code scanning / SonarCloud

Memory limits should be enforced Medium test

Specify a memory limit for this container. See more on SonarQube Cloud
test/e2e/rolloutplugin/statefulset-canary-inline-analysis-success.yaml
app: inline-analysis-success
spec:
terminationGracePeriodSeconds: 1
containers:

Check warning

Code scanning / SonarCloud

Service account permissions should be restricted Medium test

Bind this resource's automounted service account to RBAC or disable automounting. See more on SonarQube Cloud
test/e2e/rolloutplugin/statefulset-canary-inline-analysis-success.yaml
spec:
terminationGracePeriodSeconds: 1
containers:
- name: busybox

Check warning

Code scanning / SonarCloud

Memory limits should be enforced Medium test

Specify a memory limit for this container. See more on SonarQube Cloud
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Hariharasuthan99 @hariharasuthan1a