theforeman/foremanctl

Review dynflow-sidekiq exec command

Aperta

#72 aperta il 29 gen 2025

Vedi su GitHubGuida contributor
 (2 commenti) (0 reazioni) (0 assegnatari)Python (38 fork)auto 404
foremangood first issue

Metriche repository

Star
 (25 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Based on https://github.com/evgeni/talks/pull/4#discussion_r1932083989 we should review the command:

https://github.com/theforeman/foreman-quadlet/blob/177348fc149e74a4adb6680cb239181f16cf8476/roles/foreman/tasks/main.yaml#L90

This copies the DYNFLOW_SIDEKIQ_SCRIPT (that's also set as an env var) and hardcodes the Rails environment, but do we even need to expose this in the first place? It's based on https://github.com/theforeman/foreman/blob/74ddd8851cebd21c268002e0c82a3619d2a9db66/extras/systemd/dynflow-sidekiq%40.service#L12-L20 but is it better to provide a better /usr/libexec/foreman/sidekiq-selinux wrapper script that has these defaults built in?

Guida contributor

Direzione di ricerca
Confronta il comando in foreman quadlet con il file di servizio systemd upstream. Valuta se la variabile d'ambiente DYNFLOW SIDEKIQ SCRIPT e l'ambiente Rails hardcoded siano necessari. Considera la creazione di uno script wrapper in foremanctl che includa i default.
Tech stack
pythonshell
Dominio
backenddevops
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70