ACI HTTP Plugin

[shrsr]

This ACI plugin provides the HTTP API transport methods needed to initiate
a connection to the ACI controller, send API requests and process the
response from the controller.

[lhercot]

You also seem to have missed the aci_rest module changes.

[codecov]

Codecov Report

Patch coverage: 14.89% and project coverage change: -0.36% ⚠️

Comparison is base (6b54766) 35.95% compared to head (2dfb95b) 35.60%.

❗ Current head 2dfb95b differs from pull request most recent head 0bbd2b3. Consider uploading reports for the commit 0bbd2b3 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #114      +/-   ##
==========================================
- Coverage   35.95%   35.60%   -0.36%     
==========================================
  Files         179      180       +1     
  Lines        7995     8187     +192     
  Branches     1184     1214      +30     
==========================================
+ Hits         2875     2915      +40     
- Misses       5120     5272     +152     

Flag	Coverage Δ
sanity	35.60% <14.89%> (-0.36%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed	Coverage Δ
plugins/doc_fragments/aci.py	100.00% <ø> (ø)
plugins/modules/aci_access_span_src_group.py	37.20% <0.00%> (ø)
plugins/modules/aci_access_span_src_group_src.py	25.75% <0.00%> (ø)
plugins/modules/aci_config_rollback.py	33.33% <0.00%> (-2.61%)	⬇️
plugins/modules/aci_fabric_span_src_group_src.py	30.90% <0.00%> (ø)
plugins/modules/aci_rest.py	27.27% <0.00%> (-0.34%)	⬇️
plugins/modules/aci_config_snapshot.py	27.58% <10.00%> (-1.83%)	⬇️
plugins/module_utils/aci.py	11.41% <11.53%> (+0.57%)	⬆️
plugins/httpapi/aci.py	18.06% <18.06%> (ø)
plugins/module_utils/constants.py	100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Schrottfresse]

@shrsr Any update on this? I'm still hoping that it get's fixed someday. 😀

[shrsr]

@Schrottfresse It is still in the works but we have it in our to-do-list (near future) :)

[akinross]

LGTM

[lhercot]

LGTM