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

Added fix for upstream pipeline failure. #3504

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gswami91
Copy link
Contributor

Added fix for upstream pipeline failure https://gitlab.com/dogtagpki/pki/-/pipelines/291988444

Signed-off-by: Gaurav Swami [email protected]

@gswami91 gswami91 force-pushed the fix_upstream_pipeline branch 2 times, most recently from 611c876 to 25415ef Compare April 26, 2021 14:58
--ansible-module-path tests/dogtag/pytest-ansible/common-modules \
--junit-xml pki_cert_scep_1664435_junit.xml \
-qsvvv \
tests/dogtag/pytest-ansible/pytest/ca/scep/test_bug_1664435_pki_ca_scep_enrollment.py
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it possible to merge this into the previous step (lines 100-110) so we don't have to keep modifying the workflow whenever we add new tests?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I added this as per my previous job (https://github.com/dogtagpki/pki/pull/3351/files)

Should I add jobs as below?


          -qsvvv \
          tests/dogtag/pytest-ansible/pytest/sanity/test_role_users.py
          tests/dogtag/pytest-ansible/pytest/ca/scep/test_bug_1664435_pki_ca_scep_enrollment.py

Copy link
Contributor

Choose a reason for hiding this comment

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

I'm thinking the SCEP test should be added into tests/dogtag/pytest-ansible/pytest/sanity/test_role_users.py so that if we want to run sanity test manually we only need to run just that one file (assuming SCEP test is part of sanity test).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@gswami91 gswami91 force-pushed the fix_upstream_pipeline branch 6 times, most recently from f4ff01e to 2d824f6 Compare May 3, 2021 10:58
@edewata
Copy link
Contributor

edewata commented May 4, 2021

@gswami90 Hi, I've fixed the missing pip3 issue. Could you rebase the PR? Thanks.

@gswami91 gswami91 force-pushed the fix_upstream_pipeline branch from 2d824f6 to 87356f5 Compare May 5, 2021 14:36
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.

2 participants