Monday, 2017-04-03

*** jkilpatr has quit IRC00:04
*** baoli has quit IRC00:52
*** baoli has joined #openstack-ironic00:58
*** rbudden has joined #openstack-ironic00:59
*** gcb has joined #openstack-ironic01:14
*** baoli has quit IRC01:19
zer0c00l1;0c01:22
*** zer0c00l has quit IRC01:23
*** baoli has joined #openstack-ironic01:29
*** tuanluong has joined #openstack-ironic01:35
*** baoli has quit IRC01:43
*** tovin07 has joined #openstack-ironic01:50
*** rcernin has joined #openstack-ironic01:58
*** lucas-afk has quit IRC02:23
*** lucasagomes has joined #openstack-ironic02:25
*** baoli has joined #openstack-ironic02:28
*** baoli_ has joined #openstack-ironic02:30
*** rbudden has quit IRC02:30
*** baoli has quit IRC02:33
*** amotoki has joined #openstack-ironic02:39
*** rcernin has quit IRC02:45
*** baoli_ has quit IRC02:50
*** baoli has joined #openstack-ironic02:51
*** amotoki has quit IRC03:00
*** fragatina has quit IRC03:10
*** baoli has quit IRC03:16
*** amotoki has joined #openstack-ironic03:20
*** vikrant has joined #openstack-ironic03:23
*** amotoki has quit IRC03:29
*** causten has joined #openstack-ironic03:32
*** amotoki has joined #openstack-ironic03:38
*** rama_y_ has joined #openstack-ironic03:42
*** sabeen has joined #openstack-ironic03:43
openstackgerritNisha Agarwal proposed openstack/sushy master: Adds EthernetInterface to the library  https://review.openstack.org/45253203:56
*** amotoki has quit IRC04:01
*** fragatina has joined #openstack-ironic04:01
*** gkadam has joined #openstack-ironic04:03
*** fragatina has quit IRC04:04
*** fragatina has joined #openstack-ironic04:04
*** amotoki has joined #openstack-ironic04:08
*** persia has quit IRC04:17
*** causten has quit IRC04:27
*** causten has joined #openstack-ironic04:28
*** BLZbubba has quit IRC04:30
*** g3ek has quit IRC04:31
*** causten has quit IRC04:32
*** g3ek has joined #openstack-ironic04:35
*** Nisha_Agarwal has joined #openstack-ironic04:53
*** rcernin has joined #openstack-ironic04:56
*** jrcloud has joined #openstack-ironic05:05
*** zackf has quit IRC05:06
*** udesale has joined #openstack-ironic05:11
*** udesale has quit IRC05:14
*** udesale has joined #openstack-ironic05:15
*** rcernin has quit IRC05:18
*** jaganathan has joined #openstack-ironic05:18
*** gkadam has quit IRC05:20
*** gkadam has joined #openstack-ironic05:21
*** jaganathan has quit IRC05:23
*** udesale has quit IRC05:27
*** udesale has joined #openstack-ironic05:27
*** rcernin has joined #openstack-ironic05:31
*** mjura has joined #openstack-ironic05:37
openstackgerritJulian Edwards proposed openstack/ironic-python-agent master: Prevent tests from using utils.execute()  https://review.openstack.org/45260305:37
*** wrouesnel has quit IRC05:39
*** Fdaisuke has joined #openstack-ironic05:40
*** e0ne has joined #openstack-ironic05:45
*** IRCFrEAK has joined #openstack-ironic05:49
openstackgerritNisha Agarwal proposed openstack/sushy master: Adds EthernetInterface to the library  https://review.openstack.org/45253205:50
*** IRCFrEAK has left #openstack-ironic05:50
*** e0ne has quit IRC05:52
*** yogi has joined #openstack-ironic05:59
*** oomichi has quit IRC06:03
*** oomichi has joined #openstack-ironic06:04
*** makowals_ has quit IRC06:05
*** gcb has quit IRC06:07
*** yuanying has quit IRC06:07
*** yuanying_ has joined #openstack-ironic06:13
*** ChubYann has quit IRC06:20
openstackgerritJulian Edwards proposed openstack/ironic-python-agent master: Prevent tests from using utils.execute()  https://review.openstack.org/45260306:22
*** jaganathan has joined #openstack-ironic06:22
*** e0ne has joined #openstack-ironic06:23
*** e0ne has quit IRC06:24
*** rama_y_ has quit IRC06:27
*** vikrant has quit IRC06:28
*** vikrant has joined #openstack-ironic06:28
*** oomichi has quit IRC06:29
*** oomichi has joined #openstack-ironic06:30
*** makowals has joined #openstack-ironic06:30
openstackgerritDebayan Ray proposed openstack/proliantutils master: Separate out rest infra from ris  https://review.openstack.org/45261006:31
*** tesseract has joined #openstack-ironic06:34
*** hoonetorg has quit IRC06:37
*** ralonsoh has joined #openstack-ironic06:43
*** jcoufal has joined #openstack-ironic06:46
*** aparnav has joined #openstack-ironic06:46
*** rbartal has joined #openstack-ironic06:51
*** pcaruana has joined #openstack-ironic06:57
*** udesale has quit IRC06:57
*** jaosorior has joined #openstack-ironic06:57
*** udesale has joined #openstack-ironic06:58
*** yatin_away is now known as yatinkarel06:58
*** udesale has quit IRC06:58
*** udesale has joined #openstack-ironic06:59
*** moshele has joined #openstack-ironic07:00
*** ccamacho has joined #openstack-ironic07:00
*** jcoufal has quit IRC07:04
pas-hamorning ironic :)07:05
dr_gogeta86morning guys07:12
dr_gogeta86anyone can help me to configure ironic for hpe agent-ilo07:12
*** amoralej|off is now known as amoralej07:20
*** Serlex has joined #openstack-ironic07:29
openstackgerritSofiia Andriichenko proposed openstack/ironic master: Add tempest plugin API tests for node  https://review.openstack.org/41119707:36
*** amotoki_ has joined #openstack-ironic07:41
*** amarao has joined #openstack-ironic07:42
lucasagomesmorning all07:44
*** amotoki has quit IRC07:44
tuanluongmorning ironic07:54
*** rbartal has quit IRC07:56
vsaienk0morning Ironic!07:56
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:00
*** openstackgerrit has quit IRC08:03
*** chihhsin has quit IRC08:04
*** openstackgerrit has joined #openstack-ironic08:04
openstackgerritHironori Shiina proposed openstack/ironic master: Add REST API for volume connector and volume target operation  https://review.openstack.org/21458608:04
*** rbartal has joined #openstack-ironic08:05
*** xek has quit IRC08:05
*** sabeen has quit IRC08:07
*** e0ne has joined #openstack-ironic08:08
openstackgerritSofiia Andriichenko proposed openstack/ironic master: Add tempest plugin API tests for node  https://review.openstack.org/41119708:09
*** Nisha_Agarwal has quit IRC08:14
openstackgerrityolanda.robla proposed openstack/bifrost master: Allow looking constraints file from env var  https://review.openstack.org/45207608:14
*** Nisha_Agarwal has joined #openstack-ironic08:20
*** jtomasek has joined #openstack-ironic08:22
*** racedo has joined #openstack-ironic08:24
*** nmathew has joined #openstack-ironic08:24
*** fragatina has quit IRC08:28
vsaienk0lucasagomes: I've resolved your comments https://review.openstack.org/#/c/440719/ once you have a time, please check. Thanks in advance!08:28
lucasagomesvsaienk0, hi there, sure will take a look soon-ish08:28
vsaienk0thanks!08:29
*** causten has joined #openstack-ironic08:29
lucasagomesah it's only that conditional that changed08:29
lucasagomes+a'd08:30
*** Nisha_ has joined #openstack-ironic08:32
*** openstackgerrit has quit IRC08:33
*** causten has quit IRC08:34
*** Nisha_Agarwal has quit IRC08:36
*** Nisha_brb has joined #openstack-ironic08:39
*** mgoddard has joined #openstack-ironic08:40
*** derekh has joined #openstack-ironic08:42
*** Nisha_ has quit IRC08:42
*** racedo has quit IRC08:43
dtantsurmorning ironic!08:46
* dtantsur catches up with .*08:46
*** gkadam_ has joined #openstack-ironic08:52
*** gkadam has quit IRC08:53
*** mtreinish has quit IRC08:53
dtantsurplease throw at me anything requiring my urgent attention, the backlog to go over is huge..08:54
lucasagomesdtantsur, welcome back dude o/08:54
dtantsur\o/ :)08:55
*** mtreinish has joined #openstack-ironic08:57
dtantsurvsaienk0, hi! what was the reason not to remove the "ipmi" hw type job with https://review.openstack.org/#/c/450822/ ?08:58
*** vills has joined #openstack-ironic08:59
vsaienk0dtantsur: we don't have separate job to test ipmi hardware type https://review.openstack.org/#/c/446723/09:01
dtantsurvsaienk0, oh, good, I missed this patch09:02
dtantsursorry for the noise09:02
vsaienk0dtantsur: np, I think it was done when you was on absent09:03
*** kromanenko has quit IRC09:10
vdrokmorning ironic, lucasagomes dtantsur !09:11
lucasagomeso/09:11
*** amotoki_ has quit IRC09:25
*** sambetts|afk is now known as sambetts09:25
sambettsMorning all09:25
sambettsTheJulia: thats awesome!09:27
vdrokgood morning sambetts09:30
*** openstackgerrit has joined #openstack-ironic09:31
openstackgerritSofiia Andriichenko proposed openstack/python-ironicclient master:  Negative tests for testing actions with node  https://review.openstack.org/36158309:31
*** Nisha_ has joined #openstack-ironic09:32
*** Nisha_Agarwal has joined #openstack-ironic09:33
*** Nisha_brb has quit IRC09:35
*** Nisha_ has quit IRC09:36
dtantsurmorning sambetts, vdrok09:37
vdrokdtantsur: looking at your comments on the api stability guidelines, seems like you'd be much more useful than me as api-wg liason, so, if you still want to take over, feel free to do that :)09:38
dtantsurvdrok, well, I don't mind both of us doing it :)09:39
*** Nisha_ has joined #openstack-ironic09:39
Nisha_Morning ironic'iers , lucasagomes dtantsur vdrok sambetts vsaienk009:40
vdrokmorning Nisha_09:40
Nisha_o/09:41
Nisha_dtantsur, welcome back :)09:41
dtantsurthanks :)09:41
*** Nisha_Agarwal has quit IRC09:42
vdrokdtantsur: btw, how was your vacation? :)09:42
dtantsurit was fantastic. I think I'm in love with Barcelona :)09:43
vdrokheh, I prefer rocks to sand :) but otherwise, yeah, I liked it09:44
Nisha_Hey guys, on Nova side the compute node acts as a resource provider and under which we can have multiple resource classes while on ironic side we can have only one resource class per node, or rather i would say each ironic node can have only one resource class associated09:44
Nisha_how do we then relally use a node for 'N' number of capabilities?09:45
Nisha_or get it scheduled09:45
vdrokNisha_: capabilities is a separate thing, there are traits, and multiple traits can be associated with a resource provider iiuc09:45
Nisha_as i understand all the capabilities(quantitative) will have to eventually convert to resource classes09:45
*** vills has quit IRC09:45
dtantsurvdrok, there were enough rocks as well09:46
vdrokdtantsur: oh, this means I've spent not enough time there :)09:46
dtantsurvdrok, have you checked these views from the mountains? https://www.instagram.com/p/BSQfJI-D0YC/?taken-by=creepy_owlet09:46
vdrokdtantsur: nope, that's my plan for May :)09:47
vdrokNisha_: why not make a resource class to be ironic_gold_with_4_gpus_and_2_4TB_disks?09:48
dtantsurvdrok, I highly recommend :)09:48
vdrokthat would describe all those things together09:48
Nisha_vdrok, so there are ques09:49
Nisha_1. whats the exact and correct definition of ironic gold09:49
Nisha_as i understand it is user(operator) defined09:49
Nisha_2. the number of resource say cpu, gpu, disk size etc may vary from system to system09:50
vdrokevery aspect of the quantitative resources you think will be neccesary for scheduling. if you think you need a number of gpus for scheduling, include it somehow in the resource class. it's just a string, you can decide how to form it yourself09:50
Nisha_do we itend to create as many resource classes09:50
Nisha_vdrok, but my one node can have only one resource class name09:51
dtantsurNisha_, I think the essence of this change is to re-think how and why you schedule on certain nodes. Do you really need to distinguish between nodes with 100 Gi and 110 Gi disks? Or do you need "nodes with large disks (for storage)", "nodes with smaller disks", etc?09:51
dtantsurthere is no direct capability-to-resource_class mapping, unfortunately.09:51
Nisha_dtantsur, why cant we have something similar to nova...nova structure looks reasonable i.e. act as resource provider and define resource classes and have the multiple resource classes with their values under one resource provider09:54
dtantsurNisha_, it's not "we", it's Nova, so I'm not sure I understand your question.. I recommend you chat with jroll on this changes09:54
Nisha_dtantsur, 'we' hear meant ironic ...anyway yes i will discuss with him09:55
dtantsurNisha_, I understand, but the change is made by Nova, not by Ironic.09:55
dtantsuras in, we (Ironic) are not pushing something on Nova09:56
Nisha_dtantsur, yeah :(09:57
Nisha_dtantsur, but we are also trying to via the same way...so just trying to see how just resource class tag can help us for scheduling09:58
openstackgerritOksana Voshchana proposed openstack/ironic master: WIP: SSL TEST  https://review.openstack.org/44240209:59
vdrokNisha_: in nova, there is a concept of nested resource providers, but that handles different things, like fpgas. for ironic case, there is one node, and you provide the whole node. so as for me, it makes sense for node to be a single resource class, as you can't allocate only a disk from the node09:59
Nisha_vdrok, resource class stands for one attribute09:59
dtantsurvsaienk0, could you please move https://github.com/openstack/puppet-ironic/commit/507d3877138a4fc1696e35734a860459962b61d7 to a proper (ironic::neutron) manifest (with deprecation ofc)?10:00
*** deray has joined #openstack-ironic10:00
vdrokNisha_: yes, but I still don't get the problem. if you care about eg number of gpus on a node, you include this number in the string somehow. if you don't, just don't use it10:00
Nisha_vdrok, which essentially mean that i can have , say 10, nodes under ironic and each one of them have different resources.....10:00
*** tovin07 has quit IRC10:01
Nisha_vdrok, its not only numbr of gpus correct10:01
derayHi .. g'mroning all Ironic'ers!10:01
Nisha_vdrok, it can be persistent memory size10:01
Nisha_it can be number of ssd devices10:01
vdrokNisha_: it can be everything, yes. But still the point is the same. if you care about it during scheduling, it means a separate resource class10:01
Nisha_number of nvme ssd10:01
Nisha_vdrok, ok. so if understand u , u mean that i shud have all of these quantities in resource class name for that node10:03
vdrokwhat's the difference between encoding it in the resource class or adding it to capabilities? it's basically the same10:03
vdrokyes10:03
vsaienk0dtantsur: oh.. right, we have a separate class for neutron10:03
vdrokmorning deray10:03
derayvdrok, g'morning :)10:03
deraylucasagomes, hi10:05
*** rcernin has quit IRC10:05
derayhttps://review.openstack.org/#/c/452638/10:05
deray^^^ any idea how/where to add the job definition10:05
derayhttp://logs.openstack.org/38/452638/1/check/gate-project-config-layout/bebe00e/console.html#_2017-04-03_08_44_25_79487310:05
sambettsvdrok, vsaienk0: https://bugs.launchpad.net/ironic/+bug/167899510:06
openstackLaunchpad bug 1678995 in Ironic "In flat network scenario with port groups, neutron port mac address isn't updated" [Medium,Triaged]10:06
openstackgerritGalyna Zholtkevych proposed openstack/ironic master: Detailed sequence-diagram for deploy process  https://review.openstack.org/41451210:06
*** hoonetorg has joined #openstack-ironic10:07
dtantsurderay, I suspect the "coverage-jobs" template does not define the -nv variant. do you really need it non-voting?10:07
dtantsurderay, check any project (e.g. ironic) with the coverage job already present10:08
deraydtantsur, oh .. okay .. where r u looking for the defn of coverage-jobs ?10:08
dtantsurderay, I'm not looking, just guessing. it should be in some file in jenkins/jobs/10:08
*** pc_m has quit IRC10:08
vsaienk0sambetts: I think there is also problem with portgroup and standalone_ports_supported=False for flat case10:09
*** pc_m has joined #openstack-ironic10:10
sambettsvsaienk0: hmm yeah, I don't think we are checking that for the flat case :/10:14
openstackgerritGalyna Zholtkevych proposed openstack/ironic master: Doc for console customization using systemd  https://review.openstack.org/44657110:15
sambettsvsaienk0: what do you think is the right solution out of the ones I've listed in that bug report? I think the bottom one is the nicest but it involves saving the old data and resetting it10:16
deraydtantsur, thanks .. u r right :) https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/python-jobs.yaml#L576-L58510:16
deraydtantsur, unless u define your suffix it will not be inserted10:17
*** MattMan has quit IRC10:17
*** MattMan has joined #openstack-ironic10:17
dtantsurderay, no, the suffix seems fine10:18
dtantsurderay, it does define all variants, with and without -nv. I just realized that you may be using the wrong name, it should be s/python-sushy/sushy/10:18
deraydtantsur, aha !.. good catch :) I am so sorry..10:19
dtantsurno worries, many libraries start with python- indeed10:20
derayyeah .. blind copy-paste error :)10:20
openstackgerritMerged openstack/ironic master: [Devstack] Rework VMs connection logic  https://review.openstack.org/44071910:22
*** rcernin has joined #openstack-ironic10:24
openstackgerritMerged openstack/ironic master: Remove deprecated, untested ipminative driver  https://review.openstack.org/45152610:25
dtantsur\o/ JayF ^^^10:27
vsaienk0sambetts: ideally I would like to force mac_address to be mandatory for ironic portgroup object10:27
sambettsvsaienk0: but then we can't use the neutron generated MACs right?10:28
*** erolg has joined #openstack-ironic10:29
*** Nisha_brb has joined #openstack-ironic10:31
*** stendulker has joined #openstack-ironic10:31
*** Nisha_ has quit IRC10:34
vsaienk0sambetts: right10:35
*** chas has joined #openstack-ironic10:35
*** Nisha_brb has quit IRC10:38
vsaienk0sambetts: what is the benefit from using neutron generated mac address for portgroup? We still enforce that portgroup address is unique among portgroups on ironic side10:39
openstackgerritShivanand Tendulker proposed openstack/ironic master: Remove deprecated method build_instance_info_for_deploy()  https://review.openstack.org/45269010:41
*** galyna has joined #openstack-ironic10:45
openstackgerritLucas Alvares Gomes proposed openstack/sushy-tools master: Initial Commit  https://review.openstack.org/45269210:45
lucasagomesdtantsur, deray (and others interested ^)10:45
openstackgerritShivanand Tendulker proposed openstack/ironic master: Add hardware type for HPE ProLiant servers based on iLO 4  https://review.openstack.org/43940410:46
deraylucasagomes, yeah .. gr8 .. makes sense :)10:46
dtantsurlucasagomes, quick comment (will get to it later): sushy_tools.sushy_libvirt is a bit too much. maybe rather sushy_tools.libvirt_emulator? ditto for sushy_static (it's not clear from its name what it does)10:47
dtantsurlucasagomes, babel.cfg assumes it will get translations, which I doubt10:47
lucasagomesdtantsur, right yeah I can rename those, it's that sushy_libvirt will appear as "sushy-libvirt" command10:47
lucasagomessame for static10:48
lucasagomes"sushy-static"10:48
lucasagomesso to start the emulators you just need to use those commands, that's why I translated it literally (just replacing the "_" with a "-" for the actual command)10:48
lucasagomesdtantsur, yeah I probably can remove the translations for all, it came with the cookiecutter heh10:49
lucasagomesI will delete it10:49
dtantsurhmm, I see your reasoning.. btw, I'd prefer sushy-libvirt-emulator or even just sushy-emulator (potentially accepting options like --libvirt/--something-else)10:49
lucasagomesdtantsur, fair enuff, lemme rename it10:49
dtantsurand then I'd still use sushy_tools.emulator and sushy_tools.static, but maybe it's only me.. not a blocking issue10:49
deraydtantsur, > sushy-emulator (potentially accepting options like --libvirt/--something-else) .. this sounds good10:50
lucasagomesI don't have a preference, but yeah the namespace is duplicated10:50
* dtantsur subscribes to sushy-tools in gertty10:50
deraylucasagomes, dtantsur can we have "emulator > static" and "emulator > libvirt" ?10:52
lucasagomesderay, thing is, they don't really share any code in common10:53
dtantsurdepending on what static is..10:53
lucasagomesso I would prefer to treat it as a separated tool10:53
lucasagomesIMHO makes more sense10:53
lucasagomescause for the libvirt we actually create a small REST API and the backend is libvirt10:54
deraylucasagomes, yeah I know10:54
lucasagomesbut the static is just a python http server serving some files in the filesystem10:54
yolandahi, i'm hitting this error when sending a change in requirements to newton: http://logs.openstack.org/59/452659/2/check/gate-requirements-tox-py27-check-uc-ubuntu-xenial/5d16ae5/console.html#_2017-04-03_09_37_03_97613910:54
yolandaanyway, seems to be a problem with all the clients :(10:57
openstackgerritLucas Alvares Gomes proposed openstack/sushy-tools master: Initial Commit  https://review.openstack.org/45269210:58
deraylucasagomes, so whats the plan? ``sushy_tools>libvirt`` and ``sushy_tools>static``? or ``emulator`` word getting 'prepended' to each?10:59
openstackgerritOksana Voshchana proposed openstack/ironic master: WIP: SSL TEST  https://review.openstack.org/44240210:59
lucasagomesderay, I did sushy_tools/emulator and sushy_tools/static10:59
lucasagomesI can change again if needed, naming is hard so we still need a consensus :-)10:59
deraylucasagomes, I am okay .. will go thru the code.. is it the same code content cut and pasted from sushy? or something more?11:00
lucasagomesfor the user of the tools it doesn't matter much, he will install it from pip, run "sushy-emulator" and BOOM he's now have a Redfish API that controls his local vms11:01
sambettsvsaienk0: the thing I'd prefer to avoid is having the operator make a decision which might affect the OSs operation of the bond11:01
lucasagomesderay, pretty much copy and paste, I just added a "main()" method so I can invoke it in the setup.cfg11:01
*** jkilpatr has joined #openstack-ironic11:01
lucasagomesderay, before it was part of the "if __name__ ==  __main__" conditional11:02
sambettsvsaienk0: I like the thought that if the MAC address is different from the sub-ports then the operation of that bond is entirly separate from the physical interfaces11:02
vsaienk0sambetts: we still enforce mac address for bond on user image via configdrive for case when it was automatically picked by neutron11:02
deraylucasagomes, okay11:02
sambettsvsaienk0: right, but that mac address isn't either of the physical MAC address of the links11:03
vsaienk0sambetts: ideally neutron should be aware of port groups and do all tricks with mac_address on its side11:03
openstackgerritShivanand Tendulker proposed openstack/ironic master: Remove deprecated method build_instance_info_for_deploy()  https://review.openstack.org/45269011:04
deraylucasagomes, have u copied the mock contents (some.zip file) as part of the repo now?11:05
deraylucasagomes, static one11:05
lucasagomesderay, no I have not... I wonder if we should or can do that ? Cause it has some copyrights from the DMTF on them11:05
lucasagomesso I thought about instructing ppl to download it from the DMTF site directly and running the sushy-static to serve it11:06
deraylucasagomes, oh kay .. I think its better to remain as documented only11:06
lucasagomesidk maybe I'm wrong and we can have the zips in the repo, I just didn't look much at it11:06
lucasagomesderay, yeah for now at least it's safer11:06
deraylucasagomes, yeah .. no overhead for version changes also11:07
lucasagomestrue that11:07
lucasagomesand pointing them to the website they can download whatever version they want to test it against11:07
*** athomas has joined #openstack-ironic11:08
derayokay .. bye all Ironic'ers for now .. catch u t'morrow11:08
deraylucasagomes, yeah .. makes more sense11:08
deraylucasagomes, no version lock-in as such11:09
sambettsvsaienk0: I don't know if neutron plan to support bonding at all :/ for a normal VM case it might not make any sense but for SRIOV it might11:09
deraylucasagomes, okay then bye for now .. btw I have submitted the coverage job patch .. https://review.openstack.org/#/c/452638/11:10
vsaienk0sambetts: I'm sure they will, they already added vlan aware vms support :)11:10
lucasagomesderay, yeah I just +1 that, thanks for it11:10
derayoh yeah seeing it now ..11:10
*** moshele has quit IRC11:10
sambettsvsaienk0: heh yeah, I wonder :/11:11
vsaienk0sambetts: so for now we can document that for flat case mac_address should be set for portgroup to one of its physical links to allow boot node via PXE11:11
*** mat128 has joined #openstack-ironic11:13
*** mat128 has quit IRC11:13
sambettsvsaienk0: that would be the quickest "fix"11:14
sambettsvsaienk0: but we might also need to ensure we enforce the standalone_ports_ flag check for the flat network case :/11:15
*** deray has quit IRC11:15
*** mat128 has joined #openstack-ironic11:15
*** lucasagomes is now known as lucas-hungry11:20
*** stendulker has quit IRC11:23
*** ccamacho is now known as ccamacho|lunch11:24
*** ppiela_ has joined #openstack-ironic11:27
openstackgerritYuriy Zveryanskyy proposed openstack/ironic master: Validate outlet index in SNMP driver  https://review.openstack.org/45271611:27
*** ppiela has quit IRC11:29
*** erolg has quit IRC11:36
*** erolg has joined #openstack-ironic11:36
*** tuanluong has quit IRC11:42
*** bfournie has quit IRC11:55
*** ccamacho|lunch is now known as ccamacho11:56
*** jaganathan has quit IRC11:57
*** chihhsin has joined #openstack-ironic11:57
*** dprince has joined #openstack-ironic12:00
vdrokdtantsur: couple of inspector bugs I'm not sure I can comment on - https://bugs.launchpad.net/ironic-inspector/+bug/1670461 and https://bugs.launchpad.net/ironic-inspector/+bug/1678473, when you have some time to look12:03
openstackLaunchpad bug 1670461 in Ironic Inspector "unpin functional test python3 version" [Undecided,New] - Assigned to Ramamani Yeleswarapu (ramamani-yeleswarapu)12:03
openstackLaunchpad bug 1678473 in Ironic Inspector "Flask app crash and get “[Errno 32] Broken pipe”" [Undecided,New] - Assigned to Tao Li (eric-litao)12:03
*** mjura has quit IRC12:04
*** jaypipes has joined #openstack-ironic12:05
*** trown|outtypewww is now known as trown12:05
dtantsurvdrok, done, thanks12:07
*** lucas-hungry is now known as lucasagomes12:07
vdrokthank you!12:08
*** ricardoas has left #openstack-ironic12:12
*** mat128 has quit IRC12:14
*** jcoufal has joined #openstack-ironic12:19
openstackgerritSofiia Andriichenko proposed openstack/python-ironicclient master:  Negative tests for testing actions with node  https://review.openstack.org/36158312:19
*** jaganathan has joined #openstack-ironic12:20
*** causten has joined #openstack-ironic12:30
*** causten has quit IRC12:35
*** jaosorior is now known as jaosorior_brb12:36
pas-hayolanda: re newton requirements, try depending on this one https://review.openstack.org/#/c/447025/12:42
TheJuliaGood morning everyone12:45
xavierrmorning TheJulia and ironic12:46
*** vikrant has quit IRC12:53
*** amoralej is now known as amoralej|lunch12:54
pas-hamorning TheJulia and xavierr :)12:56
xavierrhey pas-ha :)12:58
*** sasha2 has joined #openstack-ironic12:59
*** baoli has joined #openstack-ironic13:00
*** yogi has quit IRC13:02
openstackgerritYuriy Zveryanskyy proposed openstack/ironic master: Validate outlet index in SNMP driver  https://review.openstack.org/45271613:04
*** rloo has joined #openstack-ironic13:14
*** gkadam_ is now known as gkadam13:15
*** crushil has quit IRC13:16
*** gkadam has quit IRC13:17
*** srobert has joined #openstack-ironic13:18
*** rbudden has joined #openstack-ironic13:18
*** aparnav has quit IRC13:18
*** mat128 has joined #openstack-ironic13:19
*** rook|tower is now known as rook13:19
*** amarao has quit IRC13:24
*** makowals_ has joined #openstack-ironic13:27
*** amarao has joined #openstack-ironic13:28
*** ma9 has joined #openstack-ironic13:29
*** baoli has quit IRC13:29
*** makowals has quit IRC13:30
*** baoli has joined #openstack-ironic13:30
*** crushil has joined #openstack-ironic13:31
*** beekneemech is now known as bnemec13:31
*** jaosorior_brb is now known as jaosorior13:33
*** ma91 has joined #openstack-ironic13:37
*** ma9 has quit IRC13:39
*** gcb has joined #openstack-ironic13:41
*** nicodemos has quit IRC13:41
*** dtantsur is now known as dtantsur|brb13:42
jroll(late) morning everyone13:45
*** mat128 has quit IRC13:45
*** bfournie has joined #openstack-ironic13:46
*** bfournie has quit IRC13:46
mariojvmorning all13:47
jrolldtantsur|brb: rloo: so doug reviewed the ironic release request, found there's a config value dropped. see https://review.openstack.org/#/c/450954/3 and https://github.com/openstack/ironic/commit/330ba8c . two questions: 1) any problems with releasing that as 8.0.0? 2) if we want to get a reno in for that, we'll need to release this one, should we do that or just get the reno in 8.1.0?13:48
jrollhttps://github.com/openstack/ironic/commit/9bc06783ecaff5eef58535904abc12ea1598232913:48
rloojroll: oh. so if we drop a config, it needs a major bump/13:49
rloojroll: i am fine with it being 8.0.0. (What other choices are there?)13:49
lucasagomesjroll, rloo morning o/13:50
*** rbartal has quit IRC13:50
*** jaganathan has quit IRC13:50
jrollrloo: there isn't, just asking if you foresee any problems there (I don't fwiw)13:51
rloojroll: fwiw, i thought that we always did a major bump when we started a new dev cycle. I don't see any problem there. the reno is a diff question, am thinking...13:52
*** jcoufal has quit IRC13:52
jrollrloo: we usually do because we tend to remove deprecated things at the beginning of a cycle :)13:53
jrollalso, as a note13:53
jrollthe next release after this will be 9.0.0 because we dropped a driver13:53
rloojroll: oh, that explains it. maybe we should have removed more deprecated things then.13:53
TheJuliagood morning jroll, rloo13:54
rlooso i am good with the reno being in 8.1.0 although technically it should be in 8.0.0. but we've done similar things in the past, missed/updated renos later.13:54
jrollrloo: well, ocata was short, some of them aren't past the three month threshold13:54
jrollrloo: s/8.1.0/9.0.0/ but yeah13:54
rlooand big hi and good morning everyone, TheJulia, jroll, mariojv, dtantsur|brb13:54
jrollmorning lucasagomes, rloo, TheJulia :)13:56
lucasagomesTheJulia, morning13:56
*** Goneri has joined #openstack-ironic13:56
*** sabeen has joined #openstack-ironic13:57
rloojroll: so for 9.0.0 we might be able to remove some of the deprecated things...13:57
*** makowals_ has quit IRC13:58
jrollrloo: yep13:59
rloojroll: is doug ok if we punt on the reno? (I'm not totally sure what he checks for.)14:00
*** fragatina has joined #openstack-ironic14:00
jrollrloo: I didn't ask but it should be fine. release cores primarily check for version numbers14:01
*** makowals has joined #openstack-ironic14:01
*** fragatina has quit IRC14:02
*** jkilpatr has quit IRC14:02
*** nmathew has quit IRC14:02
*** jaganathan has joined #openstack-ironic14:03
rloojroll: i just looked and the rpc/object version pinning and removal of ipminative are on master now, so if we add reno for the removed config, we'd pull in these two, which i'd rather not do.14:04
*** amotoki has joined #openstack-ironic14:05
jrollrloo: right, that's why I wanted to just leave it for the next release14:06
rloojroll: yup, i was thinking about maybe rolling back the rpc pinning one, but to roll back both that and ipminative seems like too much work (not worth it)14:06
*** tuan_luong has joined #openstack-ironic14:07
jrollrloo: agree14:07
*** tuanla_ has joined #openstack-ironic14:08
vsaienk0jroll: ironic-standalone is voting now, you can close ticket/bug with community goal to support API under WSGI14:12
jrollvsaienk0: sweeeet, thanks!14:12
* jroll adds to todo list because that also involves a governance patch14:13
*** udesale has quit IRC14:17
*** amoralej|lunch is now known as amoralej14:20
vdrokmorning TheJulia xavierr jroll mariojv and rloo !14:21
rloohi vdrok!14:21
openstackgerritMario Villaplana proposed openstack/ironic master: Add rescuewait timeout periodic task  https://review.openstack.org/35315614:21
openstackgerritMario Villaplana proposed openstack/ironic master: Agent driver rescue implementation  https://review.openstack.org/40043714:21
openstackgerritMario Villaplana proposed openstack/ironic master: WIP Rescue mode tempest tests  https://review.openstack.org/45230814:21
openstackgerritMario Villaplana proposed openstack/ironic master: Add API and conductor methods for [un]rescue  https://review.openstack.org/35083114:21
mariojvhey vdrok, rloo14:22
openstackgerritBéla Vancsics proposed openstack/ironic master: Use more specific asserts in tests  https://review.openstack.org/43294114:23
*** ricardoas has joined #openstack-ironic14:24
*** baha has joined #openstack-ironic14:24
*** amotoki has quit IRC14:25
openstackgerritLucas Alvares Gomes proposed openstack/sushy-tools master: Initial Commit  https://review.openstack.org/45269214:29
*** bfournie has joined #openstack-ironic14:31
*** bfournie has left #openstack-ironic14:32
*** causten has joined #openstack-ironic14:36
*** Syed__ has joined #openstack-ironic14:41
*** sabeen has quit IRC14:42
*** chlong has joined #openstack-ironic14:45
*** rama_y_ has joined #openstack-ironic14:46
*** sabeen has joined #openstack-ironic14:49
*** baoli has quit IRC14:49
*** amarao has quit IRC14:52
dtantsur|brbmorning jroll, rloo14:54
*** dtantsur|brb is now known as dtantsur14:54
jroll\o14:54
rloowelcome back dtantsur!!!14:54
dtantsurjroll, I'm fine with 8.0.0 (I don't have any feelings about numbers, as long as they follow sem-ver)14:54
jrolldtantsur: I thought so, thanks14:54
jrolland yeah, welcome back :)14:54
dtantsurvsaienk0, jroll, " you can close ticket/bug with community goal to support API under WSGI", I think inspector is not there yet14:55
dtantsurthanks rloo, jroll :)14:55
jrolldtantsur: good point14:55
dtantsurjroll, as to "when to add a release note".. well, if we add it for 8.1, it will show in 8.1...14:55
jrolldtantsur: yeah, I know, but rloo was concerned about releasing only part of the upgrades code (and we landed some)14:56
dtantsurugh14:56
dtantsurhow far are we from landing upgrades?14:56
*** mgould|afk is now known as mgould14:56
mgouldhi everyone14:56
rloodtantsur: that upgrade patch that landed has a new config, but we didn't add a reno to that patch cuz it isn't useful.14:57
jrolldtantsur: probably pretty close but unclear14:57
rloodtantsur: farther than i thought. i was testing last week and found a big issue with the version column.14:57
* dtantsur thinks14:57
dtantsurcould you link me to the patch that merged?14:58
rloodtantsur, jroll: I mean, my aim is to fix it this week, but we also don't have multinode grenade testing and even if we did, i am paranoid enough that i am testing this slowly and painfully.14:58
jrolldtantsur: https://github.com/openstack/ironic/commit/9bc06783ecaff5eef58535904abc12ea1598232914:58
jrollit's like. fine to release it, I think14:59
rloodtantsur, jroll: maybe we should try to roll back that one upgrade patch, i suspect/hope it doesn't conflict much with the patches that landed after it.14:59
jrolldtantsur: we'd also be releasing https://github.com/openstack/ironic/commit/96eafdc866b0d1d2b1384ae818804c788bc9021f14:59
*** amotoki has joined #openstack-ironic14:59
rloojroll, dtantsur: yes, it is probably fine to release it, i just don't know for sure :)14:59
jroll(which I think is fine)14:59
jrollrloo: I can't imagine it breaks anything14:59
rloojroll, dtantsur: i doubt it will break anything. but it exists w/o any 'this is experimental, this doesn't work yet'.15:00
jrollrloo: though we should add to release_mappings before releasing that, for consistency15:00
dtantsurjroll, I suggest we mark this config option as rloo suggests (we did something similar with driver comp at one stage)15:00
dtantsurthen we can release it15:00
jrolldtantsur: as in, mark it experimental?15:01
dtantsurjroll, yep. and update all mentions of it as "it's work-in-progress, do not touch it or rely on it"15:02
jrollrloo: ^ mind writing that patch, and I'll write the reno for the o.messaging thing15:02
rloojroll: will do.15:02
jrollthanks15:02
*** makowals has quit IRC15:04
openstackgerritJim Rollenhagen proposed openstack/ironic master: Add release note for messaging alias removal  https://review.openstack.org/45278715:06
jroll^ there's the reno15:06
*** makowals has joined #openstack-ironic15:06
*** crushil has quit IRC15:09
rloodtantsur: if I add a comment everywhere I use that config option, i might as well change the code *not* to use it...?15:11
rloodtantsur: or just change the code wrt the config option, to disallow it being set...?15:12
rloodtantsur: actually, with the driver comp, we only put a comment in the config option/help, so i think that should suffice here too.15:13
*** makowals has quit IRC15:13
dr_gogeta86who can help with dnsmasq15:16
dr_gogeta86i don't have the ack signal from pxe15:16
dr_gogeta86how can I fix that ?15:16
*** baoli has joined #openstack-ironic15:21
*** baoli has quit IRC15:24
*** crushil has joined #openstack-ironic15:24
*** Serlex has quit IRC15:24
dtantsurrloo, yep, I think we should only make it clear that this option is not ready for consumption15:26
*** sayantan_ has joined #openstack-ironic15:26
rloodtantsur: ++. I looked at what we put for driver comp, am using same/similar wording :)15:26
dtantsurgood idea15:26
sambettsTheJulia: I look forword to your etherpad15:29
*** tesseract has quit IRC15:31
*** baoli has joined #openstack-ironic15:31
openstackgerritOksana Voshchana proposed openstack/ironic-python-agent master: Need test ssl in IPA  https://review.openstack.org/45280415:32
*** joanna__ has joined #openstack-ironic15:33
dr_gogeta86any pxe expert ?15:33
dr_gogeta86I'm quiet disperate15:33
openstackgerritOksana Voshchana proposed openstack/ironic master: WIP: SSL TEST  https://review.openstack.org/44240215:33
jrolldr_gogeta86: could be a missing route or something, assuming the DHCPACK is being sent. it's really more of a network troubleshooting problem, which is hard to help with :)15:34
dr_gogeta86jroll, these hpe machine make me nut15:35
jrollI've had this problem before with a missing return route from ironic-conductor to the machine itself15:35
jrollthis isn't unique to HPE15:35
dr_gogeta86jroll, returning route for ?15:35
dr_gogeta86this is a triple-o15:36
jrolldr_gogeta86: the network route from where DHCP is running to the network where pxe booting is happening15:36
TheJuliasambetts|afk: Awesome, I look forward to hearing others thoughts on the things I've run into. I just need to let things unwind in my brain today before I begin writing everything out.  And I need to do it this week because I'm at the leadership thing next week, and vacationing the following week15:36
jrollI don't know much about tripleo15:36
dr_gogeta86jroll, is the same net15:36
NobodyCamGood morning Ironic'ers15:36
mgouldmorning NobodyCam15:37
NobodyCam:) morning mgould15:37
jrolldr_gogeta86: so 'ip route get $DHCP_ADDRESS' returns the correct gateway?15:37
jrolland the ACK is being sent from the dhcp server?15:37
dr_gogeta86yep15:37
*** crushil has quit IRC15:37
jrollhm, dunno where to start then15:37
dr_gogeta86i have just the discover and the offer15:38
jrolloh right, I had that backwards, was thinking of the OFFER not reaching the host15:39
*** amotoki has quit IRC15:39
openstackgerritRuby Loo proposed openstack/ironic master: Updates to RPC and object version pinning  https://review.openstack.org/45280615:40
dr_gogeta86jroll, when i'm doing the introspection worsk file15:42
dr_gogeta86jroll, when i'm doing the introspection works fine15:42
*** mat128 has joined #openstack-ironic15:43
dtantsurmorning NobodyCam15:44
NobodyCamMorning dtantsur :) happy monday :p15:44
jrolldr_gogeta86: inspector puts down some iptables rules, if that's running on the same host as the neutron dhcp server then I could see that causing problems, so maybe check iptables15:45
* jroll doesn't know how tripleo deals with this15:45
dtantsurjroll, inspector co-exists with neutron quite fine, as neutron does its networking in a separate namespace (IIUC)15:46
jrolldtantsur: yeah, just throwing ideas out, idk how to debug networking issues over irc ¯\_(ツ)_/¯15:46
dtantsuryep15:46
dtantsuralso the offer does come..15:46
TheJuliadtantsur: it does, the only issue one might have is where to attach dnsmasq to for the packets to be received and responded to by dnsmasq for the node to boot for introspection or discovery15:47
* TheJulia mucked with it in detail over the last two days15:48
dtantsurwell, for tripleo (assuming undercloud), it's quite typical15:48
dr_gogeta86jroll, time to double check ... for a** along the way15:49
dr_gogeta86like mac antispoof and things like that15:49
TheJuliadr_gogeta86: can you confirm that the ack is actually leaving host, or is it never leaving the controller?15:50
dtantsurdr_gogeta86, check this one just in case: https://docs.openstack.org/developer/ironic/deploy/troubleshooting.html#dhcp-during-pxe-or-ipxe-is-inconsistent-or-unreliable15:50
*** ijw has joined #openstack-ironic15:51
*** pcaruana has quit IRC15:53
dr_gogeta86tnx TheJulia dtantsur15:56
*** e0ne has quit IRC15:57
*** sayantan_ has quit IRC15:59
*** sayantan_ has joined #openstack-ironic15:59
*** Nisha_Agarwal has joined #openstack-ironic16:00
TheJuliaportfast is a big deal.  I should do that on my switch. :)16:01
*** rpioso has joined #openstack-ironic16:01
sambetts+1000 to ^16:01
*** rama_y_ has quit IRC16:03
*** gcb has quit IRC16:03
*** mat128 has quit IRC16:04
TheJuliawouldn't that be a no-brainer to burn into switch port configuration?16:04
TheJuliaerr16:04
TheJuliawell, if I build a bridge across multiple switches in a  host, that would be bad16:04
* TheJulia gets a "There be dragons here" sign, and places that around switch configuration.16:05
sambettshaha16:05
Nisha_Agarwaljroll, hi16:06
jrollhey Nisha_Agarwal16:07
*** vgadiraj_ has joined #openstack-ironic16:07
*** srart has joined #openstack-ironic16:07
TheJuliaAnyone know of a really good book on openvswitch?  I installed it in a bifrost role along with neutron, and having to muck with dhcp configuration with dnsmasq was a real headache, it would silently not see the packets, even when tcpdump would gladly show packets on the interface16:07
Nisha_Agarwaljroll, i had little discussion on resource class with dtantsur and vdrok in the afternoon....had some queries16:08
*** trown is now known as trown|lunch16:08
jrollNisha_Agarwal: I read the conversation, I don't have much to add. remember that this is all a work in progress, scheduling just on the node's resource class is not the end goal, merely the first step.16:09
dtantsurTheJulia, if you find such book - lemme know please16:09
jrollNisha_Agarwal: but go ahead and ask16:09
Nisha_Agarwaljroll, so you agree on vdrok's statement that the resource class should be named with all the quantitative resources in its name if scheduling is to be done on that basis16:11
jrollNisha_Agarwal: I agree that if you want to schedule two different nodes with two different flavors, they should have different resource classes. putting all the quantitative resources in the resource class is an easy way to do that. but you could also call them 'tom' and 'pocahontas', the name really doesn't matter :)16:12
*** baha has quit IRC16:12
Nisha_Agarwaljroll, "I agree that if you want to schedule two different nodes with two different flavors, they should have different resource classes" --- what i understand from this is that even if user wants to schedule a node based on one attribute(resource) but different in quantity across different nodes, they all will have different resource classes in ironic16:15
jrollNisha_Agarwal: correct16:15
jrollNisha_Agarwal: I hope that in the future we could have multiple resource classes for a single node, but for now that is the answer16:16
Nisha_Agarwaljroll, ok.16:16
*** vills has joined #openstack-ironic16:16
Nisha_Agarwaljroll, i was reading your flavor spec also16:16
*** erolg has quit IRC16:17
*** ma91 has quit IRC16:17
Nisha_Agarwaljroll, on the flavor side what does $N in  "``resources:$CUSTOM_RESOURCE_CLASS=$N" mean?16:17
jrollNisha_Agarwal: an integer16:18
*** ma9 has joined #openstack-ironic16:18
*** mat128 has joined #openstack-ironic16:18
Nisha_Agarwaljroll, is it number of instances to be selected for $CUSTOM_RESOURCE_CLASS16:18
jrollNisha_Agarwal: for an ironic node's resource class, it will always be 1. but this is a general spec that could apply to a custom resource class for a VM host, too16:18
*** jkilpatr has joined #openstack-ironic16:18
Nisha_Agarwaljroll, Ok. so on VM side if it is greater than 1, what does that mean?16:19
jrollso maybe an operator tags some VM hosts with 2 CUSTOM_NVIDIA_GPUS, their flavor could have resources:CUSTOM_NVIDIA_GPUS=2 to claim both in the request16:19
jrollit means the flavor claims that many of that resource16:19
*** d0ugal has quit IRC16:19
Nisha_Agarwalok so to match the quantity16:20
Nisha_Agarwaland claim that16:20
jrollyes16:20
Nisha_Agarwaljroll, so currently if we want to add , say nvidia_gpu_count, for ironic nodes that is not possible?16:22
*** ma91 has joined #openstack-ironic16:22
Nisha_Agarwalwe can still add them under capabilities as capabilities will still work16:22
*** ma9 has quit IRC16:22
Nisha_Agarwalor nvdimm_size16:22
jrollNisha_Agarwal: it may be possible, but nova assumes resources are divisible. so for example a node with 2 gpus may be used for a flavor with one gpu16:23
Nisha_Agarwalyes, true16:23
jrollso there's edge cases like that to work out - all in good time16:23
*** jralph has quit IRC16:24
Nisha_Agarwaljroll, so all these cases will work with capabilities also as it has all the operators supported16:24
jrollNisha_Agarwal: yes, the ComputeCapabilitiesFilter still works16:25
jrollthat said, I don't believe that's a sustainable way to do scheduling and it needs to go away in the long term16:25
Nisha_Agarwaljroll, yes so till we have complete solution in ironic we should allow capabilities to be used16:25
jrolland will continue working to handle the use cases for that so we can kill it16:25
jrollNisha_Agarwal: I don't believe we should add first class support for quantitative things in ironic's capabilities dict at this time16:26
jrollit will only make it harder to fix16:26
jroll(I assume you're saying that we should merge your inspector spec)16:27
Nisha_Agarwaljroll, i am not saying merge it as it is...i am trying to find the best solution for quantitative attributes16:27
*** mat128 has quit IRC16:27
Nisha_Agarwalfor the ones you commented on16:27
jrollNisha_Agarwal: my preferred solution is, figure out the nova side first16:28
Nisha_Agarwaljroll, nova already has a schema for resource provider, resource class etc ...is there anything else/extra we need for ironic?16:29
Nisha_Agarwalexcept making changes to ironic virt driver16:29
jrollNisha_Agarwal: we need to solve the problem of nova assuming these resources are divisible, and also agree between nova/ironic what sort of 'standard' resources should be submitted by the ironic virt driver, and decide whether or not more custom resources should be created in a similar way16:30
jrollthe first one is the hard part, I suspect, but I think it's solvable16:31
*** mat128 has joined #openstack-ironic16:31
Nisha_Agarwaljroll, that said we need to have similar schema on ironic side? or it can be different and ironic virt driver handles the coversion to nova acceptable schema16:32
jrollNisha_Agarwal: it could probably be keys in node.properties or something, I haven't thought very much about it16:33
*** derekh has quit IRC16:35
Nisha_Agarwaljroll, ok.16:35
jrollNisha_Agarwal: in the future, could we please hold these bigger-picture discussions on the mailing list? that way other people can chime in and I also don't need to be looking at IRC for 30 minute blocks of time :)16:36
Nisha_Agarwaljroll, thanks.... sure16:36
jrollI'm happy to help, but an unscheduled 30 minute meeting is pretty disruptive to a focused work day :)16:36
Nisha_Agarwaljroll, true...i didnt expected 30 min discussion though :)16:37
jroll:)16:37
*** crushil has joined #openstack-ironic16:37
openstackgerritJoanna Taryma proposed openstack/ironic master: Add base cinder common interface  https://review.openstack.org/35562516:38
mariojvjoanna: ha, i was mid-review on that ^ :)16:40
joannamariojv: sorry :)16:40
mariojvno, don't apologize, looks like you fixed all the things i was going to comment on lol16:40
*** causten has quit IRC16:40
*** kromanenko_ has quit IRC16:41
*** causten has joined #openstack-ironic16:41
*** stendulker has joined #openstack-ironic16:42
*** stendulker has joined #openstack-ironic16:42
*** mat128 has quit IRC16:43
*** causten has quit IRC16:45
*** jrcloud is now known as zackf16:47
*** mgoddard has quit IRC16:48
openstackgerritOpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements  https://review.openstack.org/45247116:49
*** baha has joined #openstack-ironic16:49
* sambetts sambetts -> home 16:50
*** jralph has joined #openstack-ironic16:50
sambettscya'll later16:50
*** sambetts is now known as sambetts|afk16:50
mariojvlater sambetts|afk16:52
*** jralph has quit IRC16:54
*** e0ne has joined #openstack-ironic16:55
*** Syed__ has quit IRC16:55
*** ma91 has quit IRC16:55
*** aarefiev_afk is now known as aarefiev16:56
*** lucasagomes is now known as lucas-afk16:56
*** amarao has joined #openstack-ironic16:56
dtantsurplease double-check me on timezones: meeting in 3 minutes?16:56
mariojvdtantsur: yes16:56
dtantsurthnx16:57
*** vills has quit IRC16:57
*** mat128 has joined #openstack-ironic16:59
*** mgoddard has joined #openstack-ironic17:01
*** nicodemos has joined #openstack-ironic17:03
openstackgerritJoanna Taryma proposed openstack/ironic master: Wire in storage interface attach/detach operations  https://review.openstack.org/40629017:04
openstackgerritJoanna Taryma proposed openstack/ironic master: Add Cinder storage driver  https://review.openstack.org/36619717:04
openstackgerritJoanna Taryma proposed openstack/ironic master: iPXE template support for iSCSI  https://review.openstack.org/41332417:04
*** jkilpatr has quit IRC17:05
*** chlong has quit IRC17:05
*** mgould is now known as mgould|afk17:07
joannamariojv: thank you! :))17:07
*** d0ugal has joined #openstack-ironic17:14
*** mat128 has quit IRC17:15
*** dprince has quit IRC17:17
*** mat128 has joined #openstack-ironic17:17
*** chlong has joined #openstack-ironic17:21
*** amarao has quit IRC17:25
openstackgerritVasyl Saienko proposed openstack/ironic-specs master: Dynamic port groups support  https://review.openstack.org/41500317:25
*** sayantan_ has quit IRC17:28
*** jaosorior is now known as jaosorior_away17:29
vsaienk0jroll: are you working on this? "make a change to the grenade job to only upgrade conductor"17:32
jrollvsaienk0: no, I'm not17:33
openstackgerritMerged openstack/ironic master: Add release note for messaging alias removal  https://review.openstack.org/45278717:33
jrollrloo: dunno if you filed a bug yet, but looks like we're all good on that nova thing - unit tests pass and it isn't used outside of tests17:34
jrolls/nova/oslo/17:34
rloojroll: wow, you're fast. no, didn't file bug so no need to. would you mind updating the status/subteam thing?17:36
jrollrloo: already done :)17:36
rloodang, that jroll is fast!17:37
jrolltime flies when you're in an irc meeting :P17:37
*** jralph has joined #openstack-ironic17:40
*** jralph has quit IRC17:45
*** jralph has joined #openstack-ironic17:45
*** tuanla_ has quit IRC17:46
*** tuan_luong has quit IRC17:46
*** sabeen has quit IRC17:50
*** ijw has quit IRC17:51
aNuposicHi JayF, i had created the releasenote for ceph-radosgw using reno itself17:52
*** bfournie has joined #openstack-ironic17:53
jrollaNuposic: jay is out this week, anything I can help with?17:53
*** lhinds is now known as lhinds|afk17:53
*** causten has joined #openstack-ironic17:53
*** jralph has quit IRC17:53
*** jralph has joined #openstack-ironic17:55
*** causten has quit IRC17:55
aNuposicjroll, oh okay i thought he was going to be around in the morning, he has a comment https://review.openstack.org/#/c/407639/ on releasenotes so just wanted to confirm what exactly he wanted17:56
*** causten has joined #openstack-ironic17:56
aNuposicjroll, if you take a look at ^ would be great17:56
aNuposicjroll, thnx17:56
*** trown|lunch is now known as trown17:56
jrollaNuposic: oh huh, is your reno up to date? that certainly doesn't look like a reno file name17:56
*** mat128 has quit IRC17:57
*** amoralej is now known as amoralej|off17:57
*** baoli has quit IRC17:57
*** Sukhdev has joined #openstack-ironic17:57
aNuposicjroll, i had created it long time back, may be i will create it again17:57
jrollaNuposic: yeah, I'd suggest doing it again :)17:57
aNuposicjroll, yup cool :)17:58
*** mat128 has joined #openstack-ironic17:58
*** bcornec has joined #openstack-ironic17:59
*** d0ugal has quit IRC17:59
*** causten has quit IRC18:00
*** aarefiev is now known as aarefiev_afk18:00
*** jralph has quit IRC18:00
vdrokgood night everyone!18:00
dtantsurg'night vdrok18:00
*** causten has joined #openstack-ironic18:00
dtantsurI am going as well, in summer period this meeting ends at 8pm :)18:01
jrollyuriyz: vdrok: 452182 is great :D18:01
*** baoli has joined #openstack-ironic18:01
*** aNuposic has quit IRC18:01
*** mat128 has quit IRC18:01
* TheJulia goes and grabs some lunch, and goes to makes some quick changes to her test lab18:01
*** jralph has joined #openstack-ironic18:01
* jroll also lunches18:02
*** ChubYann has joined #openstack-ironic18:02
*** dprince has joined #openstack-ironic18:02
*** chas has quit IRC18:04
*** jaganathan has quit IRC18:05
*** causten has quit IRC18:05
*** causten has joined #openstack-ironic18:07
*** stendulker has quit IRC18:07
*** mgoddard has quit IRC18:08
*** jralph has quit IRC18:08
*** mat128 has joined #openstack-ironic18:09
*** harlowja has joined #openstack-ironic18:12
*** Nisha_Agarwal has quit IRC18:13
*** vills has joined #openstack-ironic18:13
*** Nisha_Agarwal has joined #openstack-ironic18:13
*** causten has quit IRC18:14
*** causten has joined #openstack-ironic18:15
*** aNuposic has joined #openstack-ironic18:15
*** dtantsur is now known as dtantsur|afk18:15
*** causten has quit IRC18:16
*** crushil has quit IRC18:16
*** causten has joined #openstack-ironic18:16
aNuposicjroll, yeah i think so i had created it in wrong way last time, no need to give .yaml while creating it with reno. Got slug this time18:17
*** causten has quit IRC18:18
*** d0ugal has joined #openstack-ironic18:20
*** athomas has quit IRC18:22
*** joanna__ has quit IRC18:22
*** ralonsoh has quit IRC18:23
*** mat128 has quit IRC18:26
*** mat128 has joined #openstack-ironic18:27
openstackgerritAnup Navare proposed openstack/ironic master: Config drive support for ceph radosgw  https://review.openstack.org/40763918:27
*** e0ne has quit IRC18:28
jrollaNuposic: cool, glad you got it sorted :)18:33
jrollTheJulia: TIL https://github.com/jriguera/ansible-ironic-standalone18:35
aNuposicjroll, yup :)18:36
aNuposicvdrok, addressed the comments on https://review.openstack.org/407639 , could you please add to your review list18:38
*** sabeen has joined #openstack-ironic18:39
*** crushil has joined #openstack-ironic18:40
*** makowals has joined #openstack-ironic18:44
*** athomas has joined #openstack-ironic18:46
openstackgerritAnup Navare proposed openstack/python-ironicclient master: Add support for RESCUE and UNRESCUE provision states  https://review.openstack.org/40834118:51
*** sayantan_ has joined #openstack-ironic18:51
openstackgerritMario Villaplana proposed openstack/ironic master: WIP Rescue mode tempest tests  https://review.openstack.org/45230818:51
aNuposicmariojv, shall i use the ^ test patch for testing DHCP config patch? have you tested patch for CoreOS?18:53
mariojvaNuposic: go ahead - i doubt it's working as-is though18:56
mariojvaNuposic: that's just a WIP. we're going to make sure that CoreOS is still working with all the updated patch sets right now, then go about fixing those tests if they need updating18:56
aNuposicmariojv, okay, i can wait then for a while, just asked :)18:56
mariojvfeel free to test + help fix it though if you find anything wrong :D18:57
openstackgerritMario Villaplana proposed openstack/ironic-python-agent master: Rescue extension for CoreOS with DHCP tenant networks  https://review.openstack.org/42352118:58
aNuposicmariojv, yup18:58
*** [1]rpioso has joined #openstack-ironic18:58
*** mat128 has quit IRC19:00
*** rpioso has quit IRC19:00
openstackgerritRamamani Yeleswarapu proposed openstack/ironic-inspector master: Unpin python3 version for functional tests  https://review.openstack.org/45290019:03
*** causten has joined #openstack-ironic19:13
*** bcornec has quit IRC19:13
TheJuliajroll: I think there is another one out there too19:16
*** joanna__ has joined #openstack-ironic19:17
*** d0ugal has quit IRC19:23
*** d0ugal has joined #openstack-ironic19:25
*** racedo has joined #openstack-ironic19:29
*** causten has quit IRC19:30
*** causten has joined #openstack-ironic19:30
*** makowals has quit IRC19:32
*** vills has quit IRC19:35
*** d0ugal has quit IRC19:36
*** chlong has quit IRC19:37
*** harlowja has quit IRC19:37
*** nicodemos has quit IRC19:47
*** racedo has quit IRC19:48
*** racedo has joined #openstack-ironic19:50
*** chlong has joined #openstack-ironic19:51
*** causten has quit IRC19:53
*** causten has joined #openstack-ironic19:58
*** sabeen has quit IRC20:00
*** crushil has quit IRC20:06
*** e0ne has joined #openstack-ironic20:08
openstackgerritVasyl Saienko proposed openstack/ironic master: Testing CI  https://review.openstack.org/45291820:08
*** e0ne has quit IRC20:09
*** e0ne has joined #openstack-ironic20:10
*** crushil has joined #openstack-ironic20:10
*** Goneri has quit IRC20:11
*** Sukhdev_ has joined #openstack-ironic20:13
*** crushil has quit IRC20:18
*** causten has quit IRC20:20
jrollvdrok: mind if joanna and I take over this RFE and change it up? https://bugs.launchpad.net/ironic/+bug/152674820:22
openstackLaunchpad bug 1526748 in Ironic "[RFE] General baremetal node auth and token passing mechanism" [Wishlist,Confirmed]20:22
*** baoli has quit IRC20:25
*** jkilpatr has joined #openstack-ironic20:25
*** Nisha_Agarwal has quit IRC20:26
*** Nisha_Agarwal has joined #openstack-ironic20:26
*** causten has joined #openstack-ironic20:28
*** e0ne has quit IRC20:28
*** Sukhdev has quit IRC20:30
*** Sukhdev_ is now known as Sukhdev20:30
*** causten_ has joined #openstack-ironic20:31
*** causten has quit IRC20:32
*** baoli has joined #openstack-ironic20:32
openstackgerritRuby Loo proposed openstack/ironic master: Node should reflect what was saved  https://review.openstack.org/45293020:33
*** Nisha_Agarwal has quit IRC20:38
*** causten_ has quit IRC20:44
*** causten has joined #openstack-ironic20:45
*** [1]rpioso has quit IRC20:48
*** [1]rpioso has joined #openstack-ironic20:49
*** causten has quit IRC20:49
mariojvdoes devstack default to netbooting baremetal instances?20:52
*** harlowja has joined #openstack-ironic20:53
*** causten has joined #openstack-ironic20:53
jrollprobably, I believe that's the current default in config20:54
jrollmariojv: ^20:54
mariojvtesting out the unrescue code again, and for some reason it's trying to netboot back into the deploy image and blowing up20:54
*** trown is now known as trown|outtypewww20:55
mariojvyeah, it is the current default, but it checks capabilities first20:55
mariojvinstance_info['capabilities']20:55
mariojvwas poking around in nova code to see if that got set somewhere during the deploy based on glance image info or something20:55
rloodtantsur|afk: just remembered. we forgot to vote wrt trello. i vote to remove it unless someone wants to maintain/keep it up-to-date.20:56
TheJulia+120:56
mariojvtesting this is going to be a bit more difficult if we have to test with netboot during unrescue :/20:56
*** vgadiraj_ has quit IRC20:57
TheJuliaI seem to remember there is a knob someplace to control that, but I think after writing emails and meetings all day, my brain has melted.20:57
jrollmariojv: I believe nova may set it, instance_info is typically nova20:58
mariojvTheJulia: yeah, the knob is [deploy]/default_boot_option21:00
*** dprince has quit IRC21:00
*** d0ugal has joined #openstack-ironic21:00
*** dprince has joined #openstack-ironic21:01
mariojvi booted a node with the cirros-0.3.5-x86_64-disk image in devstack, went active, rescue failed because i fat fingered config, now trying to unrescue + it's defaulting to try to netboot into that image21:01
mariojvi think either a) the deploy for that happened with netboot, and it's supposed to do that, and something needs to change in rescue code to handle this case better21:02
mariojvor b) the rescue code isn't setting up localboot correctly in that case21:03
* mariojv will keep digging21:03
*** causten has quit IRC21:05
*** baha has quit IRC21:05
*** causten has joined #openstack-ironic21:06
*** athomas has quit IRC21:07
thomasemThank y'all for having a Vagrantfile.21:08
thomasembtw21:08
jrolldoes that still work? lol21:08
thomasemO_O21:09
thomasemI'll let you know.21:09
jrollI mean, I don't see why it wouldn't21:09
thomasemLOL21:09
thomasemMhmm, sure.21:09
thomasemI'm giving it a go.21:09
thomasemI'll file bugs if I see issues.21:09
jrollcool, I guess it depends what you're trying to do. it should work for basic unit tests or running api21:10
jrolls/file/fix/ :D21:10
thomasem:P21:10
thomasemAhhh, wasn't sure how far it went, so that's good to know.21:10
*** causten has quit IRC21:10
thomasemStarting point, I guess.21:10
jrollit installs mysql/rabbit/ironic21:10
jrolland configs21:10
jrollso things depending on neutron wouldn't work, for example21:11
thomasemGotcha. So, is the recommendation to get Devstack going with Ironic and such, then?21:11
thomasemAhhhh21:11
thomasemOkay21:11
jrollyeah, if you want to provision machines and such I'd use devstack21:11
jrollit sets up some kvm VMs as fake baremetal machines21:11
jrollhttps://docs.openstack.org/developer/ironic/dev/dev-quickstart.html#deploying-ironic-with-devstack21:11
thomasemjroll: Makes sense. :) Thank you muchly21:12
jrollya, no problem :)21:12
*** dprince has quit IRC21:14
*** srobert has quit IRC21:18
*** ijw has joined #openstack-ironic21:25
*** zer0c00l has joined #openstack-ironic21:27
*** causten has joined #openstack-ironic21:30
alezilcd ls21:31
*** snecklifter has quit IRC21:32
*** Goneri has joined #openstack-ironic21:33
*** snecklifter has joined #openstack-ironic21:34
*** causten has quit IRC21:35
thomasem.21:37
thomasem..21:37
TheJulia:)21:38
*** sayantan_ has quit IRC21:39
thomasem:P21:41
*** sayantan_ has joined #openstack-ironic21:46
*** racedo has quit IRC21:47
*** Sukhdev_ has joined #openstack-ironic21:47
TheJuliaGoodnight folks21:49
thomasemCheers!21:50
openstackgerritOpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements  https://review.openstack.org/45247121:51
*** sayantan_ has quit IRC21:57
*** sabeen has joined #openstack-ironic21:58
openstackgerritVasyl Saienko proposed openstack/ironic master: Testing CI  https://review.openstack.org/45291822:00
*** bfournie has quit IRC22:00
*** jralph has joined #openstack-ironic22:01
*** rcernin has quit IRC22:12
*** Goneri has quit IRC22:15
*** mat128 has joined #openstack-ironic22:17
*** sabeen has quit IRC22:21
*** baoli has quit IRC22:24
*** Sukhdev has quit IRC22:26
*** sabeen has joined #openstack-ironic22:27
jrollnight y'all22:29
aNuposicnight jroll22:31
*** chlong has quit IRC22:38
*** Goneri has joined #openstack-ironic22:41
*** ijw has quit IRC22:45
joannahi, it looks like the gate is broken for gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode-ubuntu-xenial22:45
*** ijw has joined #openstack-ironic22:45
joannaI observe failures for ~4 hours already22:45
*** Goneri has quit IRC22:46
*** ijw has quit IRC22:47
*** crushil has joined #openstack-ironic22:47
*** ijw has joined #openstack-ironic22:47
TheJulia:(22:51
*** bfournie has joined #openstack-ironic22:52
*** ijw has quit IRC22:52
*** bfournie has quit IRC22:53
joannaTheJulia: I updated the whiteboard, I see the common stacktrace in the logs for this failure, it seems to fail on subnet deletion due to exisiting port allocations.22:57
TheJuliaThank you joanna22:58
*** ijw has joined #openstack-ironic23:03
*** ijw has quit IRC23:03
*** ijw has joined #openstack-ironic23:04
*** mat128 has quit IRC23:07
*** ijw has quit IRC23:09
*** rbudden has quit IRC23:14
*** sayantan_ has joined #openstack-ironic23:17
*** sayantan_ has quit IRC23:30
openstackgerritJulian Edwards proposed openstack/ironic-python-agent master: WIP: Prevent tests from using utils.execute()  https://review.openstack.org/45260323:37
*** mat128 has joined #openstack-ironic23:43
openstackgerritJulian Edwards proposed openstack/ironic-python-agent master: Prevent tests from using utils.execute()  https://review.openstack.org/45260323:43
*** [1]rpioso has quit IRC23:56
*** mat128 has quit IRC23:58

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