Friday, 2017-02-03

jrollfeels like neutron should handle the differences between dhcp servers, imo00:00
Sukhdevjroll: I can bring it at ptg with neutron team. In the meantime, if you can think of simplest hack I put to move forward, I would really appreciate it00:06
jrollSukhdev: simplest hack would be use dnsmasq imo :)00:06
* jroll knows nothing about how to do the equivalent of 'tag:!ipxe,bootfile-name' on juniper's stuff00:07
TheJuliaSukhdev: The patch likely won't help you, it is for... more so... static dhcp configurations, not neutron integrated00:07
*** rob0t_ has joined #openstack-ironic00:08
Sukhdevjroll: I do not have choice in the backend. So, a hack into  the ironic side?00:08
SukhdevTheJulia: :-)00:08
TheJuliaSukhdev: I think that is what we're suggesting00:08
TheJuliaAt a glance, I'm not sure we can do the logic that the tags provide us.... which makes me think we would have to be able to explicitly state if it is ipxe, not ipxe.  i.e. offer no upgrade capability00:09
TheJulia:(00:09
*** rob0t_ has quit IRC00:12
SukhdevH00:12
SukhdevLooks like I kinda stuck for now00:13
*** chedda has joined #openstack-ironic00:14
TheJuliaSukhdev: first step would be to disable ipxe00:14
SukhdevTheJulia: how to do that00:16
*** rloo has quit IRC00:16
*** rob0t_ has joined #openstack-ironic00:20
TheJuliaSukhdev:  IRONIC_IPXE_ENABLED=false in your local.cf00:20
SukhdevTheJulia: got it... Thanks00:21
*** rob0t_ has quit IRC00:24
openstackgerritJohn L. Villalovos proposed openstack/ironic master: WIP/DNM: Testing multinode job  https://review.openstack.org/42214900:28
*** baoli has joined #openstack-ironic00:35
*** e0ne has quit IRC00:46
*** jaosorior has quit IRC00:46
*** hw_wutianwei has joined #openstack-ironic00:51
*** rama_y has quit IRC01:04
*** crushil has joined #openstack-ironic01:20
*** wrouesnel has joined #openstack-ironic01:25
*** sdake has joined #openstack-ironic01:28
*** owalsh has joined #openstack-ironic01:28
*** owalsh-afk has quit IRC01:28
*** gcb has joined #openstack-ironic01:31
*** baoli has quit IRC01:32
*** amotoki has joined #openstack-ironic01:33
*** rob0t_ has joined #openstack-ironic01:36
openstackgerritAnnie Lezil proposed openstack/ironic-python-agent master: Collect NIC name given by BIOS  https://review.openstack.org/39640601:43
*** baoli has joined #openstack-ironic01:48
*** aNuposic has quit IRC01:52
*** chedda has quit IRC01:54
*** anteaya has quit IRC01:57
*** anteaya has joined #openstack-ironic02:04
*** rob0t_ has quit IRC02:07
*** aNuposic has joined #openstack-ironic02:09
*** baoli has quit IRC02:13
*** rob0t_ has joined #openstack-ironic02:14
*** baoli has joined #openstack-ironic02:17
*** baoli has quit IRC02:18
*** rajinir has quit IRC02:26
*** mtanino has quit IRC02:28
*** aNuposic has quit IRC02:30
*** Syed__ has quit IRC02:35
*** adu has joined #openstack-ironic02:38
*** adu has quit IRC02:42
*** amotoki_ has joined #openstack-ironic02:44
*** amotoki has quit IRC02:46
*** wrouesnel has quit IRC02:56
*** wrouesnel has joined #openstack-ironic02:56
*** amotoki_ has quit IRC02:57
*** pmannidi has quit IRC02:57
*** pmannidi has joined #openstack-ironic02:58
*** baoli has joined #openstack-ironic02:58
*** wrouesnel has quit IRC02:59
*** wrouesnel has joined #openstack-ironic02:59
*** wrouesnel has quit IRC03:01
*** wrouesnel has joined #openstack-ironic03:01
*** wrouesnel has quit IRC03:04
*** wrouesnel has joined #openstack-ironic03:04
*** crushil has quit IRC03:05
*** wrouesnel has quit IRC03:06
*** wrouesnel has joined #openstack-ironic03:06
*** sayantan_ has quit IRC03:11
*** ijw has quit IRC03:14
*** wrouesnel has quit IRC03:15
*** wrouesnel has joined #openstack-ironic03:15
*** adu has joined #openstack-ironic03:18
*** baoli has quit IRC03:19
*** wrouesnel has quit IRC03:19
*** wrouesnel has joined #openstack-ironic03:19
*** sayantani01 has joined #openstack-ironic03:20
*** baoli has joined #openstack-ironic03:23
*** crushil has joined #openstack-ironic03:25
*** owalsh has quit IRC03:33
*** owalsh_ has joined #openstack-ironic03:33
*** vikrant has joined #openstack-ironic03:34
*** baoli has quit IRC03:37
*** wrouesnel has quit IRC03:37
*** wrouesnel has joined #openstack-ironic03:37
*** amotoki has joined #openstack-ironic03:38
*** wrouesnel has quit IRC03:40
*** wrouesnel has joined #openstack-ironic03:40
*** sdake has quit IRC03:41
*** crushil has quit IRC03:41
*** amotoki has quit IRC03:45
*** wrouesnel has quit IRC03:53
*** wrouesnel has joined #openstack-ironic03:53
*** wrouesnel has quit IRC03:59
*** wrouesnel has joined #openstack-ironic03:59
*** adu has quit IRC04:01
*** owalsh_ is now known as owalsh-afk04:01
*** wrouesnel has quit IRC04:02
*** wrouesnel has joined #openstack-ironic04:02
*** crushil has joined #openstack-ironic04:07
*** sdake has joined #openstack-ironic04:07
*** wrouesnel has quit IRC04:08
*** wrouesnel has joined #openstack-ironic04:08
*** sdake_ has joined #openstack-ironic04:11
*** nmathew has joined #openstack-ironic04:12
*** sdake has quit IRC04:12
*** udesale has joined #openstack-ironic04:13
*** jaganathan has joined #openstack-ironic04:15
*** Sukhdev has quit IRC04:21
*** sayantani01 has quit IRC04:26
*** crushil has quit IRC04:33
*** crushil has joined #openstack-ironic04:33
*** crushil has quit IRC04:39
*** crushil has joined #openstack-ironic04:41
*** ijw has joined #openstack-ironic04:43
*** adu has joined #openstack-ironic04:47
*** jvgrant has quit IRC04:48
openstackgerritAparna proposed openstack/proliantutils master: Add HPSUM firmware update support  https://review.openstack.org/42200404:55
*** jvgrant has joined #openstack-ironic04:57
*** Nisha_Agarwal has joined #openstack-ironic05:06
openstackgerritMadhuri Kumari proposed openstack/ironic master: Fix directories permission  https://review.openstack.org/42707705:09
*** adu has quit IRC05:14
*** crushil has quit IRC05:16
*** crushil has joined #openstack-ironic05:16
*** ijw has quit IRC05:17
*** aparnav has joined #openstack-ironic05:29
*** sayantani01 has joined #openstack-ironic05:33
*** Sukhdev has joined #openstack-ironic05:39
*** Nisha_Agarwal has quit IRC05:44
*** sasha2 has quit IRC05:50
*** wrouesnel has quit IRC05:56
*** wrouesnel has joined #openstack-ironic05:57
openstackgerritJohn L. Villalovos proposed openstack/ironic master: WIP/DNM: Testing multinode job  https://review.openstack.org/42214906:09
*** MattMan_1 has quit IRC06:12
*** MattMan_1 has joined #openstack-ironic06:12
*** crushil has quit IRC06:13
*** rbudden has quit IRC06:14
*** vsaienko has joined #openstack-ironic06:15
*** wrouesnel has quit IRC06:16
*** wrouesnel has joined #openstack-ironic06:16
*** vsaienko has quit IRC06:17
*** wrouesnel has quit IRC06:24
*** wrouesnel has joined #openstack-ironic06:24
*** mjura has joined #openstack-ironic06:32
*** Sukhdev has quit IRC06:43
openstackgerritAparna proposed openstack/ironic master: Support `update_firmware` to hpsum based firmware update  https://review.openstack.org/42257206:44
*** sdake_ has quit IRC06:52
*** vikrant has quit IRC06:53
*** sayantani01 has quit IRC07:07
*** sayantani01 has joined #openstack-ironic07:07
*** tesseract has joined #openstack-ironic07:20
*** rcernin has joined #openstack-ironic07:26
*** athomas has joined #openstack-ironic07:28
openstackgerritDao Cong Tien proposed openstack/python-ironicclient master: Extends driver-list, driver-show supporting new hardware types  https://review.openstack.org/41927407:33
*** ChubYann has quit IRC07:44
*** sayantani01 has quit IRC07:48
*** sayantani01 has joined #openstack-ironic07:48
*** vsaienko has joined #openstack-ironic07:51
*** ralonsoh has joined #openstack-ironic08:13
openstackgerritVasyl Saienko proposed openstack/ironic master: TMP: Verify multinode nova fix  https://review.openstack.org/41416008:17
*** sayantani01 has quit IRC08:33
vdrokmorning ironic!08:35
*** jaganathan has quit IRC08:38
*** fxpester has joined #openstack-ironic08:55
*** jaganathan has joined #openstack-ironic08:55
*** rob0t_ has quit IRC08:59
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-ironic09:00
*** derekh has joined #openstack-ironic09:13
*** amarao has joined #openstack-ironic09:14
*** dtantsur|afk is now known as dtantsur09:23
dtantsurmorning ironic, morning vdrok09:23
dtantsurTGIF finally09:23
openstackgerritMerged openstack/ironic-ui master: Improvements to enroll-node service  https://review.openstack.org/42632309:27
openstackgerritMerged openstack/ironic-ui master: Remove node-action deleteNodes/deletePorts functions  https://review.openstack.org/42633409:27
openstackgerritMerged openstack/ironic-ui master: Imported Translations from Zanata  https://review.openstack.org/42666709:27
openstackgerritMerged openstack/ironic-ui master: Improve error handling for the ironic-ui service  https://review.openstack.org/42630309:28
*** udesale__ has joined #openstack-ironic09:28
*** udesale has quit IRC09:30
*** athomas has quit IRC09:34
*** mgoddard_ has joined #openstack-ironic09:37
*** lucas-afk is now known as lucasagomes09:38
lucasagomesmorning all09:38
*** chlong has quit IRC09:38
openstackgerritMarkos Chandras proposed openstack/bifrost master: roles: bifrost-ironic-install: Always download the latest iPXE files  https://review.openstack.org/42768209:43
*** udesale__ has quit IRC09:46
*** udesale has joined #openstack-ironic09:46
*** udesale has quit IRC09:46
*** amotoki has joined #openstack-ironic09:47
*** udesale has joined #openstack-ironic09:52
vdrokmorning dtantsur and lucasagomes, tgif! :)09:57
lucasagomeso/ indeed09:57
amaraoHello everyone. I found something strange in ironic: nova shows 5 hypervisor for one compute. And this is exactly number of ironic nodes I have. Is this a coincidence? Is this a bug or a feature?10:03
*** amotoki has quit IRC10:05
dtantsuramarao, this is how ironic nodes are represented in nova10:05
dtantsurso this is a feature, yes :)10:05
amaraodtantsur: Oh, thanks. A bit unexpected at first, but reasonable after some thoughts. I think it should be noted somewhere in the docs for ironic installation.10:07
dtantsuryeah, good point10:07
dtantsuramarao, mind filing a bug for this missing docs bit?10:08
openstackgerritGalyna Zholtkevych proposed openstack/ironic master: Fix object save after refresh failure  https://review.openstack.org/35474410:12
amaraodtanstsur, ack.10:13
*** jaganathan has quit IRC10:17
openstackgerritMerged openstack/bifrost master: Remove link to modindex  https://review.openstack.org/42809710:22
*** vsaienko has quit IRC10:25
*** sambetts|afk is now known as sambetts10:27
sambettsMorning all o/10:27
lucasagomessambetts, o/10:28
sambettshey lucasagomes10:30
amaraoI messing up with scheduler and I stumble on a question: how Ironic (nova-compute?) desides which ironic node is served by which compute? I'm trying to create availability zones. I created two computes, but I can't find a way to split nodes between those computes.10:30
*** udesale__ has joined #openstack-ironic10:30
*** amotoki has joined #openstack-ironic10:31
*** jaganathan has joined #openstack-ironic10:32
*** udesale__ has quit IRC10:33
*** udesale has quit IRC10:33
dtantsurmorning sambetts10:34
dtantsuramarao, I think it used a hash ring to distribute nodes. I think jroll, when he's here, can give a more detailed answer.10:35
amaraothanks.10:35
*** owalsh-afk is now known as owalsh10:35
amaraoBut is here a way to create some kind of availability zones/aggregates? Or it is ouside of current features?10:35
sambettsamarao: Ironic only exposes a single pool of resources, the only way that I can see aggregates working is having multiple nova-computes that point at different Ironics with different pools of resources10:37
amaraoBut is this possible within a single openstack installation? I mean, to have two or more ironic in one keystone endpoints catalog?10:38
amaraosambetts: I think it's possible to make some mechanism to associate given ironic node to a given nova-compute. After that all nova features with aggregates, availability zones, metas in flavors, etc, would start to work.10:39
*** amotoki has quit IRC10:40
sambettshmm that could be interesting, I'd have to think about the HA impact of that, the way it works today means that if any nova compute died the others can take over, in a scenario where a ironic node is explictly tied to a nova-compute you lose that10:42
amaraosambetts: It's can be easily fixed with few nova-computes with the same set of affinity. F.e. 2 compute nodes for aggregate1, 2 compute nodes for aggregate2.10:43
*** hw_wutianwei has quit IRC10:44
amaraoSo for now a single way to prefer some nodes over others during instance scheduling is to use different flavors with different meta...10:48
sambettsamarao: yeah flavor extra specs is one way to do it currently10:48
vdrokmorning sambetts10:49
sambettshey vdrok10:49
amaraoThanks. Mind if I file a bug/rfe for aggregates?10:49
vdrokamarao: morning, in the new resource providers world that would not be needed10:49
sambettsvdrok: oh?! will that allow us to do affinity?10:49
sambettsvdrok: e.g. have an aggregate that pick one rack over another?10:50
vdrokamarao: each ironic node is a resource provider in nova. in future you'll be able to associate resource providers to aggregates and scheduler will be able to schedule based on that10:50
amaraoWow. Sounds cool.10:50
sambettsthat sounds awesome!10:51
openstackgerritDmitry Tantsur proposed openstack/ironic master: Log an actual error message when failed to load new style credentials  https://review.openstack.org/42869010:51
vdroksambetts: if you create your aggregates that way, I think it is10:51
sambettswe should definatly get an RFE up for it, just to track the idea10:51
sambettsand then maybe vdrok you can leave a comment on that regarding the resource providers10:51
amaraoOk, I'll write it now.10:51
sambettsamarao: thanks!10:52
vdrokbut, I think it's not happening until pike, I mean, resource providers are already created in nova, but scheduler does not know yet anything about the aggregates they are associated to10:52
sambetts+1 thats why I want to track it10:52
openstackgerritGalyna Zholtkevych proposed openstack/ironic master: ETAGs maintenance for all api node's requests  https://review.openstack.org/39221310:52
sambettsbecause its totally a think that I want to see for affinity and stuff10:52
* dtantsur -> food and errands, back later10:55
*** dtantsur is now known as dtantsur|bbl10:55
*** jaganathan has quit IRC10:59
*** yolanda has quit IRC11:08
*** yolanda has joined #openstack-ironic11:10
*** jaganathan has joined #openstack-ironic11:11
openstackgerritMerged openstack/ironic-inspector master: Add plugin to process basic management LLDP TLVs  https://review.openstack.org/40649611:17
*** sdake has joined #openstack-ironic11:18
*** athomas has joined #openstack-ironic11:21
*** jaganathan has quit IRC11:23
*** amotoki has joined #openstack-ironic11:26
jrollmorning y'all11:31
sambettsmorning jroll11:31
jrollvdrok: thanks for answering that, trying to think about the hash ring + aggregates made my brain hurt11:31
jroll(I still think we may need to do some work there)11:31
vdrokmorning jroll11:32
jrollI'm thinking we'll need to make this aggregate aware or something https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L541-L54911:32
jrollidk11:32
jrolltis weird11:32
* jroll needs to make a ptg list of nova conversations to have11:33
*** cdearborn has joined #openstack-ironic11:33
vdrokjroll: hmm, the hash_ring was added purely to prevent overreporting of resources right? and as it seems to me that it won't be the case if resource providers are used, and we might just drop it?11:35
jrollvdrok: well, also so that two n-cpu processes don't both try to control a single ironic node at the same time11:35
*** jaganathan has joined #openstack-ironic11:36
jrollnova has locks on instances but those are in-memory so can't be shared between n-cpus11:36
*** bigjools has quit IRC11:36
openstackgerritMarkos Chandras proposed openstack/ironic-python-agent master: imagebuild: tinyipa: Make the install-deps.sh SUSE aware  https://review.openstack.org/42870411:37
jrolldistributed locking would help, maybe completely solve things11:37
amarao jroll, how nodes (hypervisors) are assigned to nova-computes?11:37
*** bigjools has joined #openstack-ironic11:37
*** amotoki has quit IRC11:37
jrollamarao: somewhat randomly, let me find the release notes and see if that makes sense11:37
vdrokamarao: https://specs.openstack.org/openstack/nova-specs/specs/newton/implemented/ironic-multiple-compute-hosts.html11:37
jrollah yeah, spec would work :P11:38
*** jkilpatr has quit IRC11:38
amaraovdrok: thanks. I thought to patch it for our needs to make it 'rack aware', but I looks our of my reach.11:40
*** sdake_ has joined #openstack-ironic11:43
jrollamarao: locality awareness is definitely something I (personally) want to do, but it might be a couple cycles before we get there11:45
*** sdake has quit IRC11:46
vdrokhm, yeah, locking is the problem, will have to think more about that too :)11:46
amaraojroll, Yes, that's a difference between development and operations. I will try to do something with it today, may be at monday. If I can't fix (..em... enhance), than I will try other ways around.11:47
jrollamarao: hm? what's the difference?11:48
amaraojroll:Timescale: two release cycles for development, 2 days for operations. Obviously, all I can do is a crude patches, but they will be available in two days or it's no-go.11:50
lucasagomesjroll, morning o/11:50
*** vsaienko has joined #openstack-ironic11:50
jrollamarao: yeah, I hear you, though I think much of that is driven by openstack's great desire to have something mediocre that works for everyone, as opposed to doing something quickly that works for someone and iterating :)11:53
jrollamarao: curious what you're actually trying to do here, let a nova user pick a rack? :)11:54
*** pmannidi has quit IRC11:54
amaraojroll ...and maintainability. 20 lines-patch for 'we need that feature yesteday' will cause some minor headache in the future, but is solvable in reasonable time. If such patch is not 20-lines, but 40k+ - good luck maintain it (I pass).11:55
jrollheh11:55
*** crushil has joined #openstack-ironic11:56
*** jkilpatr has joined #openstack-ironic11:57
*** lucasagomes is now known as lucas-hungry11:57
amaraoHow crazy is idea to replace hash_ring with own implementation which will use properties of node & service to provide affinity?11:57
*** fellypefca has joined #openstack-ironic11:58
*** fellypefca has quit IRC11:59
jrollamarao: yeah, this has been done, that's why I ask what your real goal is11:59
*** fellypefca has joined #openstack-ironic11:59
amaraojroll: sorry. I want to implement a way to use aggregates/availability-zones in nova. In my idea each nova-compute serves own aggregate, and provides (manages?) only nodes with given property. When instance is scheduled, it assigned to aggregate (through --availability-zone option). scheduler chooses nova-compute in that aggregate, and that nova-compute uses only hypervisors with proper meta.12:02
*** rob0t_ has joined #openstack-ironic12:02
jaganathanHi, Please let me know steps to run test cases in ironic-python-agent12:02
jrollamarao: right okay, I think you can do that somehow, melwitt and friends may have some tips when she's around, they've done something like that12:03
jrollI just don't know if the hash ring messes that up, it might12:03
*** jkilpatr has quit IRC12:04
jrolljaganathan: tox, like most openstack projects :)12:04
jrolljaganathan: this is for ironic, but would be similar, just different dependencies http://docs.openstack.org/developer/ironic/dev/dev-quickstart.html12:05
sambettsjroll: I think the problem is that all nova-computes share the same Ironic pool of resources, so you can't use an agregate to force affinity by just picking the nova-compute12:05
jrollsambetts: yeah, there's some scheduler filters involved I think?12:06
sambettsjroll: ah custom for Ironic?12:06
*** rob0t_ has quit IRC12:07
jrollsambetts: yeah something like that12:07
jrollI know yahoo does affinity12:07
jrollbut I don't know if code is up somewhere or any sort of writeup12:07
jrollthey do rack/power affinity for sure. maybe hall/room12:08
jrollnetwork even?12:08
*** vsaienko has quit IRC12:08
*** vsaienko has joined #openstack-ironic12:08
sambettsI think this falls into a similar catagory to conducotr affienty, e.g. having a conductor per rack/row is interesting too12:09
sambettsthen if you did that you could have nova-compute to conductor affienty12:09
jrollyeah, +112:09
jrolland -112:09
jrollrespectively12:10
sambettshehehe12:10
jroll:)12:10
jrollI'd love conductor + image server (or caching proxy) per rack12:10
jrolleven running inside the tor if possible12:11
jrollwould be nice for traffic to not really need to leave the rack to deploy a box12:11
sambettsyeah that would be sweet12:11
sambettsalthough we'd have to fix the IPA talking to the Ironic API thing12:12
jrollah yeah12:12
* sambetts really hates that layer break12:12
sambettsit means that I have to wire two controller boxes into my provisioning network instead of just one12:13
*** rob0t_ has joined #openstack-ironic12:17
*** jkilpatr has joined #openstack-ironic12:17
*** gcb has quit IRC12:19
amaraojroll: If images are in swift, IPA will do requests to outside of the rack.12:19
mat128good morning12:19
jrollamarao: well, yeah, I'm saying I'd love to get there, it isn't trivial today12:20
jrollmorning mat12812:20
*** rob0t_ has quit IRC12:21
*** rob0t_ has joined #openstack-ironic12:23
mat128amarao: I believe you can already attain your goal by using nova filters/weighers12:25
mat128you will only have a single nova-compute but instances of X zone will end up on X types of nodes, which seems to be what you want12:26
mat128(or multiple computes, but not in the sense of them having different config and "handling" different AZ)12:26
*** jaganathan has quit IRC12:27
*** sdake has joined #openstack-ironic12:28
mat128amarao: we built user/tenant reservation using filters for now, so the node has a tenant uuid and only that tenant can boot an instance on it12:28
*** rob0t_ has quit IRC12:28
mat128amarao: you could use any custom attribute and do the same thing12:28
mat128based on the requested availability zone12:28
*** sdake_ has quit IRC12:30
amaraomat128, I didn't get it. We got different types (e.g. mem/cpu/disk) servers in two AZ. We want users have a way to say which AZ they want to use. How it will works with weights?12:30
*** rcernin has quit IRC12:31
mat128amarao: you should use filters12:31
mat128amarao: are there servers of the same hardware in both AZ?12:32
amaraoYep. That's the problem.12:32
mat128amarao: then servers could have an extra attribute in Ironic, and a custom filter would read that and the requested AZ and filter on it12:32
amaraoAnd how user can pass this attribute at 'nova boot' time?12:33
mat128amarao: they already pass an --az attribute, no?12:33
mat128amarao: filter_properties has a request_spec12:34
mat128which probably contains it12:34
amaraoYes, they do. I don't know if it counted as 'attribute' or not.12:34
mat128amarao: the az is readable because the availability zone filter uses it12:34
mat128https://github.com/openstack/nova/blob/master/nova/scheduler/filters/availability_zone_filter.py12:34
mat128amarao: so it's just a matter of matching this to a "node" spec12:34
amaraoOh, I see the idea. Throw away AggregateFilter and pass az info directly into other filters.12:34
mat128or keep both, but yeah12:35
mat128amarao: host_state.stats contains the node information12:35
mat128amarao: we essentially do host_state.stats.get('reserved_for_tenant_id') and set that in the node properties12:35
amaraoWow. That's a really clever idea.12:36
mat128amarao: and we have this in our nova driver to expose more properties:12:37
mat128http://paste.openstack.org/show/597547/12:37
openstackgerritMerged openstack/ironic master: Validate port info before assume we may use it  https://review.openstack.org/41588112:37
mat128:)12:37
amaraomat128, thanks!12:39
openstackgerritMerged openstack/ironic master: Speed up irmc power unit tests  https://review.openstack.org/42848012:39
openstackgerritMarkos Chandras proposed openstack/bifrost master: roles: bifrost-create-vm-nodes: Randomize VM XML file  https://review.openstack.org/42871612:44
*** fellypefca has quit IRC12:52
*** rob0t_ has joined #openstack-ironic12:52
vdrokjroll: so it seems I can not get the problem with aggregates and hash ring, the scheduler will select a node in aggregate, both 'host' and 'nodename' are in the dictionary scheduler returns, and it sends a build request to that compute. Or is the problem that the ring can rebalance and node will move to different host?12:52
*** athomas has quit IRC12:52
vdrokmaybe I just don't understand nova enough tho :)12:52
*** athomas has joined #openstack-ironic12:52
*** rob0t_ has quit IRC12:57
*** jvgrant has quit IRC12:58
*** jvgrant_ has joined #openstack-ironic12:58
*** lucas-hungry is now known as lucasagomes12:59
sambettsvdrok: I see the problem as this: because the nodes are assigned to nove-computes using the hash ring, you can't ever know before hand where a node will be assigned, so you can't create an host aggregate that makes sense just based on nova-compute12:59
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Find a node by multiple attributes  https://review.openstack.org/42128513:00
vdroksambetts: yup, I just mean that if you need an aggregate==room, it does not matter if the compute is the same or not? it is not the strict node affinity in nova's sense tho13:01
*** milan has joined #openstack-ironic13:01
*** ccamacho is now known as ccamacho|lunch13:02
milanmorning Ironic! :)13:02
vdroklike I want a node in room1, not in room2, I do nova boot --availability-zone room113:02
vdrokmorning milan13:02
milanmorning vdrok! :)13:02
*** jaganathan has joined #openstack-ironic13:02
sambettsvdrok: right, so say AZ==room affinity, aggregates could be used for rack affinity for example, e.g. have a nova-compute per rack that manages only the nodes in that rack13:03
*** rob0t_ has joined #openstack-ironic13:04
vdroksambetts: ah, gotcha13:05
vdrokso we need to change that bit of hash ring only when we need nova to manage a specific bit of nodes13:05
sambettssomething like that yeah, you need a nova-compute to defined group of Ironic nodes mapping (maybe this could be a use for tags)13:08
*** rob0t_ has quit IRC13:09
sambettsvdrok: :O my brain has just clicked!13:09
vdrok:D13:09
sambettswe could add a configuration option that takes a list of tags to the nova ironic driver, that means that when it does the node-list it filters by list of tags!13:10
sambettsthen you can say this nova-compute only manages node with a tag of "rack11" or something13:10
vdroksambetts: yeah, the only problems is we don't have tags yet :)13:11
sambetts:/ yeah... sigh...13:11
vdrokwe did something like that with chassis_uuid tho13:11
vdrokassigning a specific chassis to a compute13:11
mat128sambetts: I wouldnt want to have to deploy a nova-compute per rack, there would be a ton of them13:11
*** aparnav has quit IRC13:12
*** rob0t_ has joined #openstack-ironic13:12
sambettsmat128: tis but an example ;) I guess it depends on how HA you really want to get :-P13:13
*** jaganathan has quit IRC13:13
mat128sambetts: yeah I figured :)13:14
sambettsvdrok: with tags instead of chassis id though you could have nova take a list of tags, so it could be like node-tags=rack11,rack12,rack13 and then it filter nodes with those tags13:15
sambettsor node-tags=row5,high-storage13:15
sambettshmmmm /me loves tags/labels13:16
*** rob0t_ has quit IRC13:16
*** vsaienko has quit IRC13:19
vdrokyup, that's an advantage13:19
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient master: Do not show chassis_uuid field when it is not specified  https://review.openstack.org/42781513:20
*** bfournie has quit IRC13:20
*** rob0t_ has joined #openstack-ironic13:22
*** dprince has joined #openstack-ironic13:23
*** jheroux has joined #openstack-ironic13:26
*** rob0t_ has quit IRC13:26
*** g3ek has quit IRC13:28
*** jaganathan has joined #openstack-ironic13:29
*** awiddersheim has quit IRC13:29
*** awiddersheim has joined #openstack-ironic13:30
*** crushil has quit IRC13:32
jrollsambetts: oh, that's interesting13:32
*** trown|outtypewww is now known as trown13:32
*** rob0t_ has joined #openstack-ironic13:34
jrollvdrok: the aggregate thing might actually work, idk enough about aggregates. I'm worried aggregates only care about "hosts", not "nodes", and so you might send a build request for a node to a host that doesn't know about it (which might work?). maybe it's handled in the scheduling decision, though, not sure13:34
* jroll feels like he's just throwing random words together here13:34
*** hoonetorg has quit IRC13:35
sambettshaha13:35
*** g3ek has joined #openstack-ironic13:37
*** rob0t_ has quit IRC13:39
vdrokjroll: yeah, aggregates will care about resource providers, not compute hosts anymore, I guess. it works now already with resource providers, and we still get (host, node) pair from the scheduler, so hopefully this will be happening in future too :)13:40
openstackgerritLucas Alvares Gomes proposed openstack/virtualbmc master: Fix help string about adding a VBMC using IPv4  https://review.openstack.org/42872513:41
jrollvdrok: yeah, I think we're fine eventually, I just wonder about today :)13:41
*** rob0t_ has joined #openstack-ironic13:42
*** rob0t_ has quit IRC13:47
*** hoonetorg has joined #openstack-ironic13:48
*** amotoki has joined #openstack-ironic13:48
*** vsaienko has joined #openstack-ironic13:50
openstackgerritGalyna Zholtkevych proposed openstack/python-ironicclient master: Include driver properties to osc `driver show`  https://review.openstack.org/38115313:50
*** bfournie has joined #openstack-ironic13:50
*** srobert has joined #openstack-ironic13:52
*** crushil has joined #openstack-ironic13:53
openstackgerritVasyl Saienko proposed openstack/ironic master: Add Ironic standalone tests  https://review.openstack.org/42355613:54
openstackgerritVasyl Saienko proposed openstack/ironic master: [Devstack] fix waiting resources on subnode  https://review.openstack.org/42767513:54
*** srobert has quit IRC13:55
*** srobert has joined #openstack-ironic13:55
*** rob0t_ has joined #openstack-ironic13:56
amarao mat128, I dug into scheduler and I can't find any capabilities of node (ironic node) inside 'hosts' object (hosts.next()) in _schedule function in filters. I can see meta for nova-level host, but I can't see hypervisor capabilities...13:58
openstackgerritLucas Alvares Gomes proposed openstack/virtualbmc master: Document some usage with examples  https://review.openstack.org/42873713:59
*** crushil has quit IRC14:00
*** dtantsur|bbl is now known as dtantsur14:00
*** rob0t_ has quit IRC14:01
*** rob0t_ has joined #openstack-ironic14:06
TheJuliaGood morning14:09
dtantsurmorning TheJulia, jroll, milan14:10
TheJuliagood morning dtantsur14:10
* TheJulia needs lots of coffee this morning14:10
*** rob0t_ has quit IRC14:10
* dtantsur too14:10
* jroll three14:11
*** crushil has joined #openstack-ironic14:12
*** rob0t_ has joined #openstack-ironic14:12
jrolldtantsur: I just realized neutron network_interface isn't enabled by default, is there a reason for that? vdrok ?14:15
jrollis it because extra config or?14:15
dtantsurjroll, because it fails without provisioning_network set?14:15
jrolldtantsur: right, so extra config14:16
*** baoli has joined #openstack-ironic14:16
vdrokYup, no specific reason apart from that14:16
jrollI'd like to work to make multitenancy default, I think14:16
jrollor at least enabled by default14:16
vdrokMorning TheJulia14:16
*** rob0t_ has quit IRC14:17
* jroll wants ironic to be sane, secure, prod-ready with defaults14:17
*** athomas has quit IRC14:20
*** ccamacho|lunch is now known as ccamacho14:20
TheJuliaenabled by default makes sense, I think it is only when the driver is actually loaded that it would explode without the configuration.... and if we went to supporting node level definition of provisioning networks then the check for that would actually need to be moved14:20
dtantsurjroll, I agree.. though as I mentioned some time ago, pre-defining a network in ironic.conf gives me a huge headache, given tripleo architecture..14:21
*** rob0t_ has joined #openstack-ironic14:22
*** Goneri has joined #openstack-ironic14:22
TheJuliadtantsur: would node level be better?14:23
jrolldtantsur: yeah, I would think we should fix tripleo architecture in that case14:23
jrollor we could explode late, dunno14:23
jrollor check when we set it on a node? and check if it's set on any nodes at startup?14:24
TheJulianot even check if it is set on nodes, check if it exists as part of one or more validation steps14:24
dtantsurTheJulia, much better14:25
TheJuliaerr, not even check if it is set on startup14:25
*** owalsh is now known as owalsh-brb14:25
TheJuliafor some reason my macbook is lagging really badly.  Time to close all of the web browsers14:25
dtantsurjroll, "fix tripleo architecture" is not an easy win, as you may expect :) though right now it seems like Octavia has similar requirements14:25
*** jtomasek has quit IRC14:25
jrollyeah, we could just avoid checking at startup14:25
*** mjura has quit IRC14:25
jrolldtantsur: pft how hard could it be14:25
jroll:)14:26
*** jtomasek has joined #openstack-ironic14:26
*** mjura has joined #openstack-ironic14:26
TheJuliawe need it to deploy, adding it into deploy validate() makes sense to me14:26
*** athomas has joined #openstack-ironic14:26
dtantsurTheJulia, I think I have an RFE for that, did not have time to actually code it14:26
*** rob0t_ has quit IRC14:27
TheJuliaYeah, there isa n RFE for it14:27
openstackgerritMerged openstack/bifrost master: scripts: test-bifrost.sh: Use 'unsafe' disk cache for test VMs  https://review.openstack.org/42774414:28
*** owalsh-brb has quit IRC14:29
*** rob0t_ has joined #openstack-ironic14:32
*** owalsh-brb has joined #openstack-ironic14:32
*** marlinc has quit IRC14:32
*** jcoufal has joined #openstack-ironic14:35
openstackgerritLucas Alvares Gomes proposed openstack/virtualbmc master: Fix help string about adding a VBMC using IPv4  https://review.openstack.org/42872514:36
*** rob0t_ has quit IRC14:37
jroll./ironic/conductor/base_manager.py:56:1: C901 'BaseConductorManager.init_host' is too complex (17)14:37
jroll:|14:37
dtantsurlol, I don't disagree with that14:37
jrollheh14:38
jrollyeah, we need to clean some of that up14:39
jrollI just wonder how much gets cleaned up naturally if/when we kill drivers14:39
*** marlinc has joined #openstack-ironic14:41
TheJuliapas-ha: I reviewed 427071, biggest issue that I have is the deprecations are not indicating when we're removing the old behavior14:44
lucasagomesjroll, yeah, well there's a lot of room for improvements there, we could start breaking things into small functions (register_conductor(), start_periodic_tasks(), start_consoles(), etc...)14:44
lucasagomesthis init_host just do it all and separated the logic with comments :-)14:45
jrolllucasagomes: yeah totally14:45
*** sasha21 has joined #openstack-ironic14:46
*** rob0t_ has joined #openstack-ironic14:48
*** amotoki has quit IRC14:48
*** amotoki has joined #openstack-ironic14:48
*** amotoki has quit IRC14:49
* jroll notices bcornec picked the redfish spec back up14:50
* jroll gets excited14:50
TheJuliabcornec: sorry I've not replied to the email you sent me a week or two ago.  Life has been rather.. hectic as of recent.14:51
TheJuliaIf anyone wants to review a relatively small chunk of javascript in ironic-ui https://review.openstack.org/#/c/418520/14:52
patchbotpatch 418520 - ironic-ui - Create base-port module to support create and upda...14:52
*** mtanino has joined #openstack-ironic14:52
* jroll looks14:53
TheJuliajroll: thank you14:54
openstackgerritJim Rollenhagen proposed openstack/ironic master: Improve enabled_*_interfaces config help and validation  https://review.openstack.org/42821914:54
jrollTheJulia: I totally forgot everything in horizon is using angular... /me has a sad14:54
* jroll tries to resurrect any sort of angular knowledge from the depths14:55
* TheJulia gives jroll a whisky14:55
robcresswelljroll: More of a React guy?14:55
jrollrobcresswell: yeah, I've not played with it as much as I like, but some of my friends are react people and seem to have good reasons for it14:56
*** rob0t_ has quit IRC14:56
dtantsur"review a relatively small chunk of javascript"  http://pruebakr-1.webcindario.com/nope.jpg14:56
TheJuliadtantsur: nice14:57
dtantsur:-P14:57
jrolloh I like that one14:57
TheJuliaI feel like that should be my facebook profile picture14:57
robcresswellYeah. Angular 1.x was a mess. I've found it reasonable to use at 1.5/1.6 as long as you strictly follow certain styles, but then you end up using about 1/3 of the framework :/14:57
jrollTheJulia: +A14:59
TheJuliappiela: ^^^14:59
TheJuliajroll: I'm sure ppiela thanks you :)15:00
jrollhehe15:00
jrolloh, that's the only one in the queue, nice15:00
* jroll sees y'all have been busy the last week or so15:00
*** mtanino_ has joined #openstack-ironic15:01
*** mtanino has quit IRC15:02
openstackgerritMerged openstack/ironic-ui master: Create base-port module to support create and update operations  https://review.openstack.org/41852015:02
ppielaTheJulia, jroll: I do thank you15:02
*** rob0t_ has joined #openstack-ironic15:03
jroll:)15:03
*** rob0t_ has quit IRC15:08
*** mtanino_ has quit IRC15:09
milanmorning dtantsur jroll ppiela sambetts TheJulia and all! :)15:09
jrollmorning milan :)15:10
ppielaHi milan15:10
*** pcaruana has quit IRC15:11
*** mtanino has joined #openstack-ironic15:11
robcresswellOh, question from the other day; are there any decent ways to test ironic in a standalone (noauth) way with a virtual driver/backend? I was looking for a way to test some UI work. I think Bifrost does what I want, but I'm an ironic noob so I'm unsure.15:14
robcresswellI'm aware of the fake driver, but there's only so far that will get you with testing validation etc.15:14
*** pcaruana has joined #openstack-ironic15:17
*** rob0t_ has joined #openstack-ironic15:19
*** amotoki has joined #openstack-ironic15:20
milanaarefiev_afk, hey, just read your comment on https://review.openstack.org/#/c/421285/ there was yet another issue: had to specify column type for the batch_op.alter_column for MySQL15:21
patchbotpatch 421285 - ironic-inspector - Find a node by multiple attributes15:21
aarefiev_afkmilan: this issue is easy to fix :)15:21
milanaarefiev_afk, the primary key should be OK dropping during the batch_op, now the issue seems http://logs.openstack.org/85/421285/12/check/gate-grenade-dsvm-ironic-inspector-ubuntu-xenial/e4bc510/console.html#_2017-02-03_14_28_07_73859615:22
milanaarefiev_afk, yeah, I've posted an update15:22
milanI think I might have it corrected15:22
*** chlong has joined #openstack-ironic15:24
TheJuliagood morning milan15:24
milano/ :)15:24
*** Nisha_Agarwal has joined #openstack-ironic15:25
aarefiev_afkmilan: hmm, btw did you try to run tests with mysql15:25
milanaarefiev_afk, I did in devstack15:26
*** rob0t_ has quit IRC15:26
milanaarefiev_afk, I tried re-running the migrations couple of times till I figured the stack trace ;)15:26
*** amotoki has quit IRC15:26
milan* traceback15:26
milanas the alembic is consuming it happily ;)15:27
milanaarefiev_afk, wdyt about this revision?15:27
* milan not sure what happened to the grenade test though15:27
aarefiev_afkmilan: reading grenade logs15:27
*** owalsh-brb is now known as owalsh15:28
*** jcoufal_ has joined #openstack-ironic15:34
*** rajinir has joined #openstack-ironic15:36
*** jcoufal has quit IRC15:36
*** rob0t_ has joined #openstack-ironic15:37
*** jaosorior has joined #openstack-ironic15:40
openstackgerritVasyl Saienko proposed openstack/ironic master: Add Ironic standalone tests  https://review.openstack.org/42355615:41
sambettsTheJulia: can you help robcresswell? He wants to use bifrost with VMs but I'm not sure how that works15:41
TheJuliao/ robcresswell15:41
robcresswellNot a big problem if its not an option, I'm just curious :)15:41
robcresswell\o15:42
TheJuliaOh, it is!15:42
*** rob0t_ has quit IRC15:42
milanaarefiev_afk, IMO the inspector smoke test was OK (i can see the node returning back to avail in ir-cond)15:42
*** vsaienko has quit IRC15:42
TheJuliathey would be local libvirt instances, if that works for you?15:42
milanhttp://logs.openstack.org/85/421285/12/check/gate-grenade-dsvm-ironic-inspector-ubuntu-xenial/e4bc510/console.html#_2017-02-03_14_25_01_656242 has some sort of a timeout :-/15:42
aarefiev_afkmilan: I don't see any issues, really timeout? :)15:42
milanaarefiev_afk, 2017-02-03 14:28:07.738541 |     tempest.exceptions.BuildErrorException: Server e5d55745-1eab-46ce-a834-99628cf79a29 failed to build and is in ERROR status15:43
milanaarefiev_afk, http://logs.openstack.org/85/421285/12/check/gate-grenade-dsvm-ironic-inspector-ubuntu-xenial/e4bc510/console.html#_2017-02-03_14_28_07_73854115:43
milanprovisioning failed, I dunno15:44
milanrecheck?15:44
aarefiev_afkmilan: yeah timeout while wait for callback and I don't see errors in bm logs15:44
aarefiev_afkyeah lets try15:44
milancool15:44
robcresswellTheJulia: I'm not picky. I'm really just looking for something to bounce API requests off without the overheads of actual baremetal, like ten minute power cycles :)15:45
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient master: Do not show chassis_uuid field when it is not specified  https://review.openstack.org/42781515:45
robcresswellThis is all pretty new to me, so I'm kinda wandering around the dark trying to get it to work15:46
milansambetts, hey, do you plan to review https://review.openstack.org/#/c/375045/ ? ;)15:47
patchbotpatch 375045 - ironic-inspector-specs - Endpoint to list introspection statuses15:47
TheJuliarobcresswell: Bifrost has a script in it's repo that can be used to create a set of virtual machines. The output of which is a CSV file which you can use to deploy an image to, although I'm not really sure what of what API your wanting to hit :)15:47
milansambetts, and also: https://review.openstack.org/#/c/396147/ :D15:48
patchbotpatch 396147 - ironic-inspector-specs - Spec for retrieving NUMA node information15:48
robcresswellTheJulia: Which script is that?15:49
robcresswellTheJulia: I'm just looking to bounce a standalone UI off the ironic APIs, but setting up VMs instead of actual metal for speed of testing15:49
TheJuliarobcresswell:  https://github.com/openstack/bifrost/blob/master/tools/create_vm_nodes.sh15:50
robcresswellhey neat15:51
robcresswellTheJulia: Thanks!15:51
openstackgerritGalyna Zholtkevych proposed openstack/python-ironicclient master: Include driver properties to osc `driver show`  https://review.openstack.org/38115315:51
*** rob0t_ has joined #openstack-ironic15:52
*** Syed__ has joined #openstack-ironic15:56
*** rob0t_ has quit IRC15:57
sambettsmilan: I've not reviewed the NUMA one before, and I don't really have time to sit down and go through the spec right now, I want to re-read the list one though as I've been involved with that :)15:59
*** rpioso has joined #openstack-ironic15:59
milansambetts, OK cool! Thanks for info!16:00
*** jcoufal_ has quit IRC16:03
*** jcoufal has joined #openstack-ironic16:03
*** Nisha_Agarwal has quit IRC16:06
NobodyCamGood Morning Ironic'ers ... and ...16:06
*** crushil has quit IRC16:06
*** rob0t_ has joined #openstack-ironic16:06
NobodyCamTGIF!16:06
*** crushil has joined #openstack-ironic16:06
*** jcoufal_ has joined #openstack-ironic16:07
*** aparnav has joined #openstack-ironic16:07
*** sdake_ has joined #openstack-ironic16:08
*** fxpester has quit IRC16:08
*** sdake has quit IRC16:08
*** sdake has joined #openstack-ironic16:09
*** jcoufal has quit IRC16:09
*** ccamacho has quit IRC16:11
*** jcoufal has joined #openstack-ironic16:11
*** rob0t_ has quit IRC16:11
*** sdake has quit IRC16:12
*** sdake_ has quit IRC16:12
*** ccamacho has joined #openstack-ironic16:13
*** jcoufal_ has quit IRC16:13
*** baoli has quit IRC16:13
dtantsurmorning NobodyCam16:16
dtantsuroh TGIF indeed16:16
*** [1]cdearborn has joined #openstack-ironic16:17
*** jaganathan has quit IRC16:17
lucasagomesNobodyCam, o/ morning16:19
*** rob0t_ has joined #openstack-ironic16:19
*** baoli has joined #openstack-ironic16:19
milanmorning NobodyCam! TGIF ++ :D16:19
*** dansmith is now known as superdan16:21
*** rbudden has joined #openstack-ironic16:21
* dtantsur has a day full of puppet..16:21
NobodyCamGood Morning dtantsur, lucasagomes, and milan :)16:21
* NobodyCam was getting coffee16:22
dtantsurmilan, bfournie, also FYI we broken RDO a bit with our LLDP patch, as they don't have the required 'construct' package even in Fedora :)16:22
bfourniedtantsur: uugh16:23
vdrokmorning NobodyCam :)16:24
*** mjura has quit IRC16:24
NobodyCamMorning vdrok :)16:24
bfourniedtantsur: how can we bring that construct lib patch down, it went in a month ago16:24
dtantsurwell, we're the first one to actually use it :)16:25
bfourniedtantsur: right16:25
mat128amarao: you probably need something like this in your virt driver - http://paste.openstack.org/show/597547/16:25
*** nmathew has quit IRC16:25
amaraomat128, this is patch to nova?16:26
*** rob0t_ has quit IRC16:26
mat128thats a function in virt driver, the paste I sent you is just us overriding the method, but you could easily patch it in16:26
mat128amarao: let me craft you a patch :)16:27
mat128which version are you running?16:27
amaraoWell, I tried to find capabilities at scheduler level, they are in hosts.next().stats .stats[]16:27
mat128amarao: oh, good then16:27
amaraoI will try to make a custom filter. I think it's better than patch to nova.16:27
amaraoIf I fail, I will try with nova patch. Thanks for offer.16:28
mat128amarao: indeed, thats good16:28
mat128amarao: no problem, I'm always idling here so feel free to ask16:28
amaraoRight now it looks like this: ironic node-update  acafa4ea-23ac-49f6-9368-3a97a3d09bf8 add properties/capabilities='availability-zone:az2'16:28
amaraoAnd in scheduler context it is hosts[N].stats['u'availability-zone'] == u'az2'16:29
amaraoI have no idea why it's 'stats'...16:29
mat128amarao: in my case capabilities are comma separated for some reason16:30
mat128I dont remember where this comes from, but I think nova unpacks it that way16:30
amaraomat128, hmm... I see it upacked them to dict.16:30
*** cdearborn has quit IRC16:31
mat128amarao: a random test machine dump - http://paste.openstack.org/show/597590/16:31
amaraoOh, you are keeping them not in capabilities, but in properties itself... I will try this too.16:32
mat128amarao: oh, yes16:33
*** bnemec is now known as beekneemech16:33
mat128and I think this is why we have that other thing - http://paste.openstack.org/show/597547/16:33
mat128we end up having to filter on many criterias, and putting them all in capabilities is a pita16:33
amarao... (bad suspicion) Is capablities updated by some software?16:33
mat128amarao: nothing in openstack, besides ironic-inspector possibly16:33
*** mjura has joined #openstack-ironic16:33
mat128amarao: on our side, we have custom code to export "nodes" data from a management system into Ironic16:34
amaraoOh, so inspector may override them.16:34
mat128amarao: not 100% sure, but that's where it would put hardware capabilities16:34
*** baoli has quit IRC16:34
mat128amarao: it'd be nice if inspector would derive the availability zone based on some other variable16:34
amaraomat128, ok, my first attempt will be to put AZ information somewhere else.16:34
*** rob0t_ has joined #openstack-ironic16:35
mat128amarao: maybe from kernel cmdline? then you could have 1 PXE server per AZ16:35
mat128amarao: going into a meeting, feel free to ping and I'll check later16:35
*** mat128 is now known as mat128|mtg16:35
amaraomat128: my current design will not use inspector. Just to make thinks less 'software-dependent'.16:35
amaraomat128, thank you very much!16:35
*** chlong has quit IRC16:36
milandtantsur, bfournie, fun type 2 :)16:36
*** ianychoi has joined #openstack-ironic16:37
milanalso Fedora: Freedom. Friends. Features. First. ;)16:37
*** baoli has joined #openstack-ironic16:38
ianychoiHello. I am from I18n team. Can I know when "ironic-ui" will have stable/ocata branch? It is important for I18n team to create a stable version for translation sync jobs with other Horizon related projects16:38
dtantsursambetts, quick question re your hardware. you said you can't detect ipmi address during introspection from within the node itself, right? does it mean that IPMI access to it is not possible at all or only from within the machine?16:39
*** rob0t_ has quit IRC16:39
jrollianychoi: I was thinking next week, but maybe we're ready now. TheJulia what do you think?16:41
jrollI'm not opposed to branching now16:41
dtantsurbranch early, branch often16:42
*** rob0t_ has joined #openstack-ironic16:42
jrollpls don't branch often16:42
jroll:P16:42
*** xhku is now known as fbouliane16:43
*** aparna has joined #openstack-ironic16:43
JayFfor i in {1..100}; do git checkout -b branch$i; done16:44
ianychoiI18n team now deals with total 12 horizon & related repos including ironic-ui. According to https://releases.openstack.org/ocata/schedule.html , this week is for the deadline of RC116:44
jrollianychoi: ironic follows the cycle-with-intermediary model, so our deadlines are a bit different16:45
dtantsurjroll, this is a good call, we may want to branch ironic-ui with the remaining openstack to reduce headache for the i18n team16:45
*** appu103 has joined #openstack-ironic16:45
ianychoiCurrently 5 repos do not have stable/ocata branch, and then now I want to have more conversation with details16:45
jrollianychoi: I didn't realize releasing later would affect translation work, I'm happy to branch it, just want to check with TheJulia first16:45
*** aparnav has quit IRC16:45
jrolldtantsur: agree16:45
dtantsurjust like we do with the CLI, which is also a UI :)16:45
ianychoijroll, aha I see Actually I admit that I am not familiar with cycle-with-intermediary model too much16:46
*** fbouliane has quit IRC16:47
*** rob0t_ has quit IRC16:47
jrollianychoi: looking for details, but I think we should change it for ironic-ui :)16:47
ianychoijroll, does the model apply to all horizon plugin projects?16:47
*** aparna has quit IRC16:48
*** fbouliane has joined #openstack-ironic16:48
jrollianychoi: I don't think so, and I think we need a rule to make horizon plugins cycle-with-milestones always16:48
jrollianychoi: see this page for description of the models https://releases.openstack.org/reference/release_models.html16:48
dtantsurjroll, I think changing the model is too much, just setting up a deadline like with CLI is enough16:48
dtantsurwith the model, we'll have to make M1, M2, M3, RC1..16:49
jrolldtantsur: ah yeah, good point16:49
robcresswellcycle-with-intermediary should have tagged a stable/ocata by now, or should do today I believe16:49
ianychoijroll, aha I see thanks a lot for the link oh hello robcresswell16:49
jrollrobcresswell: we have two weeks yet, -intermediary final release must happen the same time as final RCs16:49
jrollhttps://releases.openstack.org/ocata/schedule.html16:49
jrollianychoi: anyway, I will double check with TheJulia and probably make the stable/ocata branch today or monday. thanks for bringing this up :)16:50
robcresswelljroll: RC1 should be when you branch stable/ocata; everything past that should be proposed as a backport.16:50
jrollrobcresswell: -intermediary doesn't do RCs16:50
robcresswellOh, interesting16:50
robcresswellWell, I'll shut up then :)16:51
jrollheh16:51
JayFrobcresswell: basically it's "just another release" for us, except we create the stable/ branch and it's capable for backporting16:51
*** chlong has joined #openstack-ironic16:52
*** rob0t_ has joined #openstack-ironic16:52
ianychoijroll and robcresswell : Thanks. Previously, I18n created stable versions for all the targeted 12 horizon & related projects just after Hard StringFreezes but maybe I18n team needs to be more aware of RC versionings with intermediary cycle model16:52
*** mtanino has quit IRC16:53
*** mtanino has joined #openstack-ironic16:53
*** mtanino has quit IRC16:53
*** mtanino has joined #openstack-ironic16:53
ianychoijroll, then what would be the final release date for ironic-ui during Ocata cycle (not exactly just estimation would be also fine)16:54
ianychoi?16:54
*** rloo has joined #openstack-ironic16:54
*** d0ugal is now known as d0ugal|skiing16:54
jrollianychoi: the plan was for next thursday, but I can do it earlier if TheJulia and ppiela are okay with it16:55
*** rloo has quit IRC16:55
ianychoijroll, aha okay thanks! It would be much nice if there will be a stable/ocata branch yep please tell me :)16:56
ianychoiThanks :)16:56
TheJuliajroll: I'm fine with branching it now since ppiela indicated to me a few days ago that the rest of the port work would not be ready before the end of next week16:56
robcresswellJayF: Yeah, I just didn't realise it didnt follow the usual RC period. I just assumed intermediary releases did that anyway.16:56
TheJuliaAlso, sorry for the delay, was in my xar16:56
TheJuliaCar16:56
jrollTheJulia: no worries on delay, I'll go ahead and do that thing. thanks!16:56
ianychoiTheJulia, oh fine. Texting in driving is so unsafe :)16:57
*** rob0t_ has quit IRC16:57
jlvillalvsaienk0, pas-ha vdrok: On networking- generic-switch. What do you think about just putting the config values into /etc/neutron/neutron.conf ?16:59
jlvillalThis is due to the grenade issues we are running into with multi-tenant17:01
jlvillalvgadiraj, ^^17:01
pas-hadid not though about that yet, as we were just following examples of other ML2 plugins? bu we could try17:01
jlvillalpas-ha, Yeah. Having a hard time getting the config file to be used in the 'new' phase of grenade.17:01
*** rob0t_ has joined #openstack-ironic17:02
jlvillalI'm going to test it to see if it helps with the grenade issue I am having17:02
pas-hait does not want to start q-svc with an extra config file?17:02
ppielaTheJulia, jroll: I plan on having the patchsets for the remaining port work uploaded for review today.17:02
jlvillalpas-ha, No. Not for the 'new' phase. The 'old' phase is fine.17:02
jrollppiela: oh, do you want to wait for next week to release ocata, then, or is now okay?17:03
*** yolanda has quit IRC17:03
*** athomas has quit IRC17:04
ppielajroll: I would prefer to wait, can we revisit on Monday?17:05
jrollppiela: sure, I'll put up a WIP patch for now17:05
dtantsurppiela, jroll, usually releases don't happen on Friday anyway17:05
vdrokmorning jlvillal , I don't think we have a specific reason for putting it where we do now. I see the same structure folder used in networking-cisco, but in networking-arista it's just /etc/neutron. So can be changed I guess. but not sure if just changing folders will help. worth trying :)17:05
ppielajroll: thanks17:05
jlvillalvdrok, Thanks. Yeah I will give it a test17:06
*** yolanda has joined #openstack-ironic17:06
ianychoiOne more question: where can I see "cycle-with-intermediary" tag for ironic-ui repository? I cannot see such tag on http://git.openstack.org/cgit/openstack/governance/tree/reference/projects.yaml ..17:06
dtantsurianychoi, I think they moved tags to releases repo17:07
*** ipukha has joined #openstack-ironic17:07
ianychoidtantsur, okay I will find in release repo, thanks!17:07
*** rob0t_ has quit IRC17:07
sambettsdtantsur: we can use IPMI to control the machine if we want, but the IPMI server is like a bolt on API on to our own BMC, so because of that it doesn't seem to advertise its address to the kernel17:08
dtantsuroh, I see17:09
dtantsurthanks17:09
*** rob0t_ has joined #openstack-ironic17:12
*** amarao has quit IRC17:13
*** mjura has quit IRC17:14
*** g3ek has quit IRC17:15
openstackgerritMerged openstack/ironic-inspector-specs master: Spec for retrieving NUMA node information  https://review.openstack.org/39614717:16
*** chlong has quit IRC17:16
*** rob0t_ has quit IRC17:17
* sambetts -> heading off17:20
sambettshave a great weekend everyone :D17:20
sambettso/17:20
*** sambetts is now known as sambetts|afk17:20
*** rob0t_ has joined #openstack-ironic17:22
dtantsurgoing as well, have a nice weekend17:24
*** dtantsur is now known as dtantsur|afk17:25
*** g3ek has joined #openstack-ironic17:26
*** aparnav has joined #openstack-ironic17:26
*** rob0t_ has quit IRC17:27
*** appu103 has quit IRC17:28
*** aparna has joined #openstack-ironic17:29
*** cdearborn has joined #openstack-ironic17:30
*** derekh has quit IRC17:30
*** jkilpatr_ has joined #openstack-ironic17:31
*** appu103 has joined #openstack-ironic17:31
*** aNuposic has joined #openstack-ironic17:32
*** aparnav has quit IRC17:32
*** jkilpatr has quit IRC17:32
*** rob0t_ has joined #openstack-ironic17:34
*** aparna has quit IRC17:34
*** trown is now known as trown|lunch17:35
NobodyCamhave a good weekend dtantsur|afk sambetts|afk17:36
*** lucasagomes is now known as lucas-afk17:36
NobodyCamyou too lucas-afk :)17:36
NobodyCam:p17:36
lucas-afko/17:36
lucas-afksee ya, enjoy the weekend NobodyCam17:37
lucas-afkand everyone else :-)17:37
NobodyCam:)17:37
*** amarao has joined #openstack-ironic17:37
*** rob0t_ has quit IRC17:39
*** rama_y has joined #openstack-ironic17:41
*** [1]cdearborn has quit IRC17:41
*** vsaienko has joined #openstack-ironic17:41
*** ccamacho has quit IRC17:43
*** amarao has quit IRC17:48
*** rob0t_ has joined #openstack-ironic17:49
*** dprince has quit IRC17:50
*** mgoddard_ has quit IRC17:52
*** EmilienM has quit IRC17:56
*** tonyb has quit IRC17:56
*** mikal has quit IRC17:56
*** Nakato has quit IRC17:56
*** patchbot has quit IRC17:56
*** hurricanerix has quit IRC17:56
*** alex_xu has quit IRC17:56
*** mikal has joined #openstack-ironic17:56
*** tonyb has joined #openstack-ironic17:56
*** devananda has quit IRC17:56
*** Madasi has quit IRC17:56
*** EmilienM has joined #openstack-ironic17:56
*** rob0t_ has quit IRC17:56
*** cinerama has quit IRC17:56
*** sirushti has quit IRC17:56
*** zer0c00l has quit IRC17:56
*** hurricanerix has joined #openstack-ironic17:56
*** zer0c00l has joined #openstack-ironic17:56
*** alex_xu has joined #openstack-ironic17:56
*** devananda has joined #openstack-ironic17:56
*** cinerama has joined #openstack-ironic17:56
*** Nakato has joined #openstack-ironic17:57
*** sirushti has joined #openstack-ironic17:57
*** patchbot has joined #openstack-ironic17:58
*** Madasi has joined #openstack-ironic17:58
jlvillalpas-ha, vdrok vsaienk0: FYI, supposedly MultiConfigParser is deprecated and could be removed in Ocata (though I don't think it will be).  https://github.com/openstack/networking-generic-switch/blob/master/networking_generic_switch/config.py#L2018:00
*** ralonsoh has quit IRC18:02
*** rob0t_ has joined #openstack-ironic18:06
*** alex_xu has quit IRC18:06
*** alex_xu has joined #openstack-ironic18:08
mariojvis anyone doing the ironicclient work to update the node endpoints to display new driver interfaces?18:09
mariojvi know the driver endpoints have work underway, didn't see node work18:09
*** rob0t_ has quit IRC18:10
*** ipukha has quit IRC18:11
*** aparnav has joined #openstack-ironic18:12
vdrokmariojv: like this one? https://review.openstack.org/41835918:14
patchbotpatch 418359 - python-ironicclient - Add args to CLI 'node-create' for selecting hardwa... (MERGED)18:14
vdroknot sure what you mean by node endpoints18:15
*** appu103 has quit IRC18:15
vdrokas in case of nodes, they are just fields18:15
TheJuliaOkay, I'm back in front of a keyboard18:15
TheJuliaWhat a frustrating morning18:16
*** sabeen has joined #openstack-ironic18:16
vdrokjlvillal: ok, we'll look into that18:16
NobodyCam:(18:16
TheJuliaianychoi: I was parked when I was typing into irc ;)18:16
*** aparnav has quit IRC18:17
ianychoiTheJulia, it seems that you have wonderful driving + parking skills!18:17
TheJulialol!18:17
vdrokmariojv: but still, seems like OSC set and unset for these new driver fields are still not done18:17
TheJuliappiela: w/r/t to the port work, awesome! :)18:18
*** pc_m has quit IRC18:18
* vdrok leaves for the weekend18:19
*** rob0t_ has joined #openstack-ironic18:19
vdrokgood night everyone!18:19
TheJuliagoodnight vdrok18:19
openstackgerritAnup Navare proposed openstack/ironic master: Allow logical name along with UUID in port creation  https://review.openstack.org/40558618:19
jlvillalvdrok, Good night, have a great weekend!18:19
ianychoiBTW thanks a lot for today conversation. Now I am also starting communications with magnum-ui and murano-dashboard which are also cycle-with-intermediary but now here is with most active conversation :)18:19
*** dprince has joined #openstack-ironic18:20
ianychoiI think on early next week I will start to support translation sync job including ironic-ui :)18:20
TheJuliaianychoi: I thought translations were already syncing out to ironic-ui18:21
aNuposicgoodnight vdrok18:21
ianychoiTheJulia, yep that's right but it is just for master branch. I18n team targets total 12 horizon & plugin projects to include translated strings on the stable branch as much as possible and I18n PTL needs to create a stable version and translation sync job for stable branches at the same time.18:22
*** pc_m has joined #openstack-ironic18:22
mariojvvdrok: yeah that was it. have a good weekend!18:23
ianychoiSince cycle-with-intermediary model might be dealt with, I think I need to discuss it with release team during PTG18:23
mariojvi think that *might* make node show show the right thing, too18:23
TheJuliaianychoi: Interesting, because I know, at least for the last release of ironic-ui, we cut the stable branch late and all of a sudden revised translations were showing up for the stable branch.18:26
*** rob0t_ has quit IRC18:26
ianychoiTheJulia, the main problem is that master branch strings might have differences with a stable branch for a few projects and also translators need to have reviewing period so I18n team has moved to translation on stable branches after Hard StringFreeze18:30
ianychoihttp://docs.openstack.org/developer/i18n/release_management.html => Hope that it helps. Now I am seeing number 5 but now I realize that the document does not cover cycle-with-intermediary model..18:31
*** vsaienko has quit IRC18:33
*** jcoufal_ has joined #openstack-ironic18:35
*** rob0t_ has joined #openstack-ironic18:36
*** tesseract has quit IRC18:38
*** jcoufal has quit IRC18:38
*** chlong has joined #openstack-ironic18:38
*** rob0t_ has quit IRC18:41
*** Goneri has quit IRC18:43
*** rob0t_ has joined #openstack-ironic18:44
*** mgoddard_ has joined #openstack-ironic18:45
*** Sukhdev has joined #openstack-ironic18:45
TheJuliaianychoi: Hmm... I would be interested to be in this discussion at the PTG because naturally we don't want to overly burden anyone, and truthfully I want to understand the process/perspective from the i18n side.18:46
*** trown|lunch is now known as trown18:48
*** rob0t_ has quit IRC18:49
*** rob0t_ has joined #openstack-ironic18:52
openstackgerritJim Rollenhagen proposed openstack/ironic master: Improve enabled_*_interfaces config help and validation  https://review.openstack.org/42821918:54
*** sdake has joined #openstack-ironic18:56
*** cdaniel is now known as cdaniel-afk18:57
*** rob0t_ has quit IRC18:57
openstackgerritJulia Kreger proposed openstack/bifrost master: Remove my_ip_address from use  https://review.openstack.org/42897518:59
NobodyCamTheJulia: oh really. I wouldn't have thought that was it19:00
*** rob0t_ has joined #openstack-ironic19:02
TheJuliaNobodyCam: defaults are calculated out upon role load, if something is still in-flight then.... I bet the exception is hiding under the covers until the variable is acessed19:03
TheJuliaat least, that is my theory19:03
NobodyCamahh :)19:03
*** rob0t_ has quit IRC19:06
* NobodyCam watches the tests :p19:07
ianychoiTheJulia, yep I don't want your team with more burden. Thanks a lot for such consideration!19:09
*** chlong has quit IRC19:11
*** rob0t_ has joined #openstack-ironic19:12
*** jidar has joined #openstack-ironic19:12
*** crushil has quit IRC19:15
*** rob0t_ has quit IRC19:17
TheJuliaNobodyCam: didn't work :\19:17
*** jkilpatr_ has quit IRC19:18
*** ChubYann has joined #openstack-ironic19:19
*** jkilpatr has joined #openstack-ironic19:20
*** chlong has joined #openstack-ironic19:21
openstackgerritAndreas Jaeger proposed openstack/ironic master: Prepare for using standard python tests  https://review.openstack.org/42899419:21
*** rob0t_ has joined #openstack-ironic19:22
openstackgerritAndreas Jaeger proposed openstack/ironic stable/newton: Prepare for using standard python tests  https://review.openstack.org/42899819:22
*** amotoki has joined #openstack-ironic19:23
*** vsaienko has joined #openstack-ironic19:23
*** milan has quit IRC19:23
openstackgerritMerged openstack/python-ironicclient master: Do not show chassis_uuid field when it is not specified  https://review.openstack.org/42781519:24
jrollif anyone minds a review, 19:23:32          AJaeger | dtantsur|afk, jroll: Could you review https://review.openstack.org/428994 , please? This is a nop until we merge 428995 but I'd like to have it in before ocata branch19:24
patchbotpatch 428994 - ironic - Prepare for using standard python tests19:24
jrollwould help out infra19:24
jrollthanks TheJulia19:25
TheJulia:)19:25
*** vsaienko has quit IRC19:26
TheJuliaI'm a little confused why we're backporting it to stable/newton as well19:26
jrollTheJulia: job definitions are mostly branchless19:27
TheJuliaahh19:27
TheJuliayeah, that they are19:27
*** rob0t_ has quit IRC19:27
*** chlong has quit IRC19:27
jrollwe could run -db for older ones but no reason not to backport this and keep project-config "simple"19:27
openstackgerritAndreas Jaeger proposed openstack/ironic stable/mitaka: Prepare for using standard python tests  https://review.openstack.org/42900019:27
*** amotoki has quit IRC19:28
TheJulia2017-02-03 19:27:33.429543 | [E] E003: Indent not multiple of 4: '   SELECT 'HERE' from pg_roles where rolname='$DB_USER'")'19:28
TheJulia2017-02-03 19:27:33.429570 |  - /home/jenkins/workspace/gate-ironic-pep8-ubuntu-xenial/tools/test-setup.sh : L4219:28
jrollya19:28
jrollI'll take care of it19:29
TheJulia10-419:29
* jroll will forget about it otherwise19:30
openstackgerritJim Rollenhagen proposed openstack/ironic master: Prepare for using standard python tests  https://review.openstack.org/42899419:30
*** vsaienko has joined #openstack-ironic19:33
jrollhuh, how did that pass on newton19:34
*** e0ne has joined #openstack-ironic19:35
openstackgerritAndreas Jaeger proposed openstack/ironic-inspector master: Prepare for using standard python tests  https://review.openstack.org/42900419:36
*** vsaienko has quit IRC19:36
openstackgerritAndreas Jaeger proposed openstack/ironic-inspector stable/newton: Prepare for using standard python tests  https://review.openstack.org/42900619:36
openstackgerritAndreas Jaeger proposed openstack/ironic-inspector stable/mitaka: Prepare for using standard python tests  https://review.openstack.org/42900719:37
*** jcoufal has joined #openstack-ironic19:37
*** jcoufal_ has quit IRC19:37
openstackgerritJulia Kreger proposed openstack/bifrost master: Remove my_ip_address from use  https://review.openstack.org/42897519:38
*** rob0t_ has joined #openstack-ironic19:39
openstackgerritMerged openstack/python-ironicclient master: Fix cleanup of resources in OSC plugin functional tests  https://review.openstack.org/42452819:39
TheJuliajroll: bashate changes? :)19:39
openstackgerritMerged openstack/python-ironicclient master: Fix cleanup of resources in functional tests  https://review.openstack.org/42454519:39
*** vsaienko has joined #openstack-ironic19:39
TheJuliaNobodyCam: ^^^ moved the setup to later, lets see if that changes the behavior in ci testing19:40
NobodyCam:)19:40
*** chlong has joined #openstack-ironic19:41
jrollTheJulia: I can only assume so19:43
*** rob0t_ has quit IRC19:43
openstackgerritJim Rollenhagen proposed openstack/ironic stable/newton: Ensures that OneView nodes are free for use by Ironic  https://review.openstack.org/42900919:44
openstackgerritMerged openstack/python-ironicclient master: Add negative test-cases for openstack node create command  https://review.openstack.org/38194319:45
openstackgerritJim Rollenhagen proposed openstack/ironic stable/newton: Prepare for using standard python tests  https://review.openstack.org/42899819:49
openstackgerritPeter Piela proposed openstack/ironic-ui master: Fix node validation problem  https://review.openstack.org/42901219:50
openstackgerritJim Rollenhagen proposed openstack/ironic stable/mitaka: Prepare for using standard python tests  https://review.openstack.org/42900019:51
jrollJayF: if you wanna pick those stable patches up when you have a few, that'd be awesome19:52
JayFjroll: I'm still -2 on that oneview one19:52
JayFjroll: even with your repost19:52
*** jcoufal has quit IRC19:53
jrollJayF: that's a different oneview patch no?19:53
JayFjroll: I'm 99% sure that requires a newer versino of oneviewclient, and teh requirements bump is missing from the patch19:53
JayFjroll: and even if it was there, it's not allowed19:53
*** jcoufal has joined #openstack-ironic19:53
jrollJayF: what makes you think it needs a new oneview client? no code that interacts with oneview client is touched19:54
*** vsaienko has quit IRC19:54
JayFbecause apparently I'm confused19:54
JayFgoing to take a secondd look at that patch19:54
jrollheh19:54
jrollJayF: I was mostly referring to the test-setup.sh stuff though :)19:54
*** rob0t_ has joined #openstack-ironic19:55
JayFjroll: both +2'd, they need to wait to merge tho obviously19:55
jrollJayF: yeah, I thought you were in a meeting and master would land by then :)19:56
JayFI am king of multitasking19:56
JayFand I'm gone at noonish so, yeah19:56
*** vsaienko has joined #openstack-ironic19:56
JayFlol19:56
openstackgerritMerged openstack/ironic master: Log an actual error message when failed to load new style credentials  https://review.openstack.org/42869019:57
jrollah19:59
*** rob0t_ has quit IRC19:59
* jroll steps out for a bit20:02
*** jcoufal has quit IRC20:06
*** jcoufal has joined #openstack-ironic20:06
*** rob0t_ has joined #openstack-ironic20:07
*** rob0t_ has quit IRC20:11
vsaienkojroll, vdrok damn scheduler picked the same node for different instances at the same time http://paste.openstack.org/show/597618/20:12
openstackgerritJulia Kreger proposed openstack/bifrost master: Remove deprecated nginx_port option  https://review.openstack.org/42902120:12
openstackgerritJulia Kreger proposed openstack/bifrost master: Fixing typo in deprecation note  https://review.openstack.org/42902220:12
openstackgerritJulia Kreger proposed openstack/bifrost master: Remove deprecated test playbooks  https://review.openstack.org/42902320:12
vsaienkojroll, vdrok latest multinode job failure https://review.openstack.org/#/c/427675/20:13
patchbotpatch 427675 - ironic - [Devstack] fix waiting resources on subnode20:13
TheJuliaNobodyCam: cinerama some cleanup above as well20:13
*** ijw has joined #openstack-ironic20:16
*** e0ne has quit IRC20:17
openstackgerritJohn L. Villalovos proposed openstack/ironic master: WIP/DNM: Testing multinode job  https://review.openstack.org/42214920:18
*** e0ne has joined #openstack-ironic20:19
*** e0ne has quit IRC20:19
*** e0ne has joined #openstack-ironic20:19
*** Goneri has joined #openstack-ironic20:20
*** e0ne has quit IRC20:20
*** e0ne has joined #openstack-ironic20:20
*** Sukhdev has quit IRC20:20
*** e0ne has quit IRC20:21
*** e0ne has joined #openstack-ironic20:21
*** e0ne has quit IRC20:21
*** dsneddon is now known as dsneddon_afk20:21
*** e0ne has joined #openstack-ironic20:22
*** rob0t_ has joined #openstack-ironic20:22
*** jcoufal_ has joined #openstack-ironic20:22
*** e0ne has quit IRC20:22
*** jcoufal has quit IRC20:23
openstackgerritMerged openstack/ironic master: Allow duplicate execution of update node DB api method  https://review.openstack.org/42767320:24
*** jheroux has quit IRC20:26
*** hecliunyx has quit IRC20:26
*** rajinir has quit IRC20:26
*** rob0t_ has quit IRC20:26
TheJuliaNobodyCam: so all those other changes, they passed the two jobs that have been misbehaving sporatically, I guess that is a good sign :)20:29
NobodyCam:) i'll take a look in a bit20:30
TheJuliajroll: shouldn't 428219 have a regenerated sample config?20:31
openstackgerritMario Villaplana proposed openstack/ironic master: Update api-ref for dynamic drivers  https://review.openstack.org/42846020:31
mariojvcc: jroll dtantsur|afk that's ready ^20:31
*** rob0t_ has joined #openstack-ironic20:32
openstackgerritMerged openstack/ironic-inspector master: Trivial: improve logging in FSM transition  https://review.openstack.org/42785320:33
*** rob0t_ has quit IRC20:36
*** sasha21 has quit IRC20:40
*** awiddersheim has quit IRC20:42
*** rob0t_ has joined #openstack-ironic20:42
*** awiddersheim has joined #openstack-ironic20:43
*** rob0t_ has quit IRC20:47
openstackgerritMerged openstack/ironic master: Prepare for using standard python tests  https://review.openstack.org/42899420:48
*** rob0t_ has joined #openstack-ironic20:52
*** dsneddon has joined #openstack-ironic20:55
*** rob0t_ has quit IRC21:01
*** sdake has quit IRC21:06
TheJuliawow the api-ref update is a bit big :(21:22
*** mgoddard_ has quit IRC21:22
jrollTheJulia: ah damn, I keep forgetting to do that, lol21:23
jrolluno momento21:23
*** amotoki has joined #openstack-ironic21:24
jrollvsaienko: I'd rather wait for CI to be green on that, if it's okay with you21:25
jrollmariojv: holy cow that's big. you rock, thanks for doing that21:25
*** crushil has joined #openstack-ironic21:26
openstackgerritJim Rollenhagen proposed openstack/ironic master: Improve enabled_*_interfaces config help and validation  https://review.openstack.org/42821921:26
jrollTheJulia: bam21:26
TheJuliajroll: merci beaucoup21:27
jrollde nada :P21:27
*** amotoki has quit IRC21:28
*** amarao has joined #openstack-ironic21:32
mariojvyw jroll21:33
*** r-daneel has joined #openstack-ironic21:33
*** mgoddard_ has joined #openstack-ironic21:35
*** crushil has quit IRC21:35
*** bfournie has quit IRC21:41
*** chlong has quit IRC21:44
mariojvi +1ed that also - looks like JayF had +2 before so someone can probably land it (428219)21:49
mariojvsince there is also another +2 there21:49
vsaienkojroll: CI is green https://review.openstack.org/#/c/427675/21:52
patchbotpatch 427675 - ironic - [Devstack] fix waiting resources on subnode21:52
*** jcoufal has joined #openstack-ironic21:53
jrollvsaienko: +A21:53
jrollthanks for that21:54
*** jcoufal_ has quit IRC21:54
vsaienkojroll thanks!21:56
jrollnp :)21:58
*** pester has joined #openstack-ironic21:58
*** srobert_ has joined #openstack-ironic21:59
*** srobert has quit IRC22:02
*** srobert_ has quit IRC22:03
*** pester is now known as fxpester22:04
*** jcoufal has quit IRC22:06
jrollmariojv: reviewed, lookin good, just a few things22:09
jrollmariojv: side note, did you have any issues playing with the new stuff? how (non-)obvious was it?22:10
mariojvjroll: no issues except with the client really22:10
mariojvit seemed quite intuitive to me22:10
*** jcoufal has joined #openstack-ironic22:10
mariojvi think the more confusing thing will be understanding classic vs. dynamic drivers22:11
mariojvi.e. it'd be better if it was just dynamic22:11
jrollwhat was up with the client, just the default version?22:11
jrollyeah22:11
jrollI think in pike I'd like to transition everyone to dynamic drivers22:11
mariojvno, even when you specify --ironic-api-version=1.3122:12
mariojvnode-show doesn't show the interfaces22:12
jrollit does for me O_o22:12
jrollthough, only on master22:12
jrollnot the latest release22:12
mariojvoh, i was pulling down a patched version22:13
mariojvmight have been done on the wrong base22:13
jrollah probably22:13
jrollI think those patches were in separate trees22:13
mariojv(patched with the updates to driver show/list/etc)22:13
jrollya :)22:13
jrollcool, I'm glad it wasn't a horrible experience22:14
mariojvthat's this one https://review.openstack.org/#/c/419274/22:14
patchbotpatch 419274 - python-ironicclient - Extends driver-list, driver-show supporting new ha...22:14
aNuposicmariojv: rama_y: thnx for reviewing, will fix the issues.22:14
jrollI just did a last sanity check and it appears to work fine in devstack, with nova and everything22:14
aNuposicrama_y: wrt https://review.openstack.org/#/c/405586/6/ironic/api/controllers/v1/port.py@89 name would be None22:14
patchbotpatch 405586 - ironic - Allow logical name along with UUID in port creation22:14
mariojvit made a heck of a lot more sense than reading spec/code (though those weren't too difficult either)22:14
mariojvgreat22:14
jrolllol22:14
jrollthe internals are the weird part, I guess22:14
mariojvya22:14
mariojvi'm heading out, have a great weekend ironic o/22:15
mariojv(go patriots)22:15
jroll\o22:15
* jroll walks his dog, may or may not return22:15
jrollif not, see y'all monday :)22:15
*** Sukhdev has joined #openstack-ironic22:16
openstackgerritVarun Gadiraju proposed openstack/ironic master: WIP/DNM: Testing grenade multi-tenant experimental job for Tempest failures  https://review.openstack.org/40389522:21
*** dprince has quit IRC22:24
*** trown is now known as trown|outtypewww22:25
openstackgerritMerged openstack/bifrost master: Allow enroll-dynamic on remote host  https://review.openstack.org/41129922:31
openstackgerritMerged openstack/bifrost master: roles: bifrost-create-vm-nodes: Randomize VM XML file  https://review.openstack.org/42871622:32
*** sdake has joined #openstack-ironic22:32
openstackgerritPeter Piela proposed openstack/ironic-ui master: Add support for additional port attributes  https://review.openstack.org/42906622:37
*** baoli has quit IRC22:37
*** Sukhdev has quit IRC22:38
*** jcoufal has quit IRC22:39
*** owalsh is now known as owalsh-afk22:40
*** jcoufal has joined #openstack-ironic22:40
*** chlong has joined #openstack-ironic22:42
*** amarao has quit IRC22:45
*** cdearborn has quit IRC22:48
*** jcoufal has quit IRC22:50
*** jcoufal has joined #openstack-ironic22:51
*** jkilpatr has quit IRC22:51
*** jaosorior has quit IRC22:52
openstackgerritJohn L. Villalovos proposed openstack/ironic master: WIP/DNM: Testing multinode job  https://review.openstack.org/42214922:52
*** vsaienko has quit IRC23:00
*** mgoddard_ has quit IRC23:00
*** jcoufal has quit IRC23:06
*** dsneddon has quit IRC23:07
*** jcoufal has joined #openstack-ironic23:09
*** mgoddard_ has joined #openstack-ironic23:10
*** sdake has quit IRC23:11
*** dsneddon has joined #openstack-ironic23:14
joannavdrok: Thank you for the review :) I replied to the comments :)23:18
*** pmannidi has joined #openstack-ironic23:20
*** jcoufal has quit IRC23:21
*** jcoufal has joined #openstack-ironic23:21
*** amotoki has joined #openstack-ironic23:24
*** Sukhdev has joined #openstack-ironic23:25
*** jkilpatr has joined #openstack-ironic23:25
*** pmannidi has quit IRC23:26
*** pmannidi has joined #openstack-ironic23:29
*** amotoki has quit IRC23:29
openstackgerritMerged openstack/ironic master: [Devstack] fix waiting resources on subnode  https://review.openstack.org/42767523:29
*** mgoddard_ has quit IRC23:36
*** adu has joined #openstack-ironic23:46
openstackgerritPeter Piela proposed openstack/ironic-ui master: Add support for editing Ironic network ports  https://review.openstack.org/42911123:49
*** adu has quit IRC23:55

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