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

Minimal Python container to build az aro extension #3490

Merged
merged 4 commits into from
Jun 7, 2024

Conversation

tsatam
Copy link
Collaborator

@tsatam tsatam commented Apr 1, 2024

(This PR recreates #3439 off of in-repository branch for CI checks)

Which issue this PR addresses:

Small step for ARO-5088

What this PR does / why we need it:

  • Creates a new Makefile target to build the az aro extension with no other dependencies
  • Adds a new containerfile to create a build container for the above

Test plan for issue:

Manually invoke target and ensure it creates an aro-${VERSION}-py2.py3-none-any.whl file within ./python/az/aro/dist.

Is there any documentation that needs to be updated for this PR?

Eventually, https://msazure.visualstudio.com/AzureRedHatOpenShift/_wiki/wikis/ARO.wiki/188187/AzureCLI-preview-extension-publishing# should be updated to include this target.

Additional Notes

  • This container does not perform the verification steps performed by the azdev command (style/linter/test). azdev requires that it be run within the context of a Git repository, as well as from within a specific virtualenv.

SudoBrendan
SudoBrendan previously approved these changes Apr 1, 2024
cadenmarchese
cadenmarchese previously approved these changes Apr 1, 2024
Copy link
Collaborator

@cadenmarchese cadenmarchese left a comment

Choose a reason for hiding this comment

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

makes sense 👍

@cadenmarchese
Copy link
Collaborator

/azp run ci,e2e

Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@tsatam tsatam closed this Apr 17, 2024
@tsatam tsatam reopened this Apr 17, 2024
@github-actions github-actions bot added the needs-rebase branch needs a rebase label May 10, 2024
Copy link

Please rebase pull request.

@tsatam tsatam force-pushed the tsatam/ARO-5088-python-multistage-containerfile branch from 3b4e7bf to 70e1d7e Compare May 10, 2024 21:14
@tsatam tsatam requested a review from petrkotas as a code owner May 10, 2024 21:14
@tsatam tsatam dismissed stale reviews from cadenmarchese and SudoBrendan via c16b4f2 May 10, 2024 21:18
@tsatam tsatam force-pushed the tsatam/ARO-5088-python-multistage-containerfile branch from 70e1d7e to c16b4f2 Compare May 10, 2024 21:18
Copy link
Collaborator

@SudoBrendan SudoBrendan left a comment

Choose a reason for hiding this comment

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

Just a quick typo fix. Otherwise, builds fine!

Makefile Outdated Show resolved Hide resolved
@tsatam tsatam force-pushed the tsatam/ARO-5088-python-multistage-containerfile branch from c16b4f2 to 10d3bff Compare June 5, 2024 20:07
@github-actions github-actions bot removed the needs-rebase branch needs a rebase label Jun 5, 2024
@tsatam tsatam force-pushed the tsatam/ARO-5088-python-multistage-containerfile branch from 10d3bff to c2d1aaa Compare June 5, 2024 20:26
@hawkowl
Copy link
Collaborator

hawkowl commented Jun 6, 2024

/azp run ci, e2e

Copy link

Azure Pipelines successfully started running 2 pipeline(s).

Copy link

github-actions bot commented Jun 6, 2024

Please rebase pull request.

SudoBrendan
SudoBrendan previously approved these changes Jun 6, 2024
Copy link
Collaborator

@SudoBrendan SudoBrendan left a comment

Choose a reason for hiding this comment

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

LGTM

@tsatam tsatam force-pushed the tsatam/ARO-5088-python-multistage-containerfile branch from c2d1aaa to 257e1cd Compare June 6, 2024 21:04
@github-actions github-actions bot removed the needs-rebase branch needs a rebase label Jun 6, 2024
Copy link
Collaborator

@tony-schndr tony-schndr left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@shubhadapaithankar shubhadapaithankar left a comment

Choose a reason for hiding this comment

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

LGTM

@SudoBrendan
Copy link
Collaborator

/azp run ci

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@SudoBrendan SudoBrendan merged commit 1a7df46 into master Jun 7, 2024
19 checks passed
@SudoBrendan SudoBrendan deleted the tsatam/ARO-5088-python-multistage-containerfile branch July 24, 2024 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants