| *** lujinluo has joined #openstack-ironic | 00:04 | |
| *** rloo has quit IRC | 00:09 | |
| *** rama_y has quit IRC | 00:09 | |
| *** jcoufal has quit IRC | 00:19 | |
| *** hshiina has joined #openstack-ironic | 00:19 | |
| *** jcoufal has joined #openstack-ironic | 00:20 | |
| *** adu has joined #openstack-ironic | 00:21 | |
| *** sdake has quit IRC | 00:23 | |
| *** hwcomcn has joined #openstack-ironic | 00:24 | |
| openstackgerrit | Merged openstack/ironic: Log full config only once in conductor https://review.openstack.org/348459 | 00:26 |
|---|---|---|
| *** jcoufal has quit IRC | 00:28 | |
| *** adu has quit IRC | 00:37 | |
| *** jcoufal has joined #openstack-ironic | 00:39 | |
| *** hoangcx has joined #openstack-ironic | 00:44 | |
| *** hwcomcn has quit IRC | 00:55 | |
| *** hshiina has quit IRC | 00:56 | |
| *** hoangcx2 has quit IRC | 00:58 | |
| *** jaybeale has joined #openstack-ironic | 01:01 | |
| *** lujinluo has quit IRC | 01:06 | |
| *** lujinluo has joined #openstack-ironic | 01:06 | |
| *** jcoufal has quit IRC | 01:06 | |
| *** jcoufal has joined #openstack-ironic | 01:08 | |
| *** jaybeale has quit IRC | 01:12 | |
| *** sdake has joined #openstack-ironic | 01:13 | |
| *** sdake has quit IRC | 01:13 | |
| *** sdake has joined #openstack-ironic | 01:13 | |
| *** sdake_ has joined #openstack-ironic | 01:15 | |
| *** sdake has quit IRC | 01:18 | |
| *** jcoufal has quit IRC | 01:20 | |
| *** jcoufal has joined #openstack-ironic | 01:20 | |
| *** harshs has quit IRC | 01:26 | |
| *** bvandewa has quit IRC | 01:28 | |
| *** harshs has joined #openstack-ironic | 01:28 | |
| *** yibo has joined #openstack-ironic | 01:31 | |
| *** yibo has quit IRC | 01:35 | |
| *** yibo has joined #openstack-ironic | 01:35 | |
| *** yibo has quit IRC | 01:35 | |
| *** yibo has joined #openstack-ironic | 01:38 | |
| *** harshs has quit IRC | 01:49 | |
| *** jcoufal has quit IRC | 01:51 | |
| *** jcoufal has joined #openstack-ironic | 01:52 | |
| *** lujinluo has quit IRC | 02:11 | |
| *** lujinluo has joined #openstack-ironic | 02:11 | |
| *** jcoufal has quit IRC | 02:12 | |
| *** PollyZ has joined #openstack-ironic | 02:12 | |
| *** PollyZ has quit IRC | 02:18 | |
| *** sdake_ has quit IRC | 02:19 | |
| *** PollyZ has joined #openstack-ironic | 02:26 | |
| *** joprovost has quit IRC | 02:28 | |
| *** PollyZ has quit IRC | 02:31 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements https://review.openstack.org/348672 | 02:34 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/348673 | 02:34 |
| *** yibo has quit IRC | 02:38 | |
| *** yibo has joined #openstack-ironic | 02:38 | |
| *** PollyZ has joined #openstack-ironic | 02:40 | |
| *** PollyZ has quit IRC | 02:44 | |
| *** harshs has joined #openstack-ironic | 02:48 | |
| *** PollyZ has joined #openstack-ironic | 02:53 | |
| *** sabeen1 has quit IRC | 02:55 | |
| *** yuanying_ has quit IRC | 02:57 | |
| *** sabeen has joined #openstack-ironic | 02:58 | |
| *** PollyZ has quit IRC | 02:58 | |
| *** thrash is now known as thrash|g0ne | 03:00 | |
| *** PollyZ has joined #openstack-ironic | 03:07 | |
| *** PollyZ has quit IRC | 03:11 | |
| *** joprovost has joined #openstack-ironic | 03:14 | |
| *** PollyZ has joined #openstack-ironic | 03:20 | |
| *** chihhsin has quit IRC | 03:25 | |
| *** lujinluo has quit IRC | 03:25 | |
| *** PollyZ has quit IRC | 03:25 | |
| *** harshs has quit IRC | 03:32 | |
| *** PollyZ has joined #openstack-ironic | 03:34 | |
| *** chihhsin has joined #openstack-ironic | 03:37 | |
| *** PollyZ has quit IRC | 03:39 | |
| *** Sukhdev has joined #openstack-ironic | 03:40 | |
| *** PollyZ has joined #openstack-ironic | 03:48 | |
| *** appu103_ has joined #openstack-ironic | 03:49 | |
| *** yuanying has joined #openstack-ironic | 03:51 | |
| *** PollyZ has quit IRC | 03:52 | |
| *** lujinluo has joined #openstack-ironic | 03:59 | |
| *** chihhsin has quit IRC | 03:59 | |
| *** chihhsin has joined #openstack-ironic | 04:00 | |
| *** PollyZ has joined #openstack-ironic | 04:01 | |
| *** PollyZ has quit IRC | 04:06 | |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils: Discover Storage details https://review.openstack.org/341370 | 04:08 |
| *** vmud213 has joined #openstack-ironic | 04:12 | |
| *** links has joined #openstack-ironic | 04:13 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Ipmitool management driver for Inject NMI https://review.openstack.org/348721 | 04:14 |
| *** PollyZ has joined #openstack-ironic | 04:15 | |
| *** joprovost has quit IRC | 04:18 | |
| *** joprovost has joined #openstack-ironic | 04:19 | |
| *** PollyZ has quit IRC | 04:20 | |
| *** Sukhdev has quit IRC | 04:24 | |
| *** Sukhdev has joined #openstack-ironic | 04:25 | |
| *** PollyZ has joined #openstack-ironic | 04:28 | |
| *** vmud213 has quit IRC | 04:29 | |
| *** Sukhdev has quit IRC | 04:29 | |
| *** vmud213 has joined #openstack-ironic | 04:29 | |
| *** PollyZ has quit IRC | 04:34 | |
| *** krtaylor has joined #openstack-ironic | 04:35 | |
| *** livelace has joined #openstack-ironic | 04:54 | |
| *** PollyZ has joined #openstack-ironic | 05:09 | |
| *** PollyZ has quit IRC | 05:14 | |
| *** amitkqed has quit IRC | 05:19 | |
| *** amitkqed has joined #openstack-ironic | 05:20 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: iRMC management driver for Inject NMI https://review.openstack.org/348732 | 05:21 |
| *** PollyZ has joined #openstack-ironic | 05:36 | |
| *** PollyZ has quit IRC | 05:41 | |
| *** amotoki has joined #openstack-ironic | 05:46 | |
| openstackgerrit | Swapnil Kulkarni (coolsvap) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 05:49 |
| *** PollyZ has joined #openstack-ironic | 05:50 | |
| *** Fdaisuke has joined #openstack-ironic | 05:52 | |
| *** _Fdaisuke_ has quit IRC | 05:54 | |
| *** PollyZ has quit IRC | 05:55 | |
| openstackgerrit | Teng Fei proposed openstack/ironic-inspector: Modify the SQL's string from double quote to single quote. https://review.openstack.org/347795 | 05:58 |
| *** harlowja_at_home has joined #openstack-ironic | 05:59 | |
| *** PollyZ has joined #openstack-ironic | 06:04 | |
| *** PollyZ has quit IRC | 06:09 | |
| *** e0ne has joined #openstack-ironic | 06:12 | |
| *** PollyZ has joined #openstack-ironic | 06:17 | |
| *** harlowja_at_home has quit IRC | 06:20 | |
| *** mjura has joined #openstack-ironic | 06:22 | |
| *** PollyZ has quit IRC | 06:22 | |
| *** mjura has quit IRC | 06:29 | |
| *** mjura has joined #openstack-ironic | 06:29 | |
| *** wlfightup has joined #openstack-ironic | 06:30 | |
| *** PollyZ has joined #openstack-ironic | 06:31 | |
| wlfightup | Hello everybody, I'm from Hong Kong. Nice to meet you | 06:33 |
| wlfightup | I am pleased to learn with you. I hope you will take care of me | 06:33 |
| *** loki_ has joined #openstack-ironic | 06:36 | |
| *** PollyZ has quit IRC | 06:36 | |
| *** tesseract- has joined #openstack-ironic | 06:37 | |
| *** hoangcx has quit IRC | 06:42 | |
| *** PollyZ has joined #openstack-ironic | 06:44 | |
| *** pcaruana has joined #openstack-ironic | 06:45 | |
| *** PollyZ has quit IRC | 06:50 | |
| *** hoangcx has joined #openstack-ironic | 06:55 | |
| openstackgerrit | David Lenwell proposed openstack/ironic: Add REST API for volume connector operation https://review.openstack.org/214586 | 06:57 |
| *** Fdaisuke has quit IRC | 06:59 | |
| *** ccamacho has joined #openstack-ironic | 06:59 | |
| *** Fdaisuke has joined #openstack-ironic | 07:00 | |
| *** appu103_ is now known as aparnav | 07:01 | |
| openstackgerrit | David Lenwell proposed openstack/ironic: Add REST API for volume connector operation https://review.openstack.org/214586 | 07:02 |
| *** zhenguo has joined #openstack-ironic | 07:04 | |
| *** bvandewa has joined #openstack-ironic | 07:16 | |
| *** Fdaisuke has quit IRC | 07:17 | |
| *** Fdaisuke has joined #openstack-ironic | 07:17 | |
| *** vishwanathj has quit IRC | 07:21 | |
| *** davidlenwell has quit IRC | 07:22 | |
| *** PollyZ has joined #openstack-ironic | 07:25 | |
| *** mkrai_ has joined #openstack-ironic | 07:27 | |
| *** PollyZ has quit IRC | 07:30 | |
| *** ccamacho is now known as ccamacho|afk | 07:33 | |
| *** davidlenwell has joined #openstack-ironic | 07:34 | |
| *** PollyZ has joined #openstack-ironic | 07:39 | |
| *** PollyZ has quit IRC | 07:43 | |
| *** bvandewa has quit IRC | 07:45 | |
| *** hoangcx has quit IRC | 07:48 | |
| *** hoangcx has joined #openstack-ironic | 07:50 | |
| *** ccamacho|afk is now known as ccamacho | 07:51 | |
| *** jpich has joined #openstack-ironic | 07:55 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:00 | |
| *** e0ne has quit IRC | 08:00 | |
| openstackgerrit | David Lenwell proposed openstack/ironic: Add REST API for volume connector operation https://review.openstack.org/214586 | 08:01 |
| *** jralph_ has joined #openstack-ironic | 08:01 | |
| *** jralph has quit IRC | 08:04 | |
| lucasagomes | morning all | 08:05 |
| tangchen | Hi, Lucas | 08:06 |
| tangchen | morning | 08:06 |
| *** PollyZ has joined #openstack-ironic | 08:06 | |
| tangchen | lucasagomes: About the functional test issue, I have a question if you have time. | 08:07 |
| lucasagomes | tangchen, sure, if I know the answer :-) | 08:07 |
| *** fragatina has quit IRC | 08:09 | |
| *** derekh has joined #openstack-ironic | 08:09 | |
| tangchen | lucasagomes: When I was trying to add some basic classes and functions for functional test, I found that in the api unit test, it actually started a server using pecan. | 08:09 |
| tangchen | lucasagomes: And it actually sent the api requests to the server and got the responses. | 08:10 |
| lucasagomes | right | 08:11 |
| tangchen | lucasagomes: So it looks more like functional tests. Am I right ? | 08:11 |
| *** PollyZ has quit IRC | 08:11 | |
| lucasagomes | true, unittests should probably mock everything | 08:11 |
| tangchen | lucasagomes: OK, if so, then what shall we expect the real functional tests do ? Or shall we move the unit test part to the functional dir ? | 08:12 |
| lucasagomes | but still it's kinda flaky to actually guarantee that we can run it all with py3, no? I was thinking on running the tempest suit against it | 08:12 |
| lucasagomes | tangchen, we could fix those unittests to mock stuff instead of start an api server, but that would be a different problem IMO | 08:13 |
| *** electrofelix has joined #openstack-ironic | 08:13 | |
| lucasagomes | I was more thinking in the line of having jobs like pxe_{ipa, ssh} etc... running on python 3 | 08:13 |
| tangchen | lucasagomes: Yes. That is different. What I am thinking is that maybe we need to split the unit and functional tests first. They are now mixed. | 08:15 |
| lucasagomes | tangchen, that would be nice, but I would look at it as a different problem | 08:15 |
| lucasagomes | both can be done in parallel if needed | 08:15 |
| lucasagomes | and both needs fixing yeah | 08:16 |
| tangchen | lucasagomes: OK. So, I'll report another bug and link them together. :) | 08:16 |
| lucasagomes | tangchen, cool, thanks a lot! | 08:16 |
| tangchen | lucasagomes: Thanks for the info. :) | 08:16 |
| lucasagomes | yw | 08:16 |
| tangchen | lucasagomes: BTW, I modified the description of the functional test REF. https://bugs.launchpad.net/ironic/+bug/1491670 | 08:19 |
| openstack | Launchpad bug 1491670 in Ironic "[RFE] Ironic functional tests and hacking checks" [Wishlist,In progress] - Assigned to Tang Chen (tangchen) | 08:19 |
| tangchen | lucasagomes: Please help to review if you have any comments on it. :) | 08:19 |
| lucasagomes | tangchen, a-ha will do | 08:19 |
| tangchen | Thx | 08:19 |
| *** PollyZ has joined #openstack-ironic | 08:20 | |
| *** PollyZ has quit IRC | 08:25 | |
| tangchen | lucasagomes: Done. https://bugs.launchpad.net/ironic/+bug/1607679 :) | 08:30 |
| openstack | Launchpad bug 1607679 in Ironic "[REF] Split unit and functional tests" [Undecided,New] - Assigned to Tang Chen (tangchen) | 08:30 |
| lucasagomes | o/ cheers | 08:30 |
| *** mkrai_ has quit IRC | 08:31 | |
| *** bvandewa has joined #openstack-ironic | 08:32 | |
| *** PollyZ has joined #openstack-ironic | 08:33 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Updated tests for db migration scripts https://review.openstack.org/344246 | 08:34 |
| *** PollyZ has quit IRC | 08:38 | |
| *** daemontool has joined #openstack-ironic | 08:44 | |
| *** PollyZ has joined #openstack-ironic | 08:47 | |
| *** PollyZ has quit IRC | 08:52 | |
| *** e0ne has joined #openstack-ironic | 08:52 | |
| *** amotoki has quit IRC | 08:56 | |
| *** amotoki has joined #openstack-ironic | 09:00 | |
| *** PollyZ has joined #openstack-ironic | 09:00 | |
| *** PollyZ has quit IRC | 09:05 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node provision state change notification https://review.openstack.org/348437 | 09:05 |
| *** e0ne has quit IRC | 09:05 | |
| *** mgould|afk is now known as mgould | 09:11 | |
| mgould | morning Ironic! | 09:11 |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic power I/F for soft reboot and soft power off https://review.openstack.org/216730 | 09:15 |
| *** mbound has joined #openstack-ironic | 09:17 | |
| *** bvandewa has quit IRC | 09:27 | |
| *** PollyZ has joined #openstack-ironic | 09:28 | |
| *** vmud213 has quit IRC | 09:30 | |
| *** vmud213 has joined #openstack-ironic | 09:30 | |
| *** PollyZ has quit IRC | 09:32 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Ipmitool power driver for soft reboot and soft power off https://review.openstack.org/216738 | 09:37 |
| *** fragatina has joined #openstack-ironic | 09:38 | |
| *** PollyZ has joined #openstack-ironic | 09:41 | |
| openstackgerrit | David Lenwell proposed openstack/ironic: Add REST API for volume connector operation https://review.openstack.org/214586 | 09:44 |
| *** PollyZ has quit IRC | 09:46 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: iRMC power driver for soft reboot and soft power off https://review.openstack.org/216743 | 09:50 |
| *** PollyZ has joined #openstack-ironic | 09:55 | |
| *** Romanenko_K has joined #openstack-ironic | 09:56 | |
| *** sambetts|afk is now known as sambetts | 09:58 | |
| sambetts | Morning all | 09:58 |
| *** PollyZ has quit IRC | 10:00 | |
| *** e0ne has joined #openstack-ironic | 10:01 | |
| mgould | morning sambetts | 10:06 |
| *** e0ne has quit IRC | 10:12 | |
| *** milan has quit IRC | 10:16 | |
| mgould | is it possible to stop Gerrit from emailing me every time someone comments on a review I've commented on? | 10:19 |
| *** milan has joined #openstack-ironic | 10:25 | |
| *** sdake has joined #openstack-ironic | 10:25 | |
| *** mbound has quit IRC | 10:31 | |
| *** hoangcx has quit IRC | 10:33 | |
| *** hoangcx has joined #openstack-ironic | 10:34 | |
| jroll | mgould: heavy filtering :D | 10:37 |
| jroll | morning y'all | 10:37 |
| sambetts | mgould, jroll: morning | 10:37 |
| mgould | morning jroll! | 10:37 |
| * jroll looks at folders... Gerrit (47446) | 10:38 | |
| mgould | also: dammit! | 10:38 |
| mgould | won't anyone think of the poor SMTP servers? | 10:38 |
| jroll | lol | 10:38 |
| sambetts | Looks like your not the only one that wants it https://bugs.chromium.org/p/gerrit/issues/detail?id=989' | 10:39 |
| sambetts | https://bugs.chromium.org/p/gerrit/issues/detail?id=989 | 10:39 |
| mgould | open since 2014 :-( | 10:40 |
| *** sabeen has quit IRC | 10:45 | |
| *** PollyZ has joined #openstack-ironic | 10:49 | |
| *** loki_ has quit IRC | 10:52 | |
| *** PollyZ has quit IRC | 10:54 | |
| *** vsaienk0 has quit IRC | 10:55 | |
| *** lujinluo has quit IRC | 10:56 | |
| *** e0ne has joined #openstack-ironic | 10:58 | |
| *** vsaienk0 has joined #openstack-ironic | 10:58 | |
| *** fragatina has quit IRC | 11:00 | |
| openstackgerrit | David Lenwell proposed openstack/ironic: Add REST API for volume target operation https://review.openstack.org/285221 | 11:01 |
| *** e0ne has quit IRC | 11:01 | |
| openstackgerrit | David Lenwell proposed openstack/ironic: Add REST API for volume connector operation https://review.openstack.org/214586 | 11:04 |
| openstackgerrit | David Lenwell proposed openstack/ironic: Add REST API for volume target operation https://review.openstack.org/285221 | 11:05 |
| openstackgerrit | David Lenwell proposed openstack/ironic: Add REST API for volume connector operation https://review.openstack.org/214586 | 11:08 |
| *** hoangcx2 has joined #openstack-ironic | 11:09 | |
| *** yuriyz has quit IRC | 11:10 | |
| *** Romanenko_K has quit IRC | 11:10 | |
| openstackgerrit | David Lenwell proposed openstack/ironic: Add REST API for volume target operation https://review.openstack.org/285221 | 11:10 |
| *** hoangcx has quit IRC | 11:13 | |
| *** lucasagomes is now known as lucas-hungry | 11:14 | |
| *** Romanenko_K has joined #openstack-ironic | 11:24 | |
| *** sdake has quit IRC | 11:27 | |
| *** ijw has joined #openstack-ironic | 11:28 | |
| *** PollyZ has joined #openstack-ironic | 11:30 | |
| *** ijw has quit IRC | 11:30 | |
| *** ijw has joined #openstack-ironic | 11:31 | |
| openstackgerrit | David Lenwell proposed openstack/ironic: Add REST API for volume target operation https://review.openstack.org/285221 | 11:31 |
| *** mbound has joined #openstack-ironic | 11:32 | |
| *** yuriyz has joined #openstack-ironic | 11:32 | |
| *** PollyZ has quit IRC | 11:35 | |
| openstackgerrit | David Lenwell proposed openstack/ironic: Add REST API for volume target operation https://review.openstack.org/285221 | 11:36 |
| *** mbound has quit IRC | 11:36 | |
| *** ijw has quit IRC | 11:37 | |
| *** tangchen has quit IRC | 11:40 | |
| *** tangchen_ has quit IRC | 11:40 | |
| *** thrash|g0ne is now known as thrash | 11:41 | |
| *** tangchen_ has joined #openstack-ironic | 11:42 | |
| *** tangchen has joined #openstack-ironic | 11:42 | |
| *** PollyZ has joined #openstack-ironic | 11:43 | |
| Romanenko_K | Hi, Ironic team! ) | 11:46 |
| Romanenko_K | Thanks for +2s and Workflow for patch about tests for provision state commands. This patch depends on following patch, which should be merged first, because baremetal_node_create() method is defined in it: https://review.openstack.org/#/c/301687/ | 11:48 |
| *** PollyZ has quit IRC | 11:48 | |
| *** e0ne has joined #openstack-ironic | 11:51 | |
| *** tangchen has quit IRC | 11:58 | |
| *** tangchen_ has quit IRC | 11:58 | |
| TheJulia | Good morning | 12:03 |
| *** hwcomcn has joined #openstack-ironic | 12:05 | |
| *** hwcomcn has quit IRC | 12:07 | |
| *** hwcomcn has joined #openstack-ironic | 12:07 | |
| *** hwcomcn has quit IRC | 12:09 | |
| *** jjohnson2 has joined #openstack-ironic | 12:09 | |
| *** hwcomcn has joined #openstack-ironic | 12:10 | |
| *** PollyZ has joined #openstack-ironic | 12:11 | |
| *** hwcomcn has quit IRC | 12:11 | |
| *** hwcomcn has joined #openstack-ironic | 12:12 | |
| *** hwcomcn has quit IRC | 12:13 | |
| *** trown|outtypewww is now known as trown | 12:13 | |
| sambetts | Morning TheJulia | 12:13 |
| *** hwcomcn has joined #openstack-ironic | 12:14 | |
| *** ijw has joined #openstack-ironic | 12:16 | |
| *** PollyZ has quit IRC | 12:16 | |
| *** MattMan has quit IRC | 12:17 | |
| *** MattMan has joined #openstack-ironic | 12:17 | |
| *** lucas-hungry is now known as lucasagomes | 12:20 | |
| lucasagomes | sambetts, TheJulia jroll morning | 12:20 |
| sambetts | Hey lucasagomes | 12:20 |
| jroll | morning lucasagomes :) | 12:21 |
| * TheJulia makes a sad frowny face as all the volume connector info stuff needs a rebasing. | 12:24 | |
| *** PollyZ has joined #openstack-ironic | 12:24 | |
| sambetts | jroll: I hope my refactor of the project-config stuff didn't break the multitenant job | 12:25 |
| sambetts | jroll: I saw your conversation in the backscroll, did it turn out to be a problem there? | 12:25 |
| jroll | sambetts: I think I got distracted, I couldn't find a problem in the code though... PROJECTS=ironic is in devstack-virtual-ironic which that job inherits from | 12:26 |
| openstackgerrit | Sam Betts proposed openstack/ironic: DO NOT MERGE Testing Multitenant Job https://review.openstack.org/348859 | 12:27 |
| *** ccamacho is now known as ccamacho|lunch | 12:28 | |
| sambetts | :/ jroll ^ we can use this for debugging | 12:28 |
| *** PollyZ has quit IRC | 12:29 | |
| *** appu1034 has joined #openstack-ironic | 12:37 | |
| *** aparnav has quit IRC | 12:37 | |
| *** PollyZ has joined #openstack-ironic | 12:38 | |
| *** thiagop has joined #openstack-ironic | 12:38 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add baremetal port set command to OSC plugin https://review.openstack.org/347287 | 12:39 |
| *** PollyZ has quit IRC | 12:43 | |
| *** ijw has quit IRC | 12:47 | |
| *** ijw has joined #openstack-ironic | 12:48 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add baremetal port unset command to OSC plugin https://review.openstack.org/347402 | 12:52 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add baremetal port set command to OSC plugin https://review.openstack.org/347287 | 12:52 |
| vdrok | sambetts: I think multitenant job won't run on that test change | 12:53 |
| thiagop | good morning, Ironicers | 12:53 |
| thiagop | TGIF | 12:53 |
| vdrok | sambetts: because of https://github.com/openstack-infra/project-config/blob/master/zuul/layout.yaml#L1191 | 12:53 |
| vdrok | morning thaig | 12:53 |
| sambetts | vdrok: I thought it was running on all changes now? or is it an experimental? | 12:53 |
| vdrok | ugh, morning thiagop :) | 12:53 |
| thiagop | vdrok: o/ | 12:53 |
| sambetts | vdrok: ooooooh | 12:53 |
| sambetts | vdrok: I see | 12:54 |
| vdrok | sambetts: yup | 12:54 |
| vdrok | you just need to change a line in some code file | 12:54 |
| openstackgerrit | Sam Betts proposed openstack/ironic: DO NOT MERGE Testing Multitenant Job https://review.openstack.org/348859 | 12:54 |
| sambetts | vdrok: updated ^ | 12:54 |
| vdrok | +2 :D | 12:55 |
| thiagop | speaking of it vdrok, should the ironic filter be filtering test-only patches? | 12:55 |
| thiagop | I find it awkward... | 12:56 |
| thiagop | L1195 | 12:56 |
| vdrok | thiagop: hm, why? if changes are only to unittests, only unittest/pep8 jobs are run | 12:57 |
| vdrok | and we don't have functional tests yet | 12:57 |
| *** vmud213 has quit IRC | 12:57 | |
| *** fumie has quit IRC | 12:57 | |
| thiagop | vdrok: aw, right. Because our functional tests are inside the tempest plugin | 12:57 |
| thiagop | gotcha | 12:57 |
| *** vmud213 has joined #openstack-ironic | 12:57 | |
| *** jcoufal has joined #openstack-ironic | 12:58 | |
| vdrok | btw, small project-config change - https://review.openstack.org/#/c/345521/ | 13:04 |
| vdrok | brb | 13:04 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Introduce VolumeConnector object https://review.openstack.org/214584 | 13:04 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Introduce new RPCs to support volume connector operation https://review.openstack.org/214585 | 13:04 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Introduce new RPCs to support volume target operations https://review.openstack.org/285220 | 13:04 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Introduce VolumeTarget object https://review.openstack.org/285219 | 13:04 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add volume_targets table to save target information https://review.openstack.org/285218 | 13:04 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add volume_connector table to save connector information https://review.openstack.org/200983 | 13:04 |
| *** links has quit IRC | 13:07 | |
| *** ccamacho|lunch is now known as ccamacho | 13:08 | |
| *** ianw has quit IRC | 13:10 | |
| *** ianw has joined #openstack-ironic | 13:10 | |
| *** ijw has quit IRC | 13:10 | |
| xavierr | good morning ironic :) | 13:16 |
| xavierr | it's friday again o/ | 13:16 |
| TheJulia | good morning xavierr | 13:17 |
| mgould | morning xavierr thiagop! | 13:17 |
| *** rbudden has joined #openstack-ironic | 13:18 | |
| thiagop | mgould: o/ | 13:18 |
| *** PollyZ has joined #openstack-ironic | 13:19 | |
| xavierr | mgould TheJulia :D | 13:19 |
| *** mjura has quit IRC | 13:23 | |
| *** PollyZ has quit IRC | 13:23 | |
| mgould | morning TheJulia | 13:25 |
| TheJulia | good morning mgould | 13:26 |
| *** appu1034 has quit IRC | 13:31 | |
| *** PollyZ has joined #openstack-ironic | 13:32 | |
| *** joprovost has quit IRC | 13:35 | |
| *** mbound has joined #openstack-ironic | 13:36 | |
| *** PollyZ has quit IRC | 13:37 | |
| *** catintheroof has quit IRC | 13:42 | |
| *** mgoddard__ has joined #openstack-ironic | 13:45 | |
| vdrok | morning xavierr TheJulia and mgould | 13:48 |
| *** mgoddard_ has quit IRC | 13:49 | |
| *** dansmith is now known as superdan | 13:49 | |
| xavierr | vdrok: hi there \o | 13:50 |
| *** amotoki has quit IRC | 13:58 | |
| lucasagomes | xavierr, morning | 14:01 |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector: Add GenericLocalLinkConnectionHook processing hook https://review.openstack.org/321082 | 14:05 |
| xavierr | hey lucasagomes ;) | 14:05 |
| *** hwcomcn has quit IRC | 14:16 | |
| *** joprovost has joined #openstack-ironic | 14:20 | |
| *** anush_ has joined #openstack-ironic | 14:23 | |
| *** causten has joined #openstack-ironic | 14:24 | |
| *** skelso has joined #openstack-ironic | 14:24 | |
| *** joprovost1 has joined #openstack-ironic | 14:25 | |
| *** joprovost has quit IRC | 14:25 | |
| *** joprovost1 is now known as joprovost | 14:25 | |
| *** PollyZ has joined #openstack-ironic | 14:29 | |
| *** wajdi has joined #openstack-ironic | 14:30 | |
| *** mgoddard__ has quit IRC | 14:31 | |
| *** mgoddard has joined #openstack-ironic | 14:31 | |
| openstackgerrit | Michael Turek proposed openstack/ironic: Prevent URL collisions with sub-controllers: nodes/ports https://review.openstack.org/314514 | 14:33 |
| openstackgerrit | Michael Turek proposed openstack/ironic: Add classes for Portgroups API enablement https://review.openstack.org/347549 | 14:33 |
| TheJulia | cinder makes my head hurt | 14:33 |
| mgould | TheJulia: :-( | 14:33 |
| thiagop | TheJulia: don't forget that today is international beer day | 14:34 |
| TheJulia | thiagop: wait?!? what?!?! | 14:34 |
| *** harshs has joined #openstack-ironic | 14:34 | |
| * mgould googles | 14:34 | |
| thiagop | (every friday is international beer day) | 14:34 |
| mgould | Aug 7, sadly | 14:34 |
| * mgould wonders if we can celebrate Orthodox International Beer Day | 14:34 | |
| mgould | it is International Tiger Day: http://tigerday.org/ | 14:35 |
| * mgould wonders if anywhere around here serves Tiger beer | 14:35 | |
| thiagop | I saw a notice that a tiger killed a woman in china this week... | 14:35 |
| mgould | :-( | 14:35 |
| mgould | they are huge ferocious predators, when all's said and done | 14:36 |
| mgould | predators gonna predate | 14:36 |
| thiagop | It was in a safari like zoo, another woman got off the vehicle probably having an argue with his husband, the tiger got her and the other went to help, but the tiger attacked her mother instead | 14:36 |
| vdrok | also System Administrator Appreciation Day | 14:36 |
| thiagop | pretty shocking video... | 14:37 |
| * mgould congratulates the office sysadmin | 14:37 | |
| TheJulia | it is also apparently crazy kitten day | 14:37 |
| * TheJulia senses she will need to take the signifigant other out to lunch | 14:37 | |
| mgould | TheJulia: there's are non-crazy kittens? | 14:37 |
| thiagop | lol | 14:38 |
| mgould | thiagop: that's pretty awful, but she really shouldn't have got out of the car | 14:39 |
| thiagop | mgould: yeah... | 14:39 |
| TheJulia | mgould: my first cat was not as crazy as the kitten we have now as well | 14:41 |
| *** david-lyle has joined #openstack-ironic | 14:42 | |
| * mgould has only had one pair of cats, and they were both pretty crazy when they were kittens | 14:43 | |
| mgould | though I suppose Haggis was generally more excitable than Josie | 14:43 |
| *** jcoufal_ has joined #openstack-ironic | 14:44 | |
| *** jcoufal has quit IRC | 14:47 | |
| *** david-lyle has quit IRC | 14:47 | |
| *** harshs has quit IRC | 14:47 | |
| *** harshs has joined #openstack-ironic | 14:53 | |
| *** anush_ has quit IRC | 15:01 | |
| sambetts | jroll, vdrok: I can see whats going wrong with the multitenant job | 15:05 |
| jroll | oh? | 15:05 |
| sambetts | for some reason this line: export PROJECTS="openstack/networking-generic-switch $PROJECTS" isn't working right, so PROJECTS is ending up with only networking-generic-switch in it... | 15:06 |
| vdrok | huh | 15:06 |
| sambetts | e.g. the $PROJECTS part isn't populating the rest of it for some reason | 15:06 |
| vdrok | that's a separate builder now? | 15:07 |
| sambetts | right, but the separate builder populates a bash file with this then sourced in the main builder | 15:07 |
| sambetts | I wonder if we change it to be export PROJECTS+="..." instead if it'll work, because that seems to be working for the rest of it :/ | 15:08 |
| openstackgerrit | Akilan Pughazhendi proposed openstack/ironic: Updating dev quickstart to include compatiblity for newest distros https://review.openstack.org/346894 | 15:09 |
| thiagop | sambetts: I think I saw something on the devstack-gate scripts that made me conclude that editting PROJECTS is a bad idea | 15:10 |
| *** jistr is now known as jistr|call | 15:10 | |
| thiagop | sambetts: that's why I use DEVSTACK_GATE_PROJECTS_OVERRIDE | 15:11 |
| sambetts | vdrok: really not sure why its not working the same logic locally is working fine :/ | 15:11 |
| vdrok | sambetts: yeah, that's strange | 15:11 |
| sambetts | thiagop: oh? perhaps we need to look into that | 15:11 |
| vdrok | inspector job does the same | 15:11 |
| *** jcoufal_ has quit IRC | 15:11 | |
| thiagop | sambetts: you're running locally with devstack-gate or without? | 15:12 |
| *** jcoufal has joined #openstack-ironic | 15:12 | |
| *** harshs has quit IRC | 15:13 | |
| *** harshs has joined #openstack-ironic | 15:13 | |
| vdrok | thiagop: heh, 'add <project> to $PROJECTS' is what the logs say when failure happens :) | 15:13 |
| vdrok | pretty bad joke if we should not do that :) | 15:13 |
| *** e0ne has quit IRC | 15:14 | |
| thiagop | vdrok: https://github.com/openstack-infra/devstack-gate/blob/master/devstack-vm-gate-wrap.sh#L44 | 15:14 |
| thiagop | vdrok: I think you'll need to += | 15:14 |
| sambetts | thiagop: not using devstack-gate locally, and when i mean locally I mean just the same code path in bash | 15:14 |
| sambetts | and like vdrok says it works fine in the ironic-inspector tempest jobs | 15:15 |
| thiagop | vdrok: do you have a link handy for the multitenant job config? | 15:15 |
| thiagop | sambetts: ^ | 15:15 |
| sambetts | https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic.yaml | 15:16 |
| vdrok | thiagop: it's in every run - http://logs.openstack.org/18/285218/10/check/gate-tempest-dsvm-ironic-multitenant-network-nv/67947f3/logs/localrc.txt.gz | 15:16 |
| *** anush_ has joined #openstack-ironic | 15:16 | |
| vdrok | or if you mean jenkins config it's what sambetts said | 15:17 |
| *** jistr|call is now known as jistr | 15:17 | |
| thiagop | uhh, I think I know what's happening | 15:18 |
| *** PollyZ has quit IRC | 15:18 | |
| thiagop | lemme check | 15:19 |
| *** harshs has quit IRC | 15:19 | |
| *** skelso has quit IRC | 15:21 | |
| *** PollyZ has joined #openstack-ironic | 15:21 | |
| *** ayoung has joined #openstack-ironic | 15:22 | |
| vdrok | maybe devstack-virtual-ironic should be moved higher than ironic-multitenant builder? | 15:22 |
| vdrok | doesn't seem so... | 15:23 |
| thiagop | vdrok: I think the problem isn't that the PROJECTS is empty, but that ironic isn't within it | 15:23 |
| *** david-lyle has joined #openstack-ironic | 15:23 | |
| sambetts | vdrok: then devstack-virtual-ironic would be run before ironic-multitenant has populated the extra-vars file | 15:23 |
| sambetts | thiagop: we know that, the problem is for some reason our job builder scripts are behaving weird | 15:24 |
| sambetts | thiagop: they aren't doing what we expect them to do | 15:24 |
| sambetts | so PROJECTS is ending up missing a bunch of things from it | 15:24 |
| sambetts | The way the JJB jobs work for Ironic now, is that we have a main shell script, that does the actual devstack-gate run, and that shell script sources a file we create dynamically by using other smaller builders | 15:26 |
| sambetts | we do this so that we can enable specifc features on the fly without bloating the main shell script | 15:27 |
| *** PollyZ has quit IRC | 15:27 | |
| thiagop | sambetts: http://logs.openstack.org/18/285218/10/check/gate-tempest-dsvm-ironic-multitenant-network-nv/67947f3/console.html#_2016-07-29_13_57_00_945821 | 15:27 |
| sambetts | in the main shell script we export PROJECTS with all the projects we need, the multitenant builder puts this line in the extra-vars file that is sourced export PROJECTS="...genric-network-switch $PROJECTS" | 15:28 |
| sambetts | for some reason the PROJECT variable exported in the main script isn't being picked up | 15:28 |
| sambetts | even though it should be | 15:28 |
| *** PollyZ has joined #openstack-ironic | 15:30 | |
| *** ijw has joined #openstack-ironic | 15:30 | |
| *** mgoddard_ has joined #openstack-ironic | 15:30 | |
| thiagop | sambetts: theory: the devstack-virtual-ironic script is calling bash on the first line | 15:30 |
| thiagop | sambetts: would that end up cleaning the exported variables? | 15:30 |
| *** harshs has joined #openstack-ironic | 15:31 | |
| sambetts | devstack-virtual-ironic sources the extra-vars-file https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic.yaml#L195 | 15:31 |
| *** harshs has quit IRC | 15:32 | |
| thiagop | uhmm | 15:32 |
| sambetts | try this locally http://paste.openstack.org/show/543919/ | 15:33 |
| sambetts | this is what its effectvly doing | 15:33 |
| *** causten has quit IRC | 15:34 | |
| sambetts | but for some reason in the gate its not populating the $EGGS in the sourced file even though it should | 15:34 |
| *** mgoddard has quit IRC | 15:34 | |
| *** causten has joined #openstack-ironic | 15:34 | |
| *** bfournie has joined #openstack-ironic | 15:36 | |
| vdrok | aha https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic.yaml#L811 | 15:36 |
| vdrok | this job fails too on inspector | 15:36 |
| vdrok | with the same symptoms | 15:37 |
| sambetts | :/ seems like something to do with the way sourcing a file is working on the version of bash in the gate... | 15:37 |
| vdrok | maybe source is somehow run under different user? | 15:38 |
| *** causten has quit IRC | 15:38 | |
| *** sdake has joined #openstack-ironic | 15:39 | |
| sambetts | that doesn't make any sense, why aren't exported vars making it through to the sourced file... | 15:40 |
| *** ijw has quit IRC | 15:40 | |
| sambetts | that just seems wrong in all accounts, unless they've somehow turned of environment variables passing to sub-shells | 15:40 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add volume_connector table to save connector information https://review.openstack.org/200983 | 15:40 |
| devananda | g'morning, all | 15:41 |
| sambetts | Hey devananda | 15:41 |
| vdrok | morning devananda | 15:41 |
| thiagop | morning devananda | 15:41 |
| devananda | JayF: it turns out, oslo.policy doesn't allow there to _not_ be a policy file. it's an open bug in the project ... | 15:41 |
| *** PollyZ has quit IRC | 15:41 | |
| JayF | devananda: ...that's a pretty terrible bug | 15:41 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Introduce VolumeConnector object https://review.openstack.org/214584 | 15:42 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Introduce new RPCs to support volume connector operation https://review.openstack.org/214585 | 15:42 |
| *** PollyZ has joined #openstack-ironic | 15:43 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add volume_targets table to save target information https://review.openstack.org/285218 | 15:43 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Introduce VolumeTarget object https://review.openstack.org/285219 | 15:43 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Introduce new RPCs to support volume target operations https://review.openstack.org/285220 | 15:43 |
| devananda | https://bugs.launchpad.net/oslo.policy/+bug/1603329 | 15:44 |
| openstack | Launchpad bug 1603329 in oslo.policy "Allow policy file to not exist" [Undecided,Confirmed] - Assigned to Kseniya Tychkova (ktychkova) | 15:44 |
| sambetts | vdrok: the logic works locally in bash :'( http://paste.openstack.org/show/543922/ | 15:48 |
| vdrok | yeah that works for me too | 15:50 |
| sambetts | vdrok: looks like the addition DEVSTACK_LOCAL_CONFIGS aren't getting through either :'( | 15:51 |
| sambetts | seriously... why isn't sourcing a file in bash working period in the gate... | 15:53 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for openstackclient plugin https://review.openstack.org/301687 | 15:53 |
| *** skelso has joined #openstack-ironic | 15:56 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Add keystone policy support to Ironic https://review.openstack.org/325599 | 15:56 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introducing node introspection state management https://review.openstack.org/348943 | 15:56 |
| *** vishwanathj has joined #openstack-ironic | 15:57 | |
| *** jcoufal has quit IRC | 15:59 | |
| *** jcoufal has joined #openstack-ironic | 15:59 | |
| *** pcaruana has quit IRC | 16:01 | |
| sambetts | vdrok: asking in infra now | 16:01 |
| *** mgoddard has joined #openstack-ironic | 16:01 | |
| *** mgoddard_ has quit IRC | 16:01 | |
| vdrok | heh, separate shell for every builder | 16:02 |
| sambetts | vdrok: yeah, thats why I have to write it to a file | 16:07 |
| sambetts | vdrok: and then source it | 16:07 |
| sambetts | vdrok: if they were all run in one shell there would be no need for the sourcing magic | 16:07 |
| *** rama_y has joined #openstack-ironic | 16:10 | |
| *** vmud213 has quit IRC | 16:10 | |
| *** vmud213 has joined #openstack-ironic | 16:10 | |
| *** harshs has joined #openstack-ironic | 16:10 | |
| *** skelso has quit IRC | 16:11 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Update documentation for keystone policy support https://review.openstack.org/345556 | 16:12 |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Mask configdrive contents in API responses https://review.openstack.org/326768 | 16:12 |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Add keystone policy support to Ironic https://review.openstack.org/325599 | 16:12 |
| *** jcoufal has quit IRC | 16:13 | |
| *** jcoufal has joined #openstack-ironic | 16:14 | |
| *** bvandewa has joined #openstack-ironic | 16:14 | |
| *** bvandewa has quit IRC | 16:15 | |
| milan | sambetts, hi :) | 16:17 |
| milan | sambetts, I could use some early feedback on an inspector patch | 16:17 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Extend parse_root_device_hints to support operators https://review.openstack.org/348953 | 16:17 |
| sambetts | vdrok, jroll: Worked out what the problem was | 16:18 |
| milan | sambetts, mostly whether it isn't a complete piece of ... ;) https://review.openstack.org/#/c/348943/1 | 16:18 |
| sambetts | milan: sure I'll add it to my list for Monday, I'll be heading out soon | 16:18 |
| milan | sambetts, thanks | 16:18 |
| milan | same here actually ;) | 16:18 |
| sambetts | vdrok, jroll: https://review.openstack.org/#/c/348949/ | 16:19 |
| vdrok | sambetts: huh, cool! | 16:19 |
| jroll | sambetts: O_O | 16:20 |
| sambetts | vdrok: basically it was trying to substitute the variable at the point it wrote the code into ironic-extra-vars, instead of processing it when we sourced the file | 16:20 |
| sambetts | adding the quotes onto the EOF prevents any processing of the code, until we source the script in ironic-virtual-devstack late | 16:21 |
| sambetts | later* | 16:21 |
| sambetts | so my bash logic was sound, its just the pre-processing of the script screwing with everything | 16:22 |
| sambetts | vdrok: also I was wrong, the DEVSTACK_LOCAL_CONFIGs we were +=ing worked fine | 16:22 |
| sambetts | so it was just the substitutions that busted things | 16:23 |
| vdrok | great finding, thanks! | 16:24 |
| sambetts | well with the size of my refactor 2 jobs breaking out of however many jobs we have isn't bad :) | 16:24 |
| vdrok | yeah, not bad at all! | 16:24 |
| *** trown is now known as trown|lunch | 16:25 | |
| sambetts | still need to update few of the other jobs to fit into the new naming scheme though | 16:25 |
| * sambetts -> home | 16:26 | |
| sambetts | night all | 16:26 |
| *** jpich has quit IRC | 16:26 | |
| *** sambetts is now known as sambetts|afk | 16:27 | |
| vdrok | good night sambetts|afk | 16:28 |
| *** tesseract- has quit IRC | 16:30 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Verify JSON response of driver commands https://review.openstack.org/336600 | 16:34 |
| lucasagomes | folks I'm calling it a day here | 16:36 |
| lucasagomes | have a great evening, enjoy the weekend | 16:36 |
| lucasagomes | monday is national holiday here, I will try to make it to the meeting | 16:36 |
| TheJulia | goodnight lucasagomes | 16:39 |
| lucasagomes | see ya :-) | 16:40 |
| *** lucasagomes is now known as lucas-dinner | 16:41 | |
| jroll | I believe the multi-compute stuff is ready to review, if anyone's interested | 16:41 |
| jroll | https://review.openstack.org/#/c/348443/ | 16:41 |
| jroll | came out much nicer than I thought it would | 16:41 |
| *** e0ne has joined #openstack-ironic | 16:45 | |
| *** e0ne has quit IRC | 16:46 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for openstackclient plugin https://review.openstack.org/348962 | 16:48 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for openstackclient plugin https://review.openstack.org/301687 | 16:54 |
| openstackgerrit | Tang Chen proposed openstack/ironic: [WIP] Split API root unit and functional tests https://review.openstack.org/348965 | 16:55 |
| *** thrash is now known as thrash|biab | 16:56 | |
| *** tangchen has joined #openstack-ironic | 16:56 | |
| *** tangchen_ has joined #openstack-ironic | 16:56 | |
| *** PollyZ has quit IRC | 16:56 | |
| *** derekh has quit IRC | 16:57 | |
| *** tangchen_ has quit IRC | 17:01 | |
| *** tangchen has quit IRC | 17:01 | |
| *** wajdi_ has joined #openstack-ironic | 17:01 | |
| *** ijw has joined #openstack-ironic | 17:02 | |
| *** wajdi_ has quit IRC | 17:02 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for openstackclient plugin https://review.openstack.org/301687 | 17:02 |
| *** vmud213 has quit IRC | 17:02 | |
| *** wajdi_ has joined #openstack-ironic | 17:02 | |
| NobodyCam | Morning ironic'ers... says the man on the road | 17:03 |
| thiagop | good finding sambetts|afk , have a nice weekend | 17:04 |
| *** wajdi has quit IRC | 17:05 | |
| thiagop | morning NobodyCam | 17:06 |
| vdrok | good morning NobodyCam ! | 17:06 |
| *** wajdi has joined #openstack-ironic | 17:07 | |
| Romanenko_K | Hello, NobodyCam ) | 17:07 |
| *** skelso has joined #openstack-ironic | 17:08 | |
| *** wajdi_ has quit IRC | 17:10 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [pxe] https://review.openstack.org/309612 | 17:11 |
| *** ChrisMead has quit IRC | 17:12 | |
| *** skelso has quit IRC | 17:14 | |
| *** ccamacho has quit IRC | 17:15 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [pxe] https://review.openstack.org/309612 | 17:15 |
| *** harshs has quit IRC | 17:17 | |
| *** milan has quit IRC | 17:17 | |
| *** ijw has quit IRC | 17:18 | |
| *** harshs has joined #openstack-ironic | 17:21 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-specs: Add new out-of-band properties to ilo drivers https://review.openstack.org/338138 | 17:22 |
| xavierr | morning NobodyCam o/ | 17:22 |
| openstackgerrit | Thiago Paiva Brito proposed openstack/ironic: Onetime boot when set_boot_device isn't persistent https://review.openstack.org/340596 | 17:23 |
| thiagop | two line "too much easy" review for you folks to take a look ^^ | 17:23 |
| *** loki_ has joined #openstack-ironic | 17:24 | |
| thiagop | And if you have a quarter of hour more, please review our child: https://review.openstack.org/#/c/286192/ | 17:25 |
| *** skelso has joined #openstack-ironic | 17:28 | |
| *** harshs has quit IRC | 17:30 | |
| loki_ | review please https://review.openstack.org/#/c/272658/ | 17:30 |
| *** loki__ has joined #openstack-ironic | 17:33 | |
| *** loki_ has quit IRC | 17:35 | |
| *** skelso has quit IRC | 17:37 | |
| *** loki__ has quit IRC | 17:38 | |
| openstackgerrit | Michael Turek proposed openstack/ironic: Add classes for Portgroups API enablement https://review.openstack.org/347549 | 17:38 |
| *** trown|lunch is now known as trown | 17:39 | |
| *** loki_ has joined #openstack-ironic | 17:39 | |
| *** Romanenko_K has quit IRC | 17:41 | |
| * mgould -> home; good night! | 17:44 | |
| *** mgould is now known as mgould|afk | 17:44 | |
| thiagop | good night mgould|afk | 17:46 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-specs: Enhance ilo drivers to do inband inspection https://review.openstack.org/201904 | 17:48 |
| *** harshs has joined #openstack-ironic | 17:49 | |
| *** e0ne has joined #openstack-ironic | 17:50 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Cleanup internal_info if auto cleaning is disabled https://review.openstack.org/348983 | 17:51 |
| * vdrok leaves too | 17:53 | |
| vdrok | have a great weekend! | 17:53 |
| *** links has joined #openstack-ironic | 17:54 | |
| *** mjturek1 has quit IRC | 17:57 | |
| *** livelace has quit IRC | 17:59 | |
| *** harshs has quit IRC | 18:00 | |
| *** harshs has joined #openstack-ironic | 18:00 | |
| *** joprovost1 has joined #openstack-ironic | 18:01 | |
| thiagop | have a nice weekend vdrok | 18:02 |
| *** joprovost has quit IRC | 18:03 | |
| *** joprovost1 is now known as joprovost | 18:03 | |
| *** mjturek1 has joined #openstack-ironic | 18:04 | |
| JayF | devananda: https://review.openstack.org/#/c/325599 has my +2 with nits; but still am curious about what we're going to do with policy.json.sample; it's not valid JSON so it seems a little weird to have it in the repo | 18:06 |
| JayF | devananda: it feels like you meant to only have one of policy.json.sample || policy.yaml.sample committed | 18:07 |
| thiagop | I'd vote for a valid json too, but with the comments if possible | 18:07 |
| JayF | no such thing as valid json with comments :) | 18:07 |
| JayF | JSON standard doesn't allow for comments | 18:08 |
| thiagop | JayF: so, move it into the docs page (with some formatting, of course)? | 18:08 |
| *** sabeen has joined #openstack-ironic | 18:08 | |
| JayF | I mean, I don't care that much beyond it being clear (my preference is to just use the yaml), but whatever we include needs to be generated by that tox target | 18:09 |
| JayF | and I don't think the policy generator makes valid json so... | 18:09 |
| JayF | and I'm -1 to putting it in docs b/c like the sample config, it'll change frequently as we add new features, so it should be managed like that sample config | 18:09 |
| thiagop | My fear is that I understand the policy and the endpoints, but new operators may not | 18:09 |
| JayF | sure; but a policy.yaml.sample explains it just as well as something else | 18:11 |
| JayF | the docs (which I'm about to start reviewing now) should generically cover policy; but because the default applied policy is "living", it needs to be managed outside of docs | 18:11 |
| *** PollyZ has joined #openstack-ironic | 18:12 | |
| *** skelso has joined #openstack-ironic | 18:12 | |
| *** ijw has joined #openstack-ironic | 18:13 | |
| JayF | if a specs core could look at and land this: https://review.openstack.org/#/c/345039/ it'd be helpful, since the keystone code following those changes is about to land soon :) | 18:14 |
| *** mtanino has joined #openstack-ironic | 18:14 | |
| *** mjturek1 has quit IRC | 18:18 | |
| *** loki_ has quit IRC | 18:18 | |
| *** anush_ has quit IRC | 18:18 | |
| *** anush_ has joined #openstack-ironic | 18:19 | |
| *** ijw has quit IRC | 18:20 | |
| thiagop | Folks, I'm kinda sick so I'm calling it a day earlier | 18:23 |
| thiagop | Have a nice weekend | 18:23 |
| *** thiagop has quit IRC | 18:23 | |
| NobodyCam | Morning thiagop xavierr vdrok :) | 18:24 |
| *** loki__ has joined #openstack-ironic | 18:25 | |
| JayF | o/ feel better | 18:26 |
| natorious | why isn't the image deploy process part of the agent hardware managers? | 18:26 |
| *** electrofelix has quit IRC | 18:28 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Enhance ilo drivers to do inband-inspection This fix allow ilo drivers to do inband inspection using ironic-inspector when `oob_inspection_only` config is set to False. https://review.openstack.org/343577 | 18:28 |
| natorious | been trying to think beyond legacy boot | 18:29 |
| JayF | I think, generally, that the deploy process (including how the image is written) is generally driven by the deploy driver | 18:31 |
| JayF | so maybe the right place to change that process would be via a different conductor deploy driver | 18:32 |
| JayF | but I'm curious what you wanna do | 18:32 |
| natorious | I'd like to be able to trigger deploy steps similarly to how you can clean steps | 18:32 |
| natorious | for easier rapid itteration | 18:33 |
| JayF | do you have a solid example so I can understand better? | 18:33 |
| JayF | We have talked about the idea of "deploy steps" with prepare_image and write_configdrive being implied as final two steps in any case | 18:34 |
| JayF | but I don't think it's been codified in a spec or gone beyond "hallway conversations" | 18:34 |
| natorious | sure, so say your deving on uefi and want to just run one part of that say like write uefi part | 18:34 |
| *** thrash|biab is now known as thrash | 18:34 | |
| JayF | So you're wanting to break up the deploy into smaller pieces | 18:35 |
| natorious | or just want to write the image to a raid set and then check to make sure its how it should be | 18:35 |
| natorious | yeah | 18:35 |
| JayF | so like using standalone mode it's easier to develop on a given piece | 18:35 |
| JayF | I think the idea of deploy steps, like clean steps, would permit that somewhat | 18:35 |
| JayF | and would be interesting to pursue | 18:35 |
| JayF | I'd def. review that spec if you put it up as an idea :) | 18:35 |
| natorious | noted :) | 18:36 |
| *** rbudden has quit IRC | 18:41 | |
| JayF | jroll: is https://review.openstack.org/#/c/204641/ still valid at all with the new scheduling stuff? I'm thinking no? | 18:42 |
| *** jcoufal has quit IRC | 18:43 | |
| *** jcoufal has joined #openstack-ironic | 18:44 | |
| *** rama_y has quit IRC | 18:45 | |
| *** harshs has quit IRC | 18:46 | |
| *** skelso has quit IRC | 18:47 | |
| *** loki__ has quit IRC | 18:47 | |
| *** sdake has quit IRC | 18:47 | |
| *** skelso has joined #openstack-ironic | 18:52 | |
| *** rama_y has joined #openstack-ironic | 18:53 | |
| *** achanda has joined #openstack-ironic | 18:55 | |
| *** hoonetorg has quit IRC | 18:56 | |
| openstackgerrit | Stephanie Miller proposed openstack/python-ironicclient: Add volume_connector support https://review.openstack.org/214786 | 18:57 |
| *** ijw has joined #openstack-ironic | 18:57 | |
| openstackgerrit | Stephanie Miller proposed openstack/python-ironicclient: Add volume_target support https://review.openstack.org/285233 | 18:57 |
| *** PollyZ has quit IRC | 19:02 | |
| *** bks has joined #openstack-ironic | 19:02 | |
| *** ijw has quit IRC | 19:03 | |
| *** bnemec is now known as beekneemech | 19:12 | |
| *** skelso has quit IRC | 19:12 | |
| *** penick has joined #openstack-ironic | 19:14 | |
| *** ijw has joined #openstack-ironic | 19:15 | |
| *** joprovost has quit IRC | 19:19 | |
| *** joprovost has joined #openstack-ironic | 19:19 | |
| *** links has quit IRC | 19:20 | |
| *** bvandewa has joined #openstack-ironic | 19:22 | |
| *** PollyZ has joined #openstack-ironic | 19:24 | |
| *** ijw has quit IRC | 19:25 | |
| *** skelso has joined #openstack-ironic | 19:33 | |
| *** sdake has joined #openstack-ironic | 19:35 | |
| *** bvandewa has quit IRC | 19:35 | |
| *** skelso has quit IRC | 19:39 | |
| *** joprovost1 has joined #openstack-ironic | 19:44 | |
| *** joprovost has quit IRC | 19:47 | |
| *** joprovost1 is now known as joprovost | 19:47 | |
| TheJulia | devananda: I've been thinking about 325599, would it be awful to bump the microversion as well just to signify "Hey, policy support was inserted, although is API wide regardless of version any pending API changes might need it too" | 19:51 |
| JayF | TheJulia: We didn't change the api. | 19:51 |
| JayF | TheJulia: unless we were going to not enforce policy for earlier microversions that doesn't make sense. | 19:52 |
| *** krtaylor has quit IRC | 19:52 | |
| TheJulia | Well, technically we have, we're changing the policy handling, it is just not self-apparent nor negotiable. I'm thinking of a force people to encounter a merge conflict on any api changes such that visibility is raised | 19:55 |
| * TheJulia might just be a horrible person though | 19:55 | |
| TheJulia | s/have/are/ | 19:55 |
| openstackgerrit | Xavier proposed openstack/python-oneviewclient: Add auditing instrumentation for python-oneviewclient https://review.openstack.org/337281 | 19:56 |
| *** skelso has joined #openstack-ironic | 19:58 | |
| JayF | TheJulia: should microversion be used for a developer signal? I don't think so; I think it's intended to signal API changes to clients | 19:58 |
| *** ijw has joined #openstack-ironic | 19:58 | |
| *** sdake has quit IRC | 19:59 | |
| *** e0ne has quit IRC | 20:04 | |
| TheJulia | Or just edit the line before or after, we don't actually have to bump the version. I guess my only concern is that it is a bit of a major change that breaks the convention that we've built via microversions thus far. Each change breaks the next change in line until it is fixed. It might be in release notes, but that doesn't mean awarness may be raised unless someone actively encounters a bloc | 20:05 |
| TheJulia | k that has to be navigated around. I guess it is that or a checklist, or a test that actively hunts for any missing policy definitions in the code (which actually might be a better idea now that I think about it). | 20:05 |
| devananda | TheJulia: I do not think changing the api version is a reasonable to thing to do with the policy change | 20:06 |
| TheJulia | ok | 20:07 |
| devananda | TheJulia: because it presents a false contract: if we claim that >= v1.21 there is policy support, it implies that a client can ask for v<1.21 and bypass all the policy | 20:07 |
| JayF | TheJulia: I suspect the more prudent path is for us (you, me, whoever) as cores to dig through API changes, and tag them with a -1 support policy if they don't | 20:07 |
| *** ijw has quit IRC | 20:07 | |
| devananda | TheJulia: and it's not a change in the semantics of any API request at all | 20:07 |
| devananda | the same admin user that could do $things before can still do exactly the same $things | 20:08 |
| devananda | we're enabling the operator to adjust when ironic throws a NotAuthorized error -- not changing what exceptions can be returned, or how the REST API itself works | 20:08 |
| TheJulia | JayF: Agreed, although at the same time we likely need testing for it anyway | 20:08 |
| devananda | I've been thinking about ways to enforce that every API endpoint has a policy applied to it | 20:09 |
| devananda | but haven't written up a test for that yet | 20:09 |
| devananda | I suppose that would be really useful, eh? | 20:09 |
| TheJulia | yeah, a nice red flag that there is a major problem potentially | 20:09 |
| devananda | JayF: to your question about policy.json.sample - that file is YAML and, unsurprisingly, works fine | 20:10 |
| JayF | devananda: then we should probably nuke the policy.yaml.sample also in that patchset | 20:10 |
| devananda | oh. doh. | 20:10 |
| JayF | devananda: my concern was more that we provided both, and only generated one | 20:10 |
| devananda | I thought I had deleted that | 20:10 |
| devananda | (merge error) | 20:10 |
| JayF | devananda: although I'd prefer it to say yaml if it is yaml *shrug* otherwise | 20:10 |
| devananda | well, I would too, except oslo | 20:10 |
| JayF | devananda: you wanna fix the couple other small nits and I'll give it a clean +@? | 20:11 |
| *** baoli has quit IRC | 20:11 | |
| JayF | *+2 | 20:11 |
| devananda | oslo.policy documents *everywhere* that it's a .json file | 20:11 |
| JayF | yeah, I know, that's ... annoying | 20:11 |
| devananda | even though it's not | 20:11 |
| *** rbudden has joined #openstack-ironic | 20:11 | |
| * devananda looks to see what other nits are there | 20:11 | |
| *** bvandewa has joined #openstack-ironic | 20:12 | |
| devananda | JayF: anything else besides commit message typo? | 20:12 |
| JayF | devananda: whitespace in the tox | 20:13 |
| JayF | and the thing in the release notes is now irreleveant | 20:13 |
| devananda | what is? | 20:13 |
| *** joprovost has quit IRC | 20:13 | |
| JayF | commands = # in your tox.init that oyu added has trailing whitespace | 20:14 |
| JayF | the release note comment was continued yaml vs json confusion | 20:14 |
| openstackgerrit | Akilan Pughazhendi proposed openstack/ironic: Updating dev quickstart to include compatiblity for newest distros https://review.openstack.org/346894 | 20:15 |
| *** ijw has joined #openstack-ironic | 20:21 | |
| *** rama_y has quit IRC | 20:21 | |
| *** ijw has quit IRC | 20:27 | |
| JayF | devananda: did you mean to push a patch set up and didn't? /me itching to +2 it, lol | 20:29 |
| JayF | I'm excited about policy landing | 20:29 |
| *** rama_y has joined #openstack-ironic | 20:31 | |
| *** achanda has quit IRC | 20:34 | |
| devananda | JayF: I have it ready, then started working on a new test case | 20:34 |
| devananda | I can do that in a follow on | 20:34 |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Update documentation for keystone policy support https://review.openstack.org/345556 | 20:34 |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Mask configdrive contents in API responses https://review.openstack.org/326768 | 20:34 |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Add keystone policy support to Ironic https://review.openstack.org/325599 | 20:34 |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: [WIP] Hide driver[_internal]_info from observer role https://review.openstack.org/345076 | 20:34 |
| JayF | either way is fine :) I'm just excited and that makes me impatient | 20:34 |
| JayF | lol | 20:34 |
| devananda | tadaa! | 20:35 |
| *** wajdi has quit IRC | 20:42 | |
| *** wajdi has joined #openstack-ironic | 20:42 | |
| JayF | TheJulia: https://review.openstack.org/#/c/325599 I think we should land this, unless there's a good reason not to. Had Multiple +2s, and the only things fixed were minor nits | 20:46 |
| *** wajdi has quit IRC | 20:49 | |
| *** jjohnson2 has quit IRC | 20:58 | |
| *** trown is now known as trown|outtypewww | 20:59 | |
| *** bvandewa has quit IRC | 21:04 | |
| *** sabeen has quit IRC | 21:23 | |
| *** skelso has quit IRC | 21:27 | |
| *** ppiela has quit IRC | 21:28 | |
| *** aNupoisc has joined #openstack-ironic | 21:30 | |
| *** adnavare has joined #openstack-ironic | 21:30 | |
| *** adnavare has quit IRC | 21:33 | |
| *** aNupoisc has quit IRC | 21:33 | |
| *** aNupoisc has joined #openstack-ironic | 21:33 | |
| *** bvandewa has joined #openstack-ironic | 21:35 | |
| *** daemontool has quit IRC | 21:41 | |
| jroll | JayF: 204641 probably isn't needed, no. I need to take some time to go back and clean up all the old artifacts of this stuff | 21:42 |
| JayF | jroll: that's what I thought; I skipped reviewing it | 21:42 |
| jroll | cool | 21:42 |
| JayF | jroll: you should totally land policy while you're here | 21:42 |
| JayF | jroll: all the nits are fixed, I have a clean +2 on it | 21:42 |
| jroll | JayF: I just -1d it :) | 21:42 |
| jroll | it's still a yaml file | 21:43 |
| jroll | in the sample | 21:43 |
| JayF | but it's rendered fine by oslo policy | 21:43 |
| jroll | huh? | 21:43 |
| JayF | imo making that file be valid json is a bug in the config generator oslo provides | 21:43 |
| jroll | https://review.openstack.org/#/c/325599/20/etc/ironic/policy.json.sample | 21:43 |
| jroll | is not json | 21:43 |
| JayF | yep, oslo policy takes yaml input too | 21:43 |
| jroll | I understand | 21:43 |
| JayF | and the only reason we don't name it yaml is all the oslo policy docs say json all over | 21:43 |
| jroll | but we shouldn't call it .json imo | 21:43 |
| JayF | and default to json | 21:43 |
| jroll | -.- | 21:44 |
| JayF | yep, exactly | 21:44 |
| JayF | I don't love it either, but the bugs are oslo.policy's, not ours | 21:44 |
| *** dims_ has quit IRC | 21:44 | |
| jroll | I'm seeing this in scrollback now but I hadn't read that far | 21:44 |
| *** ayoung has quit IRC | 21:44 | |
| jroll | can we at least drop a todo or bug ref or something? | 21:44 |
| JayF | drop it where? | 21:44 |
| JayF | not in that generated config file | 21:44 |
| jroll | idk, tox.ini? | 21:45 |
| jroll | or maybe even in that sample | 21:45 |
| JayF | I guess we could? I honestly wouldn't consider it a good reason to hold it up, but if you do you can -1 it :( | 21:45 |
| JayF | not in the sample | 21:45 |
| JayF | because that's generated | 21:45 |
| jroll | why? | 21:45 |
| jroll | ugh | 21:45 |
| JayF | yeah, that's why I'm like, lets make sure there's a bug against the config generator | 21:45 |
| JayF | and move on | 21:45 |
| *** dims has joined #openstack-ironic | 21:46 | |
| jroll | oslopolicy-sample-generator --namespace=ironic.api --output-file=etc/ironic/policy.json.sample && echo "# this isn't actually json, sorry\n $(cat policy.json.sample)" > policy.json.sample | 21:46 |
| jroll | :) | 21:46 |
| jroll | is there a bug against the generator? | 21:46 |
| JayF | I'll make sure there is right meow | 21:47 |
| jroll | k | 21:47 |
| JayF | there's another bug, you can't run oslo policy stuff w/o a config file at all | 21:47 |
| JayF | so even in a clean ironic install you need the file there | 21:47 |
| JayF | or it breaks | 21:47 |
| JayF | that bug is filed, and is very unsavory | 21:47 |
| jroll | unsavory why? | 21:48 |
| jroll | as in, annoying for ops? | 21:48 |
| JayF | yes. | 21:48 |
| JayF | an empty config file being required is silly | 21:48 |
| jroll | yeah | 21:48 |
| JayF | jroll: https://bugs.launchpad.net/oslo.policy/+bug/1607968 | 21:51 |
| openstack | Launchpad bug 1607968 in oslo.policy "oslopolicy-sample-generator does not output json, but docs always reference json" [Undecided,New] | 21:51 |
| jroll | ok thanks | 21:51 |
| jroll | let me re-review | 21:51 |
| *** rbudden has quit IRC | 21:51 | |
| *** PollyZ has quit IRC | 21:51 | |
| openstackgerrit | Merged openstack/ironic-specs: Update the keystone-policy spec https://review.openstack.org/345039 | 21:52 |
| *** bvandewa has quit IRC | 21:57 | |
| jroll | woo, policy approved | 21:57 |
| bks | https://review.openstack.org/#/c/325230 | 22:01 |
| *** superdan has quit IRC | 22:07 | |
| *** PollyZ has joined #openstack-ironic | 22:08 | |
| *** bks has quit IRC | 22:08 | |
| *** achanda has joined #openstack-ironic | 22:13 | |
| *** ijw has joined #openstack-ironic | 22:14 | |
| *** PollyZ has quit IRC | 22:16 | |
| *** ijw has quit IRC | 22:24 | |
| devananda | woot! | 22:28 |
| *** jcoufal_ has joined #openstack-ironic | 22:33 | |
| *** jcoufal has quit IRC | 22:34 | |
| JayF | Our grenade job is apparently building tinyipa from source | 22:36 |
| JayF | http://logs.openstack.org/24/348624/1/gate/gate-grenade-dsvm-ironic/0f6c6d8/logs/grenade.sh.txt.gz#_2016-07-29_22_19_59_330 | 22:37 |
| JayF | and it just caused a job to fail (the linked one0 | 22:37 |
| JayF | the default in stable/mitaka is apparently to build the ramdisk https://github.com/openstack/ironic/blob/stable/mitaka/devstack/lib/ironic#L97 | 22:38 |
| JayF | and we don't change that in the grenade job | 22:39 |
| JayF | https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L134 | 22:39 |
| JayF | we apparently default it to true at master as well | 22:39 |
| JayF | http://logs.openstack.org/24/348624/1/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa/0187ac8/logs/devstacklog.txt.gz#_2016-07-29_03_15_25_489 | 22:40 |
| JayF | and we're building tinyipa in the non -src jobs too | 22:40 |
| JayF | aye carumba! | 22:40 |
| devananda | :-/ | 22:42 |
| JayF | I'm going to find an old patchset to see if this was introduced by the refactoring just landing | 22:43 |
| JayF | devananda: wdyt about just changing that devstack default? | 22:43 |
| *** jcoufal_ has quit IRC | 22:43 | |
| JayF | a job run Jul 20 had the option set to False in localrc http://logs.openstack.org/68/344668/2/check/gate-tempest-dsvm-ironic-agent_ssh/27e9b20/logs/localrc.txt.gz | 22:44 |
| devananda | as an aside, the project-config refactor seems to have also added a lot of new job runs to each patch | 22:45 |
| devananda | JayF: if we change the default in devstack on master, it won't fix the issue on stable/mitaka | 22:45 |
| devananda | I think it's reasonable for devstack to default to download the image rather than build it every time, fwiw | 22:46 |
| *** rama_y has quit IRC | 22:46 | |
| JayF | I'm going to put that change up | 22:46 |
| *** rama_y has joined #openstack-ironic | 22:46 | |
| JayF | and file a bug about the change in behavior | 22:46 |
| JayF | do you have an example list of jobs before/after you can add as a comment to it? | 22:46 |
| devananda | not off hand -- I'd just look at any patch set from a few weeks ago vs. now | 22:47 |
| JayF | sambetts wrote that refactor, right? | 22:48 |
| JayF | going to assign that bug to him when I file it since he might have an idea what's up | 22:48 |
| *** Guest41489 has joined #openstack-ironic | 22:49 | |
| openstackgerrit | Merged openstack/python-ironicclient: Add internal_info field to port https://review.openstack.org/341552 | 22:49 |
| *** harshs has joined #openstack-ironic | 22:54 | |
| *** harshs has quit IRC | 22:54 | |
| JayF | devananda: I believe the "more jobs" problem is xenial | 22:58 |
| *** harshs has joined #openstack-ironic | 23:00 | |
| *** Guest41489 has quit IRC | 23:02 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Devstack should use a prebuilt ramdisk by default https://review.openstack.org/349112 | 23:05 |
| JayF | devananda: ^ | 23:05 |
| *** harshs has quit IRC | 23:06 | |
| *** jcoufal has joined #openstack-ironic | 23:07 | |
| JayF | nevermind, that can't merge until the project-config bit is set | 23:08 |
| JayF | or else the -src jobs on IPA will behave wrongly | 23:08 |
| JayF | I think that's a bug I'm going to leave for the morning :( | 23:08 |
| JayF | s/the/monday/ | 23:08 |
| *** harshs has joined #openstack-ironic | 23:15 | |
| *** achanda has quit IRC | 23:17 | |
| *** Guest39119 has joined #openstack-ironic | 23:26 | |
| *** thrash is now known as thrash|wknd | 23:32 | |
| *** daemontool has joined #openstack-ironic | 23:34 | |
| *** anush_ has quit IRC | 23:46 | |
| *** jcoufal_ has joined #openstack-ironic | 23:48 | |
| *** jcoufal has quit IRC | 23:49 | |
| *** vishwanathj has quit IRC | 23:52 | |
| *** Guest39119 is now known as dansmith | 23:52 | |
| *** achanda has joined #openstack-ironic | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!