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.

Sign up for GitHub

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

Jump to bottom

NCO does not allow _vrfy functions for cp, cd, mv, ln, rm, and mkdir #1021

Closed
chan-hoo opened this issue Feb 15, 2024 · 1 comment · Fixed by #1074
Closed

NCO does not allow _vrfy functions for cp, cd, mv, ln, rm, and mkdir #1021

chan-hoo opened this issue Feb 15, 2024 · 1 comment · Fixed by #1074
Assignees
@MichaelLueken
Labels
NCO EE2 compliance For EE2 compliance to meet NCO implementation standards

Comments

@chan-hoo
Copy link
Collaborator

Description

NCO does not allow cp_vrfy, cd_vrfy, mv_vrfy, ln_vrfy, rm_vrfy, and mkdir_vrfy in production

Solution

Remove _vrfy from the basic unix commands cp, cd, mv, ln, rm, and mkdir.
@chan-hoo chan-hoo added the NCO EE2 compliance For EE2 compliance to meet NCO implementation standards label Feb 15, 2024
@mkavulich
Copy link
Collaborator

See also #861, another good reason to remove these functions.

@MichaelLueken MichaelLueken self-assigned this Apr 18, 2024
@MichaelLueken MichaelLueken mentioned this issue Apr 19, 2024
Merged
16 tasks
MichaelLueken added a commit that referenced this issue Apr 30, 2024
@MichaelLueken
[develop] Update weather model hash and remove "_vrfy" from bash comm…
eea4c29 
…ands (#1074)

The weather model hash has been updated to 4f32a4b (April 15).

Additionally, _vrfy has been removed from the cd, cp, ln, mkdir, mv, and rm bash commands in jobs, scripts, ush, and ush/bash_utils. The modified commands don't function as intended (issue #861) and aren't accepted by NCO (issue #1021).
@MichaelLueken MichaelLueken mentioned this issue Apr 30, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MichaelLueken MichaelLueken

Labels
NCO EE2 compliance For EE2 compliance to meet NCO implementation standards
Projects
None yet
Milestone
No milestone
3 participants
@mkavulich @chan-hoo @MichaelLueken