Skip to content

refactor: improve code quality in virt-manager hook scripts#14

Open
rocketguedes wants to merge 4 commits intobiglinux:mainfrom
rocketguedes:refactor/code-quality-improvements
Open

refactor: improve code quality in virt-manager hook scripts#14
rocketguedes wants to merge 4 commits intobiglinux:mainfrom
rocketguedes:refactor/code-quality-improvements

Conversation

@rocketguedes
Copy link
Copy Markdown

@rocketguedes rocketguedes commented Dec 27, 2024

Pequenas refatorações nos scripts para simplificar e melhorar a legibilidade, com a adição do libvirt como dependência na Action no hook, já que parte do script depende do virsh.

@gitbarnabedikartola
Copy link
Copy Markdown
Member

massa rocketguedes, adoramos receber PRs.
sobre o service do systemD eu tb prefiro usar o "--now" ou invés de "start + enable", lembro de no inicio ter usado, mas salvo engano (já faz tem que escrevi) para o "virsh net-define ...", "virsh net-star ..." e " virsh net-autostart ..." funcionarem o service precisa estar iniciado, então move-lo para depois dos comandos vai quebrar.
acho que tentei colocar o enable --now antes mas por algum motivo que já me lembro cheguei a conclusão que serara-lo em 2 etapas deu mesmo erro.

a parte do "Depends = libvirt" faz tempo que mexo com ela, vc sabe me dizer como ela se comporta?

agora um ponto apenas cosmético, em scripts tentamos (nem sempre fazemos, mas...) manter um padrão snake_case ou camelCase, mas como falei é puro preciosismo estético.

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.

2 participants