Differential D8181 Diff 29548 site-modules/profile/templates/swh/deploy/scrubber/swh-scrubber-checker-postgres@.service.erb
Changeset View
Changeset View
Standalone View
Standalone View
site-modules/profile/templates/swh/deploy/scrubber/swh-scrubber-checker-postgres@.service.erb
Show All 11 Lines | |||||
<%- end -%> | <%- end -%> | ||||
<%- if !@sentry_environment.nil? and !@sentry_environment.empty? -%> | <%- if !@sentry_environment.nil? and !@sentry_environment.empty? -%> | ||||
Environment=SWH_SENTRY_ENVIRONMENT=<%= @sentry_environment %> | Environment=SWH_SENTRY_ENVIRONMENT=<%= @sentry_environment %> | ||||
<%- end -%> | <%- end -%> | ||||
<%- if !@sentry_swh_package.nil? and !@sentry_swh_package.empty? -%> | <%- if !@sentry_swh_package.nil? and !@sentry_swh_package.empty? -%> | ||||
Environment=SWH_MAIN_PACKAGE=<%= @sentry_swh_package %> | Environment=SWH_MAIN_PACKAGE=<%= @sentry_swh_package %> | ||||
<%- end -%> | <%- end -%> | ||||
Environment=SWH_SCRUBBER_CLI_EXTRA_ARGS="--help" | Environment=SWH_SCRUBBER_CLI_EXTRA_ARGS="--help" | ||||
Environment=SWH_CONFIG_FILENAME= | |||||
User=<%= @user %> | User=<%= @user %> | ||||
Group=<%= @group %> | Group=<%= @group %> | ||||
Type=simple | Type=simple | ||||
ExecStart=/usr/bin/swh scrubber --config-file <%= @config_file %> check storage $SWH_SCRUBBER_CLI_EXTRA_ARGS | ExecStart=/usr/bin/swh scrubber check storage $SWH_SCRUBBER_CLI_EXTRA_ARGS | ||||
olasd: Doesn't the --config-file already implicitly use SWH_CONFIG_FILENAME? | |||||
Done Inline Actionsyes, i think so. i'll check. ardumont: yes, i think so. i'll check. | |||||
Done Inline ActionsIt should be fine without it indeed [1] [1] https://forge.softwareheritage.org/source/swh-scrubber/browse/master/swh/scrubber/cli.py$68-70 ardumont: It should be fine without it indeed [1]
[1] https://forge.softwareheritage.org/source/swh… | |||||
[Install] | [Install] | ||||
WantedBy=multi-user.target | WantedBy=multi-user.target |
Doesn't the --config-file already implicitly use SWH_CONFIG_FILENAME?