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

Does not detect Cheyenne with full hostname #595

Closed
BruceKropp-Raytheon opened this issue Jul 26, 2023 · 5 comments
Closed

Does not detect Cheyenne with full hostname #595

BruceKropp-Raytheon opened this issue Jul 26, 2023 · 5 comments

Comments

@BruceKropp-Raytheon
Copy link

The script ush/detect_machine.sh does not find a match for Cheyenne. It tries to match a entry using 'hostname -f', but on Cheyenne this returns something of the form chadmin3.ucar.edu . There is no pattern that matches this, so this script exits with MACHINE_ID="" .

@BruceKropp-Raytheon
Copy link
Author

Steps to reproduce (for branch 'develop' and for PR#571):

  • source /glade/scratch/epicufsrt/jenkins/workspace/sandbox/emc-gsi-build/gsi/ush/detect_machine.sh
    ++ case $(hostname -f) in
    +++ hostname -f
    ++ MACHINE_ID=
    ++ '[' = hera ']'
    /glade/scratch/epicufsrt/jenkins/workspace/sandbox/emc-gsi-build/gsi/ush/detect_machine.sh: line 43: [: =: unary operator expected
    ++ '[' = cheyenne ']'
    /glade/scratch/epicufsrt/jenkins/workspace/sandbox/emc-gsi-build/gsi/ush/detect_machine.sh: line 43: [: =: unary operator expected
  • echo MACHINE_ID=
    MACHINE_ID=

@DavidHuber-NOAA
Copy link
Collaborator

I see now that the hostname is incorrect for chadmin nodes inside detect_machine.sh. A new line should be added after line 30 of detect_machine.sh:

30 chadmin[1-6].ib0.cheyenne.ucar.edu) MACHINE_ID=cheyenne ;; ### cheyenne1-6
31 chadmin[1-6].cheyenne.ucar.edu) MACHINE_ID=cheyenne ;; ### cheyenne1-6

I don't have access to the chadmin nodes on Cheyenne. Could you test this solution?

@BruceKropp-Raytheon
Copy link
Author

platform=cheyenne
hostname=chadmin3.ucar.edu
COMPILER=intel

  • hostname
    chadmin3
  • hostname -f
    chadmin3.ucar.edu

@RussTreadon-NOAA
Copy link
Contributor

The GSI Handling Review team is going through open issues to see which, if any, may be closed. Can this issue be closed?

@DavidHuber-NOAA
Copy link
Collaborator

Yes, this was fixed and merged with #571.

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

No branches or pull requests

3 participants