HECC NAS platforms, take da-utils out, again#96
Conversation
* Implement matrix strategy for coding norms * same for clone * fix syntax
|
There is a CI-workflow test for cloning cmake and oops. It uses If I take the https://github.com/GEOS-ESM/jedi_bundle/actions/runs/19304740441/job/55208905177?pr=96
So I'm confused how our Tier2 tests are running... |
Well, sort of. The machine pleiades will go away, but I think the I believe you can only get to the Rome and Cascade Lake nodes on Aitken from the
Nope! NAS is PBS so you need to use |
I believe it's because the swell tier2 tests are run under the gmao_ci user on discover, which does have git credentials, whereas these are run somewhere else |
We might be able to add a GitHub secret for this? Maybe have the |
This would be good, do we need SI admin privilieges? Having both repos in the |
|
I pushed up a change that I think works: I added the It's a bit confusing. This: seems bad but this: and then: seem good? |
|
Well, our clone logic is failing us again.. Looks like INFO JediBundle: Branch of jedicmake will be cloned from @mathomp4, we floated the idea of using |
|
@Dooruk Even mepo would have the same issue. It's still running git clone underneath! But I'm going to try a few more things. This should be possible! |
|
@Dooruk Okay. Update. I can duplicate what the CI is now seeing. It sees: and on discover, I see: It's like something is missing(?) with |
|
@mathomp4 are you are saying you can't clone I was able to it with this branch just now with these modules loaded: |
|
@Dooruk One issue might be that it seems to be |
Good thought, they are defined here properly: jedi_bundle/src/jedi_bundle/config/bundles/build-order.yaml Lines 1 to 7 in 5a54019 Seems like a git-credential issue.. I tried two gpt suggestions but couldn't figure it out and I can't test more without blowing up your email boxes with notifications. Is this the |
|
Sorry if this is not relevant: |
|
Turns out we may end up having to using a combination of https://github.com/JCSDA-internal/crtm/issues/456#issuecomment-3590446987 So while it is odd that we were not able to resolve the CI-workflow issue it is becoming more imperative to use |
Note that using a mix is fine in any case (mepo or not). It's just bad to use |
|
I had to make some changes to this PR and it deviated from the origin. I hope to use mepo for cloning in near future.. Shows two check are pending, hope they go through soon. Lot of hoping.. |

Introduces
aitkenandpleiades(this one will get decommisioned soon).Takesunfortunately we couldn't figure this outJCSDAout from defaultbuild.yaml.da-utilscan't be installed withiodaconv, so for now let's take it out.