| opendevreview | Takashi Kajinami proposed openstack/octavia master: DNM: Checking gate https://review.opendev.org/c/openstack/octavia/+/803426 | 08:11 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/octavia master: Enable debug of pip-extra-reqs/pip-missing-reqs https://review.opendev.org/c/openstack/octavia/+/803461 | 11:00 |
| opendevreview | Pierre-Yves Jourel proposed openstack/octavia master: Implements notifications for octavia https://review.opendev.org/c/openstack/octavia/+/803362 | 12:53 |
| rm_work | ah did we cancel the meeting for TWO weeks? | 16:03 |
| rm_work | I guess so :D | 16:03 |
| digitalsimboja | Hello Johnsom | 17:42 |
| digitalsimboja | Please take a look at the Gerrit patchset | 17:42 |
| digitalsimboja | https://review.opendev.org/c/openstack/kuryr-tempest-plugin/+/803244 | 17:42 |
| johnsom | Hi | 17:42 |
| digitalsimboja | I added the octavia-tempest-plugin on the zuul.yaml | 17:43 |
| digitalsimboja | But still throws errors | 17:43 |
| johnsom | That can only go on "- job:" sections, not the "- project:" section | 17:43 |
| digitalsimboja | On previous patchsets I added it to the last stable branch | 17:43 |
| digitalsimboja | Okay let me chack | 17:43 |
| digitalsimboja | check | 17:44 |
| johnsom | digitalsimboja You may need to put it in here: https://opendev.org/openstack/kuryr-kubernetes/src/branch/master/.zuul.d/octavia.yaml | 17:46 |
| johnsom | Though I'm not sure which job(s) you need it in | 17:47 |
| digitalsimboja | kuryr-kubernetes-tempest | 17:47 |
| digitalsimboja | I need to add it on the kuryr-kubernetes-tempest one | 17:48 |
| johnsom | So this one: https://opendev.org/openstack/kuryr-kubernetes/src/branch/master/.zuul.d/octavia.yaml#L67 | 17:48 |
| johnsom | Adding it to kuryr-kubernetes-tempest-octavia-base would include it there | 17:49 |
| digitalsimboja | okay amending... | 17:52 |
| digitalsimboja | The repo is the main Kuryr-kubernetes repo? Confirm | 18:00 |
| johnsom | Yeah, it looks like kuryr is spliting tests across repos | 18:01 |
| digitalsimboja | so I need to commit that change and get it merged before it can effect the change? | 18:03 |
| digitalsimboja | I guess? | 18:03 |
| digitalsimboja | Otherwise I would get the 'mananger' issue? | 18:04 |
| johnsom | You can also use "Depends-On: <patch URL" in your plugin patch commit message. | 18:08 |
| digitalsimboja | So I have to make the patch on the main repo then | 18:11 |
| digitalsimboja | to get the URL | 18:11 |
| digitalsimboja | Thanks | 18:11 |
| *** gregraka is now known as gregraka-afk | 21:40 | |
| *** gregraka-afk is now known as gregraka | 21:40 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!