Skip to content

Change Travis to PG 11 #1691

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

Merged
merged 1 commit into from
Mar 23, 2019
Merged

Change Travis to PG 11 #1691

merged 1 commit into from
Mar 23, 2019

Conversation

sgrif
Copy link
Contributor

@sgrif sgrif commented Mar 21, 2019

Been testing against 11 locally for a while, but figured we should get a
travis run on this

We're running PG 11 in production now, so we should test against the
same version in CI. I had to put us on the newer xenial image to get
this to work, since trusty leaves 9.2 running if it doesn't recognize
the newer version you're trying to run, which caused issues.

There's no built-in cookbook for 11 yet, so I've had to install it from
apt and copy some additional minimal configuration to get things to
work.
@sgrif
Copy link
Contributor Author

sgrif commented Mar 22, 2019

Alright, finally got this configured correctly. This is ready for review.

@sgrif sgrif requested a review from jtgeibel March 22, 2019 20:48
@jtgeibel
Copy link
Member

The changes and build logs look good to me.

@bors r+

@bors
Copy link
Contributor

bors commented Mar 23, 2019

📌 Commit a2c7a9c has been approved by jtgeibel

@bors
Copy link
Contributor

bors commented Mar 23, 2019

⌛ Testing commit a2c7a9c with merge 7f07091...

bors added a commit that referenced this pull request Mar 23, 2019
Change Travis to PG 11

Been testing against 11 locally for a while, but figured we should get a
travis run on this
@bors
Copy link
Contributor

bors commented Mar 23, 2019

☀️ Test successful - checks-travis
Approved by: jtgeibel
Pushing 7f07091 to master...

@bors bors merged commit a2c7a9c into rust-lang:master Mar 23, 2019
@sgrif sgrif deleted the sg-pg-11 branch March 25, 2019 21:25
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