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.

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

Fix 1865455 juju show-action-status not listing all #11297

Merged
merged 1 commit into from Mar 10, 2020
Merged

Conversation

@hpidcock
Copy link
Member

hpidcock commented Mar 9, 2020

Description of change

When passed an empty prefix/id, juju show-action-status
now has the old behaviour where it would list all actions.

QA steps

  • bootstrap
  • deploy charm with actions
  • run several actions
  • juju show-action-status
  • should return all actions

Documentation changes

N/A

Bug reference

https://bugs.launchpad.net/juju/+bug/1865455

When passed an empty prefix/id, juju show-action-status
now has the old behaviour where it would list all actions.
@hpidcock hpidcock requested a review from wallyworld Mar 9, 2020
@hpidcock

This comment has been minimized.

Copy link
Member Author

hpidcock commented Mar 9, 2020

!!build!!

@hpidcock

This comment has been minimized.

Copy link
Member Author

hpidcock commented Mar 10, 2020

$$merge$$

@jujubot jujubot merged commit a17f8b6 into juju:2.7 Mar 10, 2020
4 of 5 checks passed
4 of 5 checks passed
Client Tests (ubuntu-latest) Client Tests (ubuntu-latest)
Details
Lint
Details
Schema
Details
merge-multi-juju Build started for merge commit.
Details
check-multi-juju Build finished.
Details
jujubot added a commit that referenced this pull request Mar 13, 2020
#11314

## Description of change

Merge 2.7 bringing in these PRs:

#11286 speed up juju status by handling units better
#11296 fix LP:1860083 machine availability zone
#11291 various refactorings (method renames, tools finder/getter handle k8s broker)
#11297 juju show-action-status not listing all
#11300 k8s controllers can start lxd containers
#11304 protect against nil profile from the charm
#11310 speed up processing of machines in juju status
#11306 ensure k8s juju upgrades select correct agent version
#11313 fix watcher race in ControllerAddressSuite
#11302 fix LP:1866658 jujud called in makefile before it is built

## QA steps

Run unit tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.