| joanna | I'd try updating extra with valid {"vif_port_id": "<uuid>"} manually | 00:00 |
|---|---|---|
| bigjools | maybe I need to step up to Newton | 00:00 |
| bigjools | the problem is I am not sure what value should be in there | 00:00 |
| bigjools | it's a Neutron port uuid AFAICT but I don't have any neutron ports apart from the dhcp server running on the provisioning network | 00:01 |
| joanna | port id from neutron's port table | 00:01 |
| joanna | so that's the rootcause - if neutron didn't create them, I'd check neutron's logs | 00:01 |
| bigjools | should ironic-port create have also created a neutron port? | 00:01 |
| * bigjools checks | 00:02 | |
| *** jcoufal_ has quit IRC | 00:03 | |
| *** sacharya has joined #openstack-ironic | 00:03 | |
| *** crushil has quit IRC | 00:04 | |
| bigjools | no errors :/ | 00:04 |
| joanna | any info on creating ports? | 00:04 |
| joanna | did you run any neutron commands? net-create, subnet-create, etc? | 00:05 |
| bigjools | I tailed the log and it didn't show anything apart from logging a connection's IP | 00:05 |
| bigjools | yeah I have a provisioning net/subnet | 00:06 |
| joanna | and ironic log after running port-create? | 00:06 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: DNR/DNM: grenade multitenancy https://review.openstack.org/429942 | 00:06 |
| bigjools | nothing in the log apart from logging connection IP | 00:07 |
| bigjools | which is odd because I have debug logging on | 00:07 |
| joanna | huh... | 00:07 |
| *** crushil has joined #openstack-ironic | 00:07 | |
| *** sacharya has quit IRC | 00:07 | |
| *** alineb has joined #openstack-ironic | 00:08 | |
| bigjools | oh hmm my provisioning network is not shared, is that a problem? | 00:10 |
| joanna | not sure, you can try sharing it... | 00:11 |
| joanna | is ml2 plugin configured? | 00:11 |
| bigjools | yes | 00:11 |
| bigjools | making it shared didn't help | 00:13 |
| joanna | any not-consumed messages on rabbit? | 00:13 |
| joanna | I think I'm getting to the point where I have no idea - but at least we know what didn't fail :) | 00:13 |
| bigjools | heh. No Rabbit looks ok | 00:15 |
| bigjools | I'll compare to devstack and see what's going on there | 00:16 |
| joanna | sorry I couldn't help :( | 00:21 |
| mat128 | rybridges2: I kind of inherited core reviewer "rights" or "permissions" over pyghmi because I happened to know a bit of the protocol and I have an interest. tl;dr I dont really have an answer for you. | 00:21 |
| bigjools | joanna: no worries, you've already been quite useful actually! Thank you. | 00:22 |
| joanna | :) | 00:22 |
| mat128 | rybridges2: if you help me, we can make this a better project :) | 00:23 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: log if 'flat' interface and no cleaning network https://review.openstack.org/429944 | 00:27 |
| *** rbudden has quit IRC | 00:29 | |
| *** sdake has joined #openstack-ironic | 00:31 | |
| *** sdake has quit IRC | 00:45 | |
| jroll | bigjools: if you're running without neutron, you will need [dhcp]/dhcp_provider=True in config | 00:47 |
| bigjools | I am using neutron | 00:47 |
| jroll | (on mitaka, on newton is a bit different) | 00:47 |
| jroll | ah | 00:47 |
| jroll | so then, what you're going to want to do (since nova usually does this bit) | 00:47 |
| bigjools | but let me check that config | 00:47 |
| jroll | is do a neutron port-create on the network you want | 00:47 |
| jroll | and then ironic node-update add extra/vif_port_id=$neutron_port_uuid | 00:48 |
| jroll | (also, I totally meant [dhcp]/dhcp_provider=none for not using neutron, and [dhcp]/dhcp_provider=neutron for use with neutron) | 00:48 |
| bigjools | I was just coming to the conclusion that nova makes the port... thanks, let me try that | 00:48 |
| jroll | :) | 00:48 |
| bigjools | (yeah I am set to use neutron in the config) | 00:48 |
| *** hoangcx has joined #openstack-ironic | 00:49 | |
| *** tuanluong has joined #openstack-ironic | 00:55 | |
| bigjools | success! | 00:56 |
| bigjools | thanks jroll | 00:56 |
| jroll | wooooo | 00:56 |
| jroll | not a problem | 00:56 |
| *** SerenaFeng has joined #openstack-ironic | 00:56 | |
| bigjools | should the node appear as a hypervisor in nova? | 00:56 |
| jroll | yes | 00:57 |
| bigjools | I only see the machine where the VM is guested | 00:57 |
| jroll | nova hypervisor-stats ~= ironic node-list | 00:57 |
| bigjools | the one running n-compute | 00:57 |
| * jroll rusty on that, one sec | 00:58 | |
| bigjools | well they appear in my working devstack so I guess I have a problem on this deployment | 00:58 |
| jroll | it may depend on the provision state and such | 00:59 |
| jroll | e.g. if an ironic node is in maintenance mode, it reports 0 resources (e.g. 0/0MB RAM used) | 00:59 |
| jroll | it still shows in "count" for stats and in hypervisor-list, though | 01:00 |
| jroll | ^ are those the commands you're using, or? | 01:00 |
| jroll | s/stats/hypervisor-stats | 01:00 |
| *** phuongnh has joined #openstack-ironic | 01:01 | |
| bigjools | I use hypervisor list and it only shows the machine running nova-compute | 01:01 |
| jroll | hm, is that set to use the ironic virt driver in config? | 01:02 |
| bigjools | yes | 01:02 |
| jroll | O_o | 01:02 |
| jroll | what's the hypervisor hostname? a uuid or actual host? | 01:02 |
| jroll | also is nova-compute talking to ironic properly? :) | 01:02 |
| bigjools | oh shitting hell, something reset it to libvirt driver. My bad, sorry. | 01:03 |
| jroll | lol, no worries | 01:03 |
| * bigjools blames ansible | 01:03 | |
| jroll | I best get off the computer for the night, but feel free to keep bugging us if you have more questions and such :) | 01:04 |
| jroll | see y'all tomorrow | 01:04 |
| bigjools | cheers joanna | 01:05 |
| bigjools | and cheers jroll | 01:05 |
| bigjools | expect more bugging tomorrow :) | 01:05 |
| *** jrcloud has joined #openstack-ironic | 01:10 | |
| *** SerenaFeng has quit IRC | 01:14 | |
| *** jrcloud has quit IRC | 01:14 | |
| *** SerenaFeng has joined #openstack-ironic | 01:15 | |
| *** alexpilo_ has quit IRC | 01:16 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Validate the network interface before cleaning https://review.openstack.org/429952 | 01:17 |
| *** wrouesnel has quit IRC | 01:17 | |
| *** wrouesnel has joined #openstack-ironic | 01:19 | |
| *** sacharya has joined #openstack-ironic | 01:19 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Validate the network interface before cleaning https://review.openstack.org/429952 | 01:19 |
| *** zhangjl has joined #openstack-ironic | 01:21 | |
| *** BadCub has joined #openstack-ironic | 01:24 | |
| *** sacharya has quit IRC | 01:24 | |
| *** aweeks has joined #openstack-ironic | 01:24 | |
| *** jkilpatr has quit IRC | 01:25 | |
| *** alexpilotti has joined #openstack-ironic | 01:27 | |
| *** rloo has quit IRC | 01:27 | |
| *** baoli has joined #openstack-ironic | 01:31 | |
| *** alexpilotti has quit IRC | 01:31 | |
| *** chedda has quit IRC | 01:32 | |
| *** alexpilotti has joined #openstack-ironic | 01:34 | |
| *** rob0t__ has joined #openstack-ironic | 01:35 | |
| *** alexpilotti has quit IRC | 01:38 | |
| *** wrouesnel has quit IRC | 01:50 | |
| *** wrouesnel has joined #openstack-ironic | 01:51 | |
| *** wrouesnel has quit IRC | 01:55 | |
| *** wrouesnel has joined #openstack-ironic | 01:55 | |
| *** wrouesnel has quit IRC | 01:57 | |
| *** wrouesnel has joined #openstack-ironic | 01:57 | |
| *** sacharya has joined #openstack-ironic | 01:59 | |
| *** wrouesnel has quit IRC | 02:01 | |
| *** owalsh is now known as owalsh-afk | 02:01 | |
| *** wrouesnel has joined #openstack-ironic | 02:01 | |
| *** wrouesnel has quit IRC | 02:02 | |
| *** wrouesnel has joined #openstack-ironic | 02:02 | |
| *** vsaienko has quit IRC | 02:03 | |
| *** dsneddon has quit IRC | 02:03 | |
| *** spartacloud has quit IRC | 02:10 | |
| *** gcb has joined #openstack-ironic | 02:12 | |
| *** jkilpatr has joined #openstack-ironic | 02:20 | |
| *** alexpilotti has joined #openstack-ironic | 02:32 | |
| *** alexpilotti has quit IRC | 02:37 | |
| *** wrouesnel has quit IRC | 02:39 | |
| *** wrouesnel has joined #openstack-ironic | 02:40 | |
| *** wrouesnel has quit IRC | 02:42 | |
| *** wrouesnel has joined #openstack-ironic | 02:43 | |
| *** wrouesnel has quit IRC | 02:44 | |
| *** wrouesnel has joined #openstack-ironic | 02:44 | |
| *** aNuposic has quit IRC | 02:45 | |
| openstackgerrit | Nam Nguyen Hoai proposed openstack/virtualbmc master: Delete support py33 and py34 https://review.openstack.org/430000 | 02:45 |
| *** jkilpatr has quit IRC | 02:48 | |
| *** wrouesnel has quit IRC | 02:51 | |
| *** wrouesnel has joined #openstack-ironic | 02:55 | |
| *** ianychoi has quit IRC | 02:59 | |
| *** ianychoi has joined #openstack-ironic | 02:59 | |
| *** stendulker has joined #openstack-ironic | 03:04 | |
| *** jkilpatr has joined #openstack-ironic | 03:06 | |
| openstackgerrit | gengchc2 proposed openstack/ironic master: Use https instead of http for git.openstack.org https://review.openstack.org/430018 | 03:22 |
| *** vikrant has joined #openstack-ironic | 03:27 | |
| *** alexpilotti has joined #openstack-ironic | 03:32 | |
| *** owalsh has joined #openstack-ironic | 03:35 | |
| *** owalsh-afk has quit IRC | 03:35 | |
| *** alexpilotti has quit IRC | 03:37 | |
| *** jkilpatr has quit IRC | 03:40 | |
| *** links has joined #openstack-ironic | 03:41 | |
| *** spartacloud has joined #openstack-ironic | 03:48 | |
| *** rbudden has joined #openstack-ironic | 03:56 | |
| *** ijw has joined #openstack-ironic | 03:58 | |
| *** SerenaFeng has quit IRC | 03:58 | |
| openstackgerrit | Tovin Seven proposed openstack/python-ironicclient master: Add OSProfiler support to Ironicclient https://review.openstack.org/362331 | 03:59 |
| *** Nisha_Agarwal has joined #openstack-ironic | 04:02 | |
| *** ijw has quit IRC | 04:03 | |
| *** baoli has quit IRC | 04:13 | |
| *** rajinir has quit IRC | 04:16 | |
| *** Nakato_ is now known as Nakato | 04:18 | |
| *** nmathew has joined #openstack-ironic | 04:24 | |
| *** spartacloud has quit IRC | 04:26 | |
| *** alexpilotti has joined #openstack-ironic | 04:35 | |
| *** crushil has quit IRC | 04:39 | |
| *** spartacloud has joined #openstack-ironic | 04:40 | |
| *** alexpilotti has quit IRC | 04:40 | |
| *** spartacloud has quit IRC | 04:41 | |
| *** nicodemos has quit IRC | 04:48 | |
| *** nicodemos has joined #openstack-ironic | 04:48 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic master: Follow-up iRMC power driver for soft reboot/poff https://review.openstack.org/425593 | 04:51 |
| *** sdake has joined #openstack-ironic | 04:52 | |
| *** alexpilotti has joined #openstack-ironic | 04:53 | |
| *** adu has joined #openstack-ironic | 04:55 | |
| *** alexpilotti has quit IRC | 04:58 | |
| *** adu has quit IRC | 05:03 | |
| *** adu has joined #openstack-ironic | 05:11 | |
| *** Daviey has quit IRC | 05:11 | |
| *** rbudden has quit IRC | 05:15 | |
| *** sdake_ has joined #openstack-ironic | 05:35 | |
| *** adu has left #openstack-ironic | 05:36 | |
| *** sdake has quit IRC | 05:36 | |
| *** SerenaFeng has joined #openstack-ironic | 05:39 | |
| *** aparnav has joined #openstack-ironic | 05:40 | |
| *** sasha2 has quit IRC | 05:43 | |
| openstackgerrit | jiang wei proposed openstack/python-ironicclient master: Support i18n for baremetal driver, portgroup cmds https://review.openstack.org/421087 | 05:45 |
| *** sacharya has quit IRC | 05:48 | |
| *** links has quit IRC | 05:51 | |
| openstackgerrit | Aparna proposed openstack/ironic master: Doc for sanitize erase support in iLO drivers https://review.openstack.org/408441 | 05:54 |
| *** rcernin has joined #openstack-ironic | 06:00 | |
| *** moshele has joined #openstack-ironic | 06:00 | |
| *** moshele has quit IRC | 06:03 | |
| *** rcernin has quit IRC | 06:03 | |
| openstackgerrit | Aparna proposed openstack/ironic master: Doc for sanitize erase support in iLO drivers https://review.openstack.org/408441 | 06:03 |
| *** rcernin has joined #openstack-ironic | 06:04 | |
| *** links has joined #openstack-ironic | 06:08 | |
| *** MattMan_1 has quit IRC | 06:14 | |
| *** MattMan_1 has joined #openstack-ironic | 06:14 | |
| *** links has quit IRC | 06:22 | |
| *** sdake_ has quit IRC | 06:22 | |
| *** sabeen has quit IRC | 06:31 | |
| *** jaganathan has joined #openstack-ironic | 06:31 | |
| *** Syed__ has quit IRC | 06:35 | |
| *** links has joined #openstack-ironic | 06:38 | |
| *** Nisha_Agarwal has quit IRC | 06:44 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 06:44 | |
| *** moshele has joined #openstack-ironic | 06:48 | |
| *** fxpester has joined #openstack-ironic | 06:49 | |
| *** alexpilotti has joined #openstack-ironic | 06:55 | |
| *** udesale has joined #openstack-ironic | 06:55 | |
| *** alexpilotti has quit IRC | 06:59 | |
| *** links has quit IRC | 07:07 | |
| openstackgerrit | avnish proposed openstack/bifrost master: Enable DeprecationWarning in test environments https://review.openstack.org/430098 | 07:11 |
| *** makowals has joined #openstack-ironic | 07:18 | |
| *** links has joined #openstack-ironic | 07:20 | |
| *** jaganathan has quit IRC | 07:20 | |
| *** makowals_ has quit IRC | 07:21 | |
| *** jaganathan has joined #openstack-ironic | 07:22 | |
| *** galyna1 has joined #openstack-ironic | 07:28 | |
| *** links has quit IRC | 07:30 | |
| *** tesseract has joined #openstack-ironic | 07:31 | |
| pas-ha | morning ironic :) | 07:34 |
| *** makowals has quit IRC | 07:34 | |
| *** ChubYann has quit IRC | 07:37 | |
| *** links has joined #openstack-ironic | 07:43 | |
| *** makowals has joined #openstack-ironic | 07:44 | |
| *** pcaruana has joined #openstack-ironic | 07:46 | |
| *** pcaruana has quit IRC | 07:48 | |
| *** makowals has quit IRC | 07:51 | |
| *** pcaruana has joined #openstack-ironic | 07:55 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add should_bind_port property to NetworkInterface https://review.openstack.org/429893 | 08:02 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Fix log formating in ironic/common/neutron https://review.openstack.org/430111 | 08:02 |
| galyna1 | morning all :) | 08:02 |
| *** makowals has joined #openstack-ironic | 08:03 | |
| *** Nisha_Agarwal has quit IRC | 08:03 | |
| *** alexpilotti has joined #openstack-ironic | 08:04 | |
| *** alexpilotti has quit IRC | 08:09 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 08:10 | |
| *** makowals has quit IRC | 08:13 | |
| *** makowals has joined #openstack-ironic | 08:15 | |
| *** alexpilotti has joined #openstack-ironic | 08:34 | |
| *** Nisha_Agarwal has quit IRC | 08:37 | |
| *** alexpilotti has quit IRC | 08:38 | |
| *** makowals has quit IRC | 08:42 | |
| *** mjura has joined #openstack-ironic | 08:44 | |
| *** priteau has joined #openstack-ironic | 08:45 | |
| mkrai | TheJulia: Hi | 08:47 |
| *** Nisha_Agarwal has joined #openstack-ironic | 08:48 | |
| openstackgerrit | Merged openstack/proliantutils master: Add HPSUM firmware update support https://review.openstack.org/422004 | 08:49 |
| *** milan has joined #openstack-ironic | 08:53 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Retry to connect to RPC if failed https://review.openstack.org/376462 | 08:54 |
| *** makowals has joined #openstack-ironic | 08:55 | |
| *** makowals has quit IRC | 08:56 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-ironic | 09:00 | |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: Adds network check in upgrade phase in devstack https://review.openstack.org/428043 | 09:03 |
| vdrok | good morning ironic | 09:11 |
| *** makowals has joined #openstack-ironic | 09:12 | |
| *** makowals has quit IRC | 09:15 | |
| *** makowals has joined #openstack-ironic | 09:17 | |
| *** Daviey has joined #openstack-ironic | 09:21 | |
| sergek | Morning vdrok galyna1 pas-ha | 09:22 |
| *** milan has quit IRC | 09:22 | |
| vdrok | morning sergek pas-ha galyna1 and mkrai | 09:22 |
| *** milan has joined #openstack-ironic | 09:23 | |
| *** rbudden has joined #openstack-ironic | 09:23 | |
| *** lucas-afk is now known as lucasagomes | 09:29 | |
| lucasagomes | morning all | 09:29 |
| *** strigazi1AFK is now known as strigazi | 09:29 | |
| Nisha_Agarwal | morning all | 09:30 |
| *** derekh has joined #openstack-ironic | 09:33 | |
| *** mgoddard_ has joined #openstack-ironic | 09:34 | |
| *** litao has quit IRC | 09:34 | |
| *** litao has joined #openstack-ironic | 09:35 | |
| *** SerenaFeng has quit IRC | 09:35 | |
| *** SerenaFeng has joined #openstack-ironic | 09:36 | |
| *** rbudden has quit IRC | 09:39 | |
| *** SerenaFeng has quit IRC | 09:42 | |
| *** zhangjl1 has joined #openstack-ironic | 09:47 | |
| *** zhangjl has quit IRC | 09:48 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add should_bind_port property to NetworkInterface https://review.openstack.org/429893 | 09:49 |
| vdrok | morning lucasagomes and Nisha_Agarwal | 09:51 |
| Nisha_Agarwal | vdrok, :) morning | 09:52 |
| *** tovin07 has quit IRC | 09:56 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Configure and use SSL-related requests options https://review.openstack.org/398992 | 09:57 |
| *** jschwarz has quit IRC | 10:01 | |
| *** jschwarz has joined #openstack-ironic | 10:01 | |
| *** hoangcx has quit IRC | 10:08 | |
| *** skazi has quit IRC | 10:11 | |
| jtomasek | dtantsur|afk: ping when you're around | 10:13 |
| *** aparnav has quit IRC | 10:14 | |
| *** aparnav has joined #openstack-ironic | 10:15 | |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: WIP: TEST GRENADE DEV https://review.openstack.org/397126 | 10:19 |
| *** rob0t__ has quit IRC | 10:21 | |
| lucasagomes | didn't we have a WIP redfish driver using pytohn-redfish proposed at some point ? | 10:22 |
| * lucasagomes can't find the patches | 10:22 | |
| *** skazi has joined #openstack-ironic | 10:23 | |
| *** udesale has quit IRC | 10:31 | |
| *** udesale has joined #openstack-ironic | 10:31 | |
| *** mvpnitesh has joined #openstack-ironic | 10:34 | |
| mvpnitesh | @Dmitry Tantsur: Please review this https://review.openstack.org/#/c/379436/ | 10:37 |
| patchbot | patch 379436 - python-ironicclient - Fixed wrap from taking negative values | 10:37 |
| *** zhangjl1 has quit IRC | 10:42 | |
| *** amarao has joined #openstack-ironic | 10:42 | |
| *** sambetts_ is now known as sambetts | 10:43 | |
| vdrok | lucasagomes: I don't think so, I've seen only spec | 10:44 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Add port groups configuration documentation https://review.openstack.org/429826 | 10:45 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Add port groups configuration documentation https://review.openstack.org/429826 | 10:46 |
| *** stendulker has quit IRC | 10:48 | |
| *** nmathew has quit IRC | 10:51 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Pass session directly to swiftclient https://review.openstack.org/430197 | 10:55 |
| lucasagomes | vdrok, I see yeah that looks like it :-/ | 11:01 |
| *** dtantsur|afk is now known as dtantsur | 11:01 | |
| dtantsur | morning ironic | 11:02 |
| dtantsur | jtomasek, ping | 11:02 |
| jtomasek | dtantsur: hi, I've been looking at http://docs.openstack.org/developer/ironic-inspector/usage.html#discovery | 11:03 |
| jtomasek | dtantsur: I am trying to figure out if it would be possible to configure that behavior explicitly for a certain introspection run | 11:04 |
| jtomasek | dtantsur: so it would be possible to automatically register nodes from GUI | 11:04 |
| dtantsur | jtomasek, "for certain introspection run" implies per node, and we don't have nodes yet. so this is kind of chicken-and-egg | 11:04 |
| dtantsur | jtomasek, if you have node information, you don't need discovery. if you do not, you probably cannot start it from GUI | 11:05 |
| jtomasek | dtantsur: I see, when is that hook run actually? I thought it is done during introspection | 11:05 |
| dtantsur | jtomasek, it's run for unknown nodes to implement e.g. discovery. if you know something about a node, it's not discovery any more | 11:06 |
| dtantsur | with discovery nodes just show up, and then you can specify more data about them | 11:06 |
| TheJulia | lucasagomes: maybe on someone's github, but I don't remember a proposed driver. | 11:06 |
| dtantsur | lucasagomes, I haven't seen one either | 11:07 |
| TheJulia | Good morning everyone | 11:07 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Fix object save after refresh failure https://review.openstack.org/354744 | 11:07 |
| lucasagomes | TheJulia, dtantsur yeah :-/ I must have confused it with something else | 11:07 |
| lucasagomes | TheJulia, morning | 11:07 |
| jtomasek | dtantsur: ok, so how/when is that discovery triggered? My usecase is that we'd like to automatically register nodes from GUI (as an alternative option to registering them using instackenv.json) | 11:08 |
| jtomasek | (TripleO UI) | 11:08 |
| sambetts | jtomasek: discovery is for unknown nodes to **automatically** be discovered, if you are adding nodes manually you want to enroll them directy in Ironic and then trigger inspection on them | 11:09 |
| dtantsur | jtomasek, discovery happens when new nodes are powered on and boot IPA, no matter how exactly it's achieved | 11:09 |
| dtantsur | jtomasek, I'm not sure how exactly you plan "to automatically register nodes from GUI" in this context. could you specify the exact workflow? | 11:10 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Keystone session for swift-temp-url generation https://review.openstack.org/410764 | 11:11 |
| *** phuongnh has quit IRC | 11:12 | |
| jtomasek | dtantsur: so currently we need to register nodes in tripleo-ui either by providing instackenv.json or manually add them via form. What we think would be nice is if we could let user provide something like a subnet and ipmi credentials and ironic would register all nodes it finds | 11:13 |
| vdrok | morning dtantsur , TheJulia | 11:14 |
| sambetts | jtomasek: for inspector auto discovery to work like that you just need to wrap an ipmi power on command in a for loop, as long as the nodes are able to be booted by inspector, simply turning them on is enough to make them be discovered | 11:16 |
| dtantsur | jtomasek, this is not implemented, and is definitely not in scope for ironic-inspector. there were similar asks for some vendor hardware (HPE, Dell), but we don't have the mechanism at all. | 11:16 |
| mvpnitesh | dtantsur: Please review this https://review.openstack.org/#/c/379436/ | 11:17 |
| patchbot | patch 379436 - python-ironicclient - Fixed wrap from taking negative values | 11:17 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Cleanup hung iscsi session https://review.openstack.org/388092 | 11:17 |
| jtomasek | dtantsur: ok, so do I understand it correctly, that we can autodiscover nodes once they boot up but I have no mean to power them on until Ironic knows about them? | 11:17 |
| *** owalsh_ has joined #openstack-ironic | 11:18 | |
| dtantsur | mvpnitesh, I've reviewed it already | 11:18 |
| *** owalsh has quit IRC | 11:18 | |
| dtantsur | jtomasek, this is quite right | 11:18 |
| jtomasek | dtantsur, sambetts: ok, thanks | 11:19 |
| dtantsur | np | 11:19 |
| mvpnitesh | dtantsur: Thank you | 11:19 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Add port groups configuration documentation https://review.openstack.org/429826 | 11:23 |
| *** makowals_ has joined #openstack-ironic | 11:24 | |
| *** makowals has quit IRC | 11:25 | |
| openstackgerrit | jingtao liang proposed openstack/ironic master: Using utcnow instead of now in qemu.py https://review.openstack.org/430207 | 11:28 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic master: SNMP agent support for OOB inspection for iLO Drivers https://review.openstack.org/372428 | 11:30 |
| *** rob0t__ has joined #openstack-ironic | 11:32 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Retry to connect to RPC if failed https://review.openstack.org/376462 | 11:36 |
| *** rob0t__ has quit IRC | 11:36 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic master: Add ironic portgroup CRUD notifications https://review.openstack.org/427281 | 11:37 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Keystone session for swift-temp-url generation https://review.openstack.org/410764 | 11:41 |
| *** spartacloud has joined #openstack-ironic | 11:43 | |
| *** owalsh_ is now known as owalsh | 11:43 | |
| mkrai | TheJulia: Hi, I wanted to talk about https://review.openstack.org/#/c/427077/ | 11:45 |
| patchbot | patch 427077 - ironic - Fix directories permission | 11:45 |
| *** spartacloud has quit IRC | 11:47 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-specs master: Tenant-aware baremetal nodes https://review.openstack.org/415512 | 11:54 |
| *** rob0t__ has joined #openstack-ironic | 11:54 | |
| *** rob0t__ has quit IRC | 11:58 | |
| *** jschwarz has quit IRC | 12:00 | |
| *** jkilpatr has joined #openstack-ironic | 12:00 | |
| *** rob0t__ has joined #openstack-ironic | 12:02 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Allow to attach/detach VIFs to active ironic nodes https://review.openstack.org/424723 | 12:05 |
| *** rob0t__ has quit IRC | 12:06 | |
| *** tuanluong has quit IRC | 12:07 | |
| *** rob0t__ has joined #openstack-ironic | 12:12 | |
| mat128 | morning | 12:12 |
| vdrok | morning mat128 | 12:12 |
| milan | morning Ironic! :) | 12:13 |
| mat128 | jtomasek: the feature you are describing is possibly out of scope for ironic-inspector (whose job is to profile a machine booted into a specific env) but definitely not out of scope of the Ironic project | 12:13 |
| milan | morning aarefiev, dtantsur, jtomasek, lucasagomes, mat128, sambetts TheJulia vdrok and all! :) | 12:13 |
| lucasagomes | milan, o/ | 12:14 |
| vdrok | morning milan :) | 12:15 |
| milan | aarefiev, hey :) just reported https://bugs.launchpad.net/ironic-inspector/+bug/1662494 ; is it close to what you experienced? | 12:16 |
| openstack | Launchpad bug 1662494 in Ironic Inspector "missing error state transitions in the inspect code path" [Undecided,New] - Assigned to milan k (vetrisko) | 12:16 |
| *** rob0t__ has quit IRC | 12:16 | |
| *** rob0t__ has joined #openstack-ironic | 12:17 | |
| aarefiev | milan: morning, thanks, I have patch for this, was going to file a bug | 12:17 |
| milan | aarefiev, cool, mind uploading? dtantsur might want to include that in the release | 12:18 |
| *** ccamacho is now known as ccamacho|lunch | 12:18 | |
| *** lucasagomes is now known as lucas-hungry | 12:18 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: DNR/DNM Testing interface hotplug functionallity https://review.openstack.org/425123 | 12:18 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: [Devstack] Allow add several ports to node https://review.openstack.org/425112 | 12:18 |
| *** jcoufal has joined #openstack-ironic | 12:19 | |
| aarefiev | milan: it's small refactoring, I need to fix unit tests | 12:19 |
| milan | aarefiev, cool, thanks! /me eager to review ;) | 12:20 |
| aarefiev | milan: maybe I can split it to fix only certain issue | 12:20 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Allow to attach/detach VIFs to active ironic nodes https://review.openstack.org/424723 | 12:20 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: DNR/DNM Testing interface hotplug functionallity https://review.openstack.org/425123 | 12:20 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: [Devstack] Allow add several ports to node https://review.openstack.org/425112 | 12:20 |
| milan | aarefiev, as you wish, or push WIP, cool w/ me to check | 12:21 |
| aarefiev | milan: ok, I'll fix unit tests, and push it up | 12:22 |
| milan | aarefiev, thx! | 12:22 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic master: SNMP agent support for OOB inspection for iLO Drivers https://review.openstack.org/372428 | 12:23 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic master: SNMP agent support for OOB inspection for iLO Drivers https://review.openstack.org/372428 | 12:28 |
| *** fellypefca has joined #openstack-ironic | 12:29 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: DNR/DNM: grenade multitenancy https://review.openstack.org/429942 | 12:35 |
| *** gcb has quit IRC | 12:36 | |
| *** sdake has joined #openstack-ironic | 12:37 | |
| *** gcb has joined #openstack-ironic | 12:37 | |
| aarefiev | milan: looks like I've found more high priority bug, I'd like to see in release | 12:40 |
| *** sdake_ has joined #openstack-ironic | 12:41 | |
| *** sdake has quit IRC | 12:42 | |
| *** vsaienko has joined #openstack-ironic | 12:43 | |
| *** sborkows has joined #openstack-ironic | 12:44 | |
| *** gcb has quit IRC | 12:45 | |
| jroll | morning y'all | 12:47 |
| *** aparnav has quit IRC | 12:48 | |
| *** links has quit IRC | 12:48 | |
| fellypefca | jroll morning :) | 12:48 |
| dtantsur | morning jroll | 12:51 |
| vdrok | good morning jroll | 12:51 |
| nicodemos | morning all, dtantsur, jroll, fellypefca, vdrok | 12:51 |
| dtantsur | morning nicodemos | 12:52 |
| vdrok | morning nicodemos :) | 12:52 |
| nicodemos | Hey, Ironicers, Is there a documentation about the metrics library? https://github.com/openstack/ironic-lib/blob/master/ironic_lib/metrics.py | 12:52 |
| *** gcb has joined #openstack-ironic | 12:53 | |
| jroll | nicodemos: http://docs.openstack.org/developer/ironic-lib/#metrics and http://docs.openstack.org/developer/ironic/deploy/metrics.html | 12:53 |
| jroll | (library and operator docs) | 12:53 |
| nicodemos | Thanks, jroll | 12:53 |
| jroll | np | 12:54 |
| *** vikrant has quit IRC | 12:56 | |
| *** jcoufal has quit IRC | 13:01 | |
| *** Nisha_Agarwal has quit IRC | 13:03 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 13:03 | |
| dtantsur | jroll, this could use the 3rd +2 https://review.openstack.org/#/c/429694/ | 13:05 |
| patchbot | patch 429694 - ironic - [devstack] Allow using "ipmi" hardware type | 13:05 |
| jroll | dtantsur: done | 13:06 |
| dtantsur | thnx | 13:06 |
| jroll | dtantsur: whomp, your project-config change is in merge conflict | 13:07 |
| dtantsur | >_< | 13:08 |
| milan | morning fellypefca, jroll, nicodemos! :) | 13:08 |
| *** gcb has quit IRC | 13:09 | |
| jroll | hey milan | 13:09 |
| milan | aarefiev, ack, feel free to forward, I can help hack on those | 13:09 |
| milan | aarefiev, https://bugs.launchpad.net/ironic-inspector/+bug/1662504 yeah I copy-pasted the update part from you the other day O:-) | 13:11 |
| nicodemos | hey, milan | 13:11 |
| openstack | Launchpad bug 1662504 in Ironic Inspector "Migration test for d00d6e3f38c4 fails on mysql" [High,In progress] - Assigned to Anton Arefiev (aarefiev) | 13:11 |
| *** trown has joined #openstack-ironic | 13:13 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Extend standalone tests with 'ipmi' driver https://review.openstack.org/430245 | 13:14 |
| *** chlong has joined #openstack-ironic | 13:15 | |
| *** lucas-hungry is now known as lucasagomes | 13:20 | |
| *** jheroux has joined #openstack-ironic | 13:21 | |
| *** ralonsoh has joined #openstack-ironic | 13:23 | |
| *** dprince has joined #openstack-ironic | 13:24 | |
| *** ccamacho|lunch is now known as ccamacho | 13:24 | |
| openstackgerrit | M V P Nitesh proposed openstack/python-ironicclient master: Fixed wrap from taking negative values https://review.openstack.org/379436 | 13:27 |
| *** rob0t__ has quit IRC | 13:29 | |
| *** rob0t__ has joined #openstack-ironic | 13:32 | |
| openstackgerrit | M V P Nitesh proposed openstack/python-ironicclient master: Fixed wrap from taking negative values https://review.openstack.org/379436 | 13:33 |
| *** Goneri has joined #openstack-ironic | 13:36 | |
| *** hoangcx has joined #openstack-ironic | 13:36 | |
| *** sdake_ is now known as sdkae | 13:37 | |
| dtantsur | jroll, TheJulia, FYI: working on RDO packaging for ironic-ui https://review.rdoproject.org/r/#/c/4886/ :) | 13:40 |
| jroll | woot | 13:40 |
| *** sdkae is now known as sdake | 13:43 | |
| *** mvpnitesh has quit IRC | 13:44 | |
| *** amotoki has joined #openstack-ironic | 13:44 | |
| *** trown|brb has joined #openstack-ironic | 13:45 | |
| *** trown has quit IRC | 13:45 | |
| *** trown|brb is now known as trown|mtg | 13:45 | |
| *** trown|mtg is now known as trown | 13:45 | |
| *** mtanino has quit IRC | 13:47 | |
| *** baoli has joined #openstack-ironic | 13:48 | |
| *** udesale__ has joined #openstack-ironic | 13:50 | |
| *** udesale has quit IRC | 13:51 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Extend standalone tests with 'ipmi' driver https://review.openstack.org/430245 | 13:52 |
| *** cdearborn has joined #openstack-ironic | 13:53 | |
| *** vsaienko has quit IRC | 13:53 | |
| dtantsur | bcornec2, TheJulia or whoever might know: what's the relationship between https://github.com/DMTF/python-redfish-library and https://github.com/openstack/python-redfish/ ? | 13:53 |
| *** jaganathan has quit IRC | 13:56 | |
| *** vsaienko has joined #openstack-ironic | 13:58 | |
| *** baoli has quit IRC | 13:58 | |
| * lucasagomes tested it | 13:59 | |
| lucasagomes | dtantsur, doesn't seem to have any to be honest | 13:59 |
| *** baoli has joined #openstack-ironic | 13:59 | |
| *** sasha2 has joined #openstack-ironic | 13:59 | |
| *** rob0t__ has quit IRC | 14:00 | |
| *** rob0t__ has joined #openstack-ironic | 14:02 | |
| *** rob0t__ has quit IRC | 14:07 | |
| *** galyna1 has quit IRC | 14:08 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Retry to connect to RPC if failed https://review.openstack.org/376462 | 14:10 |
| *** vsaienko has quit IRC | 14:14 | |
| *** rloo has joined #openstack-ironic | 14:19 | |
| *** rob0t__ has joined #openstack-ironic | 14:22 | |
| *** rob0t__ has quit IRC | 14:26 | |
| *** mjturek has joined #openstack-ironic | 14:27 | |
| openstackgerrit | Gábor Antal proposed openstack/python-ironicclient master: Handle log message interpolation by the logger https://review.openstack.org/430273 | 14:30 |
| *** rob0t__ has joined #openstack-ironic | 14:32 | |
| *** Nisha_Agarwal has quit IRC | 14:33 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: DNR/DNM: grenade multitenancy https://review.openstack.org/429942 | 14:33 |
| *** Nisha_Agarwal has joined #openstack-ironic | 14:33 | |
| *** vsaienko has joined #openstack-ironic | 14:34 | |
| *** rob0t__ has quit IRC | 14:36 | |
| *** nmathew has joined #openstack-ironic | 14:40 | |
| sambetts | QQ When i do a rebuild, is it a full clean reinstall of the image in instance_info?? | 14:40 |
| jroll | sambetts: yes | 14:41 |
| jroll | well | 14:41 |
| *** nmathew has quit IRC | 14:41 | |
| jroll | a full clean reinstall of the image in the nova instance record | 14:41 |
| jroll | :) | 14:41 |
| *** spartacloud has joined #openstack-ironic | 14:41 | |
| sambetts | jroll: I'm using standalone right now, so manually doing set-provision-state rebuild | 14:41 |
| jroll | ah, probably instance_info then, yeah | 14:42 |
| sambetts | cool :) | 14:42 |
| *** rob0t__ has joined #openstack-ironic | 14:43 | |
| jroll | sambetts: yeah, turns out it just calls deploy | 14:43 |
| jroll | and will check for new kernel/ramdisk if it's a rebuild | 14:43 |
| jroll | see conductor/manager.py do_node_deploy | 14:43 |
| sambetts | ok awesome, thanks :) | 14:45 |
| *** rob0t__ has quit IRC | 14:47 | |
| openstackgerrit | Merged openstack/ironic master: [devstack] Allow using "ipmi" hardware type https://review.openstack.org/429694 | 14:49 |
| *** srobert has joined #openstack-ironic | 14:52 | |
| *** dyasny has quit IRC | 14:52 | |
| vsaienk0 | dtantsur: standalone tests with ipmi driver passed http://logs.openstack.org/45/430245/2/experimental/gate-ironic-dsvm-standalone-ubuntu-xenial-nv/768ba52/console.html#_2017-02-07_14_39_54_384452 | 14:53 |
| dtantsur | awesome! | 14:53 |
| *** owalsh is now known as owalsh-brb | 14:53 | |
| *** mtanino has joined #openstack-ironic | 14:54 | |
| *** rob0t__ has joined #openstack-ironic | 14:55 | |
| *** owalsh-brb has quit IRC | 14:57 | |
| *** vsaienko has quit IRC | 14:58 | |
| *** rob0t__ has quit IRC | 15:00 | |
| *** rpioso has joined #openstack-ironic | 15:00 | |
| *** owalsh-brb has joined #openstack-ironic | 15:01 | |
| *** rob0t__ has joined #openstack-ironic | 15:02 | |
| *** crushil has joined #openstack-ironic | 15:03 | |
| *** rama_y has joined #openstack-ironic | 15:03 | |
| *** rob0t__ has quit IRC | 15:06 | |
| *** vsaienko has joined #openstack-ironic | 15:06 | |
| *** dyasny has joined #openstack-ironic | 15:07 | |
| *** owalsh-brb is now known as owalsh | 15:08 | |
| *** fxpester has quit IRC | 15:11 | |
| *** rob0t__ has joined #openstack-ironic | 15:14 | |
| *** milan has quit IRC | 15:14 | |
| *** rob0t__ has quit IRC | 15:18 | |
| *** sacharya has joined #openstack-ironic | 15:19 | |
| *** trown is now known as trown|brb | 15:21 | |
| *** rob0t__ has joined #openstack-ironic | 15:22 | |
| *** dyasny has quit IRC | 15:22 | |
| *** trown|brb is now known as trown | 15:23 | |
| *** hoangcx has quit IRC | 15:24 | |
| *** dyasny has joined #openstack-ironic | 15:26 | |
| *** rob0t__ has quit IRC | 15:26 | |
| NobodyCam | Good morning ironic'ers | 15:31 |
| *** rob0t__ has joined #openstack-ironic | 15:32 | |
| dtantsur | morning NobodyCam | 15:33 |
| NobodyCam | Morning dtantsur :) | 15:33 |
| NobodyCam | It's coffee time | 15:34 |
| mat128 | sambetts: yes, full reinstall, without the cleaning part (e.g. no ISE/shred) | 15:35 |
| mat128 | oops, 1h later :( | 15:35 |
| sambetts | mat128: haha, thanks :) | 15:35 |
| *** fellypefca has quit IRC | 15:36 | |
| *** rob0t__ has quit IRC | 15:36 | |
| *** vsaienko has quit IRC | 15:38 | |
| *** moshele has quit IRC | 15:39 | |
| lucasagomes | NobodyCam, mat128 o/ | 15:41 |
| rloo | hi and good morning all, NobodyCam (early again!), dtantsur, mat128, sambetts, lucasagomes | 15:41 |
| lucasagomes | rloo, morning | 15:42 |
| rloo | dtantsur, jroll: wrt classic drivers, can a node with a classic driver also have a valid node.x_interface? eg http://docs-draft.openstack.org/60/428460/6/check/gate-ironic-api-ref/b54f5e2//api-ref/build/html/?expanded=create-node-detail | 15:42 |
| rloo | ^^ click on 'Create Node', 'detail' (cuz that link is argh). see power_interface is set. | 15:43 |
| rloo | jroll, dtantsur: that is wrt the api-ref patch: https://review.openstack.org/#/c/428460/ | 15:43 |
| patchbot | patch 428460 - ironic - Update api-ref for dynamic drivers | 15:43 |
| jroll | rloo: yeah, I guess that seems wrong | 15:43 |
| rloo | jroll, dtantsur: might be good for you to review that patch. I feel like it is a test to see if we understand how it works :) | 15:44 |
| dtantsur | morning rloo | 15:44 |
| rloo | mariojv: ^^ | 15:44 |
| dtantsur | rloo, I guess it should not (except for network and volume) | 15:44 |
| rloo | jroll: not sure if our code is allowing that or not, i thought those examples were actually generated | 15:44 |
| jroll | rloo: yes, I've been reviewing, that does seem wrong | 15:45 |
| *** rob0t__ has joined #openstack-ironic | 15:46 | |
| mariojv | thanks rloo | 15:47 |
| rloo | jroll: am looking at our code and it allows setting them regardless of classic or dynamic. as long as microversion is ok. | 15:47 |
| mariojv | not sure how the creation code got generated, will look later | 15:47 |
| mariojv | i thought that was strange though | 15:47 |
| rloo | mariojv: i think the code allowed that. we probably need to fix :) | 15:47 |
| mariojv | ack | 15:47 |
| rloo | mariojv: thx for testing :D | 15:47 |
| *** rbudden has joined #openstack-ironic | 15:47 | |
| jroll | rloo: I'm in a meeting, can you summarize in a bug or something | 15:47 |
| mariojv | rloo: jroll: should i wait to update api-ref then? | 15:47 |
| rloo | jroll: yeah, later. am going into a meeting soon. and can't summarize in a bug unless we've finished the feature :) | 15:48 |
| * jroll done with meeting now | 15:49 | |
| *** rob0t__ has quit IRC | 15:50 | |
| jroll | rloo: ok yeah, there's a bug there, sigh | 15:51 |
| rloo | jroll: no need to sigh now. If it was next week though ;) | 15:51 |
| rloo | jroll: you going to fix or do you want me to do it (later today if me) | 15:52 |
| jroll | rloo: my calendar sayd I have approximately one hour of working time today :( | 15:52 |
| rloo | jroll: no worries, you will hopefully have more time when you aren't a ptl :D I'll fix it later today. It should be an easy one. | 15:53 |
| mat128 | JayF: I think I have found the wrong root cause for my missing Babel files :( | 15:53 |
| jroll | rloo: none of this is ptl stuff today, sadly | 15:54 |
| mat128 | JayF: Nothing gets installed when using the tarball from tarballs.o.o, I got further, but IPA can't find oslo.config | 15:54 |
| mariojv | thanks for fixing rloo | 15:54 |
| jroll | just a long day of meetings | 15:54 |
| rloo | jroll: i was afraid to ask... :( Maybe you should be ptl again so you could have an excuse? ;) | 15:54 |
| jroll | heh | 15:54 |
| * jroll is still PTL for today | 15:54 | |
| * jroll tries to dream up lame duck orders to make | 15:54 | |
| mariojv | rloo: if you ping me when that's done, i can update api-ref patch then | 15:54 |
| * rloo feels sorry for jroll | 15:54 | |
| rloo | mariojv: will do | 15:54 |
| mariojv | thx | 15:54 |
| mariojv | jroll: start issuing pardons and ptl medal of freedom | 15:55 |
| * jroll removes all the -2s as 'pardons' | 15:55 | |
| mat128 | JayF: Using Github.com's tree tarball works as expected. I think it has to do with PBR not finding the right files to include in the sdist when using a tarball rather than a git tree | 15:55 |
| mat128 | JayF: I'll file a proper bug when I get to the right root cause | 15:56 |
| *** sayantani01 has joined #openstack-ironic | 15:56 | |
| *** milan has joined #openstack-ironic | 15:57 | |
| *** sayantani01 has quit IRC | 16:01 | |
| *** rob0t__ has joined #openstack-ironic | 16:02 | |
| JayF | mat128: interesting | 16:04 |
| JayF | mat128: (and others) I'm fighting an interenet outage so not going to be too much help until that's over. I'm on tethered mobile internet that barely works | 16:04 |
| *** rob0t__ has quit IRC | 16:06 | |
| dtantsur | mat128, "PBR not finding the right files" is quite common | 16:08 |
| jroll | rloo: "wrt my comment about putting dynamic drivers first before classic, maybe hold off on that until we test more and get some feedback from users" <- note we only have ipmi hardware type right now, so I think we probably consider this experimental, wdyt dtantsur | 16:09 |
| dtantsur | yeah, it's experimental for ocata (we also have irmc btw) | 16:09 |
| rloo | jroll: oh, 'experimental' is fine i think... | 16:09 |
| rloo | jroll: if it is experimental, do we want to add to api-ref? | 16:10 |
| jroll | rloo: yes | 16:10 |
| jroll | rloo: it's in the api, it should be documented | 16:10 |
| rloo | jroll: ok, then we should put in the api-ref, that dynamic drivers is experimental...? | 16:10 |
| *** mjura has quit IRC | 16:11 | |
| jroll | rloo: I think I'd rather put that in driver docs or something | 16:11 |
| jroll | the APIs aren't experimental (imo), only the feature as a whole | 16:11 |
| rloo | jroll: well, someone looking at api-ref will think it works. | 16:11 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Add support for DBDeadlock handling https://review.openstack.org/394413 | 16:11 |
| jroll | rloo: I don't think someone could use this with only the api-ref changes | 16:12 |
| jroll | also, it does work | 16:12 |
| rloo | jroll: but it is experimental... | 16:12 |
| rloo | jroll: let's see what JayF & TheJulia say. They are opinionated and support the operators :) | 16:13 |
| jroll | rloo: "works" and "is experimental" aren't mutually exclusive | 16:13 |
| JayF | Works + Experimental means "play with this for a cycle so you can know what's up next cycle" | 16:13 |
| JayF | I like that :) | 16:13 |
| *** rama_y has quit IRC | 16:13 | |
| rloo | JayF: so if you were looking at api-ref and it doesn't say that dynamic drivers is experimental, you're good with that? | 16:14 |
| JayF | I mean, I guess in my mind I don't think of the api-ref as an operations guide | 16:14 |
| jroll | I'm not opposed to adding that to api-ref, but yeah ^^ agree with that | 16:14 |
| JayF | as an operator it would certainly have not been my first stop | 16:14 |
| rloo | JayF, jroll: I'll leave it for you then. I'm not an operator. | 16:15 |
| *** rcernin has quit IRC | 16:16 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Trivial: mock sleep in introspection tests https://review.openstack.org/430340 | 16:16 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Starting -> Error transaction refactoring https://review.openstack.org/430341 | 16:16 |
| *** pcaruana has quit IRC | 16:17 | |
| JayF | mat128: https://review.openstack.org/#/c/428704 can your either reconsider or clarify your -1 review here? | 16:17 |
| patchbot | patch 428704 - ironic-python-agent - imagebuild: tinyipa: Make the install-deps.sh SUSE... | 16:17 |
| *** rob0t__ has joined #openstack-ironic | 16:17 | |
| *** jaosorior has joined #openstack-ironic | 16:18 | |
| rybridges2 | mat128: I would be interested in doing some work on pyghmi if I can find the time. First thing I would do is add a key of some sort for all of the magic numbers that are being used everywhere. Some already exist in the constants.py, but not everything | 16:20 |
| *** spartacloud is now known as zackf | 16:20 | |
| rybridges2 | Next thing i would do is try to deal with these timeout problems | 16:20 |
| *** rob0t__ has quit IRC | 16:21 | |
| *** aNuposic has joined #openstack-ironic | 16:22 | |
| *** rob0t__ has joined #openstack-ironic | 16:22 | |
| rybridges2 | for that i think we just need to find the right place to put a retry mechanism in when a timeout occurs. i have that done in the ironic code already, but would be nice to have it in pyghmi | 16:22 |
| dtantsur | hmm, who can help me with ironic-ui please? I can't see anything ironic-related in the resulting horizon... | 16:22 |
| *** kromanenko has joined #openstack-ironic | 16:25 | |
| jlvillal | vsaienk0: Trying to catch up with you. So was the grenade plugin for networking-generic-switch able to resolve the issue? I checked the latest build in Zuul and saw it fail early. | 16:25 |
| vsaienk0 | jlvillal: only one test network_basic_ops is failing, I didn't have enough time to research, but other tests that boot VMs passing | 16:26 |
| *** rob0t__ has quit IRC | 16:26 | |
| jlvillal | vsaienk0: Thanks :) | 16:30 |
| *** udesale__ has quit IRC | 16:31 | |
| vsaienk0 | jlvillal, rloo please add to your review queue https://review.openstack.org/#/c/430245 related to CI improvement | 16:31 |
| patchbot | patch 430245 - ironic - Extend standalone tests with 'ipmi' driver | 16:31 |
| *** Nisha_Agarwal has quit IRC | 16:31 | |
| jlvillal | vsaienk0: Okay. After my meeting :) | 16:31 |
| vsaienk0 | jlvillal, rloo: sorry first patch is https://review.openstack.org/#/c/423556 | 16:31 |
| patchbot | patch 423556 - ironic - Add Ironic standalone tests | 16:31 |
| *** baoli has quit IRC | 16:32 | |
| *** rob0t__ has joined #openstack-ironic | 16:32 | |
| rloo | vsaienk0 is reeling us in :) will try to look later today. are those your most important patches for this week? | 16:32 |
| *** sayantani01 has joined #openstack-ironic | 16:32 | |
| * rloo trying to focus on stuff for release this week | 16:33 | |
| lucasagomes | rybridges2, yeah pyghmi needs some code refactor indeed | 16:33 |
| *** patriciadomin has quit IRC | 16:34 | |
| *** openstackgerrit has quit IRC | 16:35 | |
| *** rob0t__ has quit IRC | 16:36 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 16:37 | |
| *** vsaienko has joined #openstack-ironic | 16:37 | |
| *** openstackgerrit has joined #openstack-ironic | 16:37 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Add safe_for_db_retry decorator https://review.openstack.org/430351 | 16:37 |
| NobodyCam | Morning rloo & lucasagomes :) o/ :) | 16:38 |
| lucasagomes | :-) | 16:39 |
| NobodyCam | may I just say congrats to all the PTL candidates!!! | 16:42 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Add port groups configuration documentation https://review.openstack.org/429826 | 16:43 |
| *** sborkows has quit IRC | 16:43 | |
| * jroll lunch and such | 16:43 | |
| NobodyCam | :L jroll only a few more hours | 16:44 |
| vdrok | leaving early today, good night everyone! | 16:44 |
| NobodyCam | night vdrok :) | 16:44 |
| jroll | NobodyCam: \o/ | 16:44 |
| *** rob0t__ has joined #openstack-ironic | 16:44 | |
| jroll | 7 hours if you're counting | 16:44 |
| NobodyCam | hehehehe :p | 16:44 |
| NobodyCam | lol | 16:44 |
| rybridges2 | lucasagomes: Do you know of any good resources for reading about the protocol that is being implemented in pyghmi? Would love to try to sink my teeth in if I get some spare time | 16:44 |
| rybridges2 | It's obvious to me that a very specific protocol is being implemented here.. Just not sure where to start reading about it/understand it. | 16:45 |
| *** srart has joined #openstack-ironic | 16:48 | |
| *** rob0t__ has quit IRC | 16:49 | |
| *** rob0t__ has joined #openstack-ironic | 16:52 | |
| lucasagomes | rybridges2, well the protocol is IPMI version 1.5 and 2.0 | 16:52 |
| rybridges2 | okay! That is helpful | 16:52 |
| lucasagomes | rybridges2, now the way it's implemented, I think only jarrod is familiar with | 16:53 |
| rybridges2 | okay | 16:54 |
| rybridges2 | I am looking around at some docs online for ipmi v2.0 | 16:54 |
| rybridges2 | found one by intel that is about 600 pages :D | 16:55 |
| lucasagomes | rybridges2, the specification you can find at the intel website | 16:55 |
| lucasagomes | rybridges2, http://www.intel.ie/content/www/ie/en/servers/ipmi/ipmi-second-gen-interface-spec-v2-rev1-1.html | 16:55 |
| lucasagomes | that said, it's not easy to read | 16:55 |
| rybridges2 | yea I was just looking at that heh | 16:55 |
| rybridges2 | pretty monolithic. i cannot even imagine how much time it took them to write all of this | 16:56 |
| *** rob0t__ has quit IRC | 16:56 | |
| lucasagomes | dtantsur, https://review.openstack.org/#/c/430366/ | 16:59 |
| patchbot | patch 430366 - python-ironicclient (stable/ocata) - Bump the LAST_KNOWN_API_VERSION to 29 in OSC | 16:59 |
| lucasagomes | rybridges2, heh indeed | 17:00 |
| *** hamzy has quit IRC | 17:03 | |
| *** derekh has quit IRC | 17:05 | |
| *** trown is now known as trown|lunch | 17:09 | |
| *** priteau has quit IRC | 17:10 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Serve boot configs from Ironic API https://review.openstack.org/369438 | 17:14 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Implement Driver API for dynamic boot config https://review.openstack.org/394398 | 17:14 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Add RPC API to get the boot config https://review.openstack.org/394399 | 17:14 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Allow not writing pxe configs to disk https://review.openstack.org/394397 | 17:14 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Convert iPXE boot config to proper Jinja template https://review.openstack.org/393462 | 17:14 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: DNM test enabled dynamic iPXE https://review.openstack.org/391943 | 17:14 |
| *** milan has quit IRC | 17:15 | |
| openstackgerrit | Bob Fournier proposed openstack/python-ironic-inspector-client master: Add new introspection commands for interface data including lldp https://review.openstack.org/419141 | 17:16 |
| *** jkilpatr_ has joined #openstack-ironic | 17:17 | |
| *** aNuposic has quit IRC | 17:18 | |
| *** jkilpatr has quit IRC | 17:19 | |
| *** jaosorior has quit IRC | 17:22 | |
| *** jaosorior has joined #openstack-ironic | 17:23 | |
| *** jaosorior has quit IRC | 17:24 | |
| *** jaosorior has joined #openstack-ironic | 17:24 | |
| *** Nisha_Agarwal has quit IRC | 17:25 | |
| *** dsneddon_afk is now known as dsneddon | 17:28 | |
| *** kromanenko has quit IRC | 17:32 | |
| *** amarao has quit IRC | 17:32 | |
| *** dprince has quit IRC | 17:34 | |
| rybridges2 | lucasagomes: I almost feel as though it would be easier to reverse engineer the pyghmi tool to understand the protocol rather than try to make sense of this document from intel. haha | 17:35 |
| *** dprince has joined #openstack-ironic | 17:35 | |
| mariojv | question about this: https://review.openstack.org/#/c/412397/32/ironic/db/sqlalchemy/alembic/versions/868cb606a74a_add_version_field_in_base_class.py | 17:36 |
| patchbot | patch 412397 - ironic - Add version column | 17:36 |
| lucasagomes | rybridges2, lol indeed, you can either look at the pyghmi implementation for python or ipmitool for C | 17:36 |
| mariojv | why do node_tags and conductor_hardware_interfaces need version fields in the DB? they don't have release mappings in the previous patch sets | 17:36 |
| mariojv | presumably because there aren't rpc objects for them | 17:36 |
| *** kvcobb has quit IRC | 17:42 | |
| *** aNuposic has joined #openstack-ironic | 17:45 | |
| *** patriciadomin has joined #openstack-ironic | 17:46 | |
| *** baoli has joined #openstack-ironic | 17:49 | |
| *** Syed__ has joined #openstack-ironic | 17:55 | |
| *** ccamacho has quit IRC | 17:56 | |
| *** srobert_ has joined #openstack-ironic | 17:57 | |
| *** mgoddard_ has quit IRC | 17:59 | |
| *** ralonsoh has quit IRC | 18:00 | |
| *** srobert has quit IRC | 18:01 | |
| *** tesseract has quit IRC | 18:01 | |
| *** lucasagomes is now known as lucas-afk | 18:02 | |
| *** alexpilotti has joined #openstack-ironic | 18:02 | |
| *** krtaylor has quit IRC | 18:03 | |
| *** vsaienko has quit IRC | 18:04 | |
| *** Goneri has quit IRC | 18:06 | |
| dtantsur | o/ | 18:09 |
| *** dtantsur is now known as dtantsur|afk | 18:09 | |
| *** trown|lunch is now known as trown | 18:15 | |
| *** dprince has quit IRC | 18:21 | |
| *** faizy has joined #openstack-ironic | 18:22 | |
| *** dprince has joined #openstack-ironic | 18:22 | |
| *** vsaienko has joined #openstack-ironic | 18:26 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-inspector master: Update documentation to deploy Ironic Inspector with DevStack https://review.openstack.org/410456 | 18:26 |
| mat128 | JayF: reconsidered https://review.openstack.org/#/c/428704/, it has a +1 from me now. My proposed changes would not add much clarity to the (not very big) script. | 18:31 |
| patchbot | patch 428704 - ironic-python-agent - imagebuild: tinyipa: Make the install-deps.sh SUSE... | 18:31 |
| mat128 | lucas-afk, rybridges2: I was looking at compiling ipmitool within a python module. ipmitool does not have a clear distinction between the "lib" and the command line client, which makes it harder to use. | 18:33 |
| mat128 | rybridges2: I had a similar goal of understanding the protocol and proposing changes to pyghmi, but god is that a heavyweight protocol :( | 18:34 |
| rybridges2 | mat128: you mean basically ripping ipmitool out of ironic and just having a python package instead? | 18:34 |
| rybridges2 | mat128: yea, i would be surprised if there are more than 10 people in this world outside of intel who have a deep understanding of that protocol | 18:34 |
| *** hamzy has joined #openstack-ironic | 18:35 | |
| rybridges2 | mat128: or by 'ipmitool' do you mean the actual C library, not ipmitool.py in ironic | 18:37 |
| sambetts | mat128, JayF: I've left a comment on that IPA patch, I'm not a fan of using a regex there | 18:37 |
| *** owalsh is now known as owalsh-afk | 18:38 | |
| * sambetts -> home | 18:38 | |
| *** sambetts is now known as sambetts|afk | 18:38 | |
| *** mgoddard_ has joined #openstack-ironic | 18:38 | |
| *** krtaylor has joined #openstack-ironic | 18:40 | |
| mariojv | rloo: TheJulia: jfyi, found a small off by one bug on https://review.openstack.org/#/c/408556 - just pinging since y'all had +2s | 18:41 |
| patchbot | patch 408556 - ironic - Add new dbsync command with first online data migr... | 18:41 |
| rloo | mariojv: thx, i probably won't look at those until after ptg | 18:46 |
| mariojv | np | 18:46 |
| *** moshele has joined #openstack-ironic | 18:49 | |
| *** vsaienko has quit IRC | 18:51 | |
| *** vsaienko has joined #openstack-ironic | 18:52 | |
| *** vsaienko has quit IRC | 18:54 | |
| *** alexpilo_ has joined #openstack-ironic | 18:55 | |
| *** alexpilotti has quit IRC | 18:55 | |
| *** vsaienko has joined #openstack-ironic | 19:04 | |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost master: Allow deploy-dynamic on remote host https://review.openstack.org/411304 | 19:08 |
| *** vsaienko has quit IRC | 19:14 | |
| *** sayantani01 has quit IRC | 19:15 | |
| *** faizy has quit IRC | 19:16 | |
| *** vsaienko has joined #openstack-ironic | 19:22 | |
| *** dprince has quit IRC | 19:27 | |
| *** jcoufal has joined #openstack-ironic | 19:28 | |
| *** mgoddard_ has quit IRC | 19:29 | |
| TheJulia | mariojv: thanks for heads up :) | 19:31 |
| *** ChubYann has joined #openstack-ironic | 19:32 | |
| *** vsaienko has quit IRC | 19:34 | |
| openstackgerrit | Merged openstack/ironic master: Use https instead of http for git.openstack.org https://review.openstack.org/430018 | 19:36 |
| *** vsaienko has joined #openstack-ironic | 19:36 | |
| *** owalsh-afk has quit IRC | 19:43 | |
| *** dprince has joined #openstack-ironic | 20:33 | |
| *** Goneri has joined #openstack-ironic | 20:34 | |
| vgadiraj | If I have a depency on another patch in gerrit and I want to "comment it out" (have the patch still in the commit message but not depended on) do I change "depends-on" to "depends-not-on"? I feel like I have seen this in the past | 20:35 |
| jroll | vgadiraj: that would work, change it to pretty much anything that doesn't match "Depends-On: ..." | 20:37 |
| vgadiraj | sounds good, was just wondering if there a standard practice :D | 20:37 |
| jroll | not that I know of :) | 20:38 |
| jroll | "eventually-depends-on" | 20:38 |
| jroll | I wouldn't merge a patch with anything like that, though | 20:38 |
| *** jaosorior has quit IRC | 20:40 | |
| openstackgerrit | Varun Gadiraju proposed openstack/ironic master: WIP/DNM: Testing grenade multi-tenant experimental job for Tempest failures https://review.openstack.org/403895 | 20:41 |
| openstackgerrit | Merged openstack/ironic master: exception from driver_factory.default_interface() https://review.openstack.org/428475 | 20:43 |
| vgadiraj | yep, this is just for testing purposes but I want to leave the original dependency in the commit message | 20:44 |
| vgadiraj | thanks jroll! | 20:44 |
| jroll | totes :) | 20:45 |
| openstackgerrit | Merged openstack/ironic-python-agent stable/newton: [TinyIPA] Avoid running rm with no parameters https://review.openstack.org/429845 | 20:45 |
| openstackgerrit | Merged openstack/ironic master: log if 'flat' interface and no cleaning network https://review.openstack.org/429944 | 20:46 |
| *** amotoki has quit IRC | 20:51 | |
| *** priteau has joined #openstack-ironic | 20:52 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: No node interface settings for classic drivers https://review.openstack.org/430435 | 20:52 |
| rloo | mariojv: ^^. (I hate unit tests.) | 20:52 |
| jroll | woo, thanks rloo | 20:53 |
| mariojv | thanks, reviewing | 20:53 |
| rloo | jroll, mariojv: i was not sure what to do (if anything) wrt backwards compat. i mean, there may already be nodes out there using classic driver, with one or more interfaces set. i think our code just ignores it anyway, but i don't know for sure. | 20:55 |
| jroll | rloo: yeah, I'd think it would, it should be fine | 20:55 |
| rloo | jroll: so no need to put any warning or anything; 'you should clean up .. cuz it is a bug'? | 20:56 |
| *** ijw has joined #openstack-ironic | 20:57 | |
| rloo | jroll: not sure who would be using dynamic driver stuff anyway. so soon. maybe the 'experimental' covers this :) | 20:57 |
| jroll | rloo: I don't know, let me look | 20:57 |
| jroll | rloo: exactly, it's probably zero impact, but might be worth a note | 20:57 |
| rloo | jroll: you want me to add a reno, or will you remember when you fix the reno for driver comp? | 20:58 |
| mariojv | rloo: had 1 comment on https://review.openstack.org/#/c/430435/ | 20:58 |
| patchbot | patch 430435 - ironic - No node interface settings for classic drivers | 20:58 |
| mariojv | didn't review tests yet | 20:58 |
| jroll | rloo: not sure, let me review the patch first please :) | 20:58 |
| rloo | jroll: ok. | 20:58 |
| rloo | mariojv: looking... | 20:58 |
| *** crushil has quit IRC | 20:58 | |
| mariojv | the function signature says node is an object at that point (not a dict) so wasn't sure if it'd work | 20:58 |
| jroll | mariojv: see the comment above that, oslo.versionedobjects is weird | 20:59 |
| * jroll doesn't know if hasattr() would work there | 20:59 | |
| rloo | mariojv: ^^. i just copied from L253. | 20:59 |
| mariojv | oh - i thought that'd be the reason for using the guard (if statement) instead of getattr(... , None) | 21:00 |
| mariojv | ah, i see where it's copied from | 21:00 |
| mariojv | i suppose it's fine then if those lines were passing tests | 21:00 |
| *** owalsh-afk has joined #openstack-ironic | 21:00 | |
| rloo | mariojv: well, assuming the tests are good ;) | 21:00 |
| mariojv | rloo: yup, it'll work https://github.com/openstack/oslo.versionedobjects/blob/1d1932400d60b0073a8254507a3ab9e87b84f4b1/oslo_versionedobjects/base.py#L321 | 21:02 |
| * mariojv does full review now | 21:02 | |
| rloo | thx mariojv! | 21:02 |
| mariojv | +1ed, but i also dunno what to do about backwards compat | 21:06 |
| mariojv | I guess it's fine based on the discussion y'all just had | 21:06 |
| *** ianw has quit IRC | 21:07 | |
| *** moshele has quit IRC | 21:07 | |
| rloo | mariojv: dunno. going to sleep on it and maybe forget about it ;) | 21:07 |
| mariojv | lol @ millions of nodes comment | 21:07 |
| rloo | mariojv: thx for the quick review. | 21:07 |
| *** rama_y has joined #openstack-ironic | 21:08 | |
| mariojv | https://www.youtube.com/watch?v=lKie-vgUGdI | 21:08 |
| *** ianw has joined #openstack-ironic | 21:08 | |
| jroll | rloo: reviewed, I think there's an easy-ish way to handle it | 21:08 |
| *** moshele has joined #openstack-ironic | 21:09 | |
| rloo | jroll: hmm... | 21:09 |
| *** harlowja has quit IRC | 21:09 | |
| jroll | (this reminds me I need to clean up renos tomorrow or so) | 21:10 |
| openstackgerrit | Jonathan Provost proposed openstack/ironic-python-agent master: Store image on disk when bigger than RAM size https://review.openstack.org/430442 | 21:10 |
| *** sdake has quit IRC | 21:11 | |
| *** crushil has joined #openstack-ironic | 21:14 | |
| *** moshele has quit IRC | 21:15 | |
| *** ijw has quit IRC | 21:16 | |
| openstackgerrit | Merged openstack/ironic master: Validate the network interface before cleaning https://review.openstack.org/429952 | 21:17 |
| *** ijw has joined #openstack-ironic | 21:20 | |
| rloo | we should revisit why we allow storage interfaces to be specified at the node level for classic drivers. make them change to using dynamic drivers :) | 21:26 |
| *** jkilpatr_ has quit IRC | 21:26 | |
| jroll | rloo: I'd be happy with that, I think it was just a timing mismatch thing | 21:27 |
| rloo | jroll: yup. we can change that after this week :) | 21:28 |
| *** sdake has joined #openstack-ironic | 21:31 | |
| *** harlowja has joined #openstack-ironic | 21:35 | |
| *** sdake_ has joined #openstack-ironic | 21:41 | |
| *** srobert_ has quit IRC | 21:41 | |
| *** jkilpatr has joined #openstack-ironic | 21:43 | |
| *** sdake has quit IRC | 21:44 | |
| *** sabeen has joined #openstack-ironic | 21:45 | |
| *** dsneddon is now known as dsneddon_afk | 21:49 | |
| *** krtaylor has quit IRC | 21:54 | |
| * rloo wants to cry. this driver instance has these interfaces: ['power', 'deploy', 'network', 'network', 'network', 'console', 'management', 'boot', 'inspect', 'raid', 'storage', 'storage', 'storage', 'vendor'] | 21:55 | |
| rloo | and that isn't even what i'm trying to fix... | 21:55 |
| jroll | O_o | 21:55 |
| mariojv | at least there's a common theme to the duplicated ones | 21:55 |
| mariojv | all are supported w/ classic | 21:55 |
| jroll | what is that from? | 21:55 |
| rloo | driver_factory. _attach_interfaces_to_driver(). driver_or_hw_type.all_interfaces. | 21:56 |
| rloo | i hate unit tests... that one is probably easy to fix. but not what i'm trying to figure out now. this driver comp stuff is not easy to understand :-( | 21:57 |
| jroll | hm, we should kill .all_interfaces, it's barely used | 21:57 |
| jroll | and kinda magical | 21:57 |
| jroll | (for my tastes) | 21:57 |
| rloo | jroll: off the top of your head. if i have a node with a classic driver, node.network_interface -- does it have to be set? | 21:57 |
| jroll | rloo: no, it isn't, the classic drivers alone were confusing enough :/ | 21:57 |
| jroll | rloo: I think it can be null but not 100%. it might get set to the default at creation time | 21:58 |
| rloo | jroll: i have a unit test failing cuz network_interface is none. not quite sure yet where that is coming from. i think i will mock that _attach_*() method for now and look into it later... | 21:59 |
| rloo | jroll: and we don't have unit tests for that _attach_* method... | 21:59 |
| *** tesseract has joined #openstack-ironic | 21:59 | |
| *** tesseract- has joined #openstack-ironic | 22:00 | |
| jroll | rloo: dunno, sorry | 22:01 |
| rloo | jroll: no worries, i'll figure it out | 22:02 |
| *** tesseract has quit IRC | 22:07 | |
| *** tesseract- has quit IRC | 22:07 | |
| *** tesseract has joined #openstack-ironic | 22:08 | |
| *** priteau has quit IRC | 22:12 | |
| *** dsneddon has joined #openstack-ironic | 22:13 | |
| *** jcoufal has quit IRC | 22:13 | |
| *** srobert has joined #openstack-ironic | 22:18 | |
| *** srobert has quit IRC | 22:18 | |
| *** jcoufal has joined #openstack-ironic | 22:18 | |
| *** srobert has joined #openstack-ironic | 22:18 | |
| *** jheroux has quit IRC | 22:22 | |
| *** sdake_ is now known as sdake | 22:23 | |
| *** baoli has quit IRC | 22:24 | |
| *** trown is now known as trown|outtypewww | 22:24 | |
| *** baoli has joined #openstack-ironic | 22:24 | |
| * TheJulia blinks at rloo's earlier comment | 22:25 | |
| *** tesseract is now known as tesseract-RH | 22:26 | |
| *** jcoufal has quit IRC | 22:28 | |
| *** baoli has quit IRC | 22:29 | |
| openstackgerrit | Merged openstack/ironic master: Fix log formating in ironic/common/neutron https://review.openstack.org/430111 | 22:31 |
| *** tesseract-RH is now known as tesseract | 22:31 | |
| openstackgerrit | Merged openstack/ironic master: Remove trailing slash from base_url in tempest plugin https://review.openstack.org/429679 | 22:32 |
| *** ijw has quit IRC | 22:32 | |
| *** vsaienko has quit IRC | 22:34 | |
| *** alexpilotti has joined #openstack-ironic | 22:38 | |
| *** alexpilo_ has quit IRC | 22:38 | |
| *** dprince has quit IRC | 22:43 | |
| *** Goneri has quit IRC | 22:44 | |
| openstackgerrit | Bob Fournier proposed openstack/ironic-inspector master: Add node_info to some messages and clean up docstrings for lldp plugin https://review.openstack.org/430464 | 22:45 |
| bigjools | hey folks, I've got a problem booting the coreos ipa image - it loads the kernel and ramdisk and then the machine reboots.. rinse and repeat ... :( | 22:49 |
| * bigjools wonders if he can bug jroll again | 22:49 | |
| openstackgerrit | Merged openstack/bifrost master: Enable DeprecationWarning in test environments https://review.openstack.org/430098 | 22:52 |
| bigjools | console output looks like this, then it reboots: http://pastebin.ubuntu.com/23950665/ | 22:52 |
| * TheJulia looks | 22:54 | |
| TheJulia | bigjools: so, crazy idea, have you tried ipxe? A few times I've seen the same thing if the packets get jumbled in blender that can occasionally be a network | 22:55 |
| bigjools | TheJulia: I am using the ipxe rom for kvm if that's what you mean? | 22:55 |
| TheJulia | Well, it is loading pxelinux.0... | 22:57 |
| TheJulia | and seems to indicate loading over tftp | 22:58 |
| bigjools | yep, you can see it loading the next stage too | 22:58 |
| bigjools | I may just try using a "regular" kernel and ramdisk to see if it does anything different | 22:59 |
| TheJulia | Worth a try, also if you haven't checked the checksums | 23:00 |
| TheJulia | I've also seen some odd behavior on some chassis with bios settings and coreos | 23:00 |
| *** tesseract has quit IRC | 23:00 | |
| *** tesseract has joined #openstack-ironic | 23:00 | |
| *** tesseract-RH has joined #openstack-ironic | 23:01 | |
| *** tesseract has quit IRC | 23:01 | |
| *** tesseract-RH has quit IRC | 23:01 | |
| *** sabeen has quit IRC | 23:05 | |
| bigjools | ok thanks | 23:05 |
| *** fragatina has joined #openstack-ironic | 23:07 | |
| TheJulia | bigjools: You may also just want to try like a fedora or debian IPA image | 23:07 |
| bigjools | ah ok I didn;t know you could get those | 23:08 |
| bigjools | are they the self-built ones from dib? | 23:08 |
| TheJulia | bigjools: Yes, using the ironic-agent element | 23:11 |
| bigjools | cool, will check that out. Thanks TheJulia. | 23:11 |
| TheJulia | No problem! | 23:11 |
| bigjools | You are all so helpful on here. | 23:11 |
| TheJulia | :) | 23:12 |
| *** ijw has joined #openstack-ironic | 23:14 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 23:15 | |
| *** ijw has quit IRC | 23:16 | |
| openstackgerrit | Merged openstack/bifrost master: Allow deploy-dynamic on remote host https://review.openstack.org/411304 | 23:16 |
| bigjools | Oh one more question - does agent_ipmitool have a hard requirement on Swift? | 23:19 |
| TheJulia | bigjools: Essentially, yes because it uses it for temporary storage for image downloads. If your doing deployments in a standalone setup, you don't need swift | 23:21 |
| TheJulia | Just, http(s) URLs | 23:21 |
| bigjools | right | 23:21 |
| bigjools | It's quite a large requirement | 23:22 |
| bigjools | the old pxe_ipmitool is deprecated right? | 23:22 |
| TheJulia | No, it is not. That will just change the deployment mechanism to be via iscsi being written from the conductor | 23:23 |
| *** sdake_ has joined #openstack-ironic | 23:24 | |
| *** sdake has quit IRC | 23:26 | |
| *** sdake_ has quit IRC | 23:27 | |
| *** sdake_ has joined #openstack-ironic | 23:31 | |
| bigjools | oh ok, so which driver is generally recommended? | 23:36 |
| bigjools | the agent one is using http right? | 23:37 |
| TheJulia | Yes, the actual lift is performed on the node being deployed as opposed to the conductor in that case | 23:37 |
| bigjools | cool | 23:38 |
| bigjools | so if I switch to pxe_ipmitool I don't need swift as it pulls images over iscsi | 23:39 |
| *** Nisha_Agarwal has quit IRC | 23:44 | |
| *** baoli has joined #openstack-ironic | 23:45 | |
| TheJulia | dtantsur|afk: Congradulations | 23:46 |
| *** cdearborn has quit IRC | 23:52 | |
| aNuposic | vsaienk0: Hi Vasyl, thanks for your review, wrt https://review.openstack.org/#/c/407639/9//COMMIT_MSG@12 how much info do you think I should put about options? | 23:56 |
| patchbot | patch 407639 - ironic - Config drive support for Ceph | 23:56 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!