| *** naohirot has joined #openstack-ironic | 00:03 | |
| *** Haomeng|2 has joined #openstack-ironic | 00:04 | |
| *** Haomeng has quit IRC | 00:07 | |
| openstackgerrit | Merged openstack/ironic: Move tests into correct directories https://review.openstack.org/215319 | 00:12 |
|---|---|---|
| *** smoriya has joined #openstack-ironic | 00:17 | |
| *** shadower49 has quit IRC | 00:23 | |
| *** shadower49 has joined #openstack-ironic | 00:23 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Fix improper exception catching https://review.openstack.org/216065 | 00:39 |
| *** amotoki has joined #openstack-ironic | 00:47 | |
| *** amotoki has quit IRC | 00:50 | |
| openstackgerrit | Merged openstack/ironic: Fix logging levels in do_node_deploy https://review.openstack.org/215617 | 00:54 |
| *** amotoki has joined #openstack-ironic | 00:56 | |
| *** zhenguo has joined #openstack-ironic | 01:01 | |
| *** achanda has joined #openstack-ironic | 01:26 | |
| *** Haomeng|2 is now known as Haomeng | 01:41 | |
| *** Haomeng has quit IRC | 01:48 | |
| *** UForgotten has quit IRC | 01:48 | |
| *** bnemec has quit IRC | 01:50 | |
| *** Haomeng has joined #openstack-ironic | 01:57 | |
| *** chenglch has joined #openstack-ironic | 01:59 | |
| *** jamielennox is now known as jamielennox|away | 02:20 | |
| *** jamielennox|away is now known as jamielennox | 02:32 | |
| *** UForgotten has joined #openstack-ironic | 02:32 | |
| *** hakimo has joined #openstack-ironic | 02:52 | |
| *** hakimo_ has quit IRC | 02:55 | |
| *** Haomeng|2 has joined #openstack-ironic | 02:58 | |
| *** ramineni has joined #openstack-ironic | 03:00 | |
| *** Haomeng has quit IRC | 03:01 | |
| *** ramineni1 has joined #openstack-ironic | 03:06 | |
| *** ramineni has quit IRC | 03:07 | |
| *** ramineni has joined #openstack-ironic | 03:10 | |
| *** ramineni1 has quit IRC | 03:11 | |
| *** ramineni1 has joined #openstack-ironic | 03:14 | |
| *** ramineni has quit IRC | 03:14 | |
| *** VikasC has joined #openstack-ironic | 03:28 | |
| *** ramineni1 has quit IRC | 03:28 | |
| *** ramineni has joined #openstack-ironic | 03:28 | |
| *** openstackgerrit has quit IRC | 03:31 | |
| *** openstackgerrit has joined #openstack-ironic | 03:32 | |
| *** saripurigopi has joined #openstack-ironic | 03:37 | |
| openstackgerrit | Merged openstack/ironic: Fix improper exception catching https://review.openstack.org/216065 | 03:40 |
| *** rameshg87 has joined #openstack-ironic | 03:45 | |
| *** coolsvap|away is now known as coolsvap | 04:06 | |
| *** chenglch has quit IRC | 04:19 | |
| *** chenglch has joined #openstack-ironic | 04:19 | |
| *** UForgotten has quit IRC | 05:01 | |
| *** UForgotten has joined #openstack-ironic | 05:01 | |
| *** chenglch has quit IRC | 05:06 | |
| *** PradeepV has joined #openstack-ironic | 05:21 | |
| PradeepV | Good morning Ironic | 05:21 |
| * PradeepV slaps Haomeng|2 around a bit with a large fishbot | 05:21 | |
| Haomeng|2 | PradeepV: good morning:) | 05:22 |
| rameshg87 | PradeepV: Haomeng|2: morning o/ | 05:22 |
| Haomeng|2 | rameshg87: morning:) | 05:23 |
| PradeepV | http://paste.openstack.org/show/425722/ | 05:23 |
| PradeepV | Haomeng, i have set the proxy for pip upgrade, still my devstack is failing | 05:23 |
| Haomeng|2 | PradeepV: still not working? did you clean the pip cache? | 05:24 |
| Haomeng|2 | PradeepV: that is odd | 05:24 |
| PradeepV | yes Haomeng | 05:25 |
| *** lazy_prince has joined #openstack-ironic | 05:25 | |
| PradeepV | can you confirm me the cache location and the proxy settings | 05:26 |
| Haomeng|2 | PradeepV: I did not run with proxy, so just google it and find some reference for you:) | 05:26 |
| PradeepV | i will try Haomeng | 05:27 |
| Haomeng|2 | PradeepV: looks like these are actions taken in docker inside | 05:27 |
| *** rameshg871 has joined #openstack-ironic | 05:27 | |
| Haomeng|2 | PradeepV: so I think we have to enable proxy for docker also | 05:27 |
| PradeepV | I have enabled the proxy for docker in /etc/default/docker.io | 05:28 |
| rameshg871 | Haomeng|2: PradeepV: have you enabled in Dockerfile ? | 05:28 |
| rameshg871 | PradeepV: you can try adding line "ENV http_proxy <proxy>" | 05:29 |
| rameshg871 | may be repeat the same for https_proxy as well | 05:29 |
| Haomeng|2 | rameshg871: yes, good idea | 05:29 |
| *** rameshg87 has quit IRC | 05:30 | |
| PradeepV | rameshg871, can you specify the docker file | 05:30 |
| *** saripurigopi has quit IRC | 05:31 | |
| rameshg871 | PradeepV: the file here - https://raw.githubusercontent.com/openstack/ironic-python-agent/master/Dockerfile | 05:32 |
| rameshg871 | PradeepV: need to add something like in http://paste.openstack.org/show/425723/ | 05:32 |
| PradeepV | rameshg871, can you specify the location of the file | 05:38 |
| Haomeng|2 | PradeepV: that is IPA Dockerfile, so you can download ipa source and add proxy into it and build again | 05:43 |
| *** Haomeng|2 is now known as Haomeng | 05:43 | |
| *** PradeepV has quit IRC | 05:47 | |
| *** jamielennox is now known as jamielennox|away | 05:47 | |
| *** yonglihe has quit IRC | 05:48 | |
| *** yonglihe has joined #openstack-ironic | 05:54 | |
| *** PradeepV has joined #openstack-ironic | 05:54 | |
| *** saripurigopi has joined #openstack-ironic | 06:11 | |
| *** lsmola has joined #openstack-ironic | 06:11 | |
| *** irenab has quit IRC | 06:13 | |
| *** ukalifon1 has joined #openstack-ironic | 06:23 | |
| openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: Add support for emulation with VirtualBox https://review.openstack.org/155647 | 06:24 |
| *** killer_prince has joined #openstack-ironic | 06:34 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic-specs: Update raid config spec to add one more RPC API https://review.openstack.org/215431 | 06:36 |
| *** lazy_prince has quit IRC | 06:37 | |
| *** max_lobur has joined #openstack-ironic | 06:44 | |
| *** achanda has quit IRC | 06:48 | |
| *** irenab has joined #openstack-ironic | 06:48 | |
| *** Nisha_away has joined #openstack-ironic | 06:55 | |
| *** ifarkas has joined #openstack-ironic | 07:04 | |
| PradeepV | rameshg871, Hi Ramesh, when running stack again with IPA the packages got installed without any issues, but again it is failing in Step7 with same error | 07:17 |
| PradeepV | rameshg871 http://paste.openstack.org/show/425787/ | 07:17 |
| rameshg871 | PradeepV: can you show me your Dockerfile | 07:17 |
| rameshg871 | PradeepV: I have little experience building agent ramdisk using docker | 07:17 |
| rameshg871 | PradeepV: but for me it the proxy configuration in docker conf file never worked | 07:18 |
| rameshg871 | PradeepV: I used to start docker.io manually exporting the environment variables http_proxy, https_proxy, | 07:18 |
| PradeepV | rameshg871 http://paste.openstack.org/show/425795/ . Here is the docker file which is in location /opt/stack/ironic-python-agent | 07:19 |
| rameshg871 | PradeepV: yeah, looks good to me | 07:20 |
| rameshg871 | PradeepV: may be give a try exporting these variables on shell and starting docker.io manually after stopping the system service | 07:20 |
| PradeepV | rameshg871, the pip update is of latest one http://paste.openstack.org/show/425796/ | 07:23 |
| *** romainh has joined #openstack-ironic | 07:23 | |
| rameshg871 | PradeepV: I think problem is related to docker not getting proxy | 07:23 |
| rameshg871 | PradeepV: I remember I used to do like | 07:24 |
| rameshg871 | sudo service docker.io stop | 07:24 |
| rameshg871 | export http_proxy=<proxy> | 07:24 |
| rameshg871 | sudo -E /usr/bin/docker.io -d | 07:24 |
| rameshg871 | iirc | 07:24 |
| PradeepV | rameshg871, thank you , i will try this once restarting the docker.io | 07:27 |
| *** VikasC_ has joined #openstack-ironic | 07:27 | |
| *** yonglihe has quit IRC | 07:29 | |
| *** VikasC has quit IRC | 07:30 | |
| *** rameshg871 is now known as rameshg87 | 07:31 | |
| *** rameshg87 is now known as rameshg87-lunch | 07:31 | |
| *** jcoufal has joined #openstack-ironic | 07:33 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: oCheck image size before provisioning for agent driver https://review.openstack.org/215254 | 07:35 |
| openstackgerrit | Om Kumar proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 07:37 |
| zhenguo | rameshg871: can you review https://review.openstack.org/#/c/192935/ again please? | 07:37 |
| Haomeng | zhenguo: I reviewed your patch, have conrecn with coments - should we reserve some RAM, such 1G for IPA OS, else we still get image download error with space issue, and it is better that we can configure such reserved ram size, make sense? | 07:41 |
| Haomeng | rameshg87-lunch: any comments about should we have the reserved ram for IPA OS? | 07:41 |
| openstackgerrit | Haomeng,Wang proposed openstack/ironic: Check image size before provisioning for agent driver https://review.openstack.org/215254 | 07:42 |
| zhenguo | Haomeng: yeah, make sense to me. | 07:42 |
| Haomeng | zhenguo: and revised the commit title to remove first "o":) | 07:42 |
| Haomeng | zhenguo: so we can discuss with ramesh after his lunch:) | 07:43 |
| Haomeng | zhenguo: no rush to change code now, we need a good solution first | 07:43 |
| Haomeng | zhenguo: thank you | 07:43 |
| zhenguo | Haomeng: ok, thanks | 07:45 |
| Haomeng | zhenguo: yw | 07:45 |
| *** rameshg87-lunch is now known as rameshg87 | 07:46 | |
| Haomeng | zhenguo: and let me add IPA experts from racespace team to help to review | 07:46 |
| rameshg87 | zhenguo: sure, will take a look at it | 07:47 |
| zhenguo | rameshg87: thanks | 07:47 |
| zhenguo | Haomeng: that's good, thanks | 07:47 |
| *** praneshp has quit IRC | 07:48 | |
| Haomeng | zhenguo: YW | 07:48 |
| rameshg87 | Haomeng: I think we just at least the amount of free ram to cache the compressed qcow2 image | 07:49 |
| * rameshg87 checks | 07:49 | |
| Haomeng | rameshg87: yes, agree with you | 07:49 |
| rameshg87 | Haomeng: yeah, it seems we uncompress the image directly to the disk - https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/shell/write_image.sh#L42 | 07:50 |
| Haomeng | rameshg87: but the current free ram is dynamic | 07:50 |
| rameshg87 | Haomeng: yeah, it depends on the amount of ram used by the agent ramdisk as well | 07:50 |
| rameshg87 | Haomeng: the compressed initrd is at least 100 mb that we know of today | 07:51 |
| rameshg87 | Haomeng: I think the loaded kernel + ramdisk might occupy around 500 mb of physical ram | 07:51 |
| Haomeng | rameshg87: yes, it is in raw format already, so zhenguo, ignore my raw format concern | 07:51 |
| Haomeng | rameshg87: yes | 07:51 |
| zhenguo | Haomeng, rameshg87: ok, so I just left 500mb for kernel and ramdisk? | 07:52 |
| Haomeng | rameshg87: not sure how ipa create ram disk, how to identfy/calucate the ram disk size | 07:52 |
| rameshg87 | Haomeng: +1 to your comment on adding a new CONF property to reserve x mb of physical ram for ipa | 07:52 |
| Haomeng | rameshg87: but not sure if it is good idea:) | 07:53 |
| Haomeng | rameshg87: because I am not very faimily with ipa ram disk creation | 07:53 |
| rameshg87 | Haomeng: but I think that's the best we could do | 07:53 |
| betherly | Morning Ironic :) | 07:53 |
| Haomeng | rameshg87: I think we can just open this patch to wait more comments:) | 07:53 |
| rameshg87 | Haomeng: but another interesting thing is when used with ironic + nova | 07:53 |
| Haomeng | betherly: morning:) | 07:53 |
| Haomeng | rameshg87: yes | 07:53 |
| rameshg87 | Haomeng: it's better to project that much less memory_mb to nova to make sure that the node doesn't get allocated | 07:54 |
| rameshg87 | Haomeng: but only for agent_* drivers | 07:54 |
| Haomeng | rameshg87: yes, you mean for nova scheduler logic? | 07:54 |
| rameshg87 | Haomeng: yes | 07:54 |
| rameshg87 | Haomeng: we do something like that for inspection where we report 1GB less than actual disk size available | 07:54 |
| rameshg87 | Haomeng: to reserve space for partition table and config drive | 07:55 |
| Haomeng | rameshg87: I have same idea before, but not sure how to apply such men size filter, which is dynamic | 07:55 |
| Haomeng | rameshg87: yes | 07:55 |
| rameshg87 | Haomeng: oh yeah, we need compare memory_mb with size of the image | 07:55 |
| Haomeng | rameshg87: yes, it is dynamic, not sure if nova filter support such dynamic filter for each instance | 07:55 |
| rameshg87 | Haomeng: we might be able to fit this logic once we have ironic's own scheduling as well | 07:56 |
| *** yuikotakada has joined #openstack-ironic | 07:56 | |
| Haomeng | rameshg87: I think nova api support, but our ironic driver does not handle such case | 07:56 |
| Haomeng | rameshg87: yes | 07:56 |
| rameshg87 | Haomeng: but it's definitely an important and interesting problem to solve :) | 07:56 |
| yuikotakada | hi, ironic! | 07:56 |
| Haomeng | yuikotakada: morning:) | 07:56 |
| Haomeng | rameshg87: yes | 07:56 |
| Haomeng | rameshg87: :) | 07:56 |
| rameshg87 | morning betherly yuikotakada | 07:56 |
| Haomeng | rameshg87: need more code cheange from nova ironic driver side | 07:56 |
| rameshg87 | yes | 07:57 |
| yuikotakada | Haomeng, rameshg87, morning :) | 07:57 |
| Haomeng | rameshg87: one concern, in nova schedule context, the image should not be in raw format if user run with non-raw such qcow2 image, so nova ironic driver can not get *true* raw size for the input image | 07:58 |
| Haomeng | yuikotakada: :) | 07:58 |
| Haomeng | rameshg87: if we can get raw size during nova scheduling | 07:58 |
| rameshg87 | Haomeng: but why do we need to compare with the raw size of the image when it's qcow2 | 07:59 |
| Haomeng | rameshg87: I think the context just have the image id, and glance does not support to get raw size for non-raw format | 07:59 |
| rameshg87 | Haomeng: we download qcow2 image to memory, and then do a qemu-img directly to write to the disk, right ? | 07:59 |
| zhenguo | Haomeng: image has a min_size property, which we used to check the raw size | 07:59 |
| Haomeng | rameshg87: yes, but in nova schedule filter context, the qcow2 image is not writ to disk | 08:00 |
| zhenguo | Haomeng: like we launch instance wih boot from volume(create from image) | 08:00 |
| Haomeng | zhenguo: min_size? | 08:00 |
| rameshg87 | Haomeng: yes, since we do uncompression directly to disk - in any case, we just need to get the amount of memory to cache the size of the glance image | 08:00 |
| Haomeng | zhenguo: dont think glance will support to get raw size for qcow2 image | 08:00 |
| zhenguo | Haomeng: yes, it's used to specify the min size it's required | 08:01 |
| rameshg87 | Haomeng: if it's uncompressed or compressed | 08:01 |
| *** Nisha_brb has joined #openstack-ironic | 08:01 | |
| Haomeng | rameshg87: yes | 08:01 |
| Haomeng | rameshg87: question - if this code - https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/shell/write_image.sh#L42 - which will convert qcow2 to raw format | 08:01 |
| Haomeng | rameshg87: https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/shell/write_image.sh#L42 | 08:02 |
| Haomeng | rameshg87: qemu-img convert -t directsync -O host_device $IMAGEFILE $DEVICE | 08:02 |
| rameshg87 | Haomeng: yes | 08:02 |
| Haomeng | rameshg87: so if input is qcow2 format, for example, it is 2G, after it is convert to raw, maybe it is 10G | 08:02 |
| Haomeng | rameshg87: glance save the qcow2 format size is 2G | 08:03 |
| rameshg87 | Haomeng: yes, and we only cache the 2G file in memory of the bare metal | 08:03 |
| Haomeng | rameshg87: so I understand in nova schedule context, we just get image type is qcow2 and size is 2G | 08:03 |
| rameshg87 | Haomeng: the 10G file is uncompressed directly to disk | 08:03 |
| Haomeng | rameshg87: ok, got | 08:03 |
| Haomeng | rameshg87: we just download qcow2 into ram disk, then uncompressed to disk, so we just calculate the input image size, whatever the type is | 08:04 |
| Haomeng | rameshg87: :) | 08:04 |
| zhenguo | so we don't need to care about the raw size right? | 08:04 |
| rameshg87 | Haomeng: yes | 08:04 |
| *** Nisha_away has quit IRC | 08:04 | |
| Haomeng | zhenguo: yes, sorry for confusing | 08:04 |
| Haomeng | :) | 08:04 |
| Haomeng | rameshg87: :) | 08:04 |
| zhenguo | Haomeng, rameshg87: thanks for the explaination. really helpful | 08:05 |
| Haomeng | zhenguo: but we need care about the *free ram* size, it is dynamic:) | 08:05 |
| rameshg87 | Haomeng: zhenguo: wc. | 08:06 |
| zhenguo | Haomeng: so we need to add a config option for that? | 08:06 |
| Haomeng | zhenguo: and it is better that we change ironic nova driver code to add such logic to pre-check the ram disk space | 08:06 |
| Haomeng | zhenguo: no such to add option, we need to wait more comments:) | 08:06 |
| Haomeng | maybe we have more good solution:) | 08:06 |
| zhenguo | Haomeng: but nova scheduler seems don't now what driver we used. | 08:07 |
| Haomeng | rameshg87: in ironic nova driver context, we can get I think | 08:07 |
| Haomeng | zhenguo: it is about how to implement, we need to get final solution first, then try to find how to change code | 08:07 |
| rameshg87 | Haomeng: yes | 08:07 |
| zhenguo | Haomeng: ok, will try. | 08:08 |
| *** mgoddard has joined #openstack-ironic | 08:08 | |
| Haomeng | so dont be rush, just wait more comments and disscustions if needed | 08:08 |
| Haomeng | zhenguo: thank you | 08:08 |
| rameshg87 | Haomeng: but I feel it's better to wait for the reservation stuff that jroll is doing to not make more changes in core of nova scheduler for ironic | 08:08 |
| Haomeng | rameshg87: ok, agree with you | 08:09 |
| *** lucasagomes has joined #openstack-ironic | 08:09 | |
| zhenguo | rameshg87: agree | 08:09 |
| zhenguo | Haomeng, rameshg87: I have an ironic driver patch about nova shceduler count available resources of deployed node https://review.openstack.org/#/c/204088/, can you have a look at it please? | 08:12 |
| Haomeng | zhenguo: sure, thanks for your patches:) | 08:13 |
| zhenguo | Haomeng: thanks | 08:13 |
| Haomeng | zhenguo: :) | 08:13 |
| rameshg87 | zhenguo: sure | 08:14 |
| *** derekh has joined #openstack-ironic | 08:14 | |
| zhenguo | rameshg87: thanks :) | 08:14 |
| Haomeng | zhenguo: however for nova patches, I just vote +1, no +2 permision:) | 08:15 |
| Haomeng | zhenguo: :) | 08:16 |
| zhenguo | Haomeng: that's enough, thanks for +1 my patch :) | 08:16 |
| Haomeng | zhenguo: are you working on HuaWei Ironic group? | 08:17 |
| Haomeng | zhenguo: welcome:) | 08:17 |
| *** jistr has joined #openstack-ironic | 08:17 | |
| Haomeng | zhenguo: I understand HuaWei enable ironic to support your systems, right? | 08:17 |
| zhenguo | Haomeng: yes, but I only do upstream, not for production | 08:18 |
| Haomeng | zhenguo: ok, that is great:) | 08:18 |
| zhenguo | Haomeng: heh | 08:19 |
| Haomeng | zhenguo: ironic is not ready for production so far I think, expecial depends on neutron physical switch support and more integration with nms I think | 08:20 |
| zhenguo | Haomeng: yeah, but we made many changes to address that, like we add a agent for ovs and attaching volumes | 08:21 |
| Haomeng | zhenguo: great jobs | 08:21 |
| zhenguo | Haomeng: but I don't think using agent is a good approach | 08:22 |
| Haomeng | zhenguo: you mean ovs agent? | 08:22 |
| Haomeng | zhenguo: or agent in user os? | 08:22 |
| zhenguo | Haomeng: no, we run an agent in the image | 08:22 |
| Haomeng | zhenguo: yes user dont want to run any 3rd agent in user's image/os | 08:23 |
| Haomeng | zhenguo: it should be security issue | 08:23 |
| zhenguo | Haomeng: yeah, that's the problem | 08:23 |
| Haomeng | zhenguo: but if your agent can be certed/trusted, that should be fine | 08:23 |
| Haomeng | zhenguo: and user can accept, and make sure low resource usage | 08:24 |
| zhenguo | Haomeng: yes, that's good | 08:26 |
| Haomeng | zhenguo: :) | 08:26 |
| zhenguo | Haomeng: thanks | 08:26 |
| Haomeng | zhenguo: :) | 08:27 |
| *** romcheg has joined #openstack-ironic | 08:28 | |
| *** Nisha_away has joined #openstack-ironic | 08:29 | |
| *** Nisha_brb has quit IRC | 08:30 | |
| *** dlpartain has joined #openstack-ironic | 08:30 | |
| *** yog_ has joined #openstack-ironic | 08:36 | |
| *** VikasC has joined #openstack-ironic | 08:41 | |
| *** VikasC_ has quit IRC | 08:42 | |
| *** jistr has quit IRC | 08:45 | |
| *** jistr has joined #openstack-ironic | 08:47 | |
| *** shadower49 is now known as shadower | 08:48 | |
| *** alexpilotti has quit IRC | 08:53 | |
| *** Marga_ has joined #openstack-ironic | 08:53 | |
| *** alexpilotti has joined #openstack-ironic | 08:53 | |
| *** Marga_ has quit IRC | 08:53 | |
| *** alexpilotti has quit IRC | 08:53 | |
| *** Marga_ has joined #openstack-ironic | 08:54 | |
| *** romcheg has quit IRC | 08:54 | |
| *** alexpilotti has joined #openstack-ironic | 08:54 | |
| *** romcheg has joined #openstack-ironic | 09:01 | |
| *** jcoufal has quit IRC | 09:03 | |
| *** jistr has quit IRC | 09:08 | |
| *** pelix has joined #openstack-ironic | 09:10 | |
| *** Marga__ has joined #openstack-ironic | 09:12 | |
| *** Marga_ has quit IRC | 09:12 | |
| *** romcheg has quit IRC | 09:14 | |
| *** romcheg has joined #openstack-ironic | 09:14 | |
| openstackgerrit | Merged openstack/ironic-inspector: Mock socket in test_bad_hostname_errors https://review.openstack.org/214558 | 09:24 |
| *** ndipanov has joined #openstack-ironic | 09:27 | |
| *** MattMan has joined #openstack-ironic | 09:33 | |
| *** degorenko has joined #openstack-ironic | 09:36 | |
| *** jcoufal has joined #openstack-ironic | 09:37 | |
| openstackgerrit | Merged openstack/ironic: Fix nits from 'HTTP constants' patch https://review.openstack.org/215840 | 09:45 |
| *** e0ne has joined #openstack-ironic | 09:51 | |
| *** romainh has quit IRC | 09:53 | |
| *** alexpilotti has quit IRC | 09:54 | |
| *** alexpilotti has joined #openstack-ironic | 09:54 | |
| *** naohirot has quit IRC | 09:56 | |
| *** Marga__ has quit IRC | 09:56 | |
| *** yuikotakada has quit IRC | 09:56 | |
| *** Marga_ has joined #openstack-ironic | 09:59 | |
| *** dlpartain has left #openstack-ironic | 10:02 | |
| *** e0ne has quit IRC | 10:08 | |
| *** e0ne has joined #openstack-ironic | 10:18 | |
| openstackgerrit | Satoru Moriya proposed openstack/ironic: Introduce VolumeConnector object https://review.openstack.org/214584 | 10:27 |
| *** Nisha_away has quit IRC | 10:30 | |
| *** Marga_ has quit IRC | 10:31 | |
| *** marzif has joined #openstack-ironic | 10:31 | |
| *** rameshg871 has joined #openstack-ironic | 10:32 | |
| *** ekarlso has quit IRC | 10:32 | |
| *** ekarlso has joined #openstack-ironic | 10:32 | |
| *** rameshg87 has quit IRC | 10:34 | |
| *** e0ne has quit IRC | 10:40 | |
| *** romainh has joined #openstack-ironic | 10:43 | |
| *** romcheg has quit IRC | 10:45 | |
| openstackgerrit | chenglch proposed openstack/ironic: Add iscsi_opc and agent_opc to manage the OpenPOWER servers https://review.openstack.org/209824 | 10:45 |
| *** jamielennox|away has quit IRC | 10:50 | |
| *** lynxman has quit IRC | 10:51 | |
| *** jistr has joined #openstack-ironic | 10:52 | |
| *** lynxman has joined #openstack-ironic | 10:53 | |
| *** adam_g has quit IRC | 10:55 | |
| *** eglute has quit IRC | 10:55 | |
| *** wendar_ has joined #openstack-ironic | 10:56 | |
| *** persia has quit IRC | 10:56 | |
| *** wendar has quit IRC | 10:56 | |
| *** jroll has quit IRC | 10:56 | |
| *** persia has joined #openstack-ironic | 10:57 | |
| *** adam_g has joined #openstack-ironic | 11:01 | |
| *** adam_g has quit IRC | 11:01 | |
| *** adam_g has joined #openstack-ironic | 11:01 | |
| *** eglute has joined #openstack-ironic | 11:02 | |
| *** jroll has joined #openstack-ironic | 11:02 | |
| *** amotoki has quit IRC | 11:02 | |
| lucasagomes | rameshg871, around? | 11:04 |
| *** jamielennox|away has joined #openstack-ironic | 11:04 | |
| *** jamielennox|away is now known as jamielennox | 11:04 | |
| *** romainh has quit IRC | 11:05 | |
| rameshg871 | lucasagomes: hi | 11:05 |
| *** romcheg has joined #openstack-ironic | 11:10 | |
| *** Haomeng|2 has joined #openstack-ironic | 11:12 | |
| lucasagomes | rameshg871, we are testing UEFI mode with pxe_ipmitool, but apparently if you build the ramdisk with dracut it fails with fatal for argument '\''IP=192.168.100.163:192.168.100.20:0.0.0.0:255.255.255.0:host-192-168-100-153::on:::off'' sorry client ip does not make sense for 'on'# | 11:12 |
| lucasagomes | just wondering if you guys have seem it? | 11:12 |
| rameshg871 | lucasagomes: no. I don't remember having tested pxe_ipmitool with uefi boot mode. | 11:14 |
| rameshg871 | lucasagomes: even in internal jenkins with use pxe_ilo instead of pxe_ipmitool | 11:15 |
| *** Haomeng has quit IRC | 11:15 | |
| lucasagomes | rameshg871, https://github.com/openstack/ironic/blob/stable/kilo/doc/source/deploy/install-guide.rst#boot-mode-support | 11:15 |
| lucasagomes | rameshg871, apparently the difference is that pxe_ilo you can switch the boot mode on the fly | 11:15 |
| lucasagomes | where pxe_ipmitool you have to configure the hardware before to boot in UEFI mode | 11:15 |
| rameshg871 | lucasagomes: yeah, but this error doesn't seem to be in ironic conductor, right ? | 11:15 |
| rameshg871 | lucasagomes: is it when ramdisk boots up ? | 11:16 |
| lucasagomes | rameshg871, no it's on the console | 11:16 |
| lucasagomes | yeah | 11:16 |
| lucasagomes | that "ip=" is in the uefi template | 11:16 |
| rameshg871 | lucasagomes: oh we still use the dib ramdisk for pxe_ilo uefi | 11:16 |
| rameshg871 | lucasagomes: are you trying with ironic-agent fedora ramdisk ? | 11:16 |
| lucasagomes | rameshg871, it's another guy testing, I'm just trying to help out. But yeah it's using the deploy-ironic element as well, but it also uses the dracut-ramdisk element | 11:17 |
| lucasagomes | (I will tell him to test it w/o the dracut module) | 11:18 |
| rameshg871 | lucasagomes: yeah we still test it with dib ramdisk and even yesterday's run has gone through fine | 11:19 |
| *** romainh has joined #openstack-ironic | 11:19 | |
| rameshg871 | "ramdisk-image-create deploy-ironic ubuntu iso" is what is used for now | 11:20 |
| lucasagomes | rameshg871, thanks! | 11:24 |
| lucasagomes | yeah I will tell him to not use that dracut element and see if it succeed | 11:24 |
| rameshg871 | okay | 11:25 |
| lucasagomes | rameshg871, thanks muh I let you know if it works | 11:27 |
| rameshg871 | lucasagomes: thanks | 11:28 |
| *** dlpartain has joined #openstack-ironic | 11:28 | |
| *** ramineni has quit IRC | 11:30 | |
| * rameshg871 leaves for the day | 11:32 | |
| *** rameshg871 has quit IRC | 11:32 | |
| *** dlpartain has left #openstack-ironic | 11:43 | |
| *** Marga_ has joined #openstack-ironic | 11:43 | |
| openstackgerrit | Juliana Motira proposed stackforge/pyghmi: Add custom OEM firmware info parsing for Lenovo ThinkServers https://review.openstack.org/214623 | 11:47 |
| *** alexpilo_ has joined #openstack-ironic | 11:53 | |
| *** lucasagomes is now known as lucas-hungry | 11:53 | |
| *** dlpartain has joined #openstack-ironic | 11:54 | |
| *** alexpilotti has quit IRC | 11:56 | |
| *** dlpartain has left #openstack-ironic | 11:56 | |
| *** thrash|wknd is now known as thrash | 11:58 | |
| *** trown|outttypeww is now known as trown | 12:04 | |
| trown | Happy Monday Ironic | 12:07 |
| *** shoutm has joined #openstack-ironic | 12:07 | |
| trown | dtantsur|afk: when you are back, I have a question about https://review.openstack.org/#/c/213159 | 12:08 |
| *** saripurigopi has quit IRC | 12:10 | |
| *** dprince has joined #openstack-ironic | 12:18 | |
| *** VikasC has quit IRC | 12:22 | |
| openstackgerrit | Juliana Motira proposed stackforge/pyghmi: Add custom OEM firmware info parsing for Lenovo ThinkServers https://review.openstack.org/214623 | 12:22 |
| *** alexpilotti has joined #openstack-ironic | 12:22 | |
| *** alexpilo_ has quit IRC | 12:22 | |
| *** jjohnson2 has joined #openstack-ironic | 12:24 | |
| *** alexpilo_ has joined #openstack-ironic | 12:25 | |
| *** alexpilotti has quit IRC | 12:27 | |
| *** daemontool__ has joined #openstack-ironic | 12:27 | |
| *** alexpilo_ has quit IRC | 12:27 | |
| *** daemontool_ has quit IRC | 12:29 | |
| openstackgerrit | Satoru Moriya proposed openstack/ironic: Introduce new RPCs to support volume connector operation https://review.openstack.org/214585 | 12:30 |
| *** jistr has quit IRC | 12:31 | |
| *** alexpilotti has joined #openstack-ironic | 12:31 | |
| *** jistr has joined #openstack-ironic | 12:33 | |
| *** alexpilotti has quit IRC | 12:33 | |
| *** alexpilotti has joined #openstack-ironic | 12:34 | |
| *** alexpilotti has quit IRC | 12:34 | |
| *** alexpilotti has joined #openstack-ironic | 12:34 | |
| *** alexpilotti has quit IRC | 12:36 | |
| *** alexpilotti has joined #openstack-ironic | 12:37 | |
| *** rloo has joined #openstack-ironic | 12:37 | |
| openstackgerrit | Satoru Moriya proposed openstack/ironic: Add REST API for volume connector operation https://review.openstack.org/214586 | 12:39 |
| *** alexpilotti has quit IRC | 12:41 | |
| *** PradeepV has quit IRC | 12:41 | |
| *** dtantsur|afk is now known as dtantsur | 12:46 | |
| dtantsur | good (ugt) afternoon, Ironic | 12:46 |
| dtantsur | trown, hi, here I am :) | 12:46 |
| *** lucas-hungry is now known as lucasagomes | 12:46 | |
| *** Marga_ has quit IRC | 12:46 | |
| lucasagomes | dtantsur, hi there | 12:47 |
| dtantsur | o/ | 12:47 |
| *** alexpilotti has joined #openstack-ironic | 12:49 | |
| trown | dtantsur: cool, I had a question about None in oslo config, because that is how I did it to begin with | 12:50 |
| trown | dtantsur: the issue I ran into was that if we later default the config option to 'swift' or 'local', how would an operator set the option to None? | 12:50 |
| trown | if it defaults to None, it is fine, but I could not figure out how to explicitly set None in the conf file | 12:51 |
| dtantsur | trown, hmm, good point, we probably need to understand literal string "none" instead. just like with auth_strategy | 12:52 |
| trown | dtantsur: cool, that works for me | 12:53 |
| openstackgerrit | Merged stackforge/pyghmi: Add custom OEM firmware info parsing for Lenovo ThinkServers https://review.openstack.org/214623 | 12:53 |
| *** alexpilotti has quit IRC | 12:53 | |
| *** [1]cdearborn has joined #openstack-ironic | 12:57 | |
| *** baoli has joined #openstack-ironic | 13:00 | |
| *** smoriya has quit IRC | 13:02 | |
| *** alexpilotti has joined #openstack-ironic | 13:09 | |
| *** jjohnson2 has quit IRC | 13:21 | |
| *** eglute has quit IRC | 13:21 | |
| *** zz_natorious has quit IRC | 13:22 | |
| *** Marga_ has joined #openstack-ironic | 13:22 | |
| *** jroll has quit IRC | 13:23 | |
| *** e0ne has joined #openstack-ironic | 13:23 | |
| *** mjturek1 has joined #openstack-ironic | 13:24 | |
| *** jjohnson2 has joined #openstack-ironic | 13:24 | |
| *** jamielennox has quit IRC | 13:26 | |
| dtantsur | ifarkas, hi! may I ask you to review a number of small inspector patches? https://review.openstack.org/#/c/212511/ https://review.openstack.org/#/c/212561/ https://review.openstack.org/#/c/212012 https://review.openstack.org/#/c/212007 https://review.openstack.org/#/c/212003 | 13:27 |
| *** irenab has quit IRC | 13:27 | |
| *** eglute has joined #openstack-ironic | 13:28 | |
| *** bapalm_ has joined #openstack-ironic | 13:28 | |
| *** thiagop has joined #openstack-ironic | 13:29 | |
| ifarkas | dtantsur, o/ I will try to squeeze them into today's tasks | 13:29 |
| dtantsur | thanks a ton! | 13:29 |
| dtantsur | some of them are actually 1-2 lines :) | 13:29 |
| ifarkas | sounds good :-) | 13:29 |
| BadCub | morning folks | 13:29 |
| thiagop | good morning Ironic | 13:30 |
| *** nicodemos has joined #openstack-ironic | 13:31 | |
| *** links has joined #openstack-ironic | 13:31 | |
| TheJulia | Good morning | 13:33 |
| thiagop | morning TheJulia BadCub | 13:33 |
| dtantsur | morning TheJulia, BadCub, thiagop and everyone else :) | 13:33 |
| thiagop | folks, we were looking into the process to enable our CI for the OneView driver and the first step is to ask for community approval to put it to vote on the project | 13:35 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Periodically checks for nodes being cleaned https://review.openstack.org/213699 | 13:35 |
| dtantsur | thiagop, I think all new CI initially starts as non-voting, no? Anyway, I would love to see more 3rd party CI | 13:35 |
| thiagop | I think so dtantsur, but anyway the instructions asks for permission from the community | 13:36 |
| *** albertoffb has joined #openstack-ironic | 13:36 | |
| *** zz_natorious has joined #openstack-ironic | 13:37 | |
| *** jroll has joined #openstack-ironic | 13:37 | |
| thiagop | I think just to be sure people wouldn't put anything strange on the CI stack | 13:37 |
| *** zz_natorious is now known as natorious | 13:37 | |
| thiagop | since our driver is not yet merged, could we put that "approval" on today's meeting agenda to be "pre-approved" to put it to work as soon as the driver is merged? | 13:38 |
| dtantsur | thiagop, well, you should definitely announce your intention on the meeting :) | 13:39 |
| thiagop | dtantsur: roger that | 13:40 |
| *** max_lobur has quit IRC | 13:40 | |
| *** max_lobur has joined #openstack-ironic | 13:40 | |
| openstackgerrit | William Stevenson proposed openstack/python-ironicclient: Updates supporting ironic-neutron integration https://review.openstack.org/206144 | 13:42 |
| *** jamielennox|away has joined #openstack-ironic | 13:43 | |
| *** jamielennox|away is now known as jamielennox | 13:43 | |
| sambetts | UGT Morning everyone o/ | 13:45 |
| thiagop | good morning sambetts | 13:45 |
| thiagop | btw, if somebody could put eyes on the OneView driver, we would appreciate very much -> https://review.openstack.org/#/c/191822/ | 13:47 |
| dtantsur | sambetts, o/ | 13:48 |
| openstackgerrit | Merged openstack/ironic-inspector: Copy gitignore from ironic https://review.openstack.org/212561 | 13:49 |
| sambetts | Morning dtantsur thiagop | 13:50 |
| rloo | hello and morning everyone -- sambetts, dtantsur, thiagop, BadCub, lucasagomes, TheJulia, ifarkas and you if i missed you :) | 13:52 |
| dtantsur | rloo, o/ | 13:52 |
| lucasagomes | rloo, sambetts thiagop good morning! | 13:52 |
| ifarkas | morning all! | 13:52 |
| openstackgerrit | Merged openstack/ironic-inspector: Add func3 tox environment for functional testing with Python 3. https://review.openstack.org/212511 | 13:52 |
| thiagop | good morning rloo | 13:53 |
| thiagop | morning too ifarkas | 13:53 |
| *** baoli_ has joined #openstack-ironic | 13:53 | |
| rloo | hi thiagop, the non-voting CI for oneview driver. I don't see any reason not to have it. But seems like something to ask on the ML since not everyone attends the meeting? I can't see why anyone would object though. | 13:55 |
| rloo | thiagop: do you have a patch to add to the CI? | 13:55 |
| thiagop | rloo: we are working on that right now, but we're facing some troubles to set it up | 13:56 |
| *** baoli has quit IRC | 13:57 | |
| thiagop | rloo: I'm just worried about that being the first step at all in the manual | 13:57 |
| rloo | thiagop: worried about what? that we wouldn't approve it so why bother trying to get it to work? | 13:57 |
| thiagop | rloo: something like that... :) | 13:58 |
| thiagop | rloo: and we're yet writing tests for it | 13:59 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Add provision state to set command https://review.openstack.org/206119 | 13:59 |
| rloo | thiagop: are we talking about 3rd party CI, or openstack's CI? | 13:59 |
| thiagop | rloo: 3rd party | 13:59 |
| *** boris-42 has joined #openstack-ironic | 14:00 | |
| rloo | thiagop: it is mentioned in your spec that there will be 3rd party CI and we DO want 3rd party CI. | 14:00 |
| *** amotoki has joined #openstack-ironic | 14:00 | |
| NobodyCam | good morning Ironicers | 14:00 |
| NobodyCam | morning rloo thiagop dtantsur lucasagomes sambetts jlvillal TheJulia jroll JoshNang and everyone else :) | 14:01 |
| thiagop | good morning NobodyCam | 14:01 |
| NobodyCam | hope everyone had a great weekend | 14:01 |
| NobodyCam | :) | 14:01 |
| *** puranamr has joined #openstack-ironic | 14:02 | |
| dtantsur | NobodyCam, morning | 14:02 |
| rloo | hiya NobodyCam | 14:02 |
| * dtantsur is just back from Prague where he applied for a Japanese visa | 14:02 | |
| rloo | dtantsur: here's hoping! | 14:03 |
| dtantsur | :) | 14:03 |
| lucasagomes | NobodyCam, good morning | 14:05 |
| thiagop | rloo: so, it's just a (good) announcement, right? | 14:06 |
| rloo | thiagop: well, an announcement isn't a 'discussion'. Just wondering what needed to be discussed. | 14:07 |
| rloo | thiagop: also, would like to encourage folks to use the mailing list since we don't tend to communicate much that way. | 14:07 |
| *** jamielennox has quit IRC | 14:08 | |
| thiagop | rloo: should I move the appointment on the meeting to announcements (and also send it on the ML)? | 14:08 |
| rloo | thiagop: no, please don't move to announcements. There is a 'OneDriver' in the subteam report -- why don't you mention it there? (and/or mailing list) | 14:09 |
| rloo | thiagop: I think there is a OneDriver in subteam report. checking. | 14:09 |
| rloo | thiagop: yup. https://etherpad.openstack.org/p/IronicWhiteBoard | 14:10 |
| *** jamielennox|away has joined #openstack-ironic | 14:10 | |
| *** jamielennox|away is now known as jamielennox | 14:11 | |
| *** marzif has quit IRC | 14:11 | |
| *** irenab has joined #openstack-ironic | 14:11 | |
| thiagop | rloo: done. Removing the topic from the open discussion section... | 14:11 |
| rloo | thx thiagop! | 14:12 |
| *** marzif has joined #openstack-ironic | 14:12 | |
| thiagop | rloo: not at all, thanks for the directions here | 14:12 |
| rloo | thiagop: yw | 14:13 |
| *** zhenguo has quit IRC | 14:17 | |
| *** puranamr has quit IRC | 14:19 | |
| *** max_lobur has quit IRC | 14:24 | |
| *** shoutm has quit IRC | 14:24 | |
| *** yog_ has quit IRC | 14:25 | |
| NobodyCam | :) | 14:27 |
| *** sambetts_ has joined #openstack-ironic | 14:27 | |
| *** shoutm has joined #openstack-ironic | 14:30 | |
| thrash | dtantsur: lucasagomes got a question for you about the osc plugin... | 14:34 |
| thrash | I'm working on adding 'openstack baremetal set --maintenance'... | 14:34 |
| dtantsur | yep | 14:35 |
| thrash | It would be cleaner (and less code) to have the command line parser take care of the 'reason' only for maintenance on | 14:35 |
| thrash | what it should be is just '--maintenance' and something like '--active' (that's the only thing I can come up with) | 14:35 |
| thrash | myu question is... what should '--active' be? :) | 14:35 |
| dtantsur | active is a valid node state, lets not introduce confusion here | 14:36 |
| thrash | ok | 14:36 |
| thrash | so what would be the opposite of --maintenance? | 14:36 |
| dtantsur | I would think about something like "openstack baremetal set maintenance --reason=" and openstack baremetal set maintenance --off" | 14:36 |
| dtantsur | thrash, ^^ | 14:36 |
| dtantsur | I don't get why "maintenance" is a flag btw | 14:36 |
| thrash | dtantsur: so that it fits in the baremetal set command. | 14:37 |
| dtantsur | what is a baremetal set command? :) | 14:37 |
| thrash | so, you could actually do a bunch of updates to a node in a single command. | 14:37 |
| dtantsur | how do you define it? | 14:37 |
| dtantsur | hmmm | 14:37 |
| thrash | openstack baremetal set --provision-state manage --maintenance $NODE | 14:38 |
| thrash | trying to follow conventions of OSC as much as possible. | 14:39 |
| dtantsur | thrash, maybe it's only me, but I'd prefer provision state verbs to become top level actions... like openstack baremetal manage | 14:39 |
| * lucasagomes doesn't know much about the OSC conventions | 14:40 | |
| dtantsur | for example "set --provision-state manage --maintenance" does not make much sense: maintenance mode means that you probably don't want any provisioning actions on it | 14:41 |
| lucasagomes | but yeah def not using --active for maintenance cause that will be confusing | 14:41 |
| *** bapalm_ has quit IRC | 14:41 | |
| dtantsur | thrash, anyway, we don't have an established antonym to "maintenance" here in ironic | 14:41 |
| thrash | is disable/enable descriptive enough if there is help text? | 14:41 |
| thrash | disable == maintenance mode, enable == !maintenance mode | 14:43 |
| dtantsur | maybe... but then we need to change how this field is shown as well | 14:43 |
| thrash | dtantsur: tangent... I do like the idea of 'openstack baremetal [active|delete|rebuild|inspect|provide|manage]' | 14:44 |
| *** irenab has quit IRC | 14:44 | |
| dtantsur | thrash, maybe worth discussing at the meeting (in ~ 2:20 IIRC) | 14:44 |
| dtantsur | or is it too late for you? | 14:44 |
| thrash | meaning in 2 hrs? | 14:44 |
| dtantsur | yep, sorry | 14:44 |
| thrash | hehe | 14:45 |
| thrash | definitely not too late, as that's noon for me. :) | 14:45 |
| *** irenab has joined #openstack-ironic | 14:45 | |
| dtantsur | cool, you can add your topic to https://wiki.openstack.org/wiki/Meetings/Ironic so that people can think about it in advance | 14:45 |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Enhance Power Interface for Soft Power Off and Inject NMI https://review.openstack.org/207897 | 14:45 |
| thrash | dtantsur: ack | 14:45 |
| *** bapalm- has joined #openstack-ironic | 14:45 | |
| *** zhenguo has joined #openstack-ironic | 14:46 | |
| *** dprince has quit IRC | 14:50 | |
| *** dprince has joined #openstack-ironic | 14:50 | |
| rloo | dtantsur, thrash: might I suggest bringing it up in the mailing list first? | 14:51 |
| rloo | dtantsur, thrash: unless it is time sensitive? | 14:51 |
| *** mtanino has joined #openstack-ironic | 14:52 | |
| rloo | dtantsur, thrash: we are bad at this, but the idea was to add topics to our meeting 2 days in advance | 14:52 |
| thrash | rloo: I'm fine with that. Not that time sensitive. | 14:52 |
| rloo | dtantsur, thrash: and I'd like folks who cannot attend meetings, to be able to voice their opinions | 14:52 |
| *** achanda has joined #openstack-ironic | 14:52 | |
| rloo | thx thrash! | 14:52 |
| dtantsur | rloo, good point! | 14:52 |
| thrash | rloo: no prob | 14:52 |
| rloo | dtantsur, thrash: in this case, I suspect it isn't too much of a problem but I do feel uncomfortable when we vote on something in the meeting, when there wasn't much of a heads-up that the something was going to be discussed/voted on. | 14:54 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic: Added CORS support middleware to Ironic https://review.openstack.org/199769 | 14:54 |
| *** shoutm has quit IRC | 15:00 | |
| jlvillal | NobodyCam, good morning | 15:00 |
| jlvillal | Good morning Ironic | 15:00 |
| thiagop | morning jlvillal | 15:01 |
| *** jcoufal has quit IRC | 15:01 | |
| *** r3mu5 has joined #openstack-ironic | 15:02 | |
| NobodyCam | mornign jlvillal :) | 15:02 |
| cinerama | morning ironicists! | 15:02 |
| *** achanda has quit IRC | 15:02 | |
| thiagop | morning cinerama | 15:03 |
| openstackgerrit | John Trowbridge proposed openstack/ironic-inspector: Store and expose introspection data https://review.openstack.org/213159 | 15:04 |
| dtantsur | jlvillal, cinerama, morning | 15:04 |
| *** mgoddard has quit IRC | 15:07 | |
| thrash | dtantsur: what about --production? | 15:07 |
| *** mgoddard has joined #openstack-ironic | 15:08 | |
| thrash | lucasagomes: ^^^ | 15:08 |
| dtantsur | also an option, though we never used anything like that :) | 15:08 |
| lucasagomes | thrash, --productor for maintenance ? | 15:09 |
| lucasagomes | production* | 15:09 |
| lucasagomes | I don't know hmm why we can't just use maintenance ? | 15:09 |
| thrash | lucasagomes: I'm trying to get rid of the explicit check for --maintenance off with a reason | 15:10 |
| thrash | the other option may be to just create a custom Action class. | 15:13 |
| lucasagomes | thrash, right hmm | 15:13 |
| thrash | but that's more code, which I am trying to avoid. :) | 15:13 |
| lucasagomes | and if instead of "set" when you wanna remove the maintenance you use "unset" ? | 15:13 |
| thrash | less code == fewer bugs | 15:13 |
| * lucasagomes is really bad naming things | 15:13 | |
| thrash | lucasagomes: actually... I think you are onto something there! | 15:13 |
| thrash | bahaha | 15:14 |
| thrash | that's it! | 15:14 |
| lucasagomes | :-) | 15:14 |
| *** amotoki has quit IRC | 15:16 | |
| * jlvillal is still trying to wake up... | 15:17 | |
| dtantsur | lucasagomes, may I ask you review IPA patch needed for inspection please? https://review.openstack.org/#/c/209889/ | 15:18 |
| dtantsur | ifarkas, I assume you're working on making wsman headers available for dracclient gate, right? | 15:19 |
| dtantsur | ifarkas, anyway, could you please update DRAC status on the whiteboard? https://etherpad.openstack.org/p/IronicWhiteBoard | 15:20 |
| ifarkas | dtantsur, not right now, but yes | 15:20 |
| dtantsur | (see "Subteam status reports") | 15:20 |
| ifarkas | sure | 15:20 |
| lucasagomes | dtantsur, sure will take a look | 15:20 |
| dtantsur | folks (especially devananda), do you think we should clean up https://etherpad.openstack.org/p/IronicWhiteBoard ? I see "Kilo ToDo List" in the top of the page :D | 15:22 |
| NobodyCam | dtantsur: lol... ya we prob should | 15:23 |
| NobodyCam | :-p | 15:23 |
| dtantsur | I'm asking just to make sure nobody want to leave it for history... | 15:23 |
| dtantsur | I can move it to "Old Stuff" section in the end though :) | 15:23 |
| NobodyCam | ya that be the first thing... we can ask in the meeting too | 15:24 |
| *** killer_prince has quit IRC | 15:30 | |
| *** zhenguo has quit IRC | 15:31 | |
| *** getvasanth has joined #openstack-ironic | 15:32 | |
| *** Sukhdev has joined #openstack-ironic | 15:33 | |
| getvasanth | Hi All, when i try to connect a baremetal, i get a "InvalidToken: Token authorization failed" error, am i missing something? using packstack | 15:34 |
| getvasanth | any pointers will help | 15:34 |
| *** jistr has quit IRC | 15:35 | |
| *** david-ly_ is now known as david-lyle | 15:38 | |
| *** jistr has joined #openstack-ironic | 15:38 | |
| *** trown is now known as trown|lunch | 15:40 | |
| dtantsur | so, I'm slowly cleaning it up... | 15:41 |
| dtantsur | does anyone still use https://etherpad.openstack.org/p/ipa-todos ? jroll, JoshNang? | 15:42 |
| *** e0ne has quit IRC | 15:42 | |
| dtantsur | also, NobodyCam, are you still working on https://github.com/NoBodyCam/PixieBootsIrcBot ? | 15:43 |
| *** e0ne has joined #openstack-ironic | 15:44 | |
| NobodyCam | dtantsur: oh I kinda stopped... but really should pick that back up | 15:46 |
| *** kenb has joined #openstack-ironic | 15:48 | |
| thiagop | getvasanth: using CLI? | 15:48 |
| JoshNang | dtantsur: i haven't looked at it in at least 6 monnths | 15:51 |
| *** getvasanth has quit IRC | 15:51 | |
| dtantsur | JoshNang, heh, I see, so it's a part of history already | 15:51 |
| JoshNang | "... is now in tree!" heh | 15:52 |
| JoshNang | long running agents and network isolation are still in progress, but have specs up, so \o/. the rest looks done | 15:52 |
| dtantsur | ok, I'm done cleaning up the whiteboard, hope devananda won't kill me :D | 15:53 |
| *** praneshp has joined #openstack-ironic | 15:54 | |
| *** kenb has quit IRC | 15:54 | |
| *** irenab has quit IRC | 15:55 | |
| JoshNang | also, good morning, y'all! | 15:56 |
| *** lazy_prince has joined #openstack-ironic | 15:56 | |
| openstackgerrit | Tomoki Sekiyama proposed openstack/ironic: Add REST API for volume connector operation https://review.openstack.org/214586 | 15:56 |
| rloo | dtantsur: wrt the kilo to-dos -- I suspect we should check and any that aren't done yet, do in liberty or decide not to do or whatever | 15:56 |
| * dtantsur is checking | 15:57 | |
| rloo | dtantsur: and thx for cleaning up! | 15:57 |
| dtantsur | checked, these TODO's are way outdated. np :) | 15:57 |
| rloo | dtantsur: so we can delete them from the history section? | 15:58 |
| dtantsur | I left it for someone else to delete (kind of review of my changes) | 15:58 |
| *** irenab has joined #openstack-ironic | 15:59 | |
| rloo | dtantsur: i don't know why we use etherpad (or maybe we didn't set it up properly) if we can't get history from it | 15:59 |
| dtantsur | hmm, good point | 15:59 |
| rloo | dtantsur: seems like the liberty summit ideas etc are still useful? to review what we were/are doing in liberty? | 16:00 |
| dtantsur | lemme check | 16:00 |
| dtantsur | rloo, fair enough, moved them back to TODO's | 16:01 |
| rloo | dtantsur: oh, this is a nice diagram. why didn't we put it in our docs? https://docs.google.com/drawings/d/1azAWh0ZfhDqEUsC14ZEBawbnAmdQ2_yl3CfOdDbPvOk/edit | 16:02 |
| dtantsur | rloo, no idea, but with switch to IPA by default it's probably of not too much value... | 16:03 |
| rloo | dtantsur: true. and updated one with ipa would be nice :) | 16:03 |
| dtantsur | TheJulia, you might want to review https://review.openstack.org/#/c/216209/1/reference/projects.yaml | 16:05 |
| *** links has quit IRC | 16:08 | |
| TheJulia | dtantsur: thanks | 16:08 |
| *** ukalifon1 has quit IRC | 16:08 | |
| dtantsur | lucasagomes, thanks for reviewing https://review.openstack.org/#/c/209889/, any particular reasons not to approve it? | 16:11 |
| lucasagomes | dtantsur, not really, (perhaps I didn't see another +2 on it) | 16:12 |
| lucasagomes | dtantsur, done | 16:12 |
| *** jistr has quit IRC | 16:13 | |
| dtantsur | thanks a lot! | 16:13 |
| *** mitchjameson has joined #openstack-ironic | 16:14 | |
| *** puranamr has joined #openstack-ironic | 16:16 | |
| *** praneshp has quit IRC | 16:17 | |
| *** marzif has quit IRC | 16:18 | |
| JoshNang | woo! | 16:20 |
| *** ifarkas has quit IRC | 16:21 | |
| *** dprince has quit IRC | 16:21 | |
| *** jcoufal has joined #openstack-ironic | 16:29 | |
| *** romcheg has quit IRC | 16:34 | |
| devananda | g'morning, all | 16:35 |
| NobodyCam | good morning devananda | 16:36 |
| *** Nisha_away has joined #openstack-ironic | 16:38 | |
| *** absubram has joined #openstack-ironic | 16:42 | |
| devananda | I feel like I have *SO* many things to do in the next three days. Ugh. | 16:42 |
| NobodyCam | :( | 16:43 |
| natorious | morning! | 16:43 |
| NobodyCam | thats what you get for taking a vacation to burn things | 16:43 |
| NobodyCam | hehehehe | 16:43 |
| NobodyCam | mornign natorious | 16:43 |
| *** wendar_ is now known as wendar | 16:45 | |
| *** Nisha_away is now known as Nisha_ | 16:46 | |
| *** derekh has quit IRC | 16:49 | |
| thiagop | morning natorious | 16:49 |
| krotscheck | I need some love on ironic-webclient reviews. Anyone have some time to look? https://review.openstack.org/#/q/status:open+project:openstack/ironic-webclient,n,z | 16:49 |
| natorious | o/ | 16:49 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/ironic: Introduce VolumeConnector object https://review.openstack.org/214584 | 16:50 |
| natorious | krotscheck: I've got some time today | 16:50 |
| krotscheck | natorious: Much obliged, none of it is functionality, I just want to clear my revew queue before crazy baby things happen | 16:51 |
| *** tsekiyama has joined #openstack-ironic | 16:51 | |
| *** trown|lunch is now known as trown | 16:52 | |
| JoshNang | krotscheck: huh, didn't realize you could use ! in .gitignore. that's handy | 16:53 |
| *** stendulker has joined #openstack-ironic | 16:54 | |
| lucasagomes | devananda, morning | 16:55 |
| krotscheck | JoshNang: I randomly ran into it while using a yeoman generator | 16:55 |
| thiagop | 4min | 16:55 |
| JayF | -meeting-3? | 16:56 |
| thiagop | yep | 16:56 |
| lucasagomes | JayF, yup | 16:56 |
| JoshNang | nice. i was just fighting with something a couple days ago that ! will solve. so \o/ | 16:56 |
| *** romainh has left #openstack-ironic | 16:57 | |
| dtantsur | devananda, morning | 16:57 |
| jlvillal | Meeting time :) | 17:00 |
| Sukhdev | lazy_prince: can we chat here for a few minutes? | 17:01 |
| lazy_prince | sure.. | 17:02 |
| Sukhdev | lazy_prince: you mentioned some irregularities - what are you referring to? | 17:02 |
| lazy_prince | umm... the way the binding profile etc needs to be passed to neutron... | 17:02 |
| Sukhdev | lazy_prince: are you tailing about the ones we found during mid-cycle sprint? | 17:02 |
| *** mitchjameson has quit IRC | 17:03 | |
| Sukhdev | lazy_prince: Oh - right - I was concerned that you found something else (in addition to that) | 17:03 |
| Sukhdev | lazy_prince: I can push another update to the spec to clarify it - moreover, I saw your patch, it seems to be doing the right thing based upon our discussion during mid-cycle sprint | 17:04 |
| lazy_prince | nope.. but that was critical stuff.. and that was not mentioned properly.. so i am insisting on getting docs ready early on to make sure not everyone ends up reworking.. | 17:04 |
| NobodyCam | we're in out weekly meeting in openstack-meeting-3 | 17:05 |
| NobodyCam | our == our | 17:05 |
| lazy_prince | sure.. feel free and i will review.. | 17:05 |
| Sukhdev | lazy_prince: got it and understood - I got concerned that you may have discovered something new :-):-) | 17:05 |
| *** romcheg has joined #openstack-ironic | 17:06 | |
| Sukhdev | lazy_prince: thanks for clarification | 17:06 |
| jlvillal | dtantsur: Bug dashboard link? I probably have it bookmarked somewhere.... | 17:10 |
| dtantsur | jlvillal, ironic-divius.rhcloud.com | 17:10 |
| *** marzif has joined #openstack-ironic | 17:12 | |
| *** Marga_ has quit IRC | 17:13 | |
| greghaynes | TheJulia: Are you aware of anyone using the ipv4_network_mac config option in bifrost | 17:16 |
| greghaynes | TheJulia: There are some fail cases im seeing with it while I port the network_metadata deal to a module | 17:16 |
| greghaynes | er, ipv4_interface_mac | 17:17 |
| *** dprince has joined #openstack-ironic | 17:17 | |
| *** marzif has quit IRC | 17:17 | |
| *** sambetts_ has quit IRC | 17:18 | |
| TheJulia | greghaynes: afaik... not off the top off my head | 17:19 |
| openstackgerrit | John Trowbridge proposed openstack/ironic-inspector: Store and expose introspection data https://review.openstack.org/213159 | 17:21 |
| openstackgerrit | Merged stackforge/pyghmi: Silently ignore broken sensors rather than error https://review.openstack.org/215332 | 17:21 |
| *** Nisha_away has joined #openstack-ironic | 17:22 | |
| *** mgoddard has quit IRC | 17:24 | |
| *** Nisha_ has quit IRC | 17:24 | |
| *** praneshp has joined #openstack-ironic | 17:25 | |
| *** mgoddard has joined #openstack-ironic | 17:25 | |
| *** lazy_prince has quit IRC | 17:32 | |
| *** achanda has joined #openstack-ironic | 17:33 | |
| *** Sukhdev has quit IRC | 17:33 | |
| *** baoli_ has quit IRC | 17:40 | |
| *** baoli has joined #openstack-ironic | 17:44 | |
| openstackgerrit | Juliana Motira proposed stackforge/pyghmi: Changing Lenovo custom get_leds method https://review.openstack.org/216355 | 17:48 |
| *** jcoufal has quit IRC | 17:52 | |
| *** Sukhdev has joined #openstack-ironic | 17:52 | |
| openstackgerrit | Merged stackforge/pyghmi: Changing Lenovo custom get_leds method https://review.openstack.org/216355 | 17:55 |
| lucasagomes | devananda, enjoy the holidays! | 18:00 |
| dtantsur | ++ | 18:00 |
| lucasagomes | tho you will be around until wednesday right? | 18:00 |
| trown | have fun burninating all the people | 18:01 |
| rloo | devananda is around for two days still, lets make sure he has stuff to do :) | 18:01 |
| *** Sukhdev has quit IRC | 18:03 | |
| *** pelix has quit IRC | 18:04 | |
| lucasagomes | rloo, ++ :-) | 18:06 |
| lucasagomes | folks I will call it a day as well | 18:06 |
| lucasagomes | have a great night everyone! | 18:06 |
| devananda | g'night, lucasagomes ! | 18:06 |
| lucasagomes | g'night! | 18:06 |
| *** lucasagomes is now known as lucas-dinner | 18:06 | |
| dtantsur | g'night everyone as well | 18:06 |
| *** baoli has quit IRC | 18:06 | |
| *** dtantsur is now known as dtantsur|afk | 18:07 | |
| devananda | g'night dtantsur|afk as well :) | 18:07 |
| *** Nisha_away is now known as Nisha_ | 18:08 | |
| thiagop | good night dtantsur|afk lucas-dinner | 18:08 |
| NobodyCam | night lucas-dinner dtantsur|afk | 18:10 |
| NobodyCam | brb | 18:10 |
| *** baoli has joined #openstack-ironic | 18:16 | |
| *** romcheg has quit IRC | 18:19 | |
| *** alexpilotti has quit IRC | 18:19 | |
| *** bapalm- is now known as bapalm_ | 18:19 | |
| *** baoli has quit IRC | 18:20 | |
| rloo | hey NobodyCam, you fixed this bug, right? https://bugs.launchpad.net/ironic/+bug/1428722 | 18:20 |
| openstack | Launchpad bug 1428722 in Ironic "Ironic drivers that utilize temp files should verify they can actually create them" [Low,In progress] - Assigned to Chris Krelle (nobodycam) | 18:20 |
| *** Nisha_ has quit IRC | 18:21 | |
| *** Nisha_ has joined #openstack-ironic | 18:21 | |
| *** baoli has joined #openstack-ironic | 18:25 | |
| *** Nisha_away has joined #openstack-ironic | 18:28 | |
| openstackgerrit | greghaynes proposed openstack/bifrost: Generate network_info json using a module https://review.openstack.org/216372 | 18:29 |
| NobodyCam | rloo: I thought so.. unless it was reopened | 18:30 |
| rloo | NobodyCam: the only thing I can think of is that your patch had 'Partial-Bug'. Not sure why though. | 18:30 |
| *** stendulker has quit IRC | 18:31 | |
| *** romcheg has joined #openstack-ironic | 18:31 | |
| *** Nisha_ has quit IRC | 18:32 | |
| openstackgerrit | greghaynes proposed openstack/bifrost: Generate network_info json using a module https://review.openstack.org/216372 | 18:32 |
| *** Nisha_away is now known as Nisha_ | 18:33 | |
| NobodyCam | rloo: it got changed several time I think as the patch worked it way to landing | 18:33 |
| rloo | NobodyCam: oh, maybe that's why. You want to mark it as Fixed? | 18:34 |
| NobodyCam | will do | 18:35 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: Add unit testing for encoding.py https://review.openstack.org/215854 | 18:35 |
| jlvillal | JoshNang: ^^ :) | 18:35 |
| NobodyCam | rloo: done and noted on the bug | 18:36 |
| *** absubram has quit IRC | 18:36 | |
| rloo | thx NobodyCam | 18:36 |
| NobodyCam | :) | 18:36 |
| *** Marga_ has joined #openstack-ironic | 18:44 | |
| *** ionutbalutoiu has joined #openstack-ironic | 18:45 | |
| *** alexpilotti has joined #openstack-ironic | 18:45 | |
| *** Marga_ has quit IRC | 18:45 | |
| *** ionutbalutoiu has quit IRC | 18:45 | |
| *** Marga_ has joined #openstack-ironic | 18:46 | |
| *** ionutbalutoiu has joined #openstack-ironic | 18:46 | |
| *** ionutbalutoiu has quit IRC | 18:46 | |
| *** ionutbalutoiu has joined #openstack-ironic | 18:47 | |
| *** ionutbalutoiu has quit IRC | 18:47 | |
| *** ionutbalutoiu has joined #openstack-ironic | 18:48 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Extend hardware manager with data needed for inspector https://review.openstack.org/209889 | 18:50 |
| devananda | Hey folks -- I'm coordinating with dhellmann right now to switch us to being able to manage our own releases | 18:50 |
| devananda | please DO NOT APPROVE ANYTHING until this switch is done. Thanks! | 18:51 |
| devananda | rloo, jroll, dtantsur|afk, lucas-dinner, NobodyCam, Haomeng|2, yuriyz, ram^D^D^D -- see note above -- please don't approve anything right now. | 18:51 |
| * JayF merges al^W none of the things | 18:53 | |
| NobodyCam | ack | 18:58 |
| NobodyCam | Ty devananda :) | 18:58 |
| NobodyCam | brb... checking on kittys | 18:59 |
| rloo | hmm, would be nice if it took say 2 weeks to do the switch ;) | 19:04 |
| *** e0ne has quit IRC | 19:04 | |
| *** puranamr has quit IRC | 19:08 | |
| *** puranamr has joined #openstack-ironic | 19:08 | |
| *** puranamr has quit IRC | 19:09 | |
| *** puranamr has joined #openstack-ironic | 19:09 | |
| devananda | rloo: if it did, or did not? | 19:12 |
| rloo | devananda: did. No one to bug us about reviewing etc :D | 19:13 |
| devananda | hehe | 19:13 |
| krotscheck | What is ramesh's IRC handle? | 19:15 |
| thiagop | but, if you'd like to give some feedback without approving rloo, we'd appreciate your thoughts on the OneView driver | 19:15 |
| thiagop | https://review.openstack.org/#/c/191822/ | 19:15 |
| openstackgerrit | Doug Hellmann proposed openstack/ironic: Switch to post-versioning https://review.openstack.org/216388 | 19:15 |
| thiagop | krotscheck: rameshg87 | 19:16 |
| rloo | thiagop: i want to focus on getting the HIGH priority stuff in before the liberty cutoff. | 19:16 |
| thiagop | rloo: ok... | 19:16 |
| krotscheck | thiagop: Thanks. He does not appear to be here :/ | 19:16 |
| rloo | thiagop: I know. If I was a faster reviewer, I'd like to get lots of stuff in. There is the openstack client stuff that started in the client, in kilo i think. i don't think that'll make it in liberty either. | 19:17 |
| thiagop | krotscheck: I think he's in Indian timezone, probably sleeping | 19:18 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Improve comparison operators for api/controllers/base.py https://review.openstack.org/213890 | 19:18 |
| thiagop | rloo: but you're very thorough | 19:19 |
| thiagop | rloo: that's good | 19:20 |
| thiagop | :) | 19:20 |
| rloo | thiagop: that's why it takes me so long. | 19:20 |
| rloo | thiagop: I know you've been helping/reviewing. If everyone did more reviews and caught stuff, it would make it faster for me to review. | 19:21 |
| thiagop | rloo: sure. Today I'm on "end-of-sprint-report" business, would like to take some time on reviews later in (my) afternoon | 19:22 |
| *** afaranha has joined #openstack-ironic | 19:23 | |
| rloo | thx thiagop | 19:25 |
| jlvillal | rloo: +1 on more reviewers. I'm trying to review. I know I should do more. It takes time to do quality reviews and hopefully helpful reviews. | 19:25 |
| rloo | jlvillal: you're doing a great job. Now how about 'dem tests :) | 19:26 |
| jlvillal | I know! Got to get that done too. | 19:26 |
| *** davideagnello has joined #openstack-ironic | 19:28 | |
| *** MattMan has left #openstack-ironic | 19:32 | |
| openstackgerrit | Lucio Seki proposed stackforge/pyghmi: Add RAID drive and controller to ThinkServer inventory https://review.openstack.org/216398 | 19:42 |
| SpamapS | TheJulia: Hey, have you thought about a 'redeploy' playbook? | 19:44 |
| SpamapS | TheJulia: I find myself doing a lot of awking to delete all the nodes and re-try | 19:45 |
| TheJulia | SpamapS: no, ricardo was going to work in that direction, the underlying shade interface supports rebuild :) | 19:45 |
| SpamapS | TheJulia: rebuild isn't always what I want tho. | 19:46 |
| TheJulia | delete my active instances and then redeploy them all? | 19:46 |
| SpamapS | TheJulia: no, just delete all the failed ones. :) | 19:47 |
| TheJulia | take a look at the one example playbook :) | 19:48 |
| SpamapS | oh right that looks like could be morphed into what I want. :) | 19:49 |
| devananda | hi folks! I'm going through aaalll the approved specs to sync the status with LP before we tag 4.0.0 | 19:50 |
| TheJulia | SpamapS: Please do feel free to submit your creation back :) | 19:51 |
| devananda | I may need a little help in determining whether something is done | 19:51 |
| devananda | JoshNang: https://blueprints.launchpad.net/ironic/+spec/deprecate-bash-ramdisk | 19:51 |
| devananda | JoshNang: looks like that should have been more than just a doc patch, but nothing else was done? I know it's asigned to lucas-dinner but he's afk and I think you tracked it? | 19:52 |
| *** r3mu5 has quit IRC | 19:54 | |
| rloo | devananda: wrt ^^ I think this needs to be approved: https://review.openstack.org/#/c/209079/ | 19:55 |
| *** afaranha has quit IRC | 19:55 | |
| devananda | ahh, ty | 19:55 |
| rloo | devananda: and according to the priorities spreadsheet, it sez it needs ironic code too | 19:55 |
| rloo | devananda: although i thought we merged that ironic code. | 19:56 |
| devananda | i couldn't find anything besides a doc update on our side yet | 19:56 |
| devananda | lucas-dinner, rloo: https://blueprints.launchpad.net/ironic/+spec/ipxe-dynamic-config looks completed as well | 19:57 |
| *** nicodemos has quit IRC | 19:57 | |
| JoshNang | devananda: sorry was lunching | 19:59 |
| rloo | devananda: wrt ipxe-dynamic-config, yeah, looks like done, but i'd have to review the spec to confirm. you need the answers today, or can it wait for lucas-dinner tomorrow? | 20:00 |
| devananda | i'm working with dhellmann now to get 4.0.0 tagged | 20:00 |
| devananda | i'd rather not wait till tomorrow | 20:01 |
| *** thiagop has quit IRC | 20:01 | |
| JoshNang | devananda: i think the only think remaining is switching the tempest tests to use IPA | 20:01 |
| devananda | if we do wait, and something else comes up / goes wrong, then we'll be just about out of time | 20:01 |
| rloo | devananda: i'll look. give me a few minute. | 20:02 |
| devananda | JoshNang: ah right. well, i'll leave that open for now then, since both external deps are unfinished | 20:02 |
| devananda | and there's not really a change inside of ironic (aside from docs) | 20:02 |
| JoshNang | right | 20:03 |
| *** Marga_ has quit IRC | 20:03 | |
| devananda | https://blueprints.launchpad.net/ironic/+spec/irmc-virtualmedia-deploy-driver appears to be done | 20:03 |
| devananda | doc patch even landed :) | 20:03 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/ironic: Introduce new RPCs to support volume connector operation https://review.openstack.org/214585 | 20:03 |
| *** albertoffb has quit IRC | 20:04 | |
| devananda | yea, naohirot's only remaining open patches are about NMI now | 20:05 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/ironic: Add REST API for volume connector operation https://review.openstack.org/214586 | 20:05 |
| *** Marga_ has joined #openstack-ironic | 20:05 | |
| rloo | devananda: I don't think the ipxe_dynamic_config is done. The spec mentions an ipxe_config vendor passthru, but that isn't in the code. yet. | 20:05 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Create a versions.py file https://review.openstack.org/213856 | 20:07 |
| devananda | rloo: oh :( | 20:07 |
| *** achanda has quit IRC | 20:07 | |
| devananda | http://specs.openstack.org/openstack/ironic-specs/specs/approved/new-boot-interface.html is also only one third done | 20:07 |
| * devananda dislikes half-implemented specs | 20:09 | |
| rloo | devananda: wrt boot interface, the agent refactoring is almost done. but yeah, not finished yet. | 20:10 |
| *** thiagop has joined #openstack-ironic | 20:10 | |
| devananda | yea | 20:12 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Stop providing 'message' property in IronicException https://review.openstack.org/215362 | 20:16 |
| devananda | https://blueprints.launchpad.net/ironic/+spec/uefi-secure-boot-pxe-ilo looks done | 20:16 |
| *** baoli has quit IRC | 20:17 | |
| *** puranamr has quit IRC | 20:18 | |
| *** puranamr has joined #openstack-ironic | 20:18 | |
| *** puranamr has quit IRC | 20:19 | |
| *** puranamr has joined #openstack-ironic | 20:19 | |
| *** absubram has joined #openstack-ironic | 20:21 | |
| devananda | Nisha_: around? do you know if https://blueprints.launchpad.net/ironic/+spec/uefi-secure-boot-pxe-ilo is completed? | 20:22 |
| Nisha_ | devananda, yes | 20:22 |
| devananda | ok - thanks | 20:22 |
| Nisha_ | devananda, request you to add blueprint https://blueprints.launchpad.net/openstack/?searchtext=remove-swift-dependency-for-ilo-drivers to ironic list | 20:23 |
| Nisha_ | the spec is merged for it | 20:23 |
| Nisha_ | and one code patch also merged | 20:23 |
| Nisha_ | spec review link was https://review.openstack.org/193478 | 20:23 |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic-specs: Update specs which have been completed https://review.openstack.org/216412 | 20:24 |
| devananda | Nisha_: I did that this morning | 20:24 |
| Nisha_ | devananda, thanks | 20:25 |
| *** Sukhdev has joined #openstack-ironic | 20:25 | |
| lucas-dinner | devananda, rloo hi oh the ipxe dynamic config is not done yet | 20:28 |
| devananda | lucas-dinner: thanks for confirming :) | 20:28 |
| lucas-dinner | devananda, rloo I had to leave it on hold to work on other things (node's stuck in some states, etc...) | 20:29 |
| lucas-dinner | but I will get back to finish it up soonish | 20:29 |
| devananda | this should now look like a complete picture of what we've completed in this cycle | 20:29 |
| devananda | https://launchpad.net/ironic/+milestone/next-liberty | 20:29 |
| lucas-dinner | all the plumbing work is already done | 20:29 |
| devananda | along with the output of my spec change above | 20:29 |
| devananda | from which we'll generate the 4.0.0 release notes | 20:29 |
| lucas-dinner | devananda, and re the deprecation of the bash ramdisk, yeah I'm waiting on some patches on disk image bulder and tripleo-ci | 20:29 |
| lucas-dinner | devananda, thanks, will take a look | 20:30 |
| *** jerrygb has joined #openstack-ironic | 20:33 | |
| *** jerrygb is now known as test0rz | 20:34 | |
| *** test0rz is now known as asd112z | 20:34 | |
| *** krtaylor has quit IRC | 20:35 | |
| rloo | thx devananda. wrt enroll, there are things mentioned in the spec (related to client, nova) that aren't done yet but I think everything related to ironic is done. | 20:35 |
| devananda | rloo: yea. some of the "do this thing later" things are not do-able until "M" release | 20:35 |
| rloo | devananda: are you also going to want to know which bugs are fixed? | 20:37 |
| devananda | rloo: nope. that's covered | 20:39 |
| devananda | rloo: just making sure that there are no other finished BP's at this point | 20:40 |
| rloo | devananda: good | 20:40 |
| *** achanda has joined #openstack-ironic | 20:40 | |
| *** achanda has quit IRC | 20:41 | |
| *** achanda has joined #openstack-ironic | 20:41 | |
| Nisha_ | devananda, is there any plan to have one more release in liberty for ironic? | 20:43 |
| devananda | Nisha_: definitely | 20:44 |
| devananda | this isn't the final code. but it is to be seen how much gets in | 20:45 |
| devananda | we just need to switch to post-versioning before then, and today is when we're doing that | 20:45 |
| Nisha_ | devananda, some of the specs which are approved but code is not merged yet, will they be considered for liberty ? | 20:46 |
| devananda | Nisha_: preference will be given to those that are already in flight over entirely new work | 20:47 |
| *** krtaylor has joined #openstack-ironic | 20:48 | |
| Nisha_ | devananda, so we can expect similar milestone releases as kilo-1, kilo-2 etc in liberty? I am asking because openstack liberty timeline is now near to closure... | 20:49 |
| *** trown is now known as trown|outttypeww | 20:56 | |
| devananda | Nisha_: nope | 21:01 |
| devananda | Nisha_: we moved away from milestone releases | 21:02 |
| devananda | and did not have an l1, l2 release | 21:02 |
| Nisha_ | devananda, i know but i am not sure what ironic in liberty will have then? thats why the ques....will it contain only those features which are complete by today? or in other words when we tag it again next time, will it be for liberty or M release? | 21:05 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: Fix Failing unit test under py34 https://review.openstack.org/216426 | 21:07 |
| devananda | Nisha_: we should be tagging at least one more time before liberty final | 21:09 |
| devananda | Nisha_: so you still have time for things to land :) | 21:09 |
| Nisha_ | devananda, ok :) that answers my ques :) | 21:10 |
| *** ionutbalutoiu has quit IRC | 21:12 | |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Add node-create example to enroll section of install-guide https://review.openstack.org/209251 | 21:14 |
| rloo | well, we may not have completed a lot of specs yet, but there sure seem to be quite a few bugs fixed... | 21:18 |
| *** tsbot_ has quit IRC | 21:19 | |
| *** Nisha_ has quit IRC | 21:19 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic-specs: Update specs which have been completed https://review.openstack.org/216412 | 21:20 |
| devananda | that ^ should now pass tests, and if it does, I'm going to self-approve the house keeping | 21:20 |
| devananda | because 4.0.0 is now out the door and that's what I wrote on launchpad :) | 21:21 |
| *** jcoufal has joined #openstack-ironic | 21:21 | |
| JoshNang | \o/ | 21:21 |
| *** absubram has quit IRC | 21:23 | |
| jlvillal | devananda: does that mean cores can start merging patches again? | 21:24 |
| openstackgerrit | greghaynes proposed openstack/bifrost: Support setting vlan in config drive https://review.openstack.org/216431 | 21:26 |
| openstackgerrit | greghaynes proposed openstack/bifrost: Support setting vlan in config drive https://review.openstack.org/216431 | 21:27 |
| thiagop | folks, I'm calling it a day | 21:36 |
| devananda | jlvillal: not quite. waiting for a merge to happen on https://review.openstack.org/#/c/216388/1 | 21:37 |
| devananda | I suppose I should have said "4.0.0 is in the doorway right now", lol | 21:37 |
| jlvillal | devananda: Okay thanks. I saw you say 4.0.0 is out the door, so I got excited :) | 21:37 |
| devananda | because it's not actually out until after that patch finishes post-merge jobs | 21:37 |
| thiagop | once again, if you can spend some time taking a look at the OneView driver, I'll send you some tapioca along with the ones promised to lucas-dinner on the summit: https://review.openstack.org/#/c/191822/ | 21:38 |
| thiagop | ;) | 21:38 |
| thiagop | good night everyone | 21:38 |
| *** thiagop has quit IRC | 21:39 | |
| openstackgerrit | Merged openstack/ironic-specs: Update specs which have been completed https://review.openstack.org/216412 | 21:40 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Create a versions.py file https://review.openstack.org/213856 | 21:45 |
| *** jjohnson2 has quit IRC | 21:52 | |
| *** [1]cdearborn has quit IRC | 22:08 | |
| devananda | ** REMINDER ** please do not approve any patches to openstack/ironic until https://review.openstack.org/#/c/216388/1 merges << rloo NobodyCam jroll dtantsur|afk lucas-dinner yuriyz Haomeng|2 | 22:09 |
| devananda | I need to go afk for two hours for errands, will keep an eye on the gate and check when I get back | 22:09 |
| devananda | oh, also, other projects are fine to keep working on - like the client, IPA, etc. it's just the server project that needs to pause until this merges. Thx! | 22:10 |
| mrda | Good morning ironic | 22:11 |
| devananda | morning, mrda! | 22:12 |
| NobodyCam | morning mrda ... back home now... how was the flight stuff... good I'z hopes | 22:13 |
| mrda | NobodyCam: It was a smooth trip, no hassles, and a good week in TX after WA :) | 22:14 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: WIP: Ansible 2.0 compatibility **DO NOT MERGE** https://review.openstack.org/196832 | 22:15 |
| mrda | I'm still mixed up from timezones, but that'll get sorted this week :) | 22:15 |
| lifeless | jlvillal: ping | 22:20 |
| jlvillal | lifeless: pong | 22:20 |
| lifeless | jlvillal: you say you've seend the error https://review.openstack.org/#/c/216426/ aims at? | 22:20 |
| lifeless | jlvillal: I'd like to figure out why - our code shouldn't ever hit that | 22:20 |
| *** bnemec has joined #openstack-ironic | 22:20 | |
| jlvillal | I do on my Fedora system | 22:20 |
| lifeless | jlvillal: it seems something somewhere is incorrectly using 'unittest.mock' | 22:20 |
| lifeless | jlvillal: (if you look at the backtrace in the bug) | 22:21 |
| lifeless | jlvillal: we use 'mock' | 22:21 |
| jlvillal | One moment | 22:21 |
| lifeless | jlvillal: precisely so we don't need to try to bug-compatible with all previous releases of Python | 22:21 |
| jlvillal | lifeless: http://pastebin.com/YTkQDpF4 | 22:22 |
| jlvillal | Is that useful? | 22:22 |
| lifeless | it confirms you have the same issue | 22:23 |
| lifeless | File "/usr/lib64/python3.4/unittest/mock.py", line 1136, in patched | 22:23 |
| lifeless | return func(*args, **keywargs) | 22:23 |
| lifeless | jlvillal: if you can figure out HTF that is getting called, we can fix the real problem | 22:23 |
| jlvillal | :) | 22:24 |
| lifeless | jlvillal: at the top of the source file | 22:24 |
| lifeless | 'import mock' | 22:24 |
| jlvillal | The file just does 'import mock' | 22:24 |
| lifeless | right | 22:24 |
| jlvillal | Is that not correct? | 22:24 |
| lifeless | thats correct | 22:24 |
| lifeless | thats not unittest.mock | 22:24 |
| lifeless | which is whats in your stacktrace | 22:24 |
| lifeless | mock is unittest.mock from Python 3.6 | 22:24 |
| jlvillal | ironic/tests/__init__.py: import unittest.mock | 22:25 |
| lifeless | unittest.mock in 3.4 has lots of bugs | 22:25 |
| jlvillal | I see | 22:25 |
| *** Sukhdev has quit IRC | 22:25 | |
| jlvillal | And then it makes that the 'mock' to use | 22:25 |
| jlvillal | lifeless: ^^ | 22:25 |
| *** asd112z has quit IRC | 22:25 | |
| lifeless | jlvillal: so yeah thats broken ass | 22:25 |
| jlvillal | lifeless: Suggestions on how to fix appreciated :) | 22:26 |
| jlvillal | I'm guessing don't do that. | 22:27 |
| jlvillal | lifeless: NOTE(viktors): We can't use mock as third-party library in python 3.4 because | 22:27 |
| jlvillal | # of bug https://code.google.com/p/mock/issues/detail?id=234 | 22:27 |
| jlvillal | # so let's use mock from standard library in python 3.x | 22:27 |
| openstackgerrit | lifeless proposed openstack/ironic: Remove broken workaround code for old mock. https://review.openstack.org/216444 | 22:27 |
| openstackgerrit | lifeless proposed openstack/ironic: Remove broken workaround code for old mock. https://review.openstack.org/216444 | 22:29 |
| lifeless | jlvillal: ^ | 22:29 |
| jlvillal | lifeless: Tested and works. Thanks. | 22:30 |
| lifeless | jlvillal: yes, I got tired of everyone accruing cruft in their systems due to carrying local workarounds for fixed bugs, so I got the ack from mfoord to backport cPython unittest.mock in mock | 22:30 |
| lifeless | so anytime anyone ever says 'due to a bug in mock' - if the bug is fixed in cPython, in any release, its fixed in 'mock' | 22:30 |
| jlvillal | lifeless: Thanks. Good to know. Appreciate the fix. | 22:31 |
| lifeless | yeah | 22:31 |
| lifeless | lifes too short to carry workarounds | 22:31 |
| lifeless | fix at ource | 22:31 |
| lifeless | *source* | 22:31 |
| jlvillal | :) | 22:31 |
| *** jcoufal has quit IRC | 22:32 | |
| *** shoutm has joined #openstack-ironic | 22:32 | |
| *** natorious is now known as zz_natorious | 22:46 | |
| * jlvillal is worried that some 3rd party library has broken the Ironic gate :( | 22:48 | |
| *** romcheg has quit IRC | 22:50 | |
| jlvillal | Can anyone else verify that tox fails? 'tox -v -v -epy27 -r' ? | 22:51 |
| jlvillal | This would be in openstack/ironic | 22:56 |
| jlvillal | mrda ^^ | 22:56 |
| lifeless | jlvillal: what are the symptoms ? | 22:58 |
| mrda | jlvillal: I'll look | 23:00 |
| jlvillal | lifeless: mrda: https://bugs.launchpad.net/ironic/+bug/1488270 | 23:01 |
| openstack | Launchpad bug 1488270 in Ironic "py27/py34 unittests failing for ironic" [Undecided,New] | 23:01 |
| jlvillal | lifeless: mrda: TypeError: Error when calling the metaclass bases | 23:01 |
| jlvillal | metaclass conflict: the metaclass of a derived class must be a (non-strict) subclass of the metaclasses of all its bases | 23:01 |
| *** Marga_ has quit IRC | 23:02 | |
| mrda | ahh metaclasses, how I do hate thee | 23:02 |
| jlvillal | lifeless: mrda: And saw it in Jenkins just now: https://review.openstack.org/#/c/213856/ But I get same error on tip without my patch. | 23:03 |
| lifeless | jlvillal: [openstack-announce] [release][oslo] oslo.versionedobjects release 0.8.0 (liberty) | 23:03 |
| *** Marga_ has joined #openstack-ironic | 23:03 | |
| * mrda wonders how this turned up | 23:03 | |
| mrda | oh, never mind | 23:03 |
| lifeless | jlvillal: try downgrading oslo.versionedobjects | 23:03 |
| jlvillal | lifeless: Will try. Thanks | 23:03 |
| *** romcheg has joined #openstack-ironic | 23:03 | |
| mrda | fwiw, I can confirm :( | 23:04 |
| jlvillal | lifeless: That is the issue. | 23:06 |
| jlvillal | Doing: +oslo.versionedobjects>=0.6.0,<0.8.0 | 23:06 |
| jlvillal | Made it work. | 23:06 |
| mrda | huh | 23:06 |
| mrda | good job jlvillal! | 23:06 |
| mrda | Question is, is there a problem with versionedobjects or in how Ironic is using that library? | 23:07 |
| jlvillal | yeah :( | 23:07 |
| *** romcheg has left #openstack-ironic | 23:08 | |
| *** Marga_ has quit IRC | 23:08 | |
| mrda | lifeless: any thoughts on ^^^ | 23:08 |
| jlvillal | mrda: I'm also asking over in #openstack-oslo | 23:09 |
| jlvillal | No responses yet | 23:10 |
| mrda | cool, excellent work jlvillal | 23:10 |
| jlvillal | Thanks. | 23:10 |
| jlvillal | mrda: I hope you had a good trip back home! | 23:11 |
| jlvillal | rloo: FYI: Gate currently broken. | 23:11 |
| mrda | jlvillal: thanks, it all went well, although I'm not yet in my timezone :) | 23:12 |
| jlvillal | Whiteboard updated. | 23:12 |
| mrda | +1 | 23:12 |
| jlvillal | mrda: As in location or body clock? | 23:12 |
| mrda | body clock | 23:12 |
| jlvillal | Yeah, that sucks | 23:12 |
| mrda | I only missed the team meeting today by 30 minutes because of sleep patterns :( | 23:13 |
| jlvillal | Maybe lintan will know how to fix https://bugs.launchpad.net/ironic/+bug/1488270 He is our oslo.versionedobjects master :) | 23:13 |
| openstack | Launchpad bug 1488270 in Ironic "oslo.versionedobjects 0.8.0 release causing py27/py34 unittests to fail in ironic" [Undecided,New] | 23:13 |
| mrda | jlvillal: I was just looking at 1390432 by cooincidence :) | 23:13 |
| lifeless | jlvillal: mrda I don't know if its a bug in o.v, or ironic's use | 23:15 |
| lifeless | jlvillal: its not 1.0 yet, so the API isn't fixed... | 23:15 |
| jlvillal | mrda: I'm sure the fix for this will be simple. I just have no idea what it is :) | 23:15 |
| *** alexpilotti has quit IRC | 23:15 | |
| jlvillal | lifeless: It only seems to be one line in a test case. So not a big breakage. | 23:15 |
| jlvillal | lifeless: Okay two lines | 23:16 |
| jlvillal | But they are identical lines | 23:16 |
| mrda | Once I get through my current crisis I'll take a look | 23:17 |
| jlvillal | mrda: Thanks | 23:17 |
| jlvillal | mrda: lifeless: I wonder if same issue will hit Nova. We copied that test_objects.py from them initially. | 23:19 |
| mrda | jlvillal: oh, have you diffed their version of that file and ours? | 23:20 |
| jlvillal | Not diff, but looked at the change log. It was copied from them originally and made Ironic specific. | 23:21 |
| rloo | hi jlvillal. you got it (the versionedobject stuff) under control? | 23:27 |
| mrda | rloo: we might need to ram a change through to fix... | 23:27 |
| mrda | rloo: we're still investigating | 23:27 |
| jlvillal | rloo: If you know how to fix it that would be great. If not, then yes. dims in oslo is looking at it. | 23:27 |
| mrda | jlvillal: cool, thanks for getting dims involved | 23:28 |
| rloo | jlvillal: no. I don't even know where/what the problem is. Don't want to spend time looking into it if you/others are. | 23:28 |
| jlvillal | rloo: Agreed. Thanks. | 23:28 |
| rloo | i wonder if this means that the patch we're waiting to merge, won't get merged. | 23:28 |
| jlvillal | Hmmmm | 23:29 |
| mrda | I think it's going to stop it | 23:29 |
| jlvillal | rloo: Yeah, it failed | 23:29 |
| jlvillal | devananda: https://review.openstack.org/#/c/216388/1 is going to fail to merge because of broken gate. | 23:30 |
| rloo | jlvillal: let's just hope my 2-week-to-get-the-release-working doesn't come true | 23:30 |
| jlvillal | Yes. I will hope :) | 23:30 |
| mrda | rloo: only at the start, we should be able to get it happening quick :) | 23:30 |
| * mrda is optomistic | 23:31 | |
| rloo | yeah, i don't think it'll take that long :) | 23:31 |
| *** achanda_ has joined #openstack-ironic | 23:31 | |
| *** alexpilotti has joined #openstack-ironic | 23:32 | |
| *** achanda has quit IRC | 23:34 | |
| *** EmilienM has quit IRC | 23:34 | |
| *** achanda_ has quit IRC | 23:35 | |
| *** alexpilotti has quit IRC | 23:36 | |
| *** EmilienM has joined #openstack-ironic | 23:36 | |
| mrda | jlvillal: well, it's good to know that that test has changed over time between Nova and Ironic | 23:36 |
| jlvillal | yeah | 23:37 |
| jlvillal | mrda: And not affecting Nova it seems. | 23:37 |
| *** lekha has quit IRC | 23:38 | |
| * mrda looks at what a NovaObjectRegistry thing is | 23:38 | |
| *** lekha has joined #openstack-ironic | 23:39 | |
| *** morganfainberg is now known as morgan | 23:42 | |
| *** alexpilotti has joined #openstack-ironic | 23:44 | |
| jlvillal | mrda: I was peeking there earlier. | 23:44 |
| jlvillal | mrda: http://blog.ionelmc.ro/2015/02/09/understanding-python-metaclasses/ Restrictions with multiple metaclasses | 23:44 |
| mrda | Metaclasses are the work of the Devil, I tell you | 23:45 |
| *** puranamr has quit IRC | 23:46 | |
| jlvillal | hah | 23:47 |
| jlvillal | mrda: Discussion with dansmith about it over on #openstack-nova | 23:51 |
| *** alexpilotti has quit IRC | 23:53 | |
| mrda | ta | 23:55 |
| *** jcoufal has joined #openstack-ironic | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!