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: ApplicationTag - tag predefined applications #414

Merged
merged 1 commit into from
Feb 15, 2022

Conversation

btorresgil
Copy link
Member

Description

Adds support for applying user-defined tags on pre-defined applications

Motivation and Context

We supported pre-defined tags and applications, and user-defined tags and applications. Now we support user-defined tags on pre-defined applications!!

Well... I'm excited.

How Has This Been Tested?

Tested on PAN-OS 9.1

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.

@btorresgil btorresgil force-pushed the feature/application-tag branch from 50515d8 to 90bcad8 Compare February 5, 2022 00:06
@btorresgil btorresgil force-pushed the feature/application-tag branch 2 times, most recently from f7e4f30 to d9a7935 Compare February 14, 2022 21:42
@btorresgil btorresgil marked this pull request as draft February 14, 2022 21:45
@btorresgil btorresgil force-pushed the feature/application-tag branch from d9a7935 to fdb374d Compare February 14, 2022 21:46
@btorresgil btorresgil marked this pull request as ready for review February 14, 2022 21:46
@btorresgil btorresgil requested a review from shinmog February 14, 2022 21:46
Copy link
Collaborator

@shinmog shinmog left a comment

Choose a reason for hiding this comment

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

lgtm

@shinmog shinmog merged commit cfee6e3 into develop Feb 15, 2022
@shinmog shinmog deleted the feature/application-tag branch February 15, 2022 04:29
alexortize pushed a commit to alexortize/pan-os-python that referenced this pull request Mar 16, 2022
github-actions bot pushed a commit that referenced this pull request Mar 29, 2022
## [1.7.0](v1.6.0...v1.7.0) (2022-03-29)

### Features

*  Add Prisma Access support ([#419](#419)) ([f93416e](f93416e)), closes [#406](#406) [#413](#413) [#396](#396)
* Add decryption rule target and negate_target ([#404](#404)) ([88a2903](88a2903))
* Add hierarchy retrieval info ([#366](#366)) ([a290b31](a290b31))
* Add hit count support for Panorama / device group rulebases ([#429](#429)) ([057822a](057822a)), closes [#388](#388)
* **panos.panorama.Panorama:** Add `refresh_shared_objects()` ([#436](#436)) ([11327a4](11327a4)), closes [#428](#428)
* Add alternative quoting for `op()` ([#367](#367)) ([b8d0379](b8d0379)), closes [#234](#234)
* Add Cloud Services plugin support ([683f3b5](683f3b5))
* Add Local User Database objects ([#385](#385)) ([d39b6de](d39b6de))
* Add objects.ApplicationOverride ([#405](#405)) ([98db86a](98db86a))
* Add policies.AuthenticationRule ([#415](#415)) ([0d70d3f](0d70d3f))
* ApplicationTag - tag predefined applications ([#414](#414)) ([cfee6e3](cfee6e3))
* Make opstate more discoverable for users ([#413](#413)) ([f4f08d0](f4f08d0)), closes [#396](#396)

### Bug Fixes

* **panos.objects.ApplicationFilter:** Fix tag path  ([#381](#381)) ([a0991f8](a0991f8))
* `delete_similar()` now breaks delete calls into chunks ([#435](#435)) ([4333377](4333377))
* Fixes `refresh()` for attrib style params ([#395](#395)) ([0b47a3a](0b47a3a)), closes [#392](#392)
* Modify show_highavailability_state method xpath for Panorama ([#430](#430)) ([37ed2ca](37ed2ca))
* **AggregateInterface:** Correct lacp_passive_pre_negotiation xpath ([6a67320](6a67320)), closes [#406](#406)
@github-actions
Copy link

🎉 This PR is included in version 1.7.0 🎉

The release is available on PyPI 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.

2 participants