Skip to content

Conversation

davepacheco
Copy link
Collaborator

This will be useful for being able to test Nexus handoffs manually. (I was doing this by hand before.)

Depends on #9023.

Base automatically changed from dap/write-notyet-records to main September 17, 2025 13:21
internal_dns 427ec88f-f467-42fa-9bbb-66a91a36103c install dataset in service fd00:1122:3344:2::1
internal_ntp 6444f8a5-6465-4f0b-a549-1993c113569c install dataset in service fd00:1122:3344:101::21
nexus 0c71b3b2-6ceb-4e8f-b020-b08675e83038 install dataset in service fd00:1122:3344:101::22
+ nexus 16a766ee-9400-4e67-9363-883670371a1b install dataset in service fd00:1122:3344:101::28
Copy link
Contributor

Choose a reason for hiding this comment

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

(Not in this PR) Should we have a column for the Nexus generation in this zones table? Sucks that it only applies to Nexus, but it seems like a pretty vital piece of info.

Copy link
Collaborator

Choose a reason for hiding this comment

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

If you want I could have it be a follow-up to #9034

IMO it's pretty handy to see the nexus_generation info co-located with the db_metdata_nexus info, because they're so tightly correlated

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah, I do think we should do this. I was thinking we might want to add a table to the blueprint show output for this, similar to DNS I think? Rather than this table.

I'm game to include it in the database state as well but I think it's important to be able to see it from just the blueprint state too (e.g., in reconfigurator-cli).

@davepacheco davepacheco enabled auto-merge (squash) September 18, 2025 16:56
@davepacheco davepacheco merged commit c698911 into main Sep 18, 2025
16 checks passed
@davepacheco davepacheco deleted the dap/reconfigurator-cli-bump-generation branch September 18, 2025 17:17
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