| jlvillal | sbezverk: That seems to be a nova error | 00:04 |
|---|---|---|
| *** ijw has joined #openstack-ironic | 00:04 | |
| jlvillal | sbezverk: Ah, you figured it out. | 00:06 |
| *** ijw has quit IRC | 00:10 | |
| *** ijw has joined #openstack-ironic | 00:10 | |
| *** ijw has quit IRC | 00:15 | |
| *** rbudden has joined #openstack-ironic | 00:16 | |
| *** mtanino__ has joined #openstack-ironic | 00:22 | |
| *** jaypipes has quit IRC | 00:23 | |
| *** Sukhdev has quit IRC | 00:33 | |
| *** vinsh has quit IRC | 00:34 | |
| *** vinsh has joined #openstack-ironic | 00:35 | |
| *** rloo has quit IRC | 00:44 | |
| *** keekz has quit IRC | 01:00 | |
| *** dmellado has quit IRC | 01:00 | |
| *** keekz has joined #openstack-ironic | 01:00 | |
| *** Administrator_ is now known as zhugaoxiao | 01:00 | |
| *** jlvillal has quit IRC | 01:01 | |
| *** dmellado has joined #openstack-ironic | 01:02 | |
| *** phuongnh has joined #openstack-ironic | 01:03 | |
| *** yuanying_ has joined #openstack-ironic | 01:04 | |
| *** oomichi has quit IRC | 01:05 | |
| *** yuanying has quit IRC | 01:06 | |
| *** jlvillal has joined #openstack-ironic | 01:06 | |
| *** oomichi has joined #openstack-ironic | 01:07 | |
| *** WeiAA has joined #openstack-ironic | 01:26 | |
| *** ijw has joined #openstack-ironic | 01:32 | |
| *** baoli has quit IRC | 01:42 | |
| *** ijw has quit IRC | 01:47 | |
| *** mat128 has joined #openstack-ironic | 01:54 | |
| *** mat128 has quit IRC | 01:55 | |
| *** crushil has joined #openstack-ironic | 01:56 | |
| openstackgerrit | Ngo Quoc Cuong proposed openstack/ironic master: Remove translation of log messages from ironic/drivers/modules/oneview https://review.openstack.org/459909 | 01:57 |
| *** fragatin_ has quit IRC | 01:57 | |
| *** fragatina has joined #openstack-ironic | 01:58 | |
| *** mat128 has joined #openstack-ironic | 02:07 | |
| *** mat128 has quit IRC | 02:10 | |
| *** rbudden has quit IRC | 02:11 | |
| *** baoli has joined #openstack-ironic | 02:13 | |
| openstackgerrit | Ngo Quoc Cuong proposed openstack/ironic master: Remove translation of log messages from ironic/drivers/modules/network https://review.openstack.org/459910 | 02:18 |
| *** WeiAA has quit IRC | 02:26 | |
| *** WeiAA has joined #openstack-ironic | 02:26 | |
| *** tuanluong has joined #openstack-ironic | 02:28 | |
| *** zhurong has joined #openstack-ironic | 02:33 | |
| *** ijw_ has joined #openstack-ironic | 02:33 | |
| *** Lambert has joined #openstack-ironic | 02:34 | |
| *** ijw_ has quit IRC | 02:37 | |
| *** baoli has quit IRC | 02:47 | |
| *** baoli has joined #openstack-ironic | 02:47 | |
| *** baoli has quit IRC | 02:49 | |
| *** WeiAA has quit IRC | 02:53 | |
| *** WeiAA has joined #openstack-ironic | 02:54 | |
| *** gcb has joined #openstack-ironic | 02:55 | |
| openstackgerrit | Ngo Quoc Cuong proposed openstack/ironic master: Remove translation of log messages from ironic/drivers/modules/drac https://review.openstack.org/459918 | 02:57 |
| openstackgerrit | Ngo Quoc Cuong proposed openstack/ironic master: Remove translation of log messages from ironic/drivers/modules/ilo https://review.openstack.org/459925 | 03:14 |
| *** zhurong has quit IRC | 03:15 | |
| *** gkadam has joined #openstack-ironic | 03:19 | |
| openstackgerrit | Hao Li proposed openstack/ironic-lib master: Retry when execute partprobe cmd failed https://review.openstack.org/443604 | 03:21 |
| *** amotoki has joined #openstack-ironic | 03:27 | |
| *** mtanino__ has quit IRC | 03:28 | |
| *** vikrant has joined #openstack-ironic | 03:37 | |
| *** amotoki has quit IRC | 03:54 | |
| openstackgerrit | Ngo Quoc Cuong proposed openstack/ironic master: Remove translation of log messages from ironic/drivers/modules/ (not include sub-folder) https://review.openstack.org/459936 | 04:03 |
| *** karthiks has joined #openstack-ironic | 04:04 | |
| *** amotoki has joined #openstack-ironic | 04:10 | |
| *** vmud213 has joined #openstack-ironic | 04:12 | |
| *** amotoki has quit IRC | 04:15 | |
| *** garyk has joined #openstack-ironic | 04:18 | |
| *** amotoki has joined #openstack-ironic | 04:20 | |
| *** jaganathan has joined #openstack-ironic | 04:21 | |
| *** fragatin_ has joined #openstack-ironic | 04:22 | |
| *** fragatina has quit IRC | 04:26 | |
| *** fragatin_ has quit IRC | 04:27 | |
| *** tuanluong_ has joined #openstack-ironic | 04:28 | |
| *** tuanluong has quit IRC | 04:32 | |
| *** tuanluong__ has joined #openstack-ironic | 04:34 | |
| *** tuanluong_ has quit IRC | 04:38 | |
| *** adisky_ has joined #openstack-ironic | 04:39 | |
| openstackgerrit | paresh sao proposed openstack/proliantutils master: Adding capability to check NVDIMM_N presence https://review.openstack.org/459535 | 04:44 |
| *** fragatina has joined #openstack-ironic | 04:45 | |
| *** fragatina has quit IRC | 04:45 | |
| *** fragatina has joined #openstack-ironic | 04:46 | |
| *** amotoki has quit IRC | 04:49 | |
| *** tuanluong has joined #openstack-ironic | 04:55 | |
| *** tuanluong_ has joined #openstack-ironic | 04:57 | |
| *** vmud213 has quit IRC | 04:57 | |
| *** amotoki has joined #openstack-ironic | 04:58 | |
| *** tuanluong__ has quit IRC | 04:58 | |
| *** tuanluong has quit IRC | 05:00 | |
| openstackgerrit | paresh sao proposed openstack/proliantutils master: Add support to get cpu virtualization capability https://review.openstack.org/422490 | 05:01 |
| *** hw_wutianwei has joined #openstack-ironic | 05:07 | |
| *** vmud213 has joined #openstack-ironic | 05:07 | |
| *** zhurong has joined #openstack-ironic | 05:08 | |
| openstackgerrit | Jaganathan Palanisamy proposed openstack/ironic-python-agent master: NUMA-topology collector https://review.openstack.org/424729 | 05:11 |
| *** Nisha_Agarwal has joined #openstack-ironic | 05:13 | |
| *** hshiina has joined #openstack-ironic | 05:16 | |
| *** udesale has joined #openstack-ironic | 05:22 | |
| *** yolanda has quit IRC | 05:28 | |
| *** e0ne has joined #openstack-ironic | 05:32 | |
| *** e0ne has quit IRC | 05:33 | |
| *** mvpnitesh has joined #openstack-ironic | 05:41 | |
| *** zhurong has quit IRC | 05:52 | |
| *** karthiks has quit IRC | 05:56 | |
| *** jaganathan has quit IRC | 05:56 | |
| *** karthiks has joined #openstack-ironic | 06:07 | |
| *** udesale__ has joined #openstack-ironic | 06:12 | |
| *** udesale has quit IRC | 06:15 | |
| *** MarginHu has joined #openstack-ironic | 06:24 | |
| *** zhurong has joined #openstack-ironic | 06:26 | |
| *** racedo has joined #openstack-ironic | 06:26 | |
| *** ChubYann has quit IRC | 06:27 | |
| *** dsneddon has joined #openstack-ironic | 06:27 | |
| openstackgerrit | Wang KaiFeng proposed openstack/ironic master: Fix cleaning documents https://review.openstack.org/459961 | 06:28 |
| *** rbartal has joined #openstack-ironic | 06:29 | |
| *** amotoki has quit IRC | 06:30 | |
| *** amotoki has joined #openstack-ironic | 06:31 | |
| *** jaianshu has joined #openstack-ironic | 06:33 | |
| *** jaianshu_ has joined #openstack-ironic | 06:33 | |
| *** zhurong has quit IRC | 06:33 | |
| *** l_jacob_ has quit IRC | 06:35 | |
| *** ijw has joined #openstack-ironic | 06:36 | |
| *** crushil has quit IRC | 06:38 | |
| *** d0ugal has quit IRC | 06:39 | |
| *** ijw has quit IRC | 06:40 | |
| *** d0ugal has joined #openstack-ironic | 06:41 | |
| *** makowals has quit IRC | 06:44 | |
| *** makowals has joined #openstack-ironic | 06:44 | |
| *** amotoki has quit IRC | 06:47 | |
| *** yolanda has joined #openstack-ironic | 06:49 | |
| *** jaganathan has joined #openstack-ironic | 06:49 | |
| *** vmud213 has quit IRC | 06:50 | |
| *** mvpnitesh has quit IRC | 06:54 | |
| *** mvpnitesh has joined #openstack-ironic | 06:55 | |
| *** chas has joined #openstack-ironic | 06:55 | |
| *** MarginHu has quit IRC | 06:57 | |
| *** udesale has joined #openstack-ironic | 06:58 | |
| *** udesale__ has quit IRC | 06:59 | |
| *** racedo has quit IRC | 07:02 | |
| *** tesseract has joined #openstack-ironic | 07:04 | |
| *** udesale__ has joined #openstack-ironic | 07:05 | |
| *** e0ne has joined #openstack-ironic | 07:07 | |
| *** udesale has quit IRC | 07:08 | |
| *** SerenaFeng has joined #openstack-ironic | 07:11 | |
| openstackgerrit | M V P Nitesh proposed openstack/ironic master: Validate ipmi_address format on ipmitool power interface validation https://review.openstack.org/436945 | 07:11 |
| pas-ha | morning ironic :) | 07:14 |
| *** rbartal has quit IRC | 07:15 | |
| *** rbartal has joined #openstack-ironic | 07:17 | |
| *** pcaruana has joined #openstack-ironic | 07:19 | |
| *** yolanda has quit IRC | 07:25 | |
| *** zhurong has joined #openstack-ironic | 07:28 | |
| *** ralonsoh has joined #openstack-ironic | 07:36 | |
| Nisha_Agarwal | pas-ha, morning :) | 07:38 |
| *** yolanda has joined #openstack-ironic | 07:41 | |
| *** vmud213 has joined #openstack-ironic | 07:43 | |
| *** athomas has joined #openstack-ironic | 07:47 | |
| *** e0ne has quit IRC | 07:53 | |
| openstackgerrit | Jacek Tomasiak proposed openstack/ironic master: Remove obsolete sentence from comment. https://review.openstack.org/459985 | 07:56 |
| *** amotoki has joined #openstack-ironic | 07:58 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:00 | |
| *** shuyingya has joined #openstack-ironic | 08:04 | |
| *** udesale has joined #openstack-ironic | 08:06 | |
| *** udesale__ has quit IRC | 08:08 | |
| *** zhurong has quit IRC | 08:08 | |
| *** lucas-afk is now known as lucasagomes | 08:10 | |
| lucasagomes | morning all | 08:12 |
| *** dtantsur|afk is now known as dtantsur | 08:14 | |
| dtantsur | morning ironic | 08:14 |
| *** tovin07 has quit IRC | 08:17 | |
| *** e0ne has joined #openstack-ironic | 08:18 | |
| *** mgoddard has joined #openstack-ironic | 08:20 | |
| *** milan|zzZZZzzz has joined #openstack-ironic | 08:22 | |
| *** yolanda has quit IRC | 08:24 | |
| *** paresh has joined #openstack-ironic | 08:25 | |
| tuanluong_ | Morning Ironic | 08:26 |
| paresh | Hi JayF | 08:26 |
| paresh | JayF Can you please review this patch https://review.openstack.org/#/c/454625/ | 08:28 |
| patchbot | patch 454625 - ironic - Change to avoid building of deploy iso when provided | 08:28 |
| *** jaosorior_away is now known as jaosorior | 08:28 | |
| *** mvpnitesh has quit IRC | 08:29 | |
| *** priteau has joined #openstack-ironic | 08:29 | |
| *** racedo has joined #openstack-ironic | 08:30 | |
| *** gkadam is now known as gkadam-afk | 08:33 | |
| *** saop_ has joined #openstack-ironic | 08:34 | |
| *** derekh has joined #openstack-ironic | 08:36 | |
| *** paresh has quit IRC | 08:37 | |
| *** mvpnitesh has joined #openstack-ironic | 08:39 | |
| *** yolanda has joined #openstack-ironic | 08:41 | |
| openstackgerrit | Mark Goddard proposed openstack/bifrost master: Allow log directories to be configured https://review.openstack.org/460002 | 08:43 |
| openstackgerrit | Mark Goddard proposed openstack/bifrost master: Support extra PXE kernel arguments for Inspector https://review.openstack.org/460003 | 08:43 |
| openstackgerrit | Mark Goddard proposed openstack/bifrost master: Support configuring inspector processing hooks https://review.openstack.org/460004 | 08:43 |
| *** vmud213 has quit IRC | 08:43 | |
| openstackgerrit | Mark Goddard proposed openstack/bifrost master: Support storage of introspection data in Nginx https://review.openstack.org/460005 | 08:43 |
| *** vmud213 has joined #openstack-ironic | 08:44 | |
| Nisha_Agarwal | lucasagomes, dtantsur morning | 08:44 |
| Nisha_Agarwal | tuanluong_, morning :) | 08:44 |
| openstackgerrit | Dao Cong Tien proposed openstack/python-ironicclient master: Extends driver-list, driver-show supporting new hardware types https://review.openstack.org/419274 | 08:45 |
| Nisha_Agarwal | dtantsur, i wanted to discuss on pxe_enabled flag | 08:45 |
| tuanluong_ | Morning Nisha_Agarwal, dtantsur lucasagomes pas-ha | 08:46 |
| *** mvpnitesh has quit IRC | 08:46 | |
| lucasagomes | o/ | 08:47 |
| dtantsur | Nisha_Agarwal, sure, it may be useful to wait for milan|zzZZZzzz and sambetts|afk though | 08:48 |
| *** milan|zzZZZzzz is now known as milan | 08:48 | |
| *** Serlex has joined #openstack-ironic | 08:48 | |
| milan | morning Ironic! :D | 08:48 |
| Nisha_Agarwal | dtantsur, sure. | 08:49 |
| Nisha_Agarwal | milan, hi | 08:49 |
| *** erolg has joined #openstack-ironic | 08:49 | |
| Nisha_Agarwal | dtantsur, milan is there now :) | 08:49 |
| milan | morning dtantsur, Nisha_Agarwal! :) | 08:49 |
| * milan reads scrollback | 08:49 | |
| Nisha_Agarwal | milan, i said nothing :D | 08:49 |
| Nisha_Agarwal | i just asked for discussion | 08:49 |
| milan | aha :D | 08:50 |
| milan | OK | 08:50 |
| milan | let's discuss! | 08:50 |
| Nisha_Agarwal | :) | 08:51 |
| milan | Nisha_Agarwal, so given what sambetts|afk said yesterday, I'd just mention that the flag not being set is meaningless because the boot method is vmedia. If someone changes the boot method to PXE, it should be recommended to switch the manage_boot flag back to True as well | 08:53 |
| Nisha_Agarwal | milan, as i read the specs earlier ones(in neutron) pxe_enabled flag was added so that neutron port is created only for those ironic ports/portgroups which has pxe_enabled flag set | 08:53 |
| milan | Nisha_Agarwal, in my understanding it is used to mark the NIC that actually booted the node over PXE | 08:53 |
| Nisha_Agarwal | milan i know the current implementation of inspector sets pxe_enabled only when it is pxe booted, but i think thats incomplete | 08:53 |
| milan | Nisha_Agarwal, maybe, but this is about PXE | 08:54 |
| Nisha_Agarwal | milan, the text in neutron spec says this | 08:55 |
| Nisha_Agarwal | If there are multiple pxe_enabled ports or portgroups, dhcpboot options will be set for all portgroups and all pxe_enabled ports not belonging to any portgroup. | 08:55 |
| Nisha_Agarwal | so there can be more than one pxe_enabled ports | 08:55 |
| Nisha_Agarwal | pxe booting will happen only on one port | 08:55 |
| Nisha_Agarwal | how does inspector then determine for other ports which is not used for pxe booting | 08:56 |
| milan | Nisha_Agarwal, from the data IPA, having inspected the node, sends back to inspector | 08:56 |
| *** aparnav has joined #openstack-ironic | 08:58 | |
| milan | Nisha_Agarwal, don't get me wrong, it's not blocking the manage_boot flag, it's IMO just something that should be mentioned as a side effect | 08:58 |
| Nisha_Agarwal | suppose the hardware has 4 macs and out of that 2 macs are capable of pxe booting an instance | 08:59 |
| milan | Nisha_Agarwal, other thing is manage_boot is mutually-exclusive with the discovery feature | 08:59 |
| Nisha_Agarwal | milan, its not about the spec right now | 08:59 |
| milan | OK :) | 08:59 |
| Nisha_Agarwal | i was under the assumption that pxe_enabled flag means the MAC is physically connected | 09:00 |
| Nisha_Agarwal | and which can boot up the instance | 09:00 |
| *** sambetts|afk is now known as sambetts | 09:00 | |
| Nisha_Agarwal | even in vmedia case the iso is booted via vmedia but the user instance boots using the MAC only | 09:00 |
| Nisha_Agarwal | milan so let me just summarize the issue | 09:01 |
| Nisha_Agarwal | suppose the scenario is: | 09:01 |
| Nisha_Agarwal | 1. the hardware has 4 macs. but the first one is not connected physically, rather the second one or third one is connected physically. | 09:02 |
| *** MattMan has quit IRC | 09:02 | |
| *** MattMan has joined #openstack-ironic | 09:02 | |
| milan | ok | 09:03 |
| Nisha_Agarwal | 2. Now after creating ironic node, i do inspection. OOB inspection is suppoed to create all ports by default | 09:03 |
| sambetts | Nisha_Agarwal: the existance of the port in Ironic should imply that it is physically connected, ports that aren't connected shouldn't be added to Ironic IMO | 09:03 |
| Nisha_Agarwal | sambetts, thats not true | 09:03 |
| Nisha_Agarwal | sambetts, yesterday also i told the same thing | 09:03 |
| Nisha_Agarwal | sambetts, default option for ironic inspection is to create all ports | 09:04 |
| Nisha_Agarwal | sambetts, which is where OOB inspection is bound to create all ports | 09:04 |
| sambetts | -1 default option for insepction is to create only the port that inspector pxe booted on | 09:04 |
| Nisha_Agarwal | sambetts, that may be true for ironic-inspector | 09:04 |
| Nisha_Agarwal | not for ironic | 09:04 |
| Nisha_Agarwal | please refer to Hardware inspection documentation | 09:05 |
| sambetts | in OOB inspection that is down to your driver to implement correctly | 09:05 |
| sambetts | in my opinion only ports that can be used should be added to Ironic | 09:05 |
| sambetts | otherwise why add them | 09:06 |
| Nisha_Agarwal | sambetts, https://docs.openstack.org/developer/ironic/deploy/inspection.html | 09:06 |
| Nisha_Agarwal | sambetts, thats true but it was asked ironic to do so | 09:06 |
| sambetts | Nisha_Agarwal: there isn't anything in there about OOB inspection added ports | 09:06 |
| Nisha_Agarwal | Inspection will also create the Bare Metal service ports for the discovered ethernet MACs. Operators will have to manually delete the Bare Metal service ports for which physical media is not connected. This is required due to the bug 1405131. | 09:06 |
| openstack | bug 1666009 in Ironic "duplicate for #1405131 [RFE] Physical network awareness" [Wishlist,Triaged] https://launchpad.net/bugs/1666009 - Assigned to Mark Goddard (mgoddard) | 09:06 |
| Nisha_Agarwal | the above text is from documentation | 09:07 |
| sambetts | Nisha_Agarwal: that sentence implys that the ports in Ironic should be removed or not added at all if they aren't connected | 09:07 |
| Nisha_Agarwal | so for OOB inspection it may or may not be possible to find if the NIC is physically connected | 09:07 |
| sambetts | Sure that might be a hardware limitation, but thats why you can use in-band inspection | 09:09 |
| sambetts | to discover those sorts of things | 09:09 |
| Nisha_Agarwal | sambetts, well from history i can tell you we had done a workaround for it | 09:09 |
| Nisha_Agarwal | https://bugs.launchpad.net/ironic/+bug/1438546 | 09:09 |
| openstack | Launchpad bug 1438546 in Ironic "ilo driver inspection requires extra parameters" [High,Fix released] - Assigned to Ramakrishnan G (rameshg87) (rameshg87) | 09:09 |
| Nisha_Agarwal | and then the above bug was filed to remove that workaround and make it default for inspection to create all ports | 09:10 |
| *** SerenaFeng has quit IRC | 09:10 | |
| Nisha_Agarwal | i think inspector has added the "all" option for add_ports that time only | 09:10 |
| Nisha_Agarwal | sambetts, so ok coming to ur point of using inband inspection | 09:11 |
| Nisha_Agarwal | we do want to do inband inspection and thats where this discussion is happening | 09:11 |
| Nisha_Agarwal | so from vmedia drivers when you boot up the iso using inspector(assuming vmedia supportt is there), pxe_enabled flag wont be set for the ironic ports | 09:12 |
| Nisha_Agarwal | and we still have the same situation where nova boot will fail if the port is not connected physically and neutron port is created for the same | 09:13 |
| sambetts | Nisha_Agarwal: you need this patch https://review.openstack.org/#/c/450104/ | 09:13 |
| patchbot | patch 450104 - ironic - Add pxe_required flag to boot interface | 09:13 |
| sambetts | Nisha_Agarwal: the current behaviour of pxe_enabled is broken | 09:13 |
| sambetts | Nisha_Agarwal: that patch fixes it for your vmedia case | 09:14 |
| Nisha_Agarwal | and as i read the neutron documentation , neutron port wont be created for any ironic ports because pxe_enabled is not set | 09:14 |
| Nisha_Agarwal | ok let me see | 09:14 |
| milan | sambetts, Nisha_Agarwal but more general: if the boot mode is set to vmedia for IPA, should it be different for the user's instance? | 09:15 |
| *** erolg has quit IRC | 09:15 | |
| * Nisha_Agarwal looking into above patch | 09:17 | |
| *** mvpnitesh has joined #openstack-ironic | 09:18 | |
| *** SerenaFeng has joined #openstack-ironic | 09:19 | |
| *** zhurong has joined #openstack-ironic | 09:22 | |
| milan | sambetts, silly question: can we have different boot methods for different node "actions" (deploy, inspect, clean...)? | 09:24 |
| *** yolanda has quit IRC | 09:26 | |
| Nisha_Agarwal | sambetts, if vmedia is used for deploy iso, even for user instance it will be vmedia. But when the user instance comes up it needs an IP to be assigned to the active MAC(means physically connected). | 09:27 |
| sambetts | milan: not unless someone writes a custom boot interface that does that, the boot interface is what decides how to boot the node, there isn't a boot mode settings | 09:28 |
| sambetts | Nisha_Agarwal: sure, why wouldn't that be the case, because all ports in Ironic should be physically connected | 09:29 |
| Nisha_Agarwal | sambetts, so if the MAC is not connected physically but has ironic node and subsequent neutron port the deploy and boot will fail | 09:29 |
| Nisha_Agarwal | sambetts, not true. in data centre you may just have one or two NICs connected | 09:29 |
| sambetts | Nisha_Agarwal: again, ports in Ironic should *all* be physically connected | 09:29 |
| sambetts | Nisha_Agarwal: you don't add the ports to Ironic that aren't connected | 09:29 |
| Nisha_Agarwal | sambetts, thats automated via inspection and it creates for all by default as i told earlier | 09:30 |
| milan | sambetts, ack, thanks, then the manage_boot for inspection won't break other node "actions"/interfaces; i.e once vmedia, always vmedia | 09:30 |
| sambetts | as you quoted me earlier, if its hardware limitation that your OOB inspection adds all ports to Ironic regardless of if they are connected or not, then "Operators will have to manually delete the Bare Metal service ports for which physical media is not connected." | 09:31 |
| Nisha_Agarwal | so my whole point is that if inband can return pxe_enabled for the MACs which are active will solve our problem when inspector support vmedia boot via ironic | 09:31 |
| sambetts | Nisha_Agarwal: In-band inspection should *remove* MACs which aren't connected | 09:31 |
| milan | Nisha_Agarwal, but that behaviour was considered a bug; see the same docs link first paragraph you shared earlier | 09:31 |
| milan | * adding all ports | 09:32 |
| sambetts | Nisha_Agarwal: thats what the "keep_ports = " option is for in the configuration https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/conf.py#L66 | 09:32 |
| Nisha_Agarwal | milan, please see the bug and patch i quoted for ilo drivers "inspect_ports" option | 09:32 |
| saop_ | Hi tuanluong_ | 09:33 |
| Nisha_Agarwal | dtantsur, might also remember all this. he was working on this earlier | 09:33 |
| dtantsur | sorry, what's the question? | 09:33 |
| tuanluong_ | hello saop_ | 09:34 |
| saop_ | tuanluong_ I replied to the comment of patch https://review.openstack.org/#/c/454625/12 please look at it | 09:34 |
| patchbot | patch 454625 - ironic - Change to avoid building of deploy iso when provided | 09:34 |
| Nisha_Agarwal | dtantsur, we (ilo drivers/OOB inspection) was asked to create ports for all MACs during inspection | 09:34 |
| Nisha_Agarwal | dtantsur, and so was done by inspector also after we were asked to do so | 09:34 |
| Nisha_Agarwal | dtantsur, do you remember this part | 09:34 |
| dtantsur | yep, I think it was devananda who felt particularly strongly about that | 09:36 |
| *** zhurong has quit IRC | 09:37 | |
| Nisha_Agarwal | dtantsur, thanks. milan sambetts ^^^ | 09:37 |
| Nisha_Agarwal | dtantsur, i am just saying that pxe_enabled which is set by inspector for only PXE booted MACs can we set it for all active MACs | 09:38 |
| sambetts | -1 | 09:38 |
| dtantsur | Nisha_Agarwal, no, you don't know if they can PXE boot | 09:38 |
| Nisha_Agarwal | dtantsur, milan sambetts reason for asking above is that when we do vmedia boot validate_interfaces() return no mac if add_ports is pxe | 09:39 |
| Nisha_Agarwal | but when we have it as active it returns active/connected MACs | 09:40 |
| Nisha_Agarwal | and we can then have ironic ports created only for those returned MACs | 09:40 |
| sambetts | right, add ports "active" is the inspector setting that makes the most sense in your environment | 09:42 |
| Nisha_Agarwal | dtantsur, to above , one general question....what do u mean "you don't know if they can PXE boot"? does it mean it may or may not have tftp working | 09:42 |
| Nisha_Agarwal | sambetts, correct | 09:42 |
| sambetts | Nisha_Agarwal: certain hardware can have the PXE firmware disabled or not usable on specific ports | 09:43 |
| *** yolanda has joined #openstack-ironic | 09:43 | |
| Nisha_Agarwal | sambetts, with pxe_enabled flag not set but ironic port created, is it expected to work for nova boot? | 09:43 |
| sambetts | Nisha_Agarwal: it is expected to work for tenant ports yes | 09:43 |
| sambetts | Nisha_Agarwal: like I pointed out the behaviour is slightly broken and vsaienk0's patch I liked fixes it | 09:44 |
| Nisha_Agarwal | sambetts, hmmmm i will have to test his patch | 09:46 |
| Nisha_Agarwal | milan, i will add the context of pxe_enabled in the spec as you say. i would have to test the behaviour with and without vsaienk0's patch | 09:48 |
| Nisha_Agarwal | sambetts, thanks | 09:48 |
| sambetts | Nisha_Agarwal: no worries :) | 09:48 |
| * Nisha_Agarwal have to go for a while. will connect back | 09:48 | |
| milan | Nisha_Agarwal, ack, thanks! | 09:48 |
| *** saop_ has quit IRC | 09:49 | |
| tuanluong_ | hello saop_, what do you think about my message? | 09:49 |
| openstackgerrit | Tovin Seven proposed openstack/ironic master: Enable OSProfiler support in Ironic https://review.openstack.org/358142 | 09:53 |
| *** Nisha_Agarwal has quit IRC | 09:53 | |
| *** WeiAA has quit IRC | 10:00 | |
| openstackgerrit | M V P Nitesh proposed openstack/ironic master: Validate ipmi_address format on ipmitool power interface validation https://review.openstack.org/436945 | 10:01 |
| *** SerenaFeng has quit IRC | 10:06 | |
| openstackgerrit | Ankit Kumar proposed openstack/ironic master: Test cases for iLO drivers in standalone ironic https://review.openstack.org/460034 | 10:13 |
| *** tuanluong_ has quit IRC | 10:15 | |
| openstackgerrit | paresh sao proposed openstack/ironic master: Change to avoid building of deploy iso when provided https://review.openstack.org/454625 | 10:17 |
| *** zackf has joined #openstack-ironic | 10:24 | |
| *** jrcloud has quit IRC | 10:27 | |
| openstackgerrit | Tran Ha Tuyen proposed openstack/ironic master: Remove logging import unused https://review.openstack.org/460038 | 10:28 |
| *** gkadam-afk is now known as gkadam | 10:30 | |
| *** gkadam has quit IRC | 10:31 | |
| *** gkadam has joined #openstack-ironic | 10:31 | |
| *** gkadam is now known as gkadam-afk | 10:31 | |
| openstackgerrit | Tran Ha Tuyen proposed openstack/ironic master: TrivialFix: Remove logging import unused https://review.openstack.org/460038 | 10:32 |
| *** Nisha_Agarwal has joined #openstack-ironic | 10:32 | |
| Nisha_Agarwal | milan, thanks for the comments on the spec. I couldnt get your one of the comment though. It would be great if you could elaborate it. | 10:34 |
| Nisha_Agarwal | milan, i have mentioned that in the patch | 10:34 |
| milan | Nisha_Agarwal, sure, let me check it again | 10:34 |
| *** vmud213 has quit IRC | 10:35 | |
| * Nisha_Agarwal available for next 10 min max. After that will login in evening/night. | 10:35 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Inspector rules API does not return all attributes. https://review.openstack.org/442876 | 10:36 |
| *** udesale has quit IRC | 10:42 | |
| *** Nisha_Agarwal has quit IRC | 10:43 | |
| *** Lambert has quit IRC | 10:44 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 10:44 | |
| *** amarao has joined #openstack-ironic | 10:45 | |
| *** jkilpatr has quit IRC | 10:45 | |
| *** Nisha_Agarwal has quit IRC | 10:45 | |
| *** jaianshu_ has quit IRC | 10:46 | |
| *** jaianshu has quit IRC | 10:46 | |
| *** aparnav has quit IRC | 10:46 | |
| *** jaianshu has joined #openstack-ironic | 10:49 | |
| *** jaianshu_ has joined #openstack-ironic | 10:49 | |
| *** aparnav has joined #openstack-ironic | 10:49 | |
| milan | aarefiev, morning! Could you please check the spec amendment for the filter drivers: https://review.openstack.org/#/c/457765/ | 10:53 |
| patchbot | patch 457765 - ironic-inspector-specs - Follow up PXE filter driver | 10:53 |
| milan | dtantsur, sambetts same thing please ;) ^ Thanks! | 10:54 |
| dtantsur | will do (after lunch, I guess) | 10:54 |
| aarefiev | milan: sure, I'll take a look today | 10:55 |
| milan | thanks! | 10:56 |
| * milan relocates | 10:56 | |
| jaganathan | milan, hi | 10:57 |
| *** lucasagomes is now known as lucas-hungry | 10:58 | |
| *** vmud213 has joined #openstack-ironic | 10:58 | |
| milan | jaganathan, hey, how's going? :) | 10:58 |
| *** phuongnh has quit IRC | 11:00 | |
| jaganathan | milan, numa_node_id, cpu_id, so can i change the key as numa_node_id+","+cpu_id? | 11:00 |
| milan | jaganathan, maybe I'm missing something, but why is the string needed in the key? I'd just use a tuple: numa_node_id, cpu_id | 11:02 |
| milan | * dict_key = numa_node_id, cpu_id | 11:02 |
| jaganathan | milan, string i added for understanding purpose. added comments separate, so i can change dict_key as numa_node_id + "," + cpu_id | 11:03 |
| milan | please, use the tuple | 11:04 |
| dtantsur | or even a dict with meaningful keys | 11:04 |
| jaganathan | milan, ok | 11:04 |
| milan | thanks! :) relocating now for real ;) | 11:05 |
| *** jkilpatr has joined #openstack-ironic | 11:05 | |
| *** SerenaFeng has joined #openstack-ironic | 11:10 | |
| *** izumi777 has quit IRC | 11:10 | |
| *** milan has quit IRC | 11:11 | |
| *** ipukha has joined #openstack-ironic | 11:13 | |
| *** leetpy has joined #openstack-ironic | 11:17 | |
| *** leetpy has quit IRC | 11:17 | |
| openstackgerrit | Jaganathan Palanisamy proposed openstack/ironic-python-agent master: NUMA-topology collector https://review.openstack.org/424729 | 11:18 |
| *** yolanda has quit IRC | 11:19 | |
| *** jaypipes has joined #openstack-ironic | 11:22 | |
| *** gkadam-afk is now known as gkadam | 11:23 | |
| *** zackf has quit IRC | 11:23 | |
| *** zackf has joined #openstack-ironic | 11:24 | |
| *** zackf has quit IRC | 11:26 | |
| *** zackf has joined #openstack-ironic | 11:26 | |
| *** zackf has quit IRC | 11:29 | |
| *** rbudden has joined #openstack-ironic | 11:34 | |
| *** Serlex has quit IRC | 11:34 | |
| *** yolanda has joined #openstack-ironic | 11:34 | |
| *** bfournie has quit IRC | 11:41 | |
| *** mat128 has joined #openstack-ironic | 11:41 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: DO NOT MERGE: testing CI with systemd https://review.openstack.org/460076 | 11:41 |
| mat128 | morning group | 11:43 |
| dtantsur | morning mat128! | 11:44 |
| dtantsur | mat128, btw do you plan on working on deploy steps in the near future? I know that yolanda does not have much time for that nowadays | 11:44 |
| mat128 | I now have my own company which means I have both less time and more time for these kinds of things. | 11:45 |
| mat128 | shuffling with priorities takes more time than I thought | 11:46 |
| mat128 | I would be more comfortable reviewing the specs and the code than doing it, for fear of not providing enough hours to the project | 11:47 |
| dtantsur | mat128, I get it (own company, nice!). I just wonder if we have to remove it from the priorities list, given that both contributors cannot dedicate time to it. | 11:58 |
| dtantsur | and maybe think about moving the ansible deploy driver in instead, as it provides a similar level of customization | 11:59 |
| *** mat128 has quit IRC | 12:00 | |
| *** SerenaFeng has quit IRC | 12:01 | |
| *** karthiks has quit IRC | 12:03 | |
| *** SerenaFeng has joined #openstack-ironic | 12:04 | |
| *** lucas-hungry is now known as lucasagomes | 12:11 | |
| *** mat128 has joined #openstack-ironic | 12:12 | |
| mat128 | dtantsur: sure, I think it may make sense | 12:13 |
| *** ccamacho is now known as ccamacho|lunch | 12:13 | |
| mat128 | dtantsur: the only alternative I see is some people from iWeb/Internap who could work on it, like fbouliane | 12:13 |
| mat128 | but I am not aware of their priorities anymore | 12:13 |
| *** bfournie has joined #openstack-ironic | 12:20 | |
| *** crushil has joined #openstack-ironic | 12:21 | |
| *** Serlex has joined #openstack-ironic | 12:21 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-specs master: Remove Pike priorities that cannot make it due to recent team changes https://review.openstack.org/460086 | 12:22 |
| dtantsur | mat128, ^^^ | 12:22 |
| *** jaianshu__ has joined #openstack-ironic | 12:23 | |
| *** jaianshu_ has quit IRC | 12:24 | |
| *** jaianshu has quit IRC | 12:24 | |
| openstackgerrit | Anshul Jain proposed openstack/ironic master: Add functionality test case to support raid configuration. https://review.openstack.org/458730 | 12:24 |
| *** SerenaFeng has quit IRC | 12:24 | |
| *** jaianshu_ has joined #openstack-ironic | 12:24 | |
| *** trown|outtypewww is now known as trown | 12:26 | |
| *** SerenaFeng has joined #openstack-ironic | 12:26 | |
| *** jaianshu__ has quit IRC | 12:28 | |
| *** jaianshu has joined #openstack-ironic | 12:29 | |
| *** marlinc has quit IRC | 12:32 | |
| *** xavierr has joined #openstack-ironic | 12:37 | |
| *** dtantsur is now known as dtantsur|brb | 12:44 | |
| jroll | mat128: congrats on starting your own thing \o/ | 12:48 |
| mat128 | ty | 12:49 |
| mat128 | I heard the unfortunate news via Twitter | 12:49 |
| jroll | (and good luck ofc) | 12:49 |
| *** jaianshu has quit IRC | 12:49 | |
| *** jaianshu_ has quit IRC | 12:49 | |
| *** shuyingya has quit IRC | 12:50 | |
| *** ccamacho|lunch has quit IRC | 12:57 | |
| *** gcb has quit IRC | 12:58 | |
| *** baoli has joined #openstack-ironic | 13:00 | |
| *** aparnav has quit IRC | 13:03 | |
| *** milan has joined #openstack-ironic | 13:03 | |
| *** SerenaFeng has quit IRC | 13:03 | |
| *** yolanda has quit IRC | 13:06 | |
| *** srobert has joined #openstack-ironic | 13:07 | |
| *** gkadam has quit IRC | 13:12 | |
| *** ccamacho has joined #openstack-ironic | 13:13 | |
| *** mat128 has quit IRC | 13:14 | |
| *** rloo has joined #openstack-ironic | 13:15 | |
| openstackgerrit | Jacek Tomasiak proposed openstack/ironic master: Remove obsolete sentence from comment. https://review.openstack.org/459985 | 13:16 |
| *** Goneri has joined #openstack-ironic | 13:16 | |
| *** causten has joined #openstack-ironic | 13:18 | |
| *** dprince has joined #openstack-ironic | 13:20 | |
| *** yolanda has joined #openstack-ironic | 13:23 | |
| *** ccamacho has quit IRC | 13:25 | |
| *** jcoufal has joined #openstack-ironic | 13:25 | |
| *** ccamacho has joined #openstack-ironic | 13:26 | |
| rloo | hi and good morning all! | 13:27 |
| mjturek | rajinir: Can you give more detail here https://bugs.launchpad.net/ironic/+bug/1685924 on why you think https://review.openstack.org/#/c/458747/ is the culprit? | 13:27 |
| openstack | Launchpad bug 1685924 in Ironic "Devstack/ironic fails with libvirt configuration" [Undecided,New] | 13:27 |
| patchbot | patch 458747 - ironic - Ensure we install latest libivrt (MERGED) | 13:27 |
| mjturek | morning rloo :) | 13:27 |
| rloo | morning mjturek! | 13:28 |
| *** shuyingya has joined #openstack-ironic | 13:28 | |
| *** baoli has quit IRC | 13:29 | |
| openstackgerrit | M V P Nitesh proposed openstack/python-ironicclient master: Fixes multiple chassis delete https://review.openstack.org/455662 | 13:29 |
| *** baoli has joined #openstack-ironic | 13:30 | |
| *** vmud213 has quit IRC | 13:31 | |
| *** shuyingya has quit IRC | 13:33 | |
| *** thomasem has left #openstack-ironic | 13:35 | |
| *** jaosorior is now known as jaosorior_away | 13:36 | |
| *** zackf has joined #openstack-ironic | 13:37 | |
| *** priteau has quit IRC | 13:38 | |
| *** fragatina has quit IRC | 13:40 | |
| *** fragatina has joined #openstack-ironic | 13:41 | |
| *** hshiina has left #openstack-ironic | 13:47 | |
| *** g3ek has quit IRC | 13:47 | |
| *** shuyingya has joined #openstack-ironic | 13:51 | |
| *** udesale has joined #openstack-ironic | 13:56 | |
| *** g3ek has joined #openstack-ironic | 13:56 | |
| *** hshiina has joined #openstack-ironic | 13:57 | |
| *** vikrant has quit IRC | 13:57 | |
| *** chlong has joined #openstack-ironic | 13:58 | |
| *** amarao has quit IRC | 13:58 | |
| *** dyasny_ has joined #openstack-ironic | 13:58 | |
| *** amarao has joined #openstack-ironic | 14:00 | |
| *** dtantsur|brb is now known as dtantsur | 14:00 | |
| dtantsur | morning rloo, mjturek | 14:00 |
| dtantsur | rloo, ack re holding deploy steps removal (my pm also wants it, so I can allocate reviewing, but not much coding) | 14:01 |
| *** dyasny has quit IRC | 14:02 | |
| rloo | dtantsur: ok | 14:02 |
| dtantsur | rloo, though tbh I start thinking that the ansible deploy driver is the best answer for customization problem | 14:03 |
| dtantsur | but one does not exclude the other | 14:03 |
| rloo | dtantsur: my biggest concern is the route the info needs to take, user -> nova -> ironic :) | 14:04 |
| rloo | dtantsur: any way, one step at a time! | 14:05 |
| *** amarao has quit IRC | 14:06 | |
| dtantsur | yeah | 14:06 |
| TheJulia | Just remember not all users will just be using nova | 14:07 |
| *** amotoki has quit IRC | 14:07 | |
| * TheJulia slowly wakes up | 14:07 | |
| dtantsur | TheJulia, morning, we do remember :) | 14:07 |
| *** amarao has joined #openstack-ironic | 14:07 | |
| * dtantsur thinks how much easier life for him would be, if TripleO used Ironic without Nova.. | 14:07 | |
| *** livelace has joined #openstack-ironic | 14:08 | |
| TheJulia | Time for heat support in ironic? | 14:08 |
| TheJulia | Err | 14:08 |
| TheJulia | Ironic support in heat | 14:08 |
| * TheJulia ducks | 14:08 | |
| pas-ha | I can revive shardy's pathces in Heat | 14:08 |
| dtantsur | pas-ha, would be fantastic! | 14:09 |
| * pas-ha remebers his old Heat days... | 14:09 | |
| TheJulia | That would be cool actually | 14:09 |
| dtantsur | even without dropping nova, it will help derekh with his work on creating CI for ironic in TripleO (I guess) | 14:09 |
| rloo | TheJulia: yup, i remember. that's another reason to do it, regardless of nova :) | 14:10 |
| * TheJulia is kind of curious what issues derekh is encountering | 14:10 | |
| *** Goneri has quit IRC | 14:11 | |
| dtantsur | TheJulia, well, it was me encountering it first. TripleO CI currently uses a heat stack to test $service in the CI | 14:11 |
| dtantsur | it's hard to write a heat template for testing ironic without heat support for ironic | 14:11 |
| * dtantsur suspects he's talking extremely confusing | 14:11 | |
| TheJulia | Not really, confusing that is | 14:12 |
| *** garyk has quit IRC | 14:14 | |
| derekh | I was firt going to focus on getting ironic installed for that CI job before trying to us it, but ya if heat supported ironic it should help | 14:15 |
| dtantsur | ++ | 14:16 |
| TheJulia | I kind of wonder how opnfv folks are scheduling their test clusters that they deploy with ironic | 14:17 |
| *** rloo has quit IRC | 14:18 | |
| * TheJulia should ask | 14:18 | |
| *** jaganathan has quit IRC | 14:18 | |
| TheJulia | Or go just dig through their code | 14:18 |
| *** rloo has joined #openstack-ironic | 14:18 | |
| *** amotoki has joined #openstack-ironic | 14:21 | |
| *** athomas has quit IRC | 14:21 | |
| *** dyasny_ has quit IRC | 14:24 | |
| *** dyasny has joined #openstack-ironic | 14:24 | |
| *** aparnav has joined #openstack-ironic | 14:26 | |
| *** athomas has joined #openstack-ironic | 14:26 | |
| fbouliane | rloo: https://review.openstack.org/#/c/456952/8 Should I file a bug for gate issues ? | 14:28 |
| patchbot | patch 456952 - ironic - VirtualPDU use libvirt group instead of libvirtd | 14:28 |
| rloo | fbouliane: yeah, if it is a bug, would be good to know. is it ironic/devstack, or general devstack. do you know/think? | 14:28 |
| *** Goneri has joined #openstack-ironic | 14:29 | |
| *** causten has quit IRC | 14:29 | |
| *** mat128 has joined #openstack-ironic | 14:30 | |
| * milan relocates back :D | 14:31 | |
| *** rpioso has joined #openstack-ironic | 14:31 | |
| fbouliane | it would be related ironic usage of devstack so yeah ironic / devstack | 14:31 |
| *** milan has quit IRC | 14:31 | |
| fbouliane | the other libvirt change didn't have any bug though https://review.openstack.org/#/c/456476/6 | 14:32 |
| patchbot | patch 456476 - ironic - [Devstack] Fix libvirt group usage (MERGED) | 14:32 |
| fbouliane | I will file a bug for this, and amend my commit with the bug ID | 14:33 |
| *** aparnav has quit IRC | 14:33 | |
| fbouliane | Thanks ! | 14:33 |
| fbouliane | oops too late | 14:34 |
| dtantsur | fbouliane, still please file a bug, link the review to it, and close it right away | 14:35 |
| dtantsur | just for reference | 14:35 |
| fbouliane | perfect | 14:35 |
| *** baoli has quit IRC | 14:36 | |
| *** amotoki has quit IRC | 14:38 | |
| *** baoli has joined #openstack-ironic | 14:38 | |
| rloo | fbouliane: oh, a bug for what you just fixed would be great. i was asking about the bug you mentioned in the commit, which is why you had to hardcode libvirt instead of $LIBVIRT_GROUP | 14:38 |
| rloo | fbouliane: or did i misunderstand the commit msg? | 14:38 |
| fbouliane | No you are right, it should be a bug | 14:39 |
| *** amotoki has joined #openstack-ironic | 14:43 | |
| *** mvpnitesh has quit IRC | 14:46 | |
| *** phuongnh has joined #openstack-ironic | 14:50 | |
| *** amotoki has quit IRC | 14:53 | |
| rajinir | mjturek> The errors are related to libvirt installation | 14:57 |
| *** jcoufal_ has joined #openstack-ironic | 15:01 | |
| NobodyCam | Good Morning Ironic'ers | 15:03 |
| TheJulia | Good morning | 15:04 |
| NobodyCam | :) good morning TheJulia :) | 15:04 |
| mjturek | gm NobodyCam | 15:04 |
| *** jcoufal has quit IRC | 15:04 | |
| NobodyCam | morning mjturek :) | 15:04 |
| *** milan has joined #openstack-ironic | 15:05 | |
| mjturek | rajinir: FWIW our dsvm's have this qemu.conf file | 15:06 |
| rajinir | mjturek> hmm. we started seeing this recently. Not sure what changed | 15:07 |
| rajinir | mjturek: could be that it is part of your image? | 15:07 |
| mjturek | rajinir: I'll check, but my guess is that it comes from the qemu we install.. don't quote me tho | 15:07 |
| rajinir | mjturek: we install as in? | 15:07 |
| rajinir | mjturek: Can you share the elements of your image? | 15:10 |
| rajinir | https://www.irccloud.com/pastebin/2bQlxChX/Elements | 15:10 |
| *** amotoki has joined #openstack-ironic | 15:11 | |
| mjturek | rajinir: that's how you built the image used by your dsvm? (not user image for target node) | 15:12 |
| mjturek | rajinir: I'm not super familiar with how our dsvm image is built. We used the same one as our CI team's nova job. I'll ask them though | 15:13 |
| rajinir | mjturek: Yes , this is how we build it. Using the disk image builder. This is why I was saying each vendor has different ways and it is not standardized | 15:13 |
| rloo | dtantsur: btw, i was 'talking' to jjohnson2 last week. he didn't know that we had taken out ipminative driver. he wanted to know whether pyghmi should stay under openstack umbrella or not. TheJulia thinks should stay. Wanted to check with you. | 15:13 |
| rloo | dtantsur: http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2017-04-19.log.html#t2017-04-19T20:02:29 | 15:13 |
| dtantsur | morning NobodyCam | 15:13 |
| NobodyCam | Morning dtantsur :) | 15:14 |
| dtantsur | rloo, I'm good with staying, we still rely on it for vbmc | 15:14 |
| rloo | morning NobodyCam! (this is role, :D) | 15:14 |
| NobodyCam | Morning rloo :) | 15:14 |
| rloo | dtantsur: ok, good. i'll let him know. jjohnson2 ^^ | 15:14 |
| NobodyCam | lol!! | 15:14 |
| rloo | NobodyCam: I also just sent him email :) | 15:16 |
| *** chas has quit IRC | 15:16 | |
| NobodyCam | him? | 15:16 |
| alezil | Good Morning Ironic. | 15:17 |
| rloo | morning alezil! | 15:17 |
| NobodyCam | Morning alezil | 15:17 |
| alezil | dtantsur: aarefiev: Could you please help me merge this Patch https://review.openstack.org/#/c/396406/ | 15:18 |
| patchbot | patch 396406 - ironic-python-agent - Collect NIC name given by BIOS | 15:18 |
| rloo | NobodyCam: yeah, jjohnson2. I think I misinterpreted your previous comment :) | 15:18 |
| dtantsur | alezil, will try to | 15:19 |
| dtantsur | also morning | 15:19 |
| milan | morning alezil, NobodyCam, rloo and patchbot! :D | 15:19 |
| alezil | Before I move on from Ironic would like to merge it :( | 15:19 |
| * NobodyCam offers rloo coffee or tea :) | 15:19 | |
| alezil | Good Morning rloo, NobodyCam, milan | 15:20 |
| NobodyCam | morning milan :) | 15:20 |
| rloo | thx NobodyCam! :D | 15:20 |
| NobodyCam | hhehee :) | 15:20 |
| dtantsur | alezil, I understand your feeling, but I also wonder if anybody needs this change nowadays | 15:22 |
| alezil | dtantsur: It was a request for feature from operators who use Ironic. | 15:24 |
| dtantsur | ok | 15:24 |
| alezil | I knew all are busy, but will really appreciate if the patch could be reviewed. :) | 15:26 |
| aarefiev | alezil, dtantsur: it would be nice to have it | 15:26 |
| aarefiev | also code was good when I saw it last time | 15:26 |
| dtantsur | cool | 15:27 |
| mjturek | rajinir: asking for some info on the bug page | 15:28 |
| rajinir | mjturek: Will update soon | 15:29 |
| *** adisky_ has quit IRC | 15:29 | |
| *** udesale has quit IRC | 15:30 | |
| *** yolanda has quit IRC | 15:31 | |
| *** causten has joined #openstack-ironic | 15:32 | |
| *** Sukhdev has joined #openstack-ironic | 15:36 | |
| *** hamzy has quit IRC | 15:39 | |
| *** Sukhdev_ has joined #openstack-ironic | 15:41 | |
| rajinir | mjturek> updated | 15:45 |
| mjturek | rajinir: interesting | 15:47 |
| *** dyasny has quit IRC | 15:49 | |
| *** jaganathan has joined #openstack-ironic | 15:49 | |
| *** watanabe_isao has joined #openstack-ironic | 15:52 | |
| *** causten has quit IRC | 15:52 | |
| *** causten has joined #openstack-ironic | 15:53 | |
| *** gcb has joined #openstack-ironic | 15:53 | |
| rajinir | <mjturek> updated the etherpad with another problem that blocked devstack installation because of nova. Since there is a fix, we are fetching that patch and keeping the build going | 15:55 |
| *** gcb has quit IRC | 15:55 | |
| *** causten has quit IRC | 15:55 | |
| mjturek | thanks rajinir | 15:55 |
| *** causten has joined #openstack-ironic | 15:56 | |
| *** gcb has joined #openstack-ironic | 15:56 | |
| *** watanabe_isao has quit IRC | 15:56 | |
| rajinir | mjturek: I'm allowed to spend only 10-15% of my time to maintain the CI, but it has taken a lot of time when pike started | 16:00 |
| mjturek | when it rains it pours :) | 16:01 |
| *** dyasny has joined #openstack-ironic | 16:02 | |
| *** chas has joined #openstack-ironic | 16:03 | |
| openstackgerrit | Merged openstack/ironic master: TrivialFix: Remove logging import unused https://review.openstack.org/460038 | 16:04 |
| *** amarao has quit IRC | 16:04 | |
| *** nicodemos has quit IRC | 16:04 | |
| rajinir | mjturek: I added more issues from the recent past that I ve seen. Can you also add your issues there, like the wait for resources | 16:05 |
| *** xavierr has quit IRC | 16:05 | |
| *** chas_ has joined #openstack-ironic | 16:07 | |
| *** xavierr has joined #openstack-ironic | 16:08 | |
| openstackgerrit | Merged openstack/ironic master: VirtualPDU use libvirt group instead of libvirtd https://review.openstack.org/456952 | 16:09 |
| *** xavierr has quit IRC | 16:09 | |
| *** chas has quit IRC | 16:09 | |
| *** rbartal has quit IRC | 16:12 | |
| *** gcb has quit IRC | 16:14 | |
| *** xavierr has joined #openstack-ironic | 16:15 | |
| alezil | Thanks aarefiev: for review. | 16:16 |
| *** sambetts is now known as sambetts|afk | 16:18 | |
| sambetts|afk | night all | 16:18 |
| *** zackf has quit IRC | 16:19 | |
| *** honza has joined #openstack-ironic | 16:20 | |
| NobodyCam | night sambetts|afk | 16:20 |
| honza | Has anything changed recently in ironic that would cause CORS issues? I'm having a hard time tracking this down. | 16:20 |
| *** ijw has joined #openstack-ironic | 16:21 | |
| *** jaosorior_away is now known as jaosorior | 16:22 | |
| *** trown is now known as trown|lunch | 16:22 | |
| *** phuongnh has quit IRC | 16:23 | |
| *** livelace has quit IRC | 16:24 | |
| *** zackf has joined #openstack-ironic | 16:24 | |
| *** jaosorior has quit IRC | 16:26 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 16:27 | |
| *** jaosorior has joined #openstack-ironic | 16:27 | |
| *** jaosorior is now known as jaosorior_away | 16:27 | |
| TheJulia | honza: not that I'm aware, but if you have some more details that you can share as to what your seeing, we might be able to point you in the right direction. | 16:28 |
| *** causten has quit IRC | 16:28 | |
| *** zackf has quit IRC | 16:28 | |
| *** causten has joined #openstack-ironic | 16:29 | |
| honza | TheJulia: I'm trying to connec to the REST API from a tripleo undercloud (GUI), and there is an issue where the Access-* headers aren't being returned in the OPTIONS call | 16:29 |
| *** aarefiev is now known as aarefiev_afk | 16:30 | |
| TheJulia | honza: so, the question I would wonder is if CORS is enabled | 16:31 |
| honza | TheJulia: Is there some master switch? I see a bunch of settings in the [cors] section in /etc/ironic/ironic.conf | 16:32 |
| *** ijw has quit IRC | 16:32 | |
| *** causten has quit IRC | 16:33 | |
| *** mgoddard has quit IRC | 16:35 | |
| openstackgerrit | Merged openstack/ironic master: Read disk identifier after config drive setup https://review.openstack.org/458740 | 16:37 |
| *** ijw has joined #openstack-ironic | 16:37 | |
| *** fragatina has quit IRC | 16:38 | |
| honza | TheJulia: I tried adding some log statements, and it seems to reject the request because it finds a forbidden header: X-OpenStack-API-Ironic | 16:39 |
| TheJulia | So, you do need to configure the cores interface, the defaults don't work. I'd have to dig to see exactly what to set | 16:39 |
| *** lucasagomes is now known as lucas-afk | 16:40 | |
| jlvillal | honza: I'm sure you have heard this before, but since I have played Overwatch I think of Hanzo :) | 16:40 |
| honza | jlvillal: bahahaha | 16:40 |
| jlvillal | :) | 16:40 |
| *** ijw has quit IRC | 16:40 | |
| TheJulia | honza: https://github.com/openstack/bifrost/blob/master/playbooks/roles/bifrost-ironic-install/templates/ironic.conf.j2#L59 is what we did in bifrost and had some people use CORS against the api endpoint | 16:42 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: DO NOT MERGE: testing CI with systemd https://review.openstack.org/460076 | 16:43 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: [devstack] fix working with USE_SYSTEMD=True https://review.openstack.org/460216 | 16:43 |
| honza | TheJulia: looks about the same, we set the origin to * | 16:43 |
| TheJulia | So I don't think the core subsystem will allow that | 16:43 |
| TheJulia | cors | 16:44 |
| TheJulia | The person that immediately comes to mind, no longer works on openstack :( | 16:44 |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic master: Change assertTrue(isinstance()) with optimal assert for ironic https://review.openstack.org/362684 | 16:45 |
| *** deadnull has joined #openstack-ironic | 16:46 | |
| *** dprince has quit IRC | 16:46 | |
| *** dprince has joined #openstack-ironic | 16:46 | |
| Nisha_Agarwal | JayF, hi | 16:46 |
| *** derekh has quit IRC | 16:46 | |
| dtantsur | honza, jtomasek may have an idea about CORS, in case you haven't asked him already | 16:47 |
| honza | dtantsur: he's currently having the same issue --- thanks! | 16:47 |
| dtantsur | oops | 16:47 |
| honza | :) | 16:48 |
| *** Sukhdev_ has quit IRC | 16:48 | |
| *** tesseract has quit IRC | 16:51 | |
| jtomasek | dtantsur: :/ I know how to setup cors on ironic in case it would not be enabled in ironic.conf but that's about it | 16:52 |
| honza | Alright, more grepping for me :) | 16:53 |
| *** udesale has joined #openstack-ironic | 16:53 | |
| jlvillal | Anyone here know about: "cgroup_device_acl" ? | 16:54 |
| jlvillal | I'm reviewing: https://review.openstack.org/#/c/453541/4/devstack/lib/ironic And I don't know what that is | 16:54 |
| patchbot | patch 453541 - ironic - [Devstack] Rework VMs connection logic | 16:54 |
| *** udesale has quit IRC | 16:58 | |
| *** ijw has joined #openstack-ironic | 17:01 | |
| honza | I think I found it. | 17:02 |
| *** ijw has quit IRC | 17:05 | |
| *** fragatina has joined #openstack-ironic | 17:05 | |
| openstackgerrit | Félix Bouliane proposed openstack/ironic master: Make SNMP UDP transport settings configurable https://review.openstack.org/396400 | 17:05 |
| *** shuyingya has quit IRC | 17:07 | |
| honza | TheJulia: are you talking about Michael Krotscheck? I forget his IRC handle. | 17:09 |
| *** ijw has joined #openstack-ironic | 17:09 | |
| TheJulia | honza: I am referring to Michael | 17:10 |
| honza | :) | 17:10 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Allow hooks to have dependencies on other hooks https://review.openstack.org/455713 | 17:14 |
| dtantsur | see you tomorrow | 17:16 |
| *** ijw has quit IRC | 17:17 | |
| *** dtantsur is now known as dtantsur|afk | 17:17 | |
| *** mgoddard has joined #openstack-ironic | 17:19 | |
| jlvillal | dtantsur|afk: Ciao! | 17:20 |
| *** hamzy has joined #openstack-ironic | 17:20 | |
| *** baha1 has joined #openstack-ironic | 17:24 | |
| *** baha1 is now known as baha | 17:24 | |
| *** dsneddon is now known as dsneddon_biab | 17:26 | |
| *** jaganathan has quit IRC | 17:27 | |
| *** e0ne has quit IRC | 17:29 | |
| *** Serlex has quit IRC | 17:29 | |
| *** makowals_ has joined #openstack-ironic | 17:33 | |
| *** makowals has quit IRC | 17:34 | |
| *** athomas has quit IRC | 17:35 | |
| *** trown|lunch is now known as trown | 17:40 | |
| *** pcaruana has quit IRC | 17:40 | |
| *** jkilpatr_ has joined #openstack-ironic | 17:44 | |
| *** Sukhdev has quit IRC | 17:46 | |
| *** jkilpatr has quit IRC | 17:46 | |
| *** causten has joined #openstack-ironic | 17:48 | |
| NobodyCam | night dtantsur|afk | 17:51 |
| *** zackf has joined #openstack-ironic | 17:55 | |
| *** xavierr has quit IRC | 17:58 | |
| *** xavierr has joined #openstack-ironic | 18:03 | |
| *** xavierr has quit IRC | 18:05 | |
| openstackgerrit | Michael Turek proposed openstack/ironic master: Enable getting volume targets by their volume_id https://review.openstack.org/460250 | 18:07 |
| *** dsneddon_biab is now known as dsneddon | 18:09 | |
| *** baoli has quit IRC | 18:13 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: DO NOT MERGE: testing CI with systemd https://review.openstack.org/460076 | 18:13 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: [devstack] fix working with USE_SYSTEMD=True https://review.openstack.org/460216 | 18:13 |
| *** ralonsoh has quit IRC | 18:17 | |
| *** chas_ has quit IRC | 18:18 | |
| *** baoli has joined #openstack-ironic | 18:21 | |
| *** chas has joined #openstack-ironic | 18:21 | |
| *** xavierr has joined #openstack-ironic | 18:24 | |
| *** ijw has joined #openstack-ironic | 18:24 | |
| *** Nisha_Agarwal has quit IRC | 18:25 | |
| *** chas has quit IRC | 18:26 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 18:26 | |
| *** milan has quit IRC | 18:27 | |
| *** sirushti has quit IRC | 18:28 | |
| *** e0ne has joined #openstack-ironic | 18:28 | |
| *** ijw has quit IRC | 18:29 | |
| *** Nisha_Agarwal has quit IRC | 18:33 | |
| *** mgoddard has quit IRC | 18:35 | |
| *** jkilpatr_ has quit IRC | 18:37 | |
| *** jkilpatr has joined #openstack-ironic | 18:39 | |
| *** chas has joined #openstack-ironic | 18:41 | |
| *** baoli has quit IRC | 18:42 | |
| *** e0ne has quit IRC | 18:44 | |
| *** fragatina has quit IRC | 18:46 | |
| *** fragatina has joined #openstack-ironic | 18:46 | |
| *** chas has quit IRC | 18:47 | |
| *** baoli has joined #openstack-ironic | 18:48 | |
| *** ChubYann has joined #openstack-ironic | 19:00 | |
| *** causten has quit IRC | 19:00 | |
| *** causten has joined #openstack-ironic | 19:04 | |
| *** fragatina has quit IRC | 19:04 | |
| *** fragatina has joined #openstack-ironic | 19:04 | |
| *** baoli has quit IRC | 19:10 | |
| *** erolg has joined #openstack-ironic | 19:11 | |
| *** baoli has joined #openstack-ironic | 19:12 | |
| *** zackf has quit IRC | 19:18 | |
| *** racedo has quit IRC | 19:18 | |
| *** chas has joined #openstack-ironic | 19:18 | |
| *** racedo has joined #openstack-ironic | 19:20 | |
| *** causten has joined #openstack-ironic | 19:23 | |
| *** chas_ has joined #openstack-ironic | 19:23 | |
| *** erolg has quit IRC | 19:23 | |
| *** chas has quit IRC | 19:25 | |
| *** causten has quit IRC | 19:27 | |
| *** racedo has quit IRC | 19:29 | |
| *** deadnull has quit IRC | 19:29 | |
| *** ijw_ has joined #openstack-ironic | 19:31 | |
| *** ijw_ has quit IRC | 19:32 | |
| *** ijw_ has joined #openstack-ironic | 19:32 | |
| *** baoli has quit IRC | 19:34 | |
| *** ijw_ has quit IRC | 19:40 | |
| *** ijw has joined #openstack-ironic | 19:40 | |
| *** ijw has quit IRC | 19:44 | |
| *** causten has joined #openstack-ironic | 19:48 | |
| *** baoli has joined #openstack-ironic | 19:56 | |
| *** baoli has quit IRC | 19:59 | |
| *** baoli has joined #openstack-ironic | 19:59 | |
| *** dprince has quit IRC | 20:00 | |
| *** JasonF has joined #openstack-ironic | 20:02 | |
| *** ijw has joined #openstack-ironic | 20:02 | |
| *** JayF has quit IRC | 20:06 | |
| *** JasonF is now known as JayF | 20:07 | |
| *** shuyingya has joined #openstack-ironic | 20:08 | |
| *** ijw has quit IRC | 20:08 | |
| *** ijw has joined #openstack-ironic | 20:09 | |
| *** shuyingya has quit IRC | 20:12 | |
| *** dsneddon is now known as dsneddon_biab | 20:20 | |
| *** baha has quit IRC | 20:23 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Rolling upgrade procedure documentation https://review.openstack.org/416565 | 20:34 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Rolling upgrades related dev documentation https://review.openstack.org/419439 | 20:39 |
| *** Sukhdev has joined #openstack-ironic | 20:39 | |
| mariojv | jlvillal: thanks for taking over on the rescue stuff | 20:40 |
| mariojv | feel free to email or irc with any questions about it | 20:40 |
| *** crushil has quit IRC | 20:41 | |
| *** mat128 has quit IRC | 20:46 | |
| *** trown is now known as trown|outtypewww | 20:49 | |
| *** Goneri has quit IRC | 21:01 | |
| *** chas_ has quit IRC | 21:01 | |
| *** chas has joined #openstack-ironic | 21:01 | |
| *** mgoddard has joined #openstack-ironic | 21:04 | |
| *** chas has quit IRC | 21:06 | |
| *** ijw has quit IRC | 21:08 | |
| *** g3ek has quit IRC | 21:09 | |
| *** xavierr has quit IRC | 21:10 | |
| *** ijw has joined #openstack-ironic | 21:10 | |
| *** chas has joined #openstack-ironic | 21:11 | |
| *** chas has quit IRC | 21:13 | |
| *** chas has joined #openstack-ironic | 21:13 | |
| *** chas has quit IRC | 21:18 | |
| *** g3ek has joined #openstack-ironic | 21:18 | |
| *** zackf has joined #openstack-ironic | 21:18 | |
| *** mgoddard has quit IRC | 21:19 | |
| *** ijw has quit IRC | 21:21 | |
| jlvillal | mariojv: Oh hey. I just rebased it. I haven't found out yet if I will take over or not. | 21:22 |
| jlvillal | mariojv: But will absolutely ping you if I have questions! :) | 21:22 |
| *** zhenguo has quit IRC | 21:22 | |
| *** dyasny has quit IRC | 21:24 | |
| *** yolanda has joined #openstack-ironic | 21:25 | |
| *** causten has quit IRC | 21:27 | |
| *** causten has joined #openstack-ironic | 21:27 | |
| *** causten has quit IRC | 21:32 | |
| *** causten has joined #openstack-ironic | 21:44 | |
| *** srobert has quit IRC | 21:44 | |
| *** jcoufal_ has quit IRC | 21:44 | |
| *** causten has quit IRC | 21:47 | |
| *** chlong has quit IRC | 21:48 | |
| *** ijw has joined #openstack-ironic | 21:48 | |
| *** Goneri has joined #openstack-ironic | 21:49 | |
| *** jkilpatr has quit IRC | 21:51 | |
| *** bfournie has quit IRC | 21:55 | |
| *** dsneddon_biab is now known as dsneddon | 22:01 | |
| *** baoli has quit IRC | 22:05 | |
| *** marlinc has joined #openstack-ironic | 22:07 | |
| *** ijw has quit IRC | 22:10 | |
| *** ijw has joined #openstack-ironic | 22:14 | |
| *** causten has joined #openstack-ironic | 22:14 | |
| *** ijw has quit IRC | 22:18 | |
| *** ijw has joined #openstack-ironic | 22:20 | |
| *** jaypipes has quit IRC | 22:26 | |
| *** rpioso has quit IRC | 22:27 | |
| *** jkilpatr has joined #openstack-ironic | 22:32 | |
| *** bfournie has joined #openstack-ironic | 22:32 | |
| *** Goneri has quit IRC | 22:41 | |
| *** sasha21 has quit IRC | 23:00 | |
| *** ijw has quit IRC | 23:07 | |
| *** ijw has joined #openstack-ironic | 23:10 | |
| *** chas has joined #openstack-ironic | 23:18 | |
| *** chas has quit IRC | 23:22 | |
| *** Fdaisuke_ has quit IRC | 23:36 | |
| *** Sukhdev has quit IRC | 23:37 | |
| *** Sukhdev has joined #openstack-ironic | 23:42 | |
| openstackgerrit | Honza Pokorny proposed openstack/ironic master: Properly allow Ironic headers in REST API https://review.openstack.org/460353 | 23:54 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!