-
Notifications
You must be signed in to change notification settings - Fork 42
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
Fix CI and bump actions dependencies to deal with GHA deprecations. #83
Conversation
03f876e
to
04973e5
Compare
This seems to work? But the unit tests shouldn't now be passing---the fix only affects the integration test? Extremely suspicious. |
e322d48
to
54148b9
Compare
@@ -34,12 +34,14 @@ jobs: | |||
secret_access_key: my_secret_access_key | |||
- name: Setup minio | |||
run: | | |||
# The version of minio is RELEASE.2022-04-01T03-41-39Z.fips |
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.
Did something change in minio that we're pinning it?
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.
I haven't dug into this. I tried to pin minio to the version we ought to have been running the last time this test succeeded. It seemed to work. 🤷
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.
OK.
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.
I thought about running a bisection, but being honest I didn't want to faff around with GHA and editing yaml.
Fixes #82.
Drive-by contribution to matrix-org/synapse#14203