Skip to content

GH Actions -> archive surefire reports (#21913). - #21875

Closed
lhanusov wants to merge 1 commit into
keycloak:mainfrom
lhanusov:lhanusov/main/CIAM-5267
Closed

GH Actions -> archive surefire reports (#21913).#21875
lhanusov wants to merge 1 commit into
keycloak:mainfrom
lhanusov:lhanusov/main/CIAM-5267

Conversation

@lhanusov

@lhanusov lhanusov commented Jul 21, 2023

Copy link
Copy Markdown
Contributor

Closes #21913

@lhanusov
lhanusov requested a review from a team as a code owner July 21, 2023 11:41
@lhanusov
lhanusov requested review from miquelsi and stianst July 21, 2023 11:42
@lhanusov

Copy link
Copy Markdown
Contributor Author

@stianst @miquelsi Hi, please can you take a look on this PR, related to surefire reports? Currently it will be triggered only with release branch 22.0 and retention policy = 1 day.

@lhanusov lhanusov self-assigned this Jul 21, 2023
@lhanusov lhanusov added area/testsuite Indicates an issue on the Testsuite area team/continuous-testing labels Jul 21, 2023
@lhanusov lhanusov added this to the 22.0.2 milestone Jul 21, 2023
@lhanusov

lhanusov commented Jul 21, 2023

Copy link
Copy Markdown
Contributor Author

Getting multiple artifacts from GH can be simply achieved by gh cli: gh run download 5621318494 -p surefire-*tests*.

@stianst stianst left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

  • Remove reference to internal JIRA.
  • Refactor into a composite action.
  • Only trigger for push and workflow_dispatch events (not for pull_requests)

Can you also elaborate on how the logs will be fetched? Retention days 1 seems a bit short perhaps?

@lhanusov lhanusov changed the title [CIAM-5267] GH Actions -> archive surefire reports. GH Actions -> archive surefire reports. Jul 24, 2023
@lhanusov
lhanusov force-pushed the lhanusov/main/CIAM-5267 branch from 7f0684e to 706c5f5 Compare July 24, 2023 14:30
@lhanusov

lhanusov commented Jul 24, 2023

Copy link
Copy Markdown
Contributor Author

@stianst @miquelsi Comments addressed #21915, in case of the artifact handling and process explanation -> Another CI will poll scm every 10 minutes and will get GH surefire artifacts based on change -> GH webhook is for internal service impossible. The CI will store it for longer period. Those artifact will be uploaded to reporting tool only once per release (will select particular revision), manual approval. Does it make sense?

@lhanusov lhanusov changed the title GH Actions -> archive surefire reports. GH Actions -> archive surefire reports (#21913). Jul 24, 2023
@lhanusov lhanusov closed this Jul 24, 2023
@lhanusov
lhanusov deleted the lhanusov/main/CIAM-5267 branch July 24, 2023 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/testsuite Indicates an issue on the Testsuite area flaky-test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GH Actions - Surefire reports artifact upload

2 participants