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.

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

Pipeline controller cloning step procedure #272

Closed
valeriano-manassero opened this issue Dec 30, 2020 · 3 comments
Closed

Pipeline controller cloning step procedure #272

valeriano-manassero opened this issue Dec 30, 2020 · 3 comments
Labels
bug Something isn't working

Comments

@valeriano-manassero
Copy link

Hi,
during some tests I noticed Pipeline controller cloning steps is taking in consideration archived task also.

This results in cloned tasks that are already in archived state.
Moreover it drives to a lot of confusions with warnings since archived steps may be more than one

WARNING - Additionally found task

I think archived task should not be used at all from a Pipeline controller.

wdyt?

@bmartinn
Copy link
Member

bmartinn commented Dec 31, 2020

Hi @valeriano-manassero
The pipeline itself is using the Task.get_task if a task ID is not provided (i.e. looking based on task project/name).
The Task.get_task will sort all the tasks based on task updated field, but and will leave out the archived tasks, so if we are selecting the first one (i.e. the most updated one), and there are more than one, we get a warning message that there are others that match the query.

It seems that stumbled upon a bug, due to some refactoring the archived were not filtered out when requested with Task.get_tasks, and this is the root cause of the warning (the correct Task was selected nonetheless the archived ones should not appear hence there should not be any warning)

BTW: regardless I would recommend to use the Task id not the Task name (which you can always change). To get the Task ID just press the ID button next to the Task name on the right info panel :)

@bmartinn bmartinn added the bug Something isn't working label Dec 31, 2020
@bmartinn
Copy link
Member

@valeriano-manassero the bug has been fixed in v0.17.4

pip install clearml==0.17.4

@valeriano-manassero
Copy link
Author

I can confirm this is now ok.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants