Friday, 2017-03-10

« Thursday, 2017-03-09 Index Saturday, 2017-03-11 »
openstackgerritJohn L. Villalovos proposed openstack/ironic master: Don't 'enable_plugin ironic' in grenade settings  https://review.openstack.org/44404300:00
openstackgerritJohn L. Villalovos proposed openstack/ironic master: WIP/DNM: Testing multi-tenant multi-node grenade job  https://review.openstack.org/42214900:02
*** dsneddon has joined #openstack-ironic00:05
*** rpioso has quit IRC00:11
openstackgerritJohn L. Villalovos proposed openstack/ironic stable/newton: WIP/DNM NEWTON: Testing multi-tenant multi-node grenade job  https://review.openstack.org/44397800:12
*** Fdaisuke has quit IRC00:14
*** rloo has quit IRC00:18
*** gfhellma_ has quit IRC00:22
*** crushil has joined #openstack-ironic00:26
openstackgerritJoanna Taryma proposed openstack/ironic-specs master: Response header based retries  https://review.openstack.org/44404600:28
*** jaypipes has quit IRC00:29
*** crushil has quit IRC00:32
openstackgerritAnup Navare proposed openstack/ironic-python-agent master: Add glean package in requirements.txt  https://review.openstack.org/44334100:39
*** tuanluong has joined #openstack-ironic00:41
aNuposicjlvillal: enhanced the commit message ^00:42
*** hamzy has joined #openstack-ironic00:43
*** gcb has quit IRC00:44
*** zhurong has joined #openstack-ironic00:56
*** priteau has joined #openstack-ironic00:56
*** Lambert has joined #openstack-ironic00:58
*** tovin07 has joined #openstack-ironic00:58
*** priteau has quit IRC01:01
*** dsneddon has quit IRC01:01
*** yuanying_ has quit IRC01:05
*** SerenaFeng has joined #openstack-ironic01:06
*** crushil has joined #openstack-ironic01:09
*** phuongnh has joined #openstack-ironic01:18
*** sayantan_ has joined #openstack-ironic01:19
*** yuanying has joined #openstack-ironic01:19
*** snecklifter_ has quit IRC01:20
*** kvcobb has joined #openstack-ironic01:24
*** snecklifter_ has joined #openstack-ironic01:24
openstackgerritTao Li proposed openstack/ironic master: Address a shell sytax mistake  https://review.openstack.org/44354601:25
*** frontrunner has joined #openstack-ironic01:43
openstackgerritHao Li proposed openstack/ironic-lib master: Retry when execute partprobe cmd failed  https://review.openstack.org/44406101:52
*** crushil has quit IRC01:58
openstackgerritHao Li proposed openstack/ironic-lib master: Retry when execute partprobe cmd failed  https://review.openstack.org/44360402:16
*** yuanying has quit IRC02:20
*** yuanying has joined #openstack-ironic02:21
*** sayantan_ has quit IRC02:35
*** fragatin_ has joined #openstack-ironic02:38
*** dsneddon has joined #openstack-ironic02:39
*** fragatina has quit IRC02:42
*** fragatin_ has quit IRC02:42
*** yuanying has quit IRC02:48
*** priteau has joined #openstack-ironic02:57
*** priteau has quit IRC03:02
*** Goneri has quit IRC03:08
*** frontrunner has quit IRC03:09
*** goldyfruit has joined #openstack-ironic03:10
*** SerenaFeng has quit IRC03:30
*** kvcobb has quit IRC03:32
*** links has joined #openstack-ironic03:38
*** fragatina has joined #openstack-ironic03:38
*** wrouesnel has quit IRC03:40
*** wrouesnel has joined #openstack-ironic03:41
*** crushil has joined #openstack-ironic03:41
*** fragatina has quit IRC03:41
*** fragatina has joined #openstack-ironic03:41
*** fragatin_ has joined #openstack-ironic03:42
*** fragatina has quit IRC03:46
*** nmathew has joined #openstack-ironic03:57
*** rbudden has quit IRC04:06
openstackgerritRuby Loo proposed openstack/ironic master: Add RPC and object version pinning  https://review.openstack.org/40749104:09
*** yuanying has joined #openstack-ironic04:10
*** Sukhdev has joined #openstack-ironic04:14
*** dyasny has quit IRC04:17
*** vmud213 has joined #openstack-ironic04:18
*** gcb has joined #openstack-ironic04:19
*** jaganathan has joined #openstack-ironic04:20
*** vmud213 has quit IRC04:26
*** vmud213 has joined #openstack-ironic04:27
*** jaganathan has quit IRC04:28
*** udesale has joined #openstack-ironic04:29
*** aparnav has joined #openstack-ironic04:32
openstackgerritTovin Seven proposed openstack/ironic master: Enable OSProfiler support in Ironic  https://review.openstack.org/35814204:33
*** crushil has quit IRC04:38
*** fragatin_ has quit IRC04:42
*** goldyfruit has quit IRC04:42
*** fragatina has joined #openstack-ironic04:48
*** priteau has joined #openstack-ironic04:58
*** priteau has quit IRC05:03
*** Sukhdev has quit IRC05:05
*** sasha21 has quit IRC05:11
*** amotoki has joined #openstack-ironic05:13
*** vmud213 has quit IRC05:24
*** vmud213 has joined #openstack-ironic05:24
*** SerenaFeng has joined #openstack-ironic05:43
*** vikrant has joined #openstack-ironic05:47
openstackgerritTao Li proposed openstack/ironic master: Address a shell syntax mistake  https://review.openstack.org/44354605:50
*** yogi has joined #openstack-ironic05:54
*** mtanino has joined #openstack-ironic05:57
*** udesale__ has joined #openstack-ironic06:00
*** udesale has quit IRC06:02
*** Lambert has quit IRC06:03
*** jaosorior has joined #openstack-ironic06:09
*** yogi has quit IRC06:10
*** crushil has joined #openstack-ironic06:14
*** Lambert has joined #openstack-ironic06:16
*** udesale has joined #openstack-ironic06:19
*** udesale__ has quit IRC06:21
openstackgerritHao Li proposed openstack/ironic master: Add udevadm cmd into ironic-lib.filters  https://review.openstack.org/44410906:24
*** yogi has joined #openstack-ironic06:25
*** jaganathan has joined #openstack-ironic06:28
*** SerenaFeng has quit IRC06:30
tovin07hello guys06:31
tovin07I saw that py35 test failed too many time with this message06:32
tovin07ERROR: InvocationError: '/home/jenkins/workspace/gate-ironic-python35/.tox/py35/bin/ostestr'06:32
*** phuongnh has quit IRC06:32
tovin07Is there anything wrong with our py35 gate?06:32
*** phuongnh has joined #openstack-ironic06:33
tovin07Example: https://review.openstack.org/#/c/358142/06:33
patchbotpatch 358142 - ironic - Enable OSProfiler support in Ironic06:33
tovin07Running test on local machine is OK, but gate failed06:33
*** rcernin has joined #openstack-ironic06:35
*** dsneddon has quit IRC06:37
*** mjura has joined #openstack-ironic06:41
*** mjura has quit IRC06:43
*** mjura has joined #openstack-ironic06:43
*** yogi has quit IRC06:45
*** amotoki has quit IRC06:50
*** yogi has joined #openstack-ironic06:58
*** priteau has joined #openstack-ironic06:59
*** priteau has quit IRC07:03
*** SerenaFeng has joined #openstack-ironic07:06
*** moshele has joined #openstack-ironic07:13
*** dsneddon has joined #openstack-ironic07:19
*** amotoki has joined #openstack-ironic07:23
*** venki1 has quit IRC07:23
*** venki has joined #openstack-ironic07:24
*** fragatina has quit IRC07:36
*** xek has quit IRC07:37
*** xek has joined #openstack-ironic07:37
*** tesseract has joined #openstack-ironic07:42
*** pmannidi has quit IRC07:44
*** xek has quit IRC07:45
*** xek has joined #openstack-ironic07:45
*** ChubYann has quit IRC07:51
pas-hamorning ironic08:08
*** phuongnh has quit IRC08:12
*** nmathew- has joined #openstack-ironic08:13
*** nmathew has quit IRC08:16
*** moshele has quit IRC08:17
*** nmathew- has quit IRC08:18
*** nmathew- has joined #openstack-ironic08:19
*** vsaienk0 has joined #openstack-ironic08:22
vsaienk0morning Ironic!08:22
*** nmathew- has quit IRC08:24
*** aarefiev_afk is now known as aarefiev08:24
aarefievmorning all!08:25
*** g3ek has quit IRC08:27
openstackgerritVasyl Saienko proposed openstack/ironic master: Add Ironic standalone tests  https://review.openstack.org/42355608:30
openstackgerritJun Yao proposed openstack/ironic-inspector master: Adds node state to the API response data  https://review.openstack.org/43641508:32
*** jaosorior is now known as jaosorior_breakf08:33
*** nmathew has joined #openstack-ironic08:37
*** g3ek has joined #openstack-ironic08:37
*** racedo has joined #openstack-ironic08:38
vsaienk0dtantsur, lucasagomes: could you please add to your review queue https://review.openstack.org/#/c/430851 we are pretty close in landing that, and I would like to switch ironic-standalone tests to run API under apache https://review.openstack.org/#/c/444156/08:39
patchbotpatch 430851 - ironic - Add wsgi handling to ironic-api in devstack08:39
patchbotpatch 444156 - openstack-infra/project-config - Run ironic-api under Apache in ironic-standalone08:39
*** crushil has quit IRC08:48
*** jaosorior_breakf is now known as jaosorior08:48
*** amoralej|off is now known as amoralej08:51
openstackgerritzack chen proposed openstack/ironic-python-agent master: collected logs is 'bytes' type.  https://review.openstack.org/43890108:55
*** deray has joined #openstack-ironic08:56
*** venki has left #openstack-ironic08:58
*** yuanying has quit IRC08:58
*** venki has joined #openstack-ironic08:59
*** priteau has joined #openstack-ironic08:59
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-ironic09:00
*** milan has joined #openstack-ironic09:04
*** priteau has quit IRC09:04
deraygood morning pas-ha, vsaienk0 aarefiev and all Ironicers !09:06
pas-hamorning deray09:08
deraypas-ha, o/09:08
openstackgerritTovin Seven proposed openstack/ironic master: Enable OSProfiler support in Ironic  https://review.openstack.org/35814209:11
*** amarao has joined #openstack-ironic09:14
*** mgoddard has joined #openstack-ironic09:18
openstackgerritYongfeng Du proposed openstack/bifrost master: Serial-console renamed by diskimage-builder  https://review.openstack.org/44417309:21
*** ralonsoh has joined #openstack-ironic09:22
*** priteau has joined #openstack-ironic09:22
*** alineb has joined #openstack-ironic09:25
*** nmathew has quit IRC09:27
*** Lambert has quit IRC09:31
*** lucas-afk is now known as lucasagomes09:34
lucasagomesmorning all09:34
*** udesale has quit IRC09:41
*** yuanying has joined #openstack-ironic09:41
*** amarao has quit IRC09:43
deraylucasagomes, g'morrning :)09:44
*** Lambert has joined #openstack-ironic09:44
lucasagomesderay, yo morning09:45
*** amarao has joined #openstack-ironic09:45
*** derekh has joined #openstack-ironic09:47
*** Lambert has quit IRC09:48
*** Lambert has joined #openstack-ironic09:48
milanmorning Ironic! :)09:50
milanmorning deray, lucasagomes, pas-ha! :)09:50
pas-hamorning milan :)09:50
deraymilan, g'morning o/09:51
milanlucasagomes, pixie made it on time! :D09:52
lucasagomesmilan, oh yeah09:56
lucasagomesmilan, he will enjoy the icy world soon :D09:56
*** moshele has joined #openstack-ironic09:58
*** MattMan has quit IRC10:04
*** MattMan has joined #openstack-ironic10:04
*** ccamacho has joined #openstack-ironic10:05
*** SerenaFeng has quit IRC10:10
*** zhurong has quit IRC10:10
*** moshele has quit IRC10:15
openstackgerritMarkos Chandras proposed openstack/bifrost master: roles: bifrost-ironic-install: Install python-pymysql for ironic-dbsync  https://review.openstack.org/44419710:23
*** hw_wutianwei has quit IRC10:23
*** dtantsur|afk is now known as dtantsur10:23
dtantsurmorning ironic!10:23
*** mgould|afk is now known as mgould10:23
mgouldmorning ironic!10:23
deraydtantsur, mgould g'morning10:24
mgouldmorning lucasagomes deray pas-ha vsaienk0 vdrok milan dtantsur tovin0710:24
dtantsurvsaienk0, note that depends-on on the project-config patch will make it wait on https://review.openstack.org/#/c/434869/ merging, which is unlikely to happen. that patch need to change its review ID10:25
patchbotpatch 434869 - ironic (stable/ocata) - DNM WIP wsgi ironic-api in devstack10:25
vsaienk0o/ mgould10:25
tovin07hello mgould10:25
dtantsurmorning vsaienk0, mgould, lucasagomes, deray, milan, pas-ha, aarefiev, tovin0710:25
*** sambetts|afk is now known as sambetts10:25
sambettsMorning all10:26
dtantsur.... and sambetts :)10:26
sambettshaha hey dtantsur10:26
lucasagomeshey sambetts dtantsur mgould everyone o/10:27
*** SerenaFeng has joined #openstack-ironic10:27
vsaienk0dtantsur: yeah, that was cherry-pick to stable branch to make sure that grenade works, and it will not be definitely landed. Thanks for notice10:27
*** tovin07 has quit IRC10:27
mgouldmorning sambetts10:28
vsaienk0dtantsur: I think I will remove depends-on once related change is merged10:29
aarefievmorning dtantsur, sambetts, mgould, deray10:29
mgouldmornign aarefiev10:29
derayaarefiev, o/ :)10:32
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Add bindep.txt file  https://review.openstack.org/44420110:32
*** openstackgerrit has quit IRC10:33
sambettsvsaienk0: we made a mistake in Newton nova virt with regards to the binding_host_id... https://github.com/openstack/nova/blob/b34809dfae862e37b1362b73fd258af8d12adae5/nova/virt/ironic/driver.py#L1265 should have been 'node.network_interface != "flat"' because they way this is written is preventing me creating my own network interfaces10:34
*** openstackgerrit has joined #openstack-ironic10:34
openstackgerritMerged openstack/ironic-specs master: Add priorities for the Pike cycle  https://review.openstack.org/43971010:34
dtantsur\o/10:35
sambetts\o/10:35
vsaienk0sambetts: why it prevents you from writing custom network interface?10:35
*** SerenaFeng has quit IRC10:36
sambettsvsaienk0: because nova always tries to bind the port for my new network interface because its not call and can't be called "neutron"10:36
*** SerenaFeng has joined #openstack-ironic10:36
vsaienk0sambetts: but nova doesn't care about baremetal port status, and it will not be bound until we set local_link_information in biding profile10:37
sambettsvsaienk0: the port isn't baremetal vnic type though right? so it tries to bind it to what ever neutron agent is supposed to be running with the nova compute, but we're not running a neutron agent, so it blows up the deployment with binding-fialed10:38
sambettsvsaienk0: and because my new interface is not called "neutron" it populates the host_id and triggers binding...10:39
vsaienk0sambetts: but even if binding is failed nova shouldn't care about that for ironic virt driver10:41
sambettsvsaienk0: it does and it fails the deployment10:41
vsaienk0sambetts: I'm wondered why nova doesn't care that port is not bound than10:42
sambettsvsaienk0: in our CI the port does actually get bound on the neutron side, because we're running openvswitch-agent along side the ironic node compute10:43
sambettsvsaienk0: when you remove that it fails10:43
*** SerenaFeng has quit IRC10:47
milanmorning and TGIF! :D10:47
sambettsTGIF milan!10:47
milanhey aarefiev sambetts, vsaienk0! :D10:47
*** SerenaFeng has joined #openstack-ironic10:47
vdrokgood morning all :)10:48
milanmorning vdrok! :)10:48
aarefievmilan: hey, thanks for update spec, I'll also try to add some more details today10:48
milanaarefiev, no prob! :)10:49
vdrokmorning mgould dtantsur aarefiev vsaienk0 sambetts deray lucasagomes pas-ha !10:49
pas-hamorning vdrok :)10:49
vsaienk0sambetts: I'm wondered if nova doesn't care about port status for ironic, why it cares if tried to bind port but failed?10:49
vdrokand good morning milan :)10:49
lucasagomeso/10:50
lucasagomesdtantsur, o/ priorities10:50
sambettsvsaienk0: in stable/newton IF network_interface != "neutron" then nova DOES care about the port binding status10:50
sambettsvsaienk0: https://github.com/openstack/nova/blob/stable/newton/nova/network/neutronv2/api.py#L43710:51
sambettsvsaienk0: that passes when we are used the neutron interface, or using the flat interface WITH a openvswitch-agent being successly bound for no reason10:53
vsaienk0sambetts: tenant port can't be bound (it remain DOWN) when nova just created it until we set binding_porfile later in ironic, so I'm wondered why nova doesn't care that port is DOWN when there were no attempt to bind it (host_id is empty) and if we set host_id we expect it to be bound than.10:53
milanaarefiev,  btw, I'm still unsure about the pxe filtering, whether to have an update method or a blacklist_node/whitelist_node; the spec atm has the update way but feels strange because of exposing the update period10:53
*** deray has quit IRC10:53
milanaarefiev, and the update call is also a bit odd interface10:54
milanaarefiev, on the other hand, if we have a blacklist_something/whitelist_something, what should be the parameter? a node_info? the boot nic mac address?10:54
sambettsvsaienk0: in *stable/newton* nova will try to bind the port, because it'll set the host_id, IF my network_interface != "neutron"10:54
aarefievmilan: give me some time to read the spec, I'll post my thoughts there10:55
sambettsvsaienk0: and my network_interface == "sams_network_interface"10:55
milanaarefiev, sure, thanks! I'm just brain dumping here ;)10:55
*** vills has joined #openstack-ironic10:55
*** SerenaFeng has quit IRC10:56
milanaarefiev, i'm atm actually more fond of the black_/white_list node_info tbh than the update10:56
aarefievmilan: I'm not sure about update period in this new filter interface10:56
milanaarefiev, yeah, moreover it feels like an internal matter of a filter implementation10:57
aarefievmilan: imo it's more related to ironic/inspector sync10:57
sambettsvsaienk0: thats where the mistake is, the if statement that check if we should set the host_id or not is the wrong way around, it checks if the network interface IS neutron, instead of checking if the network interface IS NOT flat10:57
vsaienk0sambetts: but we can have custom flat network interfaces10:58
aarefievmilan: how it would differ in drivers10:58
milanaarefiev, yeah, if a filter needs its own sync it should take care about it (like Noop doesn't have to use it)10:58
milanaarefiev, or a driver counting on a discovery network won't need it either10:58
sambettsvsaienk0: and they can bind the port in Ironic if they want to, but there is no way for me to stop nova binding now unless I use the  neutron network interface10:59
milanor,..... hmmmm10:59
vsaienk0sambetts: I was tried to get rid of this some time ago. There are patches to nova that allows to set vnic_type to baremetal when virt driver is ironic. But I'm stuck on flat case which requires an ML2 driver that will handle it10:59
milanaarefiev, actually, some blacklist items might need removing if a node is removed from ironic,10:59
milanaarefiev, so I'm not sure :-/10:59
*** Nisha_Agarwal has joined #openstack-ironic11:00
sambettsvsaienk0: working on it ;) https://github.com/Tehsmash/networking-static-baremetal/blob/master/networking_static_baremetal/mech.py11:00
sambettsvsaienk0: this is temp location, but this is why I wanted to use my own network interface11:00
sambettsin ironic11:01
sambettsvsaienk0: so I could set the vnic_type to baremetal and stuff11:01
vsaienk0sambetts: I proposed a patches to Neutron and they rejected them, anyway we have networking-baremetal repo right now, so we will fix that in Pike11:01
sambettsvsaienk0: I need it for newton11:01
sambettsvsaienk0: and if we hadn't derped on the if logic I would be able to do it in just drivers11:01
vsaienk0sambetts: I think we should always return None there, and update host_id on ironic side if needed.11:02
aarefievmilan:  I see it on beyond abstraction level, like some inspector manager call ironic and check if filtering should be updated, and call filter update some nodes if needed11:02
vsaienk0s/if needed/when needed/11:02
sambettsvsaienk0: thats what we do in Ocata, which is fine, but I need it in newton11:02
sambetts:(11:03
milanaarefiev, so basically inspector would do the "periodic" checks instead of the filters?11:04
milan* on the filter behalf11:04
milancould work11:04
vsaienk0sambetts: yeah, why not just pick that small change?11:04
milanaarefiev, the filter_driver.update_filter() would just go through the node cache and pick up things from there?11:05
vsaienk0sambetts: have you also checked? https://github.com/openstack/neutron/blob/96a51ad7427f9b6f2bd32ecd7a0fcae2e6257017/neutron/conf/common.py#L10511:06
milanaarefiev, dtantsur, sambetts btw & fyi I'm PTO next week, no patch updates :P11:06
dtantsurmilan, oh, Iceland time finally?11:07
aarefievmilan: looks like we could pass specific list of nodes only11:07
aarefievI guess11:07
sambettsvsaienk0: yeah I'm looking at that now, we can't pick the whole change because that would also require picking the changes on the Ironic side to make the flat network interface bind the port, but changing the if should be fine11:07
milandtantsur, exactly :D11:07
sambettsvsaienk0: yeah, the problem is this logic https://github.com/openstack/nova/blob/stable/newton/nova/network/neutronv2/api.py#L17211:07
dtantsuraarefiev, I guess one of us can move it forward11:07
aarefievyeah  don't worry milan11:07
aarefievand have good PTO11:08
milanaarefiev, thanks! :)11:08
*** Nisha_Agarwal has quit IRC11:08
milanaarefiev, but the list would have double-meaning, nope? like how to tell which to blacklist and which to whitelist?11:08
dtantsurmilan, try not to fall into a volcano, and don't forget Ina on a glacier :D11:08
*** Nisha_Agarwal has joined #openstack-ironic11:09
milandtantsur, yeah :DDD11:09
milanlol11:09
sambettsvsaienk0: do you have any idea how Neutron/Nova pick the binding:vnic_type for a port??11:09
* milan takes Pixie to the volcanoes too, so this poor guy gonna have a lot of type-2 fun ;)11:09
milandtantsur, ^11:09
dtantsurtrue11:09
aarefievmilan: yeah, good point, I guess this part would depends on driver impl11:10
milanvdrok, pixie might arrive w/ a burn here and there ;)11:10
openstackgerritGalyna Zholtkevych proposed openstack/ironic-specs master: API Evolution - ETAG identifiers  https://review.openstack.org/38199111:10
milanaarefiev, yeah, so I thought that the driver would anyway need to walk thru the node cache11:11
vsaienk0sambetts: I think it is always normal at the moment11:11
milanaarefiev, some filters might require more info from the cache too11:11
milanlike the infiniband thing11:11
vsaienk0sambetts: https://review.openstack.org/#/c/339143/11:12
patchbotpatch 339143 - nova - Set vnic_type correctly for ironic (ABANDONED)11:12
vsaienk0sambetts: https://review.openstack.org/#/c/339143/14/nova/network/neutronv2/api.py@100911:13
patchbotpatch 339143 - nova - Set vnic_type correctly for ironic (ABANDONED)11:13
sambettsvsaienk0: thanks :)11:13
vdrokmilan: hah, it's Iceland right? Just don't put it inside the volcano to take a photo, that's easy :D11:14
milanvdrok, lol :D11:15
vdrokI doubt the usual fire can burn it!11:15
milan:DDD yeah Pixie is sort of a Chuck Norris :D11:15
*** Lambert has quit IRC11:16
aarefievmilan: from other hand, hmm periodic update, it should only block all new nodes if needed, no?11:19
vsaienk0sambetts: I'm wondered if the current nova logic is right, ie it doesn't care about port status for ironic virt dirver, but it cares about binding failure. Because bound port is not always active port.11:19
*** Nisha_Agarwal has quit IRC11:20
*** Nisha_Agarwal has joined #openstack-ironic11:20
milanaarefiev, and unblock nodes we know are inspected11:27
aarefievmilan: should be this periodic update11:27
aarefievafter introspection is finished we could take care about particular node11:28
milanaarefiev, you mean the update method would deal w/ both the cases as it's now in inspector?11:30
milanaarefiev, or you want to have a separate method to unblock?11:31
milan* and block11:31
milan?11:31
aarefievI'm not saying it should be one method, but always relying on periodic update doesn't sound good11:32
aarefievwe could tracking node filter during it's life11:32
milanaarefiev, atm the update is called both periodically and once there's a known change such as start/stop introspection11:32
aarefievand periodically update only new nodes11:33
aarefievyeah11:33
milanaarefiev, so blacklist_node, whitelist_node, update_filter?11:33
milan* methods?11:33
*** yuriyz has joined #openstack-ironic11:37
*** vills has quit IRC11:37
*** vills has joined #openstack-ironic11:37
*** links has quit IRC11:37
aarefievmilan: seems over complicated11:38
openstackgerritLucas Alvares Gomes proposed openstack/ironic master: Add redfish driver  https://review.openstack.org/43898211:38
openstackgerritLucas Alvares Gomes proposed openstack/ironic master: Add documentation for the redfish driver  https://review.openstack.org/44253711:38
milanaarefiev, yeah, simplest is what we've got now, just the update I guess11:39
*** Nisha_Agarwal has quit IRC11:39
milanaarefiev, from another pow its about synchronising node_cache w/ a filter11:40
*** vills_ has joined #openstack-ironic11:43
*** vills has quit IRC11:43
aarefievmilan: node_cache doesn't know about not inspected node, right ?11:44
milanaarefiev, we keep the state, so it does11:45
milanwhat is waiting is being inspected11:45
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362811:45
aarefievmilan: if you don't call inspect, there no this node in cache, and some drivers need to block this node11:46
aarefievlike in iptable case11:46
milanaarefiev, right, so it's bot the cache and ironic node list that have to be in sync in the filter11:47
*** venki has quit IRC11:47
*** venki has joined #openstack-ironic11:48
*** vills has joined #openstack-ironic11:49
*** vills_ has quit IRC11:49
* milan thinks it would have been easier if inspector kept the node_cache items as long as those exist in ironic11:53
milanaarefiev, ^11:54
*** links has joined #openstack-ironic11:54
milanmaybe it's an anti feature to expire those records if those are still in Ironic ;)11:54
aarefievmilan: yeah, we will duplicate ironic db, and how often inspecting runs on same node11:56
aarefievI think about 2 methods now11:56
dtantsurmilan, hmm, I think it was introduced before we had a thread to sync nodes with ironic11:57
openstackgerritMarkos Chandras proposed openstack/ironic master: requirements.txt: Add PyMySQL dependency for ironic-dbsync  https://review.openstack.org/44424911:57
dtantsurmilan, I agree we can consider dropping it now11:57
milandtantsur, I just wanted to ping you about it :D11:58
*** vills has quit IRC11:58
dtantsurfeel free to propose a patc11:58
dtantsurh11:58
milanyeah, maybe a bug too11:59
milanaarefiev, sambetts you guys agree on dropping the inspector cache expiration in case the nodes are still in ironic?12:00
aarefievmilan: I wondering about this node default state12:01
milanaarefiev, currently it can be starting or enrolling12:03
milanor node absent12:03
aarefievstarting or enrolling would be confusing12:04
aarefievI guess new state12:05
milanaarefiev, how would you call it?12:05
openstackgerritDao Cong Tien proposed openstack/python-ironicclient master: Extends driver-list, driver-show supporting new hardware types  https://review.openstack.org/41927412:05
*** SerenaFeng has joined #openstack-ironic12:07
openstackgerritMarkos Chandras proposed openstack/ironic master: requirements.txt: Add PyMySQL dependency for ironic-dbsync  https://review.openstack.org/44424912:07
aarefievmilan: give the name is the most hard part, I think now that introducing new state helps with initial starting transition12:07
* milan lunch, sry, back in ~1h12:07
milanaarefiev, thought the same...12:08
aarefievthere are bugs like timeout on starting and so on12:08
milanyup12:08
milanaarefiev, talk to you in a bit12:08
aarefievsure, have a nice meal milan12:09
*** hshiina has joined #openstack-ironic12:09
vsaienk0dtantsur: have a question about your comment https://review.openstack.org/#/c/423556/. Do you want to use api_microversion_fixture for the whole tests instead of changing api version per specific request?12:10
patchbotpatch 423556 - ironic - Add Ironic standalone tests12:10
vsaienk0something like https://github.com/openstack/ironic/blob/f3380855f7595b3176b1136e91cfafa165882585/ironic_tempest_plugin/tests/api/admin/test_nodestates.py#L9612:10
aarefievFYI: oslo.messaging bug about spam in conductor logs https://bugs.launchpad.net/oslo.messaging/+bug/167178112:11
openstackLaunchpad bug 1671781 in oslo.messaging "rabbit polling spams into logs" [Undecided,Confirmed]12:11
dtantsurvsaienk0, rather for each test. well, if they all test the same, then for all tests.12:12
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362812:12
*** phuongnh has joined #openstack-ironic12:13
vsaienk0dtantsur: we setup resources in classmethod it allows to have single resource per different tests, but useFixture() is instance method I doubt that we can use it12:15
*** snecklifter_ has quit IRC12:15
*** jaganathan has quit IRC12:16
dtantsurhmm, I think useFixture is usually used on self, but I may be wrong12:17
*** e0ne has joined #openstack-ironic12:17
dtantsurmy main point is that picking API version per call is not how people expect it to be used12:17
*** lucasagomes is now known as lucas-hungry12:21
*** jkilpatr has joined #openstack-ironic12:21
*** aparnav has quit IRC12:25
*** venki has quit IRC12:27
*** snecklifter_ has joined #openstack-ironic12:28
*** venki has joined #openstack-ironic12:28
openstackgerritMerged openstack/ironic master: Add wsgi handling to ironic-api in devstack  https://review.openstack.org/43085112:33
*** vsaienko has joined #openstack-ironic12:33
*** racedo has quit IRC12:40
openstackgerritMerged openstack/ironic master: Fix updating node.driver to classic  https://review.openstack.org/43902612:42
openstackgerritMerged openstack/ironic master: Update the Ironic Upgrade guide  https://review.openstack.org/43919612:42
*** fragatina has joined #openstack-ironic12:46
*** yolanda has quit IRC12:46
vsaienk0dtantsur: I'm confused how microversion tests are working at the moment they using global variable https://github.com/openstack/ironic/blob/5603a215b726ca465eed2867d830821bb24b1a72/ironic_tempest_plugin/tests/api/admin/api_microversion_fixture.py#L25 there might be races between concurrent tests12:47
*** e0ne has quit IRC12:48
dtantsuroh12:50
dtantsurjroll, morning. I need your opinion on https://review.openstack.org/#/c/439404/12:55
patchbotpatch 439404 - ironic - Add hardware types for HPE ProLiant servers based ...12:55
*** frontrunner has joined #openstack-ironic12:55
vsaienk0dtantsur: ah, so concurrent tests are launched in separate process so even global variables will not be visible among them12:59
milanaarefiev, thx, /me back :)13:00
openstackgerritOksana Voshchana proposed openstack/ironic master: WIP: SSL TEST  https://review.openstack.org/44240213:01
milanaarefiev, so you want to have a state, say "default"? And we'd set the node into it before we do the starting transition?13:02
aarefievmilan: not sure yet, this is separate feature actually13:03
dtantsurvsaienk0, TIL13:03
milanaarefiev, OK13:03
aarefievmilan: I'm going to post some comments to spec13:04
aarefievabout filtering13:04
milanaarefiev, ack, thanks!13:04
*** yolanda has joined #openstack-ironic13:06
*** bfournie has quit IRC13:07
*** vmud213 has quit IRC13:07
*** gcb has quit IRC13:08
*** SerenaFeng has quit IRC13:10
*** lucas-hungry is now known as lucasagomes13:10
*** jjohnson2 has joined #openstack-ironic13:12
openstackgerritMerged openstack/ironic-inspector-specs master: Add upgrades and backwards compatibility section  https://review.openstack.org/43955613:14
*** des_consolado has quit IRC13:14
*** links has quit IRC13:17
nicodemosmorning, all! \o/13:22
openstackgerritSofiia Andriichenko proposed openstack/python-ironicclient master: Negative tests for testing actions with port.  https://review.openstack.org/25935413:22
*** moshele has joined #openstack-ironic13:24
milanguys, https://review.openstack.org/#/c/436415/9..13/ironic_inspector/test/unit/test_main.py@193 I'm wondering whether an unittest should be api microversion dependant, i.e whether it should check more than the latest version13:24
patchbotpatch 436415 - ironic-inspector - Adds node state to the API response data13:24
milanWDYT?13:24
* milan -0.6 on it ;)13:25
milanmorning nicodemos! :)13:25
*** phuongnh has quit IRC13:25
dtantsurmilan, -2 here13:26
milandtantsur, alright :)13:26
dtantsurunit tests represent the current state of the code13:26
milanyup, my take on it too13:26
vsaienk0sambetts: dtantsur please add to your review queue https://review.openstack.org/#/c/444233/ probably and all others :)13:29
patchbotpatch 444233 - networking-baremetal - Initial commit from cookiecutter13:29
*** yolanda has quit IRC13:29
*** yolanda has joined #openstack-ironic13:30
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362813:31
*** links has joined #openstack-ironic13:31
openstackgerritVasyl Saienko proposed openstack/ironic master: Add Ironic standalone tests  https://review.openstack.org/42355613:32
*** moshele has quit IRC13:32
*** vills has joined #openstack-ironic13:37
openstackgerritGalyna Zholtkevych proposed openstack/ironic-specs master: API Evolution - ETAG identifiers  https://review.openstack.org/38199113:37
dtantsurvsaienk0, ack, adding this project to gertty13:37
*** e0ne has joined #openstack-ironic13:38
*** links has quit IRC13:39
openstackgerritOksana Voshchana proposed openstack/ironic master: WIP: SSL TEST  https://review.openstack.org/44240213:41
*** bfournie has joined #openstack-ironic13:43
*** rbudden has joined #openstack-ironic13:49
*** e0ne has quit IRC13:53
*** e0ne has joined #openstack-ironic13:53
openstackgerritMerged openstack/ironic-specs master: Follow-up patch for the Redfish spec  https://review.openstack.org/44317713:55
openstackgerritMerged openstack/ironic-specs master: Allow spec to have RFE from any ironic project  https://review.openstack.org/44061013:56
*** cdearborn has joined #openstack-ironic13:59
*** tuanluong has quit IRC14:01
*** vsaienko has quit IRC14:02
lucasagomesstrange, http://logs.openstack.org/37/442537/2/check/gate-ironic-docs-ubuntu-xenial/f17b456/console.html#_2017-03-10_11_45_37_894815 I wonder how it worked before14:08
lucasagomesand it works for me locally for some reason14:08
*** sasha21 has joined #openstack-ironic14:10
*** goldyfruit has joined #openstack-ironic14:11
*** racedo has joined #openstack-ironic14:11
*** baoli has joined #openstack-ironic14:13
*** baoli has quit IRC14:16
openstackgerritVladyslav Drok proposed openstack/ironic master: Do not load credentials on import in tempest plugin clients.py  https://review.openstack.org/44431514:18
*** hshiina has quit IRC14:18
openstackgerritVasyl Saienko proposed openstack/ironic master: Add ironic standlaone test with ipmi dynamic driver  https://review.openstack.org/43912614:20
*** amoralej is now known as amoralej|lunch14:21
*** baoli has joined #openstack-ironic14:24
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362814:26
*** e0ne has quit IRC14:27
*** e0ne has joined #openstack-ironic14:27
*** e0ne has quit IRC14:28
*** e0ne has joined #openstack-ironic14:29
*** galyna has joined #openstack-ironic14:33
openstackgerritShivanand Tendulker proposed openstack/ironic master: Add hardware type for HPE ProLiant servers based on iLO 4  https://review.openstack.org/43940414:36
openstackgerritGalyna Zholtkevych proposed openstack/ironic-python-agent master: Correct failure message output when downloading  https://review.openstack.org/44107214:36
*** jaypipes has joined #openstack-ironic14:38
*** jaypipes is now known as leakypipes14:39
openstackgerritGalyna Zholtkevych proposed openstack/ironic-python-agent master: Correct failure message output when downloading  https://review.openstack.org/44107214:43
*** crushil has joined #openstack-ironic14:44
*** vills_ has joined #openstack-ironic14:44
openstackgerritJohn L. Villalovos proposed openstack/ironic master: DNM/WIP: Debugging grenade upgrade  https://review.openstack.org/44433514:47
*** yogi has quit IRC14:47
openstackgerritOksana Voshchana proposed openstack/ironic master: Adds use wsgi without port  https://review.openstack.org/44433714:49
*** leakypipes has quit IRC14:49
*** vikrant has quit IRC14:50
*** jaypipes has joined #openstack-ironic14:54
*** jaypipes is now known as leakypipes14:54
*** mjura has quit IRC15:01
dtantsurlucasagomes, using mocked version vs using the real module?15:01
*** mjturek has quit IRC15:01
lucasagomesdtantsur, so, that was what I thought but I build the docs locally and it works even when sushy is not installed15:02
lucasagomesthat said, I need to add a spec_set to the 3rd_party_mocks there15:02
lucasagomesI wasn't doing that15:02
* lucasagomes is investigating atm15:02
*** mtanino has quit IRC15:12
*** chlong_ has joined #openstack-ironic15:15
mariojvgood morning ironic15:22
mariojvaNuposic: i dunno, i doubt it15:22
mariojvaNuposic: that'd also require that you have information that you don't have at boot time15:22
*** vills has quit IRC15:23
*** jaosorior has quit IRC15:24
vsaienk0dtantsur: what do you think about https://review.openstack.org/#/c/419274/11/ironicclient/v1/driver.py@4115:28
patchbotpatch 419274 - python-ironicclient - Extends driver-list, driver-show supporting new ha...15:28
*** Syed__ has joined #openstack-ironic15:32
lucasagomesmariojv, morning15:34
openstackgerritMarkos Chandras proposed openstack/bifrost master: requirements.txt: Add PyMySQL dependency for ironic-dbsync  https://review.openstack.org/44419715:34
mariojvmorning lucasagomes15:35
*** sbezverk has quit IRC15:35
*** jcoufal has joined #openstack-ironic15:35
openstackgerritVasyl Saienko proposed openstack/ironic master: Add Ironic standalone tests  https://review.openstack.org/42355615:36
*** mjturek has joined #openstack-ironic15:37
*** venki has quit IRC15:39
NobodyCamGood Morning Ironic'ers15:39
*** amoralej|lunch is now known as amoralej15:39
dtantsurmorning mariojv, NobodyCam15:40
dtantsurvsaienk0, responded there15:40
mariojvhi dtantsur15:40
NobodyCammorning dtantsur :)15:40
*** venki has joined #openstack-ironic15:40
milanmorning mariojv, NobodyCam! :) TGIF!15:41
mariojvtgif!15:41
mariojvmorning milan15:41
milan:)15:41
NobodyCamor yea TGIF, Morning milan :)15:41
*** e0ne has quit IRC15:41
* milan just found the longest traceback in quite some time: http://logs.openstack.org/78/443878/1/check/check-osc-plugins/ba7f194/console.html#_2017-03-09_18_33_24_50029015:41
NobodyCamor = oh15:41
milanseems like the pbr dependency bump made someone recursively unhappy :)15:42
NobodyCamwow :)15:43
*** vills_ has quit IRC15:43
milandtantsur, ^^15:43
dtantsurwow15:44
milanhttps://review.openstack.org/#/c/443878/115:45
patchbotpatch 443878 - python-ironic-inspector-client (stable/ocata) - Updated from global requirements15:45
milanit's this patch ^15:45
openstackgerritOksana Voshchana proposed openstack/ironic master: Adds use wsgi without port  https://review.openstack.org/44433715:48
milanhttp://paste.openstack.org/show/602281/ the recursion15:48
*** Nisha_Agarwal has joined #openstack-ironic15:48
*** jcoufal has quit IRC15:52
*** jcoufal has joined #openstack-ironic15:54
*** amarao has quit IRC15:55
openstackgerritMerged openstack/ironic master: Correct typos in doc files  https://review.openstack.org/43777615:56
JayFmight be worth a review from other ironic folks if you have time: https://review.openstack.org/#/c/441544/15:57
patchbotpatch 441544 - nova - virt: implement get_inventory() for Ironic15:58
openstackgerritOksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi  https://review.openstack.org/43729915:58
*** rcernin has quit IRC15:59
*** dyasny has joined #openstack-ironic16:01
mgouldmorning mariojv NobodyCam, TGIF :-)16:03
NobodyCamhappy Friday mgould :)16:03
*** Goneri has joined #openstack-ironic16:04
vdrokmorning mariojv NobodyCam and JayF16:05
NobodyCamMorning vdrok :)16:05
*** yolanda has quit IRC16:06
JayFo/16:06
vdrokJayF: yeah, I'm currently trying to deploy that locally16:06
JayFvdrok: I put a -1 on it, I'm 80% sure it won't work16:06
*** yolanda has joined #openstack-ironic16:06
*** e0ne has joined #openstack-ironic16:07
openstackgerritLucas Alvares Gomes proposed openstack/ironic master: Add redfish driver  https://review.openstack.org/43898216:07
openstackgerritLucas Alvares Gomes proposed openstack/ironic master: Add documentation for the redfish driver  https://review.openstack.org/44253716:07
*** e0ne has quit IRC16:08
NobodyCamlucasagomes: \o/ very cool!16:08
lucasagomesNobodyCam, :D16:08
lucasagomesNobodyCam, yeah, the spec was approved this week so :-)16:08
NobodyCamawesome sauce !16:09
aslezil_ milan: Could you add this patch https://review.openstack.org/#/c/396406/ to your review list/queue.16:11
patchbotpatch 396406 - ironic-python-agent - Collect NIC name given by BIOS16:11
*** dyasny has quit IRC16:11
*** vills has joined #openstack-ironic16:11
*** baha has joined #openstack-ironic16:11
*** dyasny has joined #openstack-ironic16:11
lucasagomesmariojv, btw, if you have some time mind re-looking this https://review.openstack.org/#/c/443159/ ?16:11
patchbotpatch 443159 - ironic-lib - Improve logs for the root device hints mechanism16:11
*** yolanda has quit IRC16:11
openstackgerritKyrylo Romanenko proposed openstack/ironic master: Add functional API tests for portgroups  https://review.openstack.org/43687816:12
*** e0ne has joined #openstack-ironic16:12
openstackgerritOksana Voshchana proposed openstack/ironic master: WIP: SSL TEST  https://review.openstack.org/44240216:14
mjturekmariojv: cool if I handle the comments on the rescue/unrescue CLI code?16:14
mgouldwould time spent writing to iscsi show up in the "user" part of time(1)/16:15
mgould*?16:15
mariojvlucasagomes: yeah will take a look later today16:15
mariojvmjturek: feel free, thanks16:15
mjtureknp!16:15
lucasagomesmariojv, cool thanks, cause I think that part of the code is out of the scope for what the patch is doing16:15
lucasagomesanyway, check it out when u have some time16:15
lucasagomesthanks16:15
jlvillalThe perils of working from home: http://www.bbc.com/news/world-39232538  :)16:15
mgouldheh16:16
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Add bindep.txt file  https://review.openstack.org/44420116:16
*** sayantan_ has joined #openstack-ironic16:17
*** yolanda has joined #openstack-ironic16:18
*** chlong_ has quit IRC16:19
milanaslezil_, hey, qq: https://review.openstack.org/#/c/406790/ relation to https://review.openstack.org/#/c/396406/ : shouldn't that bios info be present under the bios namespace that spec introduces?16:19
patchbotpatch 406790 - ironic-inspector-specs - Collect Bios, Memory and Processor Info from Dmide...16:19
patchbotpatch 396406 - ironic-python-agent - Collect NIC name given by BIOS16:19
jlvillalpas-ha, I was asking a question about that the other day (bindep). If we should use that instead of what is in devstack/files/{debs,rpms}/ironic.  Do you know?16:20
jlvillalpas-ha, Ah, and reading the commit message answers that :)16:21
pas-hayep :) and spectacular failure of all python-* jobs in the previous change-set :)16:21
pas-hawith this might shave some time off from them, as the fallback bindep used by infra is kind of huge :)16:22
jlvillalpas-ha :)  Can we remove the files/{debs,rpms}/ironic files?16:22
pas-hanope. DevStack does not use bindep currently, and still relies on those16:22
jlvillalpas-ha, So the gate doesn't process bindep first? Before starting devstack?16:22
pas-habut - we definitely need to comb thru them, see the 'undefined' profile16:23
pas-hadoes not seem to16:23
jlvillalSad :(16:23
jlvillalpas-ha, Thanks for doing that! :)16:23
pas-hastill not getting all test dependencies... why is infra not installing the mysql by itself when the gates do strictly need it?...16:24
*** sayantan_ has quit IRC16:24
*** e0ne has quit IRC16:25
*** [1]cdearborn has joined #openstack-ironic16:25
openstackgerritAnton Arefiev proposed openstack/ironic master: Inspection: fail fast if inspector returns error  https://review.openstack.org/44439616:26
*** yolanda has quit IRC16:26
*** yolanda has joined #openstack-ironic16:27
*** e0ne has joined #openstack-ironic16:27
aslezil_milan :  https://review.openstack.org/#/c/406790/ regarding this , This patch collects the data given by Dmidecode. But  https://review.openstack.org/#/c/396406/ collects the bios given NIC's name  using biosdevname utility. The spec for that is https://review.openstack.org/#/c/404372/. What is does in brief is the biosdevname utility collects the Bios given16:28
aslezil_name of NIC's and it adds  to the data collected from default collector. The biosdevname is added under inventory-->interfaces-->biosdevname.16:28
patchbotpatch 406790 - ironic-inspector-specs - Collect Bios, Memory and Processor Info from Dmide...16:28
patchbotpatch 396406 - ironic-python-agent - Collect NIC name given by BIOS16:28
patchbotpatch 404372 - ironic-inspector-specs - Collect NIC name given by BIOS (MERGED)16:28
aslezil_milan: each interface has its ``biosdevname`` with value or None.16:29
*** aarefiev is now known as aarefiev_afk16:32
mgouldpas-ha: left some comments16:32
*** sayantan_ has joined #openstack-ironic16:32
mgouldI think libguestfs is needed to build IPA?16:32
milanaslezil_, yeah, but the other thing is w/ dmidecode spec we have a "bios" substructure in introspection data; I'm wondering whether the nic bios info shouldn't go there for consistency sake16:32
milanaslezil_, but I guess since the spec is in place already16:34
milanaslezil_, the data is fine to go to the nic16:34
*** jaosorior has joined #openstack-ironic16:34
vdrokaaaand, the new logo arrived!16:34
pas-haimo this one is the best so far16:35
milanaslezil_, but I see the kernel--bios name sync dmidecode wouldn't be able to guess16:35
pas-habasically simplified original version :)16:35
milanaslezil_, OK, cool never mind me ;)16:35
*** e0ne has quit IRC16:36
*** e0ne has joined #openstack-ironic16:37
aslezil_milan: sorry for confusion :)16:37
mgouldvdrok: yeah, I like this one16:37
mgouldI think it's as close to the existing pixieboots as is possible within the style guidelines16:38
*** cdearborn has quit IRC16:38
vdrokmgould: it has drumsticks, they are not natural objects, they should redraw! :D16:38
milanaslezil_, no worries, I confused myself :D16:38
mgouldvdrok: nonono, they're clearly, er, dandelions16:39
mgoulddefinitely something natural16:39
mgould[please don't start that again :-)]16:39
vdroklol16:39
*** e0ne_ has joined #openstack-ironic16:40
*** vsaienko has joined #openstack-ironic16:41
*** e0ne has quit IRC16:41
*** jaosorior has quit IRC16:42
*** alexpilotti has quit IRC16:42
*** e0ne_ has quit IRC16:42
openstackgerritVasyl Saienko proposed openstack/ironic master: Add Ironic standalone tests  https://review.openstack.org/42355616:42
*** vsaienko has quit IRC16:43
*** e0ne has joined #openstack-ironic16:43
*** alexpilotti has joined #openstack-ironic16:43
*** derekh has quit IRC16:43
sambettsPixie boots the peanut with a face16:43
JayFLOL16:45
*** dims_ has quit IRC16:45
JayFpeanut with a faaaaaaace16:45
mgouldsambetts: do you know what we're using libguestfs for?16:47
*** alexpilotti has quit IRC16:47
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Add bindep.txt file  https://review.openstack.org/44420116:48
*** e0ne has quit IRC16:48
sambettsmgould: the tinyipa scripts don't use it, maybe coreos ones do? #16:50
mgouldsambetts: ah, maybe16:51
sambettsmgould: where are you seeing libguestfs requirements?16:51
mgouldsambetts: https://review.openstack.org/#/c/444201/16:52
patchbotpatch 444201 - ironic - Add bindep.txt file16:52
* mgould can't find any explicit uses in DIB16:52
*** e0ne has joined #openstack-ironic16:53
*** vsaienko has joined #openstack-ironic16:53
mgouldpas-ha, are you sure we need it?16:53
sambettsmgould: taking a quick look at the coreos build stuff I don't see any deps there16:53
mgouldyeah16:54
mgouldI'm starting to think it's useful for looking inside disk images, but not actually needed16:54
*** e0ne has quit IRC16:54
mgouldoh, it's an optional dependency of nova16:56
*** logan_ has joined #openstack-ironic16:56
*** gfhellma has joined #openstack-ironic16:56
*** logan_ is now known as Guest2958916:57
*** bfournie has quit IRC16:57
pas-hawhat about dib, does it need something apart from qemu-img and squashfs-tools?16:58
*** e0ne has joined #openstack-ironic16:58
*** logan- has quit IRC16:59
openstackgerritJohn L. Villalovos proposed openstack/ironic master: DNM/WIP: Debugging grenade upgrade  https://review.openstack.org/44433517:01
*** e0ne has quit IRC17:02
*** ralonsoh has quit IRC17:03
*** Guest29589 is now known as logan-17:03
*** alexpilotti has joined #openstack-ironic17:04
*** racedo has quit IRC17:04
*** vsaienko has quit IRC17:05
*** vsaienko has joined #openstack-ironic17:05
mgouldpas-ha: `git grep guestf` returns no hits in my DIB checkout17:05
joannamorning pas-ha, mgould17:06
mgouldmorning joanna17:06
joannamorning vdrok, JayF, aslezil_17:06
JayFo/17:06
vdrokmorning joanna17:08
*** vsaienko has quit IRC17:09
dtantsuroh new Pixie variant!17:09
dtantsurlucasagomes, checked it out already?17:09
lucasagomesdtantsur, hah lemme see17:10
lucasagomesdtantsur, hmm out of everything proposed seems to be the best one17:10
lucasagomesIMHO ofc17:10
JayF+217:11
aslezil_morning joanna:17:11
*** vsaienko has joined #openstack-ironic17:11
dtantsuryep17:11
lucasagomescommented17:12
* milan calls it a day17:13
milangood night Ironic! :)17:13
*** milan has quit IRC17:13
lucasagomesmilan, g'night!17:13
* lucasagomes is going soon too17:13
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Neutron notifications  https://review.openstack.org/44077817:17
joannanight milan :)17:17
*** rpioso has joined #openstack-ironic17:20
pas-hag'night all, and have a nice weekend :)17:20
lucasagomespas-ha, night!17:21
lucasagomescalling it a day as well17:21
lucasagomeshave a great evening/weekend all17:21
*** lucasagomes is now known as lucas-afk17:22
*** vills has quit IRC17:23
*** vills has joined #openstack-ironic17:23
*** vills has quit IRC17:24
*** Goneri has quit IRC17:25
*** e0ne has joined #openstack-ironic17:26
*** e0ne has quit IRC17:26
*** Goneri has joined #openstack-ironic17:27
*** e0ne has joined #openstack-ironic17:29
*** jcoufal_ has joined #openstack-ironic17:32
*** bfournie has joined #openstack-ironic17:32
*** alexpilo_ has joined #openstack-ironic17:33
*** jcoufal has quit IRC17:35
*** alexpilotti has quit IRC17:37
*** vishwanathj has quit IRC17:37
*** vishwanathj has joined #openstack-ironic17:37
dtantsurgoing as well, see you17:41
*** dtantsur is now known as dtantsur|afk17:41
openstackgerritVasyl Saienko proposed openstack/ironic master: Add Ironic standalone tests  https://review.openstack.org/42355617:44
*** alineb has quit IRC17:45
openstackgerritGalyna Zholtkevych proposed openstack/ironic-python-agent master: Correct failure message output when downloading  https://review.openstack.org/44107217:46
*** vills has joined #openstack-ironic17:48
*** vills_ has joined #openstack-ironic17:49
vdroknight pas-ha lucas-afk and dtantsur|afk17:51
*** vills has quit IRC17:53
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Neutron notifications  https://review.openstack.org/44077817:58
jlvillalvdrok, 8PM on a Friday night where you are ;)18:00
vdrokthrowing out some shitty code :)18:00
jlvillalheh. I don't believe it!18:00
jlvillalThat's my job! ;)18:00
vdrokjust take a look at that patch :D OTOH, I know what to write in the spec now18:01
jlvillalExcellent news for the spec18:01
*** mgoddard has quit IRC18:01
*** e0ne has quit IRC18:02
*** jkilpatr has quit IRC18:03
*** Nisha_Agarwal has quit IRC18:03
*** Nisha_Agarwal has joined #openstack-ironic18:03
*** Goneri has quit IRC18:03
mariojvlucas-afk: +2+W your patch18:05
mariojvlooking forward to the follow up18:05
*** jkilpatr has joined #openstack-ironic18:06
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Neutron notifications  https://review.openstack.org/44077818:07
*** Nisha_Agarwal has quit IRC18:09
vdrokgood night everyone, /me leaves for the weekend18:14
openstackgerritJay Faulkner proposed openstack/ironic-specs master: Add spec for specific faults support  https://review.openstack.org/33411318:15
rookon the cleaning I am seeing : Failed on step {}.18:15
rookwhat should I have seen {} ?18:15
joannarook: is there any other info?18:18
joannalike timeout or something?18:18
rookyeah, states a timeout was reached18:18
rookbut at what step?18:18
joannait means that node's clean_step was empty18:19
rookAsync execution of cleanup_cleanwait_timeout failed with error:18:19
joanna:q18:19
joannasorry, irc not vim :D18:20
rookCan not transition from state 'clean failed' on event 'fail' (no defined transition)18:20
aNuposicrook: you may need to put back to manageable state via CLI if it is in 'clean failed'18:22
*** vills_ has quit IRC18:25
joannarook: from what I see cleanup_cleanwait_timeout is executed when a node is unresponsive (no heartbeats). But I may be wrong :)18:26
JayFrook: I suspect you have a problem with booting the ramdisk altogether18:28
JayFrook: and since IPA can contribute to what clean steps are run, if it's not able to boot the ramdisk for cleaning, it wouldn't have a specific step it failed on18:29
JayFrook: havev you had this node successfully pxe boot (clean or deploy) before?18:29
rookJayF: yup, PXEd fine18:30
rookneed to pull a console18:30
*** g3ek has quit IRC18:37
*** fragatina has quit IRC18:39
sambettsNight all, /me is traveling next week so might be sparadically online18:41
*** sambetts is now known as sambetts|afk18:41
joannagnight, sambetts|afk18:43
* mgould -> home; have a good weekend!18:43
*** mgould is now known as mgould|afk18:43
*** g3ek has joined #openstack-ironic18:46
*** sayantan_ has quit IRC18:51
*** jcoufal has joined #openstack-ironic18:51
openstackgerritMerged openstack/ironic-lib master: Improve logs for the root device hints mechanism  https://review.openstack.org/44315918:53
*** Goneri has joined #openstack-ironic18:53
*** jcoufal_ has quit IRC18:54
openstackgerritJoanna Taryma proposed openstack/ironic master: Remove overwriting the default value of db_max_retries.  https://review.openstack.org/44086518:55
openstackgerritJohn L. Villalovos proposed openstack/ironic master: grenade: Only 'enable_plugin ironic' if not already in conf  https://review.openstack.org/44433518:58
*** mgoddard has joined #openstack-ironic19:00
openstackgerritJohn L. Villalovos proposed openstack/ironic master: grenade: Only 'enable_plugin ironic' if not already in conf  https://review.openstack.org/44433519:03
openstackgerritJohn L. Villalovos proposed openstack/ironic master: WIP/DNM: Testing multi-tenant multi-node grenade job  https://review.openstack.org/42214919:06
*** vsaienko has quit IRC19:10
*** vsaienko has joined #openstack-ironic19:12
openstackgerritGalyna Zholtkevych proposed openstack/ironic-python-agent master: Correct failure message output when downloading  https://review.openstack.org/44107219:17
*** vsaienko has quit IRC19:20
*** vills has joined #openstack-ironic19:23
*** bfournie has quit IRC19:26
*** sayantan_ has joined #openstack-ironic19:28
jlvillalgalyna, Are you there? I wonder if when changing that exception if we should store the image_id too.19:28
*** vsaienko has joined #openstack-ironic19:28
jlvillalgalyna, Or I might be over-engineering it...19:28
jlvillalgalyna, +219:30
*** jkilpatr has quit IRC19:33
*** jcoufal_ has joined #openstack-ironic19:34
*** bnemec is now known as beekneemech19:34
*** jcoufal has quit IRC19:35
*** mgoddard has quit IRC19:36
*** jkilpatr has joined #openstack-ironic19:36
*** vills has quit IRC19:40
*** vills has joined #openstack-ironic19:41
*** vills has quit IRC19:41
*** chlong_ has joined #openstack-ironic19:42
*** jkilpatr has quit IRC19:44
openstackgerritJohn L. Villalovos proposed openstack/ironic master: grenade: Only 'enable_plugin ironic' if not already in conf  https://review.openstack.org/44433519:48
*** dims has joined #openstack-ironic19:49
*** jcoufal has joined #openstack-ironic19:52
*** Goneri has quit IRC19:54
*** jcoufal_ has quit IRC19:54
*** fragatina has joined #openstack-ironic19:55
*** jkilpatr has joined #openstack-ironic19:58
*** vsaienko has quit IRC19:59
*** bfournie has joined #openstack-ironic20:00
*** jcoufal_ has joined #openstack-ironic20:01
*** bfournie has quit IRC20:01
*** vsaienko has joined #openstack-ironic20:02
*** _vsaienko has joined #openstack-ironic20:04
*** vsaienko has quit IRC20:04
*** jcoufal has quit IRC20:04
*** sambetts|afk has quit IRC20:06
*** sambetts_ has joined #openstack-ironic20:08
*** ChubYann has joined #openstack-ironic20:08
*** crushil has quit IRC20:10
*** dyasny has quit IRC20:12
*** goldyfruit has quit IRC20:15
hamzyI am trying to follow https://docs.openstack.org/developer/openstack-ansible-os_ironic/ocata/configure-ironic.html#deploy-a-baremetal-node-kicked-with-ironic and am having difficulty providing a valid neutron net-create command20:16
hamzyInvalid input for operation: physical_network 'public' unknown for flat provider network.20:16
*** bfournie has joined #openstack-ironic20:19
*** vishwanathj has quit IRC20:25
*** vishwanathj has joined #openstack-ironic20:26
*** _vsaienko has quit IRC20:27
*** vsaienko has joined #openstack-ironic20:28
*** vsaienko has quit IRC20:29
*** vsaienko has joined #openstack-ironic20:30
*** _vsaienko has joined #openstack-ironic20:33
*** vsaienko has quit IRC20:33
*** jcoufal_ has quit IRC20:34
*** _vsaienko has quit IRC20:35
*** jcoufal has joined #openstack-ironic20:35
jlvillalSeems odd that the Python 3 test job appears to be running Python 2.7: http://logs.openstack.org/56/423556/80/experimental/gate-tempest-dsvm-ironic-ipa-partition-pxe_ipmitool-tinyipa-python3-ubuntu-xenial-nv/9a5945b/logs/screen-ir-api.txt.gz?20:35
*** vsaienko has joined #openstack-ironic20:36
*** amoralej is now known as amoralej|off20:44
*** sayantan_ has quit IRC20:57
*** sayantan_ has joined #openstack-ironic21:00
*** sayantan_ has quit IRC21:01
*** sayantan_ has joined #openstack-ironic21:02
*** jcoufal_ has joined #openstack-ironic21:03
*** jcoufal has quit IRC21:04
*** sayantan_ has quit IRC21:05
*** Goneri has joined #openstack-ironic21:07
*** vsaienko has quit IRC21:09
*** vsaienko has joined #openstack-ironic21:14
*** sayantan_ has joined #openstack-ironic21:20
*** vsaienko has quit IRC21:29
*** jtomasek has quit IRC21:30
openstackgerritJoanna Taryma proposed openstack/ironic-specs master: Clean invalid conflict response usages  https://review.openstack.org/41419421:31
*** vsaienko has joined #openstack-ironic21:34
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362821:36
openstackgerritJohn L. Villalovos proposed openstack/ironic master: grenade: Only 'enable_plugin ironic' if not already in conf  https://review.openstack.org/44433521:38
openstackgerritJohn L. Villalovos proposed openstack/ironic master: grenade: Remove enabling of ironic devstack plugin  https://review.openstack.org/44450921:42
*** vsaienko has quit IRC21:43
*** jcoufal has joined #openstack-ironic21:48
aNuposicFolks, has anyone seen py35 failing with tox? I am getting "No module named pysqlite2", is that new module got shipped inside py35?21:49
aNuposicrama_y, ^21:49
aNuposicI followed https://docs.openstack.org/developer/ironic/dev/dev-quickstart.html?highlight=py35#optional-installing-py35-requirements to install py35 inside my devstack environment21:49
*** vishwanathj has quit IRC21:49
*** jcoufal_ has quit IRC21:50
rama_yaNuposic, I too am getting the same errors21:50
*** vishwanathj has joined #openstack-ironic21:50
*** frontrunner has quit IRC21:50
jlvillalaNuposic, rama_y: Have you tried:  tox -vvv -epy35 -r21:59
jlvillal-r is the key21:59
*** jcoufal has quit IRC22:01
*** jcoufal has joined #openstack-ironic22:02
rama_yjlvillal, aNuposic, tried the above.22:05
rama_ygetting the same errors.22:05
*** g3ek has quit IRC22:07
jlvillalrama_y, $ tox -vvv -epy35 -r22:07
jlvillalworked for me. On master ironic.22:07
jlvillalI made sure to update to latest22:07
aNuposicjlvillal, rama_y let me try with -r22:08
aNuposicjlvillal, rama_y what does -r do? sorry not played that much with tox22:09
jlvillalaNuposic, $ tox --help22:09
aNuposicah recreate got it22:10
*** priteau has quit IRC22:14
*** mgoddard has joined #openstack-ironic22:15
*** g3ek has joined #openstack-ironic22:16
*** Goneri has quit IRC22:18
*** jcoufal_ has joined #openstack-ironic22:18
jlvillalaNuposic, Any change for your test run?22:19
*** jcoufal has quit IRC22:19
aNuposicjlvillal, nope, i am pulling latest code, i had it pulled a week or so back22:20
aNuposicjlvillal, that might help22:20
*** chlong_ has quit IRC22:21
*** baha has quit IRC22:24
*** jjohnson2 has quit IRC22:25
*** fragatin_ has joined #openstack-ironic22:28
aNuposicjlvillal, still the same error22:28
*** fragatina has quit IRC22:32
openstackgerritJonathan Provost proposed openstack/ironic-python-agent stable/newton: Store image on disk when bigger than RAM size  https://review.openstack.org/44452522:40
*** mgoddard has quit IRC22:47
openstackgerritJonathan Provost proposed openstack/ironic-python-agent master: Store image on disk when bigger than RAM size  https://review.openstack.org/43044222:48
rama_yjlvillal, aNuposic: so this worked for me: installed libsqlite3-dev and reinstalled python3.5.22:51
rama_ypy35 tests passed.22:51
aNuposicrama_y, i installed libsqlite3-dev22:51
aNuposicbut i did not reinstalled python3.522:51
rama_yDid the tests pass?22:52
aNuposici installed libsqlite3-dev after installing python3.522:52
aNuposicrama_y, nope22:52
aNuposicrama_y, but may be i should reinstall python3.522:52
rama_yYes22:52
*** sasha21 has quit IRC22:53
*** fragatin_ has quit IRC23:02
*** fragatina has joined #openstack-ironic23:03
aNuposicrama_y, jlvillal yup configuring py35 again after installing libsqlite3-dev worked, sweet :)23:12
jlvillalaNuposic, rama_y Glad you got it working :)23:12
jlvillalaNuposic, rama_y Maybe one of you could update the docs.23:12
aNuposicprobably it should be added in devstack docs23:13
aNuposicyes exactly i was going to say23:13
aNuposicjlvillal,23:13
jlvillalaNuposic, In the System Prerequisites23:14
jlvillalThe following packages cover the prerequisites for a local development environment...23:14
jlvillalThat area23:14
rama_yjlvillal, aNuposic, I was thinking the same. Do you think just a patch is sufficient or a bug needs to be entered as well?23:14
jlvillalI would just do a patch.23:14
aNuposiccool, so rama_y do you want to update or i can go ahead and update it23:14
jlvillalPlease provide package for both Ubuntu and Fedora23:14
rama_yaNuposic, please go ahead.23:15
aNuposicrama_y, /o\23:15
*** vills has joined #openstack-ironic23:17
*** [1]cdearborn has quit IRC23:17
*** jcoufal has joined #openstack-ironic23:17
aNuposicrama_y, cool i will do that23:17
rama_yaNuposic, jlvillal, thanks.23:19
aNuposicjlvillal, yes will update that for both ubuntu and fedora23:19
aNuposicthanks23:19
*** jcoufal_ has quit IRC23:19
jlvillalaNuposic, Thanks23:21
*** tesseract has quit IRC23:29
*** sayantan_ has quit IRC23:30
*** jkilpatr has quit IRC23:34
*** e0ne has joined #openstack-ironic23:41
openstackgerritAnup Navare proposed openstack/ironic master: Allow logical name in port creation  https://review.openstack.org/40558623:42
*** baoli has quit IRC23:43
*** baoli has joined #openstack-ironic23:45
*** baoli has quit IRC23:48
*** jjohnson2 has joined #openstack-ironic23:56
« Thursday, 2017-03-09 Index Saturday, 2017-03-11 »

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!