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(module/panos_edl): New module for External Dynamic Lists #512

Merged
merged 13 commits into from
Dec 12, 2023

Conversation

sebastianczech
Copy link
Contributor

Description

PR delivers new module for External Dynamic Lists.

Motivation and Context

Class Edl is available in pan-os-python, but there was no module in Ansible for External Dynamic Lists yet.

How Has This Been Tested?

Code was tested by configuring EDL in Panorama working in the cloud.

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.

Copy link
Contributor

@FoSix FoSix left a comment

Choose a reason for hiding this comment

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

👍🏻

@shinmog
Copy link
Collaborator

shinmog commented Oct 31, 2023

This requires a rebase since I fixed the previous commit messages.

@sebastianczech
Copy link
Contributor Author

I rebased it

@sebastianczech sebastianczech merged commit 1e33995 into develop Dec 12, 2023
18 checks passed
@sebastianczech sebastianczech deleted the module-panos-edl branch December 12, 2023 11:59
github-actions bot pushed a commit that referenced this pull request Dec 13, 2023
## [2.19.0](v2.18.0...v2.19.0) (2023-12-13)

### Features

* **module/panos_edl:** New module for External Dynamic Lists ([#512](#512)) ([1e33995](1e33995))
Copy link

🎉 This PR is included in version 2.19.0 🎉

The release is available on Ansible Galaxy and GitHub release

Posted by semantic-release bot

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.

3 participants