-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Uninstall Docker Compose v1 from CI so it's not used for tests #2114
Conversation
56cc302
to
1633845
Compare
docker-compose
in v2b598c1a
to
c9eeeb2
Compare
a9c1fc7
to
9f09714
Compare
@@ -11,24 +11,24 @@ echo "Creating backup..." | |||
touch $(pwd)/sentry/backup.json | |||
chmod 666 $(pwd)/sentry/backup.json | |||
# Command here matches exactly what we have in our docs https://develop.sentry.dev/self-hosted/backup/#backup | |||
docker-compose run -v $(pwd)/sentry:/sentry-data/backup --rm -T -e SENTRY_LOG_LEVEL=CRITICAL web export /sentry-data/backup/backup.json | |||
$dc run -v $(pwd)/sentry:/sentry-data/backup --rm -T -e SENTRY_LOG_LEVEL=CRITICAL web export /sentry-data/backup/backup.json |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
using $dc
since our scripts do some pre-script setup that is failing CI (error-handling, dc-detect-version, detect-platform, trap)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yisss, exactly the cases we wanted to fix.
.github/workflows/test.yml
Outdated
# Docker Compose v1 is installed here, remove it | ||
sudo rm -f "/usr/local/bin/docker-compose" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's move this up, next to other rm
commands.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be after downloading docker-compose, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't really matter. I'd do it before to be consistent and not accidentally delete the downloaded one.
@@ -11,24 +11,24 @@ echo "Creating backup..." | |||
touch $(pwd)/sentry/backup.json | |||
chmod 666 $(pwd)/sentry/backup.json | |||
# Command here matches exactly what we have in our docs https://develop.sentry.dev/self-hosted/backup/#backup | |||
docker-compose run -v $(pwd)/sentry:/sentry-data/backup --rm -T -e SENTRY_LOG_LEVEL=CRITICAL web export /sentry-data/backup/backup.json | |||
$dc run -v $(pwd)/sentry:/sentry-data/backup --rm -T -e SENTRY_LOG_LEVEL=CRITICAL web export /sentry-data/backup/backup.json |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yisss, exactly the cases we wanted to fix.
This will uninstall docker compose v1.29 from CI so
docker-compose
cannot be used.CI runs to indicate this is the case: