Skip to content

Conversation

@lerouxb
Copy link
Contributor

@lerouxb lerouxb commented Jan 13, 2025

Still a draft because it has to git clone and start the compass autoupdate server. Which I'm running separately locally during testing for now.

To start the update server:

~/mongo/compass-mongodb-com % PORT=8080 UPDATE_CHECKER_ALLOW_DOWNGRADES=true npm run start

}
});

// Exit child process if main process exits
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copied from https://github.com/kraenhansen/bufout which we should probably just use.

console.log(appPath);

try {
// TODO: start the autoupdate server
Copy link
Contributor Author

Choose a reason for hiding this comment

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

An actual TODO for this PR. We could also start the autoupdate server on the outside and just leave it running, but I think we'll want to run it with a different configuration once we get to the "autoupdate TO" tests.

set -e
docker build -t compass/web ./
docker run -p 8080:8080 -t -e PORT=8080 -e UPDATE_CHECKER_ALLOW_DOWNGRADES=true -d compass/web
Copy link
Contributor Author

@lerouxb lerouxb Jan 14, 2025

Choose a reason for hiding this comment

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

The plan is to run two of these: One that will downgrade the newly packaged compass to the most recent release like this, the other will upgrade from the most recent release to this newly packaged one. This newly packaged version isn't released yet, so we'll have to make some changes to compass-mongodb-com to be able to tell it about the new package. See https://jira.mongodb.org/browse/COMPASS-8761

echo "DEV_VERSION_IDENTIFIER: $DEV_VERSION_IDENTIFIER"
echo "EVERGREEN_BUCKET_NAME: $EVERGREEN_BUCKET_NAME"
echo "EVERGREEN_BUCKET_KEY_PREFIX: $EVERGREEN_BUCKET_KEY_PREFIX"
echo "export DEV_VERSION_IDENTIFIER=$DEV_VERSION_IDENTIFIER"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is to make it easy to copy/paste these into your local shell when working on the smoketests. Otherwise you have to transform the string manually every time.

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.

3 participants