| *** chas has quit IRC | 00:02 | |
| *** hoangcx has joined #openstack-ironic | 00:03 | |
| *** hoangcx2 has quit IRC | 00:04 | |
| openstackgerrit | Merged openstack/python-ironicclient: Update help shown for node-delete https://review.openstack.org/356650 | 00:09 |
|---|---|---|
| openstackgerrit | Merged openstack/ironic: Update hacking test-requirement https://review.openstack.org/359414 | 00:12 |
| openstackgerrit | Merged openstack/ironic: Add node serial console documentation https://review.openstack.org/293872 | 00:12 |
| openstackgerrit | Merged openstack/ironic: Removed unneeded vlan settings from neutron config https://review.openstack.org/354685 | 00:13 |
| *** rybridges1 has joined #openstack-ironic | 00:16 | |
| *** bfournie has joined #openstack-ironic | 00:16 | |
| *** rybridges has quit IRC | 00:16 | |
| *** rybridges1 has quit IRC | 00:18 | |
| *** rybridges has joined #openstack-ironic | 00:18 | |
| *** hoangcx has quit IRC | 00:25 | |
| *** jistr has quit IRC | 00:27 | |
| *** rloo has quit IRC | 00:29 | |
| *** jistr has joined #openstack-ironic | 00:30 | |
| openstackgerrit | Merged openstack/python-ironicclient: Add baremetal port delete command to OSC plugin https://review.openstack.org/346075 | 00:31 |
| *** syed_ has quit IRC | 00:42 | |
| *** mtanino has quit IRC | 00:44 | |
| *** wajdi has joined #openstack-ironic | 00:46 | |
| *** rajinir has quit IRC | 00:55 | |
| *** hoangcx has joined #openstack-ironic | 00:59 | |
| *** chlong has joined #openstack-ironic | 01:19 | |
| *** aNupoisc has quit IRC | 01:26 | |
| openstackgerrit | Merged openstack/ironic: Fix iLO drivers inconsistent boot mode default value https://review.openstack.org/345231 | 01:37 |
| *** thrash is now known as thrash|g0ne | 01:50 | |
| *** rama_y has quit IRC | 01:52 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic power interface for soft reboot and soft power off https://review.openstack.org/216730 | 02:04 |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Ipmitool power driver for soft reboot and soft power off https://review.openstack.org/216738 | 02:07 |
| *** sdake_ has joined #openstack-ironic | 02:08 | |
| *** Sukhdev has quit IRC | 02:09 | |
| *** sabeen has joined #openstack-ironic | 02:11 | |
| *** sdake has quit IRC | 02:12 | |
| *** chas has joined #openstack-ironic | 02:14 | |
| *** sdake_ has quit IRC | 02:17 | |
| *** chas has quit IRC | 02:18 | |
| *** sdake has joined #openstack-ironic | 02:19 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: iRMC power driver for soft reboot and soft power off https://review.openstack.org/216743 | 02:22 |
| *** sdake has quit IRC | 02:23 | |
| *** sdake has joined #openstack-ironic | 02:29 | |
| *** sabeen has quit IRC | 02:41 | |
| *** sabeen has joined #openstack-ironic | 02:45 | |
| *** rm_work has quit IRC | 02:48 | |
| *** sabeen has quit IRC | 02:51 | |
| *** rm_work has joined #openstack-ironic | 02:52 | |
| *** dansmith has quit IRC | 02:54 | |
| *** dansmith has joined #openstack-ironic | 03:00 | |
| *** dansmith is now known as Guest71645 | 03:01 | |
| *** sdake has quit IRC | 03:12 | |
| *** sdake has joined #openstack-ironic | 03:13 | |
| *** adu has joined #openstack-ironic | 03:14 | |
| *** baoli_ has quit IRC | 03:14 | |
| *** adu has quit IRC | 03:32 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Timing metrics: iLO drivers https://review.openstack.org/352745 | 03:34 |
| *** chlong has quit IRC | 03:37 | |
| *** appu103_ has joined #openstack-ironic | 03:37 | |
| *** adu has joined #openstack-ironic | 03:39 | |
| *** Sukhdev has joined #openstack-ironic | 03:43 | |
| *** sdake has quit IRC | 03:56 | |
| *** yonglihe has quit IRC | 04:04 | |
| *** links has joined #openstack-ironic | 04:07 | |
| *** chlong has joined #openstack-ironic | 04:10 | |
| *** aparnav has joined #openstack-ironic | 04:13 | |
| *** appu103_ has quit IRC | 04:13 | |
| *** chas has joined #openstack-ironic | 04:15 | |
| *** chas has quit IRC | 04:20 | |
| *** sdake has joined #openstack-ironic | 04:22 | |
| *** yogi has joined #openstack-ironic | 04:36 | |
| *** vsaienko has joined #openstack-ironic | 04:44 | |
| *** jaosorior has joined #openstack-ironic | 04:46 | |
| *** chas has joined #openstack-ironic | 04:46 | |
| *** syed_ has joined #openstack-ironic | 04:49 | |
| *** chas has quit IRC | 04:50 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/360889 | 05:08 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/360890 | 05:08 |
| *** vsaienko has quit IRC | 05:09 | |
| *** Sukhdev has quit IRC | 05:13 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/360897 | 05:13 |
| *** chlong has quit IRC | 05:21 | |
| *** vsaienko has joined #openstack-ironic | 05:23 | |
| *** vsaienko has quit IRC | 05:25 | |
| *** aparnav has quit IRC | 05:35 | |
| *** appu103_ has joined #openstack-ironic | 05:35 | |
| *** adu has quit IRC | 05:57 | |
| *** siva_ has joined #openstack-ironic | 06:07 | |
| *** siva_ has quit IRC | 06:08 | |
| *** chas has joined #openstack-ironic | 06:12 | |
| *** fragatin_ has joined #openstack-ironic | 06:12 | |
| *** fragatina has quit IRC | 06:15 | |
| *** fragatin_ has quit IRC | 06:17 | |
| *** milan has joined #openstack-ironic | 06:20 | |
| *** evgenyl has quit IRC | 06:20 | |
| *** evgenyl has joined #openstack-ironic | 06:22 | |
| *** rcernin has joined #openstack-ironic | 06:25 | |
| *** fragatina has joined #openstack-ironic | 06:36 | |
| *** fragatina has quit IRC | 06:41 | |
| *** vishwanathj has joined #openstack-ironic | 06:43 | |
| *** tesseract- has joined #openstack-ironic | 06:44 | |
| *** hshiina2 has joined #openstack-ironic | 06:48 | |
| *** aNupoisc has joined #openstack-ironic | 06:49 | |
| *** praneshp has quit IRC | 06:55 | |
| *** amotoki has joined #openstack-ironic | 06:56 | |
| *** ccamacho has quit IRC | 06:58 | |
| *** syed_ has quit IRC | 07:02 | |
| *** jaosorior is now known as jaosorior_brb | 07:22 | |
| vsaienk0 | mjturek: I'm going to put my patch at the bottom of the chain and will resolve pep8 errors in portgroup patches | 07:25 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Fix typo in policy.json.sample https://review.openstack.org/360939 | 07:29 |
| *** anteaya has quit IRC | 07:38 | |
| *** ifarkas_afk is now known as ifarkas | 07:41 | |
| *** dtantsur|afk is now known as dtantsur | 07:46 | |
| dtantsur | Morning Ironic | 07:46 |
| *** jpich has joined #openstack-ironic | 07:47 | |
| *** appu103_ has quit IRC | 07:48 | |
| *** aNupoisc has quit IRC | 07:58 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Add functional tests for CLI https://review.openstack.org/357864 | 07:59 |
| *** zzzeek has quit IRC | 08:00 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Switch to osc-lib instead of cliff https://review.openstack.org/360461 | 08:00 |
| *** zzzeek has joined #openstack-ironic | 08:00 | |
| dtantsur | did not merge for some reason ^^^, rebased and reapproved | 08:01 |
| openstackgerrit | Rodion Promyshlennikov proposed openstack/python-ironicclient: Add baremetal port list command to OSC plugin https://review.openstack.org/346722 | 08:04 |
| *** sdake has quit IRC | 08:05 | |
| *** derekh has joined #openstack-ironic | 08:05 | |
| *** sdake has joined #openstack-ironic | 08:05 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 08:10 |
| *** milan has quit IRC | 08:17 | |
| *** openstackgerrit has quit IRC | 08:18 | |
| *** openstackgerrit has joined #openstack-ironic | 08:19 | |
| *** milan has joined #openstack-ironic | 08:19 | |
| *** electrofelix has joined #openstack-ironic | 08:27 | |
| *** lucas-dinner is now known as lucasagomes | 08:28 | |
| *** vishwanathj has quit IRC | 08:36 | |
| vsaienk0 | Morning dtantsur, sambetts, lucasagomes, vdrok, aarefiev and all online Ironic'ers! | 08:36 |
| *** vishwanathj has joined #openstack-ironic | 08:37 | |
| *** hshiina2 has quit IRC | 08:37 | |
| dtantsur | vsaienk0, o/ | 08:37 |
| lucasagomes | dtantsur, vsaienk0 good morning | 08:39 |
| aarefiev | morning vsaienk0, dtantsur, lucasagomes | 08:39 |
| dtantsur | morning lucasagomes, aarefiev | 08:39 |
| lucasagomes | aarefiev, hi there | 08:41 |
| *** sdake has quit IRC | 08:48 | |
| openstackgerrit | Rodion Promyshlennikov proposed openstack/python-ironicclient: Verify JSON response of chassis commands https://review.openstack.org/336596 | 08:48 |
| *** amotoki has quit IRC | 08:54 | |
| *** vsaienko has joined #openstack-ironic | 08:55 | |
| *** vishwanathj has quit IRC | 08:56 | |
| *** vishwanathj has joined #openstack-ironic | 08:57 | |
| *** athomas has joined #openstack-ironic | 08:59 | |
| *** jaosorior_brb is now known as jaosorior | 08:59 | |
| *** jhesketh has quit IRC | 09:06 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/360890 | 09:06 |
| sambetts | Morning all | 09:08 |
| openstackgerrit | Tang Chen proposed openstack/python-ironicclient: Add rst doc file for baremetal node commands https://review.openstack.org/361009 | 09:08 |
| *** jhesketh has joined #openstack-ironic | 09:09 | |
| openstackgerrit | Merged openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/360897 | 09:10 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/360889 | 09:11 |
| tangchen | Morning, all. :) | 09:13 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Add match_root_device_hints() to the utils.py module https://review.openstack.org/358000 | 09:14 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Extend parse_root_device_hints to support operators https://review.openstack.org/348953 | 09:14 |
| lucasagomes | sambetts, tangchen hey there | 09:14 |
| sambetts | o/ lucasagomes | 09:14 |
| tangchen | Hi, lucasagomes, sambetts. :) | 09:14 |
| sambetts | o/ tangchen | 09:15 |
| openstackgerrit | Merged openstack/python-ironic-inspector-client: Add functional tests for CLI https://review.openstack.org/357864 | 09:20 |
| openstackgerrit | Merged openstack/python-ironic-inspector-client: Switch to osc-lib instead of cliff https://review.openstack.org/360461 | 09:20 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Optionally power off the node when needed https://review.openstack.org/360330 | 09:21 |
| dtantsur | morning sambetts, tangchen | 09:23 |
| tangchen | Morning, dtantsur. | 09:25 |
| ifarkas | morning all | 09:25 |
| dtantsur | hey ifarkas | 09:25 |
| openstackgerrit | Tang Chen proposed openstack/python-ironicclient: Add rst doc file for baremetal node commands https://review.openstack.org/361009 | 09:39 |
| *** vsaienko has quit IRC | 09:43 | |
| *** alexpilotti has joined #openstack-ironic | 09:46 | |
| *** alexpilotti has left #openstack-ironic | 09:46 | |
| *** xek has joined #openstack-ironic | 09:47 | |
| *** xek has quit IRC | 09:47 | |
| *** xek has joined #openstack-ironic | 09:48 | |
| *** mpsairam has quit IRC | 09:55 | |
| vdrok | good morning everyon | 10:02 |
| rcarrillocruz | NobodyCam , cinerama : hiya, i'd appreciate if you could +A if you think is ok https://review.openstack.org/#/c/356506/ and https://review.openstack.org/#/c/354626/ , final patches needed for infracloud | 10:06 |
| *** vishwanathj has quit IRC | 10:06 | |
| *** vishwanathj has joined #openstack-ironic | 10:07 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add classes for Portgroups API enablement https://review.openstack.org/347549 | 10:11 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/332177 | 10:11 |
| milan | morning aarefiev, dtantsur, lucasagomes, sambetts, vdrok! :) | 10:11 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add individual_mode_supported to portgroup - DB https://review.openstack.org/361053 | 10:11 |
| milan | and morning Ironic! :) | 10:11 |
| aarefiev | morning milan | 10:11 |
| dtantsur | hey milan, vdrok | 10:11 |
| vdrok | morning milan aarefiev dtantsur | 10:12 |
| milan | aarefiev, dtantsur, sambetts: the stacks patch decorator sucks :-/ | 10:12 |
| * milan removes it | 10:12 | |
| milan | we'll have just node_info.fsm_event | 10:12 |
| milan | that should be enough | 10:12 |
| milan | at least the patch will be much shorter | 10:13 |
| milan | *s/stacks/states/ | 10:15 |
| aarefiev | what was the problem there | 10:15 |
| milan | aarefiev, if a function consumes exceptions no way to move to error state in the decorator. sambetts found it yesterday in the process._reapply fund | 10:16 |
| milan | also, it doesn't give much value added | 10:17 |
| milan | we could do the same if the functions called e.g. node_info.fsm_event(istate.Events.reapply) | 10:17 |
| aarefiev | milan: hmm, what's wrong with catching all exceptions | 10:18 |
| aarefiev | in decorator | 10:18 |
| milan | plus the node_info.finish() may perform either finished/error transition on its own | 10:18 |
| milan | aarefiev, the error won't bubble up the _reapply as it does catch everything | 10:19 |
| milan | so it should perform the error transition too I guess | 10:19 |
| milan | * node_info.finished might do it actually | 10:20 |
| aarefiev | ok, I'll take a look closer, it's hard to imagine not seeing code :) | 10:21 |
| aarefiev | I didn't have time to test a code on this week | 10:22 |
| milan | aarefiev, no worries, thanks for checking! | 10:23 |
| * milan relocates. bbib | 10:23 | |
| *** yogi has quit IRC | 10:28 | |
| *** milan has quit IRC | 10:33 | |
| *** hoangcx has quit IRC | 10:38 | |
| *** chas_ has joined #openstack-ironic | 10:40 | |
| *** chas has quit IRC | 10:42 | |
| lucasagomes | dtantsur, #357983 code-wise looks good, left a nit and question inline | 10:48 |
| lucasagomes | and need tests ofc (it's WIP now, I know) | 10:48 |
| dtantsur | lucasagomes, yes, it is possible to have duplicate names, so I have to document it to avoid people shooting their leg | 10:49 |
| lucasagomes | dtantsur, yeah, cause that may return the wrong UUID then | 10:49 |
| lucasagomes | dtantsur, perhaps we really should raise an error or something ? | 10:49 |
| dtantsur | lucasagomes, I think we should, will fix | 10:49 |
| lucasagomes | not sure it would be ideal to have a node booting on a random network | 10:49 |
| lucasagomes | dtantsur, cool. Yeah +1 for raising an error preventing it and have the documentation to say to _not_ create networks with duplicated names | 10:50 |
| lucasagomes | or, if that's the case use UUID instead | 10:50 |
| dtantsur | ++ | 10:50 |
| *** vishwanathj has quit IRC | 10:53 | |
| rook | lucasagomes: couple of questions - sometimes I see my Dell R730xd not respond to a ipmi reboot, so it just sits there and in ironic node list i see "wait-callback" at which point, I manually reboot the host. Should ironic "retry" to kick the host via ipmi? | 10:53 |
| *** vishwanathj has joined #openstack-ironic | 10:54 | |
| rook | lucasagomes: my second question, I think is solved, but yesterday I was seeing where a instance would be rescheduled, however that rescheduling didn't remove the instance_uuid from ironic. | 10:54 |
| lucasagomes | rook, is it using the _ipmitool driver or dell one? | 10:54 |
| openstackgerrit | Imre Farkas proposed openstack/ironic: DRAC RAID configuration https://review.openstack.org/176319 | 10:54 |
| rook | ipmi | 10:54 |
| lucasagomes | rook, cause I think Ironic would fail the deployment if the reboot command from ipmitool didn't work (it will retry a couple of time, I will check) | 10:54 |
| ifarkas | vdrok, morning, test added ^ ;-) | 10:54 |
| lucasagomes | rook, the problem is if the ipmitool commands returns success but the node didn't do anything | 10:55 |
| lucasagomes | then Ironic wouldn't know | 10:55 |
| rook | lucasagomes right, however couldn't you check the state after you issue the reboot? | 10:56 |
| lucasagomes | rook, #2 it could be related to this https://review.openstack.org/#/c/341253/ | 10:56 |
| rook | even though it returns "OK" you could check to see if the host is in the desired state? | 10:56 |
| * rook doesn't know, just spitballing ideas. | 10:56 | |
| lucasagomes | rook, yeah that would be a way to fix it, we currently trust the ipmitool tool AFAICT | 10:56 |
| sambetts | rook: its hard to check if a node goes from power on to power on | 10:56 |
| lucasagomes | there's that too ^ | 10:57 |
| lucasagomes | sambetts, is it possible to check the last time it was powered on via the BMC ? | 10:57 |
| lucasagomes | not sure, probably not via ipmitool either | 10:57 |
| rook | ^ is what I am looking at now. | 10:57 |
| * lucasagomes googles | 10:57 | |
| sambetts | lucasagomes, rook: IPMItool reboot, is actually a power off followed by a power on, so it should fail if it doesn't power off | 10:59 |
| dtantsur | rook, remving instance_uuid was fixed yesterday in nova IIRC | 10:59 |
| rook | dtantsur: nice - i think you might of mentioned that. | 11:00 |
| sambetts | lucasagomes, rook: it does power off then waits for that to be true, then powers it back on | 11:00 |
| sambetts | and waits for success or error on that | 11:00 |
| lucasagomes | sambetts, oh interesting yes | 11:00 |
| lucasagomes | sambetts, it keeps checking if the node was really powered on ? | 11:00 |
| rook | sambetts so, ironic will issue a power off, check the status, then power on, if it is in the desired state of power off? | 11:00 |
| sambetts | it issues power off, waits until that is true or it encounters a failure, if successful it issues a power on and then waits for sucess or error on that | 11:01 |
| rook | hm, wonder why I had to manually reboot this host to clear the "wait-callback" | 11:02 |
| sambetts | rook: did you see if the node rebooted when Ironic issued it? because it might have actually rebooted but failed to PXE boot for some reason | 11:03 |
| rook | sambetts I issued it via racadm :/ | 11:04 |
| rook | i didn't check the state, however I issued a reboot. | 11:04 |
| * rook didn't want the deploy to timeout, time was of essence. | 11:05 | |
| sambetts | :/ hard to debug in cases like that unless things generate logs | 11:06 |
| lucasagomes | sambetts, indeed | 11:07 |
| lucasagomes | and even if it fails to pxe boot, usually it remembers on with a "no bootable device" message | 11:08 |
| * lucasagomes haven't seem it being powered on yet | 11:08 | |
| sambetts | lucasagomes: depends on the boot devices, I've seen nodes that aren't cleaned failing PXE (for whatever reason) then booting the harddrive | 11:09 |
| lucasagomes | sambetts, yeah it can fallback to others indeed | 11:09 |
| *** thrash|g0ne is now known as thrash | 11:09 | |
| *** lucasagomes is now known as lucas-hungry | 11:21 | |
| *** vsaienko has joined #openstack-ironic | 11:22 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Add 'inspect_ports' as a 'driver_info' parameter to iLO drivers https://review.openstack.org/325230 | 11:33 |
| jlvillal | Hi lucas-hungry | 11:41 |
| vdrok | morning ifarkas lucas-hungry sambetts and jlvillal | 11:41 |
| jlvillal | lucas-hungry, Regarding https://review.openstack.org/#/c/348953/ I was wondering if you saw my comment to the whole patch set? | 11:41 |
| *** vsaienko has quit IRC | 11:42 | |
| jlvillal | Good morning vdrok lucas-hungry sambetts dtantsur vsaienk0 and everyone else :) | 11:42 |
| * jlvillal is unsure why he is awake at 4:42 AM | 11:42 | |
| dtantsur | доброе утро jlvillal, you're early today | 11:42 |
| jlvillal | dtantsur, yes and I'm not happy about it ;) | 11:43 |
| vdrok | jlvillal: maybe you're excited about the accepted talk?:) | 11:44 |
| dtantsur | I can imagine :) | 11:44 |
| dtantsur | oh, really? congrats! | 11:44 |
| *** amotoki has joined #openstack-ironic | 11:44 | |
| jlvillal | vdrok, Maybe! :D | 11:44 |
| jlvillal | dtantsur, Thanks. vsaienk0 and I's talk got approved. | 11:44 |
| dtantsur | cool! mine was not | 11:44 |
| vdrok | :( | 11:44 |
| vsaienk0 | morning jlvillal | 11:45 |
| * jlvillal hopes he gets approved to go the summit. Thinks he has a good chance. | 11:45 | |
| jlvillal | dtantsur, :( | 11:45 |
| *** vishwanathj has quit IRC | 11:48 | |
| *** vishwanathj has joined #openstack-ironic | 11:49 | |
| *** ashestakov has joined #openstack-ironic | 11:50 | |
| *** chas_ has quit IRC | 11:51 | |
| *** jaosorior has quit IRC | 11:53 | |
| *** jaosorior has joined #openstack-ironic | 11:54 | |
| jroll | jlvillal: vsaienk0 \o/ | 11:56 |
| jroll | morning y'all | 11:56 |
| jlvillal | Thanks jroll :) | 11:56 |
| jroll | iOS folks, don't forget to update, nasty bug fix | 11:56 |
| jroll | (apple iOS, not cisco) | 11:57 |
| dtantsur | morning jroll | 11:57 |
| vdrok | morning jroll | 11:57 |
| mat128 | morning everyone | 11:59 |
| openstackgerrit | Tuan proposed openstack/ironic: Fix formatting strings in LOG.error https://review.openstack.org/361127 | 12:03 |
| *** baoli has joined #openstack-ironic | 12:03 | |
| *** Goneri has joined #openstack-ironic | 12:04 | |
| *** milan has joined #openstack-ironic | 12:04 | |
| * milan back | 12:04 | |
| *** baoli has quit IRC | 12:07 | |
| vdrok | morning mat128 | 12:10 |
| *** baoli has joined #openstack-ironic | 12:11 | |
| *** pece has joined #openstack-ironic | 12:16 | |
| *** baoli_ has joined #openstack-ironic | 12:20 | |
| jroll | vdrok: I assume https://review.openstack.org/#/c/326061/ and https://review.openstack.org/#/c/329625/ are still needed? | 12:20 |
| * jlvillal just learned that WhatsApp defaulted to sharing data with Facebook :( https://www.whatsapp.com/faq/general/26000016 on how to turn it off | 12:21 | |
| TheJulia | Good morning everyone | 12:21 |
| vdrok | jroll: yup, and there is another one now, according to your suggestion https://review.openstack.org/359285 | 12:22 |
| vdrok | morning TheJulia | 12:22 |
| TheJulia | jlvillal: That is why I'm very selective with my application usage on the intenrets | 12:22 |
| jlvillal | TheJulia, Good idea. I am probably more lazy about it than I should be... | 12:22 |
| *** dprince has joined #openstack-ironic | 12:23 | |
| TheJulia | jlvillal: my fiancee works in security... so I'm reminded of things often :) | 12:23 |
| jlvillal | heh | 12:23 |
| *** baoli has quit IRC | 12:23 | |
| jroll | vdrok: thanks! | 12:24 |
| ifarkas | jroll, morning! have you seen Doug's comment on https://review.openstack.org/#/c/360559/ ? | 12:24 |
| ifarkas | jroll, wdyt about adding release:independent like in case of dracclient? | 12:25 |
| jroll | ifarkas: I'm good with independent | 12:25 |
| *** lucas-hungry is now known as lucasagomes | 12:25 | |
| lucasagomes | vdrok, jlvillal jroll mat128 TheJulia ifarkas morning | 12:26 |
| lucasagomes | jlvillal, hey, lemme take a look again | 12:26 |
| TheJulia | question, out of curiosity, baremetal nodes, does anyone burn in any sort of NTP time sync/hardware clock reset upon boot of IPA? | 12:26 |
| *** MattMan has quit IRC | 12:26 | |
| *** MattMan has joined #openstack-ironic | 12:27 | |
| jlvillal | lucasagomes, It's okay if you just thought, "meh" and ignored it :) | 12:27 |
| jroll | ifarkas: actually, should we have stable branches for this, though? | 12:27 |
| mat128 | TheJulia: unless IPA/CoreOS does it on it's own, no | 12:27 |
| jroll | TheJulia: we kinda do | 12:27 |
| mat128 | TheJulia: that would probably be a good idea | 12:27 |
| * jroll finds it | 12:27 | |
| * TheJulia is asking because she had an interesting ticket cross her downstream mailbox overnight | 12:28 | |
| * jroll suddenly wonders why we haven't upstreamed this | 12:28 | |
| mat128 | time for patches :) | 12:28 |
| TheJulia | It would make sense with crypto in place | 12:28 |
| jroll | I'll just send it as a (hacky) patch | 12:28 |
| jroll | yeah | 12:28 |
| ifarkas | jroll, yeah, we can have it. what do I need to do for that? | 12:28 |
| jroll | TheJulia: we did it when we started seeing swift ssl validation failing :) | 12:28 |
| jroll | ifarkas: cycle-with-intermediary | 12:29 |
| ifarkas | jroll, oh right :-) | 12:29 |
| lucasagomes | jlvillal, oh I missed the hint_{name, expression} things | 12:29 |
| jroll | ifarkas: not sure if we need a stable branch (I don't think it will change often), but if we want one that's how | 12:29 |
| lucasagomes | jlvillal, probably just looked at the comments in code, sorry | 12:29 |
| TheJulia | jroll: pretty much, I think the issue that they hit was wait callback would never complete to the ironic API in the first place | 12:29 |
| lucasagomes | I agree it's more readable, I will update it | 12:29 |
| jroll | TheJulia: yeah, that too | 12:29 |
| jlvillal | lucasagomes, No worries. I wasn't sure what you thought. | 12:29 |
| jlvillal | lucasagomes, Thanks. | 12:30 |
| lucasagomes | jlvillal, early morning when I looked at it, probably the coffee didn't make effect at the time yet :D | 12:30 |
| jlvillal | heh | 12:30 |
| * jlvillal will see if he can fall asleep for 90 minutes and then wake up for work | 12:30 | |
| ifarkas | jroll, what happens if a project has cycle-with-intermediary tag but won't have any patch in a cycle? | 12:30 |
| jroll | ifarkas: I'm not sure, I think it gets bumped to independent | 12:31 |
| ifarkas | jroll, ok, setting it to cycle-with-intermediary for now, and we might change in the future | 12:32 |
| ifarkas | jroll, should I do the same with dracclient? | 12:32 |
| ifarkas | it's not set to independent | 12:32 |
| jroll | ifarkas: okay, I'll need to +1 that | 12:32 |
| ifarkas | it's now set to independent | 12:32 |
| jroll | sure, can't hurt | 12:32 |
| jroll | oh | 12:32 |
| jroll | ifarkas: we can't change it this late in the cycle, afaik | 12:32 |
| jroll | could chat with dhellmann about it | 12:32 |
| ifarkas | right | 12:32 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic-python-agent: Enforce upper-constraints when building ramdisks https://review.openstack.org/360795 | 12:32 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic-python-agent: Enforce upper-constraints when building ramdisks https://review.openstack.org/360793 | 12:33 |
| ifarkas | jroll, well, I think we can wait for the next cycle. dracclient is not yet using wsmanclient yet. that's the plan for the next cycle | 12:33 |
| jroll | ifarkas: okay, let's just make wsmanclient independent for now, I think | 12:34 |
| ifarkas | ack | 12:34 |
| jroll | thanks, ping me for a +1 on that | 12:34 |
| ifarkas | sure | 12:34 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic-python-agent: Enforce upper-constraints when building ramdisks https://review.openstack.org/360795 | 12:35 |
| mat128 | ^ jlvillal: made the mitaka backport dependent on yours | 12:35 |
| ifarkas | jroll, done: https://review.openstack.org/#/c/361144 | 12:37 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-python-agent: Add NTP configuration for the CoreOS image https://review.openstack.org/361146 | 12:39 |
| jroll | TheJulia: ^^ | 12:39 |
| jlvillal | mat128, I did that last night. | 12:40 |
| TheJulia | jroll: pretty much exactly what I figured :) | 12:41 |
| mat128 | yes I've seen it in the comments you left | 12:41 |
| jroll | TheJulia: also suggestions welcome on not using 4.2.2.1 or whatever :) | 12:41 |
| jlvillal | mat128, Did you find an issue? | 12:41 |
| mat128 | nope, everything worked | 12:41 |
| TheJulia | Well 129.6.15.29 is time-b.nist.gov :) | 12:41 |
| jlvillal | cool | 12:41 |
| mat128 | you can click "Toggle CI" to see the results | 12:41 |
| mat128 | and I inspected the logs and found it actually honored upper-constraints for the right branch | 12:42 |
| jlvillal | woot!! | 12:42 |
| mat128 | I also found out that, by using the logic proposed by tonyb we don't even have to read tox | 12:42 |
| jroll | TheJulia: please comment in gerrit, I am very easily distracted yet this morning | 12:42 |
| * jroll needs more coffee | 12:42 | |
| openstackgerrit | Tuan proposed openstack/ironic-inspector: Fix formatting strings in LOG.error https://review.openstack.org/361148 | 12:42 |
| TheJulia | 10-4 | 12:42 |
| mat128 | because the jenkins slaves have the file the hardcoded path | 12:42 |
| mat128 | TheJulia: 4.2.2.1 is only used to determine if routing is up | 12:44 |
| *** vsaienko has joined #openstack-ironic | 12:45 | |
| TheJulia | mat128: I was tryign to be a little silly :) | 12:45 |
| mat128 | Ahh | 12:47 |
| mat128 | jroll: commented =) | 12:47 |
| mjturek | thanks vsaienk0! | 12:49 |
| * TheJulia apparently needs more coffee | 12:49 | |
| *** jvgrant__ has joined #openstack-ironic | 12:50 | |
| *** amotoki has quit IRC | 12:50 | |
| *** jvgrant_ has quit IRC | 12:51 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent: Fix IPA for stable/mitaka with noauth mode https://review.openstack.org/359991 | 12:51 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add individual_mode_supported to portgroup - DB https://review.openstack.org/361053 | 12:52 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Rely on portgroup individual_mode_supported https://review.openstack.org/360647 | 12:52 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add classes for Portgroups API enablement https://review.openstack.org/347549 | 12:52 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/332177 | 12:52 |
| TheJulia | dtantsur: I'll rebase 360018, but 359991 is absolutely required for IPA to function with stable/mitaka in noauth mode. | 12:53 |
| jroll | mat128: good idea | 12:53 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent: Re-use API client for Heartbeat operations https://review.openstack.org/360018 | 12:53 |
| jlvillal | mat128, Is this a problem? http://logs.openstack.org/55/358855/9/check/gate-tempest-dsvm-ironic-ipa-partition-pxe_ssh-coreos-src/ae5ec21/logs/devstacklog.txt.gz#_2016-08-24_20_07_01_334 | 12:53 |
| dtantsur | TheJulia, why? we don't expect 406 to ever be returned there | 12:53 |
| TheJulia | dtantsur: try it, it is exactly what is returned | 12:53 |
| jlvillal | mat128, I don't see the ...?h=stable/... | 12:53 |
| mjturek | vsaienk0 ^ that fix the test_walk failure? | 12:53 |
| vsaienk0 | mjturek: you mean db test? | 12:54 |
| mjturek | yep | 12:54 |
| dtantsur | TheJulia, well, I guess it's a bug in ironic then... I specifically switched to 404 by reviewer's request | 12:54 |
| mat128 | jlvillal: are you sure this was a job running on a stable branch? | 12:54 |
| vsaienk0 | mjturek: yes, I've fixed that issue | 12:54 |
| mjturek | cool! | 12:54 |
| mat128 | jlvillal: 358855 is the commit in master | 12:54 |
| mat128 | jlvillal: ahh | 12:54 |
| jlvillal | mat128, Oh, I clicked the link you had posted | 12:55 |
| mat128 | jlvillal: I pasted the wrong link | 12:55 |
| mat128 | lmao | 12:55 |
| jlvillal | mat128, https://review.openstack.org/#/c/360793/ | 12:55 |
| * mat128 needs his first coffee | 12:55 | |
| TheJulia | dtantsur: I think it is still an issue in IPA since turning off authorization still means that when trying to do the lookup, the stable/mitaka ironic returns 406 because it doesn't know what to do with the request. | 12:55 |
| mat128 | jlvillal: http://logs.openstack.org/93/360793/1/check/gate-tempest-dsvm-ironic-ipa-partition-pxe_ssh-coreos-src/e20f91d/logs/devstacklog.txt.gz#_2016-08-25_21_36_37_465 | 12:56 |
| *** amotoki has joined #openstack-ironic | 12:56 | |
| TheJulia | s/authorization/authentication/ | 12:57 |
| dtantsur | TheJulia, it should return 404, because there is no such URL... | 12:57 |
| * dtantsur is confused | 12:57 | |
| *** alaski is now known as lascii | 12:57 | |
| jroll | does it return 406 for all invalid urls in noauth mode? O_o | 12:58 |
| TheJulia | that is a really good question, one moment | 12:58 |
| * jroll tests | 12:58 | |
| jlvillal | mat128, That looks better :) | 12:59 |
| mat128 | =) | 12:59 |
| TheJulia | hmm... 404 so far | 12:59 |
| * TheJulia looks up the url it sends to | 12:59 | |
| jroll | stack@jroll-devstack:~/ironic$ curl -i http://localhost:6385/v1/lookup ; echo | 13:00 |
| jroll | HTTP/1.1 404 Not Found | 13:00 |
| jroll | same for /v1/foobar | 13:00 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Add match_root_device_hints() to the utils.py module https://review.openstack.org/358000 | 13:00 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Extend parse_root_device_hints to support operators https://review.openstack.org/348953 | 13:00 |
| lucasagomes | jlvillal, ^ hope it address it all :D | 13:00 |
| TheJulia | but wouldn't it be a post? | 13:00 |
| jroll | ah yeah | 13:01 |
| TheJulia | nope, explicit get | 13:01 |
| jroll | I get a 405 on POST | 13:01 |
| *** mjura has quit IRC | 13:01 | |
| jroll | 404 on GET | 13:01 |
| jlvillal | lucasagomes, Thanks! | 13:01 |
| jroll | even adding ?addresses=foo | 13:02 |
| jlvillal | mat128, The messages are slightly confusing. As it talks about tox, but I'm assuming it actually doesn't use the tox value. | 13:02 |
| mat128 | jlvillal: I have *just* seen your rev 2 on the stable/mitaka | 13:02 |
| mat128 | yeah.. I guess we should print that only if we're falling back to tox | 13:02 |
| mat128 | let me fix that | 13:02 |
| jlvillal | mat128, That's what I said earlier ;) | 13:02 |
| *** amotoki has quit IRC | 13:02 | |
| jlvillal | mat128, Or at least what I was trying to say earlier. But I may not have spoken as clearly as I thought I was. | 13:03 |
| mat128 | when you said you were missing the cherry pick to mitaka, I thought you had done it and simply provided the link | 13:03 |
| lucasagomes | jroll, I wonder if it's another wsme magic | 13:03 |
| mat128 | I failed at reading | 13:03 |
| TheJulia | jroll, I have a sneaking supiciojn it is the microversion | 13:03 |
| * lucasagomes thinks 404 on POST is fine | 13:03 | |
| * jlvillal blames it being 5am when he was talking | 13:03 | |
| * mat128 notices that he still fails at reading :) | 13:03 | |
| mat128 | oh, west coast | 13:03 |
| jlvillal | mat128, Yep! | 13:03 |
| mat128 | you,re up already? | 13:03 |
| jlvillal | mat128, Not because I wanted to. I just did. | 13:03 |
| jroll | TheJulia: aha! | 13:04 |
| jroll | TheJulia: yeah, that's it :( | 13:04 |
| jroll | lucasagomes: the 405 probably is yeah | 13:04 |
| TheJulia | yup, just confirmed it, it is the header | 13:04 |
| TheJulia | That woudl have saved some headache if it got logged to my ramdisk console :( | 13:05 |
| lucasagomes | :-/ | 13:05 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic-python-agent: Enforce upper-constraints when building ramdisks https://review.openstack.org/358855 | 13:06 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent: Fix IPA for stable/mitaka with noauth mode https://review.openstack.org/359991 | 13:06 |
| mat128 | jlvillal: updated, will update cherry picks once we agree on the main one | 13:06 |
| jlvillal | mat128, Good plan | 13:07 |
| *** afaranha has joined #openstack-ironic | 13:10 | |
| rcarrillocruz | folks, i've noticed that core os image must have some sort of logo or something now, because monitoring my bifrost deploys via textcons on the nodes i get on boot image 'monitor is in graphics mode' blah | 13:10 |
| jlvillal | mat128, Regarding: <mat128> I also found out that, by using the logic proposed by tonyb we don't even have to read tox | 13:10 |
| rcarrillocruz | anyone knows how to tweak that to get output? it was superuseful to see what the IPA was doing to debug issues | 13:11 |
| jlvillal | mat128, I think we do still need to have the tox.ini code. Since when I run tox on my local workstation, I only have the value from tox.ini | 13:11 |
| jlvillal | mat128, I assume you know that. But wanted to be sure :) | 13:11 |
| jlvillal | rcarrillocruz, I bet JayF will know, once he wakes up :) | 13:13 |
| *** sdake has joined #openstack-ironic | 13:14 | |
| rcarrillocruz | jlvillal: thx | 13:15 |
| *** sdake_ has joined #openstack-ironic | 13:15 | |
| *** cdearborn has joined #openstack-ironic | 13:16 | |
| mat128 | jlvillal: im about to leave for the office, but yes thats why we still have that code | 13:16 |
| *** dtantsur is now known as dtantsur|brb | 13:16 | |
| TheJulia | rcarrillocruz: is console=ttyS0 set on the pxe_append_params line? | 13:16 |
| rcarrillocruz | let me check | 13:17 |
| mat128 | rcarrillocruz: we recently upgraded the CoreOS version | 13:17 |
| mat128 | rcarrillocruz: nothing to that effect was listed as a change, but it might seem minor to them | 13:17 |
| mat128 | rcarrillocruz: https://github.com/openstack/ironic-python-agent/commit/3b64950247b5dae0aed5b3c756d89e275f1e192b | 13:18 |
| jlvillal | jroll, FYI: IPA stable/mitaka build is broken. Fix posted here: https://review.openstack.org/360823 Might look very familiar :) | 13:18 |
| rcarrillocruz | mat128: you're everywhere! | 13:18 |
| rcarrillocruz | thx for the link :-) | 13:19 |
| jroll | jlvillal: +W, terrible that we missed that | 13:19 |
| *** sdake has quit IRC | 13:19 | |
| * jlvillal hangs his head in shame. | 13:19 | |
| jlvillal | I didn't realize that mitaka was also broken :( | 13:19 |
| *** [1]cdearborn has joined #openstack-ironic | 13:20 | |
| TheJulia | It is friday, no hanging head in shame! | 13:21 |
| TheJulia | :) | 13:21 |
| mat128 | rcarrillocruz: haha | 13:22 |
| mat128 | small world | 13:22 |
| openstackgerrit | Ricardo Araújo Santos proposed openstack/ironic: Reusing oneview_client when possible https://review.openstack.org/358041 | 13:23 |
| jlvillal | TheJulia, :) | 13:23 |
| *** wajdi has quit IRC | 13:24 | |
| xavierr | morning Ironic | 13:27 |
| xavierr | mat128, TheJulia: I'm using bool_from_string as you suggested ;) | 13:28 |
| xavierr | ty | 13:28 |
| *** aswadr_ has joined #openstack-ironic | 13:28 | |
| TheJulia | xavierr: thank you! | 13:28 |
| *** anteaya has joined #openstack-ironic | 13:29 | |
| *** amotoki has joined #openstack-ironic | 13:31 | |
| *** cdearborn has quit IRC | 13:34 | |
| *** sdake_ has quit IRC | 13:37 | |
| *** amotoki has quit IRC | 13:38 | |
| *** sdake has joined #openstack-ironic | 13:40 | |
| *** mgoddard_ has joined #openstack-ironic | 13:44 | |
| *** links has quit IRC | 13:45 | |
| *** mgoddard has quit IRC | 13:47 | |
| *** causten has joined #openstack-ironic | 13:50 | |
| *** amotoki has joined #openstack-ironic | 13:53 | |
| *** pece has quit IRC | 13:56 | |
| *** thiagop has joined #openstack-ironic | 13:59 | |
| *** causten has quit IRC | 14:00 | |
| thiagop | Good morning, ladies and gentlemen | 14:00 |
| *** anush_ has joined #openstack-ironic | 14:00 | |
| jlvillal | thiagop, Good morning | 14:01 |
| jroll | heya thiagop | 14:01 |
| sambetts | woah Hi everyone! many people arrived while I was in meetings :-P | 14:01 |
| *** causten has joined #openstack-ironic | 14:01 | |
| openstackgerrit | Sofiia Andriichenko proposed openstack/python-ironicclient: Add tests for testing node-delete command https://review.openstack.org/251458 | 14:01 |
| openstackgerrit | Merged openstack/ironic-python-agent: Use constraints for all the things https://review.openstack.org/360823 | 14:03 |
| TheJulia | Good morning thiagop, sambetts | 14:05 |
| lucasagomes | thiagop, morning | 14:05 |
| thiagop | jlvillal jroll TheJulia lucasagomes sambetts o/ | 14:06 |
| *** mgoddard_ has quit IRC | 14:09 | |
| *** mgoddard has joined #openstack-ironic | 14:10 | |
| *** syed_ has joined #openstack-ironic | 14:11 | |
| jlvillal | mat128, Commented on https://review.openstack.org/#/c/358855/ Let me know what you think please. | 14:13 |
| *** cdearborn has joined #openstack-ironic | 14:13 | |
| *** dtantsur|brb is now known as dtantsur | 14:15 | |
| vsaienk0 | jroll, sambetts, dtantsur, lucasagomes: I've uploaded a new version of portgroup patches. Please review them https://review.openstack.org/#/q/topic:bug/1526403+status:open | 14:15 |
| dtantsur | morning thiagop, xavierr | 14:16 |
| thiagop | dtantsur: o/ | 14:17 |
| *** rama_y has joined #openstack-ironic | 14:17 | |
| *** mtanino has joined #openstack-ironic | 14:17 | |
| *** michauds has joined #openstack-ironic | 14:18 | |
| *** ifarkas is now known as ifarkas_afk | 14:21 | |
| *** Guest71645 is now known as dansmith | 14:25 | |
| *** Ng has quit IRC | 14:25 | |
| *** vdrok has quit IRC | 14:25 | |
| *** [1]cdearborn has quit IRC | 14:27 | |
| *** Ng has joined #openstack-ironic | 14:28 | |
| *** vdrok has joined #openstack-ironic | 14:28 | |
| *** dprince has quit IRC | 14:30 | |
| *** dprince has joined #openstack-ironic | 14:31 | |
| *** chlong has joined #openstack-ironic | 14:32 | |
| *** causten has quit IRC | 14:32 | |
| *** causten has joined #openstack-ironic | 14:34 | |
| mat128 | jlvillal: python get-pip.py apparently behaves like invoking pip directly. Avoid having to install it. | 14:39 |
| mat128 | jlvillal: "$CHROOT_CMD python /tmp/get-pip.py" actually installs pip, thus it doesn't require -c | 14:40 |
| *** jheroux has joined #openstack-ironic | 14:40 | |
| *** wajdi has joined #openstack-ironic | 14:40 | |
| *** cdearborn has quit IRC | 14:42 | |
| *** cdearborn has joined #openstack-ironic | 14:43 | |
| jlvillal | mat128, But you changed the other get-pip command? | 14:51 |
| mat128 | because it invokes get-pip as if it was pip | 14:51 |
| mat128 | ./get-pip.py install bleh | 14:51 |
| mat128 | when you python get-pip.py, that simply installs pip | 14:51 |
| jlvillal | mat128, Don't we care about constraints when it installs pip? | 14:52 |
| openstackgerrit | Merged openstack/ironic-python-agent: Re-use API client for Heartbeat operations https://review.openstack.org/360018 | 14:52 |
| mat128 | pip itself? | 14:52 |
| jlvillal | mat128, Yeah. Not sure if it installs anything else. | 14:52 |
| mat128 | checking if it accepts -c | 14:52 |
| mat128 | if it does, we'll pass it | 14:53 |
| lucasagomes | vdrok, dtantsur when you have some time mind taking a look at https://review.openstack.org/#/c/360512/ ? | 14:53 |
| mat128 | pip guys should buy the get-pip.py domain | 14:53 |
| lucasagomes | the documentation is broken due that ^ (see bug) | 14:53 |
| jlvillal | heh | 14:53 |
| jroll | jlvillal: mat128: pip isn't in upper-constraints | 14:54 |
| jroll | so I wouldn't worry about it | 14:54 |
| jroll | general rule is always use latest pip | 14:54 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Support names for {cleaning,provisioning}_network https://review.openstack.org/357983 | 14:55 |
| jlvillal | mat128, Seems like pip depends on virtualenv and scripttest | 14:55 |
| jroll | jlvillal: eh? | 14:55 |
| jlvillal | jroll, https://pypi.python.org/pypi/pip | 14:55 |
| jlvillal | At the bottom | 14:55 |
| *** early has quit IRC | 14:55 | |
| jlvillal | Unless I'm mis-reading that requires section | 14:55 |
| jroll | jlvillal: extra == 'testing' | 14:55 |
| vdrok | lucasagomes: will do | 14:55 |
| jroll | means it's an opt-in extra | 14:56 |
| jlvillal | Ah | 14:56 |
| jroll | so pip install pip[testing] requires that | 14:56 |
| jroll | but not just pip | 14:56 |
| *** HenryG has quit IRC | 14:56 | |
| jroll | that did weird me out, given virtualenv depends on pip :P | 14:56 |
| jlvillal | heh | 14:56 |
| jroll | (actually bundles pip, but ya know) | 14:56 |
| jlvillal | jroll, mat128 Okay makes sense. Changing my vote :) | 14:56 |
| mat128 | :) | 14:56 |
| jlvillal | mat128, +2. Thanks for all the work on this! | 14:57 |
| * jlvillal thinks mat128 had no idea when he volunteered ;) | 14:57 | |
| mat128 | haha | 14:58 |
| mat128 | always happy to help | 14:58 |
| jlvillal | Great! | 14:58 |
| mat128 | I also submitted to diskimage-builder a similar change | 14:58 |
| mat128 | with a "depends-on" | 14:58 |
| mat128 | but that doesnt work because it git clones directly | 14:58 |
| mat128 | but that made me think of our idea to have an official dib-built image | 14:59 |
| jlvillal | Another hot day in Oregon. 36C predicted | 14:59 |
| mat128 | is there any traction? I found a review about it, but what did we choose? | 14:59 |
| mat128 | ouch | 14:59 |
| jlvillal | mat128, luckily low humidity here. | 15:00 |
| jlvillal | mat128, Not sure on the dib thing. | 15:00 |
| * jlvillal gets ready to go to work. | 15:00 | |
| sambetts | mat128: we decided that we need to have it supported in devstack and have a CI job for it before we can support an offical DIB build | 15:00 |
| mat128 | a CI job on dib changes or IPA changes? | 15:01 |
| mat128 | I mean | 15:01 |
| *** jistr is now known as jistr|call | 15:01 | |
| sambetts | mat128: all IPA and Ironic changes, like we do for the other ramdisks | 15:01 |
| mat128 | Octavia (LBaaS) builds an appliance "image", and they just pull dib as a dependency during build | 15:01 |
| mat128 | ok | 15:01 |
| *** HenryG has joined #openstack-ironic | 15:01 | |
| jroll | yeah, I like that model | 15:01 |
| mat128 | so we do want to build official DIB images | 15:01 |
| *** early has joined #openstack-ironic | 15:02 | |
| jroll | keep the element in IPA repo | 15:02 |
| mat128 | yes | 15:02 |
| mat128 | ok | 15:02 |
| mat128 | but first step is | 15:02 |
| mat128 | have devstack use dib to build from source? | 15:02 |
| mat128 | then when that works we'll just enable post jobs? | 15:02 |
| mat128 | and keep the ironic-agent element in the dib repo for now? | 15:02 |
| mat128 | (just want to have a clear view on this so I can tackle it when I have time) | 15:03 |
| JayF | cinerama was working on this | 15:03 |
| JayF | I think there's some prereq work in dib itself to make it support this | 15:03 |
| sambetts | there is a patch to add it to devstack too | 15:03 |
| JayF | there's a bug open with some of the deets | 15:03 |
| sambetts | if its not already merged | 15:03 |
| * mat128 reads | 15:03 | |
| mat128 | https://bugs.launchpad.net/ironic-python-agent/+bug/1590935 | 15:04 |
| openstack | Launchpad bug 1590935 in ironic-python-agent "RFE: DIB image should be officially supported in IPA" [Wishlist,In progress] - Assigned to Stephane Miller (stephaneeee) | 15:04 |
| dtantsur | JayF, could you please check my comments on https://review.openstack.org/357963 | 15:04 |
| JayF | https://bugs.launchpad.net/ironic-python-agent/+bug/1590935 | 15:04 |
| mat128 | oh I see | 15:04 |
| mat128 | that's mostly around moving the element | 15:04 |
| sambetts | mat128: DIB IPA is already supported in our devstack plugin, we just need a CI job \ | 15:04 |
| *** rcernin has quit IRC | 15:04 | |
| sambetts | mat128: it merged a little while ago, I forgot | 15:05 |
| mat128 | oh | 15:05 |
| mat128 | https://review.openstack.org/#/c/264579/30/devstack/lib/ironic | 15:05 |
| JayF | dtantsur: I figured the explanation would be something like that, I did say you could merge it over my -1 if I was wrong :P | 15:05 |
| mat128 | so next step is having a CI job for it | 15:05 |
| JayF | dtantsur: going to flip my vote and land it | 15:05 |
| dtantsur | thanks! just wanted to make sure you're fine with it, this is not too urgent :) | 15:06 |
| *** dprince has quit IRC | 15:07 | |
| *** dprince has joined #openstack-ironic | 15:07 | |
| *** vsaienk0|mb has joined #openstack-ironic | 15:08 | |
| *** dprince has quit IRC | 15:08 | |
| *** dprince has joined #openstack-ironic | 15:08 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/ironic-python-agent: Trust certificate based on fingerprint https://review.openstack.org/358457 | 15:09 |
| stevemar | d0ugal: fyi https://review.openstack.org/#/c/360728/ | 15:10 |
| stevemar | d0ugal: you can undo your requirements change now | 15:10 |
| d0ugal | stevemar: I spotted that, thanks - it was lucky timing and saved me :) | 15:10 |
| stevemar | d0ugal: hehe, yep | 15:11 |
| d0ugal | stevemar: I had to anyway, I made the requirements check mad | 15:11 |
| stevemar | ah | 15:11 |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Update documentation for keystone policy support https://review.openstack.org/345556 | 15:11 |
| *** sdake_ has joined #openstack-ironic | 15:11 | |
| d0ugal | stevemar: I was in the middle of doing horrible hacks to catch the import error when somebody linked to that review. | 15:11 |
| lucasagomes | vdrok, another one (small) if you have time https://review.openstack.org/#/c/361288/ | 15:12 |
| *** jistr|call is now known as jistr | 15:12 | |
| vdrok | done :) | 15:13 |
| lucasagomes | vdrok, thanks a lot | 15:13 |
| stevemar | d0ugal: yeah, i was told that linking all the check-osc-plugin jobs *is* the right thing to do, but it was cause a huge backlog with the gate | 15:13 |
| *** xavierr has quit IRC | 15:13 | |
| *** vsaienko has quit IRC | 15:13 | |
| *** xavierr has joined #openstack-ironic | 15:14 | |
| d0ugal | stevemar: I do think it makes sense, once we release next week we will be able to remove our git requirements link. So I'd like to see it enabled again at some point | 15:14 |
| *** sdake has quit IRC | 15:15 | |
| *** vsaienk0|mb has quit IRC | 15:15 | |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic-python-agent: Enforce upper-constraints when building ramdisks https://review.openstack.org/360795 | 15:17 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic-python-agent: Enforce upper-constraints when building ramdisks https://review.openstack.org/360793 | 15:24 |
| mat128 | ^cherrypicks updated | 15:24 |
| *** sergek has quit IRC | 15:26 | |
| openstackgerrit | Rodion Promyshlennikov proposed openstack/python-ironicclient: Add tests for openstackclient plugin https://review.openstack.org/301687 | 15:26 |
| openstackgerrit | Rodion Promyshlennikov proposed openstack/python-ironicclient: Add negative tests for baremetal node commands https://review.openstack.org/337719 | 15:26 |
| *** sergek has joined #openstack-ironic | 15:26 | |
| *** sabeen has joined #openstack-ironic | 15:27 | |
| *** michauds has quit IRC | 15:27 | |
| *** derekh has quit IRC | 15:30 | |
| *** vsaienko has joined #openstack-ironic | 15:31 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Mask instance secrets in API responses https://review.openstack.org/326768 | 15:32 |
| devananda|OPS | one more +2 needed on this doc update for keystone po.icy: https://review.openstack.org/345556 | 15:32 |
| JayF | that one has been stewing a while too | 15:33 |
| *** pcaruana has quit IRC | 15:33 | |
| *** vsaienko has quit IRC | 15:34 | |
| *** tesseract- has quit IRC | 15:38 | |
| lucasagomes | staging drivers docs are not fixed o/ http://ironic-staging-drivers.rtfd.io/ | 15:40 |
| lucasagomes | now* | 15:40 |
| * lucasagomes bad typo | 15:40 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add tempest tests for node Adoption https://review.openstack.org/344975 | 15:40 |
| *** rama_y has quit IRC | 15:47 | |
| *** vsaienko has joined #openstack-ironic | 15:48 | |
| *** vsaienko has quit IRC | 15:51 | |
| devananda|OPS | JayF: another one stewing a while - https://review.openstack.org/326768 | 15:51 |
| *** vsaienko has joined #openstack-ironic | 15:53 | |
| *** vsaienko has quit IRC | 15:56 | |
| *** michauds has joined #openstack-ironic | 15:58 | |
| *** jaosorior has quit IRC | 15:58 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Restrict /heartbeat to only nodes in LOOKUP_ALLOWED states https://review.openstack.org/353696 | 16:01 |
| *** milan has quit IRC | 16:01 | |
| *** praneshp has joined #openstack-ironic | 16:02 | |
| *** tangchen has quit IRC | 16:02 | |
| jlvillal | Rackspace going private: http://ir.rackspace.com/phoenix.zhtml?c=221673&p=irol-newsArticle&ID=2197686 Interesting. /me hopes they continue to kick butt in the upstream! | 16:04 |
| *** vsaienko has joined #openstack-ironic | 16:04 | |
| *** vsaienko has quit IRC | 16:05 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Support names for {cleaning,provisioning}_network https://review.openstack.org/357983 | 16:05 |
| dtantsur | lucasagomes, sambetts, bfournie, should be done now ^^ | 16:05 |
| dtantsur | jlvillal, me too :) | 16:06 |
| *** e0ne has joined #openstack-ironic | 16:06 | |
| lucasagomes | dtantsur, looking | 16:06 |
| jroll | I'll be continuing to work upstream, that's all I can really comment on that :) | 16:06 |
| *** chlong has quit IRC | 16:07 | |
| jlvillal | jroll: Of course | 16:09 |
| openstackgerrit | Tuan proposed openstack/ironic: Fix formatting strings in LOG.error https://review.openstack.org/361127 | 16:09 |
| *** praneshp has quit IRC | 16:18 | |
| NobodyCam | Good Morning Ironic'ers | 16:18 |
| NobodyCam | and OFC .... | 16:19 |
| NobodyCam | TGIF | 16:19 |
| *** chlong has joined #openstack-ironic | 16:19 | |
| *** tangchen has joined #openstack-ironic | 16:20 | |
| dtantsur | morning NobodyCam :) | 16:20 |
| sambetts | TGIF NobodyCam o/ | 16:20 |
| NobodyCam | hey Morning dtantsur and sambetts :) | 16:20 |
| *** e0ne has quit IRC | 16:21 | |
| *** aNupoisc has joined #openstack-ironic | 16:22 | |
| lucasagomes | dtantsur, reviwed, left another question inline | 16:22 |
| lucasagomes | and suggestion | 16:22 |
| *** rama_y has joined #openstack-ironic | 16:23 | |
| lucasagomes | dtantsur, also if you have a second: https://review.openstack.org/#/c/361288/ this just fix the iboot mock in staging | 16:23 |
| dtantsur | lucasagomes, not sure I get which "objects" you're referring to in your questions... I'm logging/outputting only the request name and the list of UUIDs... | 16:23 |
| dtantsur | sure, will check | 16:23 |
| lucasagomes | dtantsur, oh, maybe I overlooked! | 16:24 |
| lucasagomes | dtantsur, I thought it was the whole network representation | 16:24 |
| lucasagomes | dtantsur, my bad, I missed the list comprehension there | 16:25 |
| NobodyCam | morning lucasagomes :) | 16:26 |
| lucasagomes | NobodyCam, hey there | 16:26 |
| NobodyCam | happy friday :) | 16:26 |
| lucasagomes | NobodyCam, you too, I'm almost calling it a day! | 16:29 |
| lucasagomes | TGIF | 16:29 |
| * dtantsur is actually calling it a day | 16:30 | |
| dtantsur | have a great weekend, see you all on Monday! | 16:30 |
| NobodyCam | have a good night dtantsur | 16:30 |
| NobodyCam | and weekend | 16:30 |
| lucasagomes | dtantsur, enjoy the weekend | 16:30 |
| *** dtantsur is now known as dtantsur|afk | 16:30 | |
| *** fragatina has joined #openstack-ironic | 16:31 | |
| *** fragatina has quit IRC | 16:33 | |
| *** fragatina has joined #openstack-ironic | 16:33 | |
| vdrok | morning NobodyCam and night dtantsur|afk :) | 16:34 |
| * sambetts -> home \ | 16:34 | |
| sambetts | night everyone o/ | 16:34 |
| sambetts | have a great weekend | 16:34 |
| NobodyCam | morning vdrok | 16:34 |
| *** sambetts is now known as sambetts|afk | 16:34 | |
| NobodyCam | night sambetts | 16:34 |
| sambetts|afk | cya NobodyCam | 16:34 |
| NobodyCam | have a good one | 16:34 |
| *** praneshp has joined #openstack-ironic | 16:35 | |
| *** david-lyle has joined #openstack-ironic | 16:36 | |
| *** afaranha has quit IRC | 16:36 | |
| *** milan has joined #openstack-ironic | 16:37 | |
| *** ashestakov has quit IRC | 16:37 | |
| *** nicodemos has quit IRC | 16:38 | |
| lucasagomes | sambetts|afk, g'night, enjoy the weekend | 16:38 |
| lucasagomes | I'm also calling it a day | 16:38 |
| lucasagomes | have a great evening all! | 16:39 |
| *** hshiina has quit IRC | 16:39 | |
| *** lucasagomes is now known as lucas-afk | 16:39 | |
| *** electrofelix has quit IRC | 16:39 | |
| vdrok | night sambetts|afk and lucas-afk | 16:40 |
| *** jpich has quit IRC | 16:40 | |
| NobodyCam | night lucas-afk | 16:40 |
| openstackgerrit | Merged openstack/ironic: Switch Inspector interface to pass keystoneauth sessions https://review.openstack.org/357963 | 16:42 |
| openstackgerrit | Merged openstack/ironic: Fix typo in policy.json.sample https://review.openstack.org/360939 | 16:44 |
| *** rcernin has joined #openstack-ironic | 16:44 | |
| *** links has joined #openstack-ironic | 16:45 | |
| *** praneshp has quit IRC | 16:45 | |
| openstackgerrit | Michael Turek proposed openstack/ironic: Add classes for Portgroups API enablement https://review.openstack.org/347549 | 16:49 |
| mjturek | vdrok: ^ addressed your comment. Sorry I removed that case! | 16:50 |
| mjturek | ahhh and reno | 16:50 |
| vdrok | mjturek: no problem ,thanks :) | 16:50 |
| vdrok | yup | 16:50 |
| mjturek | will do :) | 16:51 |
| vdrok | mjturek: oh, and please move test_roots down too, see the comment on the second patch | 16:51 |
| vdrok | the more I review the more I think this split brings more pain than value :) | 16:53 |
| *** mjturek has quit IRC | 16:55 | |
| openstackgerrit | Tuan proposed openstack/ironic-inspector: Fix formatting strings in LOG.error https://review.openstack.org/361148 | 16:56 |
| *** bfournie has quit IRC | 16:58 | |
| *** sabeen has quit IRC | 16:58 | |
| openstackgerrit | Tuan proposed openstack/ironic-inspector: Fix formatting strings in LOG.error https://review.openstack.org/361148 | 16:59 |
| *** anush_ has quit IRC | 17:00 | |
| *** mjturek has joined #openstack-ironic | 17:02 | |
| *** chlong has quit IRC | 17:02 | |
| *** praneshp has joined #openstack-ironic | 17:04 | |
| mat128 | tonyb: if you have time to re-review the upper-constraints stuff, I'd be grateful :) https://review.openstack.org/#/c/358855/ | 17:05 |
| openstackgerrit | Thiago Paiva Brito proposed openstack/ironic: Onetime boot when set_boot_device isn't persistent https://review.openstack.org/340596 | 17:08 |
| jlvillal | mat128: FYI: He should be online on Monday I think. Australia. So their Friday is done with. | 17:09 |
| mat128 | oooh | 17:09 |
| jlvillal | mat128: They live in the future!!!! | 17:09 |
| mat128 | was hoping of closing this topic this week | 17:09 |
| mat128 | yup :) | 17:09 |
| mat128 | reminds me of Y2K | 17:09 |
| jlvillal | heh | 17:09 |
| *** [1]cdearborn has joined #openstack-ironic | 17:11 | |
| *** nicodemos has joined #openstack-ironic | 17:13 | |
| *** rcernin has quit IRC | 17:22 | |
| *** fragatina has quit IRC | 17:23 | |
| *** cdearborn has quit IRC | 17:25 | |
| *** sabeen has joined #openstack-ironic | 17:26 | |
| *** bfournie has joined #openstack-ironic | 17:27 | |
| openstackgerrit | Tuan proposed openstack/ironic: Fix formatting strings in LOG.error https://review.openstack.org/361127 | 17:28 |
| *** aswadr_ has quit IRC | 17:31 | |
| openstackgerrit | Tuan proposed openstack/ironic-inspector: Fix formatting strings in LOG.error https://review.openstack.org/361148 | 17:32 |
| openstackgerrit | Tuan proposed openstack/ironic: Fix formatting strings in LOG.error https://review.openstack.org/361127 | 17:35 |
| *** vishwanathj has quit IRC | 17:38 | |
| openstackgerrit | Xavier proposed openstack/ironic: Fix for check if dynamic allocation model is enabled https://review.openstack.org/357425 | 17:39 |
| *** vishwanathj has joined #openstack-ironic | 17:39 | |
| xavierr | vdrok ^^^ :) | 17:39 |
| *** ricardoas has left #openstack-ironic | 17:48 | |
| *** jheroux has quit IRC | 17:49 | |
| openstackgerrit | Merged openstack/ironic-specs: Implement rescue mode https://review.openstack.org/171878 | 17:53 |
| jroll | oh snap | 17:54 |
| jroll | JayF: ^ \o/ | 17:54 |
| JayF | what the f | 17:54 |
| JayF | woooooow | 17:54 |
| JayF | <3 devananda|OPS | 17:54 |
| *** adu has joined #openstack-ironic | 17:57 | |
| *** fragatina has joined #openstack-ironic | 17:58 | |
| *** mjturek has quit IRC | 18:00 | |
| *** causten has quit IRC | 18:02 | |
| *** vsaienko has joined #openstack-ironic | 18:02 | |
| *** fragatin_ has joined #openstack-ironic | 18:02 | |
| *** sdake_ has quit IRC | 18:02 | |
| openstackgerrit | Merged openstack/bifrost: Install the net-tools package in scripts/env-setup.sh https://review.openstack.org/359922 | 18:04 |
| *** causten has joined #openstack-ironic | 18:04 | |
| *** fragatina has quit IRC | 18:06 | |
| *** ricardoas has joined #openstack-ironic | 18:08 | |
| devananda|OPS | quick poll - do ya'll use the dashboard from gerrit-dash-creator/dashboard/ironic.dash ? | 18:13 |
| JayF | yes | 18:14 |
| devananda|OPS | I just made a new one, based on some of the queries in swift's dashboard, which seem more helpful | 18:14 |
| JayF | Well, I think it's generated and put into the whiteboard, right? | 18:14 |
| JayF | can you link a shorturl to it here? | 18:14 |
| devananda|OPS | https://github.com/openstack/gerrit-dash-creator/blob/master/dashboards/ironic.dash | 18:15 |
| JayF | I meant more the rendered one :) | 18:15 |
| devananda|OPS | heh | 18:15 |
| devananda|OPS | current one - | 18:15 |
| devananda|OPS | https://review.openstack.org/#/dashboard/?foreach=%28project%3Aopenstack%2Fironic+OR+project%3Aopenstack%2Fpython%2Dironicclient+OR+project%3Aopenstack%2Fironic%2Dpython%2Dagent+OR+project%3Aopenstack%2Fironic%2Dspecs+OR+project%3Aopenstack%2Fironic%2Dlib+OR+project%3Aopenstack%2Fironic%2Dinspector+OR+project%3Aopenstack%2Fironic%2Dwebclient+OR+project%3Aopenstack%2Fbifrost%29+status%3Aopen+NOT+labe | 18:15 |
| devananda|OPS | l%3ACode%2DReview%3C%3D%2D2&title=Ironic+Inbox&My+Patches+Requiring+Attention=owner%3Aself+%28label%3AVerified%2D1%2Cjenkins+OR+label%3ACode%2DReview%2D1+OR+label%3AWorkflow%3C%3D%2D1%29&Ironic+Specs=NOT+label%3AWorkflow%3E%3D1+NOT+owner%3Aself+project%3Aopenstack%2Fironic%2Dspecs&Needs+Approval=NOT+label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D%2D1+label%3AVerified%3E%3D1%2Cjenkins+NOT+owner%3A | 18:15 |
| devananda|OPS | self+label%3ACode%2DReview%3E%3D2+NOT+label%3ACode%2DReview%2D1&Needs+Reverify=label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D%2D1+label%3AVerified%3C%3D%2D1%2Cjenkins&5+Days+Without+Feedback=NOT+label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D%2D1+label%3AVerified%3E%3D1%2Cjenkins+NOT+owner%3Aself+NOT+project%3Aopenstack%2Fironic%2Dspecs+NOT+label%3ACode%2DReview%3C%3D2+age%3A5d&No+Negative+Fe | 18:15 |
| devananda|OPS | edback+%28top+50%29=NOT+label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D%2D1+label%3AVerified%3E%3D1%2Cjenkins+NOT+owner%3Aself+NOT+project%3Aopenstack%2Fironic%2Dspecs+NOT+label%3ACode%2DReview%3C%3D%2D1+NOT+label%3ACode%2DReview%3E%3D2+limit%3A50&With+Negative+Feedback+%28top+20%29=NOT+label%3AWorkflow%3C%3D%2D1+NOT+label%3AVerified%3C%3D%2D1%2Cjenkins+NOT+owner%3Aself+NOT+project%3Aopenstack%2F | 18:15 |
| devananda|OPS | ironic%2Dspecs+label%3ACode%2DReview%2D1+limit%3A20&Unreviewed%2C+not+WIP+%28top+20%29=NOT+label%3ACode%2DReview%3C%3D2+NOT+label%3AWorkflow%3C%3D%2D1+NOT+project%3Aopenstack%2Fironic%2Dspecs+limit%3A20&Work+In+Progress+Or+Unverified+%28top+20%29=NOT+label%3AWorkflow%3E%3D1+NOT+owner%3Aself+%28label%3AWorkflow%3C%3D%2D1+OR+label%3AVerified%3C%3D%2D1%2Cjenkins%29+limit%3A20 | 18:15 |
| JayF | this is why I suggested a shorturl :) | 18:15 |
| devananda|OPS | aah. no. | 18:15 |
| JayF | I have the current one in my bookmarks toolbar :) | 18:16 |
| * devananda|OPS was not reading | 18:16 | |
| *** bnemec is now known as beekneemech | 18:16 | |
| devananda|OPS | JayF: http://bit.ly/2bN3d2F | 18:17 |
| devananda|OPS | bah. sec | 18:17 |
| devananda|OPS | JayF: nope, bit.ly is breaking on the link | 18:20 |
| JayF | "dammit jay just go render it yourself" okay :D | 18:20 |
| devananda|OPS | JayF: second link on here: http://paste.openstack.org/show/564045/ | 18:20 |
| *** vsaienko has quit IRC | 18:20 | |
| *** ntpttr___ is now known as ntpttr__ | 18:21 | |
| JayF | Starred (by devananda or jim or myself) | 18:21 |
| JayF | LOL | 18:21 |
| JayF | Going to add me to that list? Can I be in the cool kids starring club? :P | 18:21 |
| devananda|OPS | apparently the swift team collectively "star" patches to upvote them | 18:21 |
| devananda|OPS | so "starred by the PTL" is a thing they do | 18:21 |
| * jlvillal tries the dash board... | 18:21 | |
| devananda|OPS | I was just playing with it | 18:22 |
| JayF | I like this a lot better | 18:22 |
| devananda|OPS | each time a reviewer stars a patch, it gets added to a list with a weight, based on how active that reviewer is | 18:22 |
| *** vsaienko has joined #openstack-ironic | 18:22 | |
| devananda|OPS | kinda like the LP "affects me too" heat map, but for reviews | 18:22 |
| * jlvillal didn't realize could stalk him and see what patches he has starred ;) | 18:23 | |
| devananda|OPS | I didn't realize that either! now that I do, it seems pretty useful | 18:23 |
| devananda|OPS | I already star the things I want to track / follow up on | 18:23 |
| devananda|OPS | never thought that would be useful to anyone else though | 18:23 |
| * mat128 didnt know you could star a patch.. :P | 18:25 | |
| devananda|OPS | I use that as a gertty view all the time, fwiw | 18:25 |
| *** ntpttr__ has quit IRC | 18:25 | |
| devananda|OPS | is:open is:starred | 18:25 |
| *** dprince has quit IRC | 18:26 | |
| *** bnemec has joined #openstack-ironic | 18:26 | |
| *** beekneemech has quit IRC | 18:26 | |
| * mat128 wonders why https://review.openstack.org/#/c/358855/ isnt showing up | 18:26 | |
| *** bnemec is now known as beekneemech | 18:27 | |
| * mat128 signs off | 18:28 | |
| mat128 | have a good weekend everyone | 18:28 |
| *** wajdi_ has joined #openstack-ironic | 18:28 | |
| *** vsaienko has quit IRC | 18:28 | |
| *** phschwartz has quit IRC | 18:31 | |
| *** wajdi has quit IRC | 18:31 | |
| JayF | I should make my own dash | 18:34 |
| JayF | I want #1 thing in the dash to be: things I had a review on that have new patch sets since I reviewed them | 18:34 |
| JayF | with some kind of time bound on it | 18:34 |
| JayF | idk, might be a good weekend task | 18:34 |
| JayF | I have a short friday today though, so I'm getting off IRC o/ thanks for the new dash deva, I'm going to bookmark it | 18:34 |
| JayF | have a good day folks | 18:34 |
| *** wajdi has joined #openstack-ironic | 18:36 | |
| devananda|OPS | JayF: if you find a way to make a dash of that, many folks will be happy | 18:37 |
| devananda|OPS | JayF: cheers. have a good weekend! | 18:38 |
| *** wajdi_ has quit IRC | 18:39 | |
| *** rama_y has quit IRC | 18:39 | |
| *** amotoki has quit IRC | 18:40 | |
| *** vsaienko has joined #openstack-ironic | 18:43 | |
| *** Sukhdev has joined #openstack-ironic | 18:47 | |
| *** vsaienko has quit IRC | 18:48 | |
| *** vsaienko has joined #openstack-ironic | 18:49 | |
| *** wajdi_ has joined #openstack-ironic | 18:50 | |
| *** wajdi has quit IRC | 18:50 | |
| *** rama_y has joined #openstack-ironic | 18:52 | |
| *** vsaienko has quit IRC | 18:56 | |
| *** causten has quit IRC | 18:56 | |
| *** vishwanathj has quit IRC | 18:56 | |
| *** vishwanathj has joined #openstack-ironic | 18:57 | |
| *** e0ne has joined #openstack-ironic | 18:57 | |
| *** phschwartz has joined #openstack-ironic | 18:58 | |
| *** vsaienko has joined #openstack-ironic | 18:58 | |
| *** ElCoyote_ has joined #openstack-ironic | 18:59 | |
| *** vsaienko has quit IRC | 19:03 | |
| *** vsaienko has joined #openstack-ironic | 19:03 | |
| *** fragatin_ has quit IRC | 19:03 | |
| *** fragatina has joined #openstack-ironic | 19:04 | |
| *** mjturek has joined #openstack-ironic | 19:06 | |
| *** vsaienko has quit IRC | 19:09 | |
| *** causten has joined #openstack-ironic | 19:11 | |
| *** michauds has quit IRC | 19:18 | |
| *** michauds has joined #openstack-ironic | 19:18 | |
| *** ElCoyote_ has quit IRC | 19:18 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Add multi-tenancy section to security doc https://review.openstack.org/345557 | 19:22 |
| *** ElCoyote_ has joined #openstack-ironic | 19:22 | |
| *** michauds has quit IRC | 19:22 | |
| *** michauds has joined #openstack-ironic | 19:23 | |
| openstackgerrit | Merged openstack/bifrost: Change Vagrant VM to mirror memory/cpu in CI https://review.openstack.org/352921 | 19:29 |
| *** e0ne has quit IRC | 19:30 | |
| *** links has quit IRC | 19:33 | |
| *** vishwanathj has quit IRC | 19:34 | |
| *** vishwanathj has joined #openstack-ironic | 19:35 | |
| *** vsaienko has joined #openstack-ironic | 19:35 | |
| *** sabeen has quit IRC | 19:36 | |
| *** vishwanathj has quit IRC | 19:37 | |
| *** vishwanathj has joined #openstack-ironic | 19:38 | |
| *** dprince has joined #openstack-ironic | 19:43 | |
| *** tdurakov has quit IRC | 20:01 | |
| *** vdrok has quit IRC | 20:01 | |
| *** francoblanco has quit IRC | 20:02 | |
| *** betherly has quit IRC | 20:02 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 20:04 |
| *** serverascode has quit IRC | 20:04 | |
| *** NobodyCam has quit IRC | 20:04 | |
| *** TheJulia has quit IRC | 20:07 | |
| *** snizar has joined #openstack-ironic | 20:09 | |
| openstackgerrit | Sukhdev Kapur proposed openstack/ironic: Add support for Security Groups for baremetal servers https://review.openstack.org/361451 | 20:18 |
| *** Goneri has quit IRC | 20:19 | |
| Sukhdev | jroll, devananda|OPS - please have a look at this security groups patch - https://review.openstack.org/#/c/361451/ | 20:19 |
| openstackgerrit | Sukhdev Kapur proposed openstack/ironic: Add support for Security Groups for baremetal servers https://review.openstack.org/361451 | 20:22 |
| *** ElCoyote_ has quit IRC | 20:22 | |
| jroll | Sukhdev: nice, thanks, reviewed | 20:25 |
| *** e0ne has joined #openstack-ironic | 20:26 | |
| *** ElCoyote_ has joined #openstack-ironic | 20:27 | |
| *** ricardoas has left #openstack-ironic | 20:28 | |
| openstackgerrit | Fabio Dassan proposed openstack/pyghmi: Implement M4 and Kent support https://review.openstack.org/361457 | 20:35 |
| thiagop | jroll: are you around yet? | 20:37 |
| *** aNupoisc has quit IRC | 20:39 | |
| *** sabeen has joined #openstack-ironic | 20:39 | |
| jroll | thiagop: technically, yes | 20:41 |
| thiagop | I'm facing this on my devstack, but a little tired to go debugging: https://bugs.launchpad.net/nova/+bug/1594230 | 20:42 |
| openstack | Launchpad bug 1594230 in OpenStack Compute (nova) "nova hypervisor cannot show ironic node" [Undecided,In progress] - Assigned to Hironori Shiina (shiina-hironori) | 20:42 |
| thiagop | sounds familliar? | 20:42 |
| *** serverascode has joined #openstack-ironic | 20:42 | |
| thiagop | I'm using fake driver, I remember seeing this but I'm not remembering what exactly solves it (and why gats isn't affected) | 20:42 |
| thiagop | gate* | 20:42 |
| jroll | thiagop: ooo yeah | 20:42 |
| jroll | that was fixed recently | 20:43 |
| * jroll finds it | 20:43 | |
| *** ElCoyote_ has quit IRC | 20:43 | |
| *** snizar_ has joined #openstack-ironic | 20:46 | |
| *** snizar has quit IRC | 20:47 | |
| jroll | thiagop: oh this is what I was thinking of, there's a patch there | 20:50 |
| openstackgerrit | Michael Turek proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/332177 | 20:50 |
| thiagop | jroll: sorry, a patch there where? | 20:51 |
| *** wajdi_ has quit IRC | 20:51 | |
| *** snizar_ has quit IRC | 20:51 | |
| jroll | thiagop: in the bug | 20:51 |
| *** wajdi has joined #openstack-ironic | 20:51 | |
| thiagop | jroll: but that one isn't merged yet, and the gate is awesome. How so? | 20:51 |
| jroll | thiagop: lack of reviewer bandwidth? :) | 20:52 |
| jroll | I can poke nova people | 20:52 |
| * jroll reviews first to appear not lazy | 20:52 | |
| *** adu has quit IRC | 20:53 | |
| thiagop | jroll: lol, but that doesn't explains why the gate jobs aren't failing with this error | 20:53 |
| thiagop | maybe that's related to the fake driver itself.... | 20:53 |
| jroll | thiagop: we don't test that API in ironic-enabled jobs, apparently | 20:53 |
| jroll | thiagop: this is part of the reason nova folks want us running full tempest jobs | 20:54 |
| thiagop | jroll: nova appears to be using it to do resource tracking (that's why my devstack is failing by the way) | 20:55 |
| *** aNupoisc has joined #openstack-ironic | 20:55 | |
| *** bfournie has quit IRC | 20:56 | |
| jroll | thiagop: as in, nova is broken with that broken? | 20:56 |
| jroll | O_o | 20:56 |
| thiagop | jroll: broken like "die 935 'Timed out waiting for Nova hypervisor-stats vcpus >= 3'" | 20:56 |
| jroll | thiagop: oh! that's different :) | 20:57 |
| jroll | hypervisor-show is what is broken | 20:57 |
| thiagop | jroll: then tracking the problem, I reached the bug error | 20:57 |
| jroll | that bit is waiting for nova-compute to come up and be able to talk to ironic | 20:57 |
| *** dprince has quit IRC | 20:57 | |
| thiagop | jroll: in nova-api logs | 20:57 |
| jroll | thiagop: idk, it's my understanding that only hypervisor-show is broken, not hypervisor-stats | 20:58 |
| jroll | and only -stats is used in devstack | 20:58 |
| thiagop | stats returns 0 | 20:58 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introducing node introspection state management https://review.openstack.org/348943 | 20:58 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Bypassing NodeInfo.acquire_lock() to try OOL https://review.openstack.org/360311 | 20:58 |
| thiagop | but the ironic nodes seems ok | 20:59 |
| *** snizar has joined #openstack-ironic | 20:59 | |
| jroll | right, because nova-compute isn't communicating with ironic (yet) | 20:59 |
| jroll | check n-cpu logs out | 20:59 |
| jroll | could be an auth problem or something | 20:59 |
| jroll | I need to go for the night but if you remind me monday I'd be happy to help dig into this | 20:59 |
| thiagop | the only error on n-cpu is http://paste.openstack.org/show/564062/ | 21:00 |
| *** betherly has joined #openstack-ironic | 21:00 | |
| thiagop | jroll: ok, have a nice weekend | 21:00 |
| jroll | you too :) | 21:00 |
| *** clenimar has quit IRC | 21:01 | |
| *** sdake has joined #openstack-ironic | 21:02 | |
| Sukhdev | jroll : how do I add release notes - is it in this file doc/source/deploy/multitenancy.rst | 21:04 |
| jroll | Sukhdev: http://docs.openstack.org/developer/reno/usage.html | 21:04 |
| * jroll actually out now | 21:05 | |
| thiagop | jroll: the problems seems to be really related to the fake driver. When I changed the driver to agent_ssh, the vcpu's are reported on hypervisor-stats | 21:05 |
| thiagop | jroll: will look better to file a bug next week | 21:06 |
| *** tdurakov has joined #openstack-ironic | 21:06 | |
| *** vdrok has joined #openstack-ironic | 21:07 | |
| *** NobodyCam has joined #openstack-ironic | 21:08 | |
| *** Goneri has joined #openstack-ironic | 21:08 | |
| *** francoblanco has joined #openstack-ironic | 21:08 | |
| *** TheJulia has joined #openstack-ironic | 21:10 | |
| thiagop | Going out for the weekend too | 21:10 |
| thiagop | see ya | 21:10 |
| *** thiagop has quit IRC | 21:10 | |
| *** milan has quit IRC | 21:11 | |
| *** slagle has quit IRC | 21:12 | |
| openstackgerrit | Michael Turek proposed openstack/ironic: Add classes for Portgroups API enablement https://review.openstack.org/347549 | 21:12 |
| openstackgerrit | Michael Turek proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/332177 | 21:14 |
| *** mjturek has quit IRC | 21:14 | |
| *** e0ne has quit IRC | 21:18 | |
| *** vishwanathj has quit IRC | 21:22 | |
| *** vishwanathj has joined #openstack-ironic | 21:23 | |
| openstackgerrit | Xavier proposed openstack/ironic: Fix for check if dynamic allocation model is enabled https://review.openstack.org/357425 | 21:26 |
| openstackgerrit | Thiago Paiva Brito proposed openstack/ironic: Bring configurations from tempest to ironic_tempest_plugin https://review.openstack.org/355591 | 21:27 |
| *** baoli_ has quit IRC | 21:28 | |
| *** nicodemos has quit IRC | 21:32 | |
| *** ElCoyote_ has joined #openstack-ironic | 21:32 | |
| *** causten has quit IRC | 21:41 | |
| openstackgerrit | Sukhdev Kapur proposed openstack/ironic: Add support for Security Groups for baremetal servers https://review.openstack.org/361451 | 21:44 |
| *** vishwanathj has quit IRC | 21:48 | |
| *** vishwanathj has joined #openstack-ironic | 21:49 | |
| Sukhdev | jroll : I have added the unit test for SG and addressed all the comments - will work on the release notes over the weekend or whenever find some time - I have read the documentation and understand as to what is needed | 21:49 |
| *** Sukhdev has quit IRC | 21:50 | |
| *** wajdi has quit IRC | 21:52 | |
| *** wajdi has joined #openstack-ironic | 21:52 | |
| *** wajdi has quit IRC | 22:00 | |
| *** lascii is now known as alaski | 22:00 | |
| *** sdake_ has joined #openstack-ironic | 22:01 | |
| *** sdake has quit IRC | 22:03 | |
| *** [1]cdearborn has quit IRC | 22:04 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/360889 | 22:06 |
| *** vsaienko has quit IRC | 22:11 | |
| *** vishwanathj has quit IRC | 22:14 | |
| *** vishwanathj has joined #openstack-ironic | 22:15 | |
| *** vsaienko has joined #openstack-ironic | 22:18 | |
| *** rbrud has quit IRC | 22:19 | |
| *** vishwanathj has quit IRC | 22:22 | |
| *** vishwanathj has joined #openstack-ironic | 22:23 | |
| *** michauds has quit IRC | 22:25 | |
| openstackgerrit | Merged openstack/python-ironicclient: Add key_value_pairs_to_dict() method https://review.openstack.org/357363 | 22:32 |
| *** vishwanathj has quit IRC | 22:35 | |
| *** rama_y has quit IRC | 22:36 | |
| *** vishwanathj has joined #openstack-ironic | 22:36 | |
| *** ppiela has quit IRC | 22:36 | |
| *** vdrok has quit IRC | 23:00 | |
| *** betherly has quit IRC | 23:00 | |
| *** TheJulia has quit IRC | 23:01 | |
| *** francoblanco has quit IRC | 23:01 | |
| *** serverascode has quit IRC | 23:01 | |
| *** NobodyCam has quit IRC | 23:01 | |
| *** tdurakov has quit IRC | 23:01 | |
| *** praneshp has quit IRC | 23:01 | |
| *** bapalm_ has quit IRC | 23:06 | |
| *** bapalm has joined #openstack-ironic | 23:11 | |
| *** NobodyCam has joined #openstack-ironic | 23:15 | |
| *** TheJulia has joined #openstack-ironic | 23:15 | |
| *** betherly has joined #openstack-ironic | 23:15 | |
| *** vdrok has joined #openstack-ironic | 23:19 | |
| *** serverascode has joined #openstack-ironic | 23:20 | |
| *** tdurakov has joined #openstack-ironic | 23:23 | |
| *** xavierr has quit IRC | 23:28 | |
| *** romcheg has quit IRC | 23:36 | |
| *** romcheg has joined #openstack-ironic | 23:37 | |
| *** amit213 has quit IRC | 23:37 | |
| *** amit213 has joined #openstack-ironic | 23:40 | |
| *** vishwanathj has quit IRC | 23:40 | |
| *** vishwanathj has joined #openstack-ironic | 23:41 | |
| *** NobodyCam has quit IRC | 23:46 | |
| *** TheJulia has quit IRC | 23:46 | |
| *** betherly has quit IRC | 23:46 | |
| *** vdrok has quit IRC | 23:46 | |
| *** serverascode has quit IRC | 23:46 | |
| *** tdurakov has quit IRC | 23:46 | |
| *** syed_ has quit IRC | 23:52 | |
| *** snizar_ has joined #openstack-ironic | 23:53 | |
| *** snizar has quit IRC | 23:54 | |
| *** baoli has joined #openstack-ironic | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!