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 #3439

Closed

Conversation

tsatam
Copy link
Collaborator

@tsatam tsatam commented Mar 6, 2024

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.

@tsatam tsatam force-pushed the ARO-5088/python-multistage-containerfile branch from b993ac4 to 5b64c4d Compare March 6, 2024 22:27
Dockerfile.aro-azext Outdated Show resolved Hide resolved
Dockerfile.aro-azext Outdated Show resolved Hide resolved
Copy link

Please rebase pull request.

@github-actions github-actions bot added needs-rebase branch needs a rebase and removed ready-for-review labels Mar 20, 2024
@tsatam tsatam force-pushed the ARO-5088/python-multistage-containerfile branch from b043347 to 0ea1d01 Compare March 20, 2024 13:38
@github-actions github-actions bot removed the needs-rebase branch needs a rebase label Mar 20, 2024
@tsatam tsatam added ready-for-review and removed hold Hold labels Mar 20, 2024
@tsatam
Copy link
Collaborator Author

tsatam commented Mar 20, 2024

/azp run

Copy link

Azure Pipelines successfully started running 2 pipeline(s).

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.

While I think we'd all prefer for the style/lint/etc to be included, I agree with your assessment that we can explore options in subsequent PRs since it seems like a lot of additional work. This is a great MVP for us to iterate on. Thanks Tanmay!

@SudoBrendan
Copy link
Collaborator

/azp run e2e

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@github-actions github-actions bot added needs-rebase branch needs a rebase and removed ready-for-review labels Mar 25, 2024
Copy link

Please rebase pull request.

@SudoBrendan
Copy link
Collaborator

/azp run e2e

Copy link

Pull request contains merge conflicts.

@tsatam
Copy link
Collaborator Author

tsatam commented Apr 1, 2024

Rebased and opened replacement PR w/ in-repo branch for CI checks: #3490

@tsatam tsatam closed this Apr 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
loki needs-rebase branch needs a rebase
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants