Project

General

Profile

Bug #26283

hammer sync-plan update does not work with custom cron

Added by Samir Jha about 3 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Sync Plans
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1683096

Description of problem:

When trying to update a sync-plan created with interval set to custom cron, the command always fails with:

Could not update the sync plan:
Cron expression is not valid!

Version-Release number of selected component (if applicable):

~]# hammer --version
hammer (0.15.1)

How reproducible:

I tried many different commands against sync-plan created in web UI and using hammer.

Steps to Reproduce:
1. ~]# hammer sync-plan create --name BugTest --sync-date "2019/02/01" --enabled 1 --interval "custom cron" --cron-expression "1 2 3 4 5" --organization-id 1
2. ~]# hammer sync-plan info --name BugTest --organization-id 1

3. ~]# hammer sync-plan update --name BugTest --organization-id 1 --description Hello
Could not update the sync plan:
Cron expression is not valid!

4. ~]# hammer sync-plan update --name BugTest --organization-id 1 --description Hello --cron-expression "1 2 3 4 5"
Could not update the sync plan:
Cron expression is not valid!

Actual results:

Could not update the sync plan

Expected results:

Could update the sync plan

Associated revisions

Revision bdab6e48 (diff)
Added by Samir Jha about 3 years ago

Fixes #26283 - hammer sync-plan update

History

#1 Updated by The Foreman Bot about 3 years ago

  • Assignee set to Samir Jha
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/8004 added

#2 Updated by Justin Sherrill about 3 years ago

  • Triaged changed from No to Yes
  • Target version set to Katello 3.12.0

#3 Updated by Anonymous about 3 years ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF