Skip to content
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

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

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

pjlast
Copy link
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
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

@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