openshift / origin

Conformance test suite for OpenShift
http://www.openshift.org
Apache License 2.0
8.5k stars 4.7k forks source link

API-1789: tls artifacts: list ondisk locations alongside secret/configmaps #29090

Closed vrutkovs closed 1 month ago

vrutkovs commented 2 months ago

TLS artifact reports prefer in-cluster locations, so that they can be assigned metadata via annotations. This change prints other ondisk locations so that secret/configmap could be linked to the file on disk

openshift-ci-robot commented 2 months ago

@vrutkovs: This pull request references API-1789 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 "4.18.0" version, but no target version was set.

In response to [this](https://github.com/openshift/origin/pull/29090): >TLS artifact reports prefer in-cluster locations, so that they can be assigned metadata via annotations. This change prints other ondisk locations so that secret/configmap could be linked to the file on disk Instructions for interacting with me using PR comments are available [here](https://prow.ci.openshift.org/command-help?repo=openshift%2Forigin). If you have questions or suggestions related to my behavior, please file an issue against the [openshift-eng/jira-lifecycle-plugin](https://github.com/openshift-eng/jira-lifecycle-plugin/issues/new) repository.
vrutkovs commented 2 months ago

/hold

Alongside "this matches the content" we should also match by purpose - i.e. /etc/kubernetes/static-pod-resources/kube-apiserver-certs/configmaps/trusted-ca-bundle/ca-bundle.crt belongs to kube-apiserver and so on

vrutkovs commented 2 months ago

/hold cancel

deads2k commented 2 months ago

/approve

openshift-ci[bot] commented 2 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: deads2k, dinhxuanvu, sanchezl, vrutkovs

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/openshift/origin/blob/master/OWNERS)~~ [deads2k] - ~~[tls/violations/OWNERS](https://github.com/openshift/origin/blob/master/tls/violations/OWNERS)~~ [deads2k] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
openshift-ci-robot commented 2 months ago

/retest-required

Remaining retests: 0 against base HEAD 253e68c5d78a2c63cfcc1017b0aabfa2f11a000a and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-ci-robot commented 2 months ago

/retest-required

Remaining retests: 0 against base HEAD 253e68c5d78a2c63cfcc1017b0aabfa2f11a000a and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-ci-robot commented 2 months ago

/retest-required

Remaining retests: 0 against base HEAD 4f88248729e93fed2b6e0396a2f77d89594de798 and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD 4f88248729e93fed2b6e0396a2f77d89594de798 and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD 4f88248729e93fed2b6e0396a2f77d89594de798 and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD d8377a33334cc2ef02ec9929a1197b57fd8970a1 and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD d8377a33334cc2ef02ec9929a1197b57fd8970a1 and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD d8377a33334cc2ef02ec9929a1197b57fd8970a1 and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD d8377a33334cc2ef02ec9929a1197b57fd8970a1 and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD 3d2d391631c94b2152b2cc9da80760ca9d07ea2b and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD f3ba3af93fdcacb94b650ab5d59169c7ab64af77 and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD 206be1649d3f42d747c120522448674618cb2ee7 and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD 7cece3be4853a335d2f308a8182675128a91328d and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD 5a39d08c30fc05f173cde9b153423ab76d14a4d4 and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD 5a39d08c30fc05f173cde9b153423ab76d14a4d4 and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD 4307af52394607d1903c178b1b36779b90c769ff and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD 4307af52394607d1903c178b1b36779b90c769ff and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD a5dd89bb7c0a410c97767ad82adaaae74df50407 and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD a5dd89bb7c0a410c97767ad82adaaae74df50407 and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD a5dd89bb7c0a410c97767ad82adaaae74df50407 and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD a5dd89bb7c0a410c97767ad82adaaae74df50407 and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

dinhxuanvu commented 1 month ago

I put a hold on this PR until Vadim returns and looks into some of these failures as they look permanent failures. /hold

vrutkovs commented 1 month ago

/retest-required

vrutkovs commented 1 month ago

/hold cancel

gcp-builds is now passing

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD d3fd6c9ecc783f7629a9160ef7868458e2315fb5 and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-trt-bot commented 1 month ago

Job Failure Risk Analysis for sha: 06078650a6fa3daa2eaeb353e5d7be6276a15828

Job Name Failure Risk
pull-ci-openshift-origin-master-e2e-aws-ovn-serial High
[bz-kube-apiserver][invariant] alert/KubeAPIErrorBudgetBurn should not be at or above info
This test has passed 100.00% of 54 runs on jobs ['periodic-ci-openshift-release-master-ci-4.18-e2e-aws-ovn-serial' 'periodic-ci-openshift-release-master-nightly-4.18-e2e-aws-ovn-serial'] in the last 14 days.

Open Bugs
alert/KubeAPIErrorBudgetBurn should not be at or above info
alert/KubeAPIErrorBudgetBurn should not be at or above info
openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD d3fd6c9ecc783f7629a9160ef7868458e2315fb5 and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-trt-bot commented 1 month ago

Job Failure Risk Analysis for sha: 06078650a6fa3daa2eaeb353e5d7be6276a15828

Job Name Failure Risk
pull-ci-openshift-origin-master-e2e-aws-ovn-serial High
[sig-cluster-lifecycle][Feature:Machines][Serial] Managed cluster should grow and decrease when scaling different machineSets simultaneously [Timeout:30m][apigroup:machine.openshift.io] [Suite:openshift/conformance/serial]
This test has passed 100.00% of 55 runs on jobs ['periodic-ci-openshift-release-master-ci-4.18-e2e-aws-ovn-serial' 'periodic-ci-openshift-release-master-nightly-4.18-e2e-aws-ovn-serial'] in the last 14 days.
---
[sig-network][Feature:EgressIP][apigroup:operator.openshift.io] [external-targets][apigroup:user.openshift.io][apigroup:security.openshift.io] pods should have the assigned EgressIPs and EgressIPs can be updated [Skipped:Network/OpenShiftSDN] [Serial] [Suite:openshift/conformance/serial]
This test has passed 100.00% of 55 runs on jobs ['periodic-ci-openshift-release-master-ci-4.18-e2e-aws-ovn-serial' 'periodic-ci-openshift-release-master-nightly-4.18-e2e-aws-ovn-serial'] in the last 14 days.
openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD d3fd6c9ecc783f7629a9160ef7868458e2315fb5 and 2 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD a9f3a5daefc51690a7ef25ac26f6d5cbdac8a5bc and 1 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total

openshift-ci[bot] commented 1 month ago

@vrutkovs: The following tests 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/e2e-openstack-ovn 06078650a6fa3daa2eaeb353e5d7be6276a15828 link false /test e2e-openstack-ovn
ci/prow/e2e-aws-ovn-single-node 06078650a6fa3daa2eaeb353e5d7be6276a15828 link false /test e2e-aws-ovn-single-node
ci/prow/e2e-aws-ovn-ipsec-serial 06078650a6fa3daa2eaeb353e5d7be6276a15828 link false /test e2e-aws-ovn-ipsec-serial
ci/prow/e2e-aws-ovn-single-node-upgrade 06078650a6fa3daa2eaeb353e5d7be6276a15828 link false /test e2e-aws-ovn-single-node-upgrade
ci/prow/e2e-aws-ovn-single-node-serial 06078650a6fa3daa2eaeb353e5d7be6276a15828 link false /test e2e-aws-ovn-single-node-serial

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes-sigs/prow](https://github.com/kubernetes-sigs/prow/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
openshift-ci-robot commented 1 month ago

/retest-required

Remaining retests: 0 against base HEAD 87600e2ed5bfd6a8f5de18198c8db04d7d152c28 and 0 for PR HEAD 06078650a6fa3daa2eaeb353e5d7be6276a15828 in total