*** MattMan has quit IRC | 00:12 | |
*** MattMan has joined #openstack-ironic | 00:12 | |
*** zshi has joined #openstack-ironic | 00:19 | |
*** rh-jelabarre has joined #openstack-ironic | 00:23 | |
*** bfournie has joined #openstack-ironic | 00:37 | |
*** r-daneel has quit IRC | 00:40 | |
*** zshi_ has joined #openstack-ironic | 00:56 | |
*** zshi has quit IRC | 00:57 | |
*** lifeless has quit IRC | 01:06 | |
*** lifeless has joined #openstack-ironic | 01:11 | |
*** zhangfei has joined #openstack-ironic | 01:23 | |
openstackgerrit | zenghui.shi proposed openstack/python-ironicclient master: BIOS Settings support https://review.openstack.org/569318 | 01:37 |
---|---|---|
*** lifeless has quit IRC | 01:47 | |
*** lifeless has joined #openstack-ironic | 01:49 | |
*** zhangfei has quit IRC | 01:55 | |
*** r-daneel has joined #openstack-ironic | 02:01 | |
*** zhangfei has joined #openstack-ironic | 02:07 | |
*** lifeless has quit IRC | 02:19 | |
*** kiseok7 has quit IRC | 02:20 | |
*** lifeless has joined #openstack-ironic | 02:20 | |
*** hshiina has joined #openstack-ironic | 02:53 | |
*** pvc has joined #openstack-ironic | 03:35 | |
pvc | hi | 03:36 |
pvc | ipmi_address | 03:36 |
pvc | is what? | 03:36 |
pvc | eth1 ip? | 03:36 |
pvc | anyone help me setting up ironic? | 03:36 |
*** yolanda has joined #openstack-ironic | 03:44 | |
*** lifeless_ has joined #openstack-ironic | 03:46 | |
*** lifeless has quit IRC | 03:46 | |
*** yolanda_ has quit IRC | 03:46 | |
*** lifeless has joined #openstack-ironic | 03:52 | |
*** lifeless_ has quit IRC | 03:52 | |
*** bfournie has quit IRC | 04:06 | |
*** trungnv has quit IRC | 04:13 | |
*** links has joined #openstack-ironic | 04:14 | |
*** trungnv has joined #openstack-ironic | 04:14 | |
*** rh-jelabarre has quit IRC | 04:18 | |
openstackgerrit | Nguyen Van Trung proposed openstack/ironic master: Support raid configuration for BM via irmc driver https://review.openstack.org/512979 | 04:42 |
*** jaganathan has joined #openstack-ironic | 04:51 | |
*** jaganathan has quit IRC | 04:59 | |
*** kiseok7 has joined #openstack-ironic | 05:02 | |
*** tiendc has joined #openstack-ironic | 05:10 | |
openstackgerrit | Merged openstack/ironic master: Adds more `ipmitool` errors as retryable https://review.openstack.org/567217 | 05:14 |
openstackgerrit | Merged openstack/ironic master: Make method public to support out-of-band cleaning https://review.openstack.org/570109 | 05:14 |
*** e0ne has joined #openstack-ironic | 05:28 | |
*** e0ne has quit IRC | 05:33 | |
*** trungnv has quit IRC | 05:34 | |
*** marios has quit IRC | 05:46 | |
*** marios has joined #openstack-ironic | 05:47 | |
*** rbartal has joined #openstack-ironic | 05:51 | |
openstackgerrit | yolanda.robla proposed openstack/ironic-tempest-plugin master: Add manual clean step ironic standalone test https://review.openstack.org/568616 | 05:57 |
*** hshiina2 has joined #openstack-ironic | 06:01 | |
*** AlexeyAbashkin has joined #openstack-ironic | 06:04 | |
*** hshiina has quit IRC | 06:04 | |
*** pcaruana has joined #openstack-ironic | 06:26 | |
*** hshiina has joined #openstack-ironic | 06:27 | |
*** arundhati has joined #openstack-ironic | 06:29 | |
arundhati | hi | 06:30 |
*** hshiina2 has quit IRC | 06:30 | |
arundhati | While creating a ironic node with json file, The token generation is happening twice. is it a expected behavior? | 06:30 |
*** parlos has joined #openstack-ironic | 06:43 | |
*** arundhati has quit IRC | 06:55 | |
*** arundhati has joined #openstack-ironic | 06:57 | |
*** zhangfei has quit IRC | 06:58 | |
yolanda | hi dtantsur|afk , i have a question about https://review.openstack.org/#/c/568551/14/ironic_tempest_plugin/tests/scenario/ironic_standalone/test_bios.py | 07:06 |
patchbot | patch 568551 - ironic-tempest-plugin - Add bios testing to plugin | 07:06 |
yolanda | i added some comments to your review | 07:06 |
*** rcernin has quit IRC | 07:10 | |
*** jtomasek has joined #openstack-ironic | 07:14 | |
*** marlinc has joined #openstack-ironic | 07:14 | |
*** zhangfei has joined #openstack-ironic | 07:15 | |
*** jaganathan has joined #openstack-ironic | 07:25 | |
*** zhangfei has quit IRC | 07:31 | |
*** sekharvajjula has joined #openstack-ironic | 07:38 | |
*** trungnv has joined #openstack-ironic | 07:41 | |
*** arundhati has quit IRC | 07:42 | |
*** ajya has joined #openstack-ironic | 07:44 | |
*** zhangfei has joined #openstack-ironic | 07:45 | |
*** links has quit IRC | 07:50 | |
*** lifeless_ has joined #openstack-ironic | 08:06 | |
*** lifeless has quit IRC | 08:07 | |
*** arundhati has joined #openstack-ironic | 08:07 | |
*** namnh has joined #openstack-ironic | 08:08 | |
*** dougsz has joined #openstack-ironic | 08:11 | |
*** athomas has joined #openstack-ironic | 08:20 | |
*** d0ugal has joined #openstack-ironic | 08:28 | |
*** derekh has joined #openstack-ironic | 08:31 | |
*** pvc has quit IRC | 08:37 | |
*** dtantsur|afk is now known as dtantsur | 08:40 | |
dtantsur | yolanda: hi! I don't see how this is related to putting or not putting tests in one case | 08:41 |
dtantsur | they still (are supposed to) run independently | 08:42 |
*** mtsv has joined #openstack-ironic | 08:46 | |
*** lifeless has joined #openstack-ironic | 08:46 | |
*** lifeless_ has quit IRC | 08:47 | |
openstackgerrit | Derek Higgins proposed openstack/ironic master: WIP: Testing a basic zuulv3 job https://review.openstack.org/573244 | 08:47 |
yolanda | dtantsur, so i need to move my nodes to deleted state initially, to execute the cleanup step. To move to deleted, i need that the node is active. But after i finish each individual test, the node is not on that state, it's on available state | 08:51 |
yolanda | if i split the 3 cases on 3 clases, the cleanup/boot is done automatically | 08:51 |
yolanda | but if i do on the same test, the state of the node is ok for the first one, but not for the second | 08:51 |
yolanda | if i want to do on the same test, i'd have to deal with the states manually | 08:51 |
dtantsur | yolanda: no, we need to fix our tests to work individually. all tests have to be independent, I'm not sure why we have interdependencies between tests of the same case, but that's a bug IMO. | 08:52 |
yolanda | dtantsur, it's because the boot/cleanup is done per class, not per test | 08:53 |
dtantsur | yolanda: well, that's not quite right IMO. but wait, why do you have to use "deleted" if you're testing manual cleaning? | 08:54 |
yolanda | mm, i guess that for the first test, because i need to be in manageable state, and the node starts in active state | 08:56 |
yolanda | after that it will be ok to set from available to manageable | 08:56 |
dtantsur | yolanda: so you actually need to apply "manage" on a node in the setUp if the node is in "available"? | 09:00 |
yolanda | dtantsur, what i could do is on the test setup, first move to deleting there | 09:02 |
yolanda | then on each test, ensure that we are on manageable state | 09:02 |
dtantsur | yolanda: actually, it has been bothering me for quite some time that we boot instances in setUp. our tests actually test booting instances, so it should happen in the test methods. | 09:03 |
dtantsur | if we move it, you won't have this problem. | 09:03 |
dtantsur | and your tests will work faster, because you won't do unnecessary provisioning each time | 09:04 |
yolanda | dtantsur, then a general change needs to be done for the baremetal standalone manager | 09:04 |
yolanda | right? | 09:04 |
dtantsur | right, yeah | 09:04 |
openstackgerrit | Ilya Etingof proposed openstack/ironic master: Adds boot mode support to ManagementInterface https://review.openstack.org/526773 | 09:06 |
vdrok | good morning ironic | 09:07 |
dtantsur | hey vdrok | 09:10 |
vdrok | dtantsur: morning! | 09:10 |
openstackgerrit | Ilya Etingof proposed openstack/ironic master: Added Redfish boot mode management https://review.openstack.org/528314 | 09:11 |
*** e0ne has joined #openstack-ironic | 09:12 | |
etingof | good summer morning everyone! ;-) | 09:13 |
dtantsur | hey etingof! is it warm out there? | 09:13 |
etingof | dtantsur, quite warm, 29C | 09:14 |
etingof | water temperature is above 22C | 09:14 |
vdrok | morning etingof :) | 09:14 |
etingof | vdrok, o/ | 09:14 |
dtantsur | etingof: nice! it finally cooled down here a bit - from >30 to 26 :) | 09:15 |
etingof | dtantsur, oh, that's good! 30+ in the city may be hard to tolerate... | 09:16 |
dtantsur | yup | 09:16 |
etingof | but you can always cool down in a lake | 09:16 |
dtantsur | well, right | 09:16 |
dtantsur | a big less convenient when the most active part of your work day happens when it's the hottest :D | 09:17 |
yolanda | dtantsur, do you want to take that action, or do you prefer that i try to change that behaviour? | 09:19 |
dtantsur | yolanda: I won't have time today, so please do if you have some | 09:20 |
openstackgerrit | Ilya Etingof proposed openstack/ironic master: Move boot-related code to boot_mode_utils.py https://review.openstack.org/569382 | 09:23 |
yolanda | i'll try | 09:28 |
yolanda | dtantsur, so remove the boot node and cleanup node from the class methods, and then each test individually should call those methods? | 09:29 |
dtantsur | yolanda: clean up should probably stay, but not booting | 09:33 |
yolanda | dtantsur, so tests could specifically could locate nodes depending on their needs? like "try to find one node in available state" | 09:36 |
dtantsur | yolanda: mmm, I think all nodes should be available in the beginning, just as devstack creates them | 09:38 |
*** pbourke has quit IRC | 09:43 | |
*** pbourke has joined #openstack-ironic | 09:43 | |
yolanda | i could add one flag, that does the node boot if needed, or just return an available node if not set | 09:45 |
*** arnewiebalck has quit IRC | 09:47 | |
dtantsur | yolanda: when we test node boot, it has to happens inside the test | 09:48 |
dtantsur | it's just weird to have the main test action in the preparation phase.. | 09:48 |
* dtantsur -> doctor appt, back later | 09:48 | |
*** dtantsur is now known as dtantsur|bbl | 09:49 | |
yolanda | dtantsur|bbl, for baremetal standalone manager? | 09:49 |
yolanda | i don't see any specific test for booting there | 09:49 |
yolanda | initially, only the ping ip, and rescue was there | 09:50 |
yolanda | i added the cleanup and bios | 09:50 |
*** dprince has joined #openstack-ironic | 10:23 | |
*** dprince has quit IRC | 10:31 | |
*** namnh has quit IRC | 10:34 | |
openstackgerrit | Derek Higgins proposed openstack/ironic master: WIP: Testing a basic zuulv3 job https://review.openstack.org/573244 | 10:36 |
*** hshiina2 has joined #openstack-ironic | 10:42 | |
*** jrollen is now known as jroll | 10:44 | |
*** hshiina has quit IRC | 10:44 | |
jroll | morning ironic | 10:46 |
*** arundhati has quit IRC | 10:46 | |
sambetts|afk | o/ | 10:46 |
*** arundhati has joined #openstack-ironic | 10:52 | |
openstackgerrit | Olivier Bourdon proposed openstack/ironic-lib master: Expose GPT partitioning fixing method https://review.openstack.org/568518 | 10:55 |
*** zzzeek_ has quit IRC | 11:01 | |
*** zzzeek has joined #openstack-ironic | 11:03 | |
*** lucasagomes is now known as lucas-hungry | 11:15 | |
*** hshiina2 has quit IRC | 11:15 | |
*** lucas-hungry is now known as lucasagomes | 11:15 | |
*** tiendc has quit IRC | 11:16 | |
*** arundhati has quit IRC | 11:18 | |
*** jroll has quit IRC | 11:18 | |
*** jroll has joined #openstack-ironic | 11:18 | |
*** bfournie has joined #openstack-ironic | 11:23 | |
*** bfournie has quit IRC | 11:24 | |
*** arundhati has joined #openstack-ironic | 11:36 | |
*** e0ne has quit IRC | 11:38 | |
*** rh-jelabarre has joined #openstack-ironic | 11:42 | |
*** lifeless has quit IRC | 11:45 | |
*** lifeless_ has joined #openstack-ironic | 11:45 | |
*** zhangfei has quit IRC | 11:47 | |
*** e0ne has joined #openstack-ironic | 11:48 | |
openstackgerrit | yolanda.robla proposed openstack/ironic-tempest-plugin master: WIP: Do not boot nodes if not set in standalone tests https://review.openstack.org/574228 | 11:53 |
openstackgerrit | Derek Higgins proposed openstack/ironic master: WIP: Testing a basic zuulv3 job https://review.openstack.org/573244 | 11:58 |
*** parlos has quit IRC | 12:01 | |
*** arundhati has quit IRC | 12:01 | |
*** bfournie has joined #openstack-ironic | 12:09 | |
*** markguz_ has joined #openstack-ironic | 12:10 | |
*** trown has joined #openstack-ironic | 12:12 | |
*** markguz_ has quit IRC | 12:23 | |
*** arnewiebalck_ has joined #openstack-ironic | 12:24 | |
openstackgerrit | zhangzhaoshan proposed openstack/bifrost master: replace http with https https://review.openstack.org/574239 | 12:25 |
*** markguz_ has joined #openstack-ironic | 12:26 | |
*** serlex has joined #openstack-ironic | 12:37 | |
*** pcaruana has quit IRC | 12:39 | |
*** jistr is now known as jistr|mtg | 12:41 | |
*** e0ne has quit IRC | 12:42 | |
*** AlexeyAbashkin has quit IRC | 12:42 | |
*** parlos has joined #openstack-ironic | 12:45 | |
*** e0ne has joined #openstack-ironic | 12:47 | |
openstackgerrit | yolanda.robla proposed openstack/ironic-tempest-plugin master: WIP: Do not boot nodes if not set in standalone tests https://review.openstack.org/574228 | 12:56 |
*** jistr|mtg is now known as jistr | 12:57 | |
*** rloo has joined #openstack-ironic | 12:57 | |
*** arundhati has joined #openstack-ironic | 13:00 | |
*** jcoufal has joined #openstack-ironic | 13:00 | |
*** arnewiebalck_ has quit IRC | 13:01 | |
*** baha has joined #openstack-ironic | 13:02 | |
rloo | good morning ironic'ers! | 13:03 |
*** markguz_ has quit IRC | 13:03 | |
rloo | I was looking at the gate status on our whiteboard. are ironic-lib & IPA stable branches still broken? | 13:04 |
rloo | dtantsur|bbl: have you seen this? failing cuz 'No conductor service registered which supports driver fake.' Did we (you) remove the 'fake' driver? (I'm still catching up on ironic stuff so may find out on my own...) | 13:06 |
sambetts|afk | dtantsur|bbl removed it at the end of last week, https://review.openstack.org/572401 https://review.openstack.org/572454 | 13:07 |
patchbot | patch 572401 - ironic - Remove the "fake" and "fake_soft_power" classic dr... (MERGED) | 13:07 |
patchbot | patch 572454 - ironic - Remove the fake_agent, fake_pxe and fake_inspector... (MERGED) | 13:07 |
openstackgerrit | Chuck Short proposed openstack/networking-generic-switch master: Switch to using stestr https://review.openstack.org/574248 | 13:07 |
rloo | sambetts|afk: thx, that explains it then. guess we need to update the client. wonder if we should trigger tests for all our projects, just in case... | 13:07 |
rloo | sambetts|afk: now i am wondering if we were too fast with that. wonder if other folks were using 'fake' driver... | 13:08 |
sambetts|afk | its been deprecated for a significant about of time, and its still available if they move to the fake hw type | 13:09 |
rloo | (although I'm sure we'll find out cuz someone is bound to ask/mention it...) | 13:09 |
*** arnewiebalck has joined #openstack-ironic | 13:10 | |
sambetts|afk | all CI stuff should be using hw types by now anyway, but there was a problem with devstack-gate appending the "fake" driver to the end of the enabled_drivers config option because its branchless | 13:10 |
sambetts|afk | but I'm pretty sure there was a fix for that in dtantsur|bbl's patch in the devstack plugin | 13:10 |
rloo | sambetts|afk: hmm, i'm still catching up on stuff. if no one else gets to it, i'll take a look at the client. it is failing in functional tests, wonder if it is hardcoded somewhere. | 13:12 |
sambetts|afk | hmmm, its likely we've forgotten to move a test or two over the hw types, but at least we'll find them this way | 13:13 |
*** zul has quit IRC | 13:15 | |
*** Goneri has joined #openstack-ironic | 13:17 | |
openstackgerrit | Derek Higgins proposed openstack/ironic master: WIP: Testing a basic zuulv3 job https://review.openstack.org/573244 | 13:18 |
*** zul has joined #openstack-ironic | 13:19 | |
*** dtantsur|bbl is now known as dtantsur | 13:25 | |
dtantsur | yolanda: well, right. the actual test is not "ping ip", it's "boot and ping ip", but we hide the boot stage in setUp | 13:25 |
dtantsur | rloo: yeah, sorry, I might have missed the client's functional tests | 13:26 |
dtantsur | also morning | 13:26 |
rloo | dtantsur: no worries, that is what tests are for :) | 13:26 |
rloo | dtantsur: i haven't yet had a chance to look into it | 13:27 |
rloo | dtantsur: i'm 'stuck' right now, reading a thread about using storyboard. wondering if i should say something. | 13:27 |
dtantsur | rloo: about making it a goal? if you feel uncomfortable with SB (I know you do), please say it aloud :) | 13:28 |
dtantsur | we don't want people just silently suffer :) | 13:28 |
rloo | dtantsur: about why i don't think it is usable right now | 13:28 |
dtantsur | rloo: ++ I think it's a helpful feedback | 13:28 |
dtantsur | and yes, indeed, our functional tests use "fake" explicitly. wanna me fix it? | 13:28 |
rloo | dtantsur: why doesn't someone else say that though. i don't want to be negative :-( | 13:28 |
dtantsur | rloo: nobody wants to be negative? :) I can say something, I hate the search so far | 13:29 |
rloo | dtantsur: yeah, sure, make you a deal. you fix that and i'll send email. Or if you'd rather, you send email, and I'll fix :) | 13:29 |
dtantsur | my fate is to s/fake/fake-hardware/ this cycle, so I'll obey it | 13:29 |
rloo | dtantsur: thx, you fix fake stuff and send email. i think i'll also send email cuz now i feel guilty for *not* saying anything! | 13:30 |
*** markguz_ has joined #openstack-ironic | 13:30 | |
dtantsur | what is the thread's subject? I think I've lost it | 13:30 |
yolanda | dtantsur, that's the initial proposal.. https://review.openstack.org/#/c/574228/ . But what i can do is call the boot_node from the test directly, and remove from the class setup | 13:32 |
patchbot | patch 574228 - ironic-tempest-plugin - WIP: Do not boot nodes if not set in standalone tests | 13:32 |
openstackgerrit | zhulingjie proposed openstack/networking-baremetal master: fix tox python3 overrides https://review.openstack.org/574256 | 13:32 |
*** pcaruana has joined #openstack-ironic | 13:32 | |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient master: Switch functional tests to the fake-hardware hardware type https://review.openstack.org/574258 | 13:35 |
dtantsur | rloo: let's try this ^^^ | 13:35 |
rloo | dtantsur: thx, will wait for CI first to make my reviewing easier :) | 13:35 |
dtantsur | yolanda: see inline; I'd prefer to make explicit calls rather than more class-level flags | 13:36 |
yolanda | dtantsur, ok, makes sense | 13:37 |
*** cjloader_ has quit IRC | 13:51 | |
rloo | dtantsur: do you know what needs to be done here, is stable/pike broken for ipa? https://review.openstack.org/#/c/571595/ | 13:56 |
patchbot | patch 571595 - ironic-python-agent (stable/pike) - Fix Dockerfile for Debian Jessie | 13:56 |
dtantsur | lemme find | 14:02 |
*** ianychoi has quit IRC | 14:02 | |
*** markguz_ has quit IRC | 14:02 | |
*** AlexeyAbashkin has joined #openstack-ironic | 14:04 | |
rloo | I've updated the gate status (in our whiteboard). I guessed about stuff that I think were fixed :) | 14:06 |
openstackgerrit | Derek Higgins proposed openstack/ironic master: WIP: Testing a basic zuulv3 job https://review.openstack.org/573244 | 14:08 |
*** mjturek has joined #openstack-ironic | 14:10 | |
*** Alexey_Abashkin has joined #openstack-ironic | 14:11 | |
*** zul is now known as zul_ | 14:11 | |
dtantsur | rloo: https://review.openstack.org/#/c/574258/ passed CI | 14:11 |
patchbot | patch 574258 - python-ironicclient - Switch functional tests to the fake-hardware hardw... | 14:11 |
rloo | dtantsur: thx for letting me know. looking... | 14:12 |
openstackgerrit | Derek Higgins proposed openstack/ironic master: WIP: Testing a basic zuulv3 job https://review.openstack.org/573244 | 14:13 |
dtantsur | rloo: as to IPA Pike, I think we need https://review.openstack.org/#/c/522025/ squashed into the same patch | 14:13 |
*** AlexeyAbashkin has quit IRC | 14:13 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 14:13 | |
patchbot | patch 522025 - ironic-python-agent - Use 'functional' tox environment (MERGED) | 14:13 |
dtantsur | vdrok: if you're around: https://review.openstack.org/#/c/574258/ | 14:13 |
patchbot | patch 574258 - python-ironicclient - Switch functional tests to the fake-hardware hardw... | 14:13 |
*** zul_ is now known as zul | 14:14 | |
vdrok | dtantsur: yup, thanks, looking | 14:14 |
needssleep | dtantsur: can you run the meeting today? I’m stuck in the land of visa fun :( | 14:15 |
*** needssleep is now known as TheJulia | 14:15 | |
dtantsur | TheJulia: totally | 14:15 |
rloo | dtantsur: okay... my squash-fu is bad. anyone want to do that? | 14:15 |
dtantsur | rloo: I can give it a try | 14:16 |
rloo | dtantsur: thx. /me reviews the power fault patch now... | 14:16 |
dtantsur | rloo: oh, actually. the functional job is non-voting. /me rechecks | 14:18 |
*** arundhati has quit IRC | 14:18 | |
TheJulia | dtantsur: thanks | 14:20 |
NobodyCam | Good Morning Ironic'ers | 14:21 |
*** arundhati has joined #openstack-ironic | 14:24 | |
rloo | good morning NobodyCam :) | 14:25 |
dtantsur | hey NobodyCam, happy Monday :) | 14:25 |
NobodyCam | Good Morning rloo and dtantsur, its defiantly a Monday :p :) | 14:26 |
dtantsur | no kidding | 14:26 |
* dtantsur adds more coffee to the mix | 14:26 | |
NobodyCam | ++++ | 14:26 |
*** rpioso|afk is now known as rpioso | 14:30 | |
rpioso | Good morning | 14:30 |
*** cjloader has joined #openstack-ironic | 14:30 | |
NobodyCam | good morning rpioso | 14:30 |
rpioso | NobodyCam: o/ | 14:31 |
NobodyCam | ::) | 14:31 |
rloo | dtantsur: wrt that functional tox env (https://review.openstack.org/#/c/522025/). I guess we might want to backport, otherwise that non-voting job will always fail? | 14:39 |
patchbot | patch 522025 - ironic-python-agent - Use 'functional' tox environment (MERGED) | 14:39 |
dtantsur | rloo: yes, just land the dockerfile fix first | 14:39 |
dtantsur | no need to squash them | 14:39 |
rloo | dtantsur: ok. assuming that's why it fails, of course, I didn't look, did you? or we can just cherry-pick and see what happens... | 14:39 |
openstackgerrit | Merged openstack/python-ironicclient master: Switch functional tests to the fake-hardware hardware type https://review.openstack.org/574258 | 14:40 |
dtantsur | rloo: well, I looked back then when I left the comment | 14:40 |
rloo | dtantsur: good enuf for me! | 14:40 |
*** hshiina has joined #openstack-ironic | 14:45 | |
*** r-daneel has quit IRC | 14:50 | |
*** arundhati has quit IRC | 14:52 | |
dtantsur | your last chance to update the statuses :) the meeting is coming | 14:52 |
dtantsur | TheJulia: wdyt about doing some releases this week? I think ironic stable/queens has stuff, but also maybe master? | 14:56 |
TheJulia | dtantsur: I was thinking we should have sufficient stuffs to release | 14:57 |
TheJulia | Brain cells in visa application land | 14:57 |
dtantsur | TheJulia: my only concern is releasing semi-finished driver removal.. but it may be fine, since we only removed fake drivers so far | 14:57 |
dtantsur | okie, won't distract you :) will check stable branches first | 14:58 |
rloo | i just +A'd the client part for power fault, so that could be released too | 14:58 |
TheJulia | rloo: yay! | 14:58 |
dtantsur | rloo: maybe we want to wait for BIOS stuff to land in the client as well? | 14:58 |
rloo | oh. is that the only thing left for bios? | 14:58 |
TheJulia | The client is a little more important, all things considered | 14:58 |
dtantsur | rloo: no, there are more things, but I agree that the client is now the next priority | 14:59 |
* dtantsur begins the countdown to the meeting | 14:59 | |
*** stendulker has joined #openstack-ironic | 14:59 | |
dtantsur | #startmeeting ironic | 15:00 |
openstack | Meeting started Mon Jun 11 15:00:01 2018 UTC and is due to finish in 60 minutes. The chair is dtantsur. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
*** openstack changes topic to " (Meeting topic: ironic)" | 15:00 | |
openstack | The meeting name has been set to 'ironic' | 15:00 |
rloo | o/ | 15:00 |
* dtantsur waves | 15:00 | |
etingof | o/ | 15:00 |
stendulker | o/ | 15:00 |
bdodd | o/ | 15:00 |
jroll | \o | 15:00 |
hshiina | o/ | 15:00 |
TheJulia | o/ | 15:00 |
*** beekneemech is now known as bnemec | 15:00 | |
dtantsur | #link https://wiki.openstack.org/wiki/Meetings/Ironic our agenda for today | 15:00 |
dtantsur | looks pretty empty :) | 15:00 |
rpioso | o/ | 15:00 |
dtantsur | #Announcements / Reminder | 15:01 |
dtantsur | #info Some releases to be requested this week | 15:01 |
dtantsur | I don't have a full list, will check what is needed | 15:01 |
dtantsur | TheJulia: anything else to announce? | 15:01 |
dtantsur | #chair TheJulia | 15:01 |
openstack | Current chairs: TheJulia dtantsur | 15:01 |
* dtantsur assumes Julia is fighting the visa monster right as we speak | 15:02 | |
TheJulia | Not that I have on my mind at the moment, aside from intent to try and release some things this week | 15:02 |
dtantsur | cool :) anyone anything else? | 15:02 |
TheJulia | S/monster/hydra/ | 15:02 |
dtantsur | true | 15:02 |
dtantsur | #topic Review action items from previous meeting | 15:02 |
*** openstack changes topic to "Review action items from previous meeting (Meeting topic: ironic)" | 15:03 | |
dtantsur | #link http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-06-04-15.00.html | 15:03 |
dtantsur | no action items apparently? | 15:03 |
jroll | we did good! | 15:03 |
dtantsur | indeed! | 15:03 |
yolanda | can i get reviews on https://review.openstack.org/#/c/568616/ again? i changed one deprecation bug | 15:03 |
patchbot | patch 568616 - ironic-tempest-plugin - Add manual clean step ironic standalone test | 15:03 |
jroll | yeah, was a short meeting | 15:03 |
dtantsur | yolanda: sure, after the meeting | 15:03 |
dtantsur | #topic Review subteam status reports | 15:03 |
*** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)" | 15:03 | |
yolanda | doh sorry | 15:03 |
dtantsur | #link https://etherpad.openstack.org/p/IronicWhiteBoard starting with line 164 | 15:04 |
dtantsur | TheJulia: when the hydra is defeated, please elaborate on line 206 there | 15:04 |
dtantsur | "Operators are requesting network specific documentation" | 15:04 |
* TheJulia sharpens sword | 15:05 | |
dtantsur | heh | 15:05 |
*** bnemec is now known as beekneemech | 15:06 | |
*** arundhati has joined #openstack-ironic | 15:06 | |
*** beekneemech is now known as bnemec | 15:06 | |
dtantsur | TheJulia: can we remove the "Migrate to Storyboard" topic? while it's far from being perfect, the migration is done, we even have some basic bug stats | 15:06 |
rloo | wrt api versioning with nova. L230? Dmitry asked a question in patch 557850. Do we need to discuss? | 15:07 |
openstackgerrit | Derek Higgins proposed openstack/ironic master: WIP: Testing a basic zuulv3 job https://review.openstack.org/573244 | 15:07 |
patchbot | https://review.openstack.org/#/c/557850/ - python-ironicclient - Support per-call version: set_provision_state | 15:07 |
TheJulia | dtantsur: works for me | 15:07 |
dtantsur | rloo: good idea | 15:07 |
rloo | dtantsur: i didn't realize it was done. just update it saying it is done, (vs 'in progress') | 15:07 |
TheJulia | I’ve not seen the the question on the patch | 15:08 |
jroll | > After working with openstacsdk for some time and talking to Monty about all this stuff, I'm no longer sure this is the direction we should take. Can someone remind me why we cannot just deduce the correct microversion from the input? | 15:08 |
dtantsur | let's bring this on the open discussion, which is quite soon? | 15:08 |
jroll | sure | 15:09 |
rloo | dtantsur: wrt bugs then, L167. is that as of today? | 15:09 |
dtantsur | rloo: yep | 15:09 |
dtantsur | I was out for some time, so the diff is with May | 15:09 |
dtantsur | * out last Monday, I guess | 15:09 |
TheJulia | ++ to open discussion | 15:09 |
jroll | 241 untriaged out of 283? | 15:10 |
rloo | dtantsur: ok thx. i was ignoring those cuz i didn't realize they were actually updated! | 15:10 |
dtantsur | jroll: well, we have to mark them as triaged again.. | 15:10 |
dtantsur | I guess this was not moved from LP | 15:10 |
jroll | oh. | 15:10 |
jroll | :| | 15:10 |
dtantsur | yep :( | 15:10 |
rloo | that seems... wrong. 241 out of 283 untriaged? | 15:10 |
dtantsur | rloo: it may be a bug in my tool, but it also may be that we don't have the "triaged" tag on things we triaged back in LP | 15:10 |
TheJulia | Also we had consensus that we needed to wave through them. It is a lot better than six underdogs untreated | 15:10 |
rloo | TheJulia triaged a lot of them. i find it hard to believe we only triaged 42 of them. | 15:10 |
* jroll tries to figure out what autocorrect did that | 15:11 | |
jroll | s/that/there/ | 15:11 |
TheJulia | rloo: some were closed/already fixed | 15:11 |
TheJulia | Yeah, autocorrect hates me | 15:11 |
* rloo sighs | 15:11 | |
openstackgerrit | Merged openstack/python-ironicclient master: Power fault recovery: client support https://review.openstack.org/556774 | 15:12 |
dtantsur | I can double-check the tool next time | 15:12 |
dtantsur | (or anyone can, it's not much code) | 15:12 |
dtantsur | #link https://github.com/dtantsur/ironic-bug-report | 15:12 |
rloo | question about the NOTE at L268... what does it mean? someone needs to migrate the rest of the stuff into storyboard? | 15:12 |
dtantsur | I think the plan was to create a board per large effort | 15:13 |
dtantsur | kind of a trello board | 15:13 |
dtantsur | which is the way storyboard is supposed to be used | 15:13 |
dtantsur | (we use it still the old way, it's one of the sources of our problems with it) | 15:13 |
rloo | ok. not going to ask for an action item on that :) | 15:14 |
TheJulia | rloo: it was just tracking/visibility wise | 15:14 |
rloo | sambetts|afk: when you have a chance, would you please update the Cisco UCS status (L388ish). | 15:14 |
TheJulia | What dtantsur said | 15:15 |
* etingof can take a look at dtantsur's ironic-bug-report tool | 15:15 | |
dtantsur | etingof++ | 15:15 |
* rloo done with status | 15:16 | |
TheJulia | 15 more people until I reach the hydra... | 15:16 |
dtantsur | can we remove "Queens high priorities"? | 15:16 |
* rloo roots for TheJulia | 15:16 | |
TheJulia | dtantsur: yes | 15:16 |
dtantsur | or is it there for historical purposes? | 15:16 |
dtantsur | okay | 15:16 |
rloo | dtantsur: nope. not until they get moved to storyboard | 15:17 |
* etingof starts to think that TheJulia is trying to get Russian visa | 15:17 | |
rloo | dtantsur: that's what that NOTE was about, right? | 15:17 |
dtantsur | rloo: Queens priorities? no, the NOTE is for current things | 15:17 |
TheJulia | I kept it originally to make sure we wrapped the queens stuff up | 15:17 |
dtantsur | it's line 300 | 15:17 |
rloo | dtantsur: i thought it was for 'everything after this'. i mean, isn't that what storyboard is for? to track this stuff? | 15:17 |
TheJulia | It was separate | 15:18 |
dtantsur | rloo: I assumed that these things are mostly finished? | 15:18 |
dtantsur | Rescue has a nova part, but nova (IIRC) does not use SB | 15:18 |
TheJulia | I thought they were delineated with another note | 15:18 |
rloo | shall we discuss each of these now? we could. | 15:19 |
dtantsur | okay. Routed networks - done. Rescue - Nova+Tempest left. Zuul V3 is duplicated with line 276. | 15:19 |
jroll | let's delete the routed networks and zuul v3 at least then | 15:20 |
dtantsur | we could probably leave rescue with these two patches, but everything else can be removed, I guess | 15:20 |
rloo | i just looked at rescue patch (last ironic patch for rescue?) https://review.openstack.org/#/c/528699/. it links to launchpad, not storyboard. presumably we have a storyboard link for it, so it is being tracked so we can remove the rescue section? | 15:20 |
patchbot | patch 528699 - ironic-tempest-plugin - Rescue mode tempest tests | 15:20 |
*** rbartal has quit IRC | 15:21 | |
TheJulia | rloo: because nova has not migrated | 15:21 |
TheJulia | Oh, our tempest, yeah the tag should get updated | 15:22 |
TheJulia | We can’t do anything until the nova portion lands | 15:22 |
dtantsur | right | 15:22 |
rloo | give me a sec, i'm adding a comment to storyboard for resuce, to include the stuff in whiteboard. then i'll delete from whiteboard. | 15:23 |
rloo | ok, i'm going to delete the rescue mode stuff. anyone disagree? (you have 3 secs) | 15:24 |
TheJulia | No disagreement | 15:24 |
jroll | +1 | 15:24 |
rloo | gone | 15:25 |
rloo | what about the py 3.5 stuff? | 15:25 |
*** r-daneel has joined #openstack-ironic | 15:25 | |
rloo | how are we tracking that in storyboard? | 15:25 |
rloo | or err maybe we should do this after/outside this meeting | 15:25 |
dtantsur | yeah, let's maybe clean up separately.. | 15:27 |
dtantsur | I think openstack goals are tracked on SB now | 15:27 |
TheJulia | Py35 should have community wide tracking afaik | 15:27 |
dtantsur | globally | 15:27 |
dtantsur | yep | 15:27 |
TheJulia | 7 humans to the hydra | 15:28 |
rloo | so all that stuff wrt TODO, submit py3 job for IPA. is that in storyboard? or have we done it? | 15:28 |
rloo | do we create a subtask to track that? | 15:28 |
*** r-daneel_ has joined #openstack-ironic | 15:28 | |
rloo | wow, TheJulia's line is moving fast! | 15:28 |
dtantsur | yes, we need a subtask on the main story | 15:28 |
dtantsur | I can do it, if needed | 15:28 |
rloo | someone want to do it? we can have an action item! | 15:28 |
rloo | thx dtantsur! | 15:28 |
dtantsur | #action dtantsur to make sure a task is present for IPA support of python 3 | 15:29 |
dtantsur | anything else on statuses? | 15:29 |
rloo | inspector & wsgi stuff -- dunno what to do with that. can i leave it for you dtantsur to decide to delete or not from whiteboard? | 15:29 |
*** r-daneel has quit IRC | 15:29 | |
*** r-daneel_ is now known as r-daneel | 15:29 | |
dtantsur | I will do the same - find the story and leave it there | 15:30 |
rloo | thx dtantsur! | 15:30 |
*** rh-jelabarre has quit IRC | 15:30 | |
dtantsur | moving on? | 15:30 |
rloo | ++ | 15:30 |
*** arundhati has quit IRC | 15:30 | |
dtantsur | #topic Deciding on priorities for the coming week | 15:30 |
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)" | 15:30 | |
dtantsur | okay, some progress this week | 15:31 |
rloo | are the BIOS links ordered in priority? | 15:31 |
* dtantsur puts OSC first | 15:31 | |
*** rh-jelabarre has joined #openstack-ironic | 15:32 | |
dtantsur | power fault is done, wanna put the neutron events spec o nthe list? | 15:33 |
dtantsur | oh, so SB is not used for Pike-era goals | 15:33 |
dtantsur | ah, neutron even spec needs updates. vdrok are you on it? | 15:34 |
rloo | I am fine with what we have, i don't think we need to add more | 15:34 |
dtantsur | yeah, I guess it's good | 15:34 |
dtantsur | any opinions? | 15:34 |
* dtantsur hears crickets and assumes they agree | 15:35 | |
dtantsur | #topic Open Discussion | 15:36 |
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)" | 15:36 | |
dtantsur | so, ironicclient. blame mordred, he quite convinced me that we should stop exposing innocent souls to microversions. | 15:36 |
*** markguz_ has joined #openstack-ironic | 15:36 | |
rloo | so i was thinking maybe we should defer this until TheJulia is 100% focussed on this. | 15:37 |
dtantsur | esp. for set_provision_state where we actually know which state (+ args) corresponds to which version | 15:37 |
*** markguz_ has quit IRC | 15:37 | |
dtantsur | fine, just want to raise it to more folks | 15:37 |
dtantsur | #link https://review.openstack.org/570895 is what I'm trying with openstacksdk | 15:37 |
patchbot | patch 570895 - openstacksdk - [WIP] Add set_node_provision_state to baremetal Proxy | 15:37 |
*** markguz_ has joined #openstack-ironic | 15:38 | |
rloo | dtantsur: for now, shoudl we put a -2 or something so that client patch doesn't land? or does it matter whether it lands or not? | 15:38 |
vdrok | dtantsur: yeah, kind of, slowly progressing with local testing | 15:39 |
dtantsur | I have mixed feelings. We should have done this back in Kilo, and blocking it again.... | 15:39 |
dtantsur | thanks vdrok. we may make it a priority next week then. | 15:39 |
rloo | dtantsur: is there anything wrong with landing it? | 15:39 |
dtantsur | rloo: wrong... well, I think we will commit to a much lower-level interface that I'd like to. | 15:40 |
rloo | i will come clean and say i actually don't know the details of this stuff. | 15:40 |
dtantsur | on the other hand, I cannot commit to change it all over ironicclient | 15:40 |
dtantsur | maybe let's leave it as it is (and then eventually bury it in favor of openstacksdk) | 15:40 |
rloo | could we use this lower-level interface for now, until we hae something to replace it? | 15:40 |
jroll | do we advertise python-ironicclient's python sdk to be high level? | 15:40 |
dtantsur | * leave it as it is = land the patch | 15:40 |
dtantsur | jroll: we don't either way, I think | 15:40 |
jroll | yeah that's what I'm leaning toward dtantsur, let openstacksdk be the higher level interface | 15:40 |
dtantsur | that's an option. if we agree on that, I'm fine with proceeding with this patch. | 15:41 |
* jroll is okay with that, but would like TheJulia input | 15:41 | |
dtantsur | we just have to be ready, after some time when openstacksdk reaches 1.0, to start telling people to use it | 15:41 |
* dtantsur should probably stop using commas at all, at least until he gets a clue how to do in in English | 15:42 | |
rloo | that makes sense to me. wonder what other projects think, cuz i think this is a good rule of thumb, and would like it if other projects did a similar thing. | 15:42 |
*** r-daneel_ has joined #openstack-ironic | 15:42 | |
jroll | dtantsur: your commas were good there fwiw | 15:42 |
etingof | commas are even worse in Russian | 15:42 |
dtantsur | hah, guessed then! :D | 15:42 |
rloo | dtantsur: the commas there were good | 15:42 |
jroll | :) | 15:42 |
rloo | (one rule, if the stuff in the commas can be removed and the rest is a complete sentence) | 15:43 |
* dtantsur tries to summon mordred again | 15:43 | |
* mordred runs in screaming and waving his arms around | 15:43 | |
dtantsur | lol | 15:43 |
* rloo runs away | 15:43 | |
* etingof is scared | 15:43 | |
dtantsur | mordred: do you know if there is any kind of a general agreement to switch over to openstacksdk eventually? | 15:43 |
rloo | has the screaming stopped? :) | 15:43 |
dtantsur | hehe | 15:43 |
TheJulia | Ugh | 15:43 |
*** r-daneel has quit IRC | 15:44 | |
*** r-daneel_ is now known as r-daneel | 15:44 | |
mordred | dtantsur: no - we haven't _generally_ agreed to that yet - it's just the thing I keep saying over and over hoping it'll gain traction | 15:44 |
* dtantsur hopes that Ugh is related to TheJulia defeating the hydra | 15:44 | |
TheJulia | If we’re not going to expose it, I’ll go down back to re-writing our entire usage in nova... | 15:44 |
TheJulia | And we should revert my last patch to python-ironicclient | 15:44 |
mordred | I think we have a little more work to do before we can fully make that push | 15:44 |
dtantsur | TheJulia: not really. if we do what I think we should, we will just have a try..except block. | 15:44 |
TheJulia | But... there is no way that will likely ever land because we did awful things had been one | 15:44 |
dtantsur | mordred: like finishing microversions ;) | 15:44 |
mordred | but I like what you said earlier - don't try to change ironicclient- and at a point in the future we'll just migrate people over | 15:44 |
mordred | dtantsur: pish | 15:45 |
mordred | *details* | 15:45 |
* TheJulia sighs | 15:45 | |
rloo | (yeah, that's where the devil is, apparently) | 15:45 |
jroll | dtantsur: right, I think we (nova/ironic) agreed to not to the "try / except 406" thing in the last ptg | 15:45 |
* TheJulia just wants consensus and to stop trying to reinvent the wheel over... and over... and over again. | 15:45 | |
jroll | I think we should just move forward | 15:45 |
dtantsur | jroll: which is... precisely how microversions are supposed to work.... | 15:45 |
jroll | we have a clear path to good code | 15:45 |
dtantsur | anyway. I'm fine with the patch. ship it! | 15:45 |
jroll | dtantsur: "how microversions should work" has changed at least a few times since they began :) | 15:46 |
dtantsur | jroll: and one of the changes was "let's not force SDK users to figure them out" ;) | 15:46 |
dtantsur | anyway, I think we have a consensus here? | 15:46 |
jroll | yeah, let's just do the damn thing | 15:46 |
dtantsur | #agreed Do the damn thing | 15:46 |
mordred | can I stop screaming and waving my arms in the air now? | 15:46 |
dtantsur | mordred: YES | 15:46 |
mordred | awesome | 15:46 |
* mordred relaxes | 15:46 | |
TheJulia | Lol | 15:47 |
* dtantsur removes earplugs | 15:47 | |
dtantsur | #undo | 15:47 |
openstack | Removing item from minutes: #agreed Do the damn thing | 15:47 |
TheJulia | Lol | 15:47 |
dtantsur | #agreed We will start supporting specifying microversions per call in ironicclient | 15:47 |
dtantsur | something like this? | 15:47 |
jroll | ++ | 15:47 |
rloo | wasn't the RFE approved for that? :) | 15:48 |
jroll | how about #agreed don't change the plan yet again :P | 15:48 |
jroll | yeah, we've already agreed to it | 15:48 |
TheJulia | jroll: yes please | 15:48 |
dtantsur | :) | 15:49 |
dtantsur | anything else? | 15:49 |
* jroll has nothing | 15:50 | |
* etingof is wondering how to attract people to his awesome virtualbmc patch.... | 15:50 | |
etingof | https://review.openstack.org/#/c/488874/ | 15:50 |
patchbot | patch 488874 - virtualbmc - multiprocess server, ZMQ-based management cli tool | 15:50 |
jroll | beer and/or cookies usually work | 15:50 |
dtantsur | ++ | 15:50 |
rloo | (screaming and waving hands doesn't work for some people) | 15:50 |
etingof | will you guys come to Brno some day? | 15:51 |
* jroll just can't be bothered to review a giant patch to help people run a test tool in production :/ | 15:51 | |
* dtantsur hears crickets screaming and waving hands | 15:51 | |
dtantsur | etingof: will you man come to Berlin? ;) | 15:51 |
dtantsur | okay, okay, it's offtopic | 15:51 |
rloo | crickets | 15:51 |
dtantsur | jroll: tbh we have problems in the CI as well, and with long-running QE environments | 15:51 |
dtantsur | it's not only people shutting their legs | 15:52 |
dtantsur | ehmm.. shooting? | 15:52 |
jroll | lol | 15:52 |
* dtantsur has a decorative plate saying "Today has been canceled. Go back to bed." | 15:52 | |
dtantsur | so, thanks all :) | 15:52 |
dtantsur | #endmeeting | 15:53 |
*** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic" | 15:53 | |
jroll | cheers | 15:53 |
openstack | Meeting ended Mon Jun 11 15:53:01 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:53 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-06-11-15.00.html | 15:53 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-06-11-15.00.txt | 15:53 |
openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-06-11-15.00.log.html | 15:53 |
*** hshiina has quit IRC | 15:57 | |
*** lifeless_ has quit IRC | 15:59 | |
*** lifeless has joined #openstack-ironic | 16:00 | |
*** marlinc has quit IRC | 16:07 | |
*** e0ne has quit IRC | 16:09 | |
openstackgerrit | Merged openstack/ironic-python-agent stable/queens: Stop using slave_scripts/install-distro-packages.sh https://review.openstack.org/564298 | 16:10 |
*** AlexeyAbashkin has quit IRC | 16:11 | |
*** rnoriega has quit IRC | 16:13 | |
*** AlexeyAbashkin has joined #openstack-ironic | 16:13 | |
*** r-daneel has quit IRC | 16:13 | |
openstackgerrit | Derek Higgins proposed openstack/ironic master: WIP: Testing a basic zuulv3 job https://review.openstack.org/573244 | 16:16 |
*** stendulker has quit IRC | 16:16 | |
*** rnoriega has joined #openstack-ironic | 16:20 | |
*** r-daneel has joined #openstack-ironic | 16:25 | |
*** baha has quit IRC | 16:26 | |
*** r-daneel has quit IRC | 16:29 | |
*** r-daneel has joined #openstack-ironic | 16:30 | |
*** rajinir has joined #openstack-ironic | 16:32 | |
*** Alexey_Abashkin has joined #openstack-ironic | 16:35 | |
*** AlexeyAbashkin has quit IRC | 16:36 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 16:36 | |
*** rloo has quit IRC | 16:37 | |
*** Alexey_Abashkin has joined #openstack-ironic | 16:42 | |
*** AlexeyAbashkin has quit IRC | 16:44 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 16:44 | |
*** baha has joined #openstack-ironic | 16:45 | |
*** trown is now known as trown|lunch | 16:45 | |
*** rloo has joined #openstack-ironic | 16:46 | |
*** rloo has quit IRC | 16:47 | |
*** rloo has joined #openstack-ironic | 16:48 | |
*** dougsz has quit IRC | 16:49 | |
openstackgerrit | Merged openstack/ironic-python-agent stable/pike: Fix Dockerfile for Debian Jessie https://review.openstack.org/571595 | 16:52 |
openstackgerrit | Merged openstack/python-ironicclient master: fix tox python3 overrides https://review.openstack.org/573336 | 16:52 |
etingof | dtantsur, looking at ironic-bug-report script, given we have three dedicated lists for triaged/untriaged/rfes at the SB, why do we use just two of the lists computing the triaged count? | 16:55 |
dtantsur | I don't remember from the top of my head | 16:56 |
dtantsur | etingof: I guess I don't get the question. I see all 3 lists used for something. | 16:57 |
etingof | dtantsur, list #283 e.g. triaged does not seem to be used in the script | 16:57 |
dtantsur | etingof: well, we use the untriaged and total list, and triaged should be total - untriaged, no? | 16:58 |
*** derekh has quit IRC | 17:00 | |
etingof | dtantsur, perhaps, I just wonder why don't we use the triaged count as offered by the SB, but compute it based on the (probably right) assumption | 17:00 |
dtantsur | etingof: I don't really see us computing the triaged count.. It's all {bugs,rfes}, then untriaged {bugs,rfes} | 17:01 |
*** gyee has joined #openstack-ironic | 17:03 | |
*** akhilaki has joined #openstack-ironic | 17:07 | |
*** e0ne has joined #openstack-ironic | 17:21 | |
*** parlos has quit IRC | 17:24 | |
*** e0ne has quit IRC | 17:25 | |
*** trown|lunch has quit IRC | 17:35 | |
*** AlexeyAbashkin has quit IRC | 17:35 | |
dtantsur | g'night | 17:37 |
*** dtantsur is now known as dtantsur|afk | 17:37 | |
rpioso | dtantsur|afk: Nite | 17:40 |
*** e0ne has joined #openstack-ironic | 17:43 | |
*** arundhati has joined #openstack-ironic | 17:45 | |
*** r-daneel has quit IRC | 18:02 | |
*** r-daneel has joined #openstack-ironic | 18:09 | |
*** arundhati has quit IRC | 18:09 | |
etingof | TheJulia, hey, the Ironic Triage Panopticon, is it yours? I'm interested in seeing these worklists settings, but I probably lack access rights for that | 18:10 |
*** e0ne has quit IRC | 18:33 | |
*** trown has joined #openstack-ironic | 18:38 | |
*** jcoufal has quit IRC | 18:42 | |
*** r-daneel has quit IRC | 18:52 | |
anupn | Hi TheJulia: a quick question, from within my network namespace, I am unable to ping the gateway ("sudo ip netns exec qdhcp-0d06e886-5f45-42e6-86f9-6d91a3f18720 ping 192.168.200.2"). What things, I should check? The address is pingable from inside the host, but not from within the namespace | 18:53 |
openstackgerrit | Merged openstack/python-ironicclient master: Switch to using stestr https://review.openstack.org/571721 | 18:56 |
*** r-daneel has joined #openstack-ironic | 18:56 | |
*** e0ne has joined #openstack-ironic | 19:07 | |
*** ElCoyote_ has joined #openstack-ironic | 19:11 | |
*** e0ne has quit IRC | 19:11 | |
*** mmethot has quit IRC | 19:18 | |
*** Goneri has quit IRC | 19:21 | |
*** murrdoc has joined #openstack-ironic | 19:25 | |
murrdoc | is there a recommended ironic ansible playbook for standalone iron | 19:25 |
murrdoc | *ironic | 19:25 |
jroll | murrdoc: bifrost :) | 19:26 |
murrdoc | kk, looking, does it have Queens support too ? | 19:26 |
jroll | https://github.com/openstack/bifrost https://docs.openstack.org/bifrost/latest/install/index.html | 19:26 |
jroll | yes | 19:26 |
jroll | it also has optional keystone and neutron support, if you need it | 19:26 |
murrdoc | awesome thank you, also can you point me at documentation for image requirements for images to use with ironic, need to use an image with a luks encrypted root | 19:27 |
murrdoc | jroll +1 | 19:28 |
* murrdoc is thankful | 19:28 | |
jroll | murrdoc: not sure about image docs... ironic can deploy a "whole disk image" which includes the partition table and everything, that's probably the best route, I would think? | 19:28 |
*** mmethot has joined #openstack-ironic | 19:34 | |
*** e0ne has joined #openstack-ironic | 19:49 | |
*** rh-jelabarre has quit IRC | 19:54 | |
-openstackstatus- NOTICE: Zuul was restarted for a software upgrade; changes uploaded or approved between 19:30 and 19:50 will need to be rechecked | 19:56 | |
*** hoonetorg has joined #openstack-ironic | 20:05 | |
*** rh-jelabarre has joined #openstack-ironic | 20:06 | |
*** r-daneel has quit IRC | 20:11 | |
*** serlex has quit IRC | 20:17 | |
*** r-daneel has joined #openstack-ironic | 20:22 | |
*** jtomasek has quit IRC | 20:23 | |
*** rwsu has quit IRC | 20:26 | |
*** AlexeyAbashkin has joined #openstack-ironic | 20:32 | |
*** r-daneel has quit IRC | 20:33 | |
* TheJulia cries tears of the visa acquisition process | 20:33 | |
TheJulia | anupn: that really sounds like things are not bridged correctly between the namespace, which typically can happen if the vlan segmenation id does not match the infrastucture or the provider network is mapped incorretly or not mapped at all | 20:35 |
TheJulia | etingof: I can grant access if you'd like to edit it :) | 20:36 |
*** mjturek has quit IRC | 20:43 | |
*** Goneri has joined #openstack-ironic | 20:46 | |
*** Alexey_Abashkin has joined #openstack-ironic | 20:46 | |
*** AlexeyAbashkin has quit IRC | 20:47 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 20:47 | |
*** r-daneel has joined #openstack-ironic | 20:50 | |
anupn | TheJulia: I am using flat network type, so devstack will remove the VLAN tag for outgoing traffic, and add tag 1 of br-int for incoming traffic. In the case of mapping, I am actually setting OVS_NETWORK_BRIDGE in local.conf | 20:51 |
*** Alexey_Abashkin has joined #openstack-ironic | 20:51 | |
*** AlexeyAbashkin has quit IRC | 20:51 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 20:51 | |
anupn | from within the namespace if I run "route -n", I can see the gateway IP address on the tap interface | 20:51 |
anupn | hence I am confused, what is blocking the packets reaching the gateway | 20:52 |
anupn | you can see here http://paste.openstack.org/show/723220/ | 20:54 |
openstackgerrit | Aija Jaunteva proposed openstack/sushy master: Introduce BIOS API https://review.openstack.org/570890 | 20:55 |
*** trown is now known as trown|outtypewww | 20:56 | |
*** arundhati has joined #openstack-ironic | 20:56 | |
*** AlexeyAbashkin has quit IRC | 20:56 | |
TheJulia | anupn: so that is the dhcp namespace... not the router namespace | 20:56 |
TheJulia | I'd double check iptables rules | 20:56 |
anupn | TheJulia: wait, but why will q-router namespace be used, when the gateway and the tap interface are in the same network? May be i am missing something. | 20:59 |
anupn | after the packet leaves the tap interface does it comes at some other network interface before coming to the gateway network interface? Because my tcpdump on tap interface shows that the packet it sent is using proper IP address for gateway | 21:03 |
TheJulia | ideally there should be a connection between the two and you should be able to ping across the namespaces... so something is very wrong in the ovs or neutron config. What... I don't know exactly | 21:05 |
TheJulia | wait, if it is sent, is it arping? | 21:06 |
*** bfournie has quit IRC | 21:06 | |
TheJulia | if the packet is getting transmitted, then it seems lke ip forwarding or firewalling related | 21:07 |
*** r-daneel has quit IRC | 21:08 | |
*** AlexeyAbashkin has joined #openstack-ironic | 21:08 | |
*** hamzy has quit IRC | 21:08 | |
*** r-daneel has joined #openstack-ironic | 21:09 | |
*** baha has quit IRC | 21:10 | |
*** hamzy has joined #openstack-ironic | 21:11 | |
anupn | I haven't checked arping, but yes it looks like ip forwarding issue. | 21:19 |
anupn | do we need to have this underlying network interface that act as gateway to be a OVS brige? Is it fine to have it as linux bridge? any clue | 21:20 |
*** cjloader has quit IRC | 21:23 | |
*** cjloader_ has joined #openstack-ironic | 21:23 | |
*** cjloader_ has quit IRC | 21:27 | |
*** lifeless_ has joined #openstack-ironic | 21:28 | |
*** lifeless has quit IRC | 21:28 | |
*** arundhati has quit IRC | 21:30 | |
*** r-daneel has quit IRC | 21:31 | |
*** rwsu has joined #openstack-ironic | 21:34 | |
*** Goneri has quit IRC | 21:37 | |
*** lifeless_ has quit IRC | 21:38 | |
*** lifeless has joined #openstack-ironic | 21:44 | |
etingof | TheJulia, I'd appreciate if you let me take a look at the filters there | 21:46 |
openstackgerrit | Derek Higgins proposed openstack/ironic master: WIP: Testing a basic zuulv3 job https://review.openstack.org/573244 | 21:49 |
*** r-daneel has joined #openstack-ironic | 21:50 | |
*** AlexeyAbashkin has quit IRC | 21:54 | |
TheJulia | etingof: sure, opening storyboard to grant access | 21:59 |
*** e0ne has quit IRC | 21:59 | |
TheJulia | etingof: done | 22:03 |
*** rcernin has joined #openstack-ironic | 22:16 | |
*** ajya has quit IRC | 22:18 | |
*** rpioso is now known as rpioso|afk | 22:18 | |
etingof | TheJulia, thanks! though I can't see current filters somehow | 22:20 |
TheJulia | give the page like a solid minute to load | 22:20 |
etingof | TheJulia, yea, it loads okay | 22:20 |
etingof | TheJulia, hmm, if a story has many tasks belonging to different projects, but the tag 'triaged' can only be applied to the whole story, can we use the 'triaged' tag at all? | 22:22 |
etingof | in other words: can we share the 'triaged' tag among multiple projects under the same story? | 22:24 |
*** r-daneel has quit IRC | 22:26 | |
*** cjloader has joined #openstack-ironic | 22:28 | |
*** cjloader has quit IRC | 22:33 | |
*** r-daneel has joined #openstack-ironic | 22:38 | |
etingof | also, if all ironic tasks within a story are not in 'todo' state, but 'triaged' tag is not set on the story, should we consider the story as triaged? | 22:39 |
*** EmilienM|PTO is now known as EmilienM | 22:41 | |
*** bfournie has joined #openstack-ironic | 22:44 | |
etingof | should we have a tag like `ironic-triaged` not to intervene with other projects' tasks within a story...? | 22:50 |
*** r-daneel has quit IRC | 22:54 | |
openstackgerrit | Merged openstack/ironic-python-agent stable/ocata: Fix Dockerfile for Debian Jessie https://review.openstack.org/571596 | 22:57 |
TheJulia | etingof: so... tags are global, and a triaged tag already existed, so I would see it as general usage and project should expect exclusive use | 23:01 |
TheJulia | I'm not sure I understand your question on evaluating task level status... | 23:02 |
TheJulia | the plus of ironic-triaged would be we could drop a lot of the cross project stuff that doesn't apply, but shows up in the list even with ironic tasks completed. | 23:03 |
* TheJulia wonders if that was the same thing | 23:03 | |
etingof | TheJulia, so we have stories without the triaged tag set, but the tasks in these stories are being worked on already (e.g. task status is not todo) - does it imply that the story is in fact triaged? | 23:04 |
TheJulia | triaged is an assessment by another contirbutor, not an indicator if someone has pushed a patch or toggled the task flag | 23:05 |
TheJulia | task status that is | 23:05 |
etingof | even if the task is merged? | 23:06 |
* etingof is wondering if it makes sense to have completed task still untriaged | 23:07 | |
*** murrdoc has quit IRC | 23:07 | |
etingof | may be a better filter to single out untriaged ironic bugs would be: story has ironic tasks + story does not have `ironic-triaged` tag + story does not have 'rfe*' tag | 23:12 |
etingof | though global `rfe` tag can be tricky - what if it's an rfe for one task but a bugfix for another under the same story...? | 23:14 |
* etingof needssleep, good night ;) | 23:15 | |
*** akhilaki has quit IRC | 23:19 | |
*** hoonetorg has quit IRC | 23:24 | |
*** cjloader has joined #openstack-ironic | 23:29 | |
*** cjloader has quit IRC | 23:33 | |
*** arundhati has joined #openstack-ironic | 23:34 | |
*** hoonetorg has joined #openstack-ironic | 23:36 | |
TheJulia | even if merged, it is still a paperwork tracking task in that case | 23:53 |
TheJulia | heh | 23:53 |
TheJulia | yeah, the global namescope is kind of annoying :( | 23:53 |
*** markguz_ has quit IRC | 23:54 | |
* TheJulia puts away the computers | 23:54 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!