Fix the onboarding state to account for zenml login #3270
+35
−4
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Describe changes
We used to rely on the "authorized device created" event to mark the "zenml connect" step in the onboarding state.
With "zenml login" taking the place of "zenml connect" and the new login experience, this is still valid for OSS servers, but the ZenML Pro servers no longer use "authorized devices" at the zenml server level, they use "authorized devices" at the ZenML Pro control plane level.
This PR fixes this by introducing:
User-Agent
header value set for all the API requests made by the ZenML Python client (to be able to differentiate ZenML client calls from other type of calls)Pre-requisites
Please ensure you have done the following:
develop
and the open PR is targetingdevelop
. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.Types of changes