Skip to content

Conversation

rueckstiess
Copy link
Contributor

@rueckstiess rueckstiess commented Mar 1, 2017

this backport will allow 1.6 versions to connect to atlas free tier instances.

⚠️ backport to 1.6
This PR is against the 1.6-releases branch directly. A cherry-pick from the original COMPASS-742 ticket was not possible because the actual bump of data-service was handled in COMPASS-809 already, so COMPASS-742 was a no-op in Compass itself.

This backport explicitly bumps the version of data-service, which includes collection-sample@1.5.1 which is Atlas-free-tier-aware.

this will allow 1.6 versions to connect to atlas free tier instances.
@rueckstiess rueckstiess requested a review from pzrq March 1, 2017 00:40
@rueckstiess rueckstiess changed the title COMPASS-824 backport compass 742 allow disk use COMPASS-824 backport COMPASS-742 (allowDiskUse) Mar 1, 2017
@pzrq
Copy link
Contributor

pzrq commented Mar 1, 2017

Restarting Travis as it looks like the underlying AWS S3 issues.

@pzrq
Copy link
Contributor

pzrq commented Mar 1, 2017

The bad response code looked like mongodb-runner wasn't working, that appears to have been fixed in the last few minutes so going to retry these again:

screen shot 2017-03-01 at 4 39 43 pm

EDIT: No, but for some reason I can cd runner; ./bin/mongodb-runner.js start meaning mongodb-runner can work :/

EDIT EDIT: Sometimes as the push build has now passed! :S

Copy link
Contributor

@pzrq pzrq left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Just been waiting on Travis (for half the day).

I tested this locally and on my Atlas free tier and didn't find anything amiss, not even in Create Index where we still expected we might find issues.

Found it strange not having the nice shiny query bar and the sidebar DDL buttons / instance header changes.

@durran durran merged commit 36d9373 into 1.6-releases Mar 1, 2017
@durran durran deleted the COMPASS-824-backport-COMPASS-742-allowDiskUse branch March 1, 2017 15:37
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.

3 participants