Skip to content

Conversation

@ritmun
Copy link
Contributor

@ritmun ritmun commented Jan 19, 2026

Added AGENTS.md

@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 Jan 19, 2026
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 19, 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

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 19, 2026
@ritmun ritmun force-pushed the SDCICD-1728 branch 7 times, most recently from 0daef49 to 2a62144 Compare January 19, 2026 21:16

## What This Is
End-to-end testing framework for Managed services for OSD/ROSA. Provisions clusters, runs health checks, executes tests, and collects artifacts.

Choose a reason for hiding this comment

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

I think there could be some value for the agent reading this to have a simple description of how this project is used and to see the README.md for more in depth context.

E.g. "This project is used as part of CICD workflows to validate changes to OpenShift Operators within OSD/ROSA, like Prow nightly jobs"

This extra little context I think would help the agent make decisions faster about design or troubleshooting issues.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The suggested description "This project is used as part of CICD workflows to validate changes to OpenShift Operators within OSD/ROSA, like Prow nightly jobs" is specific to one use case and doesn't represent all that osde2e supports fully. So I've updated the description slightly to include its applications.

Choose a reason for hiding this comment

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

This is great! It will give agents a bit more context to run off of without throwing too much at it.

Copy link

@christophermancini christophermancini left a comment

Choose a reason for hiding this comment

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

I added one comment but otherwise I think this looks good.

@ritmun ritmun marked this pull request as ready for review January 21, 2026 18:12
@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 Jan 21, 2026
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 21, 2026

@ritmun: 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.

Copy link

@christophermancini christophermancini left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jan 21, 2026
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 21, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: christophermancini, ritmun

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:
  • OWNERS [christophermancini,ritmun]

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

@openshift-merge-bot openshift-merge-bot bot merged commit 205751a into openshift:main Jan 21, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants