aio-cli-plugin-runtime icon indicating copy to clipboard operation
aio-cli-plugin-runtime copied to clipboard

Updating alarms feed params does not change scheduled trigger execution

Open duynguyen opened this issue 5 years ago • 5 comments

In the manifest.yml file, you can specify schedule cron jobs with trigger feeds and rules.

actions:
    import-users:
      function: actions/import-user/index.js
      runtime: 'nodejs:12'
triggers:
  sync-users:
    feed: /whisk.system/alarms/interval
    inputs: 
      minutes: 1
rules:
  sync-users-rule:
    trigger: sync-users
    action: import-users

After that you might want to update the interval to be 3 minutes instead of 1. Then aio app deploy. After redeployed you can check the info of sync-users trigger. It shows 3 minutes correctly.

{
  "annotations": [
    {
      "key": "whisk-managed",
      "value": {
        ...
      }
    },
    {
      "key": "feed",
      "value": "/whisk.system/alarms/interval"
    }
  ],
  "limits": {},
  "name": "sync-users",
  "namespace": "my-namespace",
  "parameters": [
    {
      "key": "minutes",
      "value": 3
    }
  ],
  "publish": false,
  "rules": {
    "my-namespace/sync-users-rule": {
      "action": {
        "name": "import-users",
        "path": "my-namespace/test-app-alarms-0.0.1"
      },
      "status": "active"
    }
  },
  "updated": 1605187489818,
  "version": "0.0.6"
}

However the trigger is still fired every minute, like it was never updated.

This could be an issue with the alarms package itself, but creating this ticket for triaging purpose.

duynguyen avatar Nov 12 '20 14:11 duynguyen

Thanks for raising this up @duynguyen ! Do you know if the configuration might be cached by the alarm package and hence would need to be refreshed somewhere else?

meryllblanchet avatar Nov 12 '20 18:11 meryllblanchet

@meryllblanchet , yes that's possible. We might need some inspection in the alarms provider at Runtime level.

duynguyen avatar Nov 13 '20 13:11 duynguyen

This only affects the "alarms related params", e.g. minutes for /whisk.system/alarms/interval, or cron for /whisk.system/alarms/alarm. Other params are updated properly with aio rt trigger update command as well as aio app deploy after manifest change.

duynguyen avatar Jun 04 '21 10:06 duynguyen

Thanks for the update @duynguyen . Can you reproduce it using wsk CLI?

meryllblanchet avatar Jun 07 '21 09:06 meryllblanchet

this does works with the wsk CLI

moritzraho avatar May 04 '23 10:05 moritzraho