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

New default cli settings #108

Merged
merged 3 commits into from
Sep 9, 2024
Merged

New default cli settings #108

merged 3 commits into from
Sep 9, 2024

Conversation

ajfloeder
Copy link
Contributor

  • Add the deleteUnknownVolumes command line parameter, default is false to avoid accidentally removing volumes when starting nnf-ec as a command
  • Starting the HTTP server is no longer the default. You must specify -http now for that behavior.

Examples:

    ./nnf-ec
  • Initializes all PCI related items (PAXs and NVMe drives)
  • Leaves unknown NVMe namespaces alone
  • Exits after initialization
    ./nnf-ec -http -deleteUnknownVolumes
  • Provides the old default behavior where all unknown volumes are deleted after initialization and the HTTP server is started to accept redfish commands

Dockerfile Outdated
@@ -15,7 +15,7 @@
# See the License for the specific language governing permissions and
# limitations under the License.

FROM golang:1.19 as builder
FROM golang:1.19 AS builder
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can bump that to 1.21.

Copy link
Contributor

@bdevcich bdevcich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But nnf-sos will still delete unknown volumes?

Signed-off-by: Anthony Floeder <[email protected]>
@ajfloeder ajfloeder merged commit 198c529 into master Sep 9, 2024
2 checks passed
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 this pull request may close these issues.

nnf-ec default to non-destructive initialize and exit
4 participants