Skip to content
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

KATA-815 Document must-gather for developers #365

Merged
merged 5 commits into from Jan 9, 2024

Conversation

tbuskey
Copy link
Contributor

@tbuskey tbuskey commented Dec 4, 2023

https://issues.redhat.com/browse/KATA-815 Document must-gather image [developer / support facing]

Creates must-gather/must-gather-requirements.md

@tbuskey
Copy link
Contributor Author

tbuskey commented Dec 4, 2023

/assign @gkurz
/assign @littlejawa

@cpmeadors
Copy link
Contributor

/override ci/prow/check

Copy link

openshift-ci bot commented Dec 7, 2023

@cpmeadors: Overrode contexts on behalf of cpmeadors: ci/prow/check

In response to this:

/override ci/prow/check

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.

@tbuskey
Copy link
Contributor Author

tbuskey commented Dec 21, 2023

/assign @gkurz
/assign @littlejawa

must-gather/must-gather-requirements.md Outdated Show resolved Hide resolved
must-gather/must-gather-requirements.md Show resolved Hide resolved
must-gather/must-gather-requirements.md Outdated Show resolved Hide resolved
must-gather/must-gather-requirements.md Outdated Show resolved Hide resolved
@tbuskey
Copy link
Contributor Author

tbuskey commented Jan 4, 2024

@gkurz @littlejawa PTAL with my updates.

@tbuskey
Copy link
Contributor Author

tbuskey commented Jan 4, 2024

/test check
/test sandboxed-containers-operator-e2e

I'm not sure these apply...

@tbuskey
Copy link
Contributor Author

tbuskey commented Jan 5, 2024

/retest

Copy link
Contributor

@cpmeadors cpmeadors left a comment

Choose a reason for hiding this comment

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

Left a few questions

must-gather/collection-scripts/gather_nodes Outdated Show resolved Hide resolved
must-gather/collection-scripts/gather_nodes Outdated Show resolved Hide resolved
@tbuskey
Copy link
Contributor Author

tbuskey commented Jan 5, 2024

/test check
/test sandboxed-containers-operator-e2e

Copy link

openshift-ci bot commented Jan 5, 2024

@tbuskey: 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
ci/prow/sandboxed-containers-operator-e2e 0303772 link false /test sandboxed-containers-operator-e2e

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.

@tbuskey
Copy link
Contributor Author

tbuskey commented Jan 8, 2024

/override ci/prow/check

/override ci/prow/sandboxed-containers-operator-e2e

Copy link

openshift-ci bot commented Jan 8, 2024

@tbuskey: tbuskey unauthorized: /override is restricted to Repo administrators, approvers in top level OWNERS file, and the following github teams:openshift: openshift-release-oversight.

In response to this:

/override ci/prow/check

/override ci/prow/sandboxed-containers-operator-e2e

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.

@gkurz
Copy link
Member

gkurz commented Jan 8, 2024

/override ci/prow/check

Copy link

openshift-ci bot commented Jan 8, 2024

@gkurz: Overrode contexts on behalf of gkurz: ci/prow/check

In response to this:

/override ci/prow/check

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.

- All namespaces and child objects with OSC resources
- All OSC custom resource definitions (CRDs)
- sandboxed-containers/namespaces/openshift-sandboxed-containers-operator/**_*_**\_description
- versions in nodes/**_nodename_**/**_nodename_**/version
Copy link
Member

@gkurz gkurz Jan 9, 2024

Choose a reason for hiding this comment

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

Per curiosity, what is the motivation to have nodename in italic and bold ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Because it varies. For my cluster today, the node names will be
tbuskey-240109-2-d9zsw-master-0
tbuskey-240109-2-d9zsw-master-1
tbuskey-240109-2-d9zsw-master-2
tbuskey-240109-2-d9zsw-worker-eastus1-82g44
tbuskey-240109-2-d9zsw-worker-eastus2-s5d6r
tbuskey-240109-2-d9zsw-worker-eastus3-6xdl9

@cpmeadors cpmeadors merged commit eeda647 into openshift:devel Jan 9, 2024
3 of 4 checks passed
@gkurz gkurz mentioned this pull request Jan 10, 2024
@gkurz gkurz mentioned this pull request Feb 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants