Azure mass archive#53
Open
Erica Neininger (ericaneininger) wants to merge 7 commits into
Open
Conversation
22 tasks
Co-authored-by: Jon Seddon <17068361+jonseddon@users.noreply.github.com>
Collaborator
Author
|
OFFICIAL
Thanks Jon,
I've created the branch pp_gitrepo on your suite.
I've also added your retrieve_massr change to my branch, so you can test both at the same time.
I've run it as far as successfully installing the postproc app, so it should be good to go for you. The set up is a bit quick and dirty, just before you go copying this across lots of other suites !
You can run from a branch by checking out your suite, and then running the command "fcm switch pp_git_repo".
Please be aware, that if you haven't used a suite pointing at the Git mirrors before, you will need to add the following to ~/.gitconfig on the relevant EX machine -
>>>>>
[url "cazcron2:/data/users/gitassist/git_mirrors/"]
insteadOf = localmirrors:
[safe]
directory = /data/users/gitassist/git_mirrors/*
[user]
email = ***@***.***
name = Jon Rostron
<<<<<<<
There are instructions for setting Git up properly ; locally here (but I think you'll get way with just the above):
For info, particularly for Pierre 🙂 - here's the branch diff link on trac:
https://code.metoffice.gov.uk/trac/roses-u/changeset?reponame=&new=355415%40d%2Fx%2F4%2F1%2F3%2Fpp_gitrepo&old=352940%40d%2Fx%2F4%2F1%2F3%2Ftrunk
Thanks for testing this for me. Any issues, let me know.
Erica
________________________________
From: Jon Seddon ***@***.***>
Sent: Tuesday, May 12, 2026 15:28
To: MetOffice/moci ***@***.***>
Cc: Erica Pearce ***@***.***>; Assign ***@***.***>
Subject: Re: [MetOffice/moci] Azure mass archive (PR #53)
@jonseddon commented on this pull request.
________________________________
In rose-meta/archive_and_meaning/postproc/HEAD/rose-meta.conf<#53 (comment)>:
=
=This option is fixed for the lifetime of a set and CANNOT be
=changed retrospectively.
⬇️ Suggested change
- =
- =This option is fixed for the lifetime of a set and CANNOT be
- =changed retrospectively.
There's a new chres command and so the risk appetite can now be changed if required.
—
Reply to this email directly, view it on GitHub<#53 (review)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AZVSCA372FZWED3HQIIVNHD42MYHJAVCNFSM6AAAAACYRUCYCKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHM2DENZTGE2TSMZTGU>.
Triage notifications on the go with GitHub Mobile for iOS<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you were assigned.Message ID: ***@***.***>
OFFICIAL
|
Jon Seddon (jonseddon)
approved these changes
May 20, 2026
Contributor
|
Tested with Az-MASS in t-dx413/run2 and z-dx413/run1 for both values of risk appetite. Tested against moomin with m-dx413/run2 and confirmed that the changes don't affect the usage of moomin. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR Summary
Code Reviewer: Pierre Siddall (@Pierre-siddall)
Update PostProc app to permit archive to the new Azure MASS system.
closes:#51
Code Quality Checklist
(Some checks are automatically carried out via the CI pipeline)
readability of the code
Testing
acceptable (eg. kgo changes)
tests, unit tests, etc.)
rose-stem will only test the unit tests, as it does not push to MASS
I've also run a copy of the MOCI GAL test suite: u-dx886 with this branch
Test Suite Results - moci - TEST_moci53/run1
Suite Information
Task Information
✅ succeeded tasks - 99
Security Considerations
Performance Impact
performance measurements have been conducted
AI Assistance and Attribution
of Generative AI tool name (e.g., Met Office Github Copilot Enterprise,
Github Copilot Personal, ChatGPT GPT-4, etc) and I have followed the
Simulation Systems AI policy
(including attribution labels)
Documentation
confirmed that it builds correctly
Code Review