-
Notifications
You must be signed in to change notification settings - Fork 578
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
Framework: remove obsolete nightly driver scripts #11933
Comments
Motivated by #11930. |
Please keep the |
I marked apollo, artemis, kokkos-dev-2, perseus, and ride as machines to delete |
@bartlettroscoe It looks like you "own" a few of the machines in the list. Could you review and mark them as to keep or remove? |
FYI: The committers to each of these
is shown below:
|
NOTE: I marked all of the directories that I know should be kept or deleted. (It is not clear if a check means the dir should be kept or deleted.) |
@jwillenbring Could you look at the machines that you might own? |
@jhux2 I was pretty aggressive about marking things for deletion. I believe the framework has moved away from using these scripts in this way for all or nearly all of their builds. Many of these machines have not been used at all in some time. @sebrowne should review the list once for sanity as well and ensure nothing important is lost. It might be useful to look in history at some point for some starting points for future builds. That is why some of this wasn't already deleted, but I think keeping it for that reason probably causes more confusion than it is worth at this point. |
As far as I know, Framework does not use anything in this list. If it makes it through the autotester, it's fine by me. As Jim stated, we can look back in the history for information if we need it. |
Closing as fixed by #11935. |
Enhancement
@trilinos/developers
This issue is to help track clean up scripts in
Trilinos-trilinos/cmake/ctest/drivers
. If you own one of these, please indicate whether the subdirectory can be deleted. Once the machine is removed in a PR or determined to be active, it can be "checked" off.Here are the last git commits for each subdirectory:
The text was updated successfully, but these errors were encountered: