Skip to content

GEOPY-2809: Python env for Analyst on current packages from develop#194

Merged
sebhmg merged 11 commits into
mainfrom
GEOPY-2809
Jun 26, 2026
Merged

GEOPY-2809: Python env for Analyst on current packages from develop#194
sebhmg merged 11 commits into
mainfrom
GEOPY-2809

Conversation

@sebhmg

@sebhmg sebhmg commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

GEOPY-2809 - Python env for Analyst on current packages from develop
for https://github.com/MiraGeoscience/geopy-distrib/pull/89

@sebhmg sebhmg requested review from RomFloreani and Copilot June 18, 2026 21:08
@github-actions github-actions Bot changed the title Geopy 2809 GEOPY-2809: Python env for Analyst on current packages from develop Jun 18, 2026
@sebhmg sebhmg requested a review from andrewg-mira June 18, 2026 21:09

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the Azure Pipelines “minimal conda env install” template to target Python 3.12 and adjust repository/environment mapping and environment-file discovery to align with updated lockfile generation/naming.

Changes:

  • Bumped the default pythonVersion parameter from 3.10 to 3.12.
  • Updated envToVirtualRepos defaults from single repo strings to lists (to support multiple virtual repos per environment).
  • Changed the environment lockfile discovery glob to a new *-py<ver>-win-64.conda.lock.yml naming convention.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .azure-pipelines/template-test-minimal-conda-env-install.yml Outdated
@sebhmg

sebhmg commented Jun 21, 2026

Copy link
Copy Markdown
Contributor Author

The pipeline collects correctly the environment files but fails on docker login. See failure at: https://dev.azure.com/mirageoscience/DevOps/_build/results?buildId=3428&view=results

Fixing this part is beyond the scope of GEOPY-2809.
Created DEVOPS-1139 to address that failure

@RomFloreani RomFloreani left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good! Just see Copilot comment on the pattern for the new file name.

Comment thread .azure-pipelines/template-test-minimal-conda-env-install.yml Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

Comment thread .docker-images/windows-servercore-ltsc2022-micromamba/Dockerfile
Comment thread .azure-pipelines/template-test-minimal-conda-env-install.yml Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@sebhmg

sebhmg commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

ran here: https://dev.azure.com/mirageoscience/DevOps/_build/results?buildId=3443&view=results
image

But it seems it did not find the env file. Must have missed something in last commits

@sebhmg

sebhmg commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

better explicit failure message if no env files are collected. See here: https://dev.azure.com/mirageoscience/DevOps/_build/results?buildId=3444
image

@sebhmg

sebhmg commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

env files now successfully collect here: https://dev.azure.com/mirageoscience/DevOps/_build/results?buildId=3445&view=artifacts&pathAsName=false&type=publishedArtifacts

Still fails on the creation of the Docker container. To be fixed by DEVOPS-1139

@sebhmg sebhmg merged commit fd00c85 into main Jun 26, 2026
11 checks passed
@sebhmg sebhmg deleted the GEOPY-2809 branch June 26, 2026 20:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants