Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

some fixes to cartodb ci #16179

Merged
merged 23 commits into from
Feb 26, 2021
Merged

Conversation

ibrahimmenem
Copy link
Contributor

@ibrahimmenem ibrahimmenem commented Feb 23, 2021

https://app.clubhouse.io/cartoteam/story/137764/fix-cartodb-backend-tests

Now we will upload 3 log files to https://console.cloud.google.com/storage/browser/cartodb-ci-tmp-logs/test_logs_{BUILD_ID}

Logs of parallel tests:

  • test_logs_${BUILD_ID}/parallel_tests_logs
  • test_logs_${BUILD_ID}/docker_compose_logs

Logs of second try:

  • test_logs_${BUILD_ID}/serial_tests_logs

the log files will be available for 1 day, then automatically deleted

@ibrahimmenem ibrahimmenem force-pushed the print-logs-of-failed-parallel-tests branch from 8648cc6 to 33f8ba3 Compare February 25, 2021 12:06
@ibrahimmenem ibrahimmenem changed the title print-logs-of-failed-parallel-tests some fixes to cartodb ci Feb 25, 2021
@shortcut-integration
Copy link

This pull request has been linked to Clubhouse Story #137764: Fix cartodb backend tests.

@ibrahimmenem ibrahimmenem force-pushed the print-logs-of-failed-parallel-tests branch from 46875b8 to 83cfc60 Compare February 25, 2021 17:25
Copy link
Contributor

@rafatower rafatower left a comment

Choose a reason for hiding this comment

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

LGTM, just left a one-line change request

Thanks!

script/ci/secondTry.sh Outdated Show resolved Hide resolved
@ibrahimmenem ibrahimmenem merged commit dbaf138 into master Feb 26, 2021
@ibrahimmenem ibrahimmenem deleted the print-logs-of-failed-parallel-tests branch February 26, 2021 13:20
@rafatower
Copy link
Contributor

🤖 please make sure all the checks pass before merging @ibrahimmenem

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.

None yet

2 participants