Skip to content

JPERF-188: Improve stopJira to wait for Jira process to shutdown inst…#82

Closed
konradmars wants to merge 1 commit into
atlassian:masterfrom
konradmars:issue/JPERF-188-improve-stop-jira-condition
Closed

JPERF-188: Improve stopJira to wait for Jira process to shutdown inst…#82
konradmars wants to merge 1 commit into
atlassian:masterfrom
konradmars:issue/JPERF-188-improve-stop-jira-condition

Conversation

@konradmars

Copy link
Copy Markdown

JPERF-188: Improve stopJira to wait for Jira process to shutdown instead of all Java processes on the node

Renamed branch and commit

Added Changelog entry

@dagguh

dagguh commented Feb 19, 2020

Copy link
Copy Markdown
Contributor

FYI branch name doesn't really matter and the commit message can be amended and force-pushed. You have full permissions and control over your fork.

@dagguh

dagguh commented Feb 19, 2020

Copy link
Copy Markdown
Contributor

Actually, according to myself, it's testable via the existing AwsDatasetModificationIT, which is better, because it directly pertains to dataset manipulations and covers both ends of the CustomDatasetSource - saving and reusing.
We just gotta add the second Java process and again, according to myself it's about using the AtlassianSplunkForwarder.
@KonradMarszalek LMK if you wanna adapt the test. If not, we're attacking this PR with @mzyromski-atlassian anyway, so we can cover that as an exercise.

@konradmars

Copy link
Copy Markdown
Author

@KonradMarszalek LMK if you wanna adapt the test

If possible I leave you to it as I just had coffee-break contribution capacity.

@dagguh

dagguh commented Feb 19, 2020

Copy link
Copy Markdown
Contributor

Awesome, thanks for the fix and the manual test ❤️

@dagguh

dagguh commented Feb 25, 2020

Copy link
Copy Markdown
Contributor

Adopted in #84 with CI

@dagguh dagguh closed this Feb 25, 2020
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