Bug #17601

[deployment][arvados-formula] webshell needs a cron for login-sync

Added by Javier Bértoli 6 months ago. Updated 5 months ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Deployment
Target version:
Start date:
05/28/2021
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
-
Release relationship:
Auto

Description

The formula is adding the login-sync gem, but not a cron to run it every two minutes. We need to add

# SALT_CRON_IDENTIFIER:arvados_login_sync
*/2 * * * * /usr/local/bin/arvados-login-sync

Shell node needs to be registered to API to run the script, so that needs to be fixed before this ticket.


Subtasks

Task #17645: Review #17607 register-shell-nodesResolvedJavier Bértoli

History

#1 Updated by Peter Amstutz 6 months ago

#2 Updated by Javier Bértoli 6 months ago

  • Status changed from New to In Progress

#3 Updated by Javier Bértoli 6 months ago

  • Blocked by Feature #17607: [deployment][provision][arvados-formula] register shell nodes with Arvados added

#4 Updated by Javier Bértoli 6 months ago

  • Status changed from In Progress to New
  • Description updated (diff)

#5 Updated by Peter Amstutz 5 months ago

  • Target version changed from 2021-05-12 sprint to 2021-05-26 sprint

#6 Updated by Peter Amstutz 5 months ago

  • Release set to 38

#7 Updated by Javier Bértoli 5 months ago

  • Status changed from New to In Progress

Added an example of some crude orchestration to get a shell node's scoped token and vm uuid and add them in a cron for arvados-login-sync.

As these tasks require orchestration (api server must be running before these queries being possible), I added them as examples, as they're useless on a single node configuration (which is the scope of a salt formula)

commit 4040230@arvados-formula, branch 17607-register-shell-nodes.

Need testing

#8 Updated by Peter Amstutz 5 months ago

  • Target version changed from 2021-05-26 sprint to 2021-06-09 sprint

#9 Updated by Javier Bértoli 5 months ago

  • Status changed from In Progress to Feedback

Some more fixes (squashed, sorry. Habit :| )
The cron is provided as an example state.

commit:e6dace0|arvados-formula, branch 17607-register-shell-nodes

#10 Updated by Lucas Di Pentima 5 months ago

  • File test/salt/states/examples/arvados/shell/cron/add-login-sync.sls - Line 50: The grep execution on the unless clause of the example relies on the cluster prefix to be fixme instead of just a 5 alphanum string.
  • Is there a way to properly test these examples (via jenkins, or manually)? I feel that just reading the code isn't enough to catch issues.

The rest LGTM

#11 Updated by Javier Bértoli 5 months ago

#12 Updated by Javier Bértoli 5 months ago

Lucas Di Pentima wrote:

  • File test/salt/states/examples/arvados/shell/cron/add-login-sync.sls - Line 50: The grep execution on the unless clause of the example relies on the cluster prefix to be fixme instead of just a 5 alphanum string.

fixed commit:b47321d|arvados-formula

  • Is there a way to properly test these examples (via jenkins, or manually)? I feel that just reading the code isn't enough to catch issues.

Not yet. These changes require orchestration, which I haven't set up yet in jenkins

#13 Updated by Lucas Di Pentima 5 months ago

LGTM, thanks!

#14 Updated by Javier Bértoli 5 months ago

  • Blocked by deleted (Feature #17607: [deployment][provision][arvados-formula] register shell nodes with Arvados)

#15 Updated by Javier Bértoli 5 months ago

  • Status changed from Feedback to Resolved

Merged

Also available in: Atom PDF