Skip to content

Comments

Update testing-infrastructure.md - Added verbiage around Jenkins job failing after a release and how to troubleshoot it#10951

Merged
pdurbin merged 2 commits intodevelopfrom
ofahimIQSS-patch-1
Oct 22, 2024
Merged

Update testing-infrastructure.md - Added verbiage around Jenkins job failing after a release and how to troubleshoot it#10951
pdurbin merged 2 commits intodevelopfrom
ofahimIQSS-patch-1

Conversation

@ofahimIQSS
Copy link
Contributor

@ofahimIQSS ofahimIQSS commented Oct 22, 2024

Description:
After a release, it's important to update the version number in the PR to match the latest release version. Failing to do so can cause the Jenkins build to fail. Ensure that the PR version number aligns with the current Dataverse version to avoid this issue. For this PR, we are adding a bullet under the Infrastructure of Testing section of the QA Guide to provide a solution.

Added statement under QA Guide> Infrastructure of Testing> 6. When a Jenkins job fails after a release, it might be due to the version number in the 'pom.xml' file not being updated in the pull request (PR). To Verify this, open the relevant GitHub ticket, navigate to the PR branch, and go to 'dataverse>modules>dataverse-parent>pom.xml'. Look for the version number, typically shown as '6.3', and ensure it matches the current Dataverse build version.

Preview at https://dataverse-guide--10951.org.readthedocs.build/en/10951/qa/testing-infrastructure.html

After a release, it's important to update the version number in the PR to match the latest release version. Failing to do so can cause the Jenkins build to fail. Ensure that the PR version number aligns with the current Dataverse version to avoid this issue.
@ofahimIQSS ofahimIQSS requested a review from pdurbin October 22, 2024 18:59
Copy link
Member

@pdurbin pdurbin left a comment

Choose a reason for hiding this comment

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

Overall, looks good. I added a suggestion.

Co-authored-by: Philip Durbin <philip_durbin@harvard.edu>
Copy link
Member

@pdurbin pdurbin left a comment

Choose a reason for hiding this comment

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

Looks good, merging.

@pdurbin pdurbin merged commit f6a0d53 into develop Oct 22, 2024
@pdurbin pdurbin deleted the ofahimIQSS-patch-1 branch October 22, 2024 20:39
@pdurbin pdurbin removed their assignment Oct 22, 2024
@pdurbin pdurbin added this to the 6.5 milestone Oct 22, 2024
@ofahimIQSS ofahimIQSS removed their assignment Oct 22, 2024
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