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

fix(pg): update versions to be tested (includes drop support for pg 1.2) #1108

Merged
merged 2 commits into from
Aug 8, 2024

Conversation

robbkidd
Copy link
Member

@robbkidd robbkidd commented Aug 8, 2024

  • drop pg-1.2
  • add pg-1.4
    • This used to be latest! Now latest in 1.5.

With thanks to @matthewtusker for raising the issue and helping to resolve!

* pg 1.2.x does not work under Ruby >= 3.0.
* We currently support and test only Ruby >= 3.0.
@robbkidd robbkidd added the bug Something isn't working label Aug 8, 2024
@robbkidd robbkidd self-assigned this Aug 8, 2024
@robbkidd robbkidd requested review from a team August 8, 2024 17:03
@robbkidd
Copy link
Member Author

robbkidd commented Aug 8, 2024

Is this a fix!: change to bump the version number differently because of drop support?

@robbkidd robbkidd changed the title fix: update pg test versions (includes drop support for pg 1.2) fix(pg): update versions to be tested (includes drop support for pg 1.2) Aug 8, 2024
@robbkidd robbkidd mentioned this pull request Aug 8, 2024
@arielvalentin arielvalentin merged commit 19c86ca into open-telemetry:main Aug 8, 2024
58 checks passed
@github-actions github-actions bot mentioned this pull request Aug 8, 2024
@robbkidd robbkidd deleted the update-pg-test-versions branch August 8, 2024 18:54
robbkidd added a commit that referenced this pull request Aug 8, 2024
The existing loop over appraisals for a gem is swallowing any error exits returned by commands within the loop.

Resolves #1097. The specific error mentioned in that issue for failure to bundle install pg 1.2 was fixed in #1108. This change fixes the appraisal loop.

Fix appraisal loop swallowing errors:

* add an explicit exit to the appraisal loop if any command in the chain fails so that the CI step also fails

Fix other errors that were being swallowed:

* add a workaround for GEM_HOME file permission problems during bundle install
  *  introduced to GitHub runner images sometime early July 2024
* bound which version of Ruby the gruf gem tests are run against
  * its gemspec includes upper bounds on Ruby versions which cause bundler install to fail under newer Rubies
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants