*** jroll has joined #openstack-ironic | 00:00 | |
*** rloo has quit IRC | 00:00 | |
*** cjloader has quit IRC | 00:07 | |
*** fragatina has quit IRC | 00:08 | |
*** csmart has quit IRC | 00:08 | |
*** retr0h has quit IRC | 00:09 | |
*** mkrai has quit IRC | 00:09 | |
*** jovial[m] has quit IRC | 00:09 | |
*** mrunge has quit IRC | 00:14 | |
*** mjturek has quit IRC | 00:14 | |
*** jroll has quit IRC | 00:16 | |
*** jroll has joined #openstack-ironic | 00:17 | |
*** zhangfei has joined #openstack-ironic | 00:19 | |
*** hshiina has joined #openstack-ironic | 00:21 | |
*** Goneri has joined #openstack-ironic | 01:01 | |
*** cjloader has joined #openstack-ironic | 01:02 | |
*** tiendc has joined #openstack-ironic | 01:06 | |
*** cjloader has quit IRC | 01:06 | |
*** phuongnh has joined #openstack-ironic | 01:15 | |
*** liuzz_ has quit IRC | 01:23 | |
openstackgerrit | Hironori Shiina proposed openstack/ironic master: Stop verifying updated driver in creating task https://review.openstack.org/561169 | 01:30 |
---|---|---|
*** Goneri has quit IRC | 01:41 | |
*** mrunge has joined #openstack-ironic | 01:44 | |
*** rnoriega_ has joined #openstack-ironic | 02:01 | |
*** weshay_ has joined #openstack-ironic | 02:01 | |
*** lhinds|out has joined #openstack-ironic | 02:02 | |
*** cjloader has joined #openstack-ironic | 02:02 | |
*** lhinds- has joined #openstack-ironic | 02:03 | |
*** weshay has quit IRC | 02:03 | |
*** weshay has joined #openstack-ironic | 02:03 | |
*** rnoriega- has joined #openstack-ironic | 02:04 | |
*** lhinds has quit IRC | 02:05 | |
*** rnoriega has quit IRC | 02:06 | |
*** weshay_ has quit IRC | 02:06 | |
*** rnoriega_ has quit IRC | 02:06 | |
*** lhinds|out has quit IRC | 02:06 | |
*** cjloader has quit IRC | 02:07 | |
*** cjloader has joined #openstack-ironic | 02:08 | |
*** cjloader has quit IRC | 02:13 | |
openstackgerrit | Hironori Shiina proposed openstack/ironic-lib master: uncap eventlet https://review.openstack.org/560500 | 02:15 |
openstackgerrit | Hironori Shiina proposed openstack/ironic-lib master: fix lower constraints https://review.openstack.org/560585 | 02:16 |
*** trown|outtypewww has quit IRC | 02:18 | |
*** gcb has joined #openstack-ironic | 02:33 | |
*** cjloader has joined #openstack-ironic | 02:58 | |
*** zhangfei has quit IRC | 03:01 | |
*** cjloader has quit IRC | 03:06 | |
*** d0ugal_ has joined #openstack-ironic | 03:14 | |
*** zhangfei has joined #openstack-ironic | 03:14 | |
*** d0ugal has quit IRC | 03:16 | |
*** cjloader has joined #openstack-ironic | 03:16 | |
*** cjloader has quit IRC | 03:21 | |
*** fragatina has joined #openstack-ironic | 03:32 | |
*** stendulker has joined #openstack-ironic | 03:46 | |
*** cjloader has joined #openstack-ironic | 03:57 | |
*** penick_ has joined #openstack-ironic | 04:02 | |
*** ildikov has quit IRC | 04:03 | |
*** penick has quit IRC | 04:03 | |
*** cjloader has quit IRC | 04:03 | |
*** d0ugal_ has quit IRC | 04:03 | |
*** d0ugal__ has joined #openstack-ironic | 04:03 | |
*** andymccr has quit IRC | 04:05 | |
*** andymccr has joined #openstack-ironic | 04:05 | |
*** pbourke has quit IRC | 04:05 | |
*** pbourke has joined #openstack-ironic | 04:06 | |
*** phuongnh has quit IRC | 04:09 | |
*** phuongnh has joined #openstack-ironic | 04:10 | |
openstackgerrit | Hironori Shiina proposed openstack/ironic-lib master: fix lower constraints and uncap eventlet https://review.openstack.org/560500 | 04:14 |
*** asettle_ has joined #openstack-ironic | 04:14 | |
*** asettle has quit IRC | 04:16 | |
*** zer0c00l has quit IRC | 04:16 | |
openstackgerrit | Merged openstack/ironic master: Remove vifs upon teardown https://review.openstack.org/556748 | 04:19 |
*** phuongnh has quit IRC | 04:20 | |
*** phuongnh has joined #openstack-ironic | 04:21 | |
*** zer0c00l has joined #openstack-ironic | 04:22 | |
*** mvpnitesh has joined #openstack-ironic | 04:29 | |
*** zhangfei has quit IRC | 04:51 | |
*** cjloader has joined #openstack-ironic | 04:57 | |
*** zhangfei has joined #openstack-ironic | 05:07 | |
*** cjloader has quit IRC | 05:07 | |
*** sapd_ has quit IRC | 05:07 | |
*** e0ne has joined #openstack-ironic | 05:08 | |
*** e0ne has quit IRC | 05:08 | |
*** sapd_ has joined #openstack-ironic | 05:08 | |
*** marios has joined #openstack-ironic | 05:23 | |
*** links has joined #openstack-ironic | 05:23 | |
*** jovial[m] has joined #openstack-ironic | 05:23 | |
*** mkrai has joined #openstack-ironic | 05:30 | |
*** csmart has joined #openstack-ironic | 05:33 | |
*** retr0h has joined #openstack-ironic | 05:33 | |
*** hjensas has quit IRC | 05:44 | |
openstackgerrit | jiapei proposed openstack/ironic master: Fix XClarity parameters discrepancy https://review.openstack.org/561405 | 05:47 |
*** gcb has quit IRC | 05:50 | |
*** mjura has joined #openstack-ironic | 05:52 | |
*** hoonetorg has quit IRC | 05:59 | |
*** cjloader has joined #openstack-ironic | 06:02 | |
*** rbartal has joined #openstack-ironic | 06:06 | |
*** zhangfei has quit IRC | 06:10 | |
*** cjloader has quit IRC | 06:12 | |
*** hoonetorg has joined #openstack-ironic | 06:12 | |
*** pcaruana has joined #openstack-ironic | 06:16 | |
*** priteau has joined #openstack-ironic | 06:23 | |
*** zhangfei has joined #openstack-ironic | 06:24 | |
*** e0ne has joined #openstack-ironic | 06:32 | |
*** ianychoi has joined #openstack-ironic | 06:38 | |
*** parlos has joined #openstack-ironic | 06:50 | |
*** hjensas has joined #openstack-ironic | 06:50 | |
*** hjensas has quit IRC | 06:50 | |
*** hjensas has joined #openstack-ironic | 06:50 | |
*** cjloader has joined #openstack-ironic | 06:56 | |
*** rcernin has quit IRC | 06:57 | |
*** phuongnh has quit IRC | 06:59 | |
*** cjloader has quit IRC | 07:01 | |
*** tesseract has joined #openstack-ironic | 07:14 | |
*** phuongnh has joined #openstack-ironic | 07:23 | |
*** mgoddard has joined #openstack-ironic | 07:30 | |
openstackgerrit | yolanda.robla proposed openstack/ironic master: BIOS Settings: Add BIOSInterface https://review.openstack.org/507793 | 07:30 |
*** AlexeyAbashkin has joined #openstack-ironic | 07:31 | |
openstackgerrit | yolanda.robla proposed openstack/ironic master: BIOS Settings: Add DB model https://review.openstack.org/511162 | 07:37 |
openstackgerrit | yolanda.robla proposed openstack/ironic master: BIOS Settings: Add BIOSInterface https://review.openstack.org/507793 | 07:37 |
openstackgerrit | yolanda.robla proposed openstack/ironic master: Add bios_interface db field https://review.openstack.org/528609 | 07:37 |
openstackgerrit | yolanda.robla proposed openstack/ironic master: BIOS Settings: Add DB API https://review.openstack.org/511402 | 07:38 |
openstackgerrit | yolanda.robla proposed openstack/ironic master: BIOS Settings: Add BIOS caching https://review.openstack.org/512200 | 07:38 |
openstackgerrit | yolanda.robla proposed openstack/ironic master: WIP: Add Node BIOS support - REST API https://review.openstack.org/512579 | 07:38 |
openstackgerrit | yolanda.robla proposed openstack/ironic master: BIOS Settings: Add RPC object https://review.openstack.org/511714 | 07:38 |
openstackgerrit | Kaifeng Wang proposed openstack/ironic-inspector master: [WIP] Wrapping with oslo.service and SIGHUP support https://review.openstack.org/561823 | 07:52 |
*** cjloader has joined #openstack-ironic | 07:57 | |
*** lucas-brb is now known as lucasagomes | 08:04 | |
*** ianychoi has quit IRC | 08:04 | |
*** d0ugal__ has quit IRC | 08:05 | |
*** d0ugal has joined #openstack-ironic | 08:05 | |
*** d0ugal has quit IRC | 08:05 | |
*** d0ugal has joined #openstack-ironic | 08:05 | |
*** cjloader has quit IRC | 08:07 | |
*** jtomasek has quit IRC | 08:08 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Fix ``agent`` deploy interface to call ``boot.prepare_instance`` https://review.openstack.org/499050 | 08:09 |
*** ianychoi has joined #openstack-ironic | 08:10 | |
openstackgerrit | Hironori Shiina proposed openstack/ironic-python-agent master: fix lower constraints and uncap eventlet https://review.openstack.org/560501 | 08:11 |
*** dougsz has joined #openstack-ironic | 08:16 | |
*** rnoriega- is now known as rnoriega | 08:23 | |
*** stendulker has quit IRC | 08:25 | |
*** serlex has joined #openstack-ironic | 08:27 | |
*** serlex has left #openstack-ironic | 08:28 | |
*** niteshmvp has joined #openstack-ironic | 08:29 | |
openstackgerrit | zenghui.shi proposed openstack/ironic master: BIOS Settings: Add DB model https://review.openstack.org/511162 | 08:32 |
openstackgerrit | zenghui.shi proposed openstack/ironic master: BIOS Settings: Add bios_interface db field https://review.openstack.org/528609 | 08:32 |
openstackgerrit | zenghui.shi proposed openstack/ironic master: BIOS Settings: Add DB API https://review.openstack.org/511402 | 08:32 |
*** derekh has joined #openstack-ironic | 08:32 | |
*** mvpnitesh has quit IRC | 08:32 | |
openstackgerrit | Kaifeng Wang proposed openstack/ironic-inspector master: Support reload configuration on SIGHUP https://review.openstack.org/560243 | 08:35 |
*** erolg has joined #openstack-ironic | 08:36 | |
*** devananda has quit IRC | 08:36 | |
*** namnh has joined #openstack-ironic | 08:45 | |
*** Nisha_Agarwal has joined #openstack-ironic | 08:46 | |
*** Nisha_ has joined #openstack-ironic | 08:57 | |
*** jtomasek has joined #openstack-ironic | 08:57 | |
*** hshiina has quit IRC | 08:57 | |
*** Nisha_Agarwal has quit IRC | 09:00 | |
*** Nisha_ has quit IRC | 09:00 | |
*** cjloader has joined #openstack-ironic | 09:02 | |
*** cjloader has quit IRC | 09:07 | |
openstackgerrit | Kaifeng Wang proposed openstack/ironic-inspector master: Use database to store introspection_data https://review.openstack.org/514552 | 09:16 |
*** mvp_nitesh has joined #openstack-ironic | 09:26 | |
*** niteshmvp has quit IRC | 09:30 | |
*** niteshmvp has joined #openstack-ironic | 09:39 | |
*** mvp_nitesh has quit IRC | 09:42 | |
*** cjloader has joined #openstack-ironic | 10:00 | |
openstackgerrit | Debayan Ray proposed openstack/sushy master: Add storage https://review.openstack.org/561863 | 10:02 |
*** namnh has quit IRC | 10:10 | |
*** cjloader has quit IRC | 10:12 | |
*** jtomasek has quit IRC | 10:59 | |
*** cjloader has joined #openstack-ironic | 11:02 | |
*** serlex has joined #openstack-ironic | 11:05 | |
*** AlexeyAbashkin has quit IRC | 11:07 | |
*** cjloader has quit IRC | 11:07 | |
*** lucasagomes is now known as lucas-hungry | 11:14 | |
openstackgerrit | Merged openstack/ironic master: Resolve pep8 E402 errors and no longer ignore E402 https://review.openstack.org/560188 | 11:16 |
*** zhangfei has quit IRC | 11:23 | |
*** Taseer has joined #openstack-ironic | 11:26 | |
*** AlexeyAbashkin has joined #openstack-ironic | 11:28 | |
*** dprince has joined #openstack-ironic | 11:29 | |
Taseer | Hi all, I am running bifrost on centos 7. It errors out that pip does not have a "-c" option. The logs are => https://hastebin.com/omuqabojaj.sql | 11:33 |
*** jtomasek has joined #openstack-ironic | 11:39 | |
openstackgerrit | Kaifeng Wang proposed openstack/ironic-inspector master: [WIP] Introduce oslo.messaging and sync rpc call https://review.openstack.org/561915 | 11:41 |
*** Taseer has left #openstack-ironic | 11:42 | |
*** phuongnh has quit IRC | 11:43 | |
*** bfournie has quit IRC | 11:47 | |
derekh | parlos: vdrok thanks for the reviews, I've submitted the same patch to queens, if you have time to take a look, https://review.openstack.org/#/c/561919/ | 11:56 |
patchbot | patch 561919 - ironic-staging-drivers (stable/queens) - Convert ovirt url to a str | 11:56 |
*** trown has joined #openstack-ironic | 12:00 | |
*** cjloader has joined #openstack-ironic | 12:02 | |
*** cjloader has quit IRC | 12:06 | |
*** exodusftw has quit IRC | 12:14 | |
*** tiendc has quit IRC | 12:14 | |
*** lucas-hungry is now known as lucasagomes | 12:19 | |
*** bfournie has joined #openstack-ironic | 12:19 | |
*** mgoddard has quit IRC | 12:21 | |
*** exodusftw has joined #openstack-ironic | 12:21 | |
jroll | morning ironic | 12:36 |
*** hjensas has quit IRC | 12:44 | |
jroll | woo, I see ironic-grenade-dsvm-multinode-multitenant is pretty green this morning \o/ | 12:46 |
jroll | relatedly, I learned about this page this morning http://zuul.openstack.org/builds.html | 12:46 |
*** mgoddard has joined #openstack-ironic | 12:54 | |
*** rloo has joined #openstack-ironic | 12:55 | |
*** milan_ has joined #openstack-ironic | 13:01 | |
*** cjloader has joined #openstack-ironic | 13:02 | |
*** niteshmvp has quit IRC | 13:02 | |
*** cjloader has quit IRC | 13:07 | |
*** ianychoi_ has joined #openstack-ironic | 13:07 | |
*** jcoufal has joined #openstack-ironic | 13:10 | |
*** ianychoi has quit IRC | 13:10 | |
*** jcoufal_ has joined #openstack-ironic | 13:14 | |
*** Goneri has joined #openstack-ironic | 13:15 | |
* etingof is glaring at http://logs.openstack.org/73/526773/22/check/ironic-dsvm-standalone/cc62d96/job-output.txt.gz#_2018-04-17_11_24_37_552574 | 13:17 | |
*** jcoufal has quit IRC | 13:17 | |
*** jcoufal has joined #openstack-ironic | 13:21 | |
TheJulia | good morning | 13:24 |
*** jcoufal_ has quit IRC | 13:24 | |
TheJulia | that seems like a bug | 13:25 |
*** _milan_ has joined #openstack-ironic | 13:27 | |
*** milan_ has quit IRC | 13:28 | |
jroll | ya, poking around I don't see any obvious reason that patch would cause that | 13:30 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Make ansible error message clearer https://review.openstack.org/561944 | 13:37 |
jroll | ^ related, easy review, not a fix though | 13:37 |
*** links has quit IRC | 13:38 | |
jroll | even weirder, the node that validation failed for was deployed successfully earlier O_o | 13:46 |
TheJulia | information can change | 13:46 |
jroll | right | 13:46 |
TheJulia | And I think it said rescue, which makes me think it has to do with our rescue fix | 13:47 |
*** asettle_ is now known as asettle | 13:47 | |
jroll | did we merge something in nova for that? | 13:49 |
*** hjensas has joined #openstack-ironic | 13:50 | |
*** hjensas has quit IRC | 13:50 | |
*** hjensas has joined #openstack-ironic | 13:50 | |
rloo | Good morning ironic'ers, TheJulia, jroll, etingof. Things looking rosy today? | 13:50 |
jroll | rloo: maybe! :) | 13:51 |
rloo | jroll: put on your rose-coloured glasses! | 13:51 |
jroll | TheJulia: or do you mean this one? https://review.openstack.org/#/c/499050/ | 13:51 |
patchbot | patch 499050 - ironic - Fix ``agent`` deploy interface to call ``boot.prep... | 13:51 |
etingof | rloo, Sahara dust storm is over, it's sunny again \o/ | 13:51 |
jroll | which isn't merged yet, but failing in the same way | 13:51 |
rloo | etingof: :) | 13:51 |
rloo | ouch... looks like we have a different problem today... | 13:52 |
*** etingof is now known as etingof|brb | 13:52 | |
rloo | this passes the CI: https://review.openstack.org/#/c/511402/47 | 13:54 |
patchbot | patch 511402 - ironic - BIOS Settings: Add DB API | 13:54 |
*** cdearborn has joined #openstack-ironic | 13:55 | |
openstackgerrit | Merged openstack/ironic stable/pike: Pin pycodestyle to <=2.3.1 https://review.openstack.org/560556 | 13:55 |
jroll | seems it just started: http://zuul.openstack.org/builds.html?job_name=ironic-dsvm-standalone | 13:56 |
rloo | did we change something recently? | 14:00 |
jroll | I don't see anything obvious in commit logs | 14:00 |
jroll | I guess it could be some kind of race? | 14:00 |
jroll | nothing new in the tempest plugin since april 9 | 14:01 |
*** cjloader has joined #openstack-ironic | 14:01 | |
TheJulia | jroll: afaik it has not yet merged in nova, and I meant that fix, but I've not looked to figure out exaclty where it is being generated from | 14:01 |
jroll | TheJulia: it comes from ansible deploy interface validation, the 'ipmi' driver is a red herring, see https://review.openstack.org/561944 to fix that bit | 14:02 |
patchbot | patch 561944 - ironic - Make ansible error message clearer | 14:02 |
TheJulia | yeah, already reviewed that | 14:02 |
jroll | ah ok | 14:02 |
TheJulia | and rloo approved | 14:02 |
jroll | I don't see us setting capabilities in the tempest plugin | 14:03 |
jroll | oh, it's instance_info/capabilities, hrm | 14:04 |
*** r-daneel has joined #openstack-ironic | 14:04 | |
rloo | this is the test that is failing, where is the ansible deploy interface being set? https://github.com/openstack/ironic-tempest-plugin/blob/master/ironic_tempest_plugin/tests/scenario/ironic_standalone/test_basic_ops.py#L224 | 14:06 |
* TheJulia sips coffee trying to wake up | 14:06 | |
*** cjloader has quit IRC | 14:06 | |
*** baha has joined #openstack-ironic | 14:10 | |
jroll | rloo: there's a tempest test in the standalone job that uses the ansible deploy interface | 14:10 |
jroll | https://github.com/openstack/ironic-tempest-plugin/blob/master/ironic_tempest_plugin/tests/scenario/ironic_standalone/test_basic_ops.py#L186 | 14:10 |
jroll | ah, I see why you ask, something is wrong | 14:10 |
jroll | freenode is super laggy for me right now btw :| | 14:10 |
*** cjloader has joined #openstack-ironic | 14:11 | |
*** johnsom has quit IRC | 14:11 | |
rloo | well, i haven't looked at this stuff before and am unfamiliar with how these tests are done. just want to make sure i understand... | 14:11 |
jroll | enabled_deploy_interfaces = iscsi,direct,ansible | 14:11 |
jroll | so it shouldn't be using the ansible interface, hrm | 14:12 |
rloo | hmm, i thought it'd use iscsi then. it should be the first if it isn't specified. | 14:12 |
rloo | the test should test that :) | 14:12 |
jroll | right | 14:12 |
derekh | looking at one of the jobs on ironic-staging-drivers stable/queens , does anybody know if there was a fix for this in master ? http://logs.openstack.org/35/561935/1/check/ironic-staging-drivers-dsvm-all-drivers/d2ca14a/logs/devstacklog.txt.gz | 14:12 |
derekh | 2018-04-17 13:31:25.796 | error in setup command: Error parsing /opt/stack/new/ironic-staging-drivers/setup.cfg: ImportError: No module named pep8 | 14:12 |
*** johnsom has joined #openstack-ironic | 14:12 | |
jroll | ugh | 14:13 |
jroll | derekh: there is, one sec | 14:13 |
jroll | derekh: we'll need something like this on stable/queens of that repo: https://review.openstack.org/#/c/561358/ | 14:13 |
patchbot | patch 561358 - networking-generic-switch (stable/queens) - Ensure pep8 is present before trying to proceed (MERGED) | 14:13 |
jroll | same issue, no clue why pep8 isn't found, it's clearly installed | 14:14 |
* rloo is unsure whether to be worried cuz now we have 2, or not worried cuz it isn't an isolated case. | 14:15 | |
derekh | jroll: thanks, I'll see if I can try something similar for ironic-staging-drivers | 14:15 |
jroll | derekh: thanks, if you need a hand I can try to help in a bit when I finish dealing with master CI problems | 14:16 |
jroll | :) | 14:16 |
derekh | jroll: ack, thanks | 14:16 |
jroll | rloo: I think this means we're sharing nodes between tests: https://github.com/openstack/ironic-tempest-plugin/blob/master/ironic_tempest_plugin/tests/scenario/baremetal_standalone_manager.py#L384 | 14:16 |
jroll | that should be a static method, I think | 14:17 |
*** milan_ has joined #openstack-ironic | 14:17 | |
jroll | this is likely a race we just haven't noticed yet | 14:17 |
rloo | jroll: if you say so. i mean, we shoudn't be sharing, so if that is doing / sharing, then it is wrong. | 14:17 |
jroll | I could be totally wrong though, feels like we should have noticed this affecting other tests earlier | 14:18 |
jroll | rloo: it's kind of a guess based on how classmethods work | 14:18 |
*** rpioso|afk is now known as rpioso | 14:18 | |
*** cjloader has quit IRC | 14:18 | |
rpioso | Good morning, ironicers | 14:18 |
jroll | it could be that we've been missing it this whole time. and bonus, we're probably not testing what we think :) :( | 14:19 |
*** mikal has quit IRC | 14:19 | |
*** mikal has joined #openstack-ironic | 14:20 | |
*** _milan_ has quit IRC | 14:20 | |
*** mjturek has joined #openstack-ironic | 14:21 | |
rloo | jroll: i took a look at tempest code: https://github.com/openstack/tempest/blob/master/tempest/scenario/manager.py#L1110 | 14:28 |
rloo | jroll: not a class method. vs ours: https://github.com/openstack/ironic-tempest-plugin/blob/master/ironic_tempest_plugin/tests/scenario/baremetal_standalone_manager.py#L47 | 14:28 |
jroll | rloo: well, that isn't setting self.network or anything, so not quite the same | 14:28 |
rloo | jroll: yeah, i just realized that. i think i need to actually read the code which i don't want to do right now. | 14:29 |
jroll | the problem is the setting of cls.node instead of returning a node, I think | 14:29 |
* jroll is also reading code | 14:29 | |
jroll | oh, we don't create nodes in this, hm | 14:30 |
jroll | OH | 14:31 |
jroll | so we get a node and reserve it (with instance_uuid) when setting up the test | 14:31 |
jroll | and update it with any attributes specified | 14:31 |
jroll | and so if we end up picking a node that's using the ansible deploy interface, since we don't declare which one we need, it just uses that | 14:32 |
jroll | and if we land on an otherwise incompatible node, boom | 14:32 |
rloo | well, that explains it | 14:32 |
* jroll fixes | 14:33 | |
jroll | though, I don't have a good way in mind to avoid this in the future | 14:33 |
* jroll wonders why ansible driver doesn't support partition images with netboot, but pushes that thought aside for later | 14:35 | |
* jroll would rather just kill netboot | 14:35 | |
rloo | jroll: add comments? | 14:36 |
jroll | rloo: I added one in that test, however that only helps if someone sees it (or in other words copy/pastes the right test) | 14:36 |
rloo | wonder if we should comment here: https://github.com/openstack/ironic-tempest-plugin/blob/master/ironic_tempest_plugin/tests/scenario/baremetal_standalone_manager.py#L318 | 14:39 |
jroll | maybe. I'd be surprised if people read it, but it can't hurt | 14:39 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic-tempest-plugin master: Explicitly use iscsi for standalone BaremetalIpmiRescuePartitioned https://review.openstack.org/561963 | 14:39 |
jroll | or maybe a docstring on that class, since it affects more than just the deploy_interface attribut | 14:40 |
jroll | e | 14:40 |
rloo | jroll: i think the problem is that i assumed if deploy_interface wasn't specified, that it'd use the default based on the hwtype & config, but that's not what is going on. | 14:40 |
jroll | rloo: yep, agree | 14:40 |
rloo | jroll: that is used to set the *interface, after a node is selected :-( | 14:40 |
jroll | I guess another way to handle it would be to reset all of those things that aren't set | 14:41 |
* jroll makes a poc patch for that | 14:41 | |
rloo | jroll: well, that depends. what if the test does want to randomly pick a node, regardless of X_interface value. | 14:41 |
jroll | then they can set deploy_interface = random.choice(CONF.baremetal.enabled_deploy_interfaces) :P | 14:42 |
rloo | jroll: heh. that'd work too. add that to the comments then! | 14:42 |
*** mjura has quit IRC | 14:46 | |
*** cjloader has joined #openstack-ironic | 14:47 | |
jroll | hrm, we don't have a way to set back to default, I'd have to store it somewhere | 14:48 |
*** cjloader has quit IRC | 14:48 | |
*** cjloader has joined #openstack-ironic | 14:48 | |
*** jcoufal_ has joined #openstack-ironic | 14:52 | |
*** jcoufal has quit IRC | 14:54 | |
rloo | jroll: i commented on your 561963. Do we need a story for it? | 14:55 |
rloo | I'll update the whiteboard... | 14:56 |
jroll | rloo: replied | 14:57 |
jroll | I don't personally care to make a story, but can if you want ¯\_(ツ)_/¯ | 14:57 |
rloo | jroll: if we haven't tended to do this in the past, i won't bother. it isn't user-facing. | 14:58 |
TheJulia | I also feel it is a fairly small issue/fix so not really in need of a story and all | 14:58 |
TheJulia | also not user facing | 14:58 |
*** Nisha_ has joined #openstack-ironic | 15:00 | |
*** etingof|brb is now known as etingof | 15:01 | |
jroll | yeah, not sure how I want to "fix this for real", going to set it aside | 15:03 |
jroll | behind enough on downstream right now | 15:03 |
*** Nisha_ has quit IRC | 15:05 | |
rloo | TheJulia: it was more of an 'other folks may make the same mistake, here's a story about it'. | 15:09 |
rloo | and yeah, gotta do downstream stuff... :-( | 15:09 |
NobodyCam | Good Morning Ironic'ers | 15:10 |
NobodyCam | :) | 15:10 |
rloo | morning NobodyCam | 15:13 |
NobodyCam | Good Morning rloo :) | 15:13 |
*** rbartal has quit IRC | 15:14 | |
*** e0ne has quit IRC | 15:21 | |
TheJulia | rloo: w/r/t question yesterday: https://storyboard.openstack.org/#!/story/2001870 | 15:23 |
rloo | TheJulia: thx. looking... | 15:25 |
TheJulia | the long thread continues | 15:26 |
*** pcaruana has quit IRC | 15:28 | |
rloo | TheJulia: what long thread? | 15:30 |
TheJulia | trying to achieve versioning sanity for the client and nova | 15:30 |
rloo | TheJulia: In general, isn't it a good idea to provide api-version support on an ironic-per-call basis? | 15:31 |
TheJulia | I found a few more "lets do this" items but didn't link to them since they don't really matter anymore | 15:31 |
rloo | TheJulia: i understand this particular use case but I would like to step back/up and have an idea of where we're going with this new api-version parameter. | 15:31 |
* TheJulia wants to thow her laptop against the wall | 15:32 | |
TheJulia | https://bugs.launchpad.net/nova/+bug/1739440 | 15:32 |
openstack | Launchpad bug 1739440 in OpenStack Compute (nova) "Ironic virt driver is hardcoding Ironic API microversion" [Wishlist,In progress] - Assigned to Julia Kreger (juliaashleykreger) | 15:32 |
TheJulia | All I want to do right now is the needful | 15:33 |
*** baha has quit IRC | 15:35 | |
*** marios has quit IRC | 15:35 | |
* TheJulia seriously contemplates giving up on this issue | 15:40 | |
* rloo goes to lunch | 15:42 | |
*** serlex has quit IRC | 15:45 | |
*** parlos has quit IRC | 16:12 | |
*** links has joined #openstack-ironic | 16:15 | |
*** baha has joined #openstack-ironic | 16:16 | |
*** pcaruana has joined #openstack-ironic | 16:18 | |
*** gyee has joined #openstack-ironic | 16:19 | |
*** fragatina has quit IRC | 16:21 | |
*** harlowja has joined #openstack-ironic | 16:23 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: [devstack] Switch ironic to uWSGI https://review.openstack.org/507067 | 16:27 |
*** r-daneel_ has joined #openstack-ironic | 16:29 | |
*** r-daneel has quit IRC | 16:30 | |
*** r-daneel_ is now known as r-daneel | 16:30 | |
*** trown is now known as trown|lunch | 16:35 | |
*** e0ne has joined #openstack-ironic | 16:36 | |
*** e0ne has quit IRC | 16:42 | |
*** e0ne has joined #openstack-ironic | 16:43 | |
*** Nisha_Agarwal has joined #openstack-ironic | 16:44 | |
*** jcoufal_ has quit IRC | 16:51 | |
*** jcoufal has joined #openstack-ironic | 16:52 | |
*** harlowja has quit IRC | 16:52 | |
*** milan_ has quit IRC | 16:56 | |
*** lucasagomes is now known as lucas-afk | 16:57 | |
*** harlowja has joined #openstack-ironic | 16:57 | |
*** derekh has quit IRC | 16:58 | |
*** dougsz has quit IRC | 17:00 | |
*** fragatina has joined #openstack-ironic | 17:03 | |
*** mgoddard has quit IRC | 17:05 | |
*** erolg has quit IRC | 17:07 | |
*** harlowja has quit IRC | 17:20 | |
*** jcoufal_ has joined #openstack-ironic | 17:22 | |
*** jcoufal has quit IRC | 17:25 | |
* sambetts|afk --> gome | 17:35 | |
sambetts|afk | home | 17:35 |
sambetts|afk | night all | 17:35 |
sambetts|afk | o/ | 17:35 |
*** links has quit IRC | 17:35 | |
*** r-daneel_ has joined #openstack-ironic | 17:42 | |
*** mgoddard has joined #openstack-ironic | 17:42 | |
*** r-daneel has quit IRC | 17:43 | |
*** r-daneel_ is now known as r-daneel | 17:43 | |
*** links has joined #openstack-ironic | 17:47 | |
*** dsneddon has quit IRC | 17:51 | |
*** dsneddon has joined #openstack-ironic | 17:52 | |
*** trown|lunch is now known as trown | 17:53 | |
*** dsneddon has quit IRC | 17:54 | |
*** dsneddon has joined #openstack-ironic | 17:54 | |
*** mgoddard has quit IRC | 17:59 | |
*** jcoufal_ has quit IRC | 18:02 | |
*** AlexeyAbashkin has quit IRC | 18:02 | |
*** jcoufal has joined #openstack-ironic | 18:02 | |
* TheJulia goes and finds lunch | 18:02 | |
*** tesseract has quit IRC | 18:06 | |
rloo | jroll: http://logs.openstack.org/63/561963/1/check/ironic-tempest-dsvm-ironic-inspector-pike/af2bc0f/logs/devstacklog.txt.gz#_2018-04-17_15_21_57_960 | 18:10 |
rloo | jroll: it is spreading :-( That's from your patch: https://review.openstack.org/#/c/561963/ | 18:11 |
patchbot | patch 561963 - ironic-tempest-plugin - Explicitly use iscsi for standalone BaremetalIpmiR... | 18:11 |
*** jcoufal has quit IRC | 18:15 | |
*** fragatina has quit IRC | 18:18 | |
*** Nisha_Agarwal has quit IRC | 18:25 | |
jroll | rloo: what in the | 18:51 |
jroll | I may try to take this to infra again | 18:51 |
jroll | oh that's pike, hm | 18:51 |
rloo | jroll: queens failed too but i didn't look at those logs. just assumed the same thing. | 18:52 |
jroll | probably | 18:52 |
rloo | jroll: for completeness sake. yes, same failure in queens. with ironic-inspector. | 18:53 |
jroll | ya, I wonder what's different in stable that causes this | 18:54 |
* rloo wonders too | 18:55 | |
jroll | I don't have it in me to look into this today | 18:55 |
*** r-daneel_ has joined #openstack-ironic | 19:01 | |
*** r-daneel has quit IRC | 19:03 | |
*** r-daneel_ is now known as r-daneel | 19:03 | |
*** e0ne has quit IRC | 19:13 | |
*** athomas has quit IRC | 19:16 | |
*** athomas has joined #openstack-ironic | 19:21 | |
*** AlexeyAbashkin has joined #openstack-ironic | 19:25 | |
*** r-daneel_ has joined #openstack-ironic | 19:29 | |
*** r-daneel has quit IRC | 19:30 | |
*** r-daneel_ is now known as r-daneel | 19:30 | |
rpioso | I'm having some uefi fun today :-) | 19:33 |
*** AlexeyAbashkin has quit IRC | 19:34 | |
rpioso | I successfully set up an RDO undercloud. To verify it, I began by deploying a server with boot mode configured to be legacy BIOS. | 19:35 |
rpioso | That worked splendidly. | 19:35 |
rpioso | I deleted that server and its ironic node. | 19:35 |
rpioso | I changed the baremetal server's boot mode to be UEFI, configured the correct NIC port to PXE boot, and powered it off. | 19:37 |
rpioso | I then attempted to create a server, similar to what I did before -- openstack server create ... | 19:38 |
rpioso | The server's console displayed, "Booting from PXE Device 3: Integrated NIC Port 3 Partition 1" | 19:39 |
rpioso | and then | 19:39 |
rpioso | "Downloading NBP file..." | 19:39 |
rpioso | "Succeed to download NBP file" | 19:40 |
rpioso | "Boot Failed: PXE Device 3: Integrated NIC Port 3 Partition 1" | 19:40 |
rpioso | "No boot device available or Operating System detected." | 19:40 |
rpioso | Does anyone have an idea why it's failing? | 19:41 |
rpioso | Or how to proceed to troubleshoot it? | 19:41 |
*** harlowja has joined #openstack-ironic | 19:50 | |
*** cjloader has quit IRC | 19:51 | |
*** cjloader_ has joined #openstack-ironic | 19:51 | |
*** cjloader_ has quit IRC | 19:52 | |
*** cjloader has joined #openstack-ironic | 20:02 | |
openstackgerrit | Ruby Loo proposed openstack/ironic-tempest-plugin master: Add comments for [deploy|rescue]_interface https://review.openstack.org/562037 | 20:09 |
*** fragatina has joined #openstack-ironic | 20:18 | |
*** cjloader has quit IRC | 20:20 | |
*** cjloader has joined #openstack-ironic | 20:21 | |
*** MattMan has quit IRC | 20:28 | |
*** MattMan has joined #openstack-ironic | 20:29 | |
*** dprince has quit IRC | 20:31 | |
*** fragatina has quit IRC | 20:32 | |
*** fragatina has joined #openstack-ironic | 20:37 | |
*** sapd__ has joined #openstack-ironic | 20:40 | |
*** links has quit IRC | 20:42 | |
*** pcaruana has quit IRC | 20:42 | |
*** sapd_ has quit IRC | 20:43 | |
*** jtomasek has quit IRC | 20:49 | |
*** cjloader has quit IRC | 20:57 | |
*** trown is now known as trown|outtypewww | 21:01 | |
*** cjloader has joined #openstack-ironic | 21:08 | |
*** cjloader has quit IRC | 21:08 | |
*** cjloader has joined #openstack-ironic | 21:08 | |
*** baha has quit IRC | 21:10 | |
*** priteau has quit IRC | 21:22 | |
*** Goneri has quit IRC | 21:23 | |
*** bfournie has quit IRC | 21:40 | |
*** cjloader has quit IRC | 21:53 | |
*** rh-jelabarre has quit IRC | 21:58 | |
*** rcernin has joined #openstack-ironic | 22:42 | |
*** r-daneel has quit IRC | 22:46 | |
*** r-daneel has joined #openstack-ironic | 22:52 | |
*** rpioso is now known as rpioso|afk | 23:12 | |
*** r-daneel has quit IRC | 23:18 | |
*** r-daneel has joined #openstack-ironic | 23:47 | |
*** bfournie has joined #openstack-ironic | 23:49 | |
*** Goneri has joined #openstack-ironic | 23:52 | |
*** r-daneel has quit IRC | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!