Vouch request: Bobbins228 #2365
Closed
Bobbins228
started this conversation in
Vouch Request
Replies: 2 comments
-
|
/vouch |
Beta Was this translation helpful? Give feedback.
0 replies
-
|
@Bobbins228 has been vouched by @drew. You can now submit pull requests to OpenShell. Welcome aboard. Please read CONTRIBUTING.md before submitting. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
What do you want to work on?
Hi, my name is Mark Campbell and I am a Software Engineer working at Red Hat I am also in the same team as @ChristianZaccaria , @varshaprasad96 and @rhuss . I am very passionate about containerisation, Kubernetes and AI development & deployment so OpenShell is an extremely interesting technology for me. I would like to be able to help in creating a stronger project through any means whether it be feature work, testing, docs or reviews. I have experience in Kuberentes Operator development, OpenShift deployments and deploying AI in the cloud.
To start I would like to resolve a small docs bug for which I have created the GH issue #2364
I've read through the architecture documentation, contributing guidelines, AGENTS.md and have begun to familiarise myself with the codebase.
Why this change?
Currently the VM Driver docs example shows
sandbox_uidas a configuration option you can set in a gateway TOML file. The Gateway TOML parser doesn’t accept the field and it crashes with “unknown field” if you set it.See doc ref: gateway-config.mdx
The field does exist but only in the internal subprocess that the gateway spawns. Users don’t configure that subprocess directly, the gateway passes its arguments as CLI flags so there is no way to set
sandbox_uidvia the TOML file currently.This change should hopefully prevent users from accidentally inputting
sandbox_uidwhich will prevent parser crashesChecklist
Beta Was this translation helpful? Give feedback.
All reactions