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

check config before accessing it #2057

Merged
merged 1 commit into from
May 27, 2022
Merged

check config before accessing it #2057

merged 1 commit into from
May 27, 2022

Conversation

johrstrom
Copy link
Contributor

@johrstrom johrstrom commented May 26, 2022

#1950 persists in the shell app even after #1988. We missed this one check that was in mainline.

I tested this by sharing the shell app with my test user and overriding OOD_CLUSTERS with some unreadable files.

┆Issue is synchronized with this Asana task by Unito

@johrstrom johrstrom marked this pull request as ready for review May 26, 2022 14:45
@gerald-byrket
Copy link
Contributor

I'm sure this has to do with my dev setup, but when i use this branch for the shell app, i get the following.
I'm running node version 12.22.12
If i use the latest branch of the shell app, it works fine.

Error: /lib64/libstdc++.so.6: version `CXXABI_1.3.9' not found (required by /users/PZS0714/gbyrket/ondemand/ondemand_osc/apps/shell/node_modules/node-pty/build/Release/pty.node)
at Object.Module._extensions..node (internal/modules/cjs/loader.js:1057:18)
at Module.load (internal/modules/cjs/loader.js:863:32)
at Function.Module._load (internal/modules/cjs/loader.js:708:14)
at Module.require (internal/modules/cjs/loader.js:887:19)
at Module.require (/opt/ood/ondemand/root/usr/share/passenger/helper-scripts/node-loader.js:80:25)
at require (internal/modules/cjs/helpers.js:74:18)
at Object. (/users/PZS0714/gbyrket/ondemand/ondemand_osc/apps/shell/node_modules/node-pty/lib/unixTerminal.js:26:11)
at Module._compile (internal/modules/cjs/loader.js:999:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)
at Module.load (internal/modules/cjs/loader.js:863:32)

@johrstrom
Copy link
Contributor Author

I'm sure you need to module purge because you're building on a compute/login node and it executes on the web node.

Here are the full steps that should get you there. You may even want to do it in a straight up shell session because vs code is in a singularity container and that could have something to do with it here too.

git checkout fix-shell-again
rm -rf node_modules
module purge
bin/setup

@gerald-byrket
Copy link
Contributor

I'm sure you need to module purge because you're building on a compute/login node and it executes on the web node.

Here are the full steps that should get you there. You may even want to do it in a straight up shell session because vs code is in a singularity container and that could have something to do with it here too.

git checkout fix-shell-again
rm -rf node_modules
module purge
bin/setup

Thanks Jeff.

gotta remember reload ruby module. :)

module load ruby/2.7.3

Thanks @johrstrom. That did it.

Copy link
Contributor

@gerald-byrket gerald-byrket left a comment

Choose a reason for hiding this comment

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

looks good. functions properly. ready to merge.

@johrstrom johrstrom merged commit bc7792d into release_2.0 May 27, 2022
@johrstrom johrstrom deleted the fix-shell-again branch May 27, 2022 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants