Project

General

Custom queries

Profile

Actions

Bug #35414

closed

Bump sidekiq to 6.3.x

Added by Adam Ruzicka over 2 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Packaging
Target version:
-

Related issues 1 (0 open1 closed)

Has duplicate Foreman - Feature #30275: Update Sidekiq to 6.0.6 or higherDuplicateActions

Added by Adam Ruzicka over 2 years ago

Revision b0210b23 (diff)

Fixes #35414 - Bump Sidekiq to 6.3.z

As a side effect, gitlab-sidekiq-fetcher should get updated to >= 7.1.0.

Currently latest sidekiq is 6.5.5, but anything above 6.4.0 is rather
verbose about things being deprecated in sidekiq 7.0.0.

Similarly, latest redis is 4.8.0, but anything above 4.5.0 is rather
verbose about things being deprecated in redis 5.0.0. Note, those
deprecation warnings are triggered from inside of
gitlab-sidekiq-fetcher.

Added by Adam Ruzicka over 2 years ago

Revision 9b0652fe (diff)

Fixes #35414 - Drop sd_notify usage

Sidekiq should be doing this on its own behind the scenes

Added by Adam Ruzicka over 2 years ago

Revision 0ab14da1 (diff)

Refs #35414 - Fix sd_notify for multiple orchestrators

Added by Ewoud Kohl van Wijngaarden over 2 years ago

Revision ac36213f (diff)

Refs #35414 - Add sd_notify dependency to service

Puma has a soft dependency on sd_notify when systemd is used. In our
service we use this so we should make sure it's pulled in. Prior to
9b0652fed9f86fd429e237ce359422a1dba14a2a the dynflow_sidekiq group
pulled it in, which technically wasn't entirely correct but in practice
it worked. Sidekiq 6 has essentially vendored sd_notify so it was
removed.

Fixes: 9b0652fed9f86fd429e237ce359422a1dba14a2a

Added by Ewoud Kohl van Wijngaarden over 2 years ago

Revision a6cf5c63 (diff)

Refs #35414 - Log an early message on dynflow startup

When the service is marked as ready, the verification starts. When
puppet-foreman configures the service using journald logging it checks
the journal. However, the application can take a while to really start
while it's already signaled to systemd it has started up. This means
verification fails.

By logging a message before sending ready we're guaranteed that there is
something to verify.

Fixes: 0ab14da142c9539b0f8de34c32d929d2aa5de6d4

Added by Ewoud Kohl van Wijngaarden over 2 years ago

Installer - Revision 7e0b69e7 (diff)

Refs #35414 - Expect a different message in journal

In Foreman 3.5 the Dynflow startup procedure was changed. It now signals
it's ready before it's actually started up. This is because it can run
into passive mode, in case there are multiple orchestrators. This
changes the expectation so it's reliably testing it.

Actions

Also available in: Atom PDF