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

feature(smokes): Run Smoke Tests on all supported platforms #1980

Merged
merged 8 commits into from
Jul 15, 2024

Conversation

jonathanrainer
Copy link
Contributor

@jonathanrainer jonathanrainer commented Jul 14, 2024

Now that we have smoke testing established for rover dev this rolls it out to all the supported architectures.

At the moment the workflow is triggered via a cron at 9am each day, so as to test whether it's robust before we start gating PRs behind it etc. When it triggers it calculates the 3 latest (non-pre-release) versions of router and supergraph and forms a matrix with those and the supported OS's.

Those supported OSes are:

Originally there were plans to test in a glibc 2.17 environment, however router binaries do not work here (context: https://apollograph.slack.com/archives/C02UX05LF4K/p1721026197480029) so this has been scaled back, as well as the plans to test musl compiled binaries because of the lack of supergraph binaries.

A fully successful run can be seen here: https://github.com/apollographql/rover/actions/runs/9938343741/job/27452394532

@jonathanrainer jonathanrainer force-pushed the jr/all-the-smokes branch 30 times, most recently from 94347c9 to 1e2e0ae Compare July 15, 2024 10:45
@jonathanrainer jonathanrainer force-pushed the jr/all-the-smokes branch 5 times, most recently from 98fc461 to c2c5e5e Compare July 15, 2024 12:05
@jonathanrainer jonathanrainer marked this pull request as ready for review July 15, 2024 12:26
@jonathanrainer jonathanrainer requested a review from a team as a code owner July 15, 2024 12:26
@jonathanrainer jonathanrainer added this to the v0.25.0 milestone Jul 15, 2024
@jonathanrainer jonathanrainer merged commit 7c7bc8f into main Jul 15, 2024
18 checks passed
@jonathanrainer jonathanrainer deleted the jr/all-the-smokes branch July 15, 2024 14:02
@jonathanrainer jonathanrainer mentioned this pull request Jul 19, 2024
jonathanrainer added a commit that referenced this pull request Jul 22, 2024
# [0.25.0] - 2024-07-22

## 🚀 Features

- **Enable Retries For Transient Errors Connecting To Graphs/Subgraphs -
@jonathanrainer PR #1936**

This turns on retries at the HTTP level for connections to
graphs/subgraphs to minimize connection resets and cancellations. Also,
a new --subgraph-retries flag for rover dev lets you set the number of
retries allowed when trying to re-establish a connection.

- **Add `--graph-ref` flag to `rover dev` - @dotdat PR #1984**

Introduces subgraph mirroring to rover dev. Subgraph mirroring inherits
the subgraph routing URLs and schemas from an existing Studio graphref.
This makes it easy to spin up a locally running supergraph without
maintaining a supergraph config. [See
here](https://www.apollographql.com/docs/rover/commands/dev#starting-a-session-from-a-graphos-studio-variant)
for more information.

## 🐛 Fixes

- **Fixes issues related to passing filenames to `--output` -
@jonathanrainer PR #1996**

An issue was raised whereby previous versions of Rover supported passing
filenames to the `--output` flag but this was
broken in v0.24.0. This has now been fixed and the previous
functionality restored.

## 🛠 Maintenance

- **Expand Smoke Tests To Run On All Supported Platforms -
@jonathanrainer PR #1980**
- **Fix cron expression, so it runs only once per day - @jonathanrainer
PR #1986**
- **Ensure we always use the correct version of Federation when testing
- @jonathanrainer PR #1987**
- **Add manual Smoke test invocation and pin Windows to `npm@9` for
testing - @jonathanrainer PR #1989**
- **Update apollographql/router to v1.51.0 - @jonathanrainer PR #1988**
- **Update node.js packages - @jonathanrainer PR #1979**

Includes `@eslint/compat` to v1.1.1, `eslint` to v9.7.0, `node.js` to
v20.15.1, `npm` to v10.8.2 and `prettier` to v3.3.3

- **Make sure x86 Mac Tests use 'latest' supergraph plugin version -
@jonathanrainer PR #1990**
- **Make sure homebrew runs `brew update` when we use it -
@jonathanrainer PR #1993**



## 📚 Documentation

- **Adds `graph-ref` flag to dev subcommand docs - @jackonawalk PR
#1945**
- **Update schema proposals capabilities docs - @Meschreiber PR #1949**
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants