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

Bump our fork to v1.25 #7

Merged
merged 5 commits into from
Jan 25, 2024
Merged

Bump our fork to v1.25 #7

merged 5 commits into from
Jan 25, 2024

Conversation

adolsalamanca
Copy link

We're first bumping our driver to upstream version v1.25, to then introduce a small modification to avoid race conditions when we try to modify both disk size and specs (IOPS, Throughput), making only a unified request to AWS API without having to deal with 6h cooldowns.

Issue
https://github.com/timescale/forge-issues/issues/1077

Copy link

@minkimipt minkimipt left a comment

Choose a reason for hiding this comment

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

as I understand these are just the changes that happened in the repo between our release and the last release. And that this doesn't include any code that we had to control iops and throughput form the EBS driver itself.

@adolsalamanca
Copy link
Author

as I understand these are just the changes that happened in the repo between our release and the last release. And that this doesn't include any code that we had to control iops and throughput form the EBS driver itself.

Exactly, this is a 1:1 port of the version 1.25.

@adolsalamanca adolsalamanca merged commit cf9200b into timescale Jan 25, 2024
10 checks passed
@adolsalamanca adolsalamanca deleted the adol/bump-to-1.25 branch January 25, 2024 17:20
adolsalamanca added a commit that referenced this pull request Jan 26, 2024
We're bumping our driver to upstream version v1.25, without any modifications.
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.

2 participants