Skip to content

OCPNODE-4055: Add periodic CI jobs for additional-storage-support tests#78465

Closed
BhargaviGudi wants to merge 1 commit intoopenshift:mainfrom
BhargaviGudi:additional-storage-support-ci
Closed

OCPNODE-4055: Add periodic CI jobs for additional-storage-support tests#78465
BhargaviGudi wants to merge 1 commit intoopenshift:mainfrom
BhargaviGudi:additional-storage-support-ci

Conversation

@BhargaviGudi
Copy link
Copy Markdown
Contributor

@BhargaviGudi BhargaviGudi commented Apr 28, 2026

Summary

Jobs Added

Release Job Name Cluster Interval
4.22 periodic-ci-openshift-origin-release-4.22-e2e-gcp-ovn-additional-storage-support GCP 168h

Features Tested
These jobs validate the following ContainerRuntimeConfig features:

  • additionalImageStores - Read-only shared image storage
  • additionalArtifactStores - Read-only artifact storage
  • additionalLayerStores - Lazy-pull layer storage (stargz-store)

Configuration

  • FEATURE_SET: TechPreviewNoUpgrade - Required to enable the additional storage features
  • TEST_SUITE: openshift/additional-storage-support - Runs the dedicated test suite from origin repo

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Apr 28, 2026
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

openshift-ci-robot commented Apr 28, 2026

@BhargaviGudi: This pull request references OCPNODE-4055 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "5.0.0" version, but no target version was set.

Details

In response to this:

Summary

  • Add weekly periodic CI jobs to run the openshift/additional-storage-support test suite
  • Jobs test ContainerRuntimeConfig additional storage features with TechPreviewNoUpgrade feature set

Jobs Added

Release Job Name Cluster Interval
5.0 periodic-ci-openshift-origin-release-5.0-e2e-aws-ovn-additional-storage-support AWS 168h
4.22 periodic-ci-openshift-origin-release-4.22-e2e-gcp-ovn-additional-storage-support GCP 168h

Features Tested
These jobs validate the following ContainerRuntimeConfig features:

  • additionalImageStores - Read-only shared image storage
  • additionalArtifactStores - Read-only artifact storage
  • additionalLayerStores - Lazy-pull layer storage (stargz-store)

Configuration

  • FEATURE_SET: TechPreviewNoUpgrade - Required to enable the additional storage features
  • TEST_SUITE: openshift/additional-storage-support - Runs the dedicated test suite from origin repo

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 openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci openshift-ci Bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Apr 28, 2026
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Apr 28, 2026

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 28, 2026

Walkthrough

Two new periodic E2E test workflows are added to OpenShift CI configuration: one for GCP OVN with additional storage support testing in the 4.22 release configuration, and another for AWS OVN with additional storage support testing in the 5.0 release configuration. Both run on 168-hour intervals.

Changes

Cohort / File(s) Summary
New Periodic E2E Test Workflows
ci-operator/config/openshift/origin/openshift-origin-release-4.22.yaml, ci-operator/config/openshift/origin/openshift-origin-release-5.0.yaml
Addition of two new periodic GCP and AWS OVN E2E test job definitions (e2e-gcp-ovn-additional-storage-support and e2e-aws-ovn-additional-storage-support) with 168-hour intervals, configuring cluster profiles, test suites for additional storage support, and serial workflow execution.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 12
✅ Passed checks (12 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: adding periodic CI jobs for the additional-storage-support tests.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Stable And Deterministic Test Names ✅ Passed PR modifies only CI operator configuration files (YAML infrastructure-as-code), not Ginkgo test code. Check requires assessment of Ginkgo test names for dynamic content, which are not present in this PR.
Test Structure And Quality ✅ Passed PR modifies only CI job configuration YAML files, not Ginkgo test code implementations.
Microshift Test Compatibility ✅ Passed This pull request only adds CI job configuration entries without introducing new Ginkgo e2e tests.
Single Node Openshift (Sno) Test Compatibility ✅ Passed PR only adds CI job configuration entries referencing existing test suite, not new Ginkgo e2e test code.
Topology-Aware Scheduling Compatibility ✅ Passed CI job configuration files do not contain Kubernetes scheduling constraints; changes define test suite parameters only.
Ote Binary Stdout Contract ✅ Passed PR modifies only CI operator YAML configuration files defining job scheduling, not test source code or binaries subject to stdout contract checks.
Ipv6 And Disconnected Network Test Compatibility ✅ Passed PR adds only CI job configuration in YAML files without new Ginkgo e2e test code, so custom check for new tests does not apply.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@BhargaviGudi BhargaviGudi force-pushed the additional-storage-support-ci branch from 2337633 to 61c3d26 Compare May 5, 2026 11:36
@BhargaviGudi BhargaviGudi marked this pull request as ready for review May 5, 2026 11:36
@openshift-ci openshift-ci Bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 5, 2026
@openshift-ci openshift-ci Bot requested review from deads2k and p0lyn0mial May 5, 2026 11:37
@BhargaviGudi
Copy link
Copy Markdown
Contributor Author

/retest-required

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 5, 2026

@BhargaviGudi: all tests passed!

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.

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label May 5, 2026
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 5, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: BhargaviGudi, saschagrunert
Once this PR has been reviewed and has the lgtm label, please assign bertinatto for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

@cpmeadors
Copy link
Copy Markdown
Contributor

/payload-job periodic-ci-openshift-origin-release-4.22-e2e-gcp-ovn-additional-storage-support

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 5, 2026

@cpmeadors: the repo openshift/release does not contribute to the OpenShift official images, or the base branch is not currently having images promoted

@ngopalak-redhat
Copy link
Copy Markdown
Contributor

@BhargaviGudi Can you check if its covered in payload-job periodic-ci-openshift-release-main-nightly-4.22-e2e-aws-disruptive-longrunning-techpreview ?

/hold

@openshift-ci openshift-ci Bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 6, 2026
@BhargaviGudi
Copy link
Copy Markdown
Contributor Author

Decided to use payload-job periodic-ci-openshift-release-main-nightly-4.22-e2e-aws-disruptive-longrunning-techpreview job instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants