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

Tooling container $PATH is overridden #22878

Closed
l0rd opened this issue Mar 14, 2024 · 3 comments
Closed

Tooling container $PATH is overridden #22878

l0rd opened this issue Mar 14, 2024 · 3 comments
Assignees
Labels
area/editor/vscode Issues related to the Code OSS editor of Che kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system. team/B This team is responsible for the Web Terminal, the DevWorkspace Operator and the IDEs.

Comments

@l0rd
Copy link
Contributor

l0rd commented Mar 14, 2024

Describe the bug

The container quay.io/mloriedo/cloud-dev-images:golang has the following $PATH:

/go/bin:/usr/local/go/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin

but after using it as a tooling container of a Che workspace the $PATH becomes:

/checode/checode-linux-libc/ubi9/bin/remote-cli:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games

The consequence is kind of drastic as I cannot use go:

$ go version
bash: go: command not found

Che version

next (development version)

Steps to reproduce

https://che-dogfooding.apps.che-dev.x6e0.p1.openshiftapps.com/#https://github.com/l0rd/cloud-dev-images?image=quay.io/mloriedo/cloud-dev-images:openjdk&che-editor=che-incubator/che-code/insiders

Expected behavior

The folders in the original $PATH should still be in the workspace $PATH.

Runtime

OpenShift

Screenshots

No response

Installation method

OperatorHub

Environment

Amazon

Eclipse Che Logs

No response

Additional context

No response

@l0rd l0rd added the kind/bug Outline of a bug - must adhere to the bug report template. label Mar 14, 2024
@l0rd l0rd changed the title Tooling contaienr PATH environment variable is overridden Tooling contaienr $PATH is overridden Mar 14, 2024
@l0rd l0rd changed the title Tooling contaienr $PATH is overridden Tooling container $PATH is overridden Mar 14, 2024
@RomanNikitenko RomanNikitenko added the severity/P1 Has a major impact to usage or development of the system. label Mar 15, 2024
@RomanNikitenko
Copy link
Member

@ibuziuk
I don't know if it's related to VS Code side or operator area, but anyway we should take it for an investigation.

@RomanNikitenko
Copy link
Member

I did the following:

  • started Che-Code in quay.io/mloriedo/cloud-dev-images:golang container using port-forwarding (so - no dashboard, no operator)
  • then checked PATH

it was overridden:

image

@RomanNikitenko RomanNikitenko added team/B This team is responsible for the Web Terminal, the DevWorkspace Operator and the IDEs. area/editor/vscode Issues related to the Code OSS editor of Che labels Mar 15, 2024
@RomanNikitenko
Copy link
Member

btw:
PATH has correct value when I create a terminal using Terminal => New Terminal (select a Container)

image

@RomanNikitenko RomanNikitenko added status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. and removed status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. labels May 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/editor/vscode Issues related to the Code OSS editor of Che kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system. team/B This team is responsible for the Web Terminal, the DevWorkspace Operator and the IDEs.
Projects
Status: ✅ Done
Development

No branches or pull requests

2 participants