Hide Forgot
Description of problem: Seems like we do not allow intervals with year and month values without a start time, so when I try updating interval to 'P1M' it errors out with 'Imporperly formatted schedule: P1.0M' which is a bit misleading. Log shows: 2011-09-09 12:27:31,110 [ERROR][MainThread] main() @ command.py:223 - error: (500, "PulpException: 'Imporperly formatted schedule: P1.0M'", 'Traceback (most recent call last):\n File "/home/skarmark/git/pulp/src/pulp/server/webservices/controllers/decorators.py", line 48, in report_error\n return method(self, *args, **kwargs)\n File "/home/skarmark/git/pulp/src/pulp/server/webservices/controllers/decorators.py", line 127, in _auth_decorator\n value = method(self, *args, **kwargs)\n File "/home/skarmark/git/pulp/src/pulp/server/webservices/controllers/repositories.py", line 330, in PUT\n repo = api.update(id, delta)\n File "/home/skarmark/git/pulp/src/pulp/server/api/repo.py", line 674, in update\n update_repo_schedule(repo, value)\n File "/home/skarmark/git/pulp/src/pulp/server/api/scheduled_sync.py", line 173, in update_repo_schedule\n repo[\'sync_schedule\'] = validate_schedule(new_schedule)\n File "/home/skarmark/git/pulp/src/pulp/server/api/scheduled_sync.py", line 52, in validate_schedule\n interval, start, runs = dateutils.parse_iso8601_interval(schedule)\n File "/home/skarmark/git/pulp/src/pulp/common/dateutils.py", line 219, in parse_iso8601_interval\n raise isodate.ISO8601Error(\'Intervals with year and month values are not valid without a start time\')') 2011-09-09 12:27:54,717 [ERROR][MainThread] main() @ command.py:223 - error: (500, "PulpException: 'Imporperly formatted schedule: P1.0M'", 'Traceback (most recent call last):\n File "/home/skarmark/git/pulp/src/pulp/server/webservices/controllers/decorators.py", line 48, in report_error\n return method(self, *args, **kwargs)\n File "/home/skarmark/git/pulp/src/pulp/server/webservices/controllers/decorators.py", line 127, in _auth_decorator\n value = method(self, *args, **kwargs)\n File "/home/skarmark/git/pulp/src/pulp/server/webservices/controllers/repositories.py", line 330, in PUT\n repo = api.update(id, delta)\n File "/home/skarmark/git/pulp/src/pulp/server/api/repo.py", line 674, in update\n update_repo_schedule(repo, value)\n File "/home/skarmark/git/pulp/src/pulp/server/api/scheduled_sync.py", line 173, in update_repo_schedule\n repo[\'sync_schedule\'] = validate_schedule(new_schedule)\n File "/home/skarmark/git/pulp/src/pulp/server/api/scheduled_sync.py", line 52, in validate_schedule\n interval, start, runs = dateutils.parse_iso8601_interval(schedule)\n File "/home/skarmark/git/pulp/src/pulp/common/dateutils.py", line 219, in parse_iso8601_interval\n raise isodate.ISO8601Error(\'Intervals with year and month values are not valid without a start time\')') We should show this error in cli as well as it will be hard for user to figure out what is wrong with the interval as it is right according to iso8601. How reproducible: $ sudo pulp-admin repo update --id=pulp --interval P1M error: operation failed: PulpException: 'Imporperly formatted schedule: P1.0M' Expected results: $ sudo pulp-admin repo update --id=pulp --interval P1M error: operation failed: PulpException: 'Intervals with year and month values are not valid without a start time'
build: 0.238
fails_qa [root@preethi ~]# rpm -q pulp pulp-0.0.240-1.fc15.noarch [root@preethi ~]# [root@preethi ~]# [root@preethi ~]# pulp-admin repo update --id=f14 --interval P1M error: operation failed: PulpException: Improperly formatted schedule: P1.0M [root@preethi ~]#
pulp-admin repo update no longer sets the sync schedule, closing as not a bug as there's no such thing as "no longer a bug"