-
Notifications
You must be signed in to change notification settings - Fork 2.2k
SANDBOX-800: save CRs when test fails #61582
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
Changes from all commits
a68e5c4
3c984a4
e12ab95
2dc90e3
ee8480f
67ece3f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||
|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,23 @@ | ||||||||
| approvers: | ||||||||
| - alexeykazakov | ||||||||
| - fbm3307 | ||||||||
| - matousjobanek | ||||||||
| - metlos | ||||||||
| - mfrancisc | ||||||||
| - rajivnathan | ||||||||
| - ranakan19 | ||||||||
| - rsoaresd | ||||||||
| - xcoulon | ||||||||
| - jrosental | ||||||||
| options: {} | ||||||||
| reviewers: | ||||||||
| - alexeykazakov | ||||||||
| - fbm3307 | ||||||||
| - matousjobanek | ||||||||
| - metlos | ||||||||
| - mfrancisc | ||||||||
| - rajivnathan | ||||||||
| - ranakan19 | ||||||||
| - rsoaresd | ||||||||
| - xcoulon | ||||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||
| - jrosental | ||||||||
| Original file line number | Diff line number | Diff line change | ||||||
|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,23 @@ | ||||||||
| approvers: | ||||||||
| - alexeykazakov | ||||||||
| - fbm3307 | ||||||||
| - matousjobanek | ||||||||
| - metlos | ||||||||
| - mfrancisc | ||||||||
| - rajivnathan | ||||||||
| - ranakan19 | ||||||||
| - rsoaresd | ||||||||
| - xcoulon | ||||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||
| - jrosental | ||||||||
| options: {} | ||||||||
| reviewers: | ||||||||
| - alexeykazakov | ||||||||
| - fbm3307 | ||||||||
| - matousjobanek | ||||||||
| - metlos | ||||||||
| - mfrancisc | ||||||||
| - rajivnathan | ||||||||
| - ranakan19 | ||||||||
| - rsoaresd | ||||||||
| - xcoulon | ||||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||
| - jrosental | ||||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,29 @@ | ||
| { | ||
| "path": "codeready-toolchain/aws/codeready-toolchain-aws-workflow.yaml", | ||
| "owners": { | ||
| "approvers": [ | ||
| "alexeykazakov", | ||
| "fbm3307", | ||
| "matousjobanek", | ||
| "metlos", | ||
| "mfrancisc", | ||
| "rajivnathan", | ||
| "ranakan19", | ||
| "rsoaresd", | ||
| "xcoulon", | ||
| "jrosental" | ||
| ], | ||
| "reviewers": [ | ||
| "alexeykazakov", | ||
| "fbm3307", | ||
| "matousjobanek", | ||
| "metlos", | ||
| "mfrancisc", | ||
| "rajivnathan", | ||
| "ranakan19", | ||
| "rsoaresd", | ||
| "xcoulon", | ||
| "jrosental" | ||
| ] | ||
| } | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| workflow: | ||
| as: codeready-toolchain-aws | ||
| steps: | ||
| allow_best_effort_post_steps: true | ||
| pre: | ||
| - chain: ipi-aws-pre | ||
| post: | ||
| - ref: codeready-toolchain-gather | ||
| optional_on_success: true | ||
| - chain: ipi-aws-post | ||
| documentation: |- | ||
| This workflow provides is based on /ci-operator/step-registry/ipi/aws/ipi-aws-workflow.yaml. | ||
| Additionally, it adds codeready-toolchain-gather which will save the Custom Resources on ARTIFACT dir. |
| Original file line number | Diff line number | Diff line change | ||||||
|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,23 @@ | ||||||||
| approvers: | ||||||||
| - alexeykazakov | ||||||||
| - fbm3307 | ||||||||
| - matousjobanek | ||||||||
| - metlos | ||||||||
| - mfrancisc | ||||||||
| - rajivnathan | ||||||||
| - ranakan19 | ||||||||
| - rsoaresd | ||||||||
| - xcoulon | ||||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||
| - jrosental | ||||||||
| options: {} | ||||||||
| reviewers: | ||||||||
| - alexeykazakov | ||||||||
| - fbm3307 | ||||||||
| - matousjobanek | ||||||||
| - metlos | ||||||||
| - mfrancisc | ||||||||
| - rajivnathan | ||||||||
| - ranakan19 | ||||||||
| - rsoaresd | ||||||||
| - xcoulon | ||||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. So sorry! I added it here: 3c984a4 I also added jrosental to the OWNERS of the repos (if not in OWNERS of the repos that runs the e2e jobs - toolchain-e2e, host-operator, member-operator, registration-service and workload-analyzer - his name will be removed in theses. For more info, check this Slack thread):
P.S. I also added him in api and toolchain-common to update there |
||||||||
| - jrosental | ||||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,25 @@ | ||
| #!/bin/bash | ||
|
|
||
| # queue function stolen from https://steps.ci.openshift.org/reference/gather-extra ;-) | ||
| function queue() { | ||
| local TARGET="${1}" | ||
| shift | ||
| local LIVE | ||
| LIVE="$(jobs | wc -l)" | ||
| while [[ "${LIVE}" -ge 45 ]]; do | ||
| sleep 1 | ||
| LIVE="$(jobs | wc -l)" | ||
| done | ||
| echo "${@}" | ||
| if [[ -n "${FILTER:-}" ]]; then | ||
| "${@}" | "${FILTER}" >"${TARGET}" & | ||
| else | ||
| "${@}" >"${TARGET}" & | ||
| fi | ||
| } | ||
|
|
||
| # Resources | ||
| CRD_LIST=$(oc get crds -o jsonpath='{.items[?(@.spec.group=="toolchain.dev.openshift.com")].metadata.name}') | ||
| for CRD in ${CRD_LIST}; do | ||
| queue ${ARTIFACT_DIR}/${CRD}.json oc --insecure-skip-tls-verify --request-timeout=5s get ${CRD} --all-namespaces -o json | ||
| done |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,29 @@ | ||
| { | ||
| "path": "codeready-toolchain/gather/codeready-toolchain-gather-ref.yaml", | ||
| "owners": { | ||
| "approvers": [ | ||
| "alexeykazakov", | ||
| "fbm3307", | ||
| "matousjobanek", | ||
| "metlos", | ||
| "mfrancisc", | ||
| "rajivnathan", | ||
| "ranakan19", | ||
| "rsoaresd", | ||
| "xcoulon", | ||
| "jrosental" | ||
| ], | ||
| "reviewers": [ | ||
| "alexeykazakov", | ||
| "fbm3307", | ||
| "matousjobanek", | ||
| "metlos", | ||
| "mfrancisc", | ||
| "rajivnathan", | ||
| "ranakan19", | ||
| "rsoaresd", | ||
| "xcoulon", | ||
| "jrosental" | ||
| ] | ||
| } | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,14 @@ | ||
| ref: | ||
| as: codeready-toolchain-gather | ||
| best_effort: true | ||
| optional_on_success: true | ||
| from: cli | ||
| commands: codeready-toolchain-gather-commands.sh | ||
| timeout: 900s | ||
| grace_period: 60s | ||
| resources: | ||
| requests: | ||
| cpu: 300m | ||
| memory: 300Mi | ||
| documentation: |- | ||
| The pre-deprovision artifacts step collects codeready-toolchain artifacts which are either not covered by must-gather or gather-extra. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.