Skip to content

Comments

document harvesting server IT config#9936

Merged
kcondon merged 1 commit intodevelopfrom
9931_IT_options
Sep 19, 2023
Merged

document harvesting server IT config#9936
kcondon merged 1 commit intodevelopfrom
9931_IT_options

Conversation

@donsizemore
Copy link
Contributor

@donsizemore donsizemore commented Sep 19, 2023

What this PR does / why we need it:

Documents two jvm-options which must be set for harvesting server tests to succeed.

Which issue(s) this PR closes:

Special notes for your reviewer:

I stole verbiage from #9738; please correct me if it's wrong in any way.

Suggestions on how to test this:

These jvm-options are already set via Jenkins/Ansible.

Does this PR introduce a user interface change? If mockups are available, please link/include them here:

No

Is there a release notes update needed for this change?:

No

Additional documentation:

None

@pdurbin pdurbin added Feature: Developer Guide Size: 0.5 A percentage of a sprint. 0.35 hours labels Sep 19, 2023
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.

@kcondon kcondon self-assigned this Sep 19, 2023
@kcondon kcondon merged commit ca79ae6 into develop Sep 19, 2023
@kcondon kcondon deleted the 9931_IT_options branch September 19, 2023 20:38
@pdurbin pdurbin added this to the 6.1 milestone Sep 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature: Developer Guide Size: 0.5 A percentage of a sprint. 0.35 hours

Projects

None yet

Development

Successfully merging this pull request may close these issues.

dev guide: add jvm-option required for HarvestingServerIT.testMultiRecordOaiSet to succeed Update "Getting Set Up to Run REST Assured Tests"

3 participants