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.

Sign up for GitHub

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

Jump to bottom

avoid error on missing 'buildout/*.output.ipynb' to archive artifact #92

Merged
merged 2 commits into from
Nov 4, 2021
Merged

avoid error on missing 'buildout/*.output.ipynb' to archive artifact #92

merged 2 commits into from
Nov 4, 2021

Conversation

fmigneault
Copy link
Contributor

@fmigneault fmigneault commented Nov 4, 2021 •
edited by tlvu
Loading

Changes

  • Adds check to avoid error on missing buildout/*.output.ipynb to archive artifact.

Related Issue / Discussion

tlvu
tlvu requested changes Nov 4, 2021
View reviewed changes
Jenkinsfile Outdated Show resolved Hide resolved
@tlvu tlvu changed the title avoid error on missing 'buildout/env-dump/' dir to archive artifact avoid error on missing 'buildout/*.output.ipynb' to archive artifact Nov 4, 2021
@fmigneault
Copy link
Contributor Author

fmigneault commented Nov 4, 2021 •
edited
Loading

@matprov
I'm getting ERROR: No item named PAVICS-e2e-workflow-tests/ignore-optional-artifact found.
I recall providing PAVICS_E2E_WORKFLOW_TESTS_BRANCH was working when developing the Weaver test.
Do you know what might be the problem now?

edit
Ok found it. Had to force a multi-branch scan.

@fmigneault
Copy link
Contributor Author

Fixed as per: http://daccs-jenkins.crim.ca/job/PAVICS-e2e-workflow-tests/job/ignore-optional-artifact/2

@fmigneault fmigneault merged commit 9ad91bd into master Nov 4, 2021
@fmigneault fmigneault deleted the ignore-optional-artifact branch November 4, 2021 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tlvu tlvu tlvu approved these changes

@matprov matprov matprov approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fmigneault @matprov @tlvu