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.

Sign up for GitHub

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

Jump to bottom

[Bug] ntnx_ndb_time_machines_info not fetching all attributes when name is used for fetching #418

Closed
bhati-pradeep opened this issue Oct 16, 2023 · 1 comment · Fixed by #419
Closed

[Bug] ntnx_ndb_time_machines_info not fetching all attributes when name is used for fetching #418

bhati-pradeep opened this issue Oct 16, 2023 · 1 comment · Fixed by #419
Assignees
@premkarat @alaa-bish @Gevorg-Khachatryan-97
Labels
1.9.1 Bug fixes & Improvements bug Something isn't working ndb

Comments

@bhati-pradeep
Copy link
Collaborator

Describe the bug
ntnx_ndb_time_machines_info not fetching all attributes when name is used for fetching. It only gives basic info. When UUID is used it gives all attributes like metadata, sla info, snapshot schedules, etc.

To Reproduce

- name: get ndb time machines using it's name
  ntnx_ndb_time_machines_info:
    name: "TM1"
    filters:
      detailed: true

Expected behavior
It should give all attributes.
@bhati-pradeep bhati-pradeep added the bug Something isn't working label Oct 16, 2023
@bhati-pradeep bhati-pradeep added 1.9.1 Bug fixes & Improvements ndb labels Oct 16, 2023
@bhati-pradeep bhati-pradeep linked a pull request Oct 16, 2023 that will close this issue
Fix #391 and #418 #419
Merged
abhimutant pushed a commit that referenced this issue Oct 25, 2023
@bhati-pradeep
Fix #391 and #418 (#419)
0054cc9 
* 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
@bhati-pradeep bhati-pradeep mentioned this issue Nov 9, 2023
Merged
premkarat pushed a commit that referenced this issue Nov 9, 2023
@bhati-pradeep @Gevorg-Khachatryan-97
@alaa-bish @Gianlu @Nathaniel-Roberts
Release/1.9.1 (#428)
b6dc17a 
* 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]>
@bhati-pradeep
Copy link
Collaborator Author

Fixed in release v1.9.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@premkarat premkarat

@alaa-bish alaa-bish

@Gevorg-Khachatryan-97 Gevorg-Khachatryan-97

Labels
1.9.1 Bug fixes & Improvements bug Something isn't working ndb
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #391 and #418 nutanix/nutanix.ansible
4 participants
@premkarat @bhati-pradeep @alaa-bish @Gevorg-Khachatryan-97