Wednesday, 2020-05-06

*** wuchunyang has joined #openstack-lbaas01:20
*** wuchunyang has quit IRC01:25
*** sapd1_x has quit IRC01:41
*** sapd1_x has joined #openstack-lbaas02:19
*** sapd1_x has quit IRC02:29
*** sapd1_x has joined #openstack-lbaas02:43
*** vishalmanchanda has joined #openstack-lbaas03:03
*** sapd1 has quit IRC03:16
*** psachin has joined #openstack-lbaas03:37
*** ramishra has joined #openstack-lbaas03:37
*** openstack has joined #openstack-lbaas04:24
*** ChanServ sets mode: +o openstack04:24
*** gcheresh has joined #openstack-lbaas04:29
*** wuchunyang has joined #openstack-lbaas04:44
*** wuchunyang has quit IRC05:19
openstackgerritjacky06 proposed openstack/octavia-tempest-plugin master: Remove all usage of six library
*** wuchunyang has joined #openstack-lbaas05:28
*** cgoncalves has joined #openstack-lbaas05:37
*** wuchunyang has quit IRC06:08
*** yamamoto has joined #openstack-lbaas06:18
*** yamamoto has quit IRC06:24
*** ccamposr__ has joined #openstack-lbaas06:30
*** ccamposr has quit IRC06:32
*** JayLiu7319 has quit IRC06:48
*** maciejjozefczyk has joined #openstack-lbaas06:52
*** ccamposr__ has quit IRC07:11
*** rpittau|afk is now known as rpittau07:31
*** cgoncalves has quit IRC07:34
*** cgoncalves has joined #openstack-lbaas07:40
*** openstackstatus has quit IRC07:46
*** openstack has joined #openstack-lbaas07:50
*** ChanServ sets mode: +o openstack07:50
*** wuchunyang has joined #openstack-lbaas07:54
*** ccamposr__ has joined #openstack-lbaas07:55
*** ccamposr has quit IRC07:57
*** dmellado has quit IRC07:58
*** dmellado has joined #openstack-lbaas08:04
openstackgerritOpenStack Proposal Bot proposed openstack/octavia-dashboard stable/ussuri: Imported Translations from Zanata
*** salmankhan has joined #openstack-lbaas08:33
*** salmankhan has quit IRC08:44
*** salmankhan has joined #openstack-lbaas08:45
*** ataraday_ has joined #openstack-lbaas08:54
*** strigazi has joined #openstack-lbaas09:00
*** rpittau is now known as rpittau|bbl09:44
*** wuchunyang has quit IRC10:04
*** maciejjozefczyk_ has joined #openstack-lbaas10:43
*** maciejjozefczyk has quit IRC10:44
*** maciejjozefczyk has joined #openstack-lbaas10:44
*** maciejjozefczyk_ has quit IRC10:48
*** maciejjozefczyk_ has joined #openstack-lbaas10:49
*** maciejjozefczyk has quit IRC10:50
*** maciejjozefczyk has joined #openstack-lbaas10:50
*** maciejjozefczyk_ has quit IRC10:53
*** wuchunyang has joined #openstack-lbaas10:54
*** maciejjozefczyk_ has joined #openstack-lbaas10:55
*** maciejjozefczyk has quit IRC10:55
*** livelace has joined #openstack-lbaas11:01
*** maciejjozefczyk_ has quit IRC11:04
*** maciejjozefczyk_ has joined #openstack-lbaas11:05
*** servagem has joined #openstack-lbaas11:34
*** yamamoto has joined #openstack-lbaas11:40
*** yamamoto has quit IRC11:42
*** sapd1 has joined #openstack-lbaas11:46
rm_worksomeone recheck next time it finishes? I want to see minimum 4 passes in a row before merging that11:47
rm_worki *do not* trust grenade11:47
cgoncalvesrm_work, ack. grenade can explode in our hands, I get it11:51
cgoncalvesrm_work, stein gate fix:
rm_workah damn, stein gate broken? just +W'd some backports :/11:52
cgoncalvesthink they were all non-stein patches11:52
openstackgerritMerged openstack/python-octaviaclient master: Remove mocking of functools.partial
openstackgerritMerged openstack/python-octaviaclient master: Remove Babel requirement
*** sapd1_x has quit IRC11:59
*** livelace has quit IRC12:13
rm_workyep :D nice12:16
*** rpittau|bbl is now known as rpittau12:18
*** tkajinam has joined #openstack-lbaas12:34
*** cgoncalves has quit IRC13:19
*** wuchunyang has quit IRC13:25
*** cgoncalves has joined #openstack-lbaas13:25
*** TrevorV has joined #openstack-lbaas13:31
*** livelace has joined #openstack-lbaas13:34
*** maciejjozefczyk_ is now known as maciejjozefczyk13:37
haleybjohnsom: it took me a while to track that down in the neutron tree, but i could never find a way around it short of removing it13:39
openstackgerritMerged openstack/octavia stable/train: Add a periodic image build job
openstackgerritMerged openstack/octavia stable/train: Fix padding logic for UDP health daemon
openstackgerritMerged openstack/octavia stable/queens: Fix the amphora noop driver
*** livelace has quit IRC13:59
*** livelace has joined #openstack-lbaas14:01
cgoncalvesbah, rocky gate is broken again. jobs are installing py3-only octavia-tempest-plugin15:24
johnsomI was wondering when that was going to happen15:24
johnsomThe other projects are pinning to the last tagged version of the plugin.15:25
cgoncalvesI thought we/gmann had fixed those but guess not15:25
johnsomThis is how I did it for designate:
johnsomShould I put up some patches or do you want to?15:26
nmickusjohnsom I made some updates to the listeners CLI patch, could you check them before I do the pools?15:26
cgoncalvesoh, I think I remember. gmann set override-checkout on the -xenial job15:26
cgoncalveswe probably need to do the same on the xenial base jobs too15:27
johnsomnmickus Sure, give me a few minutes15:27
cgoncalvesjohnsom, thanks for the tip!15:27
KeithMnemonicjohnsom: just to cycleback about my traffic ops tempest question. for some reason it uses a go based httpd.bin which does not work in cirros15:29
KeithMnemonicprobably need to add a note about min requirements for that test15:29
johnsomKeithMnemonic We wrote it specifically for cirros. That go app is run in all of our tempest tests15:29
KeithMnemonicif i run it, it says file not found15:31
KeithMnemonici can show you15:31
KeithMnemonici boot a cirros, manually copy httpd.bin and do ./httpd.bin and it says file not found15:31
KeithMnemonicfor ubuntu, opensuse it prints a help message15:32
johnsomWell, we renamed it when we added UDP15:32
KeithMnemonicon rocky it is still httpd.bin15:32
johnsomYeah. It's a static linked, so has no other dependencies. I assume you are getting it from that directory in that repo?15:33
KeithMnemonichmm let me see where it gets that from, maybe rpm-packaging15:33
johnsomnmickus That looks good to me15:34
KeithMnemonicrpm packaging gets this one
johnsomkeithmnemonic Have you tried running it with the full path? Cirros runs with a very basic shell, maybe that is the issue you have?15:35
KeithMnemonicwhich still has httpd and not test_server15:36
johnsomYeah, we only renamed it when we added UDP to it15:36
johnsomYeah, that is the rocky build. It works fine in our tempest gate jobs....15:37
KeithMnemonic$ /home/cirros/httpd.bin15:37
KeithMnemonic-sh: /home/cirros/httpd.bin: not found15:37
johnsomWhat does ls -al give?15:37
KeithMnemonic-rwxr-xr-x    1 cirros   cirros     4293448 May  5 17:06 httpd.bin15:39
KeithMnemonicsame binary works fine on opensuse15:39
openstackgerritCarlos Goncalves proposed openstack/octavia stable/rocky: Do not run Tempest in octavia-grenade job
KeithMnemonicopensuse@leap3:~> /home/opensuse/httpd.bin -h15:40
KeithMnemonicUsage of /home/opensuse/httpd.bin:15:40
KeithMnemonic  -id string15:40
KeithMnemonic        Server ID (default "1")15:40
KeithMnemonic  -port int15:40
KeithMnemonic        TCP port to listen on (default 8080)15:40
KeithMnemonicand as i said fine on ubuntu as well15:40
johnsomHere is how we install and run in the gate job:
KeithMnemoniconce i swapped the tempest image to an opensuse it got past the check_webserver status error it was getting15:41
johnsomMaybe that version has the Go problem where they ballooned the memory required. Try "echo 1 > /proc/sys/vm/overcommit_memory" before you run it15:41
*** gcheresh has quit IRC15:41
KeithMnemonichere is the issue15:43
KeithMnemonic$ ldd  /home/cirros/httpd.bin15:43
KeithMnemonicchecking sub-depends for '/lib64/'15:43
KeithMnemonic => /lib/ (0x7fa0a17df000)15:43
KeithMnemonic => /lib/ (0x7fa0a15db000)15:43
KeithMnemonic => /lib/ (0x7fa0a1371000)15:43
KeithMnemonicchecking sub-depends for 'not found'15:43
KeithMnemonic => /lib64/ (0x00000000)15:43
KeithMnemonic => not found (0x00000000)15:43
KeithMnemonic        /lib/ => /lib/ (0x00000000)15:43
KeithMnemoniccan find libc in the default cirros15:43
KeithMnemonici checked a cirros .5 as well15:43
johnsomWe staticly compile it, so it should not be looking for shared libraries.15:46
KeithMnemoniclet me try the current version15:47
KeithMnemonicbut rpm-packaging is not pulling the current one15:47
johnsom$ file httpd.bin15:48
johnsomhttpd.bin: ELF 64-bit LSB executable, x86-64, version 1 (GNU/Linux), statically linked, for GNU/Linux 2.6.32, BuildID[sha1]=aa8b01634784c9783c1e38db9422c21f3c4e097c, stripped15:48
johnsomI serious, we run that in cirros for every patch on stable/rocky. There is something strange with your version15:48
johnsomIt also should be larger in size. I think someone recompiled it wrong for you.15:49
johnsom$ ls -al httpd.bin15:49
johnsom-rwxrwxr-x 1 johnsom johnsom 6421240 May  6 08:47 httpd.bin15:49
KeithMnemonichmm that is interesting, let me go recheck that15:50
KeithMnemonicthe rpm pkg has a bigger size as well15:50
KeithMnemonicand that is what we install from15:50
KeithMnemonicdoes this look wrong?
johnsomYes. Looks like someone didn't read the README15:52
KeithMnemonici can patch rpm-packaging15:53
KeithMnemonicthanks johnsom: for helping me track this down15:55
johnsomSure, NP15:55
johnsom#startmeeting Octavia16:01
openstackMeeting started Wed May  6 16:01:00 2020 UTC and is due to finish in 60 minutes.  The chair is johnsom. Information about MeetBot at
*** maciejjozefczyk has quit IRC16:01
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:01
*** openstack changes topic to " (Meeting topic: Octavia)"16:01
openstackThe meeting name has been set to 'octavia'16:01
johnsom#chair rm_work16:01
openstackCurrent chairs: johnsom rm_work16:01
johnsomI am not sure if rm_work is around to host today or not.  Next week is his last week as PTL for Ussuri....16:02
johnsom#topic Announcements16:02
*** openstack changes topic to "Announcements (Meeting topic: Octavia)"16:02
johnsomIf you did not see the e-mail, the PTG is open for registration now:16:03
johnsomThere is no fee to register and it will be virtual.16:03
johnsomReminder we have an etherpad:16:04
johnsomDate and times are on the etherpad as well.16:05
johnsomThe only other announcement I have is that I think our RC1 release for Ussuri is going to be our final release version.16:06
*** maciejjozefczyk has joined #openstack-lbaas16:06
johnsomI haven't heard of any critical patches.16:06
johnsomSo, happy dance for that.16:06
johnsomAny other announcements today?16:06
cgoncalves+1 for RC1 being our final release16:06
*** maciejjozefczyk_ has joined #openstack-lbaas16:07
johnsom#topic Brief progress reports / bugs needing review16:08
*** openstack changes topic to "Brief progress reports / bugs needing review (Meeting topic: Octavia)"16:08
johnsomI have been working on small backport related patches and miscellaneous stuff.16:09
johnsom I have a draft slide deck for the OpenStack "highlights" meeting next week. Ping me if you would like to review, etc.16:09
johnsomI have also started looking at porting the failover patch to the V2 code path. It probably won't be a lot of code, but is going to be slow going.16:10
johnsomAny other updates from anyone?16:10
*** maciejjozefczyk has quit IRC16:11
cgoncalvesI proposed a couple of backport patches, proposed a patch that moves grenade job to native Zuul v3 and fix grenade in stable branches16:11
johnsomNice, yeah, happy to see grenade is finally native zuul v316:12
cgoncalvesthe V2 code path is exciting! it is one step forward to defaulting to amphorav216:12
ataraday_if some help needed with that I'm around16:12
johnsomataraday_ Thank you. I was trying to wrap my head around the deciders changes yesterday. I may have some questions there.16:13
johnsomAh, one other item of note. This week is the last week for the NDSU student course.16:15
johnsomHopefully they will be interested and continue on with Octavia, grin, but ....16:15
johnsomThey did get a lot of work done, so happy about that process.16:16
johnsomWe will probably have some work left to wrap up.16:16
cgoncalvescongratulations for all you achieved and work in Octavia!16:16
cgoncalvesthanks also to Michael and Adam for mentorship16:17
johnsomAnd Kendall with the OpenStack foundation16:17
cgoncalvesyes, sorry16:17
johnsomAny other updates today?16:17
johnsom#topic Open Discussion16:18
*** openstack changes topic to "Open Discussion (Meeting topic: Octavia)"16:18
johnsomHow about other topics?16:18
johnsomThere is a proposed spec for Active/Active open:16:18
johnsomIf you have some time please review/comment16:19
johnsomI have only had time for a quick glance.16:19
johnsomSeems to be an ECMP based proposal16:19
johnsomAnything else today?16:20
johnsomOk then, thanks everyone!16:21
*** openstack changes topic to "Discussions for OpenStack Octavia | Priority bug review list:"16:21
openstackMeeting ended Wed May  6 16:21:37 2020 UTC.  Information about MeetBot at . (v 0.1.4)16:21
openstackMinutes (text):
openstackgerritMerged openstack/octavia-dashboard stable/ussuri: Imported Translations from Zanata
*** rpittau is now known as rpittau|afk16:24
*** psachin has quit IRC16:41
*** ataraday_ has quit IRC17:10
*** maciejjozefczyk has joined #openstack-lbaas17:18
*** ccamposr has joined #openstack-lbaas17:19
*** maciejjozefczyk_ has quit IRC17:20
*** irclogbot_1 has quit IRC17:20
*** maciejjozefczyk has quit IRC17:21
*** maciejjozefczyk has joined #openstack-lbaas17:21
*** ccamposr__ has quit IRC17:21
*** maciejjozefczyk_ has joined #openstack-lbaas17:22
*** irclogbot_2 has joined #openstack-lbaas17:23
*** maciejjozefczyk_ has quit IRC17:23
johnsomI am guessing we are not validating the networks in the AZ profile and we aren't expecting the "name"17:24
*** maciejjozefczyk has quit IRC17:26
*** salmankhan has quit IRC17:36
*** livelace has quit IRC17:44
*** livelace has joined #openstack-lbaas18:03
openstackgerritCarlos Goncalves proposed openstack/octavia stable/rocky: Fix Rocky v2 scenario and grenade jobs
openstackgerritNoah Mickus proposed openstack/python-octaviaclient master: Add the ability to specify TLS protocols for a pool
openstackgerritNoah Mickus proposed openstack/python-octaviaclient master: Add the ability to specify TLS protocols for a pool
*** gcheresh has joined #openstack-lbaas19:41
*** jamesdenton has quit IRC19:48
*** gcheresh has quit IRC19:56
*** jamesdenton has joined #openstack-lbaas20:04
*** gcheresh has joined #openstack-lbaas20:04
*** jamesden_ has joined #openstack-lbaas20:08
*** jamesdenton has quit IRC20:11
openstackgerritNoah Mickus proposed openstack/python-octaviaclient master: Add the ability to specify TLS protocols for a pool
openstackgerritNoah Mickus proposed openstack/python-octaviaclient master: Add the ability to specify TLS protocols for a pool
*** jamesdenton has joined #openstack-lbaas20:21
rm_workjohnsom: errr, no, we SHOULD be validating the networks? let me see, I swear i added code for that20:21
rm_workdefinitely not expecting name tho, for sure ID20:22
johnsomYeah, mgmt-net also accepts names and later bombs20:22
*** jamesden_ has quit IRC20:22
*** gcheresh has quit IRC20:43
openstackgerritMerged openstack/octavia master: Add py38 package metadata
*** servagem has quit IRC20:57
nmickusjohnsom for the tls pools patch I'm getting a patch in mere conflict error21:38
johnsomnmickus So, that means the parent patch also changed something that conflicted/overlapped with your patch. Just pull your patch down and rebase it. It will stop in the middle of the rebase, from there you can do "git status" and get a list of the files with conflicts.  Search for the <<<<< and >>>>> tags, "merge" the changes, then git add the file. Once you are done with all of those, you can do the git rebase21:56
johnsom--continue, then git review21:56
johnsomPing me if you want to do a hangout and fix it together21:57
nmickusjohnsom is there anything special i need to do with the merge commit?21:59
nmickusdo i need to do anything special to make sure its part of the same patch?22:03
johnsomNo, the change ID in the commit message should tie them together22:08
openstackgerritNoah Mickus proposed openstack/python-octaviaclient master: Add the ability to specify TLS protocols for a pool
nmickusjohnsom Same issue :/22:11
johnsomHmm, that is interesting. Let me look at the patch for a minute22:13
johnsomnmickus I wonder if it is the depends-on patch somehow22:15
openstackgerritMichael Johnson proposed openstack/octavia master: Add TLS version configuration for listeners
nmickusIt seemed weird but i though some additions to the master since I'd started it22:16
johnsomnmickus It is this patch:
johnsomThe one your patch depends on. I rebased it's parent, then when I went to rebase that patch it is showing conflict22:17
johnsomSo, rebase 721773, then recheck the client patch22:17
nmickusso i need to re base damson's patch on my machine?22:19
*** threestrands has joined #openstack-lbaas22:20
johnsomI can do it real quick if you would like, but yeah, one of us needs to rebase it22:20
nmickusI can take a shot at it22:21
nmickusjohnsom while rebasing it does not list any conflicts22:28
johnsomDid you rebase it on the parent patch or to master?22:29
nmickus i just did git rebase22:29
nmickusdo i need to git rebase master?22:29
johnsomYeah, ok, so you rebased it onto master22:30
johnsomYou changed the parent of that patch. You can confirm by doing a "git log", if the second log entry is for "837b7bf94092438e486d3e18c5b9a21bb1298f98" it is now parented on master22:31
johnsomIt likely will not pass the tests that way22:31
johnsomYou probably need to rebase --onto bbfff55bafc5c557924b4fc93b7d774a790e9e3122:32
nmickusit says 41ba26f8aa77ad3ae07f6abd6c044bdce876d2e5 is the second patch in the log entry22:32
nmickusI have not pushed anything up yet22:33
johnsomyeah, um. I don't know what that patch is...22:33
johnsomYeah, searching for that SHA I'm not finding one.22:34
johnsomMaybe just try the rebase --onto above?22:35
nmickusthe commit message looks like dawsons listener patch22:35
nmickuswill do22:35
johnsomHmm, did you make changes to his listener patch locally?22:36
nmickusNot that i'm aware of22:36
johnsomHow odd. I just don't see that SHA in gerrit. Well, see where the rebase --onto takes you22:36
nmickusDoes not point to a valid commit22:37
johnsomSo it's not in the local repo, check this out: "git fetch refs/changes/62/721362/8 && git checkout FETCH_HEAD" and put it on another branch22:38
nmickushow would i put it on another branch?22:38
johnsomI was just going to type "git checkout -b <new branch name>"22:39
johnsomgit branch will show all of the branches you have22:39
johnsomgit checkout <branch name> will let you switch between them22:39
nmickusalright it's checked out on its on branch22:41
johnsomNow you can switch back to the branch to rebase, then do the rebase onto and it should find it22:42
nmickusrebase the new branch onto the old one?22:43
johnsomYou will rebase onto
nmickusalright i have a list of conflicts22:48
*** abaindur has quit IRC22:52
*** TrevorV has quit IRC22:54
rm_workI would have done:23:07
rm_workgit review -d 72136223:07
rm_workgit review -x 72177323:07
rm_workthat's the cleanest most gerrit-ey way to rebase onto a parent23:07
*** livelace has quit IRC23:07
nmickusjohnsom I finished with all of the conflicts and now its warning me about multiple commits is it alright to go ahead?23:44

Generated by 2.17.2 by Marius Gedminas - find it at!