Skip to content
This repository was archived by the owner on Sep 16, 2020. It is now read-only.

Allow managing certain instance properties#692

Merged
AlanCoding merged 1 commit intoansible:masterfrom
AlanCoding:instance_management
Apr 10, 2019
Merged

Allow managing certain instance properties#692
AlanCoding merged 1 commit intoansible:masterfrom
AlanCoding:instance_management

Conversation

@AlanCoding
Copy link
Member

demo:

$ tower-cli instance modify 5 --enabled=true -v
*** DETAILS: Getting existing record. *****************************************
*** DETAILS: Getting the record. **********************************************
GET https://localhost:8043/api/v2/instances/5/
Params: []

*** DETAILS: Writing the record. **********************************************
PATCH https://localhost:8043/api/v2/instances/5/
Data: {'enabled': True}

Resource changed.
== ======== ======== ================= =================== ======= ================= 
id hostname capacity consumed_capacity capacity_adjustment enabled managed_by_policy 
== ======== ======== ================= =================== ======= ================= 
 5 awx_3          39                 0 1.00                   true              true
== ======== ======== ================= =================== ======= ================= 

This will require a corresponding AWX fix... linking shortly.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 64.409% when pulling 5ea8dcd on AlanCoding:instance_management into ca3f0b1 on ansible:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 64.409% when pulling 5ea8dcd on AlanCoding:instance_management into ca3f0b1 on ansible:master.

@AlanCoding AlanCoding added this to the release_3.3.4 milestone Apr 9, 2019
@AlanCoding AlanCoding merged commit cb8f768 into ansible:master Apr 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

feature_parity feature parity

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants