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 omdb expunge-testing fudge-serial <SLED_ID> #5637

Closed
wants to merge 1 commit into from

Conversation

jgallagher
Copy link
Contributor

This subcommand should unblock looped sled add/expunge testing on madrid. After expunging, run this command to overwrite the serial numbers of the now-expunged sled, allowing it to be added again with its actual serial number.

This is obviously gross and should be removed once we solve #5625, which requires transitioning sleds to the decommissioned state.

There are some very primitive checks in place. Trying to fudge the serials for a sled that isn't expunged fails:

root@oxz_switch:~# ./omdb -w db expunge-testing fudge-serial 7bd6b802-e2d6-48bb-a53c-a7ee31f6e216
Error: serial number fudging only supported for expunged sleds; sled 7bd6b802-e2d6-48bb-a53c-a7ee31f6e216 has policy InService { provision_policy: Provisionable }

And trying to fudge the serials of the same sled twice fails:

root@oxz_switch:~# ./omdb -w db expunge-testing fudge-serial ac4a39a0-3dc5-4fc8-a7ad-5217af1b8f3d
Error: sled serial number (omdb-fudged-g2-0-1714162798) already appears fudged

Copy link
Contributor

@andrewjstone andrewjstone left a comment

Choose a reason for hiding this comment

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

Just gross enough to work!

@jgallagher
Copy link
Contributor Author

Closing this in favor of #5733 which should allow us to test this without this workaround

@jgallagher jgallagher closed this May 14, 2024
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.

None yet

2 participants