Skip to content

Comments

WPB-4910 include build timestamp in s3 upload path for test logs#3621

Merged
battermann merged 9 commits intodevelopfrom
WPB-4910-include-test-run-id-in-name-of-test-logs-that-are-uploaded-to-s-3
Sep 29, 2023
Merged

WPB-4910 include build timestamp in s3 upload path for test logs#3621
battermann merged 9 commits intodevelopfrom
WPB-4910-include-test-run-id-in-name-of-test-logs-that-are-uploaded-to-s-3

Conversation

@battermann
Copy link
Contributor

@battermann battermann commented Sep 28, 2023

https://wearezeta.atlassian.net/browse/WPB-4910

Results in logs like this:

Copy logs-integration to s3://wire-server-test-logs/test-logs-0.0.2-pr.5027/integration-0.0.2-pr.5027-1695992949.log
upload: ./logs-integration to s3://wire-server-test-logs/test-logs-0.0.2-pr.5027/integration-0.0.2-pr.5027-1695992949.log

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Sep 28, 2023
@battermann battermann changed the title WPB-4910 include BUILD_NAME in s3 upload path for test logs WPB-4910 include build timestamp in s3 upload path for test logs Sep 29, 2023
@battermann battermann marked this pull request as ready for review September 29, 2023 13:36
@battermann battermann merged commit 2ca6e3a into develop Sep 29, 2023
@battermann battermann deleted the WPB-4910-include-test-run-id-in-name-of-test-logs-that-are-uploaded-to-s-3 branch September 29, 2023 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants