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

Ensure that remote-dev works with containers created by Jib #24004

Merged
merged 1 commit into from
Feb 28, 2022

Conversation

geoand
Copy link
Contributor

@geoand geoand commented Feb 28, 2022

The issue without this is that Jib overwrites the owner
of the working directory and sets it to root.
With this fix, we add a faux layer that simply
reverts the owner back to the default when the
default base image is used.

Originally discussed at: https://quarkusio.zulipchat.com/#narrow/stream/187038-dev/topic/remote.20dev.20in.202.2E7/near/273449615

The issue without this is that Jib overwrites the owner
of the working directory and sets it to root.
With this fix, we add a faux layer that simply
reverts the owner back to the default when the
default base image is used.

Originally discussed at: https://quarkusio.zulipchat.com/#narrow/stream/187038-dev/topic/remote.20dev.20in.202.2E7/near/273449615
@geoand geoand added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Feb 28, 2022
@geoand geoand merged commit 64a58fb into quarkusio:main Feb 28, 2022
@quarkus-bot quarkus-bot bot added this to the 2.8 - main milestone Feb 28, 2022
@geoand geoand deleted the jib-remote-dev branch February 28, 2022 12:06
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label Feb 28, 2022
@gsmet gsmet modified the milestones: 2.8 - main, 2.7.3.Final Feb 28, 2022
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