Landscape Charm

Merge lp://qastaging/~ack/landscape-charm/amulet-actions-support into lp://qastaging/~landscape/landscape-charm/trunk

  1. amulet-actions-support
  2. Merge into trunk
Proposed by Alberto Donato
Status: Merged
Approved by: Alberto Donato
Approved revision: 382
Merged at revision: 379
Proposed branch: lp://qastaging/~ack/landscape-charm/amulet-actions-support
Merge into: lp://qastaging/~landscape/landscape-charm/trunk
Diff against target: 73 lines (+8/-27)
1 file modified
tests/helpers.py (+8/-27)
To merge this branch: bzr merge lp://qastaging/~ack/landscape-charm/amulet-actions-support
Reviewer Review Type Date Requested Status
🤖 Landscape Builder test results Approve
Simon Poirier (community) Approve
Free Ekanayaka (community) Approve
Review via email: mp+317768@code.qastaging.launchpad.net

Commit message

Use standard amulet methods for creating and fetching actions, rather than our own code.

Description of the change

Use standard amulet methods for creating and fetching actions, rather than our own code.

To post a comment you must log in.
Revision history for this message
🤖 Landscape Builder (landscape-builder) :
review: Abstain (executing tests)
Revision history for this message
🤖 Landscape Builder (landscape-builder) wrote :

Command: make ci-test
Result: Success
Revno: 380
Branch: lp:~ack/landscape-charm/amulet-actions-support
Jenkins: https://ci.lscape.net/job/latch-test-xenial/1432/

review: Approve (test results)
lp://qastaging/~ack/landscape-charm/amulet-actions-support updated
381. By Alberto Donato

Remove debug print.

Revision history for this message
🤖 Landscape Builder (landscape-builder) :
review: Abstain (executing tests)
Revision history for this message
🤖 Landscape Builder (landscape-builder) wrote :

Command: make ci-test
Result: Success
Revno: 381
Branch: lp:~ack/landscape-charm/amulet-actions-support
Jenkins: https://ci.lscape.net/job/latch-test-xenial/1433/

review: Approve (test results)
Revision history for this message
Free Ekanayaka (free.ekanayaka) wrote :

+1

review: Approve
lp://qastaging/~ack/landscape-charm/amulet-actions-support updated
382. By Alberto Donato

Fix docstring.

Revision history for this message
🤖 Landscape Builder (landscape-builder) :
review: Abstain (executing tests)
Revision history for this message
🤖 Landscape Builder (landscape-builder) wrote :

Command: make ci-test
Result: Success
Revno: 382
Branch: lp:~ack/landscape-charm/amulet-actions-support
Jenkins: https://ci.lscape.net/job/latch-test-xenial/1434/

review: Approve (test results)
Revision history for this message
Simon Poirier (simpoir) wrote :

+1

review: Approve
Revision history for this message
🤖 Landscape Builder (landscape-builder) :
review: Abstain (executing tests)
Revision history for this message
🤖 Landscape Builder (landscape-builder) wrote :

Command: make ci-test
Result: Success
Revno: 382
Branch: lp:~ack/landscape-charm/amulet-actions-support
Jenkins: https://ci.lscape.net/job/latch-test-xenial/1435/

review: Approve (test results)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches