-
Notifications
You must be signed in to change notification settings - Fork 2
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
Release v0.0.2 #73
Merged
Merged
Release v0.0.2 #73
Conversation
This file contains 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
Contributor
bdevcich
commented
Jun 8, 2023
- RABSW-1161: Add tag verification
- Add verification to System Configuration (systems.yaml)
- Split int-test and the unit tests so they can be enabled/disabled separately
- RABSW-1162: Remove release subcommand
- Use cert-manager v1.11.1 (Use cert-manager v1.11.1 #60)
- Initialize a KIND cluster with the same tool used for a real Rabbit (Initialize a KIND cluster with the same tool used for a real Rabbit #61)
- Add helpful Makefile targets (Add helpful Makefile targets #62)
- Install lustre-fs-operator and lustre-csi-driver directly from github (Install lustre-fs-operator and lustre-csi-driver directly from github #63)
- Allow the nnf-mfu release to be specified. (Allow the nnf-mfu release to be specified. #64)
- Handle third-party services (Handle third-party services #65)
- Improve the waitCmd for cert-manager (Improve the waitCmd for cert-manager #66)
- Update the submodules (Update the submodules #67)
- Github-63: Add command line options for config file paths
- Enable k8s audit logging in KIND (Enable k8s audit logging in KIND #69)
- Improve main.go dependency (Improve main.go dependency #70)
- Update the notes for using the k8s audit logs. (Update the notes for using the k8s audit logs. #71)
- Update submodules and remote refs
Add a github workflow to verify that tags are annotated tags. Signed-off-by: Matt Richerson <[email protected]>
RABSW-1161: Add tag verification
* Add checks for: * duplicate system names * duplicate aliases * missing overlays * missing workers * duplicate rabbit node names * duplicate compute node names (in a system) * Add supporting functions to make system config eaiser to test * Add unit tests for config.go * Add Makefile to run tests locally * Add GitHub workflow to run tests * Fixed incorrectly named worker node for htx-2 * Fixed bug in main.go that prints error output twice Signed-off-by: Blake Devcich <[email protected]>
…arately You can then manually turn off the workflows in the web UI. Signed-off-by: Blake Devcich <[email protected]>
Managing the submodule releases can be done through 'git submodule' commands. Signed-off-by: Matt Richerson <[email protected]>
RABSW-1162: Remove release subcommand
Signed-off-by: Dean Roehrich <[email protected]>
) Remove any use of the cray.nnf.manager label. Anything that was relying on that will now go to any available untainted worker. Update the submodules. Signed-off-by: Dean Roehrich <[email protected]>
Add default target that does the 'go build'. Add targets for 'go fmt' and 'go vet' for the nnf-deploy code and its tests. Fix go.mod to allow this. Fix format in tests. Signed-off-by: Dean Roehrich <[email protected]>
…#63) Use "kubectl apply -k", pointing to github, to install lustre-fs-operator and lustre-csi-driver. The local copies of these repos are not referenced. Extend the "Repository" config to allow the new information to be specified. To have nnf-deploy revert to the old way of handling these repos, change the 'useRemoteK' value to false for the given repo. Add a wait phase for cert-manager, because our NNF stuff needs the cert-manager webhook to be live when they start up. Signed-off-by: Dean Roehrich <[email protected]>
Allow the nnf-mfu release to be specified. Allow a build environment to be specified for the repositories. Add an environment variable to indicate which version of nnf-mfu should be used. Signed-off-by: Dean Roehrich <[email protected]>
Create a generic way to handle third-party services like cert-manager and mpi-operator. Add a section to repositories.yaml to handle the third-party services. Allow a "wait command" to be specified, so we can wait for the webhook on a service like cert-manager. Signed-off-by: Dean Roehrich <[email protected]>
Signed-off-by: Dean Roehrich <[email protected]>
Signed-off-by: Dean Roehrich <[email protected]>
Add command line options to specify the path to the config files for systems.yaml, daemons.yaml, and repositories.yaml. The new options are "systems", "daemons", and "repos". Signed-off-by: Matt Richerson <[email protected]>
Github-63: Add command line options for config file paths
Signed-off-by: Dean Roehrich <[email protected]>
Allow make to determine whether "nnf-deploy" is out of date with "main.go". Signed-off-by: Dean Roehrich <[email protected]>
Signed-off-by: Dean Roehrich <[email protected]>
Signed-off-by: Blake Devcich <[email protected]>
8 tasks
roehrich-hpe
approved these changes
Jun 8, 2023
Do NOT use squash or rebase. |
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.