Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Wrong cron expression of harvester planification since py3 migration #2490

Closed
mayjolux opened this issue Jun 10, 2020 · 2 comments · Fixed by #2493
Closed

Wrong cron expression of harvester planification since py3 migration #2490

mayjolux opened this issue Jun 10, 2020 · 2 comments · Fixed by #2493
Assignees
@quaxsze
Labels
harvesting migration Requires or provide a data migration regression

Comments

@mayjolux
Copy link

Concerning the harvester creation and its later planification, if you program the planification to 0 2 * * * for example under the UI of EDIT>Planification, the following is added to the cron expression:

b'0' b'2' b'*' b'*' b'*', which generates an error later on.

Seems to be related to the python3 migration. Additionally:
This does not happen if you go with the default cron expression and it does not happen if you schedule it via the "udata harvest schedule"
@abulte
Copy link
Contributor

abulte commented Jun 10, 2020

Re #2489

@abulte abulte added migration Requires or provide a data migration regression harvesting labels Jun 10, 2020
@quaxsze quaxsze mentioned this issue Jun 15, 2020
Merged
@abulte
Copy link
Contributor

abulte commented Jun 15, 2020

Fixed by #2493

@abulte abulte closed this as completed Jun 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@quaxsze quaxsze

Labels
harvesting migration Requires or provide a data migration regression
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add cast if bytes in haverster's cron schedule quaxsze/udata
3 participants
@abulte @mayjolux @quaxsze