*** crushil has quit IRC | 00:03 | |
*** bfournie has joined #openstack-ironic | 00:08 | |
*** jrist-afk is now known as jrist | 00:12 | |
*** AlexeyAbashkin has joined #openstack-ironic | 00:13 | |
*** AlexeyAbashkin has quit IRC | 00:17 | |
*** Sukhdev has joined #openstack-ironic | 00:19 | |
*** tzumainn has quit IRC | 00:24 | |
openstackgerrit | Anup Navare proposed openstack/ironic-python-agent master: [WIP] Build tinyipa with python3 https://review.openstack.org/495385 | 00:27 |
---|---|---|
openstackgerrit | Anup Navare proposed openstack/ironic-python-agent master: [Test]DNM Testing if TinyIPA builds with py3 https://review.openstack.org/509652 | 00:27 |
*** rloo has quit IRC | 00:35 | |
*** zshi has joined #openstack-ironic | 00:39 | |
*** crushil has joined #openstack-ironic | 00:40 | |
*** Goneri has quit IRC | 00:42 | |
*** hw_wutianwei has joined #openstack-ironic | 00:53 | |
*** gfhellma has quit IRC | 00:58 | |
*** ppiela has quit IRC | 01:01 | |
*** phuongnh has joined #openstack-ironic | 01:04 | |
*** mat128 has joined #openstack-ironic | 01:06 | |
*** AlexeyAbashkin has joined #openstack-ironic | 01:12 | |
*** trungnv has joined #openstack-ironic | 01:13 | |
*** AlexeyAbashkin has quit IRC | 01:16 | |
*** crushil has quit IRC | 01:27 | |
*** gcb has joined #openstack-ironic | 01:29 | |
*** yangyapeng has joined #openstack-ironic | 01:31 | |
*** yangyapeng has quit IRC | 01:35 | |
*** zhangfei has joined #openstack-ironic | 01:37 | |
*** yangyapeng has joined #openstack-ironic | 01:49 | |
*** athomas has quit IRC | 01:49 | |
*** yangyapeng has quit IRC | 01:54 | |
*** rwsu has joined #openstack-ironic | 02:06 | |
*** zhangfei has quit IRC | 02:09 | |
*** jlabarre has quit IRC | 02:12 | |
*** yangyapeng has joined #openstack-ironic | 02:15 | |
*** yangyapeng has quit IRC | 02:19 | |
*** zhangfei has joined #openstack-ironic | 02:22 | |
*** trungnv has quit IRC | 02:23 | |
*** trungnv has joined #openstack-ironic | 02:34 | |
*** yangyapeng has joined #openstack-ironic | 02:34 | |
*** yangyapeng has quit IRC | 02:38 | |
*** athomas has joined #openstack-ironic | 02:47 | |
*** athomas has quit IRC | 02:52 | |
*** Sukhdev has quit IRC | 02:53 | |
*** absubram has quit IRC | 02:58 | |
*** yangyapeng has joined #openstack-ironic | 02:59 | |
*** yangyapeng has quit IRC | 03:04 | |
*** ElCoyote_1 has joined #openstack-ironic | 03:04 | |
*** ElCoyote_ has quit IRC | 03:08 | |
*** yangyapeng has joined #openstack-ironic | 03:18 | |
*** links has joined #openstack-ironic | 03:19 | |
*** harlowja has quit IRC | 03:22 | |
*** yangyapeng has quit IRC | 03:22 | |
*** athomas has joined #openstack-ironic | 03:39 | |
*** gcb has quit IRC | 03:42 | |
*** harlowja has joined #openstack-ironic | 04:01 | |
*** vsaienk0 has joined #openstack-ironic | 04:01 | |
*** yangyapeng has joined #openstack-ironic | 04:03 | |
*** yangyapeng has quit IRC | 04:08 | |
*** vsaienk0 has quit IRC | 04:09 | |
*** yangyapeng has joined #openstack-ironic | 04:24 | |
*** Sukhdev has joined #openstack-ironic | 04:25 | |
*** Nisha_Agarwal has joined #openstack-ironic | 04:26 | |
*** TuanLA has joined #openstack-ironic | 04:27 | |
*** yangyapeng has quit IRC | 04:28 | |
*** yolanda has quit IRC | 04:44 | |
*** gcb has joined #openstack-ironic | 04:46 | |
*** yangyapeng has joined #openstack-ironic | 04:48 | |
*** hshiina has joined #openstack-ironic | 04:50 | |
*** yangyapeng has quit IRC | 04:52 | |
*** mat128 has quit IRC | 05:05 | |
*** mat128 has joined #openstack-ironic | 05:06 | |
*** jaganathan has joined #openstack-ironic | 05:14 | |
*** ijw has joined #openstack-ironic | 05:17 | |
*** dpawar has joined #openstack-ironic | 05:27 | |
*** udesale has joined #openstack-ironic | 05:32 | |
*** yangyapeng has joined #openstack-ironic | 05:37 | |
*** pcaruana has joined #openstack-ironic | 05:39 | |
*** harlowja has quit IRC | 05:52 | |
*** mat128 has quit IRC | 05:59 | |
*** hshiina2 has joined #openstack-ironic | 06:00 | |
*** hshiina has quit IRC | 06:03 | |
*** zhangfei has quit IRC | 06:05 | |
*** Sukhdev has quit IRC | 06:06 | |
*** pcaruana has quit IRC | 06:13 | |
*** sasha2 has quit IRC | 06:15 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Imported Translations from Zanata https://review.openstack.org/521721 | 06:16 |
*** zhangfei has joined #openstack-ironic | 06:18 | |
*** dyasny_ has quit IRC | 06:23 | |
*** dyasny has joined #openstack-ironic | 06:23 | |
*** yolanda has joined #openstack-ironic | 06:45 | |
*** jtomasek has joined #openstack-ironic | 06:49 | |
*** vsaienk0 has joined #openstack-ironic | 06:51 | |
*** dpawar has quit IRC | 06:53 | |
*** stendulker has joined #openstack-ironic | 06:55 | |
*** trungnv has quit IRC | 06:57 | |
*** rcernin has quit IRC | 06:59 | |
*** dtantsur|afk is now known as dtantsur | 07:01 | |
*** vsaienk0 has quit IRC | 07:01 | |
*** hshiina3 has joined #openstack-ironic | 07:04 | |
*** hshiina2 has quit IRC | 07:05 | |
*** parlos has joined #openstack-ironic | 07:07 | |
*** marios has joined #openstack-ironic | 07:10 | |
*** spectr has joined #openstack-ironic | 07:11 | |
*** spectr has quit IRC | 07:12 | |
*** trungnv has joined #openstack-ironic | 07:23 | |
*** pcaruana has joined #openstack-ironic | 07:30 | |
*** mjura has joined #openstack-ironic | 07:31 | |
*** udesale has quit IRC | 07:32 | |
dtantsur | yolanda: morning! do you think we should make a proper script out of https://teknoarticles.blogspot.de/2016/12/start-using-whole-disk-images-with.html and ship it e.g. in tripleo-common? | 07:34 |
*** ijw has quit IRC | 07:37 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for cinderclient https://review.openstack.org/476171 | 07:37 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for inspectorclient https://review.openstack.org/476172 | 07:37 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Fix swiftclient creation https://review.openstack.org/476173 | 07:37 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for neutronclient https://review.openstack.org/476170 | 07:37 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Finalize migration to keystoneauth adapters https://review.openstack.org/478825 | 07:37 |
*** trungnv has quit IRC | 07:41 | |
*** trungnv has joined #openstack-ironic | 07:52 | |
yolanda | hi dtantsur , so a generic script to convert the image and create proper volumes? | 07:53 |
dtantsur | yolanda: yep | 07:53 |
dtantsur | with UEFI support that I'm trying to figure out right now.. | 07:53 |
yolanda | dtantsur, what's needed for UEFI? i heard that isn't trivial | 07:54 |
dtantsur | yolanda: a EFI partition is needed, some grub magic too | 07:54 |
*** ralonsoh has joined #openstack-ironic | 07:56 | |
yolanda | i think it will be useful , yes. Having the ability to configure it, maybe with some yaml | 07:56 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Allow to set image_url to glance API file https://review.openstack.org/474996 | 07:57 |
dtantsur | yolanda, that's what I prepared for blinky_ghost. it runs to the end, I did not test it though: http://paste.openstack.org/show/626901/ | 08:01 |
dtantsur | yolanda: is the script at https://teknoarticles.blogspot.de/2016/12/start-using-whole-disk-images-with.html the most up-to-date version? I can look into importing it somewhere.. | 08:03 |
*** priteau has joined #openstack-ironic | 08:03 | |
*** dpawar has joined #openstack-ironic | 08:06 | |
*** karthiks has quit IRC | 08:06 | |
*** d0ugal has joined #openstack-ironic | 08:08 | |
*** gcb has quit IRC | 08:12 | |
*** gcb has joined #openstack-ironic | 08:13 | |
*** vsaienk0 has joined #openstack-ironic | 08:13 | |
*** ccamacho has joined #openstack-ironic | 08:14 | |
*** hoonetorg has quit IRC | 08:18 | |
*** gcb has quit IRC | 08:19 | |
*** gcb has joined #openstack-ironic | 08:19 | |
*** udesale has joined #openstack-ironic | 08:19 | |
*** rbartal has joined #openstack-ironic | 08:19 | |
yolanda | dtantsur, i have a version on https://github.com/redhat-nfvpe/libguestfs-image-tools | 08:19 |
*** e0ne has joined #openstack-ironic | 08:20 | |
yolanda | but quite simple, not customizable... | 08:20 |
dtantsur | cool, I'll show you what I've got in a few | 08:20 |
*** tesseract has joined #openstack-ironic | 08:23 | |
*** karthiks has joined #openstack-ironic | 08:23 | |
yolanda | ok thx | 08:25 |
*** amarao has joined #openstack-ironic | 08:25 | |
*** mrch has joined #openstack-ironic | 08:33 | |
*** karthiks has quit IRC | 08:33 | |
*** karthiks has joined #openstack-ironic | 08:33 | |
*** hoonetorg has joined #openstack-ironic | 08:35 | |
*** ijw has joined #openstack-ironic | 08:38 | |
*** karimb has joined #openstack-ironic | 08:38 | |
*** mgoddard has joined #openstack-ironic | 08:45 | |
*** ijw has quit IRC | 08:45 | |
*** zhangfei has quit IRC | 08:58 | |
*** mrch has quit IRC | 09:03 | |
*** honza has quit IRC | 09:09 | |
*** trungnv has quit IRC | 09:14 | |
*** honza has joined #openstack-ironic | 09:14 | |
*** honza is now known as Guest19085 | 09:14 | |
*** zhangfei has joined #openstack-ironic | 09:15 | |
*** etingof has quit IRC | 09:16 | |
*** vsaienk0 has quit IRC | 09:17 | |
*** dtantsur_ has joined #openstack-ironic | 09:21 | |
*** dtantsur has quit IRC | 09:22 | |
*** dtantsur_ is now known as dtantsur | 09:22 | |
*** e0ne has quit IRC | 09:23 | |
*** e0ne has joined #openstack-ironic | 09:23 | |
*** vsaienk0 has joined #openstack-ironic | 09:28 | |
*** etingof has joined #openstack-ironic | 09:30 | |
*** lucas-afk is now known as lucasagomes | 09:32 | |
*** etingof has quit IRC | 09:35 | |
*** Nisha_away has joined #openstack-ironic | 09:39 | |
*** derekh has joined #openstack-ironic | 09:41 | |
*** mrch has joined #openstack-ironic | 09:42 | |
*** Nisha_Agarwal has quit IRC | 09:42 | |
*** Nisha_away has quit IRC | 09:43 | |
*** etingof has joined #openstack-ironic | 09:48 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Adds rescue_interface to base driver class https://review.openstack.org/509335 | 09:51 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Adds RPC calls for rescue interface https://review.openstack.org/509336 | 09:51 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add rescue related methods to network interface https://review.openstack.org/509342 | 09:51 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add rescuewait timeout periodic task https://review.openstack.org/353156 | 09:51 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Agent rescue implementation https://review.openstack.org/400437 | 09:51 |
*** trungnv has joined #openstack-ironic | 09:56 | |
*** rcernin has joined #openstack-ironic | 10:01 | |
*** TuanLA has quit IRC | 10:05 | |
*** jesusaur has quit IRC | 10:07 | |
openstackgerrit | Derek Higgins proposed openstack/ironic-python-agent master: Ignore IPv6 link local addresses https://review.openstack.org/520582 | 10:11 |
dtantsur | I suspect I was disconnected when I asked: | 10:11 |
dtantsur | yolanda: what are units here https://github.com/redhat-nfvpe/libguestfs-image-tools/blob/master/add-volumes-to-image.py#L43 | 10:12 |
*** blinky_ghost has joined #openstack-ironic | 10:12 | |
blinky_ghost | dtantsur: good morning, testing your script | 10:13 |
yolanda | dtantsur, MB | 10:13 |
dtantsur | blinky_ghost: great, lemme know how it goes | 10:14 |
dtantsur | also, for everyone's information (esp. TheJulia): my internet is unreliable today, I may disappear out of the sudden | 10:14 |
*** jesusaur has joined #openstack-ironic | 10:20 | |
pas-ha | strange failures in multinode grenade - n-cpu fails to start on primary node in the 'new' grenade phase http://logs.openstack.org/71/476171/26/check/ironic-grenade-dsvm-multinode-multitenant/6fe1262/logs/screen-n-cpu.txt.gz#_Nov_21_09_10_54_515324 | 10:27 |
*** jhesketh has quit IRC | 10:28 | |
pas-ha | seems like upgraded version of oslo_db/sqlalchemy are incompatible with 'old' n-cpu | 10:29 |
*** jhesketh has joined #openstack-ironic | 10:30 | |
*** ralonsoh has quit IRC | 10:34 | |
*** ijw has joined #openstack-ironic | 10:43 | |
*** milan_ has joined #openstack-ironic | 10:44 | |
pas-ha | yep, seems like fresh oslo.db was added to u-c 18 hours ago https://github.com/openstack/requirements/commit/0c44a9622c204f21ad91b1b6361c268b3ef834b8 | 10:46 |
*** ijw has quit IRC | 10:47 | |
openstackgerrit | Mark Goddard proposed openstack/ironic master: Don't collect logs from powered off nodes https://review.openstack.org/521124 | 10:48 |
pas-ha | and here is the commit that seems to have broken us https://review.openstack.org/#/c/499934 | 10:50 |
patchbot | patch 499934 - oslo.db - Remove kwarg retry_on_request in wrap_db_retry (MERGED) | 10:50 |
pas-ha | why do we upgrade dependencies when not upgrading services in the first place? | 10:50 |
*** zhangfei has quit IRC | 10:57 | |
milan_ | morning Ironic! :) | 11:04 |
milan_ | #pixiesay -mflexing broken CI? | 11:05 |
PixieBoots | ᕙʕ⇀ᴥ⇀ʔᕗ: broken CI? | 11:05 |
milan_ | morning pas-ha! :) | 11:05 |
openstackgerrit | Ankit Kumar proposed openstack/ironic-inspector master: Add py35 gate for ironic-inspector https://review.openstack.org/515036 | 11:07 |
openstackgerrit | Nguyen Hung Phuong proposed openstack/ironic master: Add additional capabilities discovery for iRMC driver https://review.openstack.org/417347 | 11:07 |
gmonteiro | Good morning pas-ha, milan_ and all :) | 11:09 |
milan_ | morning gmonteiro! :) | 11:12 |
*** stendulker_ has joined #openstack-ironic | 11:17 | |
*** stendulker has quit IRC | 11:21 | |
*** stendulker_ has quit IRC | 11:22 | |
*** AlexeyAbashkin has joined #openstack-ironic | 11:24 | |
*** udesale has quit IRC | 11:30 | |
*** jkilpatr_ has quit IRC | 11:39 | |
*** mrch has quit IRC | 11:41 | |
*** ijw has joined #openstack-ironic | 11:44 | |
*** ijw has quit IRC | 11:49 | |
openstackgerrit | Mark Goddard proposed openstack/ironic master: Refactor VIFIDPortMixin: vif_attach and vif_detach https://review.openstack.org/486126 | 11:52 |
*** jkilpatr has joined #openstack-ironic | 12:04 | |
pas-ha | morning milan_ gmonteiro | 12:14 |
pas-ha | so, re mutlinode grenade failures - it seems the culprit is n-g-s, which restarts n-cpu, which in turn picks up new dependencies and in this case breaks due to newer oslo.db | 12:15 |
*** gcb has quit IRC | 12:18 | |
*** phuongnh has quit IRC | 12:25 | |
*** Guest19085 is now known as honza | 12:27 | |
*** fellypefca has joined #openstack-ironic | 12:31 | |
fellypefca | Good morning everyone :) | 12:32 |
gmonteiro | Morning fellypefca o/ | 12:32 |
*** vsaienk0 has quit IRC | 12:34 | |
milan_ | morning fellypefca! :) | 12:38 |
* milan_ just reads pas-ha 's analysis | 12:38 | |
*** vsaienk0 has joined #openstack-ironic | 12:39 | |
milan_ | pas-ha, so older n-cpu is picking newer oslo.db... | 12:41 |
milan_ | so didn't nova miss a deprecation period? | 12:41 |
milan_ | or maybe it's just the nova's devstack plug-in that sets a deprecated (and now removed) config opt | 12:42 |
milan_ | oh I suppose to ask vdrok, hey vdrok ;) what do I do when I do bug triage? | 12:43 |
milan_ | etingof may be interested to know too :P | 12:43 |
*** yangyapeng has quit IRC | 12:44 | |
* milan_ -> coffee actually so gonna be a scroll back read :P | 12:44 | |
*** yangyapeng has joined #openstack-ironic | 12:45 | |
*** ijw has joined #openstack-ironic | 12:45 | |
*** hw_wutianwei has quit IRC | 12:47 | |
*** lucasagomes is now known as lucas-hungry | 12:49 | |
*** yangyapeng has quit IRC | 12:49 | |
*** ijw has quit IRC | 12:50 | |
*** MattMan has quit IRC | 12:57 | |
*** MattMan has joined #openstack-ironic | 12:57 | |
*** zshi has quit IRC | 12:58 | |
*** zshi has joined #openstack-ironic | 12:59 | |
*** jlabarre has joined #openstack-ironic | 13:01 | |
*** efried has quit IRC | 13:04 | |
* etingof *is* interested | 13:06 | |
*** arnewiebalck has quit IRC | 13:08 | |
*** arnewiebalck has joined #openstack-ironic | 13:13 | |
*** dprince has joined #openstack-ironic | 13:15 | |
*** yangyapeng has joined #openstack-ironic | 13:16 | |
*** efried has joined #openstack-ironic | 13:17 | |
pas-ha | milan_: etingof I've updated ironic whiteboard etherpad with what I've dug thru | 13:24 |
milan_ | pas-ha, /me checks | 13:25 |
pas-ha | btw, vdrok might be travelling right now, and will be in PST timezone for next couple of months, so don't expect him here in this time of the day :) | 13:26 |
milan_ | oooh didn't know | 13:27 |
milan_ | thx pas-ha | 13:27 |
milan_ | safe travels vdrok! :) | 13:27 |
milan_ | https://review.openstack.org/#/c/521824/1 | 13:27 |
patchbot | patch 521824 - networking-generic-switch - WIP: add multitenant grenade job | 13:27 |
milan_ | so this isn't supposed to fix the issue rather just try that not restarting n-cpu gets as around the bug | 13:28 |
milan_ | s/bug/feature | 13:28 |
* milan_ worried a bit that it does two things actually: commenting out the restart of n-cpu and adding the multinode job line; but for a test I guess it's OK | 13:29 | |
milan_ | vsaienk0, hey, or is the multinode line required to work-around the issue? | 13:29 |
*** rcernin has quit IRC | 13:30 | |
*** vsaienk0 has quit IRC | 13:38 | |
*** vsaienk0 has joined #openstack-ironic | 13:40 | |
*** ijw has joined #openstack-ironic | 13:40 | |
*** mgoddard has quit IRC | 13:40 | |
pas-ha | milan_: effectively restarting n-cpu w/o upgrading it when you have upgraded other services on this host *is* a bug in a grenade context | 13:43 |
pas-ha | milan_: that's the beauty of having zuul jobs in-tree - we could test stuff as we write it, in a single commit | 13:44 |
milan_ | pas-ha, makes sense; I'm just wondering, shouldn't the removing of the restart be a separate patch that addresses the bug? | 13:44 |
*** trown|outtypewww is now known as trown | 13:44 | |
pas-ha | besides, as it turns out, ironic's multinode grenade is dependent on n-g-s, so it should be voting on n-g-s as well (as a usual multinode job already does) | 13:45 |
*** udesale has joined #openstack-ironic | 13:45 | |
*** mgoddard has joined #openstack-ironic | 13:45 | |
pas-ha | yet another point on pulling n-g-s under ironic umbrella | 13:45 |
*** ijw has quit IRC | 13:45 | |
pas-ha | milan_: we could split it when we are sure it does fix stuff | 13:46 |
*** jaypipes has joined #openstack-ironic | 13:46 | |
milan_ | pas-ha, cool, or at least report it.. | 13:48 |
milan_ | pas-ha, good job on figuring out btw, thanks! | 13:48 |
makowals | Hi ironic | 13:50 |
*** lucas-hungry is now known as lucasagomes | 13:51 | |
makowals | Question about get_disk_label() method from deploy_utils.py -- is there any particular reason why this function is parsing instance_info.capabilities instead of node.capabilities? | 13:51 |
makowals | I would expect this to be more associated with the physical node itself than with the instance | 13:51 |
makowals | Or is there a bigger picture I'm missing? | 13:52 |
milan_ | makowals, hi, could it be that there is no more node object at the time of get_disk_label call, rather the instance "in progress"? | 13:55 |
milan_ | * in progress of getting assembled or how to call it | 13:55 |
milan_ | https://review.openstack.org/#/c/466448/ dtantsur hi +2 time? :P | 13:56 |
patchbot | patch 466448 - ironic-inspector - Introducing a dnsmasq PXE filter driver | 13:56 |
makowals | milan_: get_disk_label is called more or less at the same time as for example get_boot_mode_for_deploy, whereas the latter uses node.capabilites | 13:56 |
milan_ | makowals, interesting | 13:56 |
* milan_ git-magic | 13:56 | |
makowals | In my current setup I'm unable to make nova fill instance_info.capabilities['boot_mode'] so in the end no easy way to set this value manually | 13:56 |
makowals | Also again, at this level of my knowledge I don't really believe it really belongs to the instance info | 13:57 |
milan_ | makowals, and you tracked your issue down to get_disk_label being called on the wrong object type? | 13:57 |
makowals | milan_: Exactly this, by tracking the calls I got down to the moment of parse_instance_info_capabilities(node).get('disk_label') instead of driver_utils.get_node_capability(node, 'disk_label') | 13:58 |
milan_ | makowals, hmmm may be worth a bug | 14:00 |
* milan_ reads the deploy_utils.get_disk_label | 14:00 | |
*** jkilpatr_ has joined #openstack-ironic | 14:00 | |
*** sasha2 has joined #openstack-ironic | 14:00 | |
milan_ | https://github.com/openstack/ironic/blob/0114f71fe5978d2c0994dfb1bbd9dd1426a66031/ironic/drivers/modules/deploy_utils.py#L748 seems like a clue | 14:00 |
*** cdearborn has joined #openstack-ironic | 14:01 | |
milan_ | but why then the nested method is called parse_instance_info_capabilities(node)?? | 14:02 |
*** jkilpatr has quit IRC | 14:03 | |
*** ElCoyote_1 has quit IRC | 14:04 | |
milan_ | makowals, I suspect it works for you once you replace the parse_instance_info_capabilities with get_node_capability | 14:07 |
*** jaganathan has quit IRC | 14:12 | |
milan_ | makowals, but reading thru parse_instance_capabilities it actually pulls the capabilities dict out from node.instance_info; https://github.com/openstack/ironic/blob/0114f71fe5978d2c0994dfb1bbd9dd1426a66031/ironic/drivers/modules/deploy_utils.py#L580 | 14:12 |
milan_ | makowals, and the issue you're having is even though you set the disklabel thru nova you don't see it returned by https://github.com/openstack/ironic/blob/0114f71fe5978d2c0994dfb1bbd9dd1426a66031/ironic/drivers/modules/deploy_utils.py#L745 | 14:13 |
milan_ | * deploy_utils.get_disk_label() | 14:13 |
*** ricardoas has joined #openstack-ironic | 14:15 | |
*** Goneri has joined #openstack-ironic | 14:17 | |
* milan_ 's gut feeling is if it's a value supposed to be always passed thru nova api, the parse_instance_info_capabilities is the proper tool to use... the naming of get_node_disk_label is misleading though | 14:18 | |
*** mat128 has joined #openstack-ironic | 14:20 | |
makowals | milan_: Well, my very first question was is this value should be always passed through nova api, because at this moment it's not at all | 14:23 |
makowals | That's why the obvious observation was let's use node.capability instead of instance's | 14:23 |
milan_ | makowals, maybe the ironic-nova-driver isn't doing it's job, dunno | 14:23 |
milan_ | makowals, so how about you report a bug that you're unable to propagate the disklabel thru to the deploy interface and that it seems the info is lost somewhere in the pipeline | 14:25 |
*** ElCoyote_ has joined #openstack-ironic | 14:25 | |
makowals | This makes sense, but I'd also see a valid question "why to rely on node.instance_info.capabilities in this case instead of node.capabilites" | 14:25 |
milan_ | makowals, btw did you try replace the parse_instance_info_capabilities with the get_node_capabilities inside the get_node_disk_label? did it help? | 14:26 |
*** rloo has joined #openstack-ironic | 14:26 | |
makowals | disk_label doesn't seem to me like something what is instance dependent, so I'd personally keep nova out of the loop | 14:26 |
milan_ | makowals, can well be the case, to me at least the naming is misleading even if it turns out to be the proper tool to call | 14:27 |
milan_ | *use | 14:27 |
makowals | milan_: That's exactly what I did, use get_node_capability instead of parse_instance_info_... and it seems to behave better, I'm still testing it | 14:27 |
makowals | So I will finish my tests and make all the paperwork then | 14:27 |
milan_ | makowals, great! that would be a valuable info for the bug actually | 14:27 |
milan_ | makowals ++ | 14:28 |
rloo | hi and good morning milan_, makowals, pas-ha | 14:32 |
rloo | pas-ha: thx for working on the gate issue :-( So there was nothing wrong with making the oslo.db change. Why do we restart old n-cpu with new packages though? That seems to be an incorrect testing issue. | 14:33 |
milan_ | morning rloo! :) | 14:37 |
*** hshiina3 has quit IRC | 14:37 | |
*** ijw has joined #openstack-ironic | 14:41 | |
*** vsaienk0 has quit IRC | 14:46 | |
*** ijw has quit IRC | 14:46 | |
*** vsaienk0 has joined #openstack-ironic | 14:47 | |
*** links has quit IRC | 14:47 | |
*** rloo has quit IRC | 14:49 | |
*** rloo has joined #openstack-ironic | 14:50 | |
*** hw_wutianwei has joined #openstack-ironic | 14:52 | |
*** ohamada has joined #openstack-ironic | 14:54 | |
milan_ | rloo ++ that's my take on it as well after pas-ha explained to me what's going on | 14:55 |
*** weshay is now known as weshay_pto | 14:55 | |
dtantsur | morning rloo and everyone | 14:55 |
* dtantsur hates his internet | 14:55 | |
rloo | milan_: thx | 14:55 |
rloo | dtantsur: hiya. sorry about the internet issue, but at least you have some furniture now? | 14:56 |
*** ohamada has quit IRC | 14:56 | |
milan_ | dtantsur, hey, welcome back from the ping void | 14:56 |
dtantsur | rloo: well, yeah. A LOT to assemble :( | 14:56 |
*** ohamada has joined #openstack-ironic | 14:56 | |
*** ohamada has quit IRC | 14:56 | |
rloo | dtantsur: heh, you need an assemble-party! | 14:56 |
*** ijw has joined #openstack-ironic | 14:57 | |
dtantsur | a need a lot of whiskey | 14:57 |
* dtantsur has a lot of whiskey, fortunately | 14:57 | |
milan_ | dtantsur, if you don't have an akkuschrauber yet definitely have one | 14:57 |
dtantsur | not yet | 14:57 |
milan_ | helps a ton | 14:57 |
*** crushil has joined #openstack-ironic | 15:01 | |
pas-ha | rloo: well, the comment in the code that was restarting n-cpu sez that while neutron is being restarted (and we do need it anyway to pick installed/upgraded n-g-s), n-cpu might fail, so just to be sure we restart it as well. | 15:04 |
pas-ha | well might be not an issue any more, both multinode and multinode grenade jobs passed... | 15:05 |
rloo | pas-ha: seems like an issue with neutron & nova then, wrt upgrading? | 15:05 |
pas-ha | so yes, this was a bug in the upgrade procedure | 15:05 |
rloo | pas-ha: i mean, our hack was to get around ^^ | 15:05 |
*** hw_wutianwei has quit IRC | 15:05 | |
blinky_ghost | derekh: Hi | 15:05 |
pas-ha | I don't even think it was about upgrades in the first place, just usual single node devstack | 15:06 |
rloo | pas-ha: hopefully, they are all playing nicely now ... | 15:06 |
rloo | pas-ha: oh. :-( | 15:06 |
*** hw_wutianwei has joined #openstack-ironic | 15:06 | |
pas-ha | it seems ok now | 15:06 |
dtantsur | blinky_ghost: we're in a meeting, so please expect delays | 15:06 |
rloo | pas-ha: 'for now'... | 15:06 |
pas-ha | but let's wait for more stats of job runs | 15:06 |
*** ElCoyote_ has quit IRC | 15:06 | |
blinky_ghost | ok | 15:06 |
*** ElCoyote_ has joined #openstack-ironic | 15:06 | |
rloo | pas-ha: let's just land it and go from there, as long as we can unblock ironic's gate :) | 15:07 |
* rloo starting to appreciate the value of zuul v3... | 15:07 | |
*** udesale has quit IRC | 15:07 | |
pas-ha | yep, that's the plan | 15:08 |
*** ijw has quit IRC | 15:09 | |
*** ohamada has joined #openstack-ironic | 15:09 | |
rpioso | Good morning, ironicers! | 15:13 |
TheJulia | Good morning everyone | 15:14 |
rpioso | Morning, TheJulia :-) | 15:14 |
*** ohamada has quit IRC | 15:16 | |
rloo | morning rpioso, TheJulia | 15:17 |
*** rbartal has quit IRC | 15:19 | |
milan_ | morning rpioso, TheJulia! :) | 15:21 |
dtantsur | morning TheJulia | 15:22 |
* TheJulia goes and makes coffee | 15:27 | |
*** blinky_ghost has quit IRC | 15:28 | |
rloo | pas-ha: this RFE your proposed, what's the status, did you decide on a possible solution? do we need to discuss? https://bugs.launchpad.net/ironic/+bug/1719582 | 15:32 |
openstack | Launchpad bug 1719582 in Ironic "[RFE] complete SSL support for https image references" [Wishlist,Confirmed] - Assigned to Pavlo Shchelokovskyy (pshchelo) | 15:32 |
pas-ha | rloo: dunno, I would just re-use [glance] section for those options, spun a ksa Session from there and without auth, and use that to fetch images... oh, and document it of course :) | 15:35 |
rloo | pas-ha: oh, doesn't seem like we'd need a spec for that. if you update the rfe with that, we can discuss, hopefully approve | 15:36 |
rloo | pas-ha: i put it down to discuss in the next (2 weeks) meeting | 15:36 |
rpioso | Morning, milan_ and rloo | 15:37 |
milan_ | hey rpioso! :) | 15:37 |
*** blinky_ghost has joined #openstack-ironic | 15:40 | |
rloo | dtantsur: are you good with approving this rfe, i see milan_ has already got a fix released... https://bugs.launchpad.net/ironic/+bug/1554243 | 15:42 |
openstack | Launchpad bug 1554243 in Ironic "[RFE] In-band introspection of diskless nodes" [Wishlist,Confirmed] | 15:42 |
dtantsur | rloo: I'm good | 15:42 |
rloo | dtantsur: thx | 15:42 |
NobodyCam | Good Morning Ironic'ers | 15:43 |
dtantsur | morning NobodyCam | 15:43 |
NobodyCam | :) feels like Thursday :p | 15:43 |
milan_ | NobodyCam, morning! :) lol yeah | 15:45 |
NobodyCam | Morning milan_ :) | 15:45 |
rloo | folks, if anyone is interested in reviewing this spec, it has 2 +2: https://review.openstack.org/#/c/515717/ | 15:46 |
patchbot | patch 515717 - ironic-specs - Support a new hardware type for PRIMEQUEST MMB | 15:46 |
rloo | otherwise, I'll probably approve later today | 15:46 |
dtantsur | rloo: heh, already approved :) | 15:47 |
rloo | dtantsur: even better! | 15:47 |
*** dpawar has quit IRC | 15:55 | |
openstackgerrit | Merged openstack/ironic-specs master: Support a new hardware type for PRIMEQUEST MMB https://review.openstack.org/515717 | 15:55 |
milan_ | rloo, funny that w/r https://bugs.launchpad.net/ironic/+bug/1554243 | 15:59 |
openstack | Launchpad bug 1554243 in Ironic "[RFE] In-band introspection of diskless nodes" [Wishlist,Confirmed] | 15:59 |
milan_ | I got pinged downstream by docs team today ;) | 15:59 |
milan_ | magic | 15:59 |
rloo | milan_: i knew :D | 15:59 |
*** jistr is now known as jistr|mtg | 15:59 | |
milan_ | lol seriously? :D | 15:59 |
rloo | milan_: nope, | 16:00 |
rloo | milan_: is your leg longer now? ('pulling your leg') | 16:00 |
milan_ | ... and I almost started to believe in magic | 16:00 |
milan_ | yeah it itched a bit :D | 16:01 |
rloo | milan_: magic :) | 16:01 |
milan_ | :D | 16:01 |
milan_ | rloo I bet you know that I have to have another cup of ... | 16:02 |
openstackgerrit | Mark Goddard proposed openstack/ironic master: Don't collect logs from powered off nodes https://review.openstack.org/521124 | 16:02 |
rloo | milan_: :) | 16:02 |
*** yangyapeng has quit IRC | 16:02 | |
* milan_ -> coffee | 16:02 | |
*** yangyapeng has joined #openstack-ironic | 16:03 | |
*** links has joined #openstack-ironic | 16:03 | |
milan_ | .... strange my leg... it's odd... a bit numb.... stumbling now that I walk actually... isn't it a bit longer??? ... dunno .... | 16:04 |
*** yangyapeng has quit IRC | 16:08 | |
vdrok | good morning all! pas-ha actually I arrived last wed :) | 16:10 |
vdrok | morning milan_ rloo dtantsur ! | 16:10 |
rloo | good morning vdrok! | 16:10 |
*** mgoddard has quit IRC | 16:10 | |
milan_ | hey vdrok! :) | 16:11 |
milan_ | how's the traveling? | 16:11 |
milan_ | traveling without moving? | 16:12 |
NobodyCam | morning vdrok :) | 16:12 |
milan_ | it's magic here ;) | 16:12 |
vdrok | good morning NobodyCam ! :) | 16:12 |
NobodyCam | :) | 16:12 |
milan_ | Thursday | 16:12 |
vdrok | milan_: nah, travels were last week. it was bearable :) | 16:12 |
*** links has quit IRC | 16:12 | |
milan_ | vdrok, good to hear, so where are you actually? :) | 16:13 |
vdrok | milan_: sunnyvale | 16:13 |
vdrok | yeah, no moving | 16:13 |
vdrok | will be here until ptg i suppose | 16:13 |
milan_ | vdrok, nice! :D | 16:14 |
*** jistr|mtg is now known as jistr | 16:16 | |
milan_ | rloo or vdrok so what does one do actually when doing the bug triage? | 16:16 |
milan_ | is there some guide? | 16:16 |
rloo | milan_: that's a good question. dtantsur may know, he did this for many years.... ^^ | 16:19 |
milan_ | dtantsur, enlighten please, with a light beam of knowledge or so :D | 16:20 |
rloo | milan_: a search brought up https://wiki.openstack.org/wiki/BugTriage | 16:20 |
rloo | milan_: but i haven't read it | 16:20 |
* milan_ reads ty rloo :) | 16:20 | |
rloo | milan_: i just skimmed that, there are a lot of tasks. have fun! | 16:21 |
vdrok | milan_: what I'm doing is just going through the list of bugs on dtantsur's triage page | 16:21 |
milan_ | heh | 16:21 |
vdrok | it's referenced on the whiteboard | 16:21 |
milan_ | OK | 16:21 |
milan_ | all the rays trace back to dtantsur :P | 16:22 |
vdrok | the ones that require triage are yours :D | 16:22 |
milan_ | dtantsur deserves a wiki page :D | 16:22 |
vdrok | and then you just comment what you think about that bug, and set triaged if it's obvious how to fix or the way to fix is already proposed | 16:22 |
vdrok | or set invalid :) | 16:23 |
milan_ | cool let's try some of that! | 16:25 |
* milan_ cargoculting etherpad bullets | 16:27 | |
*** AlexeyAbashkin has quit IRC | 16:28 | |
TheJulia | dtantsur: I updated the the braindump etherpad with my thought, in green. Hopefully your enjoying assembling all the things :) | 16:29 |
*** mjura has quit IRC | 16:30 | |
milan_ | vdrok, RFE bugs are no different here from the "regular" ones, correct? | 16:31 |
* milan_ looks at https://bugs.launchpad.net/ironic/+bug/1652957 and it was tagged by rloo as needs spec so this one should have a triaged tag too, nope? | 16:32 | |
openstack | Launchpad bug 1652957 in Ironic "[RFE] Tenant-aware baremetal nodes" [Wishlist,New] - Assigned to Vladyslav Drok (vdrok) | 16:32 |
vdrok | milan_: yeah, tho you set to confirmed if you agree. it should become triaged only when spec is merged. tho everyone references the bug in spec commit message, so it becomes in progress as soon as spec is uploaded :( | 16:32 |
* rloo is bad at bug triaging; forgets to set all the various fields like status | 16:33 | |
vdrok | confirmed -- you agree with rfe, triaged -- spec is merged, the as usual, in progress, fix released | 16:33 |
vdrok | s/the/then | 16:33 |
TheJulia | rloo: don't feel bad, I'm kind of bad too :) | 16:34 |
vdrok | good morning TheJulia ! | 16:34 |
TheJulia | good morning vdrok | 16:34 |
rloo | TheJulia: if we have to be bad at something... :) | 16:34 |
milan_ | cool, my first bug triaged \0/ :D | 16:35 |
TheJulia | milan_: \o/ | 16:35 |
milan_ | Oops, actually, where do I put it on the etherpad? | 16:35 |
milan_ | In progress? | 16:35 |
milan_ | s/inprogress/incomplete | 16:35 |
rloo | milan_: be careful about incomplete -- it means they have to get back with some information and that after they do that, we need to actually read it and change the status. otherwise, some bot thing will get rid of those after x (60?) days of 'inactivity' | 16:37 |
*** ricardoas has quit IRC | 16:37 | |
milan_ | ah, so only put incomplete status bugs to the incomplete etherpad paragraph | 16:38 |
milan_ | ok | 16:38 |
rloo | milan_: dunno cuz i'm not paying attention to what you're doing. Just mentioning that :) | 16:39 |
milan_ | rloo, thanks anyways :D | 16:40 |
*** ccamacho has quit IRC | 16:42 | |
*** fragatina has quit IRC | 16:43 | |
*** fragatina has joined #openstack-ironic | 16:44 | |
*** fragatina has quit IRC | 16:46 | |
openstackgerrit | Milan Kováčik proposed openstack/ironic-specs master: Cisco Nexus POAP Driver https://review.openstack.org/416588 | 16:46 |
*** fragatina has joined #openstack-ironic | 16:46 | |
*** tzumainn has joined #openstack-ironic | 16:46 | |
TheJulia | FWIW, as a side note, a proposed goal for rocky is for teams to migrate to storyboard. If approved, it behoves us to go ahead and have our bugs pruned down as much as possible before we request that migration to be performed for us. | 16:47 |
rloo | vdrok: if you have a chance, we need this to unblock our gate: https://review.openstack.org/#/c/521824/2 | 16:50 |
patchbot | patch 521824 - networking-generic-switch - Do not restart n-cpu in devstack plugin | 16:50 |
rloo | TheJulia: so there will be some automated migration? | 16:50 |
*** fragatina has quit IRC | 16:50 | |
dtantsur | milan_: sorry, internet disappeared again | 16:51 |
milan_ | dtantsur, I knew... magic ;) | 16:51 |
dtantsur | see previous examples at https://etherpad.openstack.org/p/ironic-bug-triage | 16:51 |
*** fragatina has joined #openstack-ironic | 16:51 | |
dtantsur | TheJulia: wow, it's finally ready | 16:51 |
*** marios has quit IRC | 16:51 | |
* milan_ wonders what to do about bugs like https://bugs.launchpad.net/ironic/+bug/1630330 / https://review.openstack.org/382012 | 16:52 | |
openstack | Launchpad bug 1630330 in Ironic "[RFE] Make Resource Actions Discoverable" [Wishlist,New] | 16:52 |
patchbot | patch 382012 - ironic-specs - API Evolution - Discoverable Resource Actions | 16:52 |
milan_ | set to incomplete? | 16:52 |
TheJulia | dtantsur: the thing I read indicated missing one major feature, which is in progress now, and it seemed like something we don't do or use in ironic. I just don't remember what it is | 16:52 |
dtantsur | milan_: there is a spec for this one | 16:52 |
rloo | TheJulia: migrate from using the launchpad bug system to storyboard? or just rfes? (shows how much I know...) | 16:52 |
milan_ | after 2 years of inactivity? | 16:52 |
milan_ | sry 1 year | 16:52 |
dtantsur | mmmmmm.. yeah. dunno | 16:52 |
dtantsur | it's not completely invalid, but we're unlikely to ever get to it | 16:53 |
TheJulia | rloo: yes, although they ask for manual testing/issue correction by teams prior to teams requesting infra execute the migration. | 16:53 |
milan_ | dtantsur, exactlu | 16:53 |
milan_ | I'll do this: | 16:53 |
rloo | milan_: it isn't incomplete. just no activity? | 16:53 |
milan_ | a) add the bug link to the spec | 16:53 |
TheJulia | rloo: complete move away from launchpad | 16:53 |
milan_ | b) set incomplete? | 16:53 |
milan_ | rloo, ENOSUCHSTATE | 16:53 |
TheJulia | milan_: I would say comment and set incomplete | 16:54 |
rloo | TheJulia: interesting. | 16:54 |
milan_ | last update on the spec 1 year ago | 16:54 |
dtantsur | folks, I'll switch to IKEA stuff. the internet (semi)outage is supposed to end today late evening | 16:54 |
TheJulia | if the person comments, and really needs it, then it can be changed back | 16:54 |
TheJulia | dtantsur: have fun! | 16:54 |
rloo | huh, the RFE is incomplete? | 16:54 |
dtantsur | will start early tomorrow anyway because of the 2nd delivery | 16:54 |
milan_ | TheJulia, exactly what I think | 16:54 |
dtantsur | TheJulia: thanks :) I already hate IKEA, myself and the whole world, and I've only started :) | 16:54 |
TheJulia | milan_: it has been what we've been doing with some of the much older bugs/issues/wants | 16:54 |
milan_ | dtantsur, have a beer, so it's beerable :P | 16:55 |
*** dtantsur is now known as dtantsur|afk | 16:55 | |
* TheJulia slides dtantsur japanese whisky | 16:55 | |
dtantsur|afk | yeah, I have a bottle of something local and craft | 16:55 |
rloo | milan_: i disagree, why is that RFE incomplete? There's a spec proposed. | 16:55 |
dtantsur|afk | mmm, japanese wisky :) | 16:55 |
openstackgerrit | Merged openstack/ironic-python-agent master: Ignore IPv6 link local addresses https://review.openstack.org/520582 | 16:55 |
dtantsur|afk | c'mon, I have to assemble stuff still :) | 16:55 |
*** fragatina has quit IRC | 16:55 | |
milan_ | rloo, cause it's been inactive for 1 year the spec? | 16:55 |
TheJulia | lol | 16:55 |
rloo | milan_: i think the RFE is still valid. | 16:55 |
milan_ | rloo, OK so I'll just link the commit message with the bug? | 16:56 |
milan_ | and unassign? | 16:56 |
TheJulia | so, at some point, we need to reach consensus on dropping things that we may think are valid, but there is no viable ability, time, or resources to make happen, which would include abandoning neglected specs | 16:56 |
rloo | milan_: are we looking at the same thing? https://bugs.launchpad.net/ironic/+bug/1630330 | 16:56 |
openstack | Launchpad bug 1630330 in Ironic "[RFE] Make Resource Actions Discoverable" [Wishlist,New] | 16:56 |
TheJulia | if we don't do that clean-up, then we're just causing ourselves to have a bunch of noise to sift through | 16:56 |
*** vsaienk0 has quit IRC | 16:56 | |
milan_ | rloo, yup | 16:56 |
* dtantsur|afk agrees with TheJulia and clean up approach | 16:56 | |
* dtantsur|afk leaves for real | 16:57 | |
* TheJulia feels bad typing it... but yeah :( | 16:57 | |
rloo | i feel like it is a disservice to all the people that spent time trying to help ironic | 16:57 |
rloo | we have specs/features that take over a year to get done | 16:57 |
TheJulia | I agree, which is why I think the last time we did any spec clean-up, we gave 3-6 months for comments? | 16:57 |
rloo | and yes, we don't have enough people. but i don't think it means, delete everything beyond 1 year. | 16:58 |
rloo | does storyboard provide more fine-grain control? | 16:58 |
TheJulia | rloo: absolutely not, but we should reach consensus | 16:58 |
*** openstackstatus has quit IRC | 16:58 | |
TheJulia | rloo: dunno | 16:58 |
rloo | cuz a bug tracker is no way to manage features, etc. | 16:58 |
rloo | TheJulia: and yes, consensus is good. but we haven't reached that yet, so i disagree with making that rfe incomplete. | 16:59 |
milan_ | rloo, so if I set incomplete and the folks disagree, they'd update the spec, fixing the suggestions as requested by the reviewers | 16:59 |
*** vsaienk0 has joined #openstack-ironic | 16:59 | |
*** openstackstatus has joined #openstack-ironic | 16:59 | |
*** ChanServ sets mode: +v openstackstatus | 16:59 | |
rloo | milan_: nope | 16:59 |
milan_ | if they don't care, it's expiring, I guess it's fair | 16:59 |
rloo | milan_: i disagree with you there. | 16:59 |
milan_ | rloo, hmmm OK | 17:00 |
rloo | milan_: If the bug description is incomplete or the report is lacking the information necessary to reproduce the issue, you should ask the reporter to provide missing information, and set the bug status to Incomplete | 17:01 |
rloo | milan_: from https://wiki.openstack.org/wiki/BugTriage | 17:01 |
rloo | milan_: i just set it to 'confirmed' | 17:01 |
*** mgoddard has joined #openstack-ironic | 17:01 | |
milan_ | rloo, OK cool, I'll add the bug link to the spec commit message I guess | 17:02 |
milan_ | that OK? | 17:02 |
milan_ | ah it's there | 17:02 |
rloo | milan_: yup. | 17:02 |
milan_ | cool :) moving on | 17:02 |
milan_ | rloo, vdrok https://bugs.launchpad.net/ironic/+bug/1660351 would go to incomplete? no feedback provided yet? | 17:07 |
openstack | Launchpad bug 1660351 in Ironic "[RFE] serial console through shellinabox is not multi-tenant and has no token/password protection" [Wishlist,New] | 17:07 |
rloo | milan_: i put that down to discuss in next ironic meeting | 17:08 |
rloo | milan_: we need to decide whether it needs a spec or whatever | 17:08 |
milan_ | cool | 17:09 |
rloo | milan_: basically, i think the onus is on us to get back to them. so no, not incomplete. | 17:09 |
*** yangyapeng has joined #openstack-ironic | 17:09 | |
*** tesseract has quit IRC | 17:09 | |
milan_ | rloo, but that means it's actually confirmed then, correct? | 17:09 |
milan_ | since we didn't disapprove of it | 17:10 |
rloo | milan_: maybe, like i said, i'm bad at setting those things | 17:10 |
rloo | milan_: so yeah | 17:10 |
milan_ | ack | 17:10 |
TheJulia | rloo: agreed, things with outstanding specs proposed and no traction do need a little more thought and review. I think we should generate a list and at some point spend time to guage interest or whatever. Kind of like what we did to rank priorities. | 17:10 |
TheJulia | But that requires time :( | 17:10 |
rloo | TheJulia: yup, requires time. am wondering what storyboard buys us and whether it makes more sense to move them over to storyboard if there is finer control there. | 17:11 |
openstackgerrit | Derek Higgins proposed openstack/ironic-python-agent stable/pike: Ignore IPv6 link local addresses https://review.openstack.org/521949 | 17:11 |
rloo | TheJulia: also, ^^ procrastination | 17:11 |
*** yangyapeng has quit IRC | 17:14 | |
milan_ | TheJulia, if this one is still valid, it's some sort of a requires spec, no? https://bugs.launchpad.net/ironic/+bug/1709782 | 17:15 |
openstack | Launchpad bug 1709782 in Ironic "manual cleaning of reset_ilo_credentail() fails" [Wishlist,New] | 17:15 |
*** lucasagomes is now known as lucas-afk | 17:15 | |
milan_ | or opinion | 17:15 |
TheJulia | rloo: It is a great question, but I am not sure there would be much of a choice if it becomes a project wide goal. of course, the instructions detail how to spin up a test instance of storyboard and play with the migration :) | 17:16 |
* TheJulia looks | 17:16 | |
milan_ | procrastination indeed :D | 17:16 |
TheJulia | milan_: oh jeeze, I have to remember how that all works | 17:17 |
milan_ | Oops | 17:18 |
* milan_ hopes this won't induce a headache on TheJulia | 17:18 | |
*** fellypefca has quit IRC | 17:18 | |
rloo | https://storyboard.openstack.org/#!/page/about | 17:18 |
TheJulia | milan_: that would be like 20 lines of code aiui | 17:20 |
TheJulia | basically low hanging fruit if one looks at another argument example... I think | 17:20 |
milan_ | cool, I'd set it to confirmed then | 17:21 |
milan_ | afaictl Anshul might appreciate a response but no pressure :P | 17:22 |
milan_ | no new bugs anymore | 17:23 |
etingof | \o/ | 17:23 |
milan_ | #pixiesay -mlikeaboss nice job, milan_ ! | 17:23 |
PixieBoots | ʕ▀̿ᴥ▀̿ʔ: nice job, milan_ ! | 17:23 |
milan_ | Thank you PixieBoots, you're a kind boss! | 17:23 |
* milan_ 's eyes are crossing -> break time | 17:24 | |
rloo | great job milan_. For doing so well, you get to be the bug triager for THE REST OF THE YEAR! Applause, applause... | 17:24 |
milan_ | lol :D | 17:25 |
milan_ | #pixiesay -mlikeaboss rloo, team work :P | 17:26 |
PixieBoots | ʕ▀̿ᴥ▀̿ʔ: rloo, team work :P | 17:26 |
rloo | milan_: :) | 17:26 |
NobodyCam | /o\ \o\ /o/ \o/ | 17:26 |
TheJulia | milan_: I just tagged that one as low hanging fruit and responded to Anshul with the 100 foot view of how to do it | 17:26 |
TheJulia | It is super easy for things liek that to slip through the cracks when there are something like 395 bugs :( | 17:27 |
*** d0ugal has quit IRC | 17:30 | |
* TheJulia tries to remember what she was going to do next today | 17:31 | |
*** fragatina has joined #openstack-ironic | 17:32 | |
*** absubram has joined #openstack-ironic | 17:34 | |
* TheJulia goes and makes a cup of coffee to put her brain back on the tracks | 17:36 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-ui master: Added support for injecting non-maskable interrupts https://review.openstack.org/516144 | 17:42 |
*** pcaruana has quit IRC | 17:48 | |
*** d0ugal has joined #openstack-ironic | 17:50 | |
anupn | TheJulia, hey i had some comments on https://review.openstack.org/#/c/516144/ patch of Inject NMI of Ironic-UI | 17:52 |
patchbot | patch 516144 - ironic-ui - Added support for injecting non-maskable interrupts | 17:52 |
anupn | TheJulia, but just saw workflow +1 | 17:52 |
TheJulia | I just -1'ed it | 17:52 |
anupn | TheJulia, cool, i am in half review | 17:53 |
anupn | TheJulia, will review that patch completely | 17:53 |
anupn | TheJulia, thanks | 17:53 |
TheJulia | anupn: great catch, please feel free to revise it, since I do agree looking at the rest of the file | 17:54 |
anupn | TheJulia, sure | 17:55 |
TheJulia | crushil: sorry for not replying to your email quickly | 17:56 |
* TheJulia feels bad | 17:56 | |
crushil | TheJulia, Np. I will be coming out with the next patchset soon. :) | 17:58 |
*** derekh has quit IRC | 17:58 | |
TheJulia | awesome | 17:59 |
openstackgerrit | Merged openstack/ironic master: Imported Translations from Zanata https://review.openstack.org/521721 | 18:00 |
*** trown is now known as trown|lunch | 18:03 | |
*** jistr is now known as jistr|off|trng | 18:04 | |
milan_ | TheJulia, yeah thanks for the update, I guess they'll react on it, not that long ago | 18:08 |
* milan_ -> afk | 18:08 | |
milan_ | Night all! | 18:08 |
TheJulia | goodnight milan_! | 18:08 |
milan_ | o/ :) | 18:09 |
*** milan_ has quit IRC | 18:09 | |
*** e0ne has quit IRC | 18:10 | |
*** vsaienk0 has quit IRC | 18:18 | |
openstackgerrit | Mateusz Kowalski proposed openstack/ironic master: Use disk_label from node.capabilities https://review.openstack.org/521975 | 18:19 |
*** amarao has quit IRC | 18:22 | |
*** ccamacho has joined #openstack-ironic | 18:24 | |
*** hw_wutianwei has quit IRC | 18:26 | |
*** ijw has joined #openstack-ironic | 18:31 | |
rloo | TheJulia: if you have a few min to update, so I can scratch that off my list: https://review.openstack.org/#/c/516445/ | 18:31 |
patchbot | patch 516445 - bifrost (stable/ocata) - Cleanup stable branch backport for CI | 18:31 |
*** ccamacho has quit IRC | 18:33 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost stable/ocata: Cleanup stable branch backport for CI https://review.openstack.org/516445 | 18:34 |
TheJulia | rloo: done | 18:35 |
*** ijw has quit IRC | 18:35 | |
*** ccamacho has joined #openstack-ironic | 18:36 | |
rloo | TheJulia: thx! | 18:37 |
*** AlexeyAbashkin has joined #openstack-ironic | 18:40 | |
*** AlexeyAbashkin has quit IRC | 18:44 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Catch OSError thrown when hexdump is missing https://review.openstack.org/520223 | 18:47 |
*** blinky_ghost has quit IRC | 18:54 | |
*** Sukhdev has joined #openstack-ironic | 18:59 | |
*** racedo has quit IRC | 19:03 | |
*** trown|lunch is now known as trown | 19:03 | |
*** Sukhdev has quit IRC | 19:04 | |
*** racedo has joined #openstack-ironic | 19:04 | |
*** racedo has quit IRC | 19:08 | |
*** ijw has joined #openstack-ironic | 19:08 | |
*** hoonetorg has quit IRC | 19:09 | |
*** hoonetorg has joined #openstack-ironic | 19:10 | |
*** AlexeyAbashkin has joined #openstack-ironic | 19:11 | |
*** ijw has quit IRC | 19:12 | |
openstackgerrit | Rushil Chugh proposed openstack/ironic master: Add XClarity Driver https://review.openstack.org/519427 | 19:14 |
*** AlexeyAbashkin has quit IRC | 19:16 | |
openstackgerrit | Ruby Loo proposed openstack/bifrost master: Clean up zuul files https://review.openstack.org/521988 | 19:22 |
*** crushil has quit IRC | 19:22 | |
*** ccamacho has quit IRC | 19:28 | |
*** mgoddard has quit IRC | 19:29 | |
*** ijw has joined #openstack-ironic | 19:41 | |
*** e0ne has joined #openstack-ironic | 19:46 | |
*** ijw has quit IRC | 19:47 | |
*** ijw has joined #openstack-ironic | 19:48 | |
*** snasir has joined #openstack-ironic | 19:50 | |
snasir | Hi guys, does Ironic support Ipv6 in its Newton release ? | 19:50 |
*** jtomasek has quit IRC | 19:51 | |
openstackgerrit | Ruby Loo proposed openstack/ironic-lib stable/ocata: zuul: Centralize irrelevant-files in legacy-ironic-lib-dsvm-base https://review.openstack.org/521993 | 19:54 |
*** ijw has quit IRC | 19:55 | |
openstackgerrit | Ruby Loo proposed openstack/ironic-python-agent stable/pike: Clean-up zuul.d/ files https://review.openstack.org/521994 | 19:56 |
TheJulia | snasir: I _think_ that is around when people first started to use it, but there are some potential issues, since its use is fairly rare in those operators that provide us feedback | 19:59 |
JayF | Unless it's changed since I was working actively on the project, even our minimal v6 support required you use a virtual media driver -- not one that relied on PXE. | 20:02 |
openstackgerrit | Andreas Jaeger proposed openstack/ironic master: Update bindep.txt for doc builds https://review.openstack.org/521997 | 20:04 |
*** AJaeger has joined #openstack-ironic | 20:04 | |
AJaeger | ironic team, please review https://review.openstack.org/521997 so that we can move forward with updating documentation jobs | 20:05 |
patchbot | patch 521997 - ironic - Update bindep.txt for doc builds | 20:05 |
*** bcornec has quit IRC | 20:06 | |
*** Goneri has quit IRC | 20:10 | |
rloo | AJaeger: sure, will wait for tests to pass first :) | 20:10 |
*** AlexeyAbashkin has joined #openstack-ironic | 20:11 | |
AJaeger | rloo: thanks. | 20:11 |
AJaeger | Note I'll backport directly as well | 20:11 |
rloo | AJaeger: so much work, that governance... :) | 20:12 |
AJaeger | nicer infra jobs ;) | 20:12 |
rloo | AJaeger: how did you go from AJaeger to jaegerandi ? | 20:13 |
AJaeger | rloo: login... should I change? | 20:14 |
AJaeger | and no need to backport | 20:14 |
rloo | AJaeger: nope, whatever works for you. I was just wondering. | 20:14 |
*** AlexeyAbashkin has quit IRC | 20:15 | |
*** srart has quit IRC | 20:19 | |
*** crushil has joined #openstack-ironic | 20:20 | |
*** ijw has joined #openstack-ironic | 20:22 | |
*** bcornec has joined #openstack-ironic | 20:25 | |
TheJulia | snasir: Yeah, what JayF said. You could possibly PXE a with v4 and then everything else be v6. It just all depends on your configuration. | 20:35 |
*** ijw_ has joined #openstack-ironic | 20:41 | |
*** snasir has quit IRC | 20:41 | |
*** ijw has quit IRC | 20:44 | |
*** ijw_ has quit IRC | 20:52 | |
*** e0ne_ has joined #openstack-ironic | 20:52 | |
*** priteau has quit IRC | 20:54 | |
*** e0ne has quit IRC | 20:55 | |
*** Goneri has joined #openstack-ironic | 21:03 | |
*** ijw has joined #openstack-ironic | 21:04 | |
openstackgerrit | Ilya Etingof proposed openstack/ironic master: WIP: Redfish driver to read pending boot mode https://review.openstack.org/522019 | 21:07 |
*** ijw has quit IRC | 21:08 | |
openstackgerrit | Ruby Loo proposed openstack/virtualbmc master: Clean up zuul files https://review.openstack.org/522020 | 21:13 |
openstackgerrit | Ruby Loo proposed openstack/virtualbmc master: Clean up zuul files https://review.openstack.org/522020 | 21:15 |
*** etingof has quit IRC | 21:20 | |
*** snasir has joined #openstack-ironic | 21:21 | |
*** etingof has joined #openstack-ironic | 21:22 | |
*** e0ne_ has quit IRC | 21:22 | |
*** vishwanathj has joined #openstack-ironic | 21:23 | |
*** etingof has quit IRC | 21:31 | |
*** etingof has joined #openstack-ironic | 21:32 | |
*** dprince has quit IRC | 21:37 | |
openstackgerrit | Ruby Loo proposed openstack/ironic-python-agent master: Use 'functional' tox environment https://review.openstack.org/522025 | 21:45 |
*** jkilpatr_ has quit IRC | 21:47 | |
*** threestrands has joined #openstack-ironic | 21:48 | |
*** rcernin has joined #openstack-ironic | 21:50 | |
*** ijw has joined #openstack-ironic | 21:56 | |
*** trown is now known as trown|outtypewww | 21:58 | |
*** akhilaki has joined #openstack-ironic | 21:59 | |
*** ijw has quit IRC | 22:00 | |
*** ijw has joined #openstack-ironic | 22:00 | |
*** jkilpatr has joined #openstack-ironic | 22:05 | |
*** ijw_ has joined #openstack-ironic | 22:07 | |
*** ijw has quit IRC | 22:10 | |
*** yangyapeng has joined #openstack-ironic | 22:10 | |
*** akhilaki has quit IRC | 22:10 | |
*** hjensas has quit IRC | 22:10 | |
*** etingof has quit IRC | 22:13 | |
*** yangyapeng has quit IRC | 22:14 | |
openstackgerrit | Ruby Loo proposed openstack/bifrost master: Clean up zuul files https://review.openstack.org/521988 | 22:16 |
rloo | TheJulia: if you have a few min, this should be a quick review and AJaeger wanted it done soon: https://review.openstack.org/#/c/521997/ | 22:21 |
patchbot | patch 521997 - ironic - Update bindep.txt for doc builds | 22:21 |
rloo | or vdrok, since you're in sunny... ^^ | 22:21 |
* vdrok looks | 22:22 | |
*** akhilaki has joined #openstack-ironic | 22:24 | |
*** pmannidi has quit IRC | 22:27 | |
TheJulia | perfect timing for when I stepped away :) | 22:28 |
*** actatux has quit IRC | 22:32 | |
*** etingof has joined #openstack-ironic | 22:35 | |
*** efried is now known as fried_turkey | 22:38 | |
*** crushil has quit IRC | 22:47 | |
snasir | TheJulia: thanks... yeah i have everything else on ipv6, and the pxe network is ipv4 | 22:51 |
*** ijw_ has quit IRC | 22:51 | |
*** akhilaki has quit IRC | 22:51 | |
openstackgerrit | Ruby Loo proposed openstack/python-ironicclient master: DNM, testing zuul job definitions https://review.openstack.org/522035 | 22:52 |
*** crushil has joined #openstack-ironic | 22:57 | |
*** pmannidi has joined #openstack-ironic | 23:04 | |
openstackgerrit | Ruby Loo proposed openstack/python-ironicclient master: DNM, testing zuul job definitions https://review.openstack.org/522035 | 23:09 |
*** bnemec has quit IRC | 23:09 | |
*** jaypipes has quit IRC | 23:11 | |
*** hjensas has joined #openstack-ironic | 23:11 | |
*** bnemec has joined #openstack-ironic | 23:12 | |
*** AlexeyAbashkin has joined #openstack-ironic | 23:12 | |
*** AlexeyAbashkin has quit IRC | 23:16 | |
*** cdearborn has quit IRC | 23:17 | |
*** charcol has joined #openstack-ironic | 23:28 | |
*** crushil has quit IRC | 23:34 | |
*** ElCoyote_ has quit IRC | 23:34 | |
*** hjensas has quit IRC | 23:36 | |
*** cdearborn has joined #openstack-ironic | 23:41 | |
*** ElCoyote_ has joined #openstack-ironic | 23:50 | |
*** d0ugal has quit IRC | 23:58 | |
*** d0ugal has joined #openstack-ironic | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!