Thursday, 2018-08-16

*** vnogin has quit IRC00:26
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_cinder master: zuul: Drop distro jobs for Xenial and switch to smoke tests  https://review.openstack.org/59196100:36
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_cinder master: Drop JSON logging for Cinder  https://review.openstack.org/59208500:36
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_cinder master: Fix cinder_environment_overrides  https://review.openstack.org/58993400:36
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_cinder master: Add target_helper to backend_defaults  https://review.openstack.org/58460400:36
*** gregoryo has joined #openstack-ansible00:52
*** dave-mccowan has quit IRC01:02
*** dave-mccowan has joined #openstack-ansible01:03
cloudnullevenings01:38
*** schwicht has quit IRC01:55
*** spatel has joined #openstack-ansible02:09
spatelHey folks!02:09
*** tinwood has quit IRC02:10
spatelcloudnull: I want to set MTU 9000 for br-storage02:10
spatelI am reading this https://docs.openstack.org/project-deploy-guide/openstack-ansible/newton/app-networking.html02:10
spatelI can set container_mtu: "9000" in user_confi file02:11
*** tinwood has joined #openstack-ansible02:11
spatelafter that do i need to run setup-host playbook?02:11
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-ops master: Add basic index templates  https://review.openstack.org/59224902:37
cloudnullspatel the lxc-container create playbook should be all you need to run02:37
cloudnulljust make sure that the mtu is set accordingly on the underlying interfaces02:38
spatelOkay! you mean Host machine02:38
spatelcloudnull: thanks for the hint :)02:39
spatelDo i need to set MTU 9000 on lxcbr0 interface also?02:40
spatelcloudnull: ^^02:40
cloudnullonly if you want to02:40
cloudnullthe storage traffic shouldnt travel over that interface02:41
spateli think on infra* node there is not real use for MTU900002:41
spatelyou are right!02:41
cloudnullfor simplicity, when using jumbo frames, i set mtu 9000 on all hosts for a given interface02:42
spateloh! wait in that case i don't need to do anything.. all i need to do just set MTT 9000 on compute node by hand02:42
cloudnullyea if there's no storage on the cinder-volume hosts02:42
cloudnullip link set mtu 9000 dev $interface02:42
spatelI am not using Cinder-volume02:42
cloudnulland you're good to go02:43
cloudnullthen save that in the ifcfg files so it persists post reboot02:43
spatelmy application need CPU & memory, we have no use of storage02:43
spatelyup!!02:43
cloudnullshould be an easy change :)02:43
cloudnullno need to change anything in OSA02:44
spatelglad!!02:44
spatelOne more question! what is the best way to migrate all my VMware guest VM to Openstack instance?02:45
spatelI did google and all but look like no easy way :(02:45
spatelI have 22 VMware host and 200 VMs trying to move over openstack02:45
cloudnullyou can export the vmdk, import the image into glance, and build a new vm?02:46
spatelbut in that case i will have 200 images in glance right?02:46
cloudnullyes02:47
cloudnullno good way to directly convert the vms, that im aware of02:47
spatelthat is what i found on google02:47
cloudnullyea, that wouldnt be a  great experience02:47
cloudnulli wrote this tool a long time ago02:48
cloudnullhttps://github.com/cloudnull/InstanceSync02:48
spateli am trying to avoid creating 200 images in glance with 4TB of storage for just images.. yike!!02:48
cloudnullmight need to be updated for newer OS's02:48
cloudnullbut that was used a lot when i wrote it to migrate servers to various places.02:49
cloudnullits basically fancy rsync02:49
spateloh!! i like that02:49
spatelso it does like DD ?02:49
cloudnullcreate a VM in the new place that is of the same OS then run the script02:50
cloudnullI had a version tahat did DD but it had very inconsistent results02:50
spatelit will like disk to disk copy right?02:50
cloudnullthis is literally rsync with a bunch of extra things02:50
cloudnullthe target VM will need to be the same OS with a similar size02:51
spatellet me understand your script and try that out..02:51
cloudnullits not a block level copy02:51
spatelits file level copy02:51
cloudnullthough you could do that too. https://cloudnull.io/2011/12/dd-and-the-mighty-disk-destroyer-or-duplicator/02:51
cloudnullbottom of the post02:52
cloudnulldd to gzip over ssh02:52
cloudnullassuming nothing interupts the transfer that does work well02:52
spatelhmm! interesting02:53
cloudnullthe file level copy of the instancesync script was used a lot and had much better results.02:53
cloudnullthe post is old as is the script so YMMV02:53
spateli think i got clue from your script...!02:54
spateli can live with that if it works :)02:54
cloudnulleven if you dont use the script it should get you most of the way there.02:54
cloudnullyea live copying worked02:54
cloudnullduring huricane sandy that was used to move a lot of workloads out of Datacenters in NYC02:55
spatelinteresting..02:55
cloudnullanyways that might be a better way forward, instead of doing the vmdk export02:56
spatelcloudnull: thanks man!! it was nice hint02:56
cloudnullanytime02:56
spatelno i am not going to import 200 vmdk.. that is really ugly way02:56
spatelI am going to do rsync that is safe and easy02:57
*** spatel has quit IRC03:06
*** spatel has joined #openstack-ansible03:07
spatelcloudnull: Good nite!  thanks for your help!03:07
cloudnullyou too03:08
cloudnulltake care03:08
*** spatel has quit IRC03:08
*** udesale has joined #openstack-ansible03:36
*** shyamb has joined #openstack-ansible03:40
*** Bhujay has quit IRC03:43
*** shyamb has quit IRC03:52
*** dave-mccowan has quit IRC04:04
*** poopcat has quit IRC04:04
*** mmalchuk_ has quit IRC04:18
*** shyamb has joined #openstack-ansible04:20
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-ops master: Extend the embedded bootstap process  https://review.openstack.org/59226104:27
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-ops master: Make journalbeat install detect if it should install  https://review.openstack.org/59226404:30
openstackgerritMerged openstack/openstack-ansible-ops master: Add basic index templates  https://review.openstack.org/59224904:30
*** Bhujay has joined #openstack-ansible04:38
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-ops master: Make journalbeat install detect if it should install  https://review.openstack.org/59226404:45
cloudnullthe ceph checks in the integrated repo seem to be failing here - http://logs.openstack.org/44/587544/22/check/openstack-ansible-deploy-ceph-ubuntu-xenial/cb2a689/job-output.txt.gz#_2018-08-15_18_09_01_054177 - is that something that we're aware of or know what to do to resolve ?04:50
cloudnullif we can pacify ceph, evrardjp healthcheck change will get the gate going in a seemingly more robust and healthy way04:51
cloudnullhttps://review.openstack.org/#/c/587544/04:51
* cloudnull headed to bed04:51
cloudnullbut it'd be great if folks could have a look04:51
openstackgerritMerged openstack/openstack-ansible-lxc_hosts master: Remove the checksum fills  https://review.openstack.org/59188004:53
cloudnullalso https://review.openstack.org/#/c/592153/ - that would be good to get in so we can download and read journals04:55
cloudnullin the role tests04:55
openstackgerritMerged openstack/openstack-ansible-ops master: Use the Ansible runtime venv for pw-token-gen.py  https://review.openstack.org/59205505:04
*** Bhujay has quit IRC05:07
*** Bhujay has joined #openstack-ansible05:08
openstackgerritMerged openstack/openstack-ansible-ops master: Extend the embedded bootstap process  https://review.openstack.org/59226105:14
*** mmalchuk has joined #openstack-ansible05:15
openstackgerritMerged openstack/openstack-ansible-ops master: Make journalbeat install detect if it should install  https://review.openstack.org/59226405:22
*** pcaruana has quit IRC05:26
*** shyamb has quit IRC05:43
*** shyamb has joined #openstack-ansible05:46
*** DanyC has joined #openstack-ansible05:47
*** shyamb has quit IRC06:01
*** Bhujay has quit IRC06:02
*** Bhujay has joined #openstack-ansible06:03
*** DanyC has quit IRC06:12
*** mbuil has joined #openstack-ansible06:25
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-plugins master: [DNM] - strategy: linear: Inspect the delegated host on delegated tasks.  https://review.openstack.org/59167206:32
hwoaranggood morning06:42
*** cshen has joined #openstack-ansible06:43
*** gkadam has joined #openstack-ansible06:48
*** pcaruana has joined #openstack-ansible06:48
evrardjpmorning hwoarang06:57
openstackgerritMerged openstack/openstack-ansible-galera_client master: Use include_tasks instead of include  https://review.openstack.org/58528806:57
jrossermorning07:02
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-plugins master: [DNM] - strategy: linear: Inspect the delegated host on delegated tasks.  https://review.openstack.org/59167207:07
openstackgerritMerged openstack/openstack-ansible-galera_server master: use include_tasks instead of include  https://review.openstack.org/58526707:08
*** fghaas has joined #openstack-ansible07:16
openstackgerritMerged openstack/openstack-ansible-os_heat master: use include_tasks instead of include  https://review.openstack.org/58526507:18
evrardjpjrosser: did you had a look at the networking, or filed a bug for cloudnull ?07:20
jrosserit is fixed07:21
jrosserlook at the systemd service role07:21
jrosserprivatetmp07:21
*** shardy has joined #openstack-ansible07:21
jrosserevrardjp: i will fire off a fresh aio right now and try that all from scratch again07:22
jrosseralso patch for lxcbr0 checksums should get rid of dmeag spam07:23
evrardjpjrosser: should help on the IO07:25
evrardjpjrosser: do you have that patch handy?07:25
jrossergimme 10mins just on the tram :)07:26
evrardjpjrosser: does that look like I didn't watch the repos during my day off? :p07:26
jrossernetns is this according to cloudnull https://github.com/openstack/ansible-role-systemd_service/commit/00394f42dbf8f79f5346752fdca3951020a105df07:27
evrardjpjrosser: we need to get the freeze in ASAP07:27
evrardjpelse the freeze will look like an ugly beast real fast07:28
evrardjpthinking about https://review.openstack.org/#/c/591957/1 that should get in07:28
jrosserchecksums here https://review.openstack.org/#/c/591880/07:29
evrardjpoh well I thought this merged and we had to revert it07:31
evrardjpgood thing is not what I think07:31
evrardjpit is not*07:31
evrardjpyeah I will rebuild an AIO too07:31
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-plugins master: [DNM] - strategy: linear: Inspect the delegated host on delegated tasks.  https://review.openstack.org/59167207:36
openstackgerritMerged openstack/openstack-ansible-os_horizon master: use include_tasks instead of include  https://review.openstack.org/58529907:38
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-plugins master: [DNM] - strategy: linear: Inspect the delegated host on delegated tasks.  https://review.openstack.org/59167207:45
*** olivierbourdon38 has joined #openstack-ansible07:48
openstackgerritChandan Kumar proposed openstack/openstack-ansible-os_tempest master: [WIP] Unifing tempest role for Tripleo  https://review.openstack.org/59142408:01
chandankumarevrardjp: odyssey4me mnaser ^^08:01
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/rocky: Revert "Revert "Update UPPER_CONSTRAINTS_FILE for stable/rocky""  https://review.openstack.org/59231208:04
*** keithmnemonic[m] has quit IRC08:11
*** RossSchulman[m] has quit IRC08:11
*** evrardjp[m] has quit IRC08:11
*** linshuicheng[m] has quit IRC08:11
*** flaviosr has joined #openstack-ansible08:11
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Update ansible to 2.5.7  https://review.openstack.org/59231508:17
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests master: Update ansible to 2.5.7  https://review.openstack.org/59231708:18
*** electrofelix has joined #openstack-ansible08:20
*** DanyC has joined #openstack-ansible08:26
openstackgerritMerged openstack/openstack-ansible-tests master: When archiving logs do not compress the journals  https://review.openstack.org/59215308:29
*** rpittau has joined #openstack-ansible08:30
*** gregoryo has quit IRC08:41
evrardjpchandankumar: reviewed08:42
*** ianychoi_ has quit IRC08:42
evrardjpthanks for having a go at this08:42
*** mjwales has joined #openstack-ansible08:42
mjwalesHi guys, when using an LDAP backend do you need to manually copy the tls_cacertfile into the keystone container?08:47
*** shyamb has joined #openstack-ansible08:55
evrardjpmjwales: it's better to distribute valid certs everywhere09:12
odyssey4meevrardjp: interesting error in the update of ansible: http://logs.openstack.org/15/592315/1/check/openstack-ansible-varstest-aio_lxc-ubuntu-xenial/36a7999/job-output.txt.gz#_2018-08-16_08_37_20_68630109:17
odyssey4meI think it's because the action plugin can't be found, but am not sure.09:17
chandankumarevrardjp: odyssey4me regarding python-tempestconf https://github.com/openstack/python-tempestconf we use this to manage tempest config generation automatically09:19
chandankumarrecently we have integrated with refstack-client https://github.com/openstack/refstack-client/commit/e1744ea1e4276de04c7e37399a96c506e24e9d63 for wider adoption09:19
odyssey4meok, what does it mean to 'manage tempest configuration automatically' ?09:19
chandankumarodyssey4me: it discovers the minimal values from the deployed cloud for tempest.conf automatically if admin credentails are used it create resources for the same which can be used by tempest while testing09:20
odyssey4meOK, so you're suggesting that this can replace our templating of tempest.conf and our implementation of resources?09:21
chandankumarodyssey4me: I am not suggesting to replace it , let's have the support of both09:22
odyssey4meok, sounds good - I guess it'll need some sort of task routing for that then09:22
chandankumarif user wants to use devstack provided tempest.conf generation https://github.com/openstack-dev/devstack/blob/master/lib/tempest that can use it with the role09:22
odyssey4meI suggest that you do that in a seperate patch to the one you've pushed up.09:23
chandankumarodyssey4me: since we are unifing I am thinking of reusing our role here https://github.com/openstack/tempest/tree/master/roles also09:23
chandankumarbut step by step09:24
jrosserevrardjp: you seen this? http://logs.openstack.org/57/591357/2/check/openstack-ansible-deploy-aio_nspawn-opensuse-423/2be5d60/job-output.txt.gz#_2018-08-16_08_19_40_94173709:24
odyssey4mechandankumar: interesting, what's the scope of those roles and their purpose? are they intended for use outside of zuul?09:24
chandankumarodyssey4me: currenlty those are tied to openstack zuul jobs and re-used where tempest gates are triggered with devstack09:25
odyssey4meaha09:25
odyssey4meso would you be porting what they do over - or would we consume those after they've been generalised?09:26
chandankumarodyssey4me: what i am thinking, let's make our piece better and then propose there to reuse it09:26
odyssey4mesure, we just need to bear in mind that testing inside zuul is different from running things in production09:28
odyssey4mealthough this is tempest, which kinda sites between things09:28
odyssey4meanyway - sure, anything that helps to improve and broadens the audience is good in my books09:29
chandankumarbut the role should be written in such a way it can be used anywhere09:29
chandankumarirrespective of the enviornment09:29
odyssey4meyep09:29
odyssey4mewhat I was trying to say is that some things done for zuul are very zuul specific, and we shouldn't allow the role we use to do that - it needs to be universal09:31
odyssey4meso I think we're agreeing. :)09:31
chandankumaryup09:32
evrardjpodyssey4me: yeah it seems we need to remove static:09:35
odyssey4meevrardjp: that's a warning, not the error09:36
evrardjpjrosser: not sure what should I see09:36
odyssey4methe error appears to me that the module is config_template in that task, and ansible doesn't find it09:37
evrardjpodyssey4me: yeah09:37
evrardjpI haven't changed anything in the path09:38
evrardjpdid we change things?09:38
evrardjpis that master?09:38
evrardjpwait let me check your patch09:38
odyssey4menope, this is all working with the current ansible - but the patch update fails with that09:38
odyssey4meso it seems that something changed in ansible I guess09:38
jrosserevrardjp: from that point on it all goes horribly wrong with opensuse mirrors09:38
evrardjpyeah with a "minor" update09:39
jrosseranother thing that cropped up yesterday, some of the locations looked like they had changed09:39
evrardjpodyssey4me: maybe there is something that needs to be looked over, is that it's properly trying to load it but fails to load09:39
evrardjpdue to invalid code for that version09:40
odyssey4methat's possible too09:40
odyssey4meevrardjp: interesting that https://review.openstack.org/592317 passed just fine09:41
odyssey4meso I suspect that the issue is that we do not set where the plugins should be sourced in the integrated repo09:41
evrardjpodyssey4me: maybe https://github.com/openstack/ansible-config_template/blob/master/action/config_template.py#L660-L668 could be cherry picked to plugins and adjusted to 2.5.x09:41
odyssey4mein the role tests we do09:41
evrardjpif that's the root cause09:41
evrardjpI haven't looked09:41
evrardjpodyssey4me: that could be the cause09:42
evrardjpor09:43
evrardjp/etc/ansible/roles/ceph-ansible/library is having config-template too and conflicting09:44
evrardjpI think it needs a more thorough look09:45
evrardjpI am stabbing in the dark right now09:45
*** cshen has quit IRC09:46
odyssey4meyeah, it's clearly not urgent - I'll find some time later to look into it09:47
evrardjpodyssey4me: I think it's worth bumping to a more recent version anyway09:47
odyssey4mewell, that's why I proposed it ;)09:47
evrardjpI think the most interesting thing right now, is how the hell did tempest work while the (almost similar, if not the same) ansible equivalent doesn't work09:48
evrardjpodyssey4me: I meant 2.609:48
odyssey4meAh yes, there was a patch for that which also failed spectacularly.09:49
*** olivierbourdon38 has quit IRC09:52
*** olivierbourdon38 has joined #openstack-ansible09:53
evrardjpthat's why I think we need to cherry pick the 2.6 support into plugins, or (better!) move to use the ansible-config_template repo09:53
*** shyambiradar has joined #openstack-ansible09:57
noonedeadpunkhi everyone09:58
noonedeadpunkevrardjp: thanks for review! odyssey4me: can you also take a look at https://review.openstack.org/#/c/586318/ one more time?09:58
evrardjpnoonedeadpunk: sorry for the trouble I gave you. I think it's the best way forward to do this09:59
evrardjpwe can even clean the role up later :)09:59
noonedeadpunkI'd be glad to help with it)10:00
*** shyamb has quit IRC10:00
evrardjpnoonedeadpunk: oh then I have a few ideas :)10:00
evrardjpI can give you a few of the code guidelines and you typey typey? :D10:01
noonedeadpunkevrardjp: yes, why not?10:01
noonedeadpunkI hove some time for this10:01
evrardjphaha that was a joke but why not :)10:01
evrardjplet's merge your patch first, to avoid conflicts10:01
noonedeadpunkbut still your ideas are highly appriciated :)10:02
evrardjpso yeah, one thing I think we should do there, is to cleanup the vars/main.yml of the symlinking to what I briefly checked (may need further analysis) all the symlinked files can be discovered by proper find module, with a proper glob, searching for *.py files excluding the _init10:04
noonedeadpunkas it seems that my vision can be too complicated without any reason on it10:04
evrardjpnoonedeadpunk: well, simplicity is ultime complication :)10:04
evrardjpultimate*10:05
*** stuartgr has quit IRC10:05
evrardjpwith that find in horizon, there is no need to have conditionals10:06
noonedeadpunkevrardjp: ok, than I'll try to manage with this) sounds very reasonable, as this list of files should be maintained and it'll become more flexible in case of some custom panels10:08
evrardjpnoonedeadpunk: precisely10:09
evrardjpif we make the pattern extensible, there is 0 limit to this, but I don;t think it should be extensible by default10:10
evrardjpelse we always carry a ton of variables for no reason :)10:10
evrardjpless code, less to maintain :)10:11
noonedeadpunkand things like this https://github.com/openstack/openstack-ansible-os_horizon/blob/master/vars/main.yml#L35-L37 should be put as a separate task?10:13
evrardjpjrosser: https://review.openstack.org/#/c/587544/ shows that even with latest version of the cinder role that includes your patch to the environment, we have an race condition10:13
evrardjpnoonedeadpunk: well I was thinking about that one10:14
mjwalesevrardjp: thanks, will manually add the certs for our LDAP server to the keystone containers post deployment10:14
evrardjpit depends on where we are in policy in code, and auto discovery in horizon. That's where we need to check if this is still relevant10:14
*** cshen has joined #openstack-ansible10:15
evrardjpmjwales: I am not sure it's a great idea for your operations later, but I understand that urgencies happen.10:15
evrardjpnoonedeadpunk: if there are other repos that are using that, then we should do a similar thing10:16
evrardjpelse yeah, just add this to the list of files to link10:16
jrosserevrardjp: does the cinder role include my environment fix yet?10:17
evrardjpjrosser: horizon role could definitely benefit from mitogen's approach :D10:17
evrardjpjrosser: it does10:17
evrardjpwell10:17
evrardjpI think it has10:17
evrardjpthat's master, and unfrozen10:17
evrardjpand your patch merged right?10:18
jrosseryou mean this? https://review.openstack.org/#/c/589934/10:18
jrossertheres i ton of stuff i have in flight that just wont merge10:18
evrardjpoh still not10:18
evrardjpI just commented it in my run!10:18
evrardjpthat's too bad10:18
*** jrosser has quit IRC10:19
evrardjpI mean I commented your patch out.10:19
evrardjpdarn.10:19
evrardjpoh but you got a timeout exception in that same patch: http://logs.openstack.org/34/589934/7/check/openstack-ansible-functional-ubuntu-xenial/5cf4f46/job-output.txt.gz  :)10:20
evrardjpkindra proves the point :)10:21
evrardjpjroll: to what I check, only the scheduler needs restart.10:21
noonedeadpunkevrardjp: got it10:21
evrardjpwoops that was not for jroll but for jrosser10:21
*** jrosser has joined #openstack-ansible10:25
*** shyambiradar has quit IRC10:35
*** shyambiradar has joined #openstack-ansible10:35
*** jrosser_ has joined #openstack-ansible10:38
*** jrosser has quit IRC10:38
*** jrosser_ is now known as jrosser10:38
*** cshen has quit IRC10:39
*** olivierbourdon38 has quit IRC10:40
*** Bhujay has quit IRC10:45
*** olivierbourdon38 has joined #openstack-ansible10:47
*** shyambiradar has quit IRC10:51
*** jrosser has quit IRC10:59
*** Laszlo-74 has joined #openstack-ansible10:59
*** jrosser has joined #openstack-ansible10:59
Laszlo-74hello11:00
Laszlo-74we have problems running OSA. We frequently run out of memory.11:01
Laszlo-74is there some recommendation about the amount of RAM to be used based on the number of hosts where we're installing openstack?11:02
*** shyambiradar has joined #openstack-ansible11:02
odyssey4meLaszlo-74: specifically with regards to the ansible deployment host, right?11:03
noonedeadpunkLaszlo-74: 3GB is pretty enough for me11:03
jrosserevrardjp: my aio+healthcheck ran all the way through ok11:03
noonedeadpunkbut it's a minimum i think11:04
Laszlo-74yes, I'm talking about the ansible deployment host11:06
Laszlo-74that's where we run out of memory, and the ansible-pplaybook processes get killed by the OOM killer11:07
noonedeadpunkLaszlo-74: I had same problems with 2Gb of ram, but with 3Gb the problem has gone11:08
odyssey4meLaszlo-74: perhaps it's good to share the size of your deployment, as that has an impact11:09
noonedeadpunkyep, I was talking about absolutely basic one - with 3 computing nodes and 3 mgmt nodes11:10
*** schwicht has joined #openstack-ansible11:14
Laszlo-74I'm talking about 57 physical servers of which we have 3 controllers, 3 ceph monitors, 9 ceph osd, 2 haproxy servers, 2 zabbix, 2 network nodes, and the rest  (42) is compute11:15
*** dave-mccowan has joined #openstack-ansible11:16
jrosserodyssey4me: i did openstack-ansible playbooks/repo-build.yml -e 'repo_build_wheel_rebuild=true' and this happened..... http://paste.openstack.org/show/728182/11:16
jrossersurprised to see it putting rax mirrors in there11:16
odyssey4mejrosser: is that for an AIO?11:17
jrosseryes11:17
odyssey4meoutside of zuul, right?11:17
jrosseryep11:17
*** shyambiradar has quit IRC11:17
odyssey4mehttps://github.com/openstack/openstack-ansible/blob/master/tests/roles/bootstrap-host/tasks/prepare_aio_config.yml#L123-L13311:18
jrosserok i'll see where that leads11:18
odyssey4meit's a little script which finds the fastest infra mirror, then adds it as an index to help AIO's build faster11:18
jrosserfor outside zuul?11:19
odyssey4meyep11:19
jrosserooooh11:19
odyssey4meinside zuul we are given the closest mirror by the mirror script11:19
openstackgerritMerged openstack/openstack-ansible stable/rocky: Temporarily use tempest master for all builds  https://review.openstack.org/59135711:20
odyssey4meoutside zuul we seek one out11:20
odyssey4meit's perhaps not all that appropriate to assume that - but the AIO is a dev/test tool11:20
odyssey4meomg that patch finally merged!11:20
odyssey4meprobably just in time to revert it :p11:20
*** cshen has joined #openstack-ansible11:21
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/rocky: AIO: Tune the nova scheduler workers down  https://review.openstack.org/59106311:21
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/rocky: AIO: Tune down the glance uwsgi processes  https://review.openstack.org/59106211:22
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/queens: AIO: Tune down the glance uwsgi processes  https://review.openstack.org/59234711:22
*** shyambiradar has joined #openstack-ansible11:23
*** schwicht has quit IRC11:25
*** galaxyblr has joined #openstack-ansible11:27
*** jrosser has quit IRC11:29
*** udesale has quit IRC11:30
*** Bhujay has joined #openstack-ansible11:35
*** olivierbourdon38 has quit IRC11:38
mjwalesevrardjp: yes less than ideal. Looking at the keystone role for LDAP it does not copy any certificates into the container so probably needs some work done there11:38
*** olivierbourdon38 has joined #openstack-ansible11:39
*** jrosser has joined #openstack-ansible11:53
mnaserchandankumar: is there a tool that generates tempest configuration which you use u11:54
mnaserInside tripleo?11:54
* mnaser is getting kinda frustrated with cinder role11:56
chandankumarmnaser: https://github.com/openstack/python-tempestconf11:58
chandankumarmnaser: https://docs.openstack.org/python-tempestconf/latest/11:58
chandankumarlet us know if anything missing we can work together to add the support there11:58
chandankumarconsumed in tripleo here https://github.com/openstack/tripleo-quickstart-extras/blob/master/roles/validate-tempest/templates/configure-tempest.sh.j2#L11911:59
openstackgerritMerged openstack/ansible-hardening stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky  https://review.openstack.org/59083212:00
chandankumarodyssey4me: regarding suse rpm for tempestconf, I am pushing the spec file in openstack/rpm-packaging project12:00
mnaserchandankumar: does the tripleo team absolutely need to use it or could we just template things with Ansible instead to do the same thing?12:01
*** schwicht has joined #openstack-ansible12:02
chandankumarmnaser: tripleo is using the tool from long time, we are doing a new release which is a complete rewrite for tempest conf generation. I would keep the both way let's the user choose what they want12:02
openstackgerritMerged openstack/openstack-ansible-os_searchlight stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky  https://review.openstack.org/59096812:05
chandankumarmnaser: sorry I didnot get the template thing you mean tempest config generation without using that tool as currently done in os_tempest role?12:07
*** zul has joined #openstack-ansible12:07
noonedeadpunkguys, can anyone review https://review.openstack.org/#/c/586318/ ?12:10
openstackgerritMerged openstack/openstack-ansible-os_ceilometer stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky  https://review.openstack.org/59090112:13
odyssey4memnaser: I think a tool that does discovery and then configures openstack for tempest testing is awesome. It'd be even nicer if that was in tempest itself, actually.12:14
odyssey4meFor now, though, enabling both methods seems sensible to me.12:14
openstackgerritMerged openstack/openstack-ansible-os_cloudkitty stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky  https://review.openstack.org/59090712:15
openstackgerritMerged openstack/openstack-ansible-os_sahara stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky  https://review.openstack.org/59096512:17
openstackgerritMerged openstack/openstack-ansible-os_aodh stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky  https://review.openstack.org/59089212:18
odyssey4memnaser: with regards to the cinder role - what do we know, and where are we stuck?12:18
odyssey4meWe could perhaps disable some tests temporarily to allow patches to the role to move again while we work on figuring out the issue?12:19
mnaserodyssey4me: I’m getting into the office just now so I’m going to look at CI results12:20
odyssey4memnaser: ok, let me know how I can help - when I see patches stuck for more than a week I get concerned about it falling behind and it causing downstream issues in the integrated repo.12:20
mnaserodyssey4me, chandankumar: my question was that why don’t we just bring the tempestconf functionality into a role inside os_tempest.. that way we don’t have that external dependency and it can work on all distros12:21
guilhermespodyssey4me:  yesterday we just changed to use smoke tests and re-enable voting to bionic functional tests https://review.openstack.org/#/c/591961/312:21
guilhermespI'm looking at the results now too12:21
mnaserI started squashing things but it looks like one passes and others fail. With nothing interesting in logs12:21
mnaserodyssey4me: it would be cool if we had some sort of promotion system for integrated repo to always have latest working master hashes. That way master always works regardless of other stuff breaking12:22
odyssey4memnaser: yep, evrardjp and I have discussed and will talk at the PTG about an idea to change how we release so that we just release the last working thing, rather than have to merge SHA pins and such.12:23
mnaser++12:23
odyssey4meBeing held hostage to having to merge a patch before releasing is not good. Sometimes CI fails or is slow and it puts a lot of pressure on the release manager.12:24
mnaserIt would be good to bring in some infra/Zuul experts to see what’s the best way of doing something like that12:24
odyssey4meyep, evrardjp has already been discussing options with the releases and infra/zuul crew12:24
mnaserI agree. And with a busy gate, you probably have 3 shots of retrying.12:24
mnaserIn a day12:24
odyssey4meWell, of concern is also that we now have 20+ tests at once running, and any one of them could fail due to external factors - so it's like running a gauntlet12:25
odyssey4meSo we really need to figure out what to do there. Making tests non-voting will result in things merging that break other things, so that's not really a solution.12:25
odyssey4meReducing external dependencies can help, but our whole model currently uses external deps all over the place.12:26
openstackgerritMerged openstack/openstack-ansible-os_barbican stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky  https://review.openstack.org/59089512:27
openstackgerritMerged openstack/openstack-ansible-tests stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky  https://review.openstack.org/59101412:28
odyssey4med34dh0r53 cloudnull when you're in, please take a peek at https://review.openstack.org/#/q/topic:file-backed-vms+status:open12:30
*** schwicht has quit IRC12:37
openstackgerritMerged openstack/openstack-ansible-os_horizon stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky  https://review.openstack.org/59092612:37
*** aicilamd has joined #openstack-ansible12:42
mnaserodyssey4me: most of those tests get skipped anyways and our failure rate was so low in puppet world back then12:43
odyssey4memnaser: ok, my reference to tests was that we have 20+ functional builds going at any one time... not that each build has 20+ tests in it12:44
mnaserOh yes, sorry12:45
mnaserYeah, in that way it kinda sucks that one bad job slash node puts us a few hours behind12:45
evrardjpI think we need to think about changing our functional testing12:45
evrardjpthat's why I proposed some healthchecks12:46
evrardjpbecause eventually these, if they are accepted, can be included into each roles, as tests/healtchecks.yml12:46
evrardjpthen called independently12:46
evrardjpintegration would simply be limited to a limited set of scenarios that are always ensuring the integrated passes12:47
odyssey4mehealthchecks will make absolutely no difference to the problem I'm describing12:47
evrardjpthen the integrated gates can be summarized to periodic jobs12:48
evrardjpor to a simple set12:48
odyssey4meThe problem I'm describing is that we have a lot of very long running tests, and any one can fail due to a bad mirror - wasting what ends up very often being days of our time doing rechecks.12:48
evrardjpthat's kinda my point12:48
evrardjpby doing this you separate into small bits12:48
odyssey4meYeah, make it smaller and spread it over the repositories.12:49
evrardjpthat's the first step12:49
odyssey4meThen also every build pulls in extra things it doesn't need - like cloning all the roles and all that.12:49
chandankumarmnaser: odyssey4me I am not sure about replacing tempestconf with ansible role would be a good idea, at the end of the day it is a python project consuming it from pip or git solve the issue, what we can improve is additional resources which needs to be created while runnign different scenario tests12:49
odyssey4meIf we make that into a more pluggable system, then we'd be cutting down more opportunities for failure.12:50
openstackgerritMerged openstack/openstack-ansible-os_tempest stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky  https://review.openstack.org/59097712:50
evrardjpyeah that's kinda my second part.12:50
evrardjpI think we all agree that I can't properly express my vision in english, but I am sure you understood me :)12:50
evrardjphaha12:50
odyssey4meThen of course there's the debate over the use of extra repositories - rabbit, mariadb, etc. If we use less of those, there are less opportunities for failures too.12:51
odyssey4meless moving parts = less chance of failure12:51
evrardjpagreed12:51
evrardjpodyssey4me: don't forget for mariadb we were waiting for openstack changes12:51
evrardjpwe couldn't bump to 10.212:51
evrardjpwell12:52
evrardjpwe tried12:52
odyssey4meIf we flip to using nspawn everywhere, then we could also change more things to build faster and more reliably.12:52
odyssey4meorly? I didn't know about that?12:52
evrardjpmore reliably I don't know but faster I agree12:52
evrardjpyeah, that's why there are so many questions about 10.1 and packages.12:53
evrardjp10.1 is supposed to be dead, but our last bump was not successful.12:53
evrardjpI will try again 10.212:53
odyssey4mewell, we can make it more reliable then by pre-building images for CI that have all the software - then the build just uses the latest images available12:53
evrardjpdocker docker docker?12:53
evrardjp:)12:53
odyssey4menspawn nspawn nspawn12:53
evrardjphahah12:53
evrardjpharder to pronounce you can't win12:54
odyssey4meI reject your reality and substitute it with my own. :p12:54
evrardjphahaha12:54
*** cshen has quit IRC12:55
odyssey4melol, just as I thought - tempest 19.0.0 has just been proposed for release https://review.openstack.org/59227612:58
*** schwicht has joined #openstack-ansible13:06
*** shyambiradar has quit IRC13:08
*** shardy has quit IRC13:11
odyssey4meevrardjp: ok, so do we want to go ahead and implement https://review.openstack.org/579371 so that we can key all the branch settings off it?13:11
*** shardy has joined #openstack-ansible13:11
odyssey4meIf so, I'll fire up patches to all the roles to ensure the tox changes are done.13:12
*** udesale has joined #openstack-ansible13:12
mnaserchandankumar: i see.  well, to be honest, i'm not *super* opposed to OSA leveraging python-tempestconf if that means that tripleo will consume it and therefore we'll have more hands on the tempest role (because, i think it'd be useful)13:14
mnaserodyssey4me: yeah, flipping to nspawn + finding a way to do cow for the containers, i think that would speed things up so much13:15
*** olivierbourdon38 has quit IRC13:15
mnaserodyssey4me, evrardjp: the one thing about prebuilding images, etc.. i start feeling like we're pretty much reinventing the whole ecosystem around docker already :X13:16
*** mjwales has left #openstack-ansible13:17
*** vollman has joined #openstack-ansible13:17
odyssey4memnaser: I know what you're saying, but it's not quite the same. And if we commit to nspawn we can also commit to the ecosystem it has for building images, pushing them out, etc.13:17
mnaserodyssey4me: oh yes, absolutely, i see what you mean with that13:18
mnasermy (kinda) concern is how we might end up with such a vast range of systemd versions we'd need to deal with13:19
mnaserbecause centos13:19
odyssey4meMy only concern at this stage is that CentOS isn't keeping up with the other platforms, so we'll be stuck with the lowest common demoninator's capabilities. :/13:19
mnaseryup, my thoughts exactly13:19
jrosseri already had to change stuff to make the networkd bits work on bionic13:19
jrosserthis will be the new norm i think13:20
*** shardy has quit IRC13:20
evrardjpodyssey4me: yeah13:23
evrardjpI know13:24
evrardjp(about tempest)13:24
evrardjpI am following that so...13:24
*** shardy has joined #openstack-ansible13:25
evrardjpodyssey4me: I don't see how https://review.openstack.org/#/c/579371/6 is linked to tempest though :p13:25
evrardjpmnaser: that's my point. If ppl want to use pre-built images, they can already.13:25
odyssey4meevrardjp: https://review.openstack.org/#/c/579371/6 is not linked to tempest - it's linked to actually testing rocky, right now all our rocky branches are testing master, not rocky13:29
evrardjpyeah.13:30
evrardjpwell13:30
evrardjpI have patches that are pending merging13:30
evrardjpwe need to get them in13:30
odyssey4methe question I have is whether we want to go ahead with that change so that all tests key off the .gitreview file13:31
odyssey4meto go ahead though we need to change the tox config in every repo13:31
*** nicolasbock has joined #openstack-ansible13:32
evrardjpI am fine with that change nowadays13:32
evrardjpwe are so close to have the freeze in stable/rocky, and everything is already keyed to stable branch13:32
evrardjpwe will just backport it13:32
evrardjpas we'll do13:33
evrardjpsorry it took so long13:33
evrardjpbecause now we all have a duty to backport if necessary13:33
odyssey4meso is that a yes, I should go ahead and push up the tox changes?13:33
evrardjpyes.13:33
odyssey4meokie dokey - will do that now13:33
*** KeithMnemonic has joined #openstack-ansible13:34
evrardjpodyssey4me: haha you already pushed the revert of the revert of the upper constraints branch!13:37
evrardjpteam team team13:37
mnaserok so13:40
mnasercinder-api gets the actual create request13:40
mnaserbut it never makes it to cinder-volume13:40
odyssey4memnaser: does the scheduler get it?13:42
mnaseroh good question, i forgot about the scheduler13:42
mnaser[e30ca7ae-598b-4403-92f2-ad0458f6535e] AMQP server on 10.1.1.101:5672 is unreachable: [Errno 32] Broken pipe. Trying again in 1 seconds.: error: [Errno 32] Broken pipe13:42
mnaser[req-10d2f84f-4610-4832-9e0f-da53a21ed1e2 - - - - -] [e30ca7ae-598b-4403-92f2-ad0458f6535e] Reconnected to AMQP server on 10.1.1.101:5672 via [amqp] client with port 33706.13:42
mnaserthis is when happening on opensuse13:42
mnaserthis is even happening on opensuse*13:42
mnaserit looks like it loses connectivity, which means the 'volume created' message gets lost i think13:43
mnaserhttp://logs.openstack.org/34/589934/7/check/openstack-ansible-functional-opensuse-423/85cd5a8/logs/host/cinder-volume.service.journal.log.txt.gz look at the end here13:43
odyssey4mehmm, I wonder whether the inventory there is using correct ip addresses13:44
odyssey4meand consistent with the correct ranges for the cidr13:44
odyssey4meor perhaps there's a conflicting address13:45
mnaserit has to.. this fails non deterministically13:45
mnaserit would just not work at all if it wasn't working i think13:45
jrosserwhy does it have a 10.x address at all?13:46
mnaseri think in ci, we bring up this small mini network with 10.x ips for the different containers13:46
jrosserunless it's not an AIO of course13:46
mnaserin the role ci13:46
jrosseroh right, sure13:46
mnaserhttp://logs.openstack.org/34/589934/7/check/openstack-ansible-functional-opensuse-423/85cd5a8/logs/ethtool-eth0-cfg.txt.gz13:47
mnaserwe should install ethtool in our bootstrap13:47
jrosseri have this for the integrated build https://review.openstack.org/#/c/590424/13:47
*** throwsb1 has joined #openstack-ansible13:47
jrosserperhaps the same thing needs to go on the tests to just check that the network isnt borked from the word go13:48
mnaserjrosser: the thing is, the rabbitmq logs show that it goes up and recieves connetions13:49
mnaserso something blips at some poit13:49
mnaserpoint13:49
mnaserat least we're almost never failing on deploys13:50
mnaserits just tempest13:50
mnaserhttp://logs.openstack.org/85/592085/5/check/openstack-ansible-functional-distro_install-ubuntu-bionic/8e29416/logs/host/cinder-volume.service.journal.log.txt.gz#_Aug_16_01_35_3213:51
odyssey4methese are the vars used by lxc for the nat interface: https://github.com/openstack/openstack-ansible-tests/blob/master/test-vars.yml#L69-L7213:51
mnaserit's always the same issue, the connection gets reset13:51
mnaserhappens across all operating systems13:51
odyssey4me10.1.1.1 is the br-mgmt bridge, and the containers use 10.1.1.10x13:55
odyssey4meso no interference there13:55
mnaseri wonder if its the fact we restart the service a few times13:56
mnasercause it looks like we're dropping the systemd file13:56
mnaserenable it13:56
mnaserit spins trying to keep starting up13:56
mnaserand then somehow the disconnect/connect again from all the restarts is messing up rabbitmq..13:57
mnasercause it even goes up before the lvms are up and raises and exception and shutsd own again13:57
odyssey4meperhaps we need to change some ordering there, or make the unit depend on another service13:58
mnaseri'm thinking of moving the systemd include_role out to the end of main.yml13:58
mnaserbut we flush handlers once13:58
*** throwsb1 has quit IRC13:58
mnaserim assuming we flush handlers to bring up the api service so we can create teh cinder backends13:58
odyssey4meyes13:59
mnasercinder_lvm_config.yml is totally not needed to run while things are up13:59
mnaserso i can move that one back up, say after pre install13:59
odyssey4meit seems sensible to prepare the underlying storage prior to setting up the service that relies on it14:00
mnaserspam inc14:04
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_cinder master: zuul: Drop distro jobs for Xenial and switch to smoke tests  https://review.openstack.org/59196114:04
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_cinder master: Drop JSON logging for Cinder  https://review.openstack.org/59208514:04
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_cinder master: Fix cinder_environment_overrides  https://review.openstack.org/58993414:04
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_cinder master: Add target_helper to backend_defaults  https://review.openstack.org/58460414:04
mnaserit'll at least reduce the amount of weird restarts happening14:04
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/ansible-config_template master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59247614:04
mnaseryou know what would be nice14:05
chandankumarmnaser: sure14:05
mnaser`when_or` for ansible conditionals that are OR'd instead of AND'd14:05
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/ansible-role-python_venv_build master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59247714:06
*** mmalchuk has quit IRC14:06
evrardjpjrosser: ok I have my machine up now, and I had a few minutes to check14:07
evrardjpit seems the namespaces are better now14:07
mnaserchandankumar: but the initial review looks good, evrardjp gave some good reviews on it, so if you can follow up with those14:07
evrardjpso I can actually do commands14:07
mnaseri'm excited14:07
evrardjpjrosser: what I noticed is that traffic flows perfectly fine from outside to the veth, then to bridges, and fails into the router namespace. The iptables rules are well set, but the instance doesn't seem to be pingable on its local (private) network14:08
evrardjpjrosser: will dig deeper14:08
jrosserevrardjp: i ran it this morning and the heathcheck passed14:09
evrardjpjrosser: oh interesting14:09
evrardjpmy next step was to tap into the tap interface of the vm14:10
jrossernothing but cloudnulls latest tweek on your healthcheck patch, on xenial/master14:10
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/ansible-role-systemd_mount master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59247814:10
chandankumarmnaser: I will update the patch tomorrow14:10
mnaserchandankumar: thank you very much :)14:10
evrardjpjrosser: latest tweak?14:10
evrardjpHAHA14:11
evrardjpthat's it14:11
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/ansible-role-systemd_networkd master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59247914:11
jrossersee he pushed a few fixes https://review.openstack.org/#/c/587544/14:11
evrardjpyeah14:11
evrardjpofc14:11
evrardjpI was running my patch 1814:11
evrardjpofc this is the failure reason:14:11
evrardjpitem[1]. was not there14:11
evrardjpso dhcp was false14:11
evrardjpso the private net doesn't ahve the ip14:12
evrardjpso that's why it can't be pinged on private network14:12
evrardjpsilly me14:12
jrosserwell good job really in a way14:12
*** Bhujay has quit IRC14:12
jrosserbecasue it made us look and find the netns thing14:12
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/ansible-role-systemd_service master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59248014:12
odyssey4meevrardjp: that role sha pin for the RC will likely require updates to all the role sha's now to pick up those changes14:13
evrardjpodyssey4me: yup14:14
evrardjpbut now it's the regular process of bumping shas14:14
jrosserodyssey4me: got a momoent for repo build things?14:14
evrardjpjrosser: without netns command it was freaking harder :p14:15
odyssey4mewell, there's no role pin on the rocky branch right now14:15
odyssey4meand the pins there are fro prior to the ns fix14:15
jamesdentonevrardjp rookie14:15
evrardjpjamesdenton: hahah14:15
odyssey4mejrosser: sure, shoot14:15
evrardjpodyssey4me: we'll update14:15
evrardjpit passes gates for now14:15
jrosserodyssey4me: i've been having a poke at why the wheel build takes really so much longer in the gate check than in my local vm14:16
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59248114:16
jrosserand the last phase where pip wheel does the actual builds does a truly gigantic amount of i/o to /tmp14:16
odyssey4meyep, that's to be expected14:16
jrosserbut /tmp is slooooow14:17
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible stable/rocky: Freeze all SHAs for RC1  https://review.openstack.org/59050314:17
evrardjpokay I didn't see the freeze in zuul, so I am reproposing14:17
evrardjpweird14:17
jrosserodyssey4me: so i was wondering if making /tmp actual tmpfs rather than just a normal directory would help14:18
jrosseras it wouldnt hit actual disks14:18
evrardjpjrosser: that's not a bad idea14:18
evrardjpor mount some place as tmpfs temporarily14:18
jrosseri had a go here and see a marginal improvement in my vm14:18
jrosserby symlinking /tmp/ to /dev/shm/tmp as a hack14:19
jrosserand also setting TMPDIR environment14:19
odyssey4mehmm, would this be something we apply to all containers?14:19
evrardjpjrosser: we absolutely need to cleanup that space later though, as we are on the edge of OOM everytime14:19
jrosserevrardjp: yes indeed14:19
evrardjpand only during repo build14:19
jrosserso i certainly see it ~10% faster here, but in my local tests the wheel build stage is always super quick14:20
jrossercomparing that with what happens in the gate the wheel build is always by far the slowest stage14:20
jrosserso i have a hunch we are being hurt by pointless i/o there14:20
evrardjpjrosser: if you want to improve speed we can make git sparse clone by using versions, it's faster to process.14:20
*** galaxyblr has quit IRC14:20
evrardjpthis is also a series of seconds shaved14:21
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-apt_package_pinning master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59248314:21
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ceph_client master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59248414:22
evrardjpodyssey4me: why did you want to push a specific patch for the netns on the freeze, vs doing what we said about backporting role per role, carefully?14:22
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-galera_client master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59248514:22
evrardjpI mean it's technically a stable branch now, so we justify the things that need to get in : )14:23
odyssey4meevrardjp: the backport's already in the role - but the sha pin you have does not include it14:23
evrardjpI don't mind doing a bump of all the roles just after the freeze merges14:23
odyssey4meok, as long as it's not affecting the tests, sure14:24
evrardjpoh yeah if it breaks tests no choice14:24
odyssey4meI thought it might be affecting tests in some way.14:24
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-galera_server master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59248714:24
evrardjpnow I understand your point14:25
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-haproxy_server master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59249014:25
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-lxc_container_create master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59249114:25
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-lxc_hosts master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59249214:26
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-memcached_server master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59249314:26
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-nspawn_container_create master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59249414:27
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-nspawn_hosts master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59249514:27
evrardjpso odyssey4me jrosser : should we split https://review.openstack.org/#/c/587544/ in two patches: one for bringing healthchecks for openstack, and one for using them instead of tempest?14:27
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-openstack_hosts master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59249614:27
odyssey4meevrardjp: yes, I think so - because I'm not so sure that replacing them is something that's going to get blessing from the PTL ;)14:28
odyssey4mebut the health checks are valuable to operators14:28
evrardjpagreed14:28
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-openstack_openrc master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59249814:29
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59249914:29
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_almanach master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59250114:31
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_aodh master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59250214:31
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_barbican master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59250414:32
openstackgerritAndy Smith proposed openstack/openstack-ansible-os_barbican master: Add install packages for optional oslo.messaging amqp1 driver  https://review.openstack.org/59250614:36
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ceilometer master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59250914:41
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59251014:41
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cloudkitty master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59251114:41
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_congress master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59251214:41
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_designate master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59251314:42
mnaseryeah i think replacing health checks by tempest is a bit more of a discussion14:51
mnaserif tempest cant create a volume, the 'healthcheck' playbook wont either, so it'll just be an extra thing we're carrying to maintain14:51
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-tests master: tests-ansible-env-prep: Fix plugin and ops testing on Vagrant  https://review.openstack.org/59252714:58
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-plugins master: [DNM] - strategy: linear: Inspect the delegated host on delegated tasks.  https://review.openstack.org/59167214:59
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-plugins master: tests: Use 'ansible_connection=local' for connecting to localhost  https://review.openstack.org/59252914:59
openstackgerritVictor Palma proposed openstack/openstack-ansible-galera_server master: fix bootstrap of galera 10.2+ clusters  https://review.openstack.org/59253115:02
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-tests master: tests-ansible-env-prep: Fix plugin, tests and ops testing on Vagrant  https://review.openstack.org/59252715:08
openstackgerritVictor Palma proposed openstack/openstack-ansible-galera_server master: fix bootstrap of galera 10.2+ clusters  https://review.openstack.org/59253115:10
*** spatel has joined #openstack-ansible15:12
spatelQuick question if i am running Ceph-mon on Infra node and it has br-storage interface in that case do you think i need to set MTU9000 there?15:12
spatelmy all OSD storage node running MTU 900015:13
spatelDoes Ceph-mon need MTU 9000?15:13
openstackgerritMerged openstack/openstack-ansible-ops master: MNAIO: Enable using a data disk for file-backed VM's  https://review.openstack.org/59202715:13
spatelanyone running ceph-mon on Infra* nodes?15:14
openstackgerritJonathan Rosser proposed openstack/openstack-ansible master: Mount heavily used temporary directories onto the host tmpfs  https://review.openstack.org/59254015:14
noonedeadpunkspatel: not me. we're running ceph on infiniband, so it's not case for me...15:15
spatelnoonedeadpunk: that is cool!!15:15
jamesdentonjrosser re: limiting provider networks to certain hosts (network-only vs all). Best I can tell, the templates do not regard group membership. If neutron_provider_networks.network_mappings is defined the host gets all of them when the respective agent is configured. The provider_networks plugin may need to be updated to expose group_binds and then we can check that within the neutron plugin templates?15:16
jrosserjamesdenton: yes i think that was my conclusion, so i was unable to drive specific & different config to the compute and network hosts15:17
jamesdentonyeah. i'll try and come up with something15:18
spateljamesdenton: quick question related F5, everything working great but my novac console not working on F5, i did create 6080 VIP on F5 but it somehow doesn't like.. what else NOVAC need ? any other dependency if you know otherwise i have to do some tcpdump stuff15:18
jamesdentonis the URL correct when you try to console from Horizon? Or what does the API return for the console URL?15:20
evrardjpmnaser: with the TC discussion I am convinced we should split my commit in two. Then I let you decide where to run what.15:23
*** DanyC has quit IRC15:24
mnaserevrardjp: thanks, ill link it here for others once eavesdrop publishes15:24
evrardjpsure. I think it's good to document mordred answer here about the risk of hidden successes.15:25
*** dave-mccowan has quit IRC15:26
spateljamesdenton: I am getting this URL https://10.30.2.9:6080/vnc_lite.html?token=6abb3bed-9990-4c15-b01f-2b836726b3c8&title=centos7-tux-1(6f868285-4641-4600-b131-bc3b5872da08)15:28
spatelAnd console screen is blank with "something went wrong, connection is closed"15:28
evrardjpmnaser: also we'll need to fix the cinder issue15:28
spatelin tcpdump i can see data going back and forth15:29
jamesdentonis 10.30.2.9 the correct VIP? also - maybe it doesn't work over SSL? not sure15:29
spatelyes that is external VIP15:29
spatelhmmm!15:29
spateli have tried to change 10.30.2.9 with name also manually but it doesn't like that too15:30
spateljamesdenton: just wondering if any other ports i need to open for external VIP?15:31
jamesdentonnot for VNC15:31
*** luksky has joined #openstack-ansible15:32
spateloh! wait..15:33
spateli have two instance on openstack, just found VM-1 is not working but VM-2 is working...15:34
spatelstrange15:34
spatellet me dig more15:34
spatellet me spin more VMs15:35
spatelnew VM works!!15:36
openstackgerritDavid Wilde proposed openstack/openstack-ansible-ops master: Deploy ELK in MNAIO  https://review.openstack.org/58661215:38
jamesdentonspatel nice work15:41
openstackgerritAndy Smith proposed openstack/openstack-ansible-os_ceilometer master: Add install packages for optional oslo.messaging amqp1 driver  https://review.openstack.org/59256015:42
mnaserevrardjp: i pushed up another attempt15:46
mnaserbut i will reach out to cinder team if it doesnt go through15:46
mnaserso far so good though15:47
mnasernope, centos-7 distro just failed, so15:47
mnaserso close15:47
evrardjpwhat do you mean there mnaser ?15:48
evrardjpwhich patch?15:48
*** dave-mccowan has joined #openstack-ansible15:49
mnaserthe whole stack15:51
mnaserit hasnt published yet, i am looking on zuul.o.o15:51
mnaserbtw, any reason why we're not using the rabbitmq that's shipped in UCA/RDO ?15:51
mnaserthat one is pre-integrated/gated with openstack before they release it15:51
mnaserit's also the same one that is used in openstack ci15:52
*** aicilamd has quit IRC15:52
*** gyee has joined #openstack-ansible15:53
odyssey4memnaser: because we've got stuck on using the same release across distributions instead15:54
*** mmalchuk has joined #openstack-ansible15:54
mnaseris the difference in releases really that major to be problematic for us15:54
*** mmalchuk has quit IRC15:55
mnaseri.e. i doubt that our ansible rabbit_* modules wouldn't work15:55
odyssey4mepersonally I wouldn't mind switching our default to using UCA/RDO/OBS and letting those who want to use rabbitmq/erlang repositories directly do so as opt-in15:55
*** mmalchuk has joined #openstack-ansible15:55
mnaserodyssey4me: it's one less dependency to worry about15:55
odyssey4meI think it might be a hangover from when we didn't use UCA and the distro version was universally terrible15:55
mnaseri.e. i legit just realized that we're downloading rabbitmq from the internets15:56
odyssey4meI agree, but it's cloudnull you need to convince. ;)15:56
mnaseri mean UCA has rabbitmq that ubuntu tests/deploys, and im pretty sure upstream openstack uses that same one15:56
*** fghaas has quit IRC15:57
odyssey4methe trouble with differing versions between distributions is that testing gets hairy, because we have to cater to those differences in a single role.15:57
mnaserbut are they really that major of differences15:57
odyssey4meBut perhaps we can figure out a suitable strategy for that.15:57
*** mmalchuk_ has joined #openstack-ansible15:57
odyssey4meCertainly something worth discussing at the PTG.15:57
mnaseri cant imagine rabbitmq in centos and uca being so wildly far away from each other that things wont work interchangibly15:57
cloudnulldon't we have an option to not use the rabbit packages from pivital ?15:58
chandankumarodyssey4me: what about creating something like openstack-ansible-rpm-packaging-hardening or reuse rdoinfo for version control15:58
chandankumar?15:58
chandankumar* version pinning15:58
mnasercloudnull: well the default seems to be use directly from pivotal15:59
*** mmalchuk has quit IRC16:00
mnaserrabbitmq-server seems to be at 3.6.10 in uca16:00
cloudnullI think we use 3.7.516:01
cloudnulland while functionally there's likley not a lot of difference between those versions, there's quite a bit of difference under the hood16:01
cloudnullespecially with HA and clustering16:02
mnaserlooks like rabbitmq-server in rdo is 3.6.1616:02
mnaserso we're really doing something on our own16:02
mnaserand i know at least the rdo one is very heavily integrated in the ci16:03
cloudnullis there an issue with 3.7.x ?16:03
mnaserwell, it's being pulled from the internet so it takes ~8-9 minutes to install16:04
mnaserso we don't have any mirrors16:04
mnaserand then we're seeing this16:04
mnaserhttp://logs.openstack.org/04/584604/10/check/openstack-ansible-functional-ubuntu-bionic/d303a81/logs/host/cinder-volume.service.journal.log.txt.gz16:04
mnaserwith these other timeouts http://logs.openstack.org/04/584604/10/check/openstack-ansible-functional-ubuntu-bionic/d303a81/logs/openstack/infra1/rabbitmq/rabbit@infra1.log.txt.gz#_2018-08-16_15_13_07_88916:05
*** udesale has quit IRC16:05
mnaserand at this point i'm at a bit of a loss other than i think it might be the rabbitmq version16:05
evrardjpmnaser: probably worth asking hwoarang I think he is the last one to have tried bumping rmq16:06
cloudnulllooks like the root of that first stack was from DuplicateMessageError: Found duplicate message(01f71fa100df4a4d8468471a85bc03f6) ?16:06
cloudnulllikely an issue with nested try/except in oslo ?16:07
evrardjpmnaser: but at some point we thought: why not avoiding bumping until distros catch up16:07
mnaseroh i didnt notice that cloudnull !16:07
evrardjpso taht upgrades are easy16:07
* mnaser might have been looking at this for too long16:07
mnaserevrardjp: makes sense16:07
*** Laszlo-74 has quit IRC16:07
evrardjpmnaser: I generally try to ;p16:07
mnasercloudnull: there seems to be a connection reset above that though16:08
evrardjpI am not sure how hard it is to downgrade rabbit.16:09
evrardjpBut I remember one of the major upgrades brought a lot of perf improvements16:09
evrardjpnot sure if it's 3.6, 3.7 or 3.8+?16:10
mnaserwell maybe we can convince distros to go up16:10
mnaserrdo is more of a community based thing, we have suse16:10
mnaserbut for ubuntu.. idk16:10
evrardjpwe can ask the community at the PTG16:10
evrardjpwe know ppl :)16:10
cloudnull++16:10
*** dxiri has joined #openstack-ansible16:11
cloudnullmnaser I wonder if that connection reset issue could be related to the checksums we were adding? (and are in the process of removing)16:12
*** aicilamd has joined #openstack-ansible16:12
mnasercloudnull: it could be, but the openstack-ansible-lxc_hosts change landed so i think it should no longer be an issue?16:12
*** shardy has quit IRC16:12
mnasercloudnull: nope, it's still a thing http://logs.openstack.org/04/584604/10/check/openstack-ansible-functional-ubuntu-bionic/d303a81/logs/host/kern.log.txt.gz (warning 2mb file)16:13
mnaserhttp://logs.openstack.org/04/584604/10/check/openstack-ansible-functional-ubuntu-bionic/d303a81/job-output.txt.gz#_2018-08-16_14_43_23_89159916:13
mnaseri think we need a patch to openstack-ansible-tests16:13
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible master: Remove checksum-fill, its not needed any longer  https://review.openstack.org/59188216:14
cloudnull^ that one still needs to get in16:14
cloudnullbut its got to be something to do with the general networking given its returning a connection reset16:15
mnasercloudnull: https://github.com/openstack/openstack-ansible-tests/blob/master/test-prepare-host.yml#L239-L25816:15
mnasercan we remove all this because its in lxc_hosts ?16:16
mnaser"Add iptables rule to ensure traffic checksum is correct" is gone .. "Add iptables rule to provide internet connectivity to instances" seems to be already part of lxc_hosts here http://git.openstack.org/cgit/openstack/openstack-ansible-lxc_hosts/tree/templates/lxc-system-manage.j216:16
cloudnullyup we should remove that checksum fill call too16:16
mnaseris the /usr/local/bin/lxc-system-manage call still necessary or no cloudnull ?16:17
* cloudnull wonders if there's anymore of those 16:17
cloudnullnope not required16:17
cloudnullin fact we could probably remove all that code and replace it with bits like we have in the integrated repo16:18
openstackgerritMohammed Naser proposed openstack/openstack-ansible-tests master: Remove tasks that mangle iptables (checksum, NAT)  https://review.openstack.org/59257116:18
mnasercloudnull: ^16:18
mnaseryeah all of openstack-ansible-tests has a lot of duplication with integrated repo16:19
noonedeadpunkcloudnull: can you kindly check https://review.openstack.org/#/c/586318/ one more time?16:20
*** luksky has quit IRC16:20
spatelcloudnull: I don't think we need CHECKSUM --checksum-fill anywhere, i think its OLD school thing and recent DHCP doesn't need those too16:20
cloudnull^ totally agree16:21
cloudnullI think we're on track to purging all of that16:21
mnaserspatel: you should vote on that patch then :)16:21
spatelI am glade to test them out, anyway i did test checksum stuff and found it cause lots of issue so that is no brain16:21
mnasercloudnull: i really hope that we can eliminate most of openstack-ansible-tests by straight up calling the integrated repo16:22
spatelI am going to VOTE sure16:22
mnaserimport_playbook: os-keystone-install.yml16:22
openstackgerritMohammed Naser proposed openstack/openstack-ansible-tests master: Clone integrated repo into the base jobs  https://review.openstack.org/59257316:24
spatelmnaser: I just voted on checksum-fill patch16:26
mnasernice :)16:26
spatelI am build new lab to test rocky on CentOS7..! soon you will see flood of bug request from me :)16:28
*** vnogin has joined #openstack-ansible16:31
*** cshen has joined #openstack-ansible16:33
*** pcaruana has quit IRC16:34
*** vnogin has quit IRC16:36
mnaserspatel: it might not work at all16:37
spatel?16:37
spatelmnaser: ^^16:39
bgmccollumrocky might be "rocky" on CentOS7...16:41
mnasercentos ci has broken all cycle and ive been trying to fix it but i couldnt in time for stable branch cut16:41
mnaserand still is16:42
mnaserspeaking of, evrardjp any reason why we cut stable/rocky so early?16:42
mnaseri assume osa is cycle trailing16:42
odyssey4memnaser: why not?16:43
mnaseri figured if we did it after release, then we can cut stable/rocky after we stabilize things to upstream releases packaged16:44
mnaseri.e. rdo would have their packages out, uca too16:44
odyssey4mebut our source builds would suffer because the tests for those would be building master, not rocky16:45
odyssey4methe packaged builds aren't affected by the branch name - but source builds are16:45
mnaserright16:46
mnaserbut our dependencies that we install from distros is what i was thinking of16:46
odyssey4mesure, but that's not affected by the branch name at all16:46
odyssey4meit does mean we need to port fixes back and all, but that's fine16:47
mnaseryeah the backports are a bit of a pain lol16:47
odyssey4meover time we can try to move towards the roles not being series specific, but for now they're pretty tightly coupled16:47
odyssey4meit'd be nice to work towards that not being a problem - supporting 2-3 releases or something16:48
odyssey4mewe could then perhaps even switch away from naming our branches like we do16:48
odyssey4methey could be ansible release related, for example16:48
*** openstackgerrit has quit IRC16:49
mnaserodyssey4me: yeah but that might become difficult as things get deprecated/removed/etc16:50
* mnaser cant imagine the mess of compatiblity layer that might become16:50
mnaserhwoarang: are you making any progress on opensuse-150 jobs?  they have been nv for a very long time and take up a fairly significant amount of resources right now16:51
odyssey4meyeah, it might be - but might not ;)16:51
odyssey4meit's food for thought, not very actionable right now16:51
mnaserodyssey4me: oh agreed, it's a discussion16:51
* mnaser really hopes the checksum stuff helps fix cinder16:51
mnaserhttps://review.openstack.org/#/c/592571/ if you dont mind giving an advance +W odyssey4me16:51
odyssey4mewe could, for example, make the rmq role be totally unrelated to openstack series and instead focused on rmq release major.minor - then each distro could use the one meant for it or a deployer could use the upstream packages instead of distribution packages and choose the right role version for that16:52
mnaserodyssey4me: that i totally agree on.  our infra roles should be totally usable and id love for them to be decoupled from osa16:52
mnaseri.e: ansible-role-rabbitmq16:53
mnaseri think they are relatively un-opinionated right now16:53
odyssey4mewe have kinda agreed to that in the past, but I don't think we've had time to action it, really16:54
odyssey4mewe can't just rename the repo - we'd have to seed a new one and retire the old16:54
mnaseri'm looking at a 'sample' to see what the runs are like16:57
mnasertook 22 seconds to deploy memcached.. seems reasonable.. but with_items seems to the biggest thing that hurts, esp combined with template16:58
mnaser1 second per file transferred as it looks like it does it operation by operation16:58
*** fghaas has joined #openstack-ansible17:00
odyssey4meyep, ansible prepares a script on the deploy host, then fires up a connection to the remote host and copies the script over, then executes it17:00
mnaserodyssey4me: i think mitogen is the type of thing that might help resolve these issues i guess17:01
mnaserespecially with it's built-in delegation into containers!17:01
odyssey4mepipelining helps speed it up a little, but this is the price to be paid for doing this sort of thing without an agent on the target host17:01
mnaseri think that's a lot of what mitogen does17:01
* dw scans up17:02
mnaserand gets rid of the latency too17:02
dwmnaser: indeed mitogen helps loops if the looped task is naturally 'fast' and it's ansible overhead killing things, but let me read ^ first :)17:02
odyssey4meI think that if we really want to speed things up, we have to fundamentally change how things are done instead.17:02
mnaserdw: in this case a `with_items` + `template`17:02
mnaserso we can get rid of our connection and strategy plugin and rely on mitogen to do that17:03
*** gkadam has quit IRC17:03
mnaseri think by looking at profiling it just seems like every little iteration in ansible takes a second or two17:03
mnaserand with a large # of tasks they add up17:03
dwmnaser: with_items over a template is generally a big winner17:04
odyssey4mewe do a lot of that17:04
dwbut please give me at least over the weekend -- there are serious bugs in the stuff osa needs just now :S17:04
mnaserfor example our "update the local file system crud" is a common job17:04
mnaserall it does is ensure directory/file in with_items17:04
mnaserdw: no worries, we won't be switching for a little while but i think there is a big benefit in us taking advantage of it17:05
dwmnaser: that much is clear from my local runs so far.. we just need to flush out all the issues identified so far, but my energy is a little low this week17:05
dwi have a whole new branch with osa's name on it, but it's realistically probably at least a month out17:05
mnaserdw: hey no worries :)17:05
mnaseri have all sorts of fun things im dealing with too17:05
odyssey4medweaver: we appreciate the work you're doing?17:06
odyssey4melol, that went wrong17:06
odyssey4medw ^17:06
jrosseri just noticed that .ansible is on real disks not tmpfs17:06
jrosserthats really not great17:06
odyssey4mejrosser: .ansible? you mean for the role tests?17:06
jrosserwhere it drops all the temporary files for each task17:07
jrosserin all the containers and everywhere17:07
odyssey4meah yes, I think that can be changed17:07
jrosserthat feels like super low hanging fruit17:07
*** fghaas has quit IRC17:07
* jrosser travels17:07
odyssey4mebut on a overburdened host, things in /tmp disappear (perhaps due to some garbage collection?) - so ideally we need something a little more protected17:07
spatelI am trying to increase MTU 9000 so i did container_mtu: "9000" in openstack_user_config  and now running openstack-ansible lxc-hosts-setup.yml  but it didn't change MTU on br-storage interface17:09
spatelDo i need to run any other playbook ?17:09
odyssey4mespatel: lxc-containers-create I think17:09
odyssey4methat's the one that actually does all the container things17:10
spatelLet me run that!17:10
*** openstackgerrit has joined #openstack-ansible17:13
openstackgerritMerged openstack/openstack-ansible-os_horizon master: Implements custom theme distribution  https://review.openstack.org/58631817:13
*** DanyC has joined #openstack-ansible17:13
*** electrofelix has quit IRC17:15
*** goldenfri has joined #openstack-ansible17:17
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_glance master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59258717:18
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_gnocchi master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59258817:18
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_heat master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59258917:18
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_horizon master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59259017:18
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ironic master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59259217:19
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_keystone master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59259317:19
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_magnum master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59259417:19
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_masakari master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59259517:21
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-tests master: Convert test network setup to use the networkd role  https://review.openstack.org/59259617:22
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_molteniron master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59259717:22
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_monasca master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59259817:22
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_monasca-agent master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59259917:22
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_monasca-ui master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59260017:23
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_neutron master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59260117:23
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_nova master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59260217:23
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59260317:24
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_panko master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59260417:24
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_rally master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59260517:24
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_sahara master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59260617:25
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_searchlight master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59260717:25
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_swift master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59260817:25
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tacker master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59260917:25
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59261017:26
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_trove master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59261117:26
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_trove master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59261117:26
*** mmalchuk_ has quit IRC17:27
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_watcher master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59261217:28
*** mmalchuk has joined #openstack-ansible17:28
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_zaqar master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59261317:28
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_zun master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59261417:28
*** gkadam has joined #openstack-ansible17:29
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-pip_install master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59261517:29
mnasernice, checksum fix made it to gate17:29
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-plugins master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59261617:29
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-rabbitmq_server master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59261717:29
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59261817:30
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_server master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59261917:30
mnaseris anyone looking at the ceph issues, i remember seeing some discussions around it17:30
noonedeadpunkofftopic question - does anyone has a discount, which is known not to be used, for Berlin Summit?17:30
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-rsyslog_client master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59262017:30
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-rsyslog_server master: Use the TESTING_BRANCH env var for constraints  https://review.openstack.org/59262117:30
mnasernoonedeadpunk: didnt get an ATC code? :(17:34
openstackgerritMerged openstack/openstack-ansible-ops master: MNAIO: Use virt module to shut VM's down when saving images  https://review.openstack.org/59213117:35
openstackgerritMerged openstack/openstack-ansible-ops master: MNAIO: Extend image saving to include manifest  https://review.openstack.org/59217117:35
openstackgerritMerged openstack/openstack-ansible-ops master: MNAIO: Install jmespath  https://review.openstack.org/59217317:35
odyssey4memnaser: for ceph https://review.openstack.org/590778 contains a ceph-ansible pin to tide us over17:36
odyssey4meso we need to babysit that one17:36
mnasergr17:37
mnasergithub failed to resolve17:37
mnaserfor ansible-keepalived17:37
noonedeadpunkmnaser: actually I get, thanks to you) But probably someone has one, and he knows, that he won't use it (because of recieving free entrance or as he's not going to come) for a co-worker. As my company is not very interested in sending staff to conferences17:37
mnasernoonedeadpunk: let me see what i can do try to do17:37
mnasernoonedeadpunk: can you email me to mnaser@vexxhost.com with some info and ill try to get something17:38
*** fghaas has joined #openstack-ansible17:38
noonedeadpunkmnaser: oh, great) it's not really required, but if it's possible, I'd be really grateful17:39
mnaseri'll try to see what i can do but i can't guarantee anything :)17:39
*** hamzaachi has joined #openstack-ansible17:39
*** dave-mccowan has quit IRC17:39
odyssey4meok folks, I'm out for the night - back online tomorrow, see you soon!17:42
mnaserlater odyssey4me17:44
mnaserhttps://review.openstack.org/#/q/%22+use+include_tasks+instead+of+include%22+is:open+label:Code-Review%252B2+label:Verified%252B117:44
mnasereasy +W's if possible17:44
openstackgerritDmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_horizon stable/rocky: Implements custom theme distribution  https://review.openstack.org/59133617:47
dwjrosser_: assuming i get a gust of productivity over the weekend, how do i actually go about validating the finished OSA deploy is sane?17:49
openstackgerritAndy Smith proposed openstack/openstack-ansible-os_swift master: Add install packages for optional oslo.messaging amqp1 driver  https://review.openstack.org/59263017:49
jrosserdw: it will run a test suite at the end - tempest17:50
jrosserWhich is the source of much trouble for us currently btw17:50
*** cshen has quit IRC17:51
mnaserjrosser: yeah, if you want to push up the code, we can help review it in some way (if you made changes to osa)17:51
mnaserdw: ^17:51
dwaah :)17:51
jrosserI pushed up a test patch to enable mitogen17:52
dwmnaser: mostly i want to play catch-up with the issues jrosser has already found17:52
mnaserjrosser: ooooh do you have a link17:52
jrosserJust for folks to see where/what got changed17:52
jrossermnaser https://review.openstack.org/#/c/591236/17:53
mnaseryep looks like it died on galera -- EOF on stream; last 300 bytes received: u'ssh: Could not resolve hostname {{ galera_server_bootstrap_node }}: Temporary failure in name resolution\r\n'17:54
mnaserhttps://github.com/openstack/openstack-ansible-galera_server/blob/master/handlers/main.yml#L6517:55
jrosserThe trouble all starts with delegation I think17:55
mnaseri guess the delegate_to being templated is not something that's supported17:55
jrosserWe had a good poke at it in #mitogen the other day17:56
*** fghaas has quit IRC17:57
*** cshen has joined #openstack-ansible17:58
jrosserdw: check this out for driving openstack with ansible modules https://review.openstack.org/#/c/587544/17:59
*** poopcat has joined #openstack-ansible17:59
jrosserAnd top tip, look at the "download" link top right of all these patch pages, it's really easy to drop them onto your working tree18:00
*** hamzaachi has quit IRC18:05
openstackgerritDmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_horizon stable/queens: Implements custom theme distribution  https://review.openstack.org/59263518:05
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-tests master: Convert test network setup to use the networkd role  https://review.openstack.org/59259618:07
openstackgerritMerged openstack/openstack-ansible-tests master: Remove tasks that mangle iptables (checksum, NAT)  https://review.openstack.org/59257118:09
noonedeadpunkcloudnull: if it's possible, check please cherry-pick to rocky as well https://review.openstack.org/#/c/591336/18:09
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-nspawn_hosts master: Add SUSE 15 to the nspawn hosts role  https://review.openstack.org/59217718:10
spatelodyssey4me: Thanks for you help! MTU 9000 works after running lxc-create playbook18:15
mnaserdo we have more attendees at the ptg (cloudnull anyone else from rax coming too?)18:23
cloudnullodyssey4me d34dh0r53 devx cjloader prometheanfire, i think18:23
cjloaderyep18:24
mnasercool, eyes on https://etherpad.openstack.org/p/osa-stein-ptg (and if you can add yourself as attendees too :))18:24
d34dh0r53yep18:24
prometheanfireI'm on the list18:24
mnasercool, evrardjp will hwoarang be attending (as well as any other suse folks?)18:25
mnaserwould have been nice to have jrosser :(18:25
guilhermespI want my visaaa =( heheh18:25
cjloaderI just put my name down mnaser18:26
mnaseryeah guilhermesp is pending travel stuff (hopefully)18:26
mnaseri dunno who else to ping who was there last time18:26
mnaseri'm not sure if andymccr will continue doing openstack stuff within redhat18:26
mnaserlogan-: too perhaps?18:27
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-nspawn_hosts master: Add SUSE 15 to the nspawn hosts role  https://review.openstack.org/59217718:31
*** gkadam_ has joined #openstack-ansible18:38
*** gkadam has quit IRC18:42
*** hamzaachi has joined #openstack-ansible18:43
spotzmnaser: me too!18:58
*** DanyC has quit IRC18:59
*** gkadam_ has quit IRC19:00
openstackgerritJonathan Rosser proposed openstack/openstack-ansible master: [TEST] bionic  https://review.openstack.org/58660719:00
spotzPoor mnaser is gonna be sick of me:)19:03
*** hamzaachi has quit IRC19:08
*** hamzaachi has joined #openstack-ansible19:09
*** Leo_m has joined #openstack-ansible19:16
openstackgerritJames Denton proposed openstack/openstack-ansible master: Provides group_names for use with provider_network plugin  https://review.openstack.org/59265919:19
*** Leo_m has quit IRC19:21
openstackgerritJames Denton proposed openstack/openstack-ansible-plugins master: Modify provider_network plugin to compare group_binds to group_names  https://review.openstack.org/59266119:25
jamesdentonjrosser maybe this will do what you're thinking: https://review.openstack.org/#/q/status:open+topic:bug/1787462. Accompanying bug: https://bugs.launchpad.net/openstack-ansible/+bug/178746219:26
openstackLaunchpad bug 1787462 in openstack-ansible "group_binds for provider networks has no effect on neutron provider mappings" [Undecided,In progress] - Assigned to James Denton (james-denton)19:26
guilhermesphttps://review.openstack.org/#/c/591961/419:27
guilhermespsuccess :P19:27
*** Leo_m has joined #openstack-ansible19:27
guilhermespI was waiting for this day19:27
*** kevinshort has joined #openstack-ansible19:28
jrosserjamesdenton: ahha cool - we are redeploying the lab very soon so i can try those out for real19:31
jamesdentonyes, please do!19:34
bgmccollumso it was ignoring the group_binds...no wonder19:34
jamesdentonaye19:34
spateljamesdenton: or other i have question,  my infra-02 is primary node in OSA, i have notice if i shutdown that node for testing, my horizon is just stuck and wheel is spinning, i have check F5 LB also and F5 removed that node from pool19:36
spatelDo you guys have seen issue like that?19:36
jamesdentonbgmccollum i don't see group_binds being used anywhere, honestly19:36
jamesdentonbut i could be overlooking it19:36
spatelit seem this is galera issue..19:37
jamesdentonspatel Interesting. i would recommend, at a minimum, ensuring you have connection mirroring enabled on the galera VS19:37
jamesdentonand have the mirror interfaces setup properly on the F5s19:38
*** Leo_m has quit IRC19:38
jamesdentonwell - that would only apply to F5 failover, not pool member failure19:38
mnaseromg19:39
mnaserITS PASSING?!19:39
mnaserTHEY ALL PASSED19:39
mnaserWITH NO TIMEOUTS19:39
mnaser!!!19:39
openstackmnaser: Error: "!!" is not a valid command.19:39
mnaseri think a lot of our slowness had to do with the checksum stuff cloudnull19:39
spatelmirroring is disabled on F519:40
spateljamesdenton: ^^19:40
mnaserhttps://review.openstack.org/#/c/591961/4 https://review.openstack.org/#/c/592085/6 https://review.openstack.org/#/c/589934/8 https://review.openstack.org/#/c/584604/10 please19:40
*** Leo_m has joined #openstack-ansible19:40
mnaserappreciate reviews on those19:40
spatelwhy do we need mirroring? HAproxy doesn't use mirroring19:40
jamesdentonspatel ok - i would enable it for production, but it doesn't make a difference in this case. Can you hit the galera VIP thru the F5 when you shutdown infra02?19:40
mnasercc jrosser ^ :>19:41
spatellet me try some basic test and i get back to you in min...19:41
mnaserand spotz if oyu're around ^19:41
jamesdentonspatel The F5 will reset all of those connections and cause services to create new ones, and you may hit max_connections on the galera node19:41
* jrosser looks19:41
jamesdentonjust FYI19:41
spotzmnaser: looking19:41
spateljamesdenton: interesting if max_connection issue.. can i see that in log? let me check19:42
guilhermespmnaser: reviews asap otherwise cinder will frustrate us hahaha19:42
mnaseri reviewed the ones i'm allowed to ;p19:42
mnaseri authored the ones i didnt19:42
jamesdentonspatel you would see something logged. But anyway, i don't suspect that here. it was the first thing that came to mind, but doesn't apply to this scenario19:42
spotzheheh, they looked straightforward and all cindery:)19:44
mnaserthanks for the reviews jrosser and spotz <319:45
guilhermespmnaser: but I'm still wondering we have solid conclusions of what happaned with cinder role...was it the last change in defaults/main determinant?19:46
mnaserguilhermesp: it was a combination of things19:46
mnaserevery review on that stack had an issue somewhere19:46
spotzI try to be just a distracted ping away:)19:46
*** fghaas has joined #openstack-ansible19:47
*** fghaas has quit IRC19:53
guilhermespYeah but, every review with specific changes but all of them with transient failings across different distros.. that's why I'm still wondering what makes all the patches pass19:55
*** cshen has quit IRC20:06
*** pcaruana has joined #openstack-ansible20:13
*** cshen has joined #openstack-ansible20:21
*** hamzaachi has quit IRC20:22
openstackgerritMerged openstack/openstack-ansible-os_cinder master: zuul: Drop distro jobs for Xenial and switch to smoke tests  https://review.openstack.org/59196120:29
openstackgerritMerged openstack/openstack-ansible-os_cinder master: Drop JSON logging for Cinder  https://review.openstack.org/59208520:29
*** kevinshort has quit IRC20:33
openstackgerritMerged openstack/openstack-ansible-os_cinder master: Fix cinder_environment_overrides  https://review.openstack.org/58993420:35
openstackgerritMerged openstack/openstack-ansible-os_cinder master: Add target_helper to backend_defaults  https://review.openstack.org/58460420:35
cloudnullmnaser are you seeing faster returns now ?20:41
mnasercloudnull: i think the jobs might be running faster20:41
* cloudnull just got back to a computer20:41
mnaserlike that was pretty fast to build..20:41
mnasertook 40m for the cinder stuff, worked flawless20:41
mnaserlook at the stack, centos took 7 minutes more only20:42
mnasercloudnull: https://review.openstack.org/#/c/592207/2 would be nice20:42
mnaserwe gotta make sure this is all backported to stable/rockty20:43
mnaserrocky*20:43
cloudnull++20:43
openstackgerritMohammed Naser proposed openstack/openstack-ansible-openstack_hosts stable/rocky: Use variable with full URL for RDO repositories  https://review.openstack.org/59268320:43
openstackgerritMohammed Naser proposed openstack/openstack-ansible-tests stable/rocky: Use latest hashed RDO repository  https://review.openstack.org/59268420:43
* cloudnull shgould have made those prs use the same topic20:43
openstackgerritMohammed Naser proposed openstack/openstack-ansible-tests stable/rocky: Remove tasks that mangle iptables (checksum, NAT)  https://review.openstack.org/59268520:44
mnaseryeah20:44
mnaserall good20:44
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_cinder stable/rocky: zuul: Drop distro jobs for Xenial and switch to smoke tests  https://review.openstack.org/59268620:44
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_cinder stable/rocky: Drop JSON logging for Cinder  https://review.openstack.org/59268720:45
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_cinder stable/rocky: Fix cinder_environment_overrides  https://review.openstack.org/59268820:45
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_cinder stable/rocky: Add target_helper to backend_defaults  https://review.openstack.org/59268920:45
*** ssbarnea has quit IRC20:45
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_cinder stable/rocky: Drop JSON logging for Cinder  https://review.openstack.org/59268720:45
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_cinder stable/rocky: Fix cinder_environment_overrides  https://review.openstack.org/59268820:45
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_cinder stable/rocky: Add target_helper to backend_defaults  https://review.openstack.org/59268920:46
openstackgerritMohammed Naser proposed openstack/openstack-ansible-lxc_hosts stable/rocky: Remove the checksum fills  https://review.openstack.org/59269020:47
openstackgerritMohammed Naser proposed openstack/openstack-ansible-lxc_hosts stable/queens: Remove the checksum fills  https://review.openstack.org/59269220:47
openstackgerritMohammed Naser proposed openstack/openstack-ansible-lxc_hosts stable/pike: Remove the checksum fills  https://review.openstack.org/59269320:47
openstackgerritMohammed Naser proposed openstack/openstack-ansible-lxc_hosts stable/ocata: Remove the checksum fills  https://review.openstack.org/59269420:47
*** cshen has quit IRC20:47
*** Darcidride has joined #openstack-ansible20:51
openstackgerritAlexandre Bassel proposed openstack/openstack-ansible-ops master: fix intermittent failures when deploying base glance images  https://review.openstack.org/59269620:53
*** pcaruana has quit IRC20:53
mnasercloudnull: im thinking we should squash https://review.openstack.org/#/c/590778/ with https://review.openstack.org/#/c/591882/20:55
mnaserbecause your change will never merge cause it needs the ceph one, and i think its not being stable because of the iptables stuff20:55
cloudnulllets do it20:55
*** spatel has quit IRC20:55
*** Darcidride has quit IRC20:55
cloudnullyou want me to do that or you?20:55
mnasercloudnull: do you mind doing it please?20:56
cloudnullnp at all20:56
cloudnullone sec20:56
mnaserhopefully its far more stable20:56
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible master: Temporarily use tempest master for all builds  https://review.openstack.org/59077820:58
cloudnulllets see how that goes20:58
openstackgerritAlexandre Bassel proposed openstack/openstack-ansible-ops master: fix intermittent failures when deploying base glance images  https://review.openstack.org/59269621:00
mnasercloudnull: ill keep an eye out and +A if it goes through smoothly in ci21:02
mnaserim excited, i feel good about this21:03
mnaserthe way all 4 patches in a stack passed flawlessly in check and gate made me happy21:03
openstackgerritAlexandre Bassel proposed openstack/openstack-ansible-ops master: fix intermittent failures when deploying base glance images  https://review.openstack.org/59269621:03
*** madorn has quit IRC21:18
*** madorn has joined #openstack-ansible21:19
*** aicilamd has quit IRC21:20
*** aicilamd has joined #openstack-ansible21:20
jrossermnaser: looks like there is an ordering issue with swift21:22
mnaserjrosser: context?21:23
jrosseras in the various services start / fail with exception / restart about 200-300 times before a valid config gets put down21:23
mnasercloudnull: do you think there's someway to get nspawn images for centos somewhere other than github?21:23
mnaseroh ouch21:23
*** DanyC has joined #openstack-ansible21:24
mnaserim not all that familiar with it21:24
cloudnullmnaser if there's a better source, absolutly21:24
mnaseri'm wondering if there is somewhere else we can look for them21:24
cloudnullI just took the official cent docker image source21:25
*** aicilamd has quit IRC21:25
jrossersearch this for "Started swift"  http://logs.openstack.org/07/586607/21/check/openstack-ansible-deploy-aio_lxc-ubuntu-bionic/c4f9187/logs/host/syslog.txt.gz21:25
mnasercloudnull: yeah im wondering if it can be stored somewhere outside github that we can easily mirror21:26
cloudnullid love it if there was a published rootfs somewhere21:26
cloudnulli couldnt find one21:26
cloudnullbut if theres a place we should change to that21:27
mnasercloudnull: yeah im googling and i cant seem to find anywhere21:27
bgmccollumjrosser looks like the rings need to be dropped before the swift services are started21:27
mnaseri wonder if we can somehow pull it in from docker.io21:27
mnaserjrosser: yeah i think bgmccollum might be on it21:27
jrossermnaser: cool - it just stood out in syslog when i was looking for something else21:28
cloudnullthats where I got the current image21:28
cloudnullwhich pulls from githubn21:28
*** hamzaachi has joined #openstack-ansible21:30
mnasercloudnull: what are your thoughts of bootstrapping a container using yum21:31
cloudnullthat would work21:31
cloudnulli think odyssey4me had some work in that direction too21:31
mnaseri think i can find some old ansible code where i got that to work21:31
mnaserand itll pull from mirrors too21:31
cloudnullusing bindep LIR21:31
mnaseroh thats an interesting approach21:32
mnasercloudnull: i see that we've moved to using docker images rather than stuff out of https://us.images.linuxcontainers.org/ .. any history behind this?21:35
mnaserim thinking if we use https://us.images.linuxcontainers.org/ behind infra cache/mirrors, we might see speed ups21:36
mnaserand infra's cache is there to save us from any network issues21:36
*** aicilamd has joined #openstack-ansible21:36
mnaserhttp://mirror.ca-ymq-1.vexxhost.openstack.org:8080/images.linuxcontainers/ is a thing too21:39
openstackgerritMerged openstack/openstack-ansible-tests master: Revert "Make CentOS tests non-voting"  https://review.openstack.org/59220721:41
openstackgerritByron McCollum proposed openstack/openstack-ansible-os_swift master: [Test] Move swift ring tasks before systemd tasks  https://review.openstack.org/59274821:48
*** goldenfri has quit IRC21:49
*** aicilamd has quit IRC21:51
*** aicilamd has joined #openstack-ansible21:51
openstackgerritByron McCollum proposed openstack/openstack-ansible-os_swift master: [Test] Move swift ring tasks before systemd tasks  https://review.openstack.org/59274821:52
*** aicilamd has quit IRC21:56
bgmccollumjrosser ^ we'll see if that helps21:56
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-tests master: Convert test network setup to use the networkd role  https://review.openstack.org/59259621:56
cloudnullmnaser I'm pretty sure that we have a reverse proxy in infra for the lxc container images21:58
cloudnullhowever for the most part we dont use those images any longer.21:58
mnasercloudnull: any reason why? they seem to be fairly up to date21:58
cloudnullthe current images are SUPER minimal vs those images.21:58
mnaserah21:58
mnaserwas hoping we can leverage http://mirror.ca-ymq-1.vexxhost.openstack.org:8080/images.linuxcontainers/  again :(21:59
cloudnullwe can21:59
cloudnullits a variable21:59
mnaseri just almost finished most of the work in nspawn-hosts :p21:59
mnaserand it's still possible to go the old way21:59
cloudnullhttps://github.com/openstack/openstack-ansible-lxc_hosts/blob/master/defaults/main.yml#L188-L18921:59
mnaserwith the option of specifying the url still there22:00
cloudnullboth lxc and nspawn use the same images22:00
cloudnullyes with the legacy image option, the url can be set22:02
cloudnullhttps://github.com/openstack/openstack-ansible-lxc_hosts/blob/master/defaults/main.yml#L18622:02
cloudnullsame for nspawn22:02
cloudnullhttps://github.com/openstack/openstack-ansible-nspawn_hosts/blob/master/defaults/main.yml#L114-L11822:02
openstackgerritMohammed Naser proposed openstack/openstack-ansible-tests master: Clone integrated repo into the base jobs  https://review.openstack.org/59257322:04
openstackgerritMohammed Naser proposed openstack/openstack-ansible-tests master: Add NODEPOOL_LXC_IMAGE_PROXY for faster and more reliable images  https://review.openstack.org/59275222:04
openstackgerritMohammed Naser proposed openstack/openstack-ansible-nspawn_hosts master: Switch to downloading images from LXC mirrors  https://review.openstack.org/59275322:06
mnasercloudnull: ^ your thoughts welcome, id want to rerun it twice to see performance difference22:06
* mnaser anxiously watched the openstack-ansible patch22:11
mnaserso far so good22:11
mnasercould it have been the checksum the whole time22:11
*** schwicht has quit IRC22:12
openstackgerritMohammed Naser proposed openstack/openstack-ansible-nspawn_hosts master: Switch to downloading images from LXC mirrors  https://review.openstack.org/59275322:19
mnaserhttps://us.images.linuxcontainers.org/images/centos/7/amd64/default/20180816_02:16/22:21
mnaserlooks like it's 67M for centos22:21
mnaserbionic is 80M22:22
mnaserand it downloaded in 2s in a job, i bet it'd be even faster and more reliable with the proxy and mirrors22:22
mnaserhttp://zuul.openstack.org/builds.html?job_name=openstack-ansible-deploy-aio_metal-centos-722:27
*** chyka has joined #openstack-ansible22:27
mnaserim hoping to see the first green in months22:27
*** vnogin has joined #openstack-ansible22:39
openstackgerritMerged openstack/openstack-ansible-os_swift master: Move centos tests to voting  https://review.openstack.org/58771222:44
*** hamzaachi has quit IRC22:52
*** DanyC has quit IRC23:04
*** DanyC has joined #openstack-ansible23:04
*** vnogin has quit IRC23:05
*** chyka has quit IRC23:13
cloudnullmnaser the http://zuul.openstack.org/builds.html?job_name=openstack-ansible-deploy-aio_metal-centos-7 jobs shouldn't be using containers.23:20
cloudnullall of those would be gating on host only23:20
mnasercloudnull: yeah it's the one that is least likely to fail23:20
cloudnullah ok23:20
mnaserit was failing last time because of the cinder stuff23:20
mnaserbut we just got the fixes in so fingers crossed23:20
cloudnullnice!23:20
mnaserfor some reason the build got aborted and restarted?23:20
*** Leo_m has quit IRC23:20
mnaser:(23:20
mnaserbut still all green for that squashed one23:21
*** Leo_m has joined #openstack-ansible23:21
cloudnullmnaser in the case of ubuntu the images from http://cdimage.ubuntu.com/ubuntu-base/releases/ are about half the size of the ones  on the lxc index.23:31
cloudnullmaybe it makes sense to change cent to lxc index23:31
cloudnullsuse and ubuntu are sourcing from their respective repos23:32
* cloudnull going to eat dinner, bbl 23:33
*** DanyC has quit IRC23:34
*** aicilamd has joined #openstack-ansible23:52
*** gyee has quit IRC23:53
*** aicilamd has quit IRC23:57

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!