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

Move trilinos-for-albany to E3SM-Project/Trilinos? #19

Closed
xylar opened this issue Sep 29, 2023 · 5 comments · Fixed by #22
Closed

Move trilinos-for-albany to E3SM-Project/Trilinos? #19

xylar opened this issue Sep 29, 2023 · 5 comments · Fixed by #22

Comments

@xylar
Copy link
Collaborator

xylar commented Sep 29, 2023

@ikalash, what do you think about moving our trilinos-for-albany package to point to E3SM-Project/Trilinos in the spack/develop branch? This is where we can make tags for Compass, so it would be helpful to us here but it might cause you trouble. I don't know how closely the develop branch on the E3SM-Project fork tracks the develop branch on the main repo, or what the procedure is for updating it.

@bartgol, same question to you, what do you think? What could the issues be?

@bartgol
Copy link

bartgol commented Sep 29, 2023

I guess I answered this in #20. Bottom line: we can add some automation that keeps e3sm-project fork up-to-date with main trilinos, if that's necessary.

@ikalash
Copy link

ikalash commented Sep 29, 2023

Sorry for my non-responsiveness. I had to work on something else that's time critical this morning, and am just now getting caught up on these threads.

Just to make sure I understood: are you suggesting to use Trilinos from this repo: https://github.com/E3SM-Project/trilinos instead of from https://github.com/sandialabs/Trilinos? The only issue with this is if changes are made to Albany that go hand-in-hand with Trilinos VOTD in the main Trilinos repo. Then this will break the spack Albany build. I have no idea about the syncing process b/w the E3SM-Project and sandialabs Trilinos or who is responsible for that... I actually didn't even know until just now that there was an E3SM Trilinos fork!

I was going to offer to do this but I think you created a PR. I will take a look.

@bartgol
Copy link

bartgol commented Sep 29, 2023

I added a github action to our trilinos fork, which updates our develop branch nightly. I could increase the frequency, if that's needed, but I think in most cases one update/day will be enough (I can't make the action trigger whenever trilinos/develop is updated, since the action cannot be triggered by an event on another repo).

See action log here:

https://github.com/E3SM-Project/Trilinos/actions/runs/6357080101

After tonight, I will be able to verify that the periodicity of the action works as expected.

Note: the action can be triggered manually from here (see "Run workflow" on the top right), so if an update is really necessary during the day, anyone with write privileges can manually run the action.

@bartgol
Copy link

bartgol commented Sep 29, 2023

@xylar Given the above, I think it's safe to switch to the e3sm-project/trilinos fork.

@xylar
Copy link
Collaborator Author

xylar commented Oct 3, 2023

I can confirm that there is a solution for getting compass-* tags from https://github.com/E3SM-Project/Trilinos and still getting the rest from https://github.com/sandialabs/Trilinos in #22, so no need to move to the E3SM-Project url for all builds.

@xylar xylar closed this as completed in #22 Oct 5, 2023
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 a pull request may close this issue.

3 participants