Skip to content

Fix ocp4 e2e tests#6026

Merged
JAORMX merged 2 commits intoComplianceAsCode:masterfrom
JAORMX:fix-comp-op
Aug 28, 2020
Merged

Fix ocp4 e2e tests#6026
JAORMX merged 2 commits intoComplianceAsCode:masterfrom
JAORMX:fix-comp-op

Conversation

@JAORMX
Copy link
Copy Markdown
Contributor

@JAORMX JAORMX commented Aug 28, 2020

We needed to change our deployment strategy and upgrade the operator...

@JAORMX
Copy link
Copy Markdown
Contributor Author

JAORMX commented Aug 28, 2020

/test all

JAORMX and others added 2 commits August 28, 2020 13:58
We now need to use catalog sources instead of operator source objects.

Co-Authored-By: Jakub Hrozek <jhrozek@redhat.com>
Newer versions of the operator introduced differences in the go structs
we were using, so these need to be fixed as we upgrade.

Co-Authored-By: Jakub Hrozek <jhrozek@redhat.com>
@openshift-ci-robot
Copy link
Copy Markdown
Collaborator

@JAORMX: The following tests failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
ci/prow/e2e-aws-rhcos4-e8 e32d045 link /test e2e-aws-rhcos4-e8
ci/prow/e2e-aws-rhcos4-moderate e32d045 link /test e2e-aws-rhcos4-moderate

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/test-infra repository. I understand the commands that are listed here.

@jhrozek
Copy link
Copy Markdown
Collaborator

jhrozek commented Aug 28, 2020

During a local run, I saw:

    --- PASS: TestE2e/Parameter_setup_and_validation (0.01s)
    --- PASS: TestE2e/Operator_setup (78.14s)
    --- PASS: TestE2e/Run_first_compliance_scan (456.44s)
    --- PASS: TestE2e/Wait_for_Remediations_to_apply (30.07s)
    --- FAIL: TestE2e/Run_second_compliance_scan (581.19s)

And the nodes were still updating which seems like we still need to tune the isNodeReady function. I'm going to look into that..

@JAORMX
Copy link
Copy Markdown
Contributor Author

JAORMX commented Aug 28, 2020

Merging this. Seems we have an issue waiting for the nodes... But that's unrelated. This gets us further

@JAORMX JAORMX merged commit a6c1277 into ComplianceAsCode:master Aug 28, 2020
@jhrozek
Copy link
Copy Markdown
Collaborator

jhrozek commented Aug 30, 2020 via email

@shaneboulden shaneboulden mentioned this pull request Feb 2, 2021
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.

3 participants