theforeman/foremanctl

Review dynflow-sidekiq exec command

Offen

#72 geöffnet am 29.01.2025

Auf GitHub ansehenContributor Guide
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Python (38 Forks)auto 404
foremangood first issue

Repository-Metriken

Stars
 (25 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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?

Contributor Guide

Research-Richtung
Vergleiche den Befehl in foreman quadlet mit der upstream systemd Service Datei. Bewerte, ob die Umgebungsvariable DYNFLOW SIDEKIQ SCRIPT und die hartcodierte Rails Umgebung notwendig sind. Erwäge, ein Wrapper Skript in foremanctl zu erstellen, das Standardwerte enthält.
Tech Stack
pythonshell
Domain
backenddevops
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70