Skip to content

Proof-of-concept: update to nightly images to include sourcegraph/sourcegraph@e151326#882

Draft
pjlast wants to merge 2 commits into
mainfrom
pjlast/nightly-images
Draft

Proof-of-concept: update to nightly images to include sourcegraph/sourcegraph@e151326#882
pjlast wants to merge 2 commits into
mainfrom
pjlast/nightly-images

Conversation

@pjlast
Copy link
Copy Markdown
Contributor

@pjlast pjlast commented Dec 2, 2022

Draft PR so that changes can be reviewed. It is not intended to be merged.

I did a docker-compose up locally to ensure that all the images are being pulled correctly.

This is for a custom docker-compose.yml that points to images built on this commit: https://github.com/sourcegraph/sourcegraph/commit/623cd1da89b3837b9b7df2772d3823ba740ec30a

The 623cd1d part can be used to verify that the correct images are being used.

This is to unblock a trial for a customer that needs the fix to OIDC with TLS that is fixed by this commit, see this Slack thread: https://sourcegraph.slack.com/archives/C032Z79NZQC/p1669917233230879
And this thread: https://sourcegraph.slack.com/archives/C03D4H7UBEV/p1669830116016469

@mrnugget
Copy link
Copy Markdown
Contributor

mrnugget commented Dec 2, 2022

Looks good to me!

@mrnugget mrnugget changed the title Update to nightly images Proof-of-concept: update to nightly images to include sourcegraph/sourcegraph@e151326 Dec 2, 2022
Copy link
Copy Markdown

@kopancek kopancek left a comment

Choose a reason for hiding this comment

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

I see some version changes, looks good to me. That's pretty much all I know about docker-compose 🤷

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