Skip to content

Conversation

@kraenhansen
Copy link
Contributor

@kraenhansen kraenhansen commented Feb 14, 2025

Description

Merging this PR will:

  • Arm the RPM package on GHA running "time-to-first-query" and "auto-update-from" tests using Rocky linux via docker.

Here's a link to a green run: https://github.com/mongodb-js/compass/actions/runs/13342974727.

I've disabled the "linux_rpm test auto-update-to (compass)" tests, as they're waiting for https://github.com/10gen/compass-mongodb-com/pull/122 to deploy.

Checklist

Motivation and Context

  • Bugfix
  • New feature
  • Dependency update
  • Misc

Open Questions

Dependents

Types of changes

  • Backport Needed
  • Patch (non-breaking change which fixes an issue)
  • Minor (non-breaking change which adds functionality)
  • Major (fix or feature that would cause existing functionality to change)

@kraenhansen kraenhansen added no release notes Fix or feature not for release notes no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) labels Feb 14, 2025
@kraenhansen kraenhansen self-assigned this Feb 14, 2025
@kraenhansen kraenhansen requested a review from lerouxb February 14, 2025 22:05
if [[ "$IS_RHEL" == "true" ]]; then
npm run --unsafe-perm --workspace @mongodb-js/compass-smoke-tests start -- --package=linux_rpm --tests=time-to-first-query
# npm run --unsafe-perm --workspace @mongodb-js/compass-smoke-tests start -- --package=linux_rpm --tests=time-to-first-query
Copy link
Contributor Author

@kraenhansen kraenhansen Feb 17, 2025

Choose a reason for hiding this comment

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

I had to disable this on Evergreen, since it's not running as root and the tests now assume that: https://spruce.mongodb.com/version/67b042493a26aa0007abde21/tasks?sorts=STATUS%3AASC%3BBASE_STATUS%3ADESC&variant=smoketest-rhel-compass

@kraenhansen kraenhansen marked this pull request as ready for review February 17, 2025 08:17
@kraenhansen kraenhansen merged commit d3f8ade into main Feb 17, 2025
36 checks passed
@kraenhansen kraenhansen deleted the kh/smoke-test/rocky-linux-on-gha branch February 17, 2025 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no release notes Fix or feature not for release notes no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants