Skip to content

vdk-core: Add python version disparity warning#2242

Merged
doks5 merged 1 commit intomainfrom
person/andonova/control-cli-cleanup
Jun 14, 2023
Merged

vdk-core: Add python version disparity warning#2242
doks5 merged 1 commit intomainfrom
person/andonova/control-cli-cleanup

Conversation

@doks5
Copy link
Copy Markdown
Contributor

@doks5 doks5 commented Jun 13, 2023

If a user has set a specific python_version in their data job's config.ini file, but they develop the data job in a python environment with a different version, the execution results may be unexpected when the job is deployed.

This change adds a warning message for local data job executions, in case there is a disparity between the configured python version and the python environment that executes the job.

Testing Done: Locally tested with a data job, and verified the message appears when there is a python version disparity.

warning: 
warning:                         
warning:                   ********************************************************************************
warning:                         Python version (3.9), set in the job's config.ini file, is different
warning:                         from the python version (3.8) used to execute the data job.
warning:                         
warning:                   ********************************************************************************
warning:

@doks5 doks5 self-assigned this Jun 13, 2023
@doks5 doks5 force-pushed the person/andonova/control-cli-cleanup branch from 16bf236 to 8e9d09a Compare June 13, 2023 13:42
@doks5 doks5 requested a review from antoniivanov June 13, 2023 16:55
If a user has set a specific python_version in their data job's config.ini file,
but they develop the data job in a python environment with a different version,
the execution results may be unexpected when the job is deployed.

This change adds a warning message for local data job executions, in case there
is a disparity between the configured python version and the python environment
that executes the job.

Testing Done: Locally tested with a data job, and verified the message appears
when there is a python version disparity.

Signed-off-by: Andon Andonov <andonova@vmware.com>
@doks5 doks5 force-pushed the person/andonova/control-cli-cleanup branch from 8e9d09a to 310b1e4 Compare June 13, 2023 20:52
Copy link
Copy Markdown
Contributor

@gabrielgeorgiev1 gabrielgeorgiev1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks great

@doks5 doks5 merged commit fd8ec9f into main Jun 14, 2023
@doks5 doks5 deleted the person/andonova/control-cli-cleanup branch June 14, 2023 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants