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

Update RadMon transfer scripts to work on hera #93

Merged

Conversation

EdwardSafford-NOAA
Copy link
Collaborator

RadMon file transfer scripts were updated to perform file transfers from hera, if user has password-free access to the web server (emcrzdm).

The transfer.sh script was also made a bit smarter. It now:

  1. Checks to see if there is a copy of the html and other web services files in $TANKimg (source directory) and syncs those files with the corresponding server directory. Any new changes in $TANKimg are not overwritten though.
  2. Creates the proper directory on the server if it doesn't exist (necessary for initial installation).
  3. Syncs the $TANKimg directory and pngs sub-directory with the corresponding directory on the server.

Testing has been completed on hera and wcoss2.

Completes #92

Update transfer to work on hera.
Rm comments.
Add queue plots from hera, update documentation.
Maybe fix the intel ci error.
Copy link
Collaborator

@DavidHuber-NOAA DavidHuber-NOAA 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!

@ADCollard
Copy link
Collaborator

@EdwardSafford-NOAA Sorry was halfway through testing this when Hera went down. It looks good though.

@EdwardSafford-NOAA
Copy link
Collaborator Author

CI test issues continue on intel. The problem is with netcdf-c.4.7.4 installation. This has been an ongoing issue for some months. When it initially occurred setting the ubuntu version to 20.04 instead of "latest" did allow the test to run to completion. However the test began failing again shortly thereafter. I've set the ubuntu version back to 'latest' and for now we're going to have to live with a failed CI test.

@EdwardSafford-NOAA EdwardSafford-NOAA merged commit 835998a into NOAA-EMC:develop Jul 18, 2023
3 of 4 checks passed
@EdwardSafford-NOAA EdwardSafford-NOAA deleted the feature/fix-xfer-92 branch July 18, 2023 16:18
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

Successfully merging this pull request may close these issues.

3 participants