| *** deadnull_ has quit IRC | 00:03 | |
| *** joanna_mac has quit IRC | 00:08 | |
| openstackgerrit | Julian Edwards proposed openstack/ironic master: Update install guide to explain Nova integration https://review.openstack.org/450492 | 00:08 |
|---|---|---|
| zer0c00l | Is there a way to get the current power status of machine? | 00:08 |
| openstackgerrit | Merged openstack/ironic master: devstack: install python-dracclient if DRAC enabled https://review.openstack.org/425326 | 00:08 |
| zer0c00l | ironic node-get-power-status <uuid> | 00:09 |
| zer0c00l | But that command isn't there | 00:09 |
| openstackgerrit | Chris Smart proposed openstack/ironic master: WIP: test buildroot IPA image https://review.openstack.org/445763 | 00:25 |
| bigjools | I am coming a bit late to the Ironic party so apologies if this was already discussed, but did anyone consider either an ironic agent process or building a simple TFTP server into the conductor? | 00:32 |
| *** hoonetorg has quit IRC | 00:32 | |
| *** rloo has quit IRC | 00:41 | |
| *** Goneri has quit IRC | 00:41 | |
| *** gcb has quit IRC | 00:41 | |
| *** hoonetorg has joined #openstack-ironic | 00:45 | |
| *** tuanluong has joined #openstack-ironic | 01:00 | |
| *** phuongnh has joined #openstack-ironic | 01:03 | |
| *** MarginHu has joined #openstack-ironic | 01:04 | |
| MarginHu | hi, I don't know how to assign ssh_key_contens when using pxe_ssh driver . | 01:04 |
| *** dtantsur has quit IRC | 01:10 | |
| zer0c00l | bigjools: why? | 01:13 |
| bigjools | zer0c00l: it's one fewer moving parts to worry about when installing Ironic. TFTP is already very tightly coupled with the conductor. | 01:13 |
| *** dtantsur has joined #openstack-ironic | 01:14 | |
| zer0c00l | I don't think that is the unix way :P | 01:14 |
| bigjools | It would remove some config and external set up | 01:14 |
| bigjools | irrelevant | 01:14 |
| bigjools | This is how Ubuntu MAAS does it, and it's much easier to set up. | 01:15 |
| zer0c00l | MarginHu: driver_info ssh_key_filename | 01:15 |
| MarginHu | I can't use ssh_keyfilename in my environment. | 01:15 |
| zer0c00l | ironic node-update <uuid> driver_info/ | 01:16 |
| zer0c00l | what do you mean? you can't? | 01:16 |
| zer0c00l | do you need to use passwords? | 01:17 |
| MarginHu | yes, I must use ssh_key_contents instead of ssh_key_filename | 01:19 |
| *** e0ne has joined #openstack-ironic | 01:20 | |
| bigjools | zer0c00l: also if it's built-in to the conductor, you can generate config files on the fly without needing to write them out to actual files. | 01:22 |
| zer0c00l | bigjools: i don't get to decide what goes into ironic. I would talk to other folks in the channel | 01:24 |
| bigjools | sure, just starting the conversation! | 01:24 |
| *** e0ne has quit IRC | 01:25 | |
| zer0c00l | I was merely pointing out the fact that implementing tftp in ironic would be considered monolithic | 01:25 |
| zer0c00l | You can propose a spec | 01:25 |
| zer0c00l | if you want on how to implement tftp in ironic | 01:25 |
| zer0c00l | and let the folks decide if it is the right thing to do | 01:25 |
| bigjools | yeah | 01:25 |
| bigjools | it could at least do with an agent | 01:25 |
| bigjools | then you can fire up stuff more easily in a netns | 01:26 |
| *** alexpilotti has joined #openstack-ironic | 01:26 | |
| *** alexpilotti has quit IRC | 01:31 | |
| *** SerenaFeng has joined #openstack-ironic | 01:32 | |
| zer0c00l | MarginHu: it does supports ssh_key_contents https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ssh.py#L61 | 01:37 |
| zer0c00l | so what is the problem? | 01:37 |
| MarginHu | how to config ? | 01:39 |
| MarginHu | how to config a private key file ? | 01:40 |
| *** hoonetorg has quit IRC | 01:42 | |
| MarginHu | I try to add "\n" after each line in a private key file ,but failed | 01:44 |
| *** amotoki has joined #openstack-ironic | 01:46 | |
| zer0c00l | MarginHu: i belive something like | 01:47 |
| zer0c00l | ironic node-update <uuid> add driver_info/ssh_key_contents='<CONTENTS>' | 01:47 |
| *** SerenaFeng has quit IRC | 01:50 | |
| MarginHu | i know, how to fill the <CONTENTS> | 01:53 |
| *** hoonetorg has joined #openstack-ironic | 01:54 | |
| *** gcb has joined #openstack-ironic | 01:58 | |
| *** amotoki has quit IRC | 01:58 | |
| zer0c00l | copy paste the private key in single quote '' | 01:59 |
| *** zhurong has joined #openstack-ironic | 02:04 | |
| TheJulia | zer0c00l: if power state is being enforced, whatever is in ironic should be correct... unless the node is in error state or the hardware is reporting incorrectly | 02:06 |
| *** wrouesnel has quit IRC | 02:07 | |
| *** causten has joined #openstack-ironic | 02:08 | |
| *** causten has quit IRC | 02:08 | |
| *** causten has joined #openstack-ironic | 02:09 | |
| openstackgerrit | Merged openstack/ironic-inspector stable/ocata: Use eventlet version of subprocess https://review.openstack.org/449743 | 02:10 |
| *** causten has quit IRC | 02:13 | |
| *** yonglihe has joined #openstack-ironic | 02:20 | |
| *** e0ne has joined #openstack-ironic | 02:21 | |
| *** rbudden has quit IRC | 02:23 | |
| zer0c00l | TheJulia: i see, to get the power status of a machine. my best bet is to scrape ironic node-list ? | 02:24 |
| zer0c00l | In large environments, the sync_powerstate and power_state enforcements cannot run often | 02:24 |
| zer0c00l | For example, a node with 50k cluster, it might take a while to get the power status and update the status in the db. | 02:24 |
| zer0c00l | Or | 02:25 |
| *** SerenaFeng has joined #openstack-ironic | 02:25 | |
| *** e0ne has quit IRC | 02:25 | |
| zer0c00l | So if someone wants to quickly see the *current* power status of a node | 02:25 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: [PoC|DNM|TEST] Add fake fields in Node object and RPC https://review.openstack.org/415462 | 02:25 |
| zer0c00l | There is no way, except scrapping the 'ironic node-list' | 02:25 |
| zer0c00l | and what ironic node-list shows is power status of the node 'n hours ago' | 02:26 |
| zer0c00l | not current | 02:26 |
| *** mat128 has joined #openstack-ironic | 02:29 | |
| *** wrouesnel has joined #openstack-ironic | 02:34 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Add note about vaidation to Install Guide https://review.openstack.org/451148 | 02:36 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/451008 | 02:39 |
| TheJulia | zer0c00l: if sync power status is enabled in ironic.conf, it will be enforced by the conductor. So you should be able to scrape whatever ironic says as the power status | 02:55 |
| *** yuanying has quit IRC | 02:55 | |
| TheJulia | The default is to check every sixty seconds | 02:56 |
| *** amotoki has joined #openstack-ironic | 03:00 | |
| openstackgerrit | Merged openstack/ironic master: Remove translation of log messages from ironic/conductor https://review.openstack.org/447727 | 03:03 |
| *** vills has quit IRC | 03:07 | |
| *** links has joined #openstack-ironic | 03:12 | |
| *** raginbajin has quit IRC | 03:14 | |
| *** raginbajin has joined #openstack-ironic | 03:21 | |
| *** e0ne has joined #openstack-ironic | 03:22 | |
| *** e0ne has quit IRC | 03:23 | |
| *** sasha2 has quit IRC | 03:25 | |
| *** mat128 has quit IRC | 03:30 | |
| *** hshiina has quit IRC | 03:39 | |
| *** udesale has joined #openstack-ironic | 03:45 | |
| *** gcb has quit IRC | 03:45 | |
| *** mat128 has joined #openstack-ironic | 03:50 | |
| *** mat128 has quit IRC | 03:51 | |
| *** SerenaFeng has quit IRC | 03:52 | |
| *** gcb has joined #openstack-ironic | 03:59 | |
| *** nicodemos has joined #openstack-ironic | 04:09 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/451008 | 04:17 |
| *** e0ne has joined #openstack-ironic | 04:22 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: Convert BaseDriver.*_interfaces to tuples https://review.openstack.org/450856 | 04:24 |
| *** alexpilotti has joined #openstack-ironic | 04:26 | |
| *** e0ne has quit IRC | 04:27 | |
| *** alexpilotti has quit IRC | 04:30 | |
| *** yuanying has joined #openstack-ironic | 04:39 | |
| *** vmud213 has joined #openstack-ironic | 04:44 | |
| *** hshiina has joined #openstack-ironic | 04:48 | |
| zer0c00l | TheJulia: Sorry i was out for dinner | 05:03 |
| *** vmud213 has quit IRC | 05:04 | |
| zer0c00l | TheJulia: The default 60 seconds does not work on a large scale production environment | 05:04 |
| *** vmud213 has joined #openstack-ironic | 05:04 | |
| zer0c00l | TheJulia: think of it like 20k+ nodes | 05:05 |
| zer0c00l | So obiviously we have tweaked the sync timeout to something like every every 8hrs | 05:05 |
| zer0c00l | we do have like 20+ conductors | 05:05 |
| zer0c00l | still, spawning a new process (ipmitool) slows down the whole thing | 05:06 |
| zer0c00l | And factor in buggy bioses that basically just respond slow/never respond at all | 05:06 |
| zer0c00l | It is impossible to run this every 60 secs | 05:07 |
| zer0c00l | i was asking for a way to check power status of the node using API because, sometimes when the nodes goes into maintenance mode due to a temporary reason (i.e network connectivity to bmc goes down) | 05:08 |
| zer0c00l | when the network comes back fine, the node is still in maintenance mode | 05:08 |
| zer0c00l | It is easy for the operator to remove the node from maintenance when he can scan the node for its current power status | 05:09 |
| *** causten has joined #openstack-ironic | 05:10 | |
| *** causten has quit IRC | 05:10 | |
| zer0c00l | I believe the sync power state does not check the nodes that are in maintenance mode | 05:10 |
| *** causten has joined #openstack-ironic | 05:11 | |
| zer0c00l | So these nodes are always left in maintenanance mode forever and needs operator intervention | 05:11 |
| *** causten has quit IRC | 05:15 | |
| *** vikrant has joined #openstack-ironic | 05:20 | |
| *** e0ne has joined #openstack-ironic | 05:23 | |
| *** yogi has joined #openstack-ironic | 05:27 | |
| *** e0ne has quit IRC | 05:28 | |
| *** oomichi has quit IRC | 05:29 | |
| *** oomichi has joined #openstack-ironic | 05:33 | |
| *** rcernin has joined #openstack-ironic | 05:39 | |
| *** SerenaFeng has joined #openstack-ironic | 05:42 | |
| *** sayantan_ has joined #openstack-ironic | 05:46 | |
| *** zhurong has quit IRC | 05:54 | |
| *** jaosorior has joined #openstack-ironic | 06:14 | |
| openstackgerrit | Chris Smart proposed openstack/ironic master: WIP: test buildroot IPA image https://review.openstack.org/445763 | 06:14 |
| *** rbartal has joined #openstack-ironic | 06:17 | |
| *** rbartal_ has joined #openstack-ironic | 06:17 | |
| *** Serlex has joined #openstack-ironic | 06:21 | |
| *** fragatin_ has joined #openstack-ironic | 06:21 | |
| csmart | this time for sure | 06:21 |
| *** pcaruana has joined #openstack-ironic | 06:24 | |
| *** fragatina has quit IRC | 06:24 | |
| *** tovin07 has joined #openstack-ironic | 06:25 | |
| *** fragatin_ has quit IRC | 06:26 | |
| *** vmud213 has quit IRC | 06:36 | |
| *** fragatina has joined #openstack-ironic | 06:36 | |
| *** fragatina has quit IRC | 06:40 | |
| *** fragatina has joined #openstack-ironic | 06:40 | |
| *** gcb has quit IRC | 06:42 | |
| *** fragatina has quit IRC | 06:42 | |
| *** rbartal has quit IRC | 06:42 | |
| *** rbartal_ has quit IRC | 06:43 | |
| *** bfournie has joined #openstack-ironic | 06:43 | |
| *** fragatina has joined #openstack-ironic | 06:43 | |
| *** rbartal has joined #openstack-ironic | 06:44 | |
| *** dimak has quit IRC | 06:48 | |
| *** oanson has quit IRC | 06:48 | |
| *** gcb has joined #openstack-ironic | 06:51 | |
| *** udesale__ has joined #openstack-ironic | 06:52 | |
| *** jkilpatr has quit IRC | 06:53 | |
| *** udesale has quit IRC | 06:53 | |
| *** vills has joined #openstack-ironic | 06:57 | |
| *** athomas has joined #openstack-ironic | 06:57 | |
| *** wrouesnel has quit IRC | 07:04 | |
| *** vmud213 has joined #openstack-ironic | 07:05 | |
| *** mjura has joined #openstack-ironic | 07:06 | |
| *** mjura has quit IRC | 07:06 | |
| *** mjura has joined #openstack-ironic | 07:06 | |
| *** fxpester has joined #openstack-ironic | 07:09 | |
| *** vmud213 has quit IRC | 07:09 | |
| pas-ha | morning ironic :) | 07:10 |
| *** vmud213 has joined #openstack-ironic | 07:10 | |
| *** alex_xu has quit IRC | 07:13 | |
| *** tesseract has joined #openstack-ironic | 07:14 | |
| *** alex_xu has joined #openstack-ironic | 07:16 | |
| *** sayantan_ has quit IRC | 07:18 | |
| *** sayantan_ has joined #openstack-ironic | 07:19 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Don't pass sqlite_db in db_options.set_defaults() https://review.openstack.org/450737 | 07:19 |
| *** sayantan_ has quit IRC | 07:23 | |
| *** amoralej|off is now known as amoralej | 07:24 | |
| *** ralonsoh has joined #openstack-ironic | 07:25 | |
| *** dyasny has quit IRC | 07:26 | |
| *** oomichi has quit IRC | 07:29 | |
| *** aarefiev_afk is now known as aarefiev | 07:32 | |
| aarefiev | morning all! | 07:32 |
| *** oomichi has joined #openstack-ironic | 07:33 | |
| *** zhurong has joined #openstack-ironic | 07:34 | |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/bifrost master: [WIP]inventory: Add support for defining custom host groups https://review.openstack.org/450970 | 07:41 |
| *** jaganathan has joined #openstack-ironic | 07:42 | |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/bifrost master: [WIP]inventory: Add support for defining custom host groups https://review.openstack.org/450970 | 07:44 |
| *** tovin07 has quit IRC | 07:45 | |
| *** lucas-afk is now known as lucas-event | 07:49 | |
| *** e0ne has joined #openstack-ironic | 07:54 | |
| *** jaganathan has quit IRC | 07:56 | |
| *** vills has quit IRC | 07:56 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Add new transaction starting -> error on timeout https://review.openstack.org/437377 | 07:58 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector-specs master: Split service into API and Conductor https://review.openstack.org/435517 | 07:59 |
| *** zzzeek has quit IRC | 08:00 | |
| *** racedo has joined #openstack-ironic | 08:00 | |
| thomas_oneill | morning :) pas-ha you're Pavlo right? | 08:00 |
| pas-ha | yep | 08:01 |
| thomas_oneill | pas-ha: cool, if you have time could you possibly take a look at this and let me know what you think? I notice you're a bifrost top contributor 2nd to julia :p https://review.openstack.org/#/c/450884/ | 08:02 |
| *** zzzeek has joined #openstack-ironic | 08:03 | |
| *** amarao has joined #openstack-ironic | 08:05 | |
| thomas_oneill | it's a bit of re-write/re-arrangement of the package installs for ironic/keystone, a clean-up role, and I've also added group_vars/all, because it aided with variable precedence and I'm also a fan of having all vars in one place I think. | 08:05 |
| openstackgerrit | Tao Li proposed openstack/python-ironic-inspector-client master: Remove log translations https://review.openstack.org/447801 | 08:18 |
| *** mgoddard has joined #openstack-ironic | 08:23 | |
| openstackgerrit | Tao Li proposed openstack/python-ironicclient master: Remove log translations https://review.openstack.org/447806 | 08:28 |
| pas-ha | thomas_oneill: I like the idea, just couple of suggestions to refactor a bit | 08:29 |
| thomas_oneill | thanks will take a look :) | 08:29 |
| *** yuanying has quit IRC | 08:31 | |
| openstackgerrit | wei zhao proposed openstack/ironic master: Fix some reST field lists in docstrings Probably the most common format for documenting arguments is reST field lists [1]. This change updates some docstrings to comply with the field lists syntax. [1] http://sphinx-doc.org/domains.html#info-field-lists C https://review.openstack.org/451251 | 08:39 |
| *** gcb has quit IRC | 08:39 | |
| *** gcb has joined #openstack-ironic | 08:39 | |
| *** vmud213 has quit IRC | 08:44 | |
| *** dyasny has joined #openstack-ironic | 08:46 | |
| *** vmud213 has joined #openstack-ironic | 08:48 | |
| *** dyasny has quit IRC | 08:50 | |
| *** vills has joined #openstack-ironic | 08:59 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-specs master: Dynamic port groups support https://review.openstack.org/415003 | 09:00 |
| *** dyasny has joined #openstack-ironic | 09:03 | |
| *** vmud213 has quit IRC | 09:05 | |
| *** vmud213 has joined #openstack-ironic | 09:06 | |
| *** ccamacho has joined #openstack-ironic | 09:08 | |
| *** jaganathan has joined #openstack-ironic | 09:10 | |
| *** pmannidi has quit IRC | 09:16 | |
| *** vmud213 has quit IRC | 09:18 | |
| *** zackf has joined #openstack-ironic | 09:18 | |
| *** derekh has joined #openstack-ironic | 09:20 | |
| *** jrcloud has quit IRC | 09:22 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: [Devstack] Rework VMs connection logic https://review.openstack.org/440719 | 09:23 |
| *** jaganathan has quit IRC | 09:23 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: DNR/DNM Testing interface hotplug functionallity https://review.openstack.org/425123 | 09:23 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic master: Add api tests for portgroup update https://review.openstack.org/450855 | 09:27 |
| *** sambetts|afk is now known as sambetts | 09:30 | |
| sambetts | Morning all | 09:31 |
| vdrok | morning ironic, sambetts thomas_oneill pas-ha aarefiev | 09:34 |
| aarefiev | morning vdrok, sambetts | 09:34 |
| sambetts | zer0c00l: o/ aarefiev vdrok | 09:34 |
| sambetts | -_- | 09:35 |
| sambetts | I meant morning aarefiev vdrok :/ | 09:35 |
| vdrok | heh | 09:35 |
| sambetts | zer0c00l: sorry for the noise | 09:35 |
| *** yuanying has joined #openstack-ironic | 09:39 | |
| *** amotoki has quit IRC | 09:40 | |
| *** yuanying has quit IRC | 09:44 | |
| *** yuanying has joined #openstack-ironic | 09:47 | |
| *** moshele has joined #openstack-ironic | 09:49 | |
| *** SerenaFeng has quit IRC | 09:58 | |
| *** e0ne has quit IRC | 10:01 | |
| *** tuanluong has quit IRC | 10:01 | |
| *** e0ne has joined #openstack-ironic | 10:01 | |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/bifrost master: [WIP]inventory: Add support for defining custom host groups https://review.openstack.org/450970 | 10:12 |
| *** mjura has quit IRC | 10:14 | |
| *** mjura has joined #openstack-ironic | 10:14 | |
| *** MattMan has quit IRC | 10:14 | |
| *** MattMan has joined #openstack-ironic | 10:14 | |
| *** amotoki has joined #openstack-ironic | 10:16 | |
| *** gcb has quit IRC | 10:16 | |
| *** udesale__ has quit IRC | 10:17 | |
| *** amotoki has quit IRC | 10:17 | |
| openstackgerrit | Chris Smart proposed openstack/ironic master: WIP: test buildroot IPA image https://review.openstack.org/445763 | 10:18 |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: WIP: SSL TEST https://review.openstack.org/442402 | 10:20 |
| *** vills has quit IRC | 10:25 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: [Devstack] Rework VMs connection logic https://review.openstack.org/440719 | 10:27 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: DNR/DNM Testing interface hotplug functionallity https://review.openstack.org/425123 | 10:27 |
| *** oomichi has quit IRC | 10:29 | |
| *** moshele has quit IRC | 10:30 | |
| *** pester has joined #openstack-ironic | 10:31 | |
| *** fxpester|2 has joined #openstack-ironic | 10:31 | |
| *** amotoki has joined #openstack-ironic | 10:31 | |
| *** MarginHu has quit IRC | 10:32 | |
| *** oomichi has joined #openstack-ironic | 10:33 | |
| *** tuanluong has joined #openstack-ironic | 10:34 | |
| *** pester has quit IRC | 10:35 | |
| *** fxpester has quit IRC | 10:35 | |
| *** amotoki has quit IRC | 10:40 | |
| *** hshiina has quit IRC | 10:42 | |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: WIP: SSL TEST https://review.openstack.org/442402 | 10:46 |
| *** milan has joined #openstack-ironic | 10:47 | |
| *** mgould|afk is now known as mgould | 10:48 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: [Devstack] Rework VMs connection logic https://review.openstack.org/440719 | 10:50 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: DNR/DNM Testing interface hotplug functionallity https://review.openstack.org/425123 | 10:50 |
| milan | morning Ironic! :) | 10:50 |
| pas-ha | morning milan :) | 10:54 |
| milan | morning pas-ha! :) | 10:54 |
| milan | how's the weather in the world today ;) Brno has 19 centigrade, was sunny the whole morning now getting a bit overcast | 10:56 |
| mgould | morning pas-ha milan sambetts vdrok aarefiev | 10:56 |
| milan | morning mgould! :) | 10:57 |
| mgould | milan: grey and 9C here in Edinburgh | 10:57 |
| vdrok | morning milan and mgould | 10:57 |
| milan | morning vdrok! :) | 10:57 |
| vdrok | milan: +9 here, but sunny :) | 10:58 |
| milan | :) | 10:58 |
| *** phuongnh has quit IRC | 10:59 | |
| aarefiev | morning milan, mgould :) | 11:00 |
| milan | seems like mgould's clouds will be here tonight ;) | 11:00 |
| milan | morning aarefiev! :) | 11:00 |
| *** mat128 has joined #openstack-ironic | 11:01 | |
| *** jkilpatr has joined #openstack-ironic | 11:02 | |
| *** alexpilotti has joined #openstack-ironic | 11:06 | |
| *** jkilpatr has quit IRC | 11:07 | |
| *** jkilpatr has joined #openstack-ironic | 11:07 | |
| *** tuanluong has quit IRC | 11:07 | |
| *** amotoki has joined #openstack-ironic | 11:11 | |
| mat128 | morning | 11:12 |
| mgould | morning mat128 | 11:12 |
| *** thomas_oneill is now known as thomas-oneill | 11:16 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Remove translation of log messages Part-2 https://review.openstack.org/448221 | 11:17 |
| *** jjung has joined #openstack-ironic | 11:22 | |
| openstackgerrit | Sam Betts proposed openstack/ironic-specs master: Add spec for IPA API versioning support https://review.openstack.org/341086 | 11:30 |
| *** vills has joined #openstack-ironic | 11:31 | |
| *** yogi has quit IRC | 11:31 | |
| *** zhurong has quit IRC | 11:34 | |
| milan | morning mat128! :) what's the weather like at your place? ;) | 11:34 |
| mat128 | rainy for now :S | 11:35 |
| mat128 | and you? | 11:35 |
| *** bfournie has quit IRC | 11:39 | |
| milan | mat128, was sunny the whole morning and quite warm, 19 centigrade, now getting more cloudy | 11:40 |
| milan | so I had couple of errands in the morning and I was quite enthusiastic about the weather :) | 11:41 |
| *** vills has quit IRC | 11:43 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Permanent node cache https://review.openstack.org/451350 | 11:46 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost master: Do not overrride enable_venv by local lookup https://review.openstack.org/451351 | 11:49 |
| *** moshele has joined #openstack-ironic | 11:51 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Start passing portgroup information to Neutron https://review.openstack.org/446763 | 11:52 |
| *** amoralej is now known as amoralej|lunch | 11:56 | |
| *** athomas has quit IRC | 11:57 | |
| openstackgerrit | Merged openstack/ironic master: Copy and append to static lists https://review.openstack.org/449577 | 12:02 |
| *** athomas has joined #openstack-ironic | 12:05 | |
| *** chlong has quit IRC | 12:05 | |
| openstackgerrit | hnyang proposed openstack/ironic master: Add Error Codes to ironic api references https://review.openstack.org/449495 | 12:06 |
| *** moshele has quit IRC | 12:08 | |
| *** vmud213 has joined #openstack-ironic | 12:09 | |
| *** SerenaFeng has joined #openstack-ironic | 12:15 | |
| *** SerenaFeng has quit IRC | 12:18 | |
| csmart | jroll: JayF success, all the gating tests are passing, although I still have some more of my hacks to clean up so there'll be a few more | 12:18 |
| csmart | https://review.openstack.org/#/c/445763/ | 12:18 |
| *** bfournie has joined #openstack-ironic | 12:18 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Help a user to enable console redirection https://review.openstack.org/384578 | 12:23 |
| TheJulia | good morning everyone | 12:24 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Help a user to enable console redirection https://review.openstack.org/384578 | 12:25 |
| *** udesale has joined #openstack-ironic | 12:25 | |
| TheJulia | zer0c00l: so.... This sounds like a perfect api extension, although your going to really slow things down on your conductors of you go crawl all 20k machines for power status separately :\ | 12:26 |
| *** dprince has joined #openstack-ironic | 12:26 | |
| TheJulia | zer0c00l: we also talked at one point, but I don't think anyone really got behind it.... to automatically take nodes out of maintenance mode if they re-appear, but I feel like there was some contention on the idea. There may have also been one.... two... or three whiskeys involved during that discussion.... | 12:27 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/451008 | 12:30 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements https://review.openstack.org/451009 | 12:30 |
| *** trown|outtypewww is now known as trown | 12:30 | |
| *** rbudden has joined #openstack-ironic | 12:34 | |
| *** alexpilo_ has joined #openstack-ironic | 12:34 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: [Devstack] Rework VMs connection logic https://review.openstack.org/440719 | 12:36 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: DNR/DNM Testing interface hotplug functionallity https://review.openstack.org/425123 | 12:36 |
| *** alexpilotti has quit IRC | 12:37 | |
| *** krtaylor has quit IRC | 12:40 | |
| TheJulia | vsaienk0: reprioritized the ironic-lib bug accordingly, great catch that they were running initial newton and not most recently released newton | 12:40 |
| vsaienk0 | https://github.com/openstack/ironic-lib/commit/33af42c96556ec66087954f115697d79967efe91 | 12:42 |
| vsaienk0 | TheJulia: so I think you just need that commit ^ | 12:42 |
| vsaienk0 | TheJulia: or just use ironic-lib == 2.2.0 it already contain a fix https://github.com/openstack/ironic-lib/blame/2.2.0/ironic_lib/disk_utils.py#L565 | 12:43 |
| vsaienk0 | TheJulia: so should we mark bug as invalid or wont fix for ironic-lib 2.1.0 as it was fixed in 2.2.0 https://bugs.launchpad.net/ironic/+bug/1676422 ? | 12:45 |
| openstack | Launchpad bug 1676422 in ironic-lib "Node failed at deploy. Exception: int() argument must be a string or a number, not 'tuple'" [Medium,Incomplete] - Assigned to Julia Kreger (juliaashleykreger) | 12:45 |
| *** jcoufal has joined #openstack-ironic | 12:45 | |
| *** SerenaFeng has joined #openstack-ironic | 12:57 | |
| *** mjura has quit IRC | 12:58 | |
| *** mjura has joined #openstack-ironic | 12:59 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic master: Add safe_for_db_retry decorator https://review.openstack.org/430351 | 13:03 |
| *** lucas-event is now known as lucasagomes | 13:03 | |
| lucasagomes | morning (ugt) all! | 13:03 |
| mgould | morning lucasagomes | 13:04 |
| mgould | how was the conference? | 13:04 |
| *** sasha2 has joined #openstack-ironic | 13:04 | |
| *** trown is now known as trown|brb | 13:04 | |
| lucasagomes | mgould, was great, better than I expected | 13:05 |
| lucasagomes | mgould, I looked at sdn + nfv and read about it, but one guy presented in a nice way that many things I had doubts started making sense now | 13:05 |
| mgould | excellent | 13:05 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic master: Add names to portgroup api tempest tests https://review.openstack.org/450739 | 13:06 |
| lucasagomes | so, was worth it at least for a noob in the topic like me | 13:06 |
| mgould | lucasagomes: cool! Do you know if slides are available? | 13:06 |
| lucasagomes | mgould, they said they will publish it | 13:06 |
| lucasagomes | mgould, I can forward it to you once we get it | 13:06 |
| mgould | lucasagomes: yes please! | 13:06 |
| lucasagomes | mgould, they will (most likely) put it here: https://www.meetup.com/OpenStack-Ireland/events/238093532/ | 13:07 |
| lucasagomes | idk if you have a meetup account | 13:07 |
| mgould | yep | 13:07 |
| lucasagomes | but if so you can follow that page | 13:07 |
| mgould | cool | 13:08 |
| mgould | brb | 13:08 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic master: Add api tests for portgroup update https://review.openstack.org/450855 | 13:11 |
| *** amoralej|lunch is now known as amoralej | 13:17 | |
| *** vikrant has quit IRC | 13:20 | |
| *** links has quit IRC | 13:20 | |
| *** vikrant has joined #openstack-ironic | 13:21 | |
| *** cdearborn has joined #openstack-ironic | 13:23 | |
| *** zhurong has joined #openstack-ironic | 13:23 | |
| mariojv | morning ironic o/ | 13:24 |
| lucasagomes | mariojv, o/ morning | 13:28 |
| TheJulia | vsaienk0: likely, 2.1.3 has it fixed, sadly a packaging issue :\ | 13:28 |
| *** baoli has joined #openstack-ironic | 13:32 | |
| *** tuan_luong has joined #openstack-ironic | 13:32 | |
| *** crushil has quit IRC | 13:32 | |
| *** tuanla_ has joined #openstack-ironic | 13:32 | |
| *** mat128 has quit IRC | 13:32 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/451008 | 13:33 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements https://review.openstack.org/451009 | 13:33 |
| *** trown|brb is now known as trown | 13:35 | |
| *** gcb has joined #openstack-ironic | 13:36 | |
| *** jcoufal has quit IRC | 13:37 | |
| *** hshiina has joined #openstack-ironic | 13:38 | |
| *** jaosorior has quit IRC | 13:38 | |
| *** chlong has joined #openstack-ironic | 13:40 | |
| *** e0ne has quit IRC | 13:41 | |
| *** e0ne has joined #openstack-ironic | 13:41 | |
| openstackgerrit | Sam Betts proposed openstack/ironic-python-agent master: Include IPA Version during heartbeat https://review.openstack.org/364834 | 13:42 |
| jroll | csmart: awesome :D | 13:42 |
| jroll | (late) morning everyone | 13:42 |
| *** jaosorior has joined #openstack-ironic | 13:43 | |
| *** hshiina has quit IRC | 13:43 | |
| *** jcoufal has joined #openstack-ironic | 13:47 | |
| *** mjturek has joined #openstack-ironic | 13:48 | |
| *** crushil has joined #openstack-ironic | 13:49 | |
| *** vinsh has quit IRC | 13:52 | |
| *** vinsh has joined #openstack-ironic | 13:53 | |
| *** e0ne has quit IRC | 13:54 | |
| *** jrist has quit IRC | 13:55 | |
| *** jrist has joined #openstack-ironic | 13:57 | |
| *** Goneri has joined #openstack-ironic | 13:58 | |
| mgould | hi jroll mariojv TheJulia | 14:00 |
| vdrok | good morning TheJulia lucasagomes mariojv and jroll ! | 14:02 |
| lucasagomes | p/ | 14:02 |
| vdrok | :P | 14:02 |
| *** zhurong_ has joined #openstack-ironic | 14:06 | |
| *** zhurong has quit IRC | 14:07 | |
| *** causten has joined #openstack-ironic | 14:10 | |
| *** udesale has quit IRC | 14:11 | |
| jroll | fyi, our gate should now be functional, the dbsync thing merged | 14:14 |
| *** baha has joined #openstack-ironic | 14:15 | |
| jroll | I also released ironic-lib so the intermittent configdrive bug should be gone | 14:15 |
| *** yatinkarel has quit IRC | 14:17 | |
| *** zhurong_ has quit IRC | 14:18 | |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/bifrost master: playbooks: roles: Drop obsolete dib-utils package https://review.openstack.org/451426 | 14:19 |
| mariojv | \o/ for gate | 14:21 |
| mariojv | thanks jroll | 14:21 |
| jroll | np | 14:22 |
| *** rbudden has quit IRC | 14:22 | |
| lucasagomes | nice! | 14:23 |
| JayF | csmart: holy crap, that's super fast too! | 14:28 |
| JayF | Ironic gate running with a buildroot IPA image, passing tests pretty quickly -> https://review.openstack.org/#/c/445763/ | 14:29 |
| *** vinsh has quit IRC | 14:29 | |
| sambetts | JayF: nice! | 14:29 |
| JayF | sambetts: the link to the repo he built that out of is in the commit message | 14:30 |
| sambetts | yeah looking now | 14:30 |
| *** yatinkarel has joined #openstack-ironic | 14:31 | |
| jroll | JayF: it doesn't seem super fast | 14:31 |
| * jroll compares to random patch https://review.openstack.org/#/c/392959/36 | 14:31 | |
| jroll | seems roughly same or higher | 14:31 |
| JayF | yeah, you're right | 14:32 |
| JayF | I think I might have been thinking abotu ipa -src jobs | 14:32 |
| *** vikrant has quit IRC | 14:32 | |
| sambetts | what sort of image sizes is buildroot producing? | 14:32 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic master: Add names to portgroup api tempest tests https://review.openstack.org/450739 | 14:33 |
| *** rbudden has joined #openstack-ironic | 14:33 | |
| JayF | sambetts: I pulled the image down, looks like 23MB | 14:35 |
| sambetts | cool its save ~10mb over tinyipa | 14:35 |
| JayF | I think it may also be a kernel config just good for the gate | 14:36 |
| JayF | so missing a lot of drivers | 14:36 |
| sambetts | does it have all the required binaries installed? I'm struggling the parse if it has things like the qemu-utils installed etc | 14:37 |
| jroll | it must, if it passes CI :) | 14:37 |
| jroll | even inspector | 14:38 |
| JayF | sambetts: My suggestion to Chris (cmsart -- who btw is in .au so doesn't overlap much time with us), was to do just this -- get CI working | 14:38 |
| JayF | sambetts: I think next steps are to make it an actual usable ramdisk for Rackspace private cloud, and my hope was I could get it a priority for Queens to revamp our ramdisk offerings | 14:39 |
| *** krtaylor has joined #openstack-ironic | 14:39 | |
| openstackgerrit | Derek Higgins proposed openstack/ironic stable/ocata: Copy and append to static lists https://review.openstack.org/451441 | 14:40 |
| jroll | derekh: nice, I was going to backport that today | 14:40 |
| * jroll reviews | 14:40 | |
| JayF | derekh: jroll +2 | 14:41 |
| derekh | jroll: ya, I guessed somebody would want to | 14:42 |
| jroll | and +3 | 14:42 |
| sambetts | JayF: yeah I think that would be a good idea! | 14:42 |
| JayF | sambetts: nice, glad to know you're on board. tell a friend | 14:43 |
| JayF | sambetts: spoiler alert: I want the endgame to be supported ramdisks of dib + buildroot | 14:43 |
| *** vinsh has joined #openstack-ironic | 14:43 | |
| JayF | sambetts: I want tinyipa and coreos to both go away for similar reasons :/ | 14:43 |
| sambetts | yeah, I think that makes a lot of sense | 14:44 |
| *** fxpester|2 has quit IRC | 14:44 | |
| *** SerenaFeng has quit IRC | 14:48 | |
| *** mjturek has quit IRC | 14:48 | |
| *** galyna1 has quit IRC | 14:51 | |
| *** ijw has quit IRC | 14:52 | |
| *** amotoki has quit IRC | 14:56 | |
| *** rloo has joined #openstack-ironic | 14:58 | |
| *** jistr is now known as jistr|mtg | 15:01 | |
| *** rcernin has quit IRC | 15:05 | |
| *** rpioso has joined #openstack-ironic | 15:11 | |
| NobodyCam | Good morning ironic'ers | 15:11 |
| *** rbartal has quit IRC | 15:12 | |
| *** sayantan_ has joined #openstack-ironic | 15:12 | |
| *** rama_y_ has joined #openstack-ironic | 15:13 | |
| *** e0ne has joined #openstack-ironic | 15:13 | |
| *** mat128 has joined #openstack-ironic | 15:13 | |
| TheJulia | lesson of the day: obtaining a managed switch locally is impossible. | 15:13 |
| sambetts | TheJulia: :( no spare hardware available? | 15:14 |
| jroll | :/ | 15:14 |
| jroll | TheJulia: ebay + corp card :D | 15:14 |
| TheJulia | jroll: hehe.... although that might not fly | 15:15 |
| *** jaypipes has joined #openstack-ironic | 15:16 | |
| *** mtanino has joined #openstack-ironic | 15:16 | |
| sambetts | I think netgear do some cheap ones for home use but its depends what your trying to do | 15:16 |
| mgould | TheJulia: dumpster-dive outside failed startups? | 15:17 |
| sambetts | hahaha | 15:17 |
| jroll | mgould: all you'll find is aws coupons :D | 15:17 |
| sambetts | lol | 15:17 |
| mgould | jroll: curses :-/ | 15:18 |
| TheJulia | mgould: what jroll said :) | 15:18 |
| rloo | hi and good morning all, TheJulia, mgould, jroll, sambetts, NobodyCam! | 15:18 |
| jroll | morning :) | 15:18 |
| mgould | morning rloo | 15:18 |
| sambetts | o/ rloo | 15:18 |
| rloo | hi derekh too | 15:18 |
| derekh | rloo: good morning | 15:19 |
| rloo | jroll, derekh: wrt https://review.openstack.org/#/c/451441/, should we do a similar thing for newton (it only has core_interfaces problelm) | 15:19 |
| TheJulia | good morning rloo | 15:19 |
| rloo | and good detective work there derekh! | 15:19 |
| * rloo needs to look at bugs more often | 15:19 | |
| JayF | rloo: I think so | 15:20 |
| jroll | maybe | 15:20 |
| * jroll checks if offending patch is there | 15:20 | |
| rloo | my guess it will slow down, but not as fast, cuz there is just the core_interface. | 15:20 |
| jroll | I think it is | 15:20 |
| derekh | rloo: thanks, seems like a good idea | 15:20 |
| jroll | ya, brokenness is there https://github.com/openstack/ironic/blob/stable/newton/ironic/drivers/base.py#L171 | 15:21 |
| rloo | i suspect it is in mitaka too. | 15:21 |
| NobodyCam | Morning rloo :) | 15:21 |
| jroll | I don't think it's in mitaka | 15:21 |
| jroll | oh no it is | 15:21 |
| jroll | damn | 15:21 |
| jroll | derekh: ^ fyi, that could go all the way back to mitaka | 15:22 |
| rloo | jroll: no, not in mitaka. at least, i don't see it there. | 15:23 |
| jroll | I saw the commit | 15:23 |
| jroll | oh right, it's the appending thing, not the baredriver | 15:23 |
| jroll | which didn't happen until newton | 15:23 |
| rloo | jroll: yup, and phew! | 15:24 |
| derekh | jroll: rloo I'll submit a newton backport | 15:24 |
| rloo | thanks derekh!! | 15:24 |
| jroll | thanks | 15:25 |
| mgould | derekh++ | 15:25 |
| rloo | fwiw, the bug seems to indicate they saw the problem with newton (at least, it sez 'stock TripleO deployments with Newton'. | 15:26 |
| *** rloo has quit IRC | 15:30 | |
| *** rloo has joined #openstack-ironic | 15:30 | |
| *** alex_xu has quit IRC | 15:33 | |
| *** rama_y_ has quit IRC | 15:33 | |
| *** alex_xu has joined #openstack-ironic | 15:35 | |
| openstackgerrit | Derek Higgins proposed openstack/ironic stable/newton: Copy and append to static lists https://review.openstack.org/451459 | 15:37 |
| *** sayantan_ has quit IRC | 15:41 | |
| *** sayantan_ has joined #openstack-ironic | 15:41 | |
| *** amarao has quit IRC | 15:42 | |
| *** causten has quit IRC | 15:44 | |
| sambetts | JayF: ok just did some digging on buildroot... holycow its cool | 15:44 |
| JayF | sambetts: yeah, it's the proper way to do embedded linux :) | 15:45 |
| JayF | sambetts: I've looked into it before but never made the time to dedicate to get something working from zero -- that's why I was thrilled when chris started and had experience with it | 15:45 |
| *** sayantan_ has quit IRC | 15:45 | |
| sambetts | JayF: whats awesome is they already have things like DIB elements for building from source a hugggee range of packages, so installing the qemu utils is as simple as enabling a configuration option | 15:46 |
| *** sayantan_ has joined #openstack-ironic | 15:50 | |
| *** vgadiraj_ has joined #openstack-ironic | 15:52 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: [PoC|DNM|TEST] Add fake fields in Node object and RPC https://review.openstack.org/415462 | 15:55 |
| *** amotoki has joined #openstack-ironic | 15:57 | |
| *** tesseract has quit IRC | 15:58 | |
| *** Sukhdev has joined #openstack-ironic | 15:59 | |
| *** fragatina has quit IRC | 16:00 | |
| *** mat128 has quit IRC | 16:00 | |
| *** causten has joined #openstack-ironic | 16:02 | |
| *** Sukhdev_ has joined #openstack-ironic | 16:02 | |
| *** dyasny has quit IRC | 16:02 | |
| *** amotoki has quit IRC | 16:03 | |
| *** trown is now known as trown|lunch | 16:03 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic-python-agent master: Collect NIC name given by BIOS https://review.openstack.org/396406 | 16:04 |
| vsaienk0 | jroll: dtantsur please add to your review queue https://review.openstack.org/#/c/450822/ | 16:04 |
| *** Goneri has quit IRC | 16:05 | |
| *** jtaryma has joined #openstack-ironic | 16:05 | |
| JayF | vsaienk0: we're 100% sure we're getting test coverage to cover those 3 jobs we're removing | 16:05 |
| JayF | vsaienk0: if so, holy crap that's huge | 16:05 |
| *** Sukhdev_ has quit IRC | 16:05 | |
| *** e0ne has quit IRC | 16:06 | |
| *** e0ne has joined #openstack-ironic | 16:06 | |
| JayF | vsaienk0: hmm.. you're leaving all the uefi jobs, presumedly the standalone doesn't use uefi. I wonder if in this patch we could flip the nova/neutron stuff to use a uefi job, and kill the 4th one too | 16:06 |
| jroll | rloo: I'm about ready to land this, but have a question: https://review.openstack.org/#/c/407491/38/ironic/common/release_mappings.py@48 | 16:07 |
| vsaienk0 | JayF: please have look standalone job covers all cases (pxe|agent|IPMI wholedisk|partitioned + direct http links | 16:07 |
| JayF | vsaienk0: that's what I thought, wdyt about my idea to flip out the bios job for a uefi job for nova/neutron | 16:08 |
| rloo | jroll: oh. yes. i think you should release first. | 16:08 |
| JayF | vsaienk0: since, as proposed, your current patch leaves all the uefi ones in place | 16:08 |
| vsaienk0 | JayF: good point about uefi | 16:08 |
| *** Sukhdev_ has joined #openstack-ironic | 16:08 | |
| *** aarefiev is now known as aarefiev_afk | 16:08 | |
| jroll | rloo: the release request is up and pinned to a SHA, so I'm going to land it then | 16:08 |
| rloo | jroll: ++ | 16:08 |
| jroll | JayF: vsaienk0: let's do that in a follow-up patch that also makes the uefi job voting | 16:09 |
| rloo | jroll: i'd like to keep/try to keep all the rolling upgrades patches w/i one release. fewer things to remember to test | 16:09 |
| jroll | rloo: yeah +1 | 16:09 |
| jroll | rloo: so you're good with a followup to add 7.1 then, yes? | 16:11 |
| vsaienk0 | jroll: JayF thanks, will switch uefi to voting and made it cross project in follow-up | 16:11 |
| rloo | jroll: yup | 16:11 |
| jroll | rloo: thanks | 16:11 |
| jroll | rloo: a few other nits there but +W, thansk! | 16:12 |
| rloo | jroll: thx. i can address in followup. | 16:12 |
| * jroll sees the next patch is WIP, gets sad | 16:12 | |
| rloo | jroll: well, it actually might be done. just that i have to convince myself first. trying to do that today. | 16:13 |
| rloo | jroll: at the very least, for me, i need to be sure i understand it. i suspect i will be adding comments if nothing else. | 16:13 |
| jroll | heh, fair enough, just wanted to review while on a roll :) | 16:13 |
| * jroll lunches instead | 16:13 | |
| *** d0ugal has quit IRC | 16:17 | |
| *** Sukhdev_ has quit IRC | 16:18 | |
| rloo | mariojv, yuriyz: wrt rolling upgrades. this POC to test it by adding a node.fake_field. what does it mean wrt notifications? | 16:19 |
| rloo | mariojv, yuriyz: i don't recall now, why we decided we weren't / didn't need to deal with notification versions. but now i wonder if we do. | 16:20 |
| rloo | mariojv, yuriyz: eg, we add node.fake_field to PIke. Ocata doesn't have it. Pike's notification payloads have version bumps cuz the payloads will include node.fake_field | 16:20 |
| rloo | mariojv, yuriyz: what happens during the rolling upgrade? ocata API creates a node via Pike/pinned conductor. notification payload includes node.fake_field? is that ok? | 16:21 |
| rloo | mariojv, yuriyz: POC: https://review.openstack.org/#/c/415462/ | 16:22 |
| jroll | rloo: yeah, I think we decided notifications didn't need to be in scope for this | 16:23 |
| mariojv | rloo: jroll is right. i don't think there will be any problems, since notifications themselves are versioned and the consumer can do what they will with a newer/older version of the notification | 16:23 |
| rloo | jroll: yes, we decided. but now i am wondering why we decided cuz of that ^^. i can go over our comments to see if we documented it :) | 16:23 |
| * mariojv opens comments too | 16:24 | |
| mariojv | i looked at this a bit yesterday, forgot which patch set that happened on though | 16:24 |
| jroll | rloo: I don't see the concern, fwiw | 16:24 |
| rloo | mariojv: right, notifications are versioned. so we're good with the *user* via the ocata API getting the node info w/o fake_field, even though notification payload includes node.fake_field? | 16:24 |
| mariojv | yes | 16:25 |
| rloo | mariojv: i think it was this patch set: https://review.openstack.org/#/c/407491/ | 16:25 |
| mariojv | which version the user gets also depends on which API they hit mid-upgrade, too | 16:25 |
| mariojv | yup, looking there | 16:25 |
| rloo | mariojv: as long as we're good with it, i'm fine. | 16:25 |
| mariojv | discussion about this starts on PS20 | 16:25 |
| mariojv | ah, so my concern was more about what would happen if Node was pinned and NodePayload wasn't pinned | 16:27 |
| mariojv | i.e. let's say NodePayload is 1.31, Node is 1.30 | 16:27 |
| mariojv | i think it was fine, since you're never passing the node through to another service when emitting the notification, so it'll always "match" the notification version when emitted | 16:28 |
| mgagne | jroll: RE https://bugs.launchpad.net/nova/+bug/1619050 what closed status do you suggest for the bug? | 16:29 |
| openstack | Launchpad bug 1619050 in OpenStack Compute (nova) "Nova scheduler is too greedy when loading instances of hypervisors" [Low,In progress] - Assigned to Mathieu Gagné (mgagne) | 16:29 |
| jroll | mgagne: fix released is what we use | 16:30 |
| rloo | mariojv: ok thx. might be worth testing if i get a chance... | 16:30 |
| mariojv | ah, and in case of a missing field, we handle it properly | 16:30 |
| mgagne | thanks, done | 16:30 |
| mariojv | i.e. if NodePayload has something in the schema that's not in node 1.30 | 16:30 |
| mariojv | wait, that could be problematic... | 16:31 |
| jroll | thanks mgagne :) | 16:31 |
| mariojv | https://github.com/openstack/ironic/blob/master/ironic/objects/notification.py#L159 | 16:31 |
| mariojv | imagine NodePayload schema 1.31 has a mandatory field (non-nullable), and 1.30 doesn't have this field | 16:31 |
| mariojv | L167 exception will get hit | 16:32 |
| mariojv | idk if the object would be serialized (downgraded) at that point though | 16:32 |
| mariojv | i doubt it, but i haven't audited all the notification code for that specific issue | 16:32 |
| mariojv | i think it'd be fine, since rpc node would always match the code running on the service emitting the notification | 16:34 |
| rloo | mariojv: i think we should test just to be sure. i'll put it on my list... | 16:34 |
| mariojv | thanks rloo, let me know if you need any help | 16:35 |
| rloo | mariojv: help == everyone test this as much as possible :) | 16:35 |
| * jroll actually goes to lunch now | 16:36 | |
| *** fragatina has joined #openstack-ironic | 16:36 | |
| *** e0ne has quit IRC | 16:38 | |
| *** mjura has quit IRC | 16:38 | |
| *** sasha2 has quit IRC | 16:38 | |
| *** sasha2 has joined #openstack-ironic | 16:38 | |
| *** d0ugal has joined #openstack-ironic | 16:38 | |
| *** ChubYann has joined #openstack-ironic | 16:42 | |
| *** d0ugal has quit IRC | 16:43 | |
| *** bfournie has quit IRC | 16:44 | |
| *** jistr|mtg is now known as jistr | 16:47 | |
| *** lucasagomes is now known as lucas-afk | 16:49 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic master: Agent driver rescue implementation https://review.openstack.org/400437 | 16:51 |
| *** mgould is now known as mgould|afk | 16:53 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Permanent node cache https://review.openstack.org/451350 | 16:56 |
| *** derekh has quit IRC | 16:57 | |
| *** jtaryma has quit IRC | 16:58 | |
| *** vinsh has quit IRC | 16:58 | |
| *** fragatin_ has joined #openstack-ironic | 16:59 | |
| *** jtaryma has joined #openstack-ironic | 16:59 | |
| *** [1]cdearborn has joined #openstack-ironic | 16:59 | |
| *** amotoki has joined #openstack-ironic | 16:59 | |
| *** fragatina has quit IRC | 17:02 | |
| *** chlong has quit IRC | 17:03 | |
| *** bfournie has joined #openstack-ironic | 17:03 | |
| *** trown|lunch is now known as trown | 17:04 | |
| *** amotoki has quit IRC | 17:04 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic master: Agent driver rescue implementation https://review.openstack.org/400437 | 17:05 |
| *** links has joined #openstack-ironic | 17:07 | |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Reorganise Cisco UCSM driver code to match its name https://review.openstack.org/426814 | 17:09 |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Rename and redocument the Cisco UCSM driver https://review.openstack.org/400673 | 17:09 |
| *** sayantan_ has quit IRC | 17:10 | |
| *** sambetts is now known as sambetts|afk | 17:11 | |
| sambetts|afk | Night all o/ | 17:11 |
| *** mgoddard has quit IRC | 17:11 | |
| *** cdearborn has quit IRC | 17:13 | |
| *** links has quit IRC | 17:19 | |
| *** athomas has quit IRC | 17:25 | |
| *** sayantan_ has joined #openstack-ironic | 17:26 | |
| *** jjung has quit IRC | 17:26 | |
| *** sayantan_ has quit IRC | 17:27 | |
| *** sayantan_ has joined #openstack-ironic | 17:27 | |
| openstackgerrit | Merged openstack/ironic master: Add RPC and object version pinning https://review.openstack.org/407491 | 17:33 |
| openstackgerrit | Merged openstack/ironic stable/ocata: Copy and append to static lists https://review.openstack.org/451441 | 17:33 |
| * jroll approves newton | 17:37 | |
| *** tuanla_ has quit IRC | 17:38 | |
| *** tuan_luong has quit IRC | 17:38 | |
| *** mtanino has quit IRC | 17:40 | |
| *** jjung has joined #openstack-ironic | 17:41 | |
| *** e0ne has joined #openstack-ironic | 17:42 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 17:43 | |
| *** trown is now known as trown|brb | 17:46 | |
| *** ElCoyote_ has quit IRC | 17:51 | |
| *** pcaruana has quit IRC | 17:52 | |
| *** e0ne has quit IRC | 17:53 | |
| *** [1]rpioso has joined #openstack-ironic | 17:54 | |
| *** trown|brb is now known as trown | 17:54 | |
| *** [1]rpioso has quit IRC | 17:55 | |
| *** [2]rpioso has joined #openstack-ironic | 17:56 | |
| *** rpioso has quit IRC | 17:56 | |
| *** [2]rpioso is now known as rpioso | 17:56 | |
| *** ralonsoh has quit IRC | 17:57 | |
| *** vishwanathj has joined #openstack-ironic | 17:58 | |
| *** chlong has joined #openstack-ironic | 17:58 | |
| jlvillal | vdrok, vsaienk0 Not sure if you are still around. Is either one of you working on moving the API service to the subnode for multi-node? | 17:58 |
| vdrok | jlvillal: nope, we did not have a chance yet, morning | 17:59 |
| jlvillal | vdrok, vsaienk0 I was reading the back-scroll of the IRC log from a few days ago and I think that was the plan. Correct me if I am wrong. | 17:59 |
| mariojv | JayF: we have 3x +2 on the active node vif attach patch: https://review.openstack.org/#/c/424723 feel free to +W if your concern was addressed sufficiently | 17:59 |
| vdrok | jlvillal: yup, s/moving to the subnode/removing from the primary node/ | 17:59 |
| vdrok | there is api already on the subnode | 17:59 |
| jlvillal | vdrok, Thanks! I don't think I will have time to look at it before my vacation starts tomorrow. | 17:59 |
| jlvillal | vdrok, Ah, okay. | 18:00 |
| JayF | mariojv: my question was not a "block the merge of this" question, more a "I don't understand" | 18:00 |
| *** baoli has quit IRC | 18:00 | |
| JayF | mariojv: feel free to land it | 18:00 |
| vdrok | jlvillal: great, thanks! | 18:00 |
| mariojv | cool | 18:00 |
| *** amotoki has joined #openstack-ironic | 18:00 | |
| JayF | mariojv: fwiw if I have a question I want to answer+read before it lands, I usually vote -1 :) but thanks for being concerned | 18:00 |
| JayF | mariojv: my review on that patch not worth a lot anyway, I don't understand it well :) | 18:00 |
| *** baoli has joined #openstack-ironic | 18:01 | |
| *** jjung has quit IRC | 18:02 | |
| TheJulia | so now I really have motivation to write an ansible module for port stuffs | 18:02 |
| TheJulia | :) | 18:02 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-lib master: Make disk size check output handling consistent https://review.openstack.org/450754 | 18:04 |
| mariojv | are the standalone experimental jobs now called "gate-ironic-dsvm-functional-ubuntu-xenial" ? i thought i remembered them being called gate-ironic-something-standalone-something but i don't see it on this patch https://review.openstack.org/#/c/434405/ | 18:05 |
| *** amotoki has quit IRC | 18:05 | |
| *** mgoddard has joined #openstack-ironic | 18:05 | |
| jroll | mariojv: they're still called standalone something but that check queue run was feb 15 | 18:06 |
| jroll | needs a recheck :P | 18:06 |
| mariojv | ah, ok, so new config didn't make it in | 18:06 |
| mariojv | thx | 18:06 |
| mariojv | was confused b/c i saw this too https://github.com/openstack-infra/project-config/blob/3d07ce36cad91e939a74198fcd5c2dc33ea03e06/jenkins/jobs/ironic.yaml#L1270 will recheck it | 18:06 |
| *** amoralej is now known as amoralej|off | 18:07 | |
| jroll | yep, that's the one. moved from experimental to check | 18:07 |
| *** chlong has quit IRC | 18:07 | |
| *** alex_xu has quit IRC | 18:07 | |
| *** dsneddon has quit IRC | 18:08 | |
| *** alex_xu has joined #openstack-ironic | 18:08 | |
| *** mtanino has joined #openstack-ironic | 18:09 | |
| *** dsneddon has joined #openstack-ironic | 18:09 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: WIP: Refactor API microversion checks https://review.openstack.org/451506 | 18:13 |
| *** mgoddard has quit IRC | 18:13 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: WIP: Refactor API microversion checks https://review.openstack.org/451506 | 18:15 |
| *** ElCoyote_ has joined #openstack-ironic | 18:16 | |
| vdrok | jroll: hopefully that^^ will allow to drop a bunch of cruft from our api | 18:16 |
| *** jjung has joined #openstack-ironic | 18:16 | |
| *** crushil has quit IRC | 18:16 | |
| *** vmud213 has quit IRC | 18:17 | |
| *** mat128 has joined #openstack-ironic | 18:17 | |
| jroll | vdrok: neat | 18:17 |
| * jroll stars | 18:17 | |
| *** crushil has joined #openstack-ironic | 18:17 | |
| *** baha has quit IRC | 18:18 | |
| *** Sukhdev_ has joined #openstack-ironic | 18:18 | |
| vdrok | good night everyone! | 18:19 |
| openstackgerrit | Merged openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/450711 | 18:20 |
| *** Sukhdev_ has quit IRC | 18:21 | |
| TheJulia | ppiela: betherly: I went and explicitly tagged you both on a couple ui reviews, when you have a chance! Thank you both in advance! | 18:21 |
| *** chlong has joined #openstack-ironic | 18:21 | |
| ppiela | TheJulia: I will review this afternoon | 18:22 |
| TheJulia | Awesome! | 18:22 |
| *** dyasny has joined #openstack-ironic | 18:24 | |
| *** dyasny has quit IRC | 18:25 | |
| *** dyasny has joined #openstack-ironic | 18:26 | |
| openstackgerrit | Merged openstack/ironic-lib master: Updated from global requirements https://review.openstack.org/451010 | 18:29 |
| *** trown is now known as trown|brb | 18:29 | |
| *** baoli_ has joined #openstack-ironic | 18:32 | |
| *** chlong has quit IRC | 18:34 | |
| *** baoli has quit IRC | 18:34 | |
| *** vills has joined #openstack-ironic | 18:37 | |
| *** mat128 has quit IRC | 18:41 | |
| *** Nisha_Agarwal has quit IRC | 18:41 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/451008 | 18:42 |
| *** openstackstatus has joined #openstack-ironic | 18:44 | |
| *** ChanServ sets mode: +v openstackstatus | 18:44 | |
| *** trown|brb is now known as trown | 18:45 | |
| *** Sukhdev has quit IRC | 18:46 | |
| *** vills has quit IRC | 18:46 | |
| *** mat128 has joined #openstack-ironic | 18:47 | |
| *** vgadiraj_ has quit IRC | 18:47 | |
| openstackgerrit | Merged openstack/ironic stable/newton: Copy and append to static lists https://review.openstack.org/451459 | 18:49 |
| *** chlong has joined #openstack-ironic | 18:49 | |
| *** e0ne has joined #openstack-ironic | 18:50 | |
| *** vills has joined #openstack-ironic | 18:50 | |
| *** jcoufal has quit IRC | 18:54 | |
| *** baha has joined #openstack-ironic | 18:56 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic-python-agent master: Collect NIC name given by BIOS https://review.openstack.org/396406 | 18:59 |
| betherly | TheJulia: I'll take a look in the morning :) | 19:00 |
| *** baoli_ has quit IRC | 19:01 | |
| *** jtaryma has quit IRC | 19:01 | |
| *** amotoki has joined #openstack-ironic | 19:01 | |
| *** jcoufal has joined #openstack-ironic | 19:02 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic master: Remove deprecated, untested ipminative driver https://review.openstack.org/451526 | 19:03 |
| JayF | dtantsur: jroll ^ fyi; it's been >2wks since I emailed the list, so followed up and pushed the patch | 19:03 |
| TheJulia | betherly: thank you! | 19:04 |
| *** amotoki has quit IRC | 19:05 | |
| *** e0ne has quit IRC | 19:06 | |
| *** baoli has joined #openstack-ironic | 19:07 | |
| *** milan has quit IRC | 19:08 | |
| *** crushil has quit IRC | 19:09 | |
| *** vills has quit IRC | 19:11 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic master: Remove deprecated, untested ipminative driver https://review.openstack.org/451526 | 19:11 |
| JayF | jroll: ^ you commetned on ps2 after I added the note | 19:17 |
| JayF | jroll: but still -1'd me for it | 19:17 |
| JayF | jroll: perhaps you were reviewing ps1? | 19:17 |
| jroll | JayF: noticed it on 1, overlooked it on 2, heh | 19:17 |
| JayF | yeah, I realized after the first one I needed a 'git grep pyghmi' and saw the missing release note then | 19:17 |
| * jroll actually reviews now | 19:18 | |
| jroll | +2 with nits | 19:20 |
| JayF | i'll fix nits real soon | 19:20 |
| JayF | jroll: lol, you missed something again | 19:21 |
| jroll | no worries, I'm fine if you don't fix them | 19:21 |
| jroll | what'd I miss | 19:21 |
| JayF | jroll: "fake_ipminative" is literally in the line you commented on | 19:21 |
| *** mat128 has quit IRC | 19:21 | |
| jroll | JayF: "and" isn't | 19:21 |
| jroll | :) | 19:21 |
| JayF | OH | 19:21 |
| JayF | lol | 19:21 |
| * jroll gets the grammar hammer | 19:21 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic master: Remove deprecated, untested ipminative driver https://review.openstack.org/451526 | 19:22 |
| JayF | jroll: ^ | 19:22 |
| jroll | JayF: thanks, now we wait for dmitry to return | 19:23 |
| JayF | feel free to modify it if needed, since I'm out next week when dmitry is back :) | 19:24 |
| jroll | sure | 19:24 |
| *** fragatin_ has quit IRC | 19:25 | |
| *** fragatina has joined #openstack-ironic | 19:26 | |
| *** crushil has joined #openstack-ironic | 19:26 | |
| *** sasha2 has quit IRC | 19:31 | |
| *** openstackgerrit has quit IRC | 19:33 | |
| *** sasha2 has joined #openstack-ironic | 19:33 | |
| *** chlong has quit IRC | 19:34 | |
| *** dyasny has quit IRC | 19:35 | |
| *** rpioso has quit IRC | 19:36 | |
| *** racedo has quit IRC | 19:36 | |
| *** rpioso has joined #openstack-ironic | 19:37 | |
| *** rama_y_ has joined #openstack-ironic | 19:43 | |
| *** openstackgerrit has joined #openstack-ironic | 19:47 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Updated from global requirements https://review.openstack.org/451009 | 19:47 |
| *** d0ugal has joined #openstack-ironic | 19:49 | |
| *** dprince has quit IRC | 19:55 | |
| *** vinsh has joined #openstack-ironic | 19:55 | |
| *** Serlex has quit IRC | 19:56 | |
| *** baoli has quit IRC | 19:58 | |
| zer0c00l | TheJulia: When you said, API extension, do you mean a seperate service for running sync_power_state? | 19:58 |
| *** baoli has joined #openstack-ironic | 19:58 | |
| *** crushil has quit IRC | 19:58 | |
| TheJulia | zer0c00l: Well, it comes down to a use pattern at that point | 19:59 |
| TheJulia | do you want to poll all servers/sync the state for all servers in one shot, as in trigger load for your conductors, or do you just want to check the state of node x and y, but not z ? | 19:59 |
| *** d0ugal has quit IRC | 20:00 | |
| zer0c00l | I see | 20:00 |
| zer0c00l | Right now there is no way to do force run a 'power status' other than waiting for sync_power_state to come around | 20:01 |
| zer0c00l | So you are saying we can write an API extension to do that | 20:01 |
| zer0c00l | ? | 20:01 |
| jroll | doesn't 'ironic node-get-power-state' do that? | 20:01 |
| *** amotoki has joined #openstack-ironic | 20:02 | |
| zer0c00l | There is no node-get-power-state https://docs.openstack.org/cli-reference/ironic.html | 20:02 |
| jroll | huh, I thought there was at some point | 20:03 |
| * jroll puts down the crack | 20:03 | |
| zer0c00l | It would be nice if there is one | 20:04 |
| TheJulia | zer0c00l: I doubt there would be resistance to either possibility, "explicitly go fetch x state" or "trigger a full power state resync" | 20:04 |
| jroll | suspect folks would prefer an async resync sort of thing, followed by polling | 20:05 |
| *** amotoki has quit IRC | 20:06 | |
| TheJulia | although, at 20k+ nodes, would someone want continued polling of their power states for all of their nodes? Or are you thinking single node? | 20:06 |
| zer0c00l | TheJulia: single node | 20:06 |
| jroll | yeah, I'm -1 on "resync everything" :P | 20:07 |
| zer0c00l | yeah | 20:07 |
| zer0c00l | me too | 20:07 |
| zer0c00l | People will DOS it | 20:07 |
| TheJulia | make it a "baremetal_i_really_know_what_im_doing" role | 20:07 |
| zer0c00l | explicitly go fetch x state | 20:07 |
| TheJulia | :) | 20:07 |
| zer0c00l | async + polling will be nice | 20:07 |
| zer0c00l | We don't want to wait for ipmitool to come back after x seconds | 20:08 |
| jroll | TheJulia: you mean "baremetal_bofh", right | 20:08 |
| TheJulia | Sounds good to me to do a single node, should be a relatively easy spec to write, if you want to give that a spin zer0c00l :) | 20:09 |
| TheJulia | jroll: baremtal_dos_admin | 20:09 |
| JayF | jroll: TheJulia: It'd be strange to have that API be async though | 20:09 |
| JayF | jroll: TheJulia: What value goes into power_state / target_power_state while it's working? Nulling it would break other things against the node | 20:09 |
| jroll | JayF: it'd be strange to have a syncronous request that can take seconds waiting on the bmc though :/ | 20:10 |
| JayF | jroll: yeah, but there's no way to communicate async to the operator either | 20:10 |
| TheJulia | get boot devices? does that though | 20:10 |
| TheJulia | at least I think it does | 20:10 |
| jroll | right, they'd need to poll | 20:10 |
| jroll | TheJulia: yeah that's the one I thought was get power state | 20:10 |
| mrda | Morning Ironic | 20:11 |
| JayF | jroll: poll for what? How do I know it's been sync'd? | 20:11 |
| jroll | JayF: true that | 20:11 |
| JayF | jroll: that's the problem; it's hard for a node object to communicate "I'm getting power state" | 20:11 |
| jroll | damn | 20:11 |
| jroll | ya | 20:11 |
| zer0c00l | hrm | 20:11 |
| jroll | maybe I'm fine with sync, seems like a dos vector but /shrug | 20:11 |
| JayF | our entire ipmi-related API is a dos vector | 20:12 |
| JayF | lol | 20:12 |
| zer0c00l | hehehehe | 20:12 |
| zer0c00l | true that | 20:12 |
| JayF | ipmi as a protocol is too terrible to make it any other way | 20:12 |
| JayF | zer0c00l: btw, you're aware I just pushed this : https://review.openstack.org/451526 | 20:12 |
| JayF | zer0c00l: I know you all use it for some stuff, so it going out of tree may impact you. | 20:12 |
| zer0c00l | JayF: yes i saw that. No we are not using it. | 20:12 |
| zer0c00l | We wanted to use it | 20:13 |
| zer0c00l | Did a POC | 20:13 |
| *** PTCbro has joined #openstack-ironic | 20:13 | |
| zer0c00l | It was too flakey | 20:13 |
| zer0c00l | rybridges did the POC | 20:13 |
| JayF | that'd be one of the reasons it's getting crushinated | 20:13 |
| zer0c00l | so what do we do about the node-get-power-state? | 20:13 |
| zer0c00l | Should we write a spec? | 20:14 |
| TheJulia | I would think so, just so we have consensus. Maybe also refer to the eavesdrop log in the spec :) | 20:14 |
| zer0c00l | will do! | 20:14 |
| zer0c00l | Thanks | 20:15 |
| *** zer0c00l is now known as zer0c00l_lunch | 20:15 | |
| zer0c00l_lunch | brb | 20:15 |
| *** Goneri has joined #openstack-ironic | 20:17 | |
| *** krtaylor has quit IRC | 20:20 | |
| JayF | Is an unstack and restack supposed to update all the software you're running? | 20:23 |
| JayF | I'm setting up a physical machine to be my devstack, and so want to make it less ... disposable than I've treated devstack boxes in the past | 20:23 |
| JayF | so curious about how folks care for it to keep it up to date and such | 20:24 |
| jroll | JayF: I believe stack.sh does an apt-get upgrade, but not 100% | 20:24 |
| JayF | I'm thinking more services in git | 20:24 |
| jroll | ah | 20:24 |
| jroll | RECLONE=true/false | 20:24 |
| JayF | mainly the ones I depend on but am unlikely to actually develop | 20:24 |
| JayF | default false, I presume? | 20:24 |
| jroll | yep | 20:25 |
| jroll | that's a local.conf setting to be clear | 20:25 |
| JayF | anything else you really do or try to pay attention to in order to keep it from getting crufty? | 20:25 |
| JayF | yeah, I get it | 20:25 |
| jroll | not that I can think of | 20:25 |
| jroll | there's no compat guarantees even with unstack/stack | 20:25 |
| csmart | jroll: JayF: sambetts|afk I should be able to clean up the last few things today, with any luck | 20:26 |
| jroll | but there's also clean.sh which should blow away more than unstack | 20:26 |
| jroll | oh also RECLONE=true doesn't update the devstack repo itself, that's all on you | 20:26 |
| jroll | csmart: nice | 20:26 |
| JayF | jroll: makes sense, ty :) | 20:26 |
| csmart | then start looking into the 3rd party tests, etc - things will be missing from the kernel we need to slowly add | 20:26 |
| jroll | yeah, np | 20:26 |
| JayF | csmart: I told you if you got it working upstream folks would be super interested :D | 20:26 |
| JayF | csmart: 3rd party ironic ci is ... unstable at best. So that's a good thing to do, just be aware you have a long road of false-failures ahead | 20:27 |
| csmart | JayF: hehe, yeah I'll work on the other imagebuild patch too | 20:27 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Remove extra blank space in ClientSide error msg https://review.openstack.org/451551 | 20:27 |
| csmart | JayF: yeah, I noticed they are pretty unstable, 3rd party is not a priority, but I just wanna kick the process of by touching bas with people at least | 20:27 |
| *** trown is now known as trown|outtypewww | 20:28 | |
| openstackgerrit | Merged openstack/ironic master: Allow to attach/detach VIFs to active ironic nodes https://review.openstack.org/424723 | 20:28 |
| *** bl0m1_ has quit IRC | 20:28 | |
| csmart | *base | 20:28 |
| *** Guest6666 has quit IRC | 20:28 | |
| *** Guest6666 has joined #openstack-ironic | 20:29 | |
| *** bl0m1 has joined #openstack-ironic | 20:29 | |
| *** racedo has joined #openstack-ironic | 20:29 | |
| *** bl0m1 has quit IRC | 20:30 | |
| *** bl0m1 has joined #openstack-ironic | 20:30 | |
| *** jaypipes_ has joined #openstack-ironic | 20:30 | |
| csmart | Gah, *kick the process off by touching base | 20:31 |
| JayF | csmart: they should all have email addresses for contacts in the wiki jfyi | 20:32 |
| *** vills has joined #openstack-ironic | 20:32 | |
| *** jaypipes_ is now known as brokenpipes | 20:32 | |
| *** jaypipes has quit IRC | 20:32 | |
| csmart | JayF: thanks, yep, jroll pointed them out to me the other day. Cheers | 20:33 |
| *** racedo has quit IRC | 20:37 | |
| *** jtaryma has joined #openstack-ironic | 20:39 | |
| jtaryma | mariojv: I replied to some of your comments in etags patch | 20:39 |
| *** racedo has joined #openstack-ironic | 20:39 | |
| *** racedo has quit IRC | 20:40 | |
| mariojv | thanks jtaryma | 20:40 |
| mariojv | i'm fine with returning it in the header if that's what's in the API WG spec | 20:40 |
| *** jtaryma is now known as joanna_mac | 20:41 | |
| mariojv | +1 on not including updated_at | 20:41 |
| *** joanna_mac is now known as joanna__ | 20:41 | |
| joanna__ | mariojv: cool :) also - BFV cinder api patch passed jenkins after recheck :) I'd appreciate if you could review it :) | 20:42 |
| mariojv | in my queue :) | 20:43 |
| joanna__ | mariojv: awesome! thank you :) | 20:43 |
| *** bcornec has joined #openstack-ironic | 20:49 | |
| *** xek has quit IRC | 20:50 | |
| *** xek has joined #openstack-ironic | 20:52 | |
| *** causten has quit IRC | 20:55 | |
| *** causten has joined #openstack-ironic | 20:56 | |
| *** causten has quit IRC | 20:57 | |
| *** causten has joined #openstack-ironic | 20:57 | |
| *** amarao has joined #openstack-ironic | 20:58 | |
| *** PTCbro has quit IRC | 21:02 | |
| *** amotoki has joined #openstack-ironic | 21:02 | |
| *** amarao has quit IRC | 21:04 | |
| *** ccamacho has quit IRC | 21:04 | |
| *** mgoddard has joined #openstack-ironic | 21:06 | |
| *** mat128 has joined #openstack-ironic | 21:06 | |
| *** amotoki has quit IRC | 21:07 | |
| *** fragatin_ has joined #openstack-ironic | 21:09 | |
| *** baoli has quit IRC | 21:12 | |
| openstackgerrit | Anup Navare proposed openstack/ironic master: Config drive support for Ceph https://review.openstack.org/407639 | 21:12 |
| *** baoli has joined #openstack-ironic | 21:13 | |
| *** baoli has quit IRC | 21:13 | |
| *** causten has quit IRC | 21:13 | |
| *** causten has joined #openstack-ironic | 21:13 | |
| *** fragatina has quit IRC | 21:13 | |
| *** mgoddard has quit IRC | 21:14 | |
| *** krtaylor has joined #openstack-ironic | 21:14 | |
| *** baoli has joined #openstack-ironic | 21:14 | |
| *** vills has quit IRC | 21:17 | |
| *** joanna__ has quit IRC | 21:18 | |
| *** joanna__ has joined #openstack-ironic | 21:23 | |
| *** jjung has quit IRC | 21:24 | |
| *** ccamacho has joined #openstack-ironic | 21:24 | |
| *** yee379 has joined #openstack-ironic | 21:25 | |
| *** yee37915 has quit IRC | 21:26 | |
| *** bfournie has quit IRC | 21:28 | |
| openstackgerrit | Merged openstack/ironic master: Updated from global requirements https://review.openstack.org/451008 | 21:29 |
| *** jcoufal has quit IRC | 21:29 | |
| *** aignatov has quit IRC | 21:45 | |
| *** aignatov has joined #openstack-ironic | 21:46 | |
| mariojv | hi folks, i proposed a couple topics for the review party tomorrow for anyone interested in attending: https://etherpad.openstack.org/p/ironic-review-party | 21:48 |
| *** brokenpipes has quit IRC | 21:55 | |
| *** sayantan_ has quit IRC | 21:55 | |
| *** rbudden has quit IRC | 22:03 | |
| *** amotoki has joined #openstack-ironic | 22:03 | |
| *** sayantan_ has joined #openstack-ironic | 22:04 | |
| *** causten has quit IRC | 22:05 | |
| *** joanna__ has quit IRC | 22:07 | |
| *** amotoki has quit IRC | 22:08 | |
| *** fragatin_ has quit IRC | 22:08 | |
| *** fragatina has joined #openstack-ironic | 22:09 | |
| *** choyj has quit IRC | 22:12 | |
| *** sayantan_ has quit IRC | 22:21 | |
| *** sayantan_ has joined #openstack-ironic | 22:21 | |
| *** baoli has quit IRC | 22:21 | |
| jlvillal | aNuposic, The radosgw/ceph patch failed the releasenotes job | 22:22 |
| *** baha has quit IRC | 22:24 | |
| *** choyj has joined #openstack-ironic | 22:25 | |
| *** [1]cdearborn has quit IRC | 22:29 | |
| * TheJulia sighs that DEFAULT_VER is still 1.9 in the client | 22:31 | |
| *** wrouesnel has joined #openstack-ironic | 22:33 | |
| JayF | TheJulia: you know mariojv has a spec up, that needs reviews, to fix that? | 22:36 |
| *** sayantan_ has quit IRC | 22:36 | |
| JayF | TheJulia: https://review.openstack.org/#/c/448661/ | 22:36 |
| *** krtaylor has quit IRC | 22:38 | |
| TheJulia | Oh awesome, I hadn't seen it since I was a a conference last week. Although, if we just cut 1.12, wouldn't that mean the spec now needs to be updated since we've delivered a new version during pike? | 22:40 |
| JayF | no, for pike it will stay the default of 1.9 but print a warning if you don't specify versino | 22:41 |
| JayF | in queens we'll bump it to be latest supported version | 22:41 |
| *** joanna__ has joined #openstack-ironic | 22:42 | |
| JayF | at least that's how I read the spec | 22:42 |
| TheJulia | Yeah, I just re-read that part and your right | 22:42 |
| TheJulia | I mean, if we land it now, it should be mostly fine. as long as it is cli invocation, direct api usage shouldn't log anything I guess, but I'm kind of grumpy about the pin being there since it means I'll have to specify a newer version in code. It actually kind of almost pushes me off using the client library and just means I should engage the api directly | 22:44 |
| *** joanna_mac has joined #openstack-ironic | 22:44 | |
| *** joanna_mac has quit IRC | 22:44 | |
| JayF | TheJulia: in every downstream ironic thing we did at OnMetal, we used our own client | 22:44 |
| TheJulia | heh, I just found out they want to do direct api calls anyway | 22:44 |
| JayF | TheJulia: it's just too simple of an api not to | 22:44 |
| *** joanna__ has quit IRC | 22:44 | |
| *** joanna__ has joined #openstack-ironic | 22:44 | |
| TheJulia | Yeah, the only reason we put the pin in was to guard against the default state suddenly changing, but it actually now makes it harder to use the python client library directly across versions | 22:49 |
| JayF | yep, exactly | 22:50 |
| JayF | but we're undoing the mistake, at least for CLI use cases | 22:50 |
| JayF | undoing mistakes in openstack is hard and we (mariojv) should be lauded for attempting it :P | 22:51 |
| TheJulia | mariojv: I'll buy you beer, or fancy water in boston! :) | 22:52 |
| *** zer0c00l_lunch is now known as zer0c00l | 22:54 | |
| *** rpioso has quit IRC | 22:55 | |
| TheJulia | mariojv: I put a -1 on it because we do actually have direct client library users... and I feel like that in itself should have at least a sentence. If it already does and I've missed it, sorry! Otherwise, lgtm. | 22:57 |
| *** bfournie has joined #openstack-ironic | 22:57 | |
| *** sayantan_ has joined #openstack-ironic | 22:58 | |
| JayF | TheJulia: my rememberance from ptg was that basically the thought was CLI is a quick win, because we don't break apps | 22:59 |
| JayF | TheJulia: a little scarier to update python client, but with a max version bump I'd be +2 to that as well | 22:59 |
| TheJulia | We've been telling people to pin direct invocations for a while.... | 23:00 |
| TheJulia | I guess in my mind I thought when client was said, that it included library users as well | 23:03 |
| TheJulia | $words... why do we not have telepathic thought transfers yet? | 23:03 |
| *** amotoki has joined #openstack-ironic | 23:04 | |
| JayF | *shrug* idk; but it's a good point to talk about in the spec | 23:05 |
| * JayF out for teh day o/ | 23:06 | |
| TheJulia | JayF: I'll just tell myself that people would be horrified of the thoughts that go through my brain | 23:06 |
| TheJulia | JayF: Have a wonderful evening! | 23:06 |
| *** amotoki has quit IRC | 23:09 | |
| mariojv | thanks for reviewing TheJulia, will take a look tomorrow | 23:09 |
| *** pmannidi has joined #openstack-ironic | 23:14 | |
| aNuposic | jlvillal, yes they failed because of this http://paste.openstack.org/show/604754/ | 23:24 |
| aNuposic | jlvillal, i just listed the options that needs to be added | 23:24 |
| aNuposic | but seems that's not the way to do it | 23:24 |
| jlvillal | aNuposic, Google: rst lists | 23:25 |
| aNuposic | jlvillal, ok | 23:26 |
| jlvillal | aNuposic, If you look at the Ocata release notes you can see some lists used. Can look at how they did it. | 23:27 |
| aNuposic | jlvillal, great, will check that | 23:29 |
| *** hshiina has joined #openstack-ironic | 23:32 | |
| *** rbudden has joined #openstack-ironic | 23:41 | |
| *** mat128 has quit IRC | 23:42 | |
| aNuposic | jlvillal, will add them as bullet list, wdyt? | 23:43 |
| *** rloo has quit IRC | 23:44 | |
| jlvillal | aNuposic, I like it :) | 23:45 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!