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

feat(ansible_ee): introduce Ansible Execution Environment image build #123

Conversation

FoSix
Copy link
Contributor

@FoSix FoSix commented Oct 18, 2023

Description

CI for building the Ansible EE that can be used with Ansible Tower or AWX.

Motivation and Context

Palo Ansible Collection cannot be used with the default EE shipped with AWX, as that one lacks the required Python packages. It's the same situation when one would like to use the panos-upgrade-assurance package and corresponding Ansible modules. This PR introduces a small CI workflow that grabs all requirements for both the Ansible collection and panos-upgrade-assurance and builds an Ansible Execution Environment Docker image, by extending the default EE.

Although this PR does not introduce any changes to the code itself, it's still marked as a feature release.

How Has This Been Tested?

tested on a AWX instance

Screenshots (if appropriate)

Types of changes

  • New feature (non-breaking change which adds functionality)

Checklist

  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes if appropriate.
  • All new and existing tests passed.

@FoSix FoSix linked an issue Oct 18, 2023 that may be closed by this pull request
@github-actions
Copy link

github-actions bot commented Oct 18, 2023

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
919 904 98% 95% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: f9a98f0 by action🐍

@FoSix FoSix force-pushed the 120-move-ansible-execution-env-build-to-upgrade-assurance-repo branch from 590d62f to b711085 Compare October 18, 2023 12:00
@FoSix FoSix force-pushed the 120-move-ansible-execution-env-build-to-upgrade-assurance-repo branch from b711085 to b8fa407 Compare October 18, 2023 12:02
@github-actions
Copy link

A Preview PR in PanDev repo has been created. You can view it here.

@FoSix FoSix force-pushed the 120-move-ansible-execution-env-build-to-upgrade-assurance-repo branch from 9af845b to 5239d51 Compare October 18, 2023 13:09
@FoSix FoSix force-pushed the 120-move-ansible-execution-env-build-to-upgrade-assurance-repo branch 3 times, most recently from 026b0ec to e95f296 Compare October 18, 2023 14:14
@FoSix FoSix force-pushed the 120-move-ansible-execution-env-build-to-upgrade-assurance-repo branch from e95f296 to 16d4bcc Compare October 19, 2023 07:07
@FoSix FoSix force-pushed the 120-move-ansible-execution-env-build-to-upgrade-assurance-repo branch from 001425c to 1d6b1e0 Compare October 19, 2023 07:53
@FoSix FoSix marked this pull request as ready for review October 19, 2023 08:27
@FoSix
Copy link
Contributor Author

FoSix commented Oct 19, 2023

@jamesholland-uk @shinmog - gentlemen, what do you think about this CI, should we keep it here, next to pan-os-ansible or in a completely separate repo? Any idea how this plays with RHAAP?

@FoSix
Copy link
Contributor Author

FoSix commented Nov 16, 2023

this will be moved to a separate repo

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.

prep a repo and a workflow to build panos ansible based AWX EE image
1 participant