Tuesday, 2019-12-17

« Monday, 2019-12-16 Index Wednesday, 2019-12-18 »
rm_workwhelp, just now actually looked back and realized I f'd up the basics on that patch because i didn't run the full suite after my last change >_<00:06
rm_workfixing00:07
openstackgerritAdam Harwell proposed openstack/octavia master: Support creating an LB in a specified AZ  https://review.opendev.org/69376200:26
*** mithilarun has joined #openstack-lbaas00:43
*** mithilarun has quit IRC01:06
*** goldyfruit has quit IRC01:14
*** yamamoto has joined #openstack-lbaas01:25
*** goldyfruit has joined #openstack-lbaas01:29
*** goldyfruit has quit IRC01:43
openstackgerritHidekazu Nakamura proposed openstack/octavia master: Add install guide for Ubuntu  https://review.opendev.org/67284201:46
*** mithilarun has joined #openstack-lbaas02:00
openstackgerritHidekazu Nakamura proposed openstack/octavia master: Add install guide for Ubuntu  https://review.opendev.org/67284202:07
*** yamamoto has quit IRC02:19
*** yamamoto has joined #openstack-lbaas02:38
*** mithilarun has quit IRC03:02
openstackgerritBrian Haley proposed openstack/octavia master: Support hacking 2.0.0  https://review.opendev.org/69930203:08
openstackgerritBrian Haley proposed openstack/octavia master: Support hacking 2.0.0  https://review.opendev.org/69930203:34
*** psachin has joined #openstack-lbaas03:37
*** zainub_wahid has joined #openstack-lbaas03:58
*** goldyfruit has joined #openstack-lbaas04:01
*** yamamoto has quit IRC04:34
*** yamamoto has joined #openstack-lbaas04:40
*** goldyfruit has quit IRC05:00
*** ramishra has joined #openstack-lbaas05:10
*** yamamoto has quit IRC05:20
*** yamamoto has joined #openstack-lbaas05:41
*** yamamoto has quit IRC05:53
*** yamamoto has joined #openstack-lbaas05:54
*** yamamoto has quit IRC06:07
*** yamamoto has joined #openstack-lbaas06:47
*** yamamoto has quit IRC06:58
*** gcheresh_ has joined #openstack-lbaas07:44
*** pcaruana has joined #openstack-lbaas07:52
*** bonguardo has joined #openstack-lbaas07:55
*** mjozefcz|afk has joined #openstack-lbaas08:03
*** rpittau|afk is now known as rpittau08:05
*** yamamoto has joined #openstack-lbaas08:08
*** ataraday_ has joined #openstack-lbaas08:14
*** tkajinam has quit IRC08:18
*** tesseract has joined #openstack-lbaas08:25
*** yamamoto has quit IRC08:41
*** yamamoto has joined #openstack-lbaas08:46
*** ccamposr has joined #openstack-lbaas09:02
*** coreycb_ has joined #openstack-lbaas09:03
*** dasp has joined #openstack-lbaas09:05
*** amotoki_ has joined #openstack-lbaas09:09
*** tesseract has quit IRC09:10
*** psachin has quit IRC09:10
*** rcernin has quit IRC09:10
*** Dinesh_Bhor has quit IRC09:10
*** ataraday_ has quit IRC09:10
*** ccamposr__ has quit IRC09:10
*** sorrison has quit IRC09:10
*** jamesdenton has quit IRC09:10
*** dasp_ has quit IRC09:10
*** amotoki has quit IRC09:10
*** coreycb has quit IRC09:10
*** coreycb_ is now known as coreycb09:12
*** yamamoto has quit IRC09:13
*** Dinesh_Bhor has joined #openstack-lbaas09:16
*** tesseract has joined #openstack-lbaas09:17
*** yamamoto has joined #openstack-lbaas09:18
*** rcernin has joined #openstack-lbaas09:18
*** yamamoto has quit IRC09:23
*** zainub_wahid has quit IRC10:11
*** salmankhan has joined #openstack-lbaas10:23
*** salmankhan has quit IRC10:28
*** salmankhan has joined #openstack-lbaas10:28
*** salmankhan has quit IRC10:29
*** salmankhan has joined #openstack-lbaas10:29
*** yamamoto has joined #openstack-lbaas11:02
*** yamamoto has quit IRC11:03
*** hkominos has quit IRC11:06
*** yamamoto has joined #openstack-lbaas11:06
*** gcheresh_ has quit IRC11:07
*** gcheresh_ has joined #openstack-lbaas11:07
*** ataraday_ has joined #openstack-lbaas11:15
*** gcheresh has joined #openstack-lbaas11:17
*** gcheresh_ has quit IRC11:18
*** gcheresh has quit IRC11:23
*** yamamoto has quit IRC11:31
*** ianychoi has quit IRC11:38
*** ianychoi has joined #openstack-lbaas11:40
*** yamamoto has joined #openstack-lbaas11:41
*** farhanjamil has joined #openstack-lbaas11:42
*** yamamoto has quit IRC11:45
*** yamamoto has joined #openstack-lbaas11:54
*** yamamoto has quit IRC11:56
*** amotoki_ is now known as amotoki12:12
*** gcheresh has joined #openstack-lbaas12:14
*** farhanjamil has quit IRC12:33
*** hkominos has joined #openstack-lbaas12:35
hkominoscgoncalve: ping!12:36
*** yamamoto has joined #openstack-lbaas12:37
*** tesseract has quit IRC12:38
*** tesseract has joined #openstack-lbaas12:38
*** goldyfruit has joined #openstack-lbaas12:41
*** gcheresh_ has joined #openstack-lbaas12:43
*** gcheresh has quit IRC12:47
*** gcheresh_ has quit IRC13:01
*** hkominos has quit IRC13:07
*** hkominos has joined #openstack-lbaas13:09
*** ccamposr has quit IRC13:28
*** ccamposr has joined #openstack-lbaas13:28
*** gcheresh_ has joined #openstack-lbaas13:43
*** yamamoto has quit IRC13:49
*** yamamoto has joined #openstack-lbaas13:51
*** yamamoto has quit IRC13:51
rm_workhkominos: FYI you can also just generate a new ssh key, and upload it to nova manually as the Octavia service user... Then any new amphora will use that13:51
rm_workThough in the long term it'd be good to figure out why it's not adding it automatically13:51
hkominosrm_work: I might be mistaken but I thought from the discussion yesterday  and the log that the issue is not the fact that it does not upload but that ansible get confused and crushes because it cannot evaluate a variable despite reading it13:56
hkominosoriginal message: recursive loop detected in template string: {{ amp_ssh_key_path }}\13:58
*** tkajinam has joined #openstack-lbaas14:04
rm_workmaybe, no idea14:04
rm_workbut you can fix it temporarily if you really just want to do debugging14:04
rm_work(on the octavia side)14:05
hkominosThat is probably what I have to try. Commenting out the blocks did not work though so I have to find another way to actually use the key that is provided14:06
hkominosglory to the brave14:06
rm_workyeah but i am saying, you can manually create/upload a key after (delete the existing keypair in nova and make yours with the same name) and it will work fine14:09
rm_workso you can ignore what ansible is doing for now and deal with that later14:09
*** gcheresh_ has quit IRC14:27
*** tkajinam has quit IRC14:47
*** TrevorV has joined #openstack-lbaas14:47
ataraday_rm_work, Hi! I found that you have change similar to my https://review.opendev.org/#/c/696943/ I'm fine with your variant... But mine has been on the review for a couple of weeks, so it can be updated if you have concernes ;)14:50
rm_workah! :D14:51
rm_workdidn't see you'd worked on that14:51
rm_workis it basically the same code? lol14:51
ataraday_I've updated UpdateAmphoraInfo task, so not identically :D14:52
*** yamamoto has joined #openstack-lbaas14:54
*** yamamoto has quit IRC14:54
rm_workheh14:54
rm_workok14:54
rm_workwe can go with yours, it's probably better thought out, heh14:54
*** yamamoto has joined #openstack-lbaas14:55
*** yamamoto has quit IRC15:00
ataraday_I don't insist just deciced to point this out :)15:03
rm_workoh, mine does it *even earlier*15:21
rm_worklet me see why it's failing15:21
rm_workahh your comment15:21
rm_workheh15:21
openstackgerritAdam Harwell proposed openstack/octavia master: Update the lb_id on an amp earlier if we know it  https://review.opendev.org/69808215:22
openstackgerritAdam Harwell proposed openstack/octavia master: Update the lb_id on an amp earlier if we know it  https://review.opendev.org/69808215:23
openstackgerritAdam Harwell proposed openstack/octavia master: Update the lb_id on an amp earlier if we know it  https://review.opendev.org/69808215:24
rm_worklol individual fixes in gerrit UI augh15:25
rm_workanywho, let's see if that works15:25
rm_workataraday_: i think *as soon as possible* is best? so even we could do it in the initial create like in my patch15:25
hkominosdo we report bugs in launchpad or storyboard ?15:28
rm_workhkominos: storyboard15:31
hkominosthx!15:32
hkominosrm_work: Sanity check: Is this a typo ? https://opendev.org/openstack/tripleo-ansible/src/branch/master/tripleo_ansible/roles/octavia-undercloud/tasks/main.yml#L38 . I was expecting stat: path: and not path= . I did not see that annotation in the ansible docs.15:47
hkominosor maybe I should report this in the OOO channel15:47
rm_worki don't know anything about triple-o T_T15:47
rm_workyeah, might be better, sorry15:47
hkominosfair enough!15:47
hkominosno worries15:47
johnsomhkominos: probably best to ask in the tripleo channel.15:48
rm_worktrying to run devstack on our centos stuff is killing me15:49
johnsomOther than some cross over, the Octavia team doesn’t work on the deployment tools.15:49
rm_work /opt/stack/devstack/lib/keystone: line 447: /usr/local/bin/keystone-manage: No such file or directory15:49
rm_workwtflol15:49
rm_worki should just upload an ubuntu image again15:49
rm_workand deal with qemu being slooow15:49
hkominosjohnsom: Yes i understand.15:52
johnsomCool, we are certainly happy to help with the Octavia stuff, but there are just too many deployment tools for us to be experts (competent?) in all of them. lol15:57
rm_workjohnsom: i fixed the issues you had with the AZ patch I think15:59
johnsomrm_work Ok, I will take a look again this morning16:03
*** mjozefcz|afk has quit IRC16:06
*** tesseract has quit IRC17:00
openstackgerritGregory Thiemonge proposed openstack/python-octaviaclient master: Add amphora stats show API and CLI  https://review.opendev.org/69946617:01
*** hkominos has quit IRC17:04
ataraday_rm_work, yes, sooner better. My point was wasting time/effort with duplicate changes.17:09
rm_workah yeah17:09
rm_workjust didn't see yours17:09
*** salmankhan has quit IRC17:17
*** ramishra has quit IRC17:23
*** rpittau is now known as rpittau|afk17:25
*** mithilarun has joined #openstack-lbaas18:04
*** bonguardo has quit IRC18:22
rm_worksiuohfhiusehufishug i hate custom distros where stuff is just "a little different"18:29
*** mithilarun has quit IRC18:30
*** mithilarun has joined #openstack-lbaas18:34
*** pcaruana has quit IRC18:38
*** ataraday_ has quit IRC18:45
*** pcaruana has joined #openstack-lbaas19:18
*** TrevorV has quit IRC19:20
*** gcheresh_ has joined #openstack-lbaas19:36
*** TrevorV has joined #openstack-lbaas19:37
*** pcaruana has quit IRC19:59
*** gcheresh_ has quit IRC20:07
openstackgerritAdam Harwell proposed openstack/octavia-tempest-plugin master: Add new scenario test to create LB in specific AZ  https://review.opendev.org/69534920:28
rm_worki'm so confused by the last failure on that job, too... seems to have failed because... it ran on Xenial? O_o20:28
*** salmankhan has joined #openstack-lbaas20:41
*** pcaruana has joined #openstack-lbaas20:42
johnsomI must be looking at a different job, the tempest one passed20:44
rm_workon PS12: octavia-v2-az-dsvm-scenario-two-node FAILURE in 10m 24s (non-voting)20:45
rm_workof the tempest test20:45
*** salmankhan has quit IRC20:45
rm_work+ ./stack.sh:main:224                      :   [[ ! xenial =~ (bionic|stretch|jessie|f29|opensuse-15.0|opensuse-15.1|opensuse-tumbleweed|rhel7) ]]20:45
rm_work+ ./stack.sh:main:225                      :   echo 'WARNING: this script has not been tested on xenial'20:45
rm_workso weird20:46
johnsomAh, yeah, that was a recent change20:46
rm_workwhy would this stuff run on xenial and not bionic?20:46
rm_worki thought everything ran on bionic for a long time now20:46
johnsomWell, the two node jobs have probably bit-rotted and I bet whoever created this one copied what was there20:47
rm_workyeah but they trace back to `devstack-tempest`20:48
rm_workparent: octavia-dsvm-live-two-node-base -> parent: octavia-dsvm-base -> parent: devstack-tempest20:48
johnsomrm_work https://github.com/openstack/octavia-tempest-plugin/blob/master/zuul.d/jobs.yaml#L3-L720:48
rm_workahh lol20:49
rm_workdidn't expand that far up20:49
rm_workhokay20:49
johnsomThose were unstable in that sometimes the neutron networks didn't cross the nodes right. I never got back around to futzing with those jobs to get them stable.20:50
rm_worksetting both of those to bionic20:50
openstackgerritAdam Harwell proposed openstack/octavia-tempest-plugin master: Add new scenario test to create LB in specific AZ  https://review.opendev.org/69534920:50
rm_workwell, we kinda need them to work for this test :(20:50
rm_workand i'm just about to give up on getting this running on my cloud >_>20:50
johnsomreally?20:51
openstackgerritMerged openstack/octavia-tempest-plugin master: Add octavia-v2-dsvm-scenario-centos-8  https://review.opendev.org/69733620:52
rm_workI mean devstack20:52
*** armax has joined #openstack-lbaas20:59
johnsomrm_work So, I think this looks ok. I wish it had the flavor updates in it so we could do the conflict resolution, etc. but that can be a followup21:15
rm_workerr21:16
rm_workwhichwhat?21:16
rm_workwhich flavor updates21:16
johnsomflavor updates to support defining the AZ in a flavor definition21:16
rm_workah21:25
rm_workyeah i can write that in a followup21:26
rm_worki have a couple of followups in mind already21:26
rm_workbut they're less urgent21:26
rm_worki need THIS to merge because it has a migration <_<21:26
rm_workcan21:26
rm_work*can't pull those in downstream for testing T_T21:26
*** TrevorV has quit IRC21:40
*** pcaruana has quit IRC21:41
*** rcernin has quit IRC21:46
rm_workjohnsom: ah also: https://review.opendev.org/#/c/699047/ had a couple of typos (that don't actually affect anything because we didn't ever pass this thing by name)21:58
johnsomSigh21:59
rm_workjohnsom: you voted it as backport :D22:08
johnsomCaught the bot, or distracted reviewer in this case22:09
*** rcernin has joined #openstack-lbaas22:42
openstackgerritMerged openstack/octavia master: Support creating an LB in a specified AZ  https://review.opendev.org/69376222:49
openstackgerritMerged openstack/octavia-lib master: Missed some flavor references in the AZ methods  https://review.opendev.org/69904722:49
rm_workwhelp, finally kinda got devstack running lol22:56
johnsomrm_work I have few comments already on the client. Want me to  finish the review or do you want to get started on those?23:02
rm_workhmmm23:02
rm_workgo ahead, i'm poking at a followup for AZ23:02
rm_worki can handle them all at once23:03
johnsomOk23:03
*** armax has quit IRC23:07
*** sorrison has joined #openstack-lbaas23:09
*** tkajinam has joined #openstack-lbaas23:13
rm_worki'm also just starting to test the client myself23:16
rm_worknot sure how to access these commands...23:17
johnsomThat's because the docs are missing... grin23:19
rm_worklol23:19
johnsomOk, I think those are my comments. I will re-look it over once the docs render.23:19
rm_work[stack@adam-devstack-az-2 octavia]$ openstack loadbalancer availabilityzoneprofile list23:20
rm_workopenstack: 'loadbalancer availabilityzoneprofile list' is not an openstack command. See 'openstack --help'.23:20
johnsomYou don't have that installed23:20
rm_workhmmm yeah23:20
johnsomJust setup a venv23:20
rm_worki told devstack to install it but something must have changed, because it didn't23:20
johnsomThat is what I do23:20
rm_workweird, it WAS already set up, and pip showed it as being installed as a dev egg...23:25
rm_workbut it wasn't picking it up until i did a reinstall23:25
rm_worki thought devstack used -e but maybe it doesn't anymore23:25
rm_workjohnsom: how does the "type" for the schemas work? everything so far is "string"23:28
rm_workis "list" a valid type? what defines the types and how they're parsed?23:28
rm_workjson-schema?23:28
rm_workyeah, looks like no "list" schema type :/23:29
johnsomI did not context switch with you, so not sure what you are talking about23:29
johnsomFor the profile?23:29
johnsomYeah, you must be talking about the profile. It's "array"23:31
johnsomjavascript types23:31
johnsomas it's JSON23:31
johnsomhttps://json-schema.org/draft/2019-09/json-schema-validation.html#rfc.section.6.1.123:31
rm_workyes heh23:34
rm_workok cool23:35
openstackgerritSam Morrison proposed openstack/octavia-tempest-plugin master: Add new scenario test to create LB in specific AZ  https://review.opendev.org/69534923:46
sorrisonah, I just realised you've push up changed to this review rm_work, I'll fix that up23:47
rm_workwas just about to check what you changed :D23:47
rm_workat least the run had finished23:48
rm_workso i can see what broken23:48
rm_work*broke23:48
rm_workuhh except it timed out during amp create, lolwat23:48
rm_work*amp image create23:48
openstackgerritSam Morrison proposed openstack/octavia-tempest-plugin master: Add new scenario test to create LB in specific AZ  https://review.opendev.org/69534923:49
sorrisonYeah so basically I just did a recheck for you :-)23:49
sorrisonDidn't need to change anything23:49
sorrisonGonna be my last day tomorrow, off for 4 weeks23:50
rm_worklol23:50
rm_workahh wow23:50
rm_workok23:50
rm_workwish I could take a month off :D23:50
rm_workright now i'm working on adding "valid_vip_networks" to the amp driver AZ schema23:50
sorrisonyeah it's been a while since I've had this long off23:50
rm_workbecause I remembered that'll be per-AZ also23:50
sorrisonyeah ok nice23:51
rm_workand i'll take a look at fixing the client stuff per michael's review whenever he posts that23:51
rm_workhe might already have23:51
sorrisonAnything I can help with?23:51
rm_workhopefully by the time you get back, this code will be in-use in prod23:51
rm_worklol uhhh23:51
sorrisonthe client patch was a sed replace withe the flavor stuff so could be a few issues23:51
rm_worki think it's mostly covered, though if you want to deal with the docs on the client right now that'd be helpful23:51
sorrisonalthough gave it a good test in our env and seems ok23:52
rm_worklooks like he posted a review23:52
rm_workso yeah, if you have time to do the client patch fixes23:52
sorrisonyeah I'll take a look23:55
« Monday, 2019-12-16 Index Wednesday, 2019-12-18 »

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