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

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

rocketguedes
Copy link

@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
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