| *** Nisha_Agarwal has joined #openstack-ironic | 00:00 | |
| *** Nisha_ has joined #openstack-ironic | 00:02 | |
| *** Nisha_Agarwal has quit IRC | 00:02 | |
| *** Nisha_brb has joined #openstack-ironic | 00:04 | |
| *** Nisha_ has quit IRC | 00:04 | |
| *** Nisha_brb has quit IRC | 00:20 | |
| *** boris-42__ has quit IRC | 00:27 | |
| *** jaypipes has quit IRC | 00:31 | |
| *** vishwanathj has joined #openstack-ironic | 00:43 | |
| *** litao__ has joined #openstack-ironic | 00:52 | |
| *** gcb has quit IRC | 00:53 | |
| *** phuongnh has joined #openstack-ironic | 00:58 | |
| *** crushil has quit IRC | 01:15 | |
| *** crushil has joined #openstack-ironic | 01:15 | |
| *** rloo has quit IRC | 01:27 | |
| *** yufei has joined #openstack-ironic | 01:30 | |
| *** yufei has quit IRC | 01:32 | |
| *** harlowja has quit IRC | 01:34 | |
| *** namnh has joined #openstack-ironic | 01:34 | |
| *** yufei has joined #openstack-ironic | 01:36 | |
| *** rama_y_ has quit IRC | 01:38 | |
| *** shobhar__ has quit IRC | 01:39 | |
| *** shobhar__ has joined #openstack-ironic | 01:46 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/484079 | 01:46 |
|---|---|---|
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements https://review.openstack.org/483331 | 01:46 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-lib master: Updated from global requirements https://review.openstack.org/484506 | 01:47 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent master: Updated from global requirements https://review.openstack.org/484551 | 01:47 |
| *** baoli has joined #openstack-ironic | 01:48 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient master: Updated from global requirements https://review.openstack.org/484576 | 01:55 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-oneviewclient master: Updated from global requirements https://review.openstack.org/478085 | 01:56 |
| *** shobhar__ has quit IRC | 02:02 | |
| *** litao___ has joined #openstack-ironic | 02:06 | |
| *** mattcburns_ has joined #openstack-ironic | 02:06 | |
| *** mgkwill_ has joined #openstack-ironic | 02:06 | |
| *** samueldmq_ has joined #openstack-ironic | 02:06 | |
| *** patrickeast_ has joined #openstack-ironic | 02:06 | |
| *** ildikov_ has joined #openstack-ironic | 02:06 | |
| *** hogepodge_ has joined #openstack-ironic | 02:07 | |
| *** sai_ has joined #openstack-ironic | 02:13 | |
| *** litao__ has quit IRC | 02:13 | |
| *** mattcburns has quit IRC | 02:13 | |
| *** patrickeast has quit IRC | 02:13 | |
| *** ildikov has quit IRC | 02:13 | |
| *** sai has quit IRC | 02:13 | |
| *** rcarrillocruz has quit IRC | 02:13 | |
| *** kash has quit IRC | 02:13 | |
| *** hogepodge has quit IRC | 02:13 | |
| *** samueldmq has quit IRC | 02:13 | |
| *** mgkwill has quit IRC | 02:13 | |
| *** antonym has quit IRC | 02:13 | |
| *** litao___ is now known as litao__ | 02:13 | |
| *** mattcburns_ is now known as mattcburns | 02:13 | |
| *** ildikov_ is now known as ildikov | 02:13 | |
| *** patrickeast_ is now known as patrickeast | 02:14 | |
| *** samueldmq_ is now known as samueldmq | 02:14 | |
| *** mgkwill_ is now known as mgkwill | 02:14 | |
| *** hogepodge_ is now known as hogepodge | 02:14 | |
| *** kash has joined #openstack-ironic | 02:17 | |
| *** rcarrillocruz has joined #openstack-ironic | 02:19 | |
| *** antonym has joined #openstack-ironic | 02:19 | |
| *** gcb has joined #openstack-ironic | 02:19 | |
| *** antonym is now known as Guest73843 | 02:20 | |
| *** rama_y has joined #openstack-ironic | 02:24 | |
| *** shuangyangqian has joined #openstack-ironic | 02:38 | |
| *** baoli has quit IRC | 02:41 | |
| *** raginbajin has quit IRC | 02:41 | |
| *** shuangyangqian has quit IRC | 02:44 | |
| *** shuangyangqian has joined #openstack-ironic | 02:44 | |
| *** edleafe has quit IRC | 02:47 | |
| *** crushil has quit IRC | 02:51 | |
| *** crushil has joined #openstack-ironic | 02:52 | |
| *** edleafe has joined #openstack-ironic | 02:59 | |
| *** raginbajin has joined #openstack-ironic | 03:00 | |
| *** nicodemos has joined #openstack-ironic | 03:06 | |
| *** chlong has quit IRC | 03:11 | |
| *** rama_y has quit IRC | 03:15 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: Adds doc for restore_irmc_bios_config clean step https://review.openstack.org/483317 | 03:24 |
| *** udesale has joined #openstack-ironic | 03:30 | |
| *** harlowja has joined #openstack-ironic | 03:40 | |
| *** shuangyangqian has quit IRC | 03:42 | |
| *** tuanluong has joined #openstack-ironic | 03:46 | |
| *** aparna has joined #openstack-ironic | 03:47 | |
| *** mtanino_ has quit IRC | 03:47 | |
| *** shuangyangqian has joined #openstack-ironic | 03:52 | |
| *** links has joined #openstack-ironic | 03:55 | |
| *** shuangyangqian has quit IRC | 04:00 | |
| *** shuangyangqian has joined #openstack-ironic | 04:01 | |
| *** crushil has quit IRC | 04:02 | |
| *** yufei has quit IRC | 04:03 | |
| *** racedo has quit IRC | 04:11 | |
| *** racedo has joined #openstack-ironic | 04:13 | |
| *** fragatin_ has joined #openstack-ironic | 04:19 | |
| *** adisky__ has joined #openstack-ironic | 04:20 | |
| *** fragatin_ has quit IRC | 04:21 | |
| *** fragatin_ has joined #openstack-ironic | 04:21 | |
| *** fragatina has quit IRC | 04:23 | |
| *** sasha2 has quit IRC | 04:24 | |
| *** fragatin_ has quit IRC | 04:24 | |
| *** shuangyangqian has quit IRC | 04:25 | |
| *** shuangyangqian has joined #openstack-ironic | 04:26 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Allow setting interfaces to a node in available state https://review.openstack.org/482448 | 04:29 |
| *** hshiina has joined #openstack-ironic | 04:31 | |
| *** shuangyangqian has quit IRC | 04:34 | |
| *** shuangyangqian has joined #openstack-ironic | 04:34 | |
| openstackgerrit | Aparna proposed openstack/ironic-python-agent master: Rescue extension for CoreOS with DHCP tenant networks https://review.openstack.org/423521 | 04:35 |
| *** shuangyangqian has quit IRC | 04:37 | |
| *** shuangyangqian has joined #openstack-ironic | 04:37 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Allow setting interfaces to a node in available state https://review.openstack.org/482448 | 04:40 |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Allow setting interfaces to a node in available state https://review.openstack.org/482448 | 04:43 |
| *** shuangyangqian has quit IRC | 04:49 | |
| *** shuangyangqian has joined #openstack-ironic | 04:49 | |
| *** shuangyangqian has quit IRC | 04:50 | |
| *** shuangyangqian has joined #openstack-ironic | 04:51 | |
| *** shuangyangqian has quit IRC | 04:52 | |
| *** shuangyangqian has joined #openstack-ironic | 04:53 | |
| *** harlowja has quit IRC | 04:54 | |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Adds macs discovery https://review.openstack.org/483127 | 04:54 |
| *** shuangyangqian has quit IRC | 04:54 | |
| *** shuangyangqian has joined #openstack-ironic | 04:54 | |
| *** zzzeek has quit IRC | 04:55 | |
| *** yuanying has quit IRC | 04:58 | |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Redfish: Add the get_server_capabilities() and pci_gpu_device attribute https://review.openstack.org/481993 | 04:59 |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Redfish: Adds macs discovery https://review.openstack.org/483127 | 05:00 |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Redfish: Add the nic_capacity attributes https://review.openstack.org/481994 | 05:04 |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: DevStack: Add configuration for boot-from-volume https://review.openstack.org/466333 | 05:05 |
| *** jjung has joined #openstack-ironic | 05:09 | |
| *** bnemec has quit IRC | 05:13 | |
| *** fragatina has joined #openstack-ironic | 05:15 | |
| *** fragatina has quit IRC | 05:19 | |
| *** marios has joined #openstack-ironic | 05:22 | |
| *** rcernin has joined #openstack-ironic | 05:29 | |
| *** yufei has joined #openstack-ironic | 05:31 | |
| *** amarao has joined #openstack-ironic | 05:33 | |
| *** zzzeek has joined #openstack-ironic | 05:36 | |
| *** fragatina has joined #openstack-ironic | 05:37 | |
| *** zzzeek has quit IRC | 05:37 | |
| *** zzzeek has joined #openstack-ironic | 05:38 | |
| *** amarao has quit IRC | 05:40 | |
| *** marios has quit IRC | 05:42 | |
| *** mvpnitesh has joined #openstack-ironic | 05:43 | |
| *** marios has joined #openstack-ironic | 05:43 | |
| *** mjura has joined #openstack-ironic | 05:47 | |
| *** jjung has quit IRC | 05:56 | |
| *** dsneddon has quit IRC | 05:56 | |
| *** ccamacho|lunch has quit IRC | 05:59 | |
| *** dsneddon has joined #openstack-ironic | 06:03 | |
| *** jjung has joined #openstack-ironic | 06:09 | |
| *** shuangyangqian_ has joined #openstack-ironic | 06:09 | |
| *** shuangyangqian has quit IRC | 06:12 | |
| *** makowals has quit IRC | 06:13 | |
| openstackgerrit | fupingxie proposed openstack/ironic master: Replace http with https(Part-1) https://review.openstack.org/481516 | 06:14 |
| *** makowals has joined #openstack-ironic | 06:15 | |
| *** shuangyangqian_ has quit IRC | 06:16 | |
| *** shuangyangqian_ has joined #openstack-ironic | 06:16 | |
| *** zhangfei has joined #openstack-ironic | 06:16 | |
| *** fragatina has quit IRC | 06:26 | |
| *** fragatina has joined #openstack-ironic | 06:27 | |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Redfish: Add trusted_boot capability https://review.openstack.org/482789 | 06:31 |
| *** ChubYann has quit IRC | 06:36 | |
| *** ccamacho has joined #openstack-ironic | 06:45 | |
| vdrok | morning ironic! | 06:47 |
| *** makowals has quit IRC | 06:50 | |
| *** tesseract has joined #openstack-ironic | 07:18 | |
| *** jaganathan has joined #openstack-ironic | 07:24 | |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Redfish: Adds local_gb discovery https://review.openstack.org/483179 | 07:29 |
| *** yufei has quit IRC | 07:31 | |
| *** yufei has joined #openstack-ironic | 07:32 | |
| *** rbartal_ has joined #openstack-ironic | 07:33 | |
| *** yufei has quit IRC | 07:35 | |
| *** yufei has joined #openstack-ironic | 07:35 | |
| *** ralonsoh has joined #openstack-ironic | 07:41 | |
| openstackgerrit | Ankit Kumar proposed openstack/proliantutils master: Adds 'reset_bios_to_default' for redfish systems https://review.openstack.org/475785 | 07:47 |
| *** makowals has joined #openstack-ironic | 07:47 | |
| *** mvpnitesh has quit IRC | 07:57 | |
| *** gcb has quit IRC | 08:09 | |
| *** mgoddard_ has joined #openstack-ironic | 08:12 | |
| *** lucas-afk is now known as lucasagomes | 08:17 | |
| *** makowals has quit IRC | 08:22 | |
| *** makowals has joined #openstack-ironic | 08:24 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Improve ports API reference https://review.openstack.org/483015 | 08:25 |
| wong_ | morning | 08:27 |
| *** etingof has quit IRC | 08:27 | |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: WIP: Redfish: Discovers Storage related capabilities https://review.openstack.org/484669 | 08:30 |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: Switch to admin_token auth for standalone mode https://review.openstack.org/359061 | 08:43 |
| vdrok | dtantsur|afk: morning :) mind taking a look at ^^ if you'll have a bit of time. I think that would require a major version bump | 08:45 |
| *** etingof has joined #openstack-ironic | 08:47 | |
| openstackgerrit | Hangdong Zhang proposed openstack/python-ironicclient master: Update and optimize documentation links https://review.openstack.org/484679 | 08:55 |
| *** amarao has joined #openstack-ironic | 08:56 | |
| *** mvpnitesh has joined #openstack-ironic | 08:57 | |
| *** derekh has joined #openstack-ironic | 08:58 | |
| *** milan has joined #openstack-ironic | 09:01 | |
| milan | morning Ironic! :) | 09:01 |
| openstackgerrit | Merged openstack/ironic-ui master: Update URL home-page in documents according to document migration https://review.openstack.org/483658 | 09:01 |
| vdrok | good morning milan | 09:03 |
| *** aparnav has joined #openstack-ironic | 09:03 | |
| milan | morning vdrok! :) | 09:03 |
| *** appu103 has joined #openstack-ironic | 09:04 | |
| *** udesale__ has joined #openstack-ironic | 09:05 | |
| *** aparna has quit IRC | 09:06 | |
| openstackgerrit | Merged openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/481908 | 09:07 |
| *** aparnav has quit IRC | 09:07 | |
| *** udesale has quit IRC | 09:08 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 09:10 | |
| *** udesale has joined #openstack-ironic | 09:11 | |
| openstackgerrit | Aparna proposed openstack/proliantutils master: Add 'sriov_enabled' capability https://review.openstack.org/483458 | 09:11 |
| *** udesale__ has quit IRC | 09:13 | |
| Nisha_Agarwal | dtantsur|afk, hi | 09:18 |
| openstackgerrit | Merged openstack/python-ironicclient master: Updated from global requirements https://review.openstack.org/484576 | 09:24 |
| *** Nisha_ has joined #openstack-ironic | 09:24 | |
| *** Nisha_Agarwal has quit IRC | 09:24 | |
| *** udesale__ has joined #openstack-ironic | 09:29 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 09:30 | |
| *** Nisha_ has quit IRC | 09:30 | |
| *** udesale has quit IRC | 09:32 | |
| openstackgerrit | Merged openstack/proliantutils master: Redfish: Add 'refresh' method for HPEManager https://review.openstack.org/484264 | 09:32 |
| openstackgerrit | Mark Goddard proposed openstack/python-ironicclient master: Add physical network to port commands https://review.openstack.org/461893 | 09:32 |
| *** udesale__ has quit IRC | 09:36 | |
| *** udesale__ has joined #openstack-ironic | 09:36 | |
| *** udesale has joined #openstack-ironic | 09:40 | |
| *** makowals has quit IRC | 09:42 | |
| *** udesale__ has quit IRC | 09:42 | |
| *** Nisha_ has joined #openstack-ironic | 09:46 | |
| *** Nisha_Agarwal has quit IRC | 09:46 | |
| *** sambetts|afk is now known as sambetts | 09:46 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Updated from global requirements https://review.openstack.org/484551 | 09:46 |
| *** Dinesh_Bhor has quit IRC | 09:50 | |
| openstackgerrit | Merged openstack/ironic master: Updated from global requirements https://review.openstack.org/484079 | 09:52 |
| *** hshiina has quit IRC | 09:53 | |
| *** yufei has quit IRC | 09:57 | |
| openstackgerrit | Ankit Kumar proposed openstack/proliantutils master: Adds 'reset_ilo_credential' for redfish systems https://review.openstack.org/474191 | 09:57 |
| openstackgerrit | M V P Nitesh proposed openstack/ironic master: Allow setting {provisioning,cleaning}_network in node driver_info https://review.openstack.org/460564 | 09:58 |
| *** stendulker has joined #openstack-ironic | 10:03 | |
| stendulker | vdrok: Hi | 10:05 |
| openstackgerrit | Ankit Kumar proposed openstack/proliantutils master: Adds 'reset_ilo_credential' for redfish systems https://review.openstack.org/474191 | 10:07 |
| openstackgerrit | Aparna proposed openstack/proliantutils master: Redfish: Adds server capabilities https://review.openstack.org/484360 | 10:10 |
| *** gcb has joined #openstack-ironic | 10:11 | |
| *** jjung has quit IRC | 10:11 | |
| vdrok | hey stendulker | 10:13 |
| stendulker | vdrok: Trying to implement SNMP hardware type. There is no management interface for SNMP drivers | 10:14 |
| *** MattMan_1 has quit IRC | 10:15 | |
| stendulker | vdrok: Should we need to implement a management interface for the same? | 10:15 |
| *** MattMan_1 has joined #openstack-ironic | 10:15 | |
| *** ralonsoh has quit IRC | 10:15 | |
| *** udesale has quit IRC | 10:16 | |
| vdrok | stendulker: I'm not sure I remember, it might be using fake management, or was it WOL? I'll take a look when I'm back from lunch | 10:17 |
| stendulker | its marked as None | 10:18 |
| sambetts | stendulker: for an SNMP hardware type the management interface list would just be [noop] I think | 10:19 |
| vdrok | ++ | 10:19 |
| stendulker | vdrok: We do nohave noop management interface | 10:19 |
| stendulker | should I implement one? Or Should I implement SNMPManagement interface | 10:19 |
| vdrok | I think it should be a noop one | 10:20 |
| sambetts | yeah, I think one needs to be added like the others in ironic/drivers/modules/noop.py | 10:21 |
| stendulker | ok. I was confused as it supports only PXE boot device. so what do we return for get_supported_boot_devices() | 10:21 |
| stendulker | noop would return failure message for all calls. | 10:21 |
| *** Dinesh_Bhor has joined #openstack-ironic | 10:21 | |
| stendulker | that may not be right thing | 10:21 |
| stendulker | so is set_boot_device() | 10:22 |
| stendulker | I feel we may have to implement SNMPManagement that returns gracefuly for these calls. | 10:22 |
| stendulker | vdrok, sambetts : WDYT? | 10:23 |
| vdrok | I think noop should be the one just doing pass everywhere. | 10:24 |
| sambetts | hmmm ... the problem I see is that set_boot_device is a user facing action, so we'd have to make sure that we still return Unsupported if SNMP can't change the boot device | 10:24 |
| sambetts | which is why the current noop interfaces "fail" | 10:27 |
| *** hshiina has joined #openstack-ironic | 10:34 | |
| *** pc_m has quit IRC | 10:34 | |
| *** shuangyangqian_ has quit IRC | 10:36 | |
| *** phuongnh has quit IRC | 10:39 | |
| *** dtantsur|afk is now known as dtantsur | 10:41 | |
| dtantsur | morning ironic | 10:42 |
| dtantsur | vdrok: major version bump does not sound cool. tl;dr why? | 10:42 |
| vdrok | dtantsur: for ironic command when used in noauth mode, we change underlying implementation from httpclient to sessioclient. So error codes may change I think :( | 10:44 |
| *** jkilpatr has quit IRC | 10:46 | |
| *** pc_m has joined #openstack-ironic | 10:47 | |
| *** jjung has joined #openstack-ironic | 10:53 | |
| openstackgerrit | paresh sao proposed openstack/ironic master: Generic implementation for UEFI iSCSI boot for vendors https://review.openstack.org/308425 | 10:54 |
| dtantsur | vdrok: I guess the question is: what's the value of doing it one week before the final client release? | 10:55 |
| vdrok | To remove the httpclient class next release? :) | 10:55 |
| vdrok | Not much apart from that | 10:56 |
| dtantsur | vdrok: if we stop using it, we can remove it instantly | 10:56 |
| *** pc_m has quit IRC | 10:56 | |
| dtantsur | I don't think we expect too many downstream people to import httpclient (or am I wrong?) | 10:56 |
| vdrok | dtantsur: you mean after we remove the ironic command? | 10:57 |
| *** pc_m has joined #openstack-ironic | 10:57 | |
| dtantsur | vdrok: you said this patch is needed to remove httpclient, right? then why not remove httpclient in the same patch in early Queens? | 10:57 |
| *** Nisha_ has quit IRC | 10:58 | |
| vdrok | Because it may be used by someone | 10:58 |
| dtantsur | vdrok: in what way? shouldn't we just deprecate it then? | 10:59 |
| vdrok | I know as much as you do about its downstream usage :) | 10:59 |
| dtantsur | note that ironicclient.common.http is not a part of our public API | 10:59 |
| dtantsur | I'm just trying to understand why we need 2 breaking changes instead of one :) | 10:59 |
| vdrok | Yup, I deprecate that class in that patch. The only reason for major version bump is for switch between client classes implementation | 11:00 |
| vdrok | If we say it's not a public API, then I think we can remove right away | 11:00 |
| dtantsur | vdrok: it's something worth posting on the ML, but I think the httpclient is our implementation detail. the only public thing is error it raises - and you plan to change it right today | 11:01 |
| vdrok | Yup. Will do, thanks! | 11:01 |
| *** jistr is now known as jistr|tpb | 11:05 | |
| *** namnh has quit IRC | 11:05 | |
| *** jkilpatr has joined #openstack-ironic | 11:07 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 11:09 | |
| stendulker | dtantsur: Hi | 11:11 |
| *** makowals has joined #openstack-ironic | 11:11 | |
| stendulker | dtantsur: Implementing hardware type for SNMP driver. It does not have 'management' interface. It does not have a management interface https://github.com/openstack/ironic/blob/master/ironic/drivers/pxe.py#L128 | 11:13 |
| stendulker | The behavior of its management interface is same as FakeMangement interface https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/fake.py#L194-L215 | 11:14 |
| *** ralonsoh has joined #openstack-ironic | 11:14 | |
| stendulker | Can we use fake.management as a management interface for SNMP hardware type? | 11:14 |
| *** yufei has joined #openstack-ironic | 11:17 | |
| openstackgerrit | Milan Kováčik proposed openstack/python-ironic-inspector-client master: Introducing cli and contributor content https://review.openstack.org/484724 | 11:19 |
| *** zhangfei has quit IRC | 11:20 | |
| stendulker | sambetts, vdrok : We cannot give exception for set_boot_device() as its called by pxe boot interface unconditionally | 11:20 |
| *** lucasagomes is now known as lucas-hungry | 11:21 | |
| stendulker | We would have return silently if boot device is 'pxe'. | 11:21 |
| *** Nisha_ has joined #openstack-ironic | 11:21 | |
| stendulker | Else provisioning would fail | 11:21 |
| *** ralonsoh_ has joined #openstack-ironic | 11:24 | |
| *** ralonsoh has quit IRC | 11:24 | |
| openstackgerrit | Ankit Kumar proposed openstack/proliantutils master: Adds 'reset_bios_to_default' for redfish systems https://review.openstack.org/475785 | 11:25 |
| *** Nisha_Agarwal has quit IRC | 11:25 | |
| sambetts | stendulker: I guess you need a custom management interface that doesn't error if pxe is requested but returns unsupported for all other boot device types or something because ironic can't actually change the boot device so that would ensure that doing PXE boot followed by local boot would result in a fail | 11:26 |
| Nisha_ | dtantsur, hi | 11:27 |
| sambetts | stendulker: perhaps a "StaticallySetBootDevice" management interface would be a more generic way to put it | 11:27 |
| *** jistr|tpb is now known as jistr | 11:28 | |
| stendulker | sambetts: yes. But wht about SNMPManagement? | 11:28 |
| stendulker | sambetts: Would it be useful in any other scenario for it to be "StaticallySetBootDevice" | 11:29 |
| *** vmud213 has joined #openstack-ironic | 11:31 | |
| stendulker | sambetts: Upon second thought, "StaticallySetBootDevice" makes sense. Will add it. | 11:34 |
| stendulker | sambetts, vdrok, dtantsur: Thank you | 11:34 |
| openstackgerrit | Ankit Kumar proposed openstack/proliantutils master: Adds 'reset_bios_to_default' for redfish systems https://review.openstack.org/475785 | 11:36 |
| * etingof can help stendulker with pysnmp stuff whenever needed ;) | 11:37 | |
| *** stendulker has quit IRC | 11:39 | |
| *** appu103 has quit IRC | 11:40 | |
| *** fragatina has quit IRC | 11:42 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Allow updating interfaces on a node in available state https://review.openstack.org/482448 | 11:52 |
| TheJulia | Good morning | 11:53 |
| dtantsur | morning TheJulia | 11:55 |
| *** chlong has joined #openstack-ironic | 12:00 | |
| *** vmud213 has quit IRC | 12:01 | |
| openstackgerrit | Julia Kreger proposed openstack/python-ironicclient master: Update volume release notes to fix reno https://review.openstack.org/484493 | 12:01 |
| *** litao__ has quit IRC | 12:06 | |
| openstackgerrit | Milan Kováčik proposed openstack/python-ironic-inspector-client master: Introducing warning-is-error to docs https://review.openstack.org/484733 | 12:10 |
| *** hjensas has quit IRC | 12:12 | |
| *** tuanluong has quit IRC | 12:15 | |
| *** bfournie has quit IRC | 12:20 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent-builder master: Initial structure https://review.openstack.org/484735 | 12:21 |
| dtantsur | sambetts: let the fun begin ^^^ | 12:22 |
| sambetts | dtantsur: nice! thanks for kicking that off | 12:22 |
| dtantsur | np :) | 12:22 |
| nicodemos | morning ironic'ers, dtantsur, TheJulia | 12:22 |
| dtantsur | no real builders there for now, just the docs, reno and pep8 stuff | 12:22 |
| TheJulia | vdrok: You may just want to revise the nova attach/detach patch to remove the caching changes. Matt was up quite late last night, and it seems like we can remove the code. sambetts and dtantsur may want to toss in their $0.02 | 12:22 |
| dtantsur | TheJulia: I saw his comments, but my knowledge of Nova is lacking.. | 12:23 |
| TheJulia | I just thought about it for a couple minutes and went "how/where would the data ever be used such that the cache must be refreshed immediately" | 12:24 |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Redfish: Add the get_server_capabilities() and pci_gpu_device attribute https://review.openstack.org/481993 | 12:25 |
| TheJulia | and config drive upon rebuild likely doesn't count because I'm not sure we ever fixed the bug to allow a new configdrive to be submitted | 12:25 |
| TheJulia | Anyway, I need coffee | 12:25 |
| *** lucas-hungry is now known as lucasagomes | 12:25 | |
| dtantsur | milan: https://review.openstack.org/484735 | 12:26 |
| patchbot | patch 484735 - ironic-python-agent-builder - Initial structure | 12:26 |
| *** jjung has quit IRC | 12:29 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent-builder master: Initial structure https://review.openstack.org/484735 | 12:30 |
| *** dprince has joined #openstack-ironic | 12:34 | |
| *** rloo has joined #openstack-ironic | 12:36 | |
| *** yufei has quit IRC | 12:42 | |
| openstackgerrit | Julia Kreger proposed openstack/sushy master: Change theme to openstackdocs https://review.openstack.org/484465 | 12:44 |
| openstackgerrit | Julia Kreger proposed openstack/sushy master: Rework sushy documentation for doc migration https://review.openstack.org/484466 | 12:44 |
| *** mvpnitesh has quit IRC | 12:49 | |
| *** Goneri has joined #openstack-ironic | 12:49 | |
| *** sasha2 has joined #openstack-ironic | 12:50 | |
| *** jaypipes has joined #openstack-ironic | 12:51 | |
| *** ralonsoh_ is now known as ralonsoh | 12:52 | |
| *** ricardoas has quit IRC | 12:52 | |
| *** bfournie has joined #openstack-ironic | 12:53 | |
| *** trown|outtypewww is now known as trown | 12:54 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent-builder master: Initial structure https://review.openstack.org/484735 | 12:58 |
| *** udesale has joined #openstack-ironic | 13:08 | |
| *** hshiina has quit IRC | 13:11 | |
| *** jjung has joined #openstack-ironic | 13:12 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/proliantutils master: Redfish: Adds 'reset_ilo_credential' for redfish systems https://review.openstack.org/474191 | 13:12 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: BFV Deploy skip minor logging, logic, and test fixes https://review.openstack.org/479326 | 13:12 |
| *** baoli has joined #openstack-ironic | 13:15 | |
| *** jcoufal has joined #openstack-ironic | 13:17 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP/DNM:Temp patch to test boot-from-volume job https://review.openstack.org/476854 | 13:17 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Tempest scenario test for boot-from-volume https://review.openstack.org/472740 | 13:17 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP/DNM:Temp patch to test boot-from-volume job https://review.openstack.org/476854 | 13:17 |
| *** udesale has quit IRC | 13:27 | |
| *** udesale has joined #openstack-ironic | 13:28 | |
| *** rbudden has joined #openstack-ironic | 13:30 | |
| rloo | howdy and good morning TheJulia, dtantsur, and ironic'ers all over the world :) | 13:31 |
| TheJulia | Good morning! | 13:31 |
| dtantsur | hi rloo :) | 13:31 |
| rloo | dtantsur: qq, what do you mean by this comment? https://review.openstack.org/#/c/476779/8/ironic/objects/base.py | 13:31 |
| patchbot | patch 476779 - ironic - Modifications for rolling upgrades | 13:31 |
| rloo | dtantsur: did you want me to s/remove_unavail_fields/remove_unavailable_fields/ ? | 13:32 |
| *** ricardoas has joined #openstack-ironic | 13:32 | |
| milan | morning rloo, TheJulia! :) | 13:32 |
| openstackgerrit | Milan Kováčik proposed openstack/python-ironic-inspector-client master: Introducing warning-is-error to docs https://review.openstack.org/484733 | 13:33 |
| openstackgerrit | Milan Kováčik proposed openstack/python-ironic-inspector-client master: Introducing cli and contributor content https://review.openstack.org/484724 | 13:33 |
| dtantsur | rloo: I'd prefer to, yes | 13:34 |
| rloo | hi milan! | 13:34 |
| rloo | dtantsur: okay, will do follow up patch | 13:34 |
| openstackgerrit | Merged openstack/python-ironicclient master: Remove useless variables assignment in unit test https://review.openstack.org/482158 | 13:34 |
| dtantsur | thanks! | 13:34 |
| milan | dtantsur, just fixed your suggestion on the content headline ... | 13:35 |
| dtantsur | thanks! let us wait for the CI now | 13:35 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add new dbsync command with first online data migration https://review.openstack.org/408556 | 13:36 |
| milan | sambetts, TheJulia, guys, if you have some free time please check the docs inspector cli patchset, thanks a lot! | 13:37 |
| *** links has quit IRC | 13:38 | |
| *** vishwanathj has quit IRC | 13:38 | |
| *** jjung has quit IRC | 13:38 | |
| *** rodrigofm has joined #openstack-ironic | 13:39 | |
| *** vishwanathj has joined #openstack-ironic | 13:39 | |
| openstackgerrit | Julia Kreger proposed openstack/sushy master: Rework sushy documentation for doc migration https://review.openstack.org/484466 | 13:40 |
| *** udesale has quit IRC | 13:43 | |
| *** cdearborn has joined #openstack-ironic | 13:46 | |
| *** rama_y has joined #openstack-ironic | 13:46 | |
| *** hamzy has quit IRC | 13:47 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-ui master: Add python api docs to contributor docs https://review.openstack.org/480240 | 13:47 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Rename 'remove_unavail_fields' parameter https://review.openstack.org/484800 | 13:48 |
| *** crushil has joined #openstack-ironic | 13:48 | |
| TheJulia | milan: done :) | 13:51 |
| milan | TheJulia, nice! :) thx | 13:52 |
| * milan reviews | 13:52 | |
| vdrok | TheJulia: rloo morning | 13:53 |
| vdrok | TheJulia: will try to get to it today | 13:53 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add new dbsync command with first online data migration https://review.openstack.org/408556 | 13:54 |
| vdrok | (re nova attach patch) | 13:54 |
| rloo | hi vdrok! | 13:54 |
| TheJulia | vdrok: thanks! | 13:54 |
| TheJulia | I'm going to make another cup of coffee and step away for a few hours. Ping me if I'm needed | 13:55 |
| *** jjung has joined #openstack-ironic | 13:57 | |
| *** mtanino__ has joined #openstack-ironic | 13:57 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic master: [devstack] add support for running behind tls-proxy https://review.openstack.org/469836 | 13:59 |
| openstackgerrit | Merged openstack/sushy master: Change theme to openstackdocs https://review.openstack.org/484465 | 13:59 |
| *** persia__ is now known as persia | 14:01 | |
| xavierr | hey vdrok, may you review https://review.openstack.org/#/c/482209/ ? :) | 14:05 |
| patchbot | patch 482209 - ironic - Removes agent mixin from oneview drivers | 14:05 |
| *** baoli has quit IRC | 14:06 | |
| vdrok | xavierr: no promises but I'll try | 14:06 |
| vdrok | also morning :) | 14:06 |
| *** baoli has joined #openstack-ironic | 14:07 | |
| xavierr | morning vdrok o/ | 14:09 |
| xavierr | and ty! | 14:09 |
| *** Guest73843 is now known as antonym | 14:11 | |
| *** antonym has quit IRC | 14:11 | |
| *** antonym has joined #openstack-ironic | 14:11 | |
| *** jheroux has joined #openstack-ironic | 14:12 | |
| openstackgerrit | Merged openstack/ironic-lib master: Updated from global requirements https://review.openstack.org/484506 | 14:14 |
| *** fragatina has joined #openstack-ironic | 14:14 | |
| *** fragatina has quit IRC | 14:15 | |
| *** fragatina has joined #openstack-ironic | 14:16 | |
| *** zhangfei has joined #openstack-ironic | 14:18 | |
| *** bnemec has joined #openstack-ironic | 14:25 | |
| *** zhangfei has quit IRC | 14:30 | |
| *** hamzy has joined #openstack-ironic | 14:32 | |
| *** rama_y has quit IRC | 14:38 | |
| *** jaganathan has quit IRC | 14:42 | |
| *** zhangfei has joined #openstack-ironic | 14:44 | |
| *** rbartal_ has quit IRC | 14:49 | |
| *** keerthi has joined #openstack-ironic | 14:52 | |
| keerthi | facing scheduling issues in Ironic | 14:53 |
| keerthi | scheduling always happen only 10 at a time | 14:54 |
| keerthi | can anyone explain why it is happening ? | 14:54 |
| *** krtaylor has quit IRC | 14:55 | |
| *** trown is now known as trown|brb | 14:56 | |
| keerthi | ?? | 14:57 |
| dtantsur | keerthi: this is the nova default | 14:58 |
| sambetts | keerthi: scheduling is handle by nova, there is a setting in nova max_concurrent_builds | 14:58 |
| sambetts | that defaults to 10 | 14:58 |
| keerthi | max_concurrent_builds is only for per compute node right ? | 14:58 |
| keerthi | concurrent builds only on the single compute node right | 14:59 |
| sambetts | yes, so you can either increase that setting for the compute node running the ironic driver, or scale up your nova compute nodes running the ironic driver | 14:59 |
| keerthi | changed the compute_node setting...still no effect | 15:00 |
| *** causten has joined #openstack-ironic | 15:00 | |
| sambetts | did you restart the nova compute? | 15:01 |
| keerthi | yes sambetts | 15:01 |
| keerthi | adding this max_concurrent_builds=20 in nova.conf and restarted no effect | 15:02 |
| keerthi | is there any other we can solve this issue ? | 15:02 |
| *** milan has quit IRC | 15:03 | |
| dtantsur | I think you should restart the scheduler, not the compute | 15:04 |
| dtantsur | otherwise the nova folks on #openstack-nova may help | 15:04 |
| dtantsur | keerthi: ^^ | 15:04 |
| sambetts | you might do better to ask in the the openstack-nova channel if the nova-compute isn't using the config option correctly | 15:04 |
| keerthi | Restarted nova-scheduler too...let me check in other channel. Thanks | 15:05 |
| *** aparnav has joined #openstack-ironic | 15:07 | |
| *** jjung has quit IRC | 15:09 | |
| *** vinsh has joined #openstack-ironic | 15:11 | |
| *** aparnav has quit IRC | 15:11 | |
| *** aparnav has joined #openstack-ironic | 15:12 | |
| jlvillal | TheJulia, congrats, I see a +2 on the Nova patch for BFV :) | 15:13 |
| TheJulia | \o/ | 15:14 |
| dtantsur | \o/ | 15:14 |
| dtantsur | now eagerly hunt for another one | 15:14 |
| dtantsur | :) | 15:14 |
| *** rpioso has joined #openstack-ironic | 15:15 | |
| *** rama_y has joined #openstack-ironic | 15:15 | |
| jlvillal | dtantsur, There is that Red Hat guy who lives in Portland ;) | 15:15 |
| rloo | jlvillal, dtantsur: oh, i dunno about that red hat guy. he's tough :) | 15:16 |
| jlvillal | +1 | 15:16 |
| dtantsur | lol | 15:20 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: WIP: Experiment with decorators & keyword arguments https://review.openstack.org/253849 | 15:22 |
| *** trown|brb is now known as trown | 15:25 | |
| *** krtaylor has joined #openstack-ironic | 15:28 | |
| *** dyasny has quit IRC | 15:30 | |
| *** dyasny has joined #openstack-ironic | 15:31 | |
| *** rbudden_ has joined #openstack-ironic | 15:32 | |
| *** rcernin has quit IRC | 15:32 | |
| *** vishwana_ has joined #openstack-ironic | 15:33 | |
| *** rbudden has quit IRC | 15:34 | |
| *** rbudden_ is now known as rbudden | 15:34 | |
| *** dyasny has quit IRC | 15:37 | |
| *** dyasny has joined #openstack-ironic | 15:37 | |
| *** vishwanathj has quit IRC | 15:37 | |
| *** dyasny has quit IRC | 15:43 | |
| *** dyasny_ has joined #openstack-ironic | 15:43 | |
| *** crushil has quit IRC | 15:43 | |
| *** baoli has quit IRC | 15:44 | |
| TheJulia | valid -1 on the nova bfv patch. I just replied suggesting we just yank the fibrechannel support matrix note of partial and indicated that we are likely to ask for that to be changed next cycle with no code actually being changed in nova. | 15:44 |
| *** dyasny__ has joined #openstack-ironic | 15:45 | |
| *** cdearborn has quit IRC | 15:45 | |
| *** rpioso has quit IRC | 15:45 | |
| *** mjturek has joined #openstack-ironic | 15:46 | |
| *** mguz_ has joined #openstack-ironic | 15:47 | |
| *** dyasny_ has quit IRC | 15:47 | |
| *** vmud213 has joined #openstack-ironic | 15:48 | |
| *** baoli has joined #openstack-ironic | 15:49 | |
| *** dyasny__ has quit IRC | 15:50 | |
| *** baoli has quit IRC | 15:51 | |
| *** crushil has joined #openstack-ironic | 15:52 | |
| *** baoli has joined #openstack-ironic | 15:53 | |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: rearrange existing documentation to follow the new layout https://review.openstack.org/484343 | 15:54 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient master: [WIP] Log warning when API version is not specified for the ironic tool https://review.openstack.org/484866 | 15:56 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: rearrange existing documentation to follow the new layout https://review.openstack.org/484343 | 16:00 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: rearrange existing documentation to follow the new layout https://review.openstack.org/484343 | 16:01 |
| *** vishwana_ has quit IRC | 16:01 | |
| dtantsur | easy win for someone's commit stats (etingof?): we need to fix the docs URL in https://github.com/openstack/sushy/blob/master/README.rst | 16:01 |
| dtantsur | there are still 2 linkis to rtfd | 16:02 |
| TheJulia | :) | 16:02 |
| *** aparnav has quit IRC | 16:02 | |
| *** vishwanathj has joined #openstack-ironic | 16:02 | |
| *** rcernin has joined #openstack-ironic | 16:02 | |
| etingof | dtantsur, taken | 16:02 |
| etingof | ;-) | 16:02 |
| dtantsur | thanks! | 16:02 |
| *** baoli has quit IRC | 16:02 | |
| *** rbudden has quit IRC | 16:05 | |
| *** rbudden has joined #openstack-ironic | 16:06 | |
| openstackgerrit | Merged openstack/ironic-ui master: Introduce form-field directive https://review.openstack.org/480201 | 16:06 |
| dtantsur | bailing out, see you tomorrow | 16:07 |
| *** dtantsur is now known as dtantsur|afk | 16:07 | |
| TheJulia | goodnight dtantsur|afk | 16:08 |
| *** zhangfei has quit IRC | 16:09 | |
| openstackgerrit | Ilya Etingof proposed openstack/sushy master: fixed hosted documentation links https://review.openstack.org/484877 | 16:11 |
| openstackgerrit | Julia Kreger proposed openstack/sushy master: Rework sushy documentation for doc migration https://review.openstack.org/484466 | 16:12 |
| *** baoli has joined #openstack-ironic | 16:12 | |
| *** jcoufal has quit IRC | 16:13 | |
| NobodyCam | Good Morning Ironic'ers | 16:14 |
| *** ccamacho has quit IRC | 16:15 | |
| *** dyasny__ has joined #openstack-ironic | 16:20 | |
| *** vishwanathj has quit IRC | 16:22 | |
| *** Nisha_ has quit IRC | 16:24 | |
| *** Nisha_ has joined #openstack-ironic | 16:24 | |
| jlvillal | Hi NobodyCam :) | 16:24 |
| NobodyCam | Good Morning jlvillal :) | 16:24 |
| mgoddard_ | rloo: hi | 16:25 |
| edleafe | OK, I've gotten my Ironic devstack running and created my first instance. In the ironic driver for nova, I call self._get_node_list(), and get back one item. So far so good. Problem is that that node doesn't have a 'resource_class' attribute. | 16:25 |
| rloo | hi mgoddard_ | 16:25 |
| rloo | morning NobodyCam, jlvillal | 16:25 |
| mgoddard_ | rloo: what are your plans for https://review.openstack.org/#/c/477694? | 16:26 |
| patchbot | patch 477694 - ironic - Rolling upgrades support for port.physical_network | 16:26 |
| NobodyCam | Good morning rloo :) | 16:26 |
| TheJulia | edleafe: what api micro version are you using? | 16:26 |
| mgoddard_ | rloo: I'm happy to pick it up as I look at other upgrades pieces, just didn't want to tread on your toes | 16:26 |
| rloo | mgoddard_: oh. i forgot about that. although i did remember it for a sec. i think i was thinking i should look at it again, but that you could probably take over/roll some of that into your current api patch | 16:27 |
| edleafe | TheJulia: dunno. Whatever the default is, I guess :) | 16:27 |
| rloo | mgoddard_: sec, let me take a look | 16:27 |
| TheJulia | edleafe: so 1.9 can't see resource_class | 16:27 |
| *** amarao has quit IRC | 16:28 | |
| rloo | edleafe: are you calling self._get_node_list() from the ironic driver in nova? | 16:28 |
| edleafe | In the nova driver, the call is: node_list = self.ironicclient.call("node.list", **kwargs) | 16:28 |
| *** ricardoas has quit IRC | 16:28 | |
| TheJulia | edleafe: set to "latest" or "1.21" | 16:29 |
| TheJulia | 1.21 is where we added resource_class | 16:29 |
| TheJulia | hmm | 16:29 |
| TheJulia | oh | 16:29 |
| TheJulia | I don't think it is returned on a normal node list | 16:30 |
| edleafe | so adding something like 'version=1.21' to the ironicclient call? | 16:30 |
| rloo | edleafe: you are using master code right? | 16:30 |
| edleafe | rloo: yeah | 16:30 |
| rloo | TheJulia: master is set to use 1.29 | 16:30 |
| *** cdearborn has joined #openstack-ironic | 16:30 | |
| TheJulia | edleafe: initialization of the client, but if your in nova code doing this, you should be at a much higher version | 16:30 |
| TheJulia | hmm | 16:30 |
| rloo | TheJulia: we may have in the ironic driver, specified the set of fields to return from a get_node_list(). don't remember | 16:30 |
| TheJulia | yeah, so I bet we just don't reply in a standard node list | 16:31 |
| rloo | edleafe: you have the line in the code where it is being called? | 16:31 |
| *** rpioso has joined #openstack-ironic | 16:31 | |
| edleafe | rloo: it's code I added to the init_host of the ironic driver.py | 16:32 |
| edleafe | rloo: node_list = self._get_node_list(associated=True, limit=0) | 16:32 |
| rloo | edleafe: ok, the 'normal' response for that doesn't return the resource_class | 16:32 |
| rloo | edleafe: you can add detail=True to get *everything* about the node | 16:32 |
| edleafe | rloo: ok, let me try that | 16:32 |
| rloo | edleafe: or fields=<list of field names> | 16:32 |
| TheJulia | heh, what rloo said, I was just double checking it | 16:34 |
| *** marios has quit IRC | 16:34 | |
| edleafe | ok, detail=True returned the resource_class | 16:35 |
| rloo | mgoddard_: you can take over that patch. there are two patches (one waiting to merge, the other i just submitted) that will affect the port.py/_convert_to_version() | 16:35 |
| rloo | mgoddard_: https://review.openstack.org/484800 | 16:35 |
| patchbot | patch 484800 - ironic - Rename 'remove_unavail_fields' parameter | 16:35 |
| rloo | mgoddard_: or base it on top of https://review.openstack.org/#/c/476779/ | 16:36 |
| patchbot | patch 476779 - ironic - Modifications for rolling upgrades | 16:36 |
| rloo | mgoddard_: depends on how much of a hurry you are in :-) | 16:36 |
| rloo | mgoddard_: so as not to block your physical-network-api patch, i'm fine if you just add in that little bit of rolling upgrade. | 16:37 |
| *** derekh has quit IRC | 16:37 | |
| *** cdearborn__ has joined #openstack-ironic | 16:38 | |
| edleafe | good news: I'm getting back the node's resource_class | 16:43 |
| edleafe | bad news: it's None | 16:43 |
| edleafe | How is that set? | 16:43 |
| edleafe | (in ironic) | 16:43 |
| rloo | edleafe: heh. you'll have to be an ironic admin, and set it | 16:44 |
| rloo | edleafe: ironic node-update <node uuid> something, i forgot the syntax. | 16:44 |
| rloo | edleafe: there's also 'openstack baremetal node set' commnd | 16:44 |
| rloo | edleafe: or REST API, but CLI is easier :) | 16:45 |
| edleafe | rloo: ok, thanks | 16:45 |
| edleafe | rloo: but if I create a new node, is there a way to get that populated based on flavor/hardware? | 16:45 |
| *** causten_ has joined #openstack-ironic | 16:46 | |
| rloo | edleafe: no, don't think so. in ironic, it doesn't know anything about nova flavor. if that is what you are asking. | 16:46 |
| rloo | edleafe: maybe ironic inspector can do that but i don't know if it does it now. | 16:47 |
| edleafe | I understand that. What I'm not sure of is what change was made in ironic in Ocata? Just adding the resource_class field, or populating it in some way? | 16:47 |
| *** causten has quit IRC | 16:48 | |
| edleafe | rloo: my goal is to get that class from ironic, and use it to populate both the nova flavor and the placement inventory | 16:48 |
| mgoddard_ | rloo: thanks. I'll take over your patch | 16:48 |
| rloo | edleafe: just adding resource_class | 16:48 |
| * dtantsur|afk shows edleafe https://review.openstack.org/476968 and disappears again | 16:48 | |
| patchbot | patch 476968 - ironic - [devstack] use resource classes by default | 16:48 |
| rloo | mgoddard_: thx! | 16:49 |
| *** mjura has quit IRC | 16:49 | |
| rloo | thx to the invisible but lurking dtantsur|afk | 16:49 |
| edleafe | rloo: so is the expectation that nova will push that info to ironic? | 16:49 |
| sambetts | no, the operator populates that info | 16:50 |
| rloo | edleafe: i have no idea, but I don't think so. (I guess i have a bit of an idea). | 16:50 |
| rloo | edleafe: what sambetts said | 16:51 |
| sambetts | nova consumes it | 16:51 |
| sambetts | for scheduling | 16:51 |
| *** trown is now known as trown|lunch | 16:51 | |
| edleafe | sambetts: how/when does the operator populate the info? | 16:52 |
| edleafe | sambetts: I know that they are supposed to add the class to the nova flavor extra_specs | 16:52 |
| rloo | edleafe: before they make the node available | 16:52 |
| sambetts | when they enroll the node into ironic | 16:52 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: change get_target_version() to class method https://review.openstack.org/484890 | 16:52 |
| edleafe | sambetts: so devstack doesn't do it, but a normal operator would? | 16:52 |
| sambetts | edleafe: currently the nova side of resource class isn't finished so we don't use it in devstack | 16:53 |
| *** cdearborn has quit IRC | 16:53 | |
| edleafe | sambetts: heh, that's what I'm trying to do: finish the nova side of things | 16:53 |
| edleafe | sambetts: and since I don't have a real-world ironic deployment to play with... | 16:54 |
| sambetts | ah ok, yeah so the idea would be that the node's resource class would be provided/set by the operator instead of/at the same point as the cpu/ram/disk is set on the ironic node today | 16:55 |
| rloo | edleafe: don't know if this helps or not, but here's a bit of our install doc: https://docs.openstack.org/ironic/latest/install/enrollment.html | 16:55 |
| jlvillal | rloo, Do you remember that patch you had me -2? | 16:55 |
| rloo | jlvillal: no | 16:55 |
| rloo | jlvillal: it was tied to an RFE if i recall | 16:55 |
| rloo | jlvillal: for oneview? | 16:55 |
| jlvillal | rloo, Okay, you had asked me to -2 a patch while you were away. I was going to remove it and let you -2 it :) | 16:55 |
| edleafe | sambetts: rloo: thanks for your help. I think I have enough to hack away | 16:56 |
| jlvillal | But I forget what it was. | 16:56 |
| jlvillal | I'll see if I can figure out how to search for patches I voted -2 on. | 16:56 |
| rloo | jlvillal: i'm fine if you keep the -2, you're around for the next few months or years? :) | 16:56 |
| jlvillal | heh. Months yes, I think. Not sure about years :) | 16:56 |
| rloo | edleafe: yw :) | 16:56 |
| sambetts | edleafe: no problem thanks for working on that | 16:56 |
| * sambetts --> home | 16:57 | |
| sambetts | Night all! o/ | 16:57 |
| rloo | lunch break, back in a bit | 16:57 |
| rloo | night sambetts! | 16:57 |
| *** sambetts is now known as sambetts|afk | 16:57 | |
| sambetts|afk | cya rloo | 16:57 |
| jlvillal | TheJulia, I just put a -2 on https://review.openstack.org/#/c/472740/ I could probably be talked out of it if really necessary. | 16:59 |
| patchbot | patch 472740 - ironic - Tempest scenario test for boot-from-volume | 16:59 |
| jlvillal | TheJulia, But I think we need to get an experimental job first so that we can actually test the tempest code. | 16:59 |
| *** ralonsoh has quit IRC | 17:00 | |
| *** etingof has quit IRC | 17:03 | |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Redfish: Add the nic_capacity attributes https://review.openstack.org/481994 | 17:05 |
| *** hjensas has joined #openstack-ironic | 17:05 | |
| *** hjensas has quit IRC | 17:05 | |
| *** hjensas has joined #openstack-ironic | 17:05 | |
| *** baoli has quit IRC | 17:06 | |
| *** baoli has joined #openstack-ironic | 17:06 | |
| *** jistr is now known as jistr|off | 17:06 | |
| TheJulia | I could have sworn there was a patch proposed for the experimental pipeline someplace :( | 17:08 |
| TheJulia | then again, I know there has been resistance to adding jobs | 17:09 |
| jlvillal | TheJulia, https://review.openstack.org/#/c/480332/ which seems to be a copy of https://review.openstack.org/#/c/472740/ | 17:10 |
| patchbot | patch 480332 - openstack-infra/project-config - Add an experimental job to test ironic boot from v... | 17:10 |
| patchbot | patch 472740 - ironic - Tempest scenario test for boot-from-volume | 17:10 |
| *** keerthi has quit IRC | 17:10 | |
| TheJulia | jlvillal: you sure you have the right links? | 17:15 |
| jlvillal | TheJulia, first link is correct | 17:16 |
| jlvillal | TheJulia, second link should have been: https://review.openstack.org/#/c/470307/ | 17:17 |
| patchbot | patch 470307 - openstack-infra/project-config - WIP: Add job to test ironic boot from volume | 17:17 |
| *** ChubYann has joined #openstack-ironic | 17:20 | |
| *** baoli has quit IRC | 17:22 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic master: [TEST] [DO NOT MERGE] Test tls-proxy with IPA https://review.openstack.org/469837 | 17:22 |
| *** baoli has joined #openstack-ironic | 17:22 | |
| *** harlowja has joined #openstack-ironic | 17:24 | |
| TheJulia | jlvillal: I feel like a -2 to block the gate patch is going a little far at the moment, but it does also make sense. I would just expect you will be ready to remove your -2 it once the experimental job lands? Of course, so much chicken meet egg kind of stuff right now :( | 17:24 |
| jlvillal | TheJulia, Of course I would remove when the experimental job lands. Or if the patch is changed to depend on the experimental job. | 17:25 |
| TheJulia | jlvillal: If you could, review https://review.openstack.org/#/c/484493/ so we can begin to remedy some of it | 17:25 |
| patchbot | patch 484493 - python-ironicclient - Update volume release notes to fix reno | 17:25 |
| TheJulia | jlvillal: Awesome | 17:25 |
| jlvillal | TheJulia, Otherwise we really have no idea if it works or not. | 17:25 |
| *** lucasagomes is now known as lucas-afk | 17:25 | |
| TheJulia | well, you can't actually depends-on a project-config repo change other than to block merging | 17:25 |
| openstackgerrit | Nisha Agarwal proposed openstack/sushy master: Adds EthernetInterface to the library https://review.openstack.org/452532 | 17:25 |
| jlvillal | TheJulia, Right. But I mean if we merge the tempest change now, we have no idea if it works or not. | 17:26 |
| TheJulia | jlvillal: Agreed, and I'm not objecting in the slightest | 17:26 |
| TheJulia | more than anything, I'm confused | 17:26 |
| jlvillal | TheJulia, Thanks | 17:26 |
| TheJulia | But confusion may also be a normal state | 17:27 |
| * TheJulia expects physics jokes from here on out | 17:27 | |
| jlvillal | TheJulia, I just did an update to: https://review.openstack.org/480332 | 17:34 |
| patchbot | patch 480332 - openstack-infra/project-config - Add an experimental job to test ironic boot from v... | 17:34 |
| TheJulia | I saw it pop up :) | 17:35 |
| TheJulia | oh, you actually revised it | 17:35 |
| *** vmud213 has quit IRC | 17:35 | |
| TheJulia | Well, python-ironicclient 1.15.0 is basically blocked on the reno fix in https://review.openstack.org/#/c/484493/ which blocks the devstack change from actually being able to be merged. | 17:39 |
| patchbot | patch 484493 - python-ironicclient - Update volume release notes to fix reno | 17:39 |
| openstackgerrit | Nisha Agarwal proposed openstack/sushy master: Add local_gb discovery to sushy https://review.openstack.org/455114 | 17:39 |
| jlvillal | TheJulia, Does that mean I should put it at the top of my queue? | 17:39 |
| TheJulia | please :) | 17:40 |
| jlvillal | Looking now | 17:40 |
| TheJulia | It is just a reno fix :) | 17:40 |
| * jlvillal wonder how to spell hexadecimal or is it hexidecimal | 17:40 | |
| TheJulia | ohhhh good one! | 17:41 |
| TheJulia | hexa it looks like | 17:41 |
| JayF | jlvillal: it's spelled "h" "e" "x" | 17:41 |
| JayF | :D | 17:41 |
| jlvillal | heh :) | 17:42 |
| jlvillal | TheJulia, Is there a link to what it looks like now and how it is wrong? | 17:42 |
| TheJulia | common misspelling, enough that google autocorrects it | 17:42 |
| TheJulia | http://docs-draft.openstack.org/93/484493/5/check/gate-python-ironicclient-releasenotes/2c45e1e//releasenotes/build/html/unreleased.html | 17:42 |
| jlvillal | Found it, I think https://docs.openstack.org/releasenotes/python-ironicclient/ | 17:43 |
| TheJulia | yup | 17:43 |
| jlvillal | TheJulia, I guess I don't understand what is changed when I look at the release notes | 17:43 |
| TheJulia | look at 1.14.0 on the docs.openstack.org site | 17:44 |
| TheJulia | then compare to docs-draft change | 17:44 |
| *** mgoddard_ has quit IRC | 17:44 | |
| jlvillal | TheJulia, Ah :) Thanks! | 17:44 |
| jlvillal | TheJulia, +2 +A Thanks. | 17:44 |
| * TheJulia dances | 17:46 | |
| edleafe | rloo: TheJulia: update- using openstack baremetal node set, I was able to set the resource class on the nodes, and it is showing up in Nova. Thanks for getting me this far! | 17:51 |
| rloo | edleafe: yay! thx for fixing nova so it works with ironic :) | 17:52 |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Redfish: Add trusted_boot capability https://review.openstack.org/482789 | 17:52 |
| TheJulia | edleafe: \o/ | 17:55 |
| *** tesseract has quit IRC | 17:58 | |
| *** baoli has quit IRC | 18:00 | |
| *** baoli has joined #openstack-ironic | 18:01 | |
| *** Goneri has quit IRC | 18:07 | |
| *** deep-book-gk_ has joined #openstack-ironic | 18:09 | |
| *** deep-book-gk_ has left #openstack-ironic | 18:09 | |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Redfish: Adds macs discovery https://review.openstack.org/483127 | 18:13 |
| *** Nisha_ has quit IRC | 18:19 | |
| *** trown|lunch is now known as trown | 18:23 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 18:23 | |
| *** jralph has quit IRC | 18:25 | |
| *** xavierr has quit IRC | 18:26 | |
| openstackgerrit | Merged openstack/python-ironicclient master: Update volume release notes to fix reno https://review.openstack.org/484493 | 18:32 |
| rloo | TheJulia: ^^ wheeeeee! | 18:33 |
| TheJulia | awesome | 18:34 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: BFV Deploy skip minor logging, logic, and test fixes https://review.openstack.org/479326 | 18:40 |
| *** xavierr has joined #openstack-ironic | 18:41 | |
| *** hjensas has quit IRC | 18:42 | |
| *** baoli has quit IRC | 18:42 | |
| *** baoli has joined #openstack-ironic | 18:45 | |
| *** etingof has joined #openstack-ironic | 18:45 | |
| rloo | TheJulia: there are 2 +2 on this, do you think it needs more reviews? https://review.openstack.org/#/c/462939/ | 18:53 |
| patchbot | patch 462939 - ironic - Improve graceful shutdown of conductor process | 18:53 |
| TheJulia | I doubt it, I don't remember it being very complex | 18:57 |
| *** cdearborn_ has joined #openstack-ironic | 18:57 | |
| openstackgerrit | Merged openstack/ironic master: remove explicit directions for release notes on current branch https://review.openstack.org/467734 | 18:58 |
| *** adisky__ has quit IRC | 18:59 | |
| rloo | TheJulia: you want to +A it then? | 19:00 |
| TheJulia | sure | 19:00 |
| rloo | TheJulia: thx! I'm trying to go for the world record wrt number of patches waiting to merge... | 19:00 |
| TheJulia | rloo: lol | 19:01 |
| TheJulia | rloo: up to 43 yet? | 19:01 |
| TheJulia | doesn't look like it | 19:01 |
| rloo | TheJulia: oh, i guess it would help if i actually kept a count :) | 19:01 |
| *** Goneri has joined #openstack-ironic | 19:02 | |
| TheJulia | :) | 19:03 |
| *** chlong_ has joined #openstack-ironic | 19:03 | |
| *** Nisha_Agarwal has quit IRC | 19:04 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 19:04 | |
| TheJulia | cinerama: NobodyCam: https://review.openstack.org/#/c/483998/ may be of interest | 19:05 |
| patchbot | patch 483998 - bifrost - Allows to setup more than one nameserver | 19:05 |
| *** chlong has quit IRC | 19:05 | |
| *** cdearborn has joined #openstack-ironic | 19:07 | |
| *** Nisha_ has joined #openstack-ironic | 19:10 | |
| *** Nisha_Agarwal has quit IRC | 19:12 | |
| *** Nisha_ has quit IRC | 19:13 | |
| *** cdearborn__ has quit IRC | 19:13 | |
| openstackgerrit | Merged openstack/bifrost master: Update URL home-page in documents according to document migration https://review.openstack.org/483976 | 19:15 |
| openstackgerrit | Julia Kreger proposed openstack/sushy master: Correct sushy homepage https://review.openstack.org/484470 | 19:17 |
| *** cdearborn_ has quit IRC | 19:21 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-lib master: Rework base landing page to contributor docs https://review.openstack.org/483464 | 19:23 |
| *** vishwanathj has joined #openstack-ironic | 19:26 | |
| *** zzzeek has quit IRC | 19:29 | |
| *** crushil has quit IRC | 19:29 | |
| *** zzzeek has joined #openstack-ironic | 19:30 | |
| openstackgerrit | Merged openstack/python-ironicclient master: Update and optimize documentation links https://review.openstack.org/484679 | 19:34 |
| *** vishwanathj has quit IRC | 19:34 | |
| *** vishwanathj has joined #openstack-ironic | 19:35 | |
| *** crushil has joined #openstack-ironic | 19:37 | |
| openstackgerrit | Merged openstack/ironic master: Modifications for rolling upgrades https://review.openstack.org/476779 | 19:42 |
| rloo | wheeeeee, only needed about 10 rechecks !! | 19:45 |
| *** joseph has joined #openstack-ironic | 19:48 | |
| *** vishwanathj has quit IRC | 19:48 | |
| *** joseph is now known as Guest40047 | 19:48 | |
| *** vishwanathj has joined #openstack-ironic | 19:49 | |
| *** vishwanathj has quit IRC | 19:50 | |
| *** vishwanathj has joined #openstack-ironic | 19:51 | |
| *** dprince has quit IRC | 19:51 | |
| *** Guest40047 has quit IRC | 20:11 | |
| *** Dinesh_Bhor has quit IRC | 20:17 | |
| *** jkilpatr has quit IRC | 20:20 | |
| NobodyCam | TheJulia: +2 | 20:25 |
| *** makowals has quit IRC | 20:25 | |
| *** kash has quit IRC | 20:29 | |
| *** makowals has joined #openstack-ironic | 20:30 | |
| *** vishwanathj has quit IRC | 20:32 | |
| *** vishwanathj has joined #openstack-ironic | 20:33 | |
| *** rbudden_ has joined #openstack-ironic | 20:36 | |
| *** rbudden has quit IRC | 20:36 | |
| *** rbudden_ is now known as rbudden | 20:36 | |
| *** jkilpatr has joined #openstack-ironic | 20:37 | |
| *** vishwanathj has quit IRC | 20:39 | |
| *** vishwanathj has joined #openstack-ironic | 20:40 | |
| *** vishwanathj has quit IRC | 20:40 | |
| *** vishwanathj has joined #openstack-ironic | 20:41 | |
| openstackgerrit | Merged openstack/ironic master: Improve graceful shutdown of conductor process https://review.openstack.org/462939 | 20:44 |
| *** vishwanathj has quit IRC | 20:45 | |
| *** vishwanathj has joined #openstack-ironic | 20:46 | |
| *** vishwanathj has quit IRC | 20:47 | |
| *** vishwanathj has joined #openstack-ironic | 20:47 | |
| *** ptx0 has joined #openstack-ironic | 20:48 | |
| openstackgerrit | Xavier proposed openstack/ironic master: Removes agent mixin from oneview drivers https://review.openstack.org/482209 | 20:51 |
| *** gsteinmuller has quit IRC | 20:54 | |
| *** crushil has quit IRC | 20:55 | |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Add the capability to associate ports with portgroups https://review.openstack.org/480583 | 20:56 |
| *** trown is now known as trown|outtypewww | 20:56 | |
| *** rcernin has quit IRC | 20:57 | |
| *** xavierr has quit IRC | 21:03 | |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Add property-collection-editor directive https://review.openstack.org/481152 | 21:11 |
| *** jheroux has quit IRC | 21:28 | |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Add property-collection-editor directive https://review.openstack.org/481152 | 21:29 |
| *** ElCoyote_ has quit IRC | 21:29 | |
| *** baoli has quit IRC | 21:31 | |
| *** ChubYann has quit IRC | 21:34 | |
| *** mjturek has quit IRC | 21:34 | |
| *** rloo has quit IRC | 21:39 | |
| *** rloo has joined #openstack-ironic | 21:39 | |
| *** crushil has joined #openstack-ironic | 21:40 | |
| *** ElCoyote_ has joined #openstack-ironic | 21:44 | |
| *** rloo has quit IRC | 21:45 | |
| *** crushil has quit IRC | 21:59 | |
| *** etingof has quit IRC | 21:59 | |
| jlvillal | TheJulia, Anymore patches to review for BFV that aren't on the weekly priority list on the WhiteBoard? | 22:11 |
| *** etingof has joined #openstack-ironic | 22:15 | |
| *** crushil has joined #openstack-ironic | 22:16 | |
| *** crushil has quit IRC | 22:17 | |
| *** crushil has joined #openstack-ironic | 22:18 | |
| *** causten_ has quit IRC | 22:20 | |
| *** causten has joined #openstack-ironic | 22:21 | |
| *** etingof has quit IRC | 22:22 | |
| *** cdearborn has quit IRC | 22:23 | |
| *** crushil has quit IRC | 22:24 | |
| *** causten has quit IRC | 22:25 | |
| *** hjensas has joined #openstack-ironic | 22:34 | |
| *** hjensas has quit IRC | 22:34 | |
| *** hjensas has joined #openstack-ironic | 22:34 | |
| *** bfournie has quit IRC | 22:39 | |
| *** rbudden has quit IRC | 22:52 | |
| *** gcb has quit IRC | 23:02 | |
| *** joseph has joined #openstack-ironic | 23:07 | |
| *** joseph is now known as Guest89423 | 23:08 | |
| openstackgerrit | Merged openstack/ironic master: Add missing parameter descriptions. https://review.openstack.org/484281 | 23:11 |
| *** bfournie has joined #openstack-ironic | 23:19 | |
| *** baoli has joined #openstack-ironic | 23:20 | |
| *** bfournie has quit IRC | 23:21 | |
| *** baoli has quit IRC | 23:24 | |
| *** hjensas has quit IRC | 23:38 | |
| *** bfournie has joined #openstack-ironic | 23:39 | |
| *** jaypipes has quit IRC | 23:40 | |
| *** rloo has joined #openstack-ironic | 23:40 | |
| *** mtanino__ has quit IRC | 23:41 | |
| *** mtanino_ has joined #openstack-ironic | 23:41 | |
| *** rloo has quit IRC | 23:44 | |
| *** mtanino_ has quit IRC | 23:46 | |
| Guest89423 | Hi | 23:48 |
| Guest89423 | I am simulating Ironic provision process, and built an Ironic Python Agent ramdisk using CoreOS. The node I want to do provision is running on a KVM, and it is with 4G memory. When the node downloads an OS image, it can't donwload an image larger 161 MB into '/tmp/<uuid>'. When I increased the memory of the node to 6G, it can donwload an image around 220 MB. Is it a restriction from the CoreOS container? | 23:50 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!