Fix: Syntax Errors in Create App Security Rule Example #394
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
bhati-pradeep
approved these changes
Oct 30, 2023
|
@Nathaniel-Roberts Thank you for your valuable contribution ! |
premkarat
pushed a commit
that referenced
this pull request
Nov 9, 2023
* example fix (#420) Co-authored-by: Gevorg-Khachatryaan <Gevorg1050.1> * Add cluster URL to response only when there is cluster creation (#402) * Create cluster url when it is required * Minor fix * fiql string in filtering (#423) * fiql string in filtering * fix * Add Tests * example for filter string --------- Co-authored-by: Gevorg-Khachatryaan <Gevorg1050.1> Co-authored-by: Alaa Bishtawi <[email protected]> * Bug/issue#383 (#411) * Fix Jekyll Build * fix * Fix #391 and #418 (#419) * Create cluster url when it is required * Minor fix * Change api url for fetching time machine using name to get complete info same as uuid * Add attribute in clone modules in ndb for creation or refresh using latestSnapshot * Minor typo fix * Minor typo fix * functionality to update owner of vms (#412) * functionality to update owner of vm * sanity fix * add test --------- Co-authored-by: Gevorg-Khachatryaan <Gevorg1050.1> Co-authored-by: Alaa Bishtawi <[email protected]> * example for vms inventory (#410) Co-authored-by: Gevorg-Khachatryaan <Gevorg1050.1> * fix for custom filter and examples (#417) * fix for custom filter and examples * black fix * sanity fix * Add test --------- Co-authored-by: Gevorg-Khachatryaan <Gevorg1050.1> Co-authored-by: Alaa Bishtawi <[email protected]> * example for vms inventory (#426) * example for vms inventory * ansible cfg * Move cfg to inventory example folder * Minor name changes * Minor fix --------- Co-authored-by: Gevorg-Khachatryaan <Gevorg1050.1> Co-authored-by: Pradeepsingh Bhati <[email protected]> * Add support for PC Categories (#405) Co-authored-by: Gianluca Salvo <[email protected]> * Bug/issue#400 (#403) * fix issue #400 * fix issue #400 --------- Co-authored-by: Gevorg-Khachatryaan <Gevorg1050.1> * example for VMs with guest customization file which injects variables from the playbook (#396) Co-authored-by: Gevorg-Khachatryaan <Gevorg1050.1> * Fix: Syntax Errors in Create App Security Rule Ex (#394) * functionality to handle issue when size of new disk is absent (#393) Co-authored-by: Gevorg-Khachatryaan <Gevorg1050.1> * example fix (#392) Co-authored-by: Gevorg-Khachatryaan <Gevorg1050.1> * sanity fix * fix for issue#360 (#408) * fix for issue#360 * fix --------- Co-authored-by: Gevorg-Khachatryaan <Gevorg1050.1> * improve clone examples (#407) * imporve clone examples * Fix Examples * sanity fix * disbale tests * Enhance Acp Tests * fix disks issue when size gb is not given for clone image disks * Set release version 1.9.1 * enable image tests * enable foundation sanity test * Minor foundation test fix * Enable pc tests * Enable karbon and foundation sanity * disables tests * enables tests * enable pc tests * add tests for clones with latest snapshot. And enable clones and NDB database sanity tests * skip protection rules tests * Add random naming for acp test with user reference * Bug/191 fix database sanity test (#430) * Revert "Bug/issue#383 (#411)" This reverts commit d1a0016. * Fix Properties map * Skip jelkyll check on database sanity test build issue --------- Co-authored-by: Pradeepsingh Bhati <[email protected]> * fix ntnx_database_sanity * Add Contributaiong (#431) * Add Contributaiong * enhance contributaion * fix readme * Add comments to sample var files for simplicity * Minor typo --------- Co-authored-by: Pradeepsingh Bhati <[email protected]> * Release Notes and changelog changes * Minor typo fix * remove unrequired file * Minor release bug list change due to non cherry picks --------- Co-authored-by: Gevorg Khachatryan <[email protected]> Co-authored-by: Alaa Bishtawi <[email protected]> Co-authored-by: Gianluca Salvo <[email protected]> Co-authored-by: Gianluca Salvo <[email protected]> Co-authored-by: Nathaniel Roberts <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I tried to use this example in an Ansible Playbook and the syntax doesn't work, it also doesn't reflect the documentation directly above it. So made it work with correct syntax and tested with Nutanix Prism Central.