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

Very basic trilio update test #616

Merged
merged 1 commit into from
Aug 10, 2021

Conversation

gnuoy
Copy link
Contributor

@gnuoy gnuoy commented Aug 4, 2021

Simple test that the action to update trilio completes.

Simple test that the action to update trilio completes.
@gnuoy gnuoy marked this pull request as ready for review August 5, 2021 07:44
@gnuoy
Copy link
Contributor Author

gnuoy commented Aug 5, 2021

openstack-mirroring pushed a commit to openstack/charm-trilio-wlm that referenced this pull request Aug 6, 2021
After rereading the request from Trilio I realised that the action
should be performing the package update too. So this PR removes the
post-upgrade-configuration action and replaces it with update-trilio
which performs the package upgrades too.

func-test-pr: openstack-charmers/zaza-openstack-tests#616
Change-Id: Ia2c88560b45bdb20742e254464730b5708f8b79f
openstack-mirroring pushed a commit to openstack/openstack that referenced this pull request Aug 6, 2021
* Update charm-trilio-wlm from branch 'master'
  to 6b9b1480065d26f01a5bff79d22cbf18db7f26b7
  - Replace post-upgrade-configuration with update-trilio
    
    After rereading the request from Trilio I realised that the action
    should be performing the package update too. So this PR removes the
    post-upgrade-configuration action and replaces it with update-trilio
    which performs the package upgrades too.
    
    func-test-pr: openstack-charmers/zaza-openstack-tests#616
    Change-Id: Ia2c88560b45bdb20742e254464730b5708f8b79f
Copy link
Contributor

@ajkavanagh ajkavanagh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ajkavanagh ajkavanagh merged commit 12596b4 into openstack-charmers:master Aug 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants