Skip to content

add(upgrade-os): upgrade to hbsd14 - #13

Open
ELDiablO59152 wants to merge 11 commits into
devfrom
upgrade_to_hbsd14
Open

add(upgrade-os): upgrade to hbsd14#13
ELDiablO59152 wants to merge 11 commits into
devfrom
upgrade_to_hbsd14

Conversation

@ELDiablO59152

Copy link
Copy Markdown
Member

Added

  • [UPGRADE-OS] Upgrade script to update to HBSD14

@frikilax frikilax left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think upgrade-os-to-14 and upgrade-os-to-15 could be merged in one single upgrade-major command, with the ability to give the selected major version (or not ? maybe not necessary, unless it's easier to ask the user for next major instead of computing/guessing it).
Also: couldn't this upgrade-major function use most of the already defined helper functions in regular upgrade-pkg and upgrade-os ?

Comment thread usr/local/bin/vlt-admin
upgrade-pkg Upgrade the packages.
upgrade-os-to-14 Migrate system to HardenedBSD 14.
upgrade-os-to-15 Migrate system to HardenedBSD 15.
upgrade-mongodb Upgrade MongoDB version

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

No relation to the PR
Maybe better suited in the vulture-mongodb package

Comment on lines +296 to +297
if /sbin/zfs destroy "${_zpool}/${_dataset}@${_snap_to_delete}" || \
/sbin/zfs destroy "${_zpool}/ROOT/$(get_current_BE)/${_dataset}@${_snap_to_delete}"; then

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Prefer checking which dataset exist and show it to the user on removal

if zfs_dataset_exists "${dataset}"; then
/sbin/zfs snap "${_zpool}/${dataset}@${_snapshot_name}"
/sbin/zfs snap "${_zpool}/${dataset}@${_snapshot_name}" || \
/sbin/zfs snap "${_zpool}/ROOT/$(get_current_BE)/${dataset}@${_snapshot_name}"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

...or maybe make a helper function to get the correct fullpath of a dataset, depending on ZFS structure (seeing as several lines were updated that way)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants