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.

Sign up for GitHub

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

Jump to bottom

[JUJU-250] Update juju/replicaset/v2 dependency #13525

Merged
merged 1 commit into from
Nov 26, 2021
Merged

[JUJU-250] Update juju/replicaset/v2 dependency #13525

merged 1 commit into from
Nov 26, 2021

Conversation

wallyworld
Copy link
Member

@wallyworld wallyworld commented Nov 25, 2021 •
edited
Loading

Update the juju/replicaset/v2 dependency to fix an issue updating the mongo replicaset where node addresses change.

Supersedes #13521

QA steps

# Setup lxd instances and push required keys
$ lxc launch ubuntu:18.04 t1
$ lxc launch ubuntu:18.04 t2
$ lxc launch ubuntu:18.04 t3
$ lxc file push --uid 0 --gid 0 --mode 644  ~/.local/share/juju/ssh/juju_id_rsa.pub t1/root/.ssh/authorized_keys
$ lxc file push --uid 0 --gid 0 --mode 644  ~/.local/share/juju/ssh/juju_id_rsa.pub t2/root/.ssh/authorized_keys
$ lxc file push --uid 0 --gid 0 --mode 644  ~/.local/share/juju/ssh/juju_id_rsa.pub t3/root/.ssh/authorized_keys

# Add manual cloud with the address of `t1` (lxc list t1)
$ juju add-cloud manual-test --client
Cloud Types
  lxd
  maas
  manual
  openstack
  vsphere

Select cloud type: manual

Enter the ssh connection string for controller, username@<hostname or IP> or <hostname or IP>: root@$T1_IP_ADDRESS

Cloud "manual-test" successfully added to your local client.

# Bootstrap cloud with FAN support
FAN_UNDERLAY=`ip addr show dev lxdbr0 | grep /24 | awk '{print $2}' | sed 's/1\/24/0\/24/'`
FAN_OVERLAY='252.0.0.0/16'

$ juju bootstrap manual-test manual-test --model-default container-networking-method=fan --model-default fan-config=${FAN_UNDERLAY}=${FAN_OVERLAY}

$ juju switch controller

$ lxc list | grep RUNNING
$ juju add-machine ssh:root@$T2_IP_ADDRESS
$ juju add-machine ssh:root@$T3_IP_ADDRESS

$ juju enable-ha --to 1,2
$ juju show-controller
# check "ha-enabled" sections

# Also connect to mongo and check 'rs.status()' output

Bug reference

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

@hpidcock hpidcock added the 2.9 label Nov 25, 2021
@wallyworld wallyworld mentioned this pull request Nov 25, 2021
Closed
@wallyworld
Copy link
Member Author

$$merge$$

@wallyworld wallyworld force-pushed the update-replicaset-dep branch from 1a1b110 to 80b8602 Compare November 26, 2021 01:29
@wallyworld
Copy link
Member Author

$$merge$$

@jujubot jujubot merged commit 49e55ba into juju:2.9 Nov 26, 2021
@hmlanigan hmlanigan mentioned this pull request Nov 30, 2021
Merged
jujubot added a commit that referenced this pull request Dec 3, 2021
@jujubot
Merge pull request #13537 from hmlanigan/fix-enable-ha-integration-test
6c1be85 
#13537

Found a bug in ha tear down, #13525 , however the enable-ha integration test failed to catch it multiple times.

1. destroy model can fail without the word error, check for "Unable to get the model status from the API" as well.
2. When verifying machine tear down in ha, once there are no stopped machines and 1 started, verify none are in error state.

## QA steps

```sh
(cd tests ; ./main.sh -v controller test_enable_ha)
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hpidcock hpidcock hpidcock approved these changes

Assignees
No one assigned
Labels
2.9
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@wallyworld @hpidcock @jujubot