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

Add format support to nvme WipeDisk #142

Merged
merged 1 commit into from
May 17, 2024

Conversation

mmlb
Copy link
Member

@mmlb mmlb commented May 7, 2024

What does this PR do

Adds support for wiping nvme disks using nvme format for those devices that do not support sanitize.

@mmlb
Copy link
Member Author

mmlb commented May 7, 2024

depends on #136

@joelrebel

This comment was marked as outdated.

@mmlb mmlb force-pushed the nvme-format branch 2 times, most recently from 5665654 to 5a7144b Compare May 9, 2024 21:03
@mmlb mmlb requested a review from DoctorVin as a code owner May 9, 2024 21:03
@mmlb mmlb force-pushed the nvme-format branch 3 times, most recently from 9fd74ae to fbc982b Compare May 10, 2024 20:16
@mmlb mmlb changed the title utils/nvme: Add support for sanitize based WipeDisk Add format support to nvme WipeDisk May 10, 2024
@mmlb mmlb force-pushed the nvme-format branch 3 times, most recently from 6b8af3f to 3b665d1 Compare May 13, 2024 20:00
Copy link
Collaborator

@turegano-equinix turegano-equinix left a comment

Choose a reason for hiding this comment

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

@mmlb
Copy link
Member Author

mmlb commented May 16, 2024

we need to check if err is nil here

#142 (files)

fixed

@turegano-equinix turegano-equinix merged commit 514c594 into metal-toolbox:main May 17, 2024
5 checks passed
@mmlb mmlb deleted the nvme-format branch May 17, 2024 14:54
mmlb added a commit to metal-toolbox/vogelkop that referenced this pull request Jun 14, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
|
[github.com/metal-toolbox/ironlib](https://togithub.com/metal-toolbox/ironlib)
| `v0.2.18-0.20240611133518-3514176030a4` -> `v0.2.18` |
[![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fmetal-toolbox%2fironlib/v0.2.18?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fmetal-toolbox%2fironlib/v0.2.18?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fmetal-toolbox%2fironlib/v0.2.18-0.20240611133518-3514176030a4/v0.2.18?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fmetal-toolbox%2fironlib/v0.2.18-0.20240611133518-3514176030a4/v0.2.18?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>metal-toolbox/ironlib
(github.com/metal-toolbox/ironlib)</summary>

###
[`v0.2.18`](https://togithub.com/metal-toolbox/ironlib/releases/tag/v0.2.18)

[Compare
Source](https://togithub.com/metal-toolbox/ironlib/compare/v0.2.17...v0.2.18)

#### What's Changed

- Vc/instrument firmware by
[@&#8203;DoctorVin](https://togithub.com/DoctorVin) in
[metal-toolbox/ironlib#123
- Dockerfile: add support to build non-dist image by
[@&#8203;joelrebel](https://togithub.com/joelrebel) in
[metal-toolbox/ironlib#124
- Update actions/checkout action to v4 by
[@&#8203;renovate](https://togithub.com/renovate) in
[metal-toolbox/ironlib#112
- House/spring cleaning by [@&#8203;mmlb](https://togithub.com/mmlb) in
[metal-toolbox/ironlib#128
- ironlib is able to fill a disk with all zeros by
[@&#8203;turegano-equinix](https://togithub.com/turegano-equinix) in
[metal-toolbox/ironlib#131
- ironlib is able to detect ineffective wipes by
[@&#8203;turegano-equinix](https://togithub.com/turegano-equinix) in
[metal-toolbox/ironlib#135
- House Cleaning Part 2 - Electric Boogaloo! by
[@&#8203;mmlb](https://togithub.com/mmlb) in
[metal-toolbox/ironlib#133
- Better nvme capability detection by
[@&#8203;mmlb](https://togithub.com/mmlb) in
[metal-toolbox/ironlib#129
- Quiet down tests by [@&#8203;mmlb](https://togithub.com/mmlb) in
[metal-toolbox/ironlib#137
- More refactors, fixes, etc by
[@&#8203;mmlb](https://togithub.com/mmlb) in
[metal-toolbox/ironlib#141
- chore(deps): update docker/login-action action to v3 by
[@&#8203;renovate](https://togithub.com/renovate) in
[metal-toolbox/ironlib#118
- chore(deps): update docker/build-push-action action to v5 by
[@&#8203;renovate](https://togithub.com/renovate) in
[metal-toolbox/ironlib#113
- fix(deps): update module github.com/r3labs/diff/v2 to v3 by
[@&#8203;renovate](https://togithub.com/renovate) in
[metal-toolbox/ironlib#108
- chore(deps): update docker/metadata-action action to v5 by
[@&#8203;renovate](https://togithub.com/renovate) in
[metal-toolbox/ironlib#144
- chore(deps): update actions/setup-go action to v5 by
[@&#8203;renovate](https://togithub.com/renovate) in
[metal-toolbox/ironlib#143
- chore(deps): update module google.golang.org/protobuf to v1.33.0
\[security] by [@&#8203;renovate](https://togithub.com/renovate) in
[metal-toolbox/ironlib#126
- fix(deps): update module golang.org/x/net to v0.23.0 \[security] by
[@&#8203;renovate](https://togithub.com/renovate) in
[metal-toolbox/ironlib#116
- chore(deps): update docker/setup-buildx-action action to v3 by
[@&#8203;renovate](https://togithub.com/renovate) in
[metal-toolbox/ironlib#146
- fix(deps): update module github.com/sirupsen/logrus to v1.9.3 by
[@&#8203;renovate](https://togithub.com/renovate) in
[metal-toolbox/ironlib#100
- chore(deps): update anchore/sbom-action action to v0.15.11 by
[@&#8203;renovate](https://togithub.com/renovate) in
[metal-toolbox/ironlib#99
- fix(deps): update module github.com/tidwall/gjson to v1.17.1 by
[@&#8203;renovate](https://togithub.com/renovate) in
[metal-toolbox/ironlib#109
- fix(deps): update module golang.org/x/net to v0.24.0 by
[@&#8203;renovate](https://togithub.com/renovate) in
[metal-toolbox/ironlib#145
- chore(deps): update golangci/golangci-lint-action action to v5 by
[@&#8203;renovate](https://togithub.com/renovate) in
[metal-toolbox/ironlib#147
- fix(deps): update module github.com/stretchr/testify to v1.9.0 by
[@&#8203;renovate](https://togithub.com/renovate) in
[metal-toolbox/ironlib#101
- fix(deps): update module github.com/beevik/etree to v1.3.0 by
[@&#8203;renovate](https://togithub.com/renovate) in
[metal-toolbox/ironlib#97
- fix(deps): update module golang.org/x/net to v0.25.0 by
[@&#8203;renovate](https://togithub.com/renovate) in
[metal-toolbox/ironlib#148
- chore(deps): update github/codeql-action action to v3 by
[@&#8203;renovate](https://togithub.com/renovate) in
[metal-toolbox/ironlib#149
- Update CODEOWNERS by
[@&#8203;DoctorVin](https://togithub.com/DoctorVin) in
[metal-toolbox/ironlib#152
- Update modules that renovate is having trouble with by
[@&#8203;mmlb](https://togithub.com/mmlb) in
[metal-toolbox/ironlib#150
- chore(deps): update golangci/golangci-lint-action action to v6 by
[@&#8203;renovate](https://togithub.com/renovate) in
[metal-toolbox/ironlib#153
- Add DiskWipe support to nvme using sanitize by
[@&#8203;mmlb](https://togithub.com/mmlb) in
[metal-toolbox/ironlib#136
- Add format support to nvme WipeDisk by
[@&#8203;mmlb](https://togithub.com/mmlb) in
[metal-toolbox/ironlib#142
- Add ns delete/create support to nvme WipeDisk by
[@&#8203;mmlb](https://togithub.com/mmlb) in
[metal-toolbox/ironlib#154
- More clean ups by [@&#8203;mmlb](https://togithub.com/mmlb) in
[metal-toolbox/ironlib#158
- Add basic support for blkdiscard by
[@&#8203;ScottGarman](https://togithub.com/ScottGarman) in
[metal-toolbox/ironlib#159
- Some more clean ups by [@&#8203;mmlb](https://togithub.com/mmlb) in
[metal-toolbox/ironlib#161
- Rename Disk stuff -> Drive and teach diskwipe example more tricks by
[@&#8203;mmlb](https://togithub.com/mmlb) in
[metal-toolbox/ironlib#163

#### New Contributors

- [@&#8203;turegano-equinix](https://togithub.com/turegano-equinix) made
their first contribution in
[metal-toolbox/ironlib#131
- [@&#8203;ScottGarman](https://togithub.com/ScottGarman) made their
first contribution in
[metal-toolbox/ironlib#159

**Full Changelog**:
metal-toolbox/ironlib@v0.2.17...v0.2.18

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/metal-toolbox/vogelkop).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zOTMuMCIsInVwZGF0ZWRJblZlciI6IjM3LjM5My4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119-->
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.

3 participants