Monday, 2017-03-13

*** pmannidi has joined #openstack-ironic00:04
*** gfhellma has joined #openstack-ironic00:10
*** sayantan_ has joined #openstack-ironic00:12
*** baoli has quit IRC00:13
*** baoli has joined #openstack-ironic00:28
*** baoli has quit IRC00:28
*** fragatina has joined #openstack-ironic00:43
*** SerenaFeng has joined #openstack-ironic01:00
*** wrouesnel has joined #openstack-ironic01:03
*** phuongnh has joined #openstack-ironic01:06
*** zhurong has joined #openstack-ironic01:11
openstackgerritMatt Riedemann proposed openstack/ironic master: Deprecate support for glance v1  https://review.openstack.org/44149101:14
*** zhangjl has joined #openstack-ironic01:22
openstackgerritMatt Riedemann proposed openstack/ironic master: Deprecate support for glance v1  https://review.openstack.org/44149101:26
openstackgerritMatt Riedemann proposed openstack/ironic master: Update ironic.conf.sample  https://review.openstack.org/44468301:26
*** baoli has joined #openstack-ironic01:29
*** amotoki has quit IRC01:30
*** baoli has quit IRC01:34
*** mtanino has joined #openstack-ironic01:40
*** amotoki has joined #openstack-ironic01:43
*** gcb has joined #openstack-ironic01:44
*** yonglihe has joined #openstack-ironic01:46
*** Lambert has joined #openstack-ironic01:46
Lamberthello01:46
*** amotoki has quit IRC01:49
Lambertanybody knows how to import ddt in test code01:50
*** baoli has joined #openstack-ironic01:50
*** cppforlife_ has quit IRC01:51
*** sasha2 has joined #openstack-ironic01:53
*** amotoki has joined #openstack-ironic01:55
openstackgerritTao Li proposed openstack/ironic master: Address a shell syntax mistake  https://review.openstack.org/44354601:55
*** baoli has quit IRC01:55
*** sasha2 has quit IRC01:58
*** SerenaFeng has quit IRC02:03
*** SerenaFeng has joined #openstack-ironic02:08
*** amotoki has quit IRC02:19
*** SerenaFeng has quit IRC02:22
*** hw_wutianwei has joined #openstack-ironic02:24
*** amotoki has joined #openstack-ironic02:29
*** amotoki has quit IRC02:30
*** amotoki has joined #openstack-ironic02:30
*** baoli has joined #openstack-ironic02:51
*** baoli has quit IRC02:56
*** tuanluong has joined #openstack-ironic03:06
*** rbudden has quit IRC03:24
*** vishwanathj has quit IRC03:30
openstackgerritJun Yao proposed openstack/ironic-inspector master: Adds node state to the API response data  https://review.openstack.org/43641503:31
*** mtanino has quit IRC03:40
*** ianychoi has joined #openstack-ironic03:46
*** aparnav has joined #openstack-ironic03:48
*** sayantan_ has quit IRC03:51
*** aparnav has quit IRC03:54
*** frontrunner has quit IRC03:54
*** nmathew has joined #openstack-ironic03:57
*** nmathew- has joined #openstack-ironic03:59
*** sayantan_ has joined #openstack-ironic03:59
*** links has joined #openstack-ironic04:02
*** nmathew has quit IRC04:02
*** sayantan_ has quit IRC04:04
*** fragatina has quit IRC04:10
*** chlong_ has quit IRC04:34
*** sayantan_ has joined #openstack-ironic04:38
*** sayantan_ has quit IRC04:42
*** nmathew- has quit IRC04:49
*** sayantan_ has joined #openstack-ironic04:50
*** baoli has joined #openstack-ironic04:53
*** aparnav has joined #openstack-ironic04:54
*** sayantan_ has quit IRC04:55
*** baoli has quit IRC04:57
*** dsneddon has joined #openstack-ironic04:58
*** nmathew has joined #openstack-ironic05:15
*** vikrant has joined #openstack-ironic05:36
*** dsneddon has quit IRC05:45
*** jaganathan has joined #openstack-ironic05:47
*** dsneddon has joined #openstack-ironic05:53
*** baoli has joined #openstack-ironic05:54
openstackgerritJaganathan Palanisamy proposed openstack/ironic-python-agent master: NUMA-topology collector  https://review.openstack.org/42472905:55
*** baoli has quit IRC05:58
*** tovin07 has joined #openstack-ironic06:00
*** yogi has joined #openstack-ironic06:26
*** wrouesnel has quit IRC06:41
*** baoli has joined #openstack-ironic06:54
*** baoli has quit IRC06:59
*** vills has joined #openstack-ironic07:00
*** vills has quit IRC07:06
*** venki has quit IRC07:11
openstackgerritM V P Nitesh proposed openstack/ironic master: Validate ipmi_address format on ipmitool power interface validation  https://review.openstack.org/43694507:17
openstackgerritJun Yao proposed openstack/ironic-inspector master: Adds node state to the API response data  https://review.openstack.org/43641507:23
*** venki has joined #openstack-ironic07:28
*** ChubYann has quit IRC07:30
*** amarao has joined #openstack-ironic07:43
*** ccamacho has joined #openstack-ironic07:43
*** jtomasek has joined #openstack-ironic07:45
*** rbartal has joined #openstack-ironic07:46
*** ralonsoh has joined #openstack-ironic07:51
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:00
*** hshiina has joined #openstack-ironic08:00
openstackgerritOksana Voshchana proposed openstack/ironic master: Adds use wsgi without port  https://review.openstack.org/44433708:07
*** tovin07 has quit IRC08:07
*** mjura has joined #openstack-ironic08:07
openstackgerritOksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi  https://review.openstack.org/43729908:08
*** tovin07 has joined #openstack-ironic08:12
*** tesseract has joined #openstack-ironic08:13
*** vills has joined #openstack-ironic08:14
pas-hamorning ironic :)08:17
*** tesseract has quit IRC08:17
*** openstackgerrit has quit IRC08:18
*** pmannidi has quit IRC08:22
tuanluongmroning ironic08:23
tuanluongMornig pas-ha08:24
pas-hamorning tuanluong :)08:24
*** pcaruana has joined #openstack-ironic08:25
*** tesseract has joined #openstack-ironic08:32
*** openstackgerrit has joined #openstack-ironic08:33
openstackgerritVasyl Saienko proposed openstack/ironic master: Add Ironic standalone tests  https://review.openstack.org/42355608:33
vsaienk0morning Ironic'ers!08:38
*** e0ne has joined #openstack-ironic08:43
*** jaosorior has joined #openstack-ironic08:57
*** alineb has joined #openstack-ironic09:01
*** dtantsur|afk is now known as dtantsur09:05
dtantsurMorning ironic. I hope you had a great weekend!09:05
*** yibo has quit IRC09:06
*** tuanluong has quit IRC09:09
amaraoHello everyone? Did anyone knows what had happens with diskimage-builder? I've tried to upgrade to 2.0 and found that it's basically, gone. It wants some bash scripts which are not in the repo anymore...09:10
*** yibo has joined #openstack-ironic09:14
openstackgerritM V P Nitesh proposed openstack/ironic master: Validate ipmi_address format on ipmitool power interface validation  https://review.openstack.org/43694509:17
*** mgoddard has joined #openstack-ironic09:18
dtantsuramarao, hey! I don't think people here know much about DIB, there was a dedicated channel (can't remember its name). I know that 2.0 is nearly a rewrite of it.09:19
openstackgerritAnton Arefiev proposed openstack/ironic-inspector master: Starting -> Error transaction refactoring  https://review.openstack.org/43034109:19
*** aarefiev_afk is now known as aarefiev09:19
amaraodtantsur: Oh, thanks.09:19
dtantsuramarao, I suggest you stick with <2.0 until you're ready to switch. I haven't looked into new version either.09:19
*** lucas-afk is now known as lucasagomes09:20
lucasagomesmorning all09:20
aarefievmorning all!09:21
dtantsuramarao, https://docs.openstack.org/releasenotes/diskimage-builder/#upgrade-notes09:21
dtantsurmorning lucasagomes, aarefiev09:21
aarefievmorning dtantsur, lucasagomes09:22
lucasagomeso/09:22
*** racedo has joined #openstack-ironic09:22
*** racedo has quit IRC09:22
dtantsur"another feature of v2 is override elements – an element that appears first in the element path-list will override any built-in one (just like $PATH)" OH FINALLY!09:23
dtantsurmeaning, we can create an element called ironic-agent now in-tree :)09:23
dtantsurJayF, ^^^09:23
*** athomas has joined #openstack-ironic09:24
dtantsuramarao, ah, found: the channel is #openstack-dib09:24
openstackgerritAnton Arefiev proposed openstack/ironic-inspector master: Add new transaction starting -> error on timeout  https://review.openstack.org/43737709:26
pas-haamarao: how do you run dib? from 2.0.0 release notes "if you run it directly from a git-tree checkout, you may be affected"09:27
pas-hamorning dtantsur :)09:27
pas-hamorning lucasagomes09:28
amaraopas-ha: Well, we use it as debian package. I rebuild package for tag 2.0.0 and after installation I got: https://bugs.launchpad.net/diskimage-builder/+bug/1672304 (/usr/bin/disk-image-create09:28
openstackLaunchpad bug 1672304 in diskimage-builder "bash: /usr/lib/python2.7/dist-packages/diskimage_builder/lib/disk-image-create: No such file or directory" [Undecided,New]09:28
amaraobash: /usr/lib/python2.7/dist-packages/diskimage_builder/lib/disk-image-create: No such file or directory)09:28
amaraoAnd the issue was that there was no 'bash script' to run anywhere in the git.09:28
amaraoI already rolled back to 1.9.009:29
dtantsurworth asking on #openstack-dib, but I think the packaging has to be changed..09:30
*** tovin07 has quit IRC09:39
*** derekh has joined #openstack-ironic09:42
openstackgerritLucas Alvares Gomes proposed openstack/ironic master: Add redfish driver  https://review.openstack.org/43898209:42
openstackgerritLucas Alvares Gomes proposed openstack/ironic master: Add documentation for the redfish driver  https://review.openstack.org/44253709:42
*** athomas has quit IRC09:45
*** athomas has joined #openstack-ironic09:45
*** vmud213 has joined #openstack-ironic09:51
*** baoli has joined #openstack-ironic09:52
*** e0ne has quit IRC09:52
*** baoli has quit IRC09:56
*** MattMan has quit IRC10:05
*** MattMan has joined #openstack-ironic10:06
*** e0ne has joined #openstack-ironic10:09
openstackgerritMerged openstack/ironic master: Do not load credentials on import in tempest plugin clients.py  https://review.openstack.org/44431510:10
*** hshiina has quit IRC10:10
openstackgerritOksana Voshchana proposed openstack/ironic master: Adds use wsgi without port  https://review.openstack.org/44433710:17
openstackgerritOksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi  https://review.openstack.org/43729910:18
vdrokmorning ironic!10:19
*** zhurong has quit IRC10:21
*** zhurong has joined #openstack-ironic10:22
*** Lambert has quit IRC10:23
*** alexpilotti has joined #openstack-ironic10:24
openstackgerritOksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi  https://review.openstack.org/43729910:25
*** phuongnh has quit IRC10:29
*** hwoarang_ is now known as hwoarang10:33
lucasagomesvdrok, pas-ha o/ morning10:37
vdrokmorning lucasagomes :)10:37
*** lifeless has quit IRC10:37
*** ralonsoh_ has joined #openstack-ironic10:41
*** zhugaoxiao has quit IRC10:43
*** zhugaoxiao has joined #openstack-ironic10:44
*** ralonsoh has quit IRC10:44
*** lifeless has joined #openstack-ironic10:45
*** ralonsoh_ is now known as ralonsoh10:45
openstackgerritMerged openstack/ironic master: Update ironic.conf.sample  https://review.openstack.org/44468310:51
*** baoli has joined #openstack-ironic10:53
openstackgerritOksana Voshchana proposed openstack/ironic master: Adds use wsgi without port  https://review.openstack.org/44433710:55
openstackgerritOksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi  https://review.openstack.org/43729910:55
*** alexpilotti has quit IRC10:56
*** baoli has quit IRC10:57
openstackgerritMerged openstack/ironic master: Remove overwriting the default value of db_max_retries.  https://review.openstack.org/44086510:58
openstackgerritiswarya vakati proposed openstack/bifrost master: Pass located tests directory in oslo debug  https://review.openstack.org/44481110:58
openstackgerritiswarya vakati proposed openstack/bifrost master: Pass located tests directory in oslo debug  https://review.openstack.org/44481111:01
*** alexpilotti has joined #openstack-ironic11:01
openstackgerritPavlo Shchelokovskyy proposed openstack/bifrost master: Always write clouds.yaml  https://review.openstack.org/44482011:09
openstackgerritKyrylo Romanenko proposed openstack/ironic master: Add functional API tests for portgroups  https://review.openstack.org/43687811:10
openstackgerritGalyna Zholtkevych proposed openstack/ironic-specs master: API Evolution - ETAG identifiers  https://review.openstack.org/38199111:11
*** amotoki_ has joined #openstack-ironic11:16
*** jkilpatr has joined #openstack-ironic11:17
*** rbudden has joined #openstack-ironic11:17
*** amotoki has quit IRC11:19
*** jcoufal has joined #openstack-ironic11:20
openstackgerritGalyna Zholtkevych proposed openstack/ironic-specs master: API Evolution - ETAG identifiers  https://review.openstack.org/38199111:22
openstackgerritzack chen proposed openstack/ironic-python-agent master: Fix logs encoding in py3  https://review.openstack.org/43890111:25
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient master: Add negative tests for baremetal node commands  https://review.openstack.org/33771911:25
*** jkilpatr has quit IRC11:25
*** aparnav has quit IRC11:31
*** vills has quit IRC11:32
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient master: Add tests for node list and show with specific fields  https://review.openstack.org/38492311:33
*** gcb has quit IRC11:35
openstackgerritVasyl Saienko proposed openstack/ironic master: Add Ironic standalone tests  https://review.openstack.org/42355611:36
openstackgerritOksana Voshchana proposed openstack/ironic master: Adds use wsgi without port  https://review.openstack.org/44433711:36
galynaHello, all! :) I have a question about nookeepalive option https://bugs.launchpad.net/ironic/+bug/1618842  I know that on PTG this issue was proposed to fix in another cleaner way. Are there any updates? I could take it, cause I need to report about this bug11:37
openstackLaunchpad bug 1618842 in Ironic "[RFE] Support nokeepalive for serial console in the ipmitool driver" [Wishlist,In progress] - Assigned to Galyna Zholtkevych (gzholtkevych)11:37
openstackgerritOksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi  https://review.openstack.org/43729911:37
*** nmathew has quit IRC11:37
openstackgerritAnton Arefiev proposed openstack/ironic-inspector master: Starting -> Error transaction refactoring  https://review.openstack.org/43034111:38
*** jkilpatr has joined #openstack-ironic11:39
dtantsurgalyna, I don't remember discussing this particular issue on the PTG. we were rather talking about how to run console processes at all IIRC11:39
*** hshiina has joined #openstack-ironic11:39
galynaI suppose this could cover this issue11:39
galynadtantsur: or tell if I am mistaken, so I will work with it in some another way :)11:40
dtantsurI don't think so, to be honest11:40
galynaThanks for the answer11:42
*** zhangjl has quit IRC11:42
openstackgerritVladyslav Drok proposed openstack/ironic master: Refactor waiters in our tempest plugin  https://review.openstack.org/41068411:44
*** bfournie has quit IRC11:45
*** baoli has joined #openstack-ironic11:53
*** dprince has joined #openstack-ironic11:55
*** baoli has quit IRC11:58
*** xavierr has quit IRC12:08
*** jcoufal has quit IRC12:08
*** athomas has quit IRC12:12
dtantsurfolks, we have a lot of new subteams on the whiteboard. please provide a status update before the meeting. thanks!12:12
openstackgerritJaganathan Palanisamy proposed openstack/ironic-python-agent master: NUMA-topology collector  https://review.openstack.org/42472912:12
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient master: Add tests for node list and show with specific fields  https://review.openstack.org/38492312:15
*** bfournie has joined #openstack-ironic12:21
*** athomas has joined #openstack-ironic12:26
*** trown|outtypewww is now known as trown12:35
*** jheroux has joined #openstack-ironic12:35
openstackgerritOksana Voshchana proposed openstack/ironic master: Adds use wsgi without port  https://review.openstack.org/44433712:37
openstackgerritOksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi  https://review.openstack.org/43729912:38
* dtantsur looks at http://ci-watch.tintri.com/project?project=ironic&time=7+days and realized he has to chat with some CI folks..12:43
openstackgerritDao Cong Tien proposed openstack/python-ironicclient master: Extends driver-list, driver-show supporting new hardware types  https://review.openstack.org/41927412:44
openstackgerritPeter Piela proposed openstack/ironic-ui master: Add support for starting/stopping/accessing the serial console  https://review.openstack.org/43741912:55
openstackgerritPeter Piela proposed openstack/ironic-ui master: Add support for soft power transitions  https://review.openstack.org/44488712:55
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362812:58
*** sasha2 has joined #openstack-ironic13:01
*** vmud213 has quit IRC13:01
*** zhurong has quit IRC13:01
*** jaganathan has quit IRC13:06
*** [1]cdearborn has joined #openstack-ironic13:07
*** vsaienko has joined #openstack-ironic13:09
*** gcb has joined #openstack-ironic13:14
openstackgerritKyrylo Romanenko proposed openstack/ironic master: Add functional API tests for portgroups  https://review.openstack.org/43687813:19
*** g3ek has quit IRC13:20
*** zackf has joined #openstack-ironic13:20
*** zackf has quit IRC13:21
*** zackf has joined #openstack-ironic13:21
*** vsaienko has quit IRC13:24
*** _vsaienko has joined #openstack-ironic13:24
*** rloo has joined #openstack-ironic13:26
*** zackf has quit IRC13:26
*** g3ek has joined #openstack-ironic13:29
TheJuliao/13:32
TheJuliaGood morning folks, I'm back, just doing my expense reporting for last week first thing13:32
*** jcoufal has joined #openstack-ironic13:32
*** _vsaienko has quit IRC13:32
openstackgerritSofiia Andriichenko proposed openstack/ironic master: This adds a tempest test for creating a chassis with a specific UUID.  https://review.openstack.org/40919613:33
*** lucasagomes is now known as lucas-hungry13:35
*** frontrunner has joined #openstack-ironic13:37
*** Goneri has joined #openstack-ironic13:40
*** venki has quit IRC13:41
*** venki has joined #openstack-ironic13:42
openstackgerritAnton Arefiev proposed openstack/ironic-inspector master: Use latest API microversion in unit tests  https://review.openstack.org/44491013:46
dtantsurmorning TheJulia!13:47
*** bcornec2 has joined #openstack-ironic13:48
aarefievdtantsur : some magic for latest api version in unit tests ^^, but looks ugly13:48
aarefievmorning TheJulia13:48
dtantsuraarefiev, don't we support "latest" value? hmm13:49
aarefievdtantsur: do we?13:49
dtantsurno idea :)13:50
aarefievgrep by `latest` find nothing13:51
*** baoli has joined #openstack-ironic13:52
*** gcb has quit IRC13:52
aarefievdtantsur: I don't think we are supporting latest, worth to file a rfe if we don't have one13:52
dtantsuryeah13:57
*** leakypipes is now known as jaypipes14:00
*** baoli has quit IRC14:01
*** baoli has joined #openstack-ironic14:02
jrollmorning y'all14:04
* jroll slowly catching up14:04
*** amoralej is now known as amoralej|lunch14:04
*** amotoki_ has quit IRC14:05
vsaienk0lucas-hungry: I've resolved your comments https://review.openstack.org/#/c/423556/14:06
patchbotpatch 423556 - ironic - Add Ironic standalone tests14:06
*** gcb has joined #openstack-ironic14:07
*** _vsaienko has joined #openstack-ironic14:07
*** baoli has quit IRC14:08
*** chlong has joined #openstack-ironic14:11
*** sayantan_ has joined #openstack-ironic14:15
*** Goneri has quit IRC14:15
*** links has quit IRC14:16
*** _vs_ has joined #openstack-ironic14:17
*** _vsaienko has quit IRC14:17
*** Goneri has joined #openstack-ironic14:19
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362814:21
vdrokmorning TheJulia and jroll14:21
*** venki has quit IRC14:21
jroll\o14:22
*** baoli has joined #openstack-ironic14:22
*** hshiina has quit IRC14:22
*** baoli has quit IRC14:22
*** venki has joined #openstack-ironic14:23
*** _vsaienko has joined #openstack-ironic14:27
*** _vs_ has quit IRC14:27
*** _vsaienko has quit IRC14:27
openstackgerritKyrylo Romanenko proposed openstack/ironic master: Add functional api tests for node resource class  https://review.openstack.org/43796314:28
dtantsurmorning jroll14:29
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362814:30
*** udesale has joined #openstack-ironic14:30
vsaienk0jroll, dtantsur could you please add to your review queue https://review.openstack.org/#/c/42472314:31
patchbotpatch 424723 - ironic - Allow to attach/detach VIFs to active ironic nodes14:31
dtantsurk14:31
vsaienk0sambetts_:  ^14:31
jrollvsaienk0: sure, thanks14:31
vsaienk0thanks!14:32
*** baoli has joined #openstack-ironic14:32
pas-haTheJulia: morning14:36
*** lucas-hungry is now known as lucasagomes14:37
lucasagomesvsaienk0, thanks man, I will take a look at it soon-ish14:38
vsaienk0thanks lucasagomes14:39
pas-haTheJulia: I'm trying to setup bifrost thru our 'experimental' virtualenv feature and find numerous bugs/obstacles... e.g. ironic-eroll-dynamic role does not work when bifrost has installed ironic with 'enable_venv'. Do you have any context as for why do we need this feature at all? I tend to propose dropping it altogether...14:39
*** vikrant has quit IRC14:40
*** frontrunner has quit IRC14:41
openstackgerritMiles Gould proposed openstack/ironic-ui master: Fix typos in docstrings  https://review.openstack.org/44376214:41
openstackgerritMiles Gould proposed openstack/ironic-ui master: Fix docstrings for node validation  https://review.openstack.org/44376014:44
*** amoralej|lunch is now known as amoralej14:46
*** baoli has quit IRC14:55
*** baoli_ has joined #openstack-ironic14:55
TheJuliapas-ha: why doesn't it work? the context is automated bulk addition of nodes into ironic14:58
*** mgould|afk is now known as mgould14:59
pas-hait tries to execute os_client_config module but the os-client-config is actually in the venv https://github.com/openstack/bifrost/blob/master/playbooks/roles/ironic-enroll-dynamic/tasks/main.yml#L2515:00
pas-haTheJulia: and by 'context' I meant 'install services in venvs' feature :) not dynamic enroll  :)15:01
TheJuliapas-ha: I just realized that15:01
*** venki has quit IRC15:02
TheJuliaSo, I think part of the conundrum is that shade/os_client_config needs to get installed wherever ansible will execute the modules, sounds like it is not with the isolation in the venv.  Kind of soudns like a simple fix actually, as long as it is being installed on the target host.15:02
pas-haok, will try to fix that15:03
*** venki has joined #openstack-ironic15:03
*** sayantan_ has quit IRC15:05
mgouldhi everyone15:06
rloohi and Happy Monday morning all, pas-ha, TheJulia & jroll (welcome back), vsaienk0, lucasagomes, dtantsur, vdrok, aarefiev15:06
mgoulddoes our API accept a node name everywhere it would accept a UUID?15:06
mgouldcontext: https://review.openstack.org/#/c/443760/2/ironic_ui/api/ironic.py@17915:06
patchbotpatch 443760 - ironic-ui - Fix docstrings for node validation15:06
rloohi mgould15:06
mgouldmorning rloo15:06
lucasagomesrloo, morning o/15:06
pas-hamorning rloo :)15:06
aarefievmorning rloo :)15:06
rloomgould: i think it does. for all the ironic node commands. but not totally sure if we got them all.15:07
rloomgould: i mean, we should and i think we do but not positive :)15:07
vdrokgood morning rloo !15:08
dtantsurmorning rloo :)15:08
TheJuliappiela: I have two minor asks on 437419, they should be super quick when you have a minute15:08
mgouldrloo: thanks!15:08
*** sayantan_ has joined #openstack-ironic15:09
rloojroll: when you have a chance, wanted to know if you had thought about classic drivers and how to get driver-properties. see https://bugs.launchpad.net/ironic/+bug/167154915:09
openstackLaunchpad bug 1671549 in Ironic "[RFE] API to get driver properties for a dynamic driver + non-default interfaces" [High,New] - Assigned to Ruby Loo (rloo)15:09
vsaienk0morning rloo15:10
ppielaTheJulia: Thanks Julia, I will deal them15:10
TheJuliamgould: not everywhere, but in most places it does as I understand it15:11
dtantsura reminder #2: we got many more subteams, please provide status updates (even if it's just "nothing yet")15:11
TheJuliamgould: the to further add to confusion, id internally to the api/conductor is database row id typically. :)15:11
openstackgerritMerged openstack/ironic-ui master: Fix typos in docstrings  https://review.openstack.org/44376215:12
dtantsurmgould, I think the notable exception is port creation, where we only accept UUID. I believe there is a patch up to address it15:12
* dtantsur brb15:12
*** udesale has quit IRC15:13
openstackgerritPeter Piela proposed openstack/ironic-ui master: Add support for starting/stopping/accessing the serial console  https://review.openstack.org/43741915:13
mjturekdtantsur: just wanted to let you know that I have been watching the bug list but I don't have permission to change importance. I've been categorizing bugs here https://etherpad.openstack.org/p/ironic-bug-triage15:14
rloomjturek: you need to be a member of https://launchpad.net/~ironic-bugs15:16
mjturekrloo: oh cool I didn't know!15:16
*** sayantan_ has quit IRC15:16
rloomjturek: i think it is simple to join but i don't recall how. hopefully there is a link somewhere...15:16
mjturekrloo: just one click :)15:17
ppielaTheJulia: I am only seeing an ask for Closes-Bug, am I missing something?15:17
TheJuliappiela: release note15:17
rloomjturek: easy peasy15:17
*** zackf has joined #openstack-ironic15:18
ppielaTheJulia: It's there15:20
* TheJulia raises an eyebrow and goes and looks15:20
jrollrloo: I haven't thought about it much, honestly. off the top of my head perhaps we can do a /v1/drivers/power/ipmi/properties or so15:20
rloojroll: too bad. i mean, that you haven't thought about it much, cuz i don't want to but we need to :)15:21
TheJuliappiela: I see you added it in patchset 7, thank you15:21
*** gfhellma has quit IRC15:21
rloojroll: dtantsur threw some thoughts etc in https://etherpad.openstack.org/p/ironic-driver-properties-reform15:21
rloojroll: guess i'll spend some time thinking about it this week15:21
jrollrloo: yes, I see that, I won't have time to think about it today, maybe this week15:21
ppielaTheJulia: Phew, you had me worried15:21
rloojroll: no big hurry, but we need to decide on this, to make sure the osc-related commands for driver properties works with the new api, etc.15:22
*** beekneemech is now known as bnemec15:23
jlvillalGood morning Ironic15:23
jrollrloo: idk, I think we should be able to separate the two15:23
* jlvillal wonders why the patch he just pushed didn't get announced...15:23
rloojroll: 'should be'. quite possibly, but why take the chance.15:24
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362815:25
*** fragatina has joined #openstack-ironic15:25
vdrokmorning jlvillal15:26
* jlvillal finds out that openstackgerrit isn't yet configured to announce stable/ocata. He will work on a patch15:26
jlvillalvdrok: :)15:27
*** baoli_ has quit IRC15:27
*** amotoki has joined #openstack-ironic15:28
TheJuliappiela: Sorry for causing you to be worried, there is a distinct possibility that my brain is going to be a blended by this afternoon.15:29
TheJuliaLooking at the main ironic meeting agenda... oh my15:29
*** ianychoi has quit IRC15:29
*** rpioso has joined #openstack-ironic15:30
*** baoli has joined #openstack-ironic15:31
openstackgerritJohn L. Villalovos proposed openstack/ironic stable/newton: grenade: Only 'enable_plugin ironic' if not already in conf  https://review.openstack.org/44495015:33
*** jvgrant has joined #openstack-ironic15:33
jrollTheJulia: indeed, wow15:34
jrollsome of those feel like emails15:34
*** trown is now known as trown|brb15:34
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362815:35
TheJulia+115:35
TheJuliaAs meeting agenda items, I'm not sure we will actually get done with them in an hour15:35
JayFI think some of those are misleading.15:36
JayFLike I expect jlvillal's item, +1s all around and over15:36
JayFbecause it seems straightforward to me15:36
JayFidk15:36
JayFI agree some of it seems like overkill for the meeting, but it shouldn't take long15:37
JayF/optimizism15:37
openstackgerritMiles Gould proposed openstack/ironic-ui master: s/UUID/UUID or name/ in docstrings  https://review.openstack.org/44495415:37
*** srart has joined #openstack-ironic15:38
TheJuliaYeah, hopefully everything will be nice and quick, and if they are then we've learned we can cover more per meeting! :)15:39
jrollJayF: you underestimate our ability to talk about topics way longer than needed15:40
*** ianychoi has joined #openstack-ironic15:40
JayFwell that took all my optimism for the week15:41
JayF;)15:41
TheJuliadtantsur: Thank you for updating the links last week.  My return travel encountered weather and it... was.... a... very... long trip.15:41
*** rbartal has quit IRC15:41
TheJuliaJayF: w/r/t doc stuff, Are you still hoping to post a spec this week?15:41
JayFdsfdsgjkdflgjsdgfd15:42
JayFsorry, lost my ssh connection, lol15:42
rloojroll, JayF: so why do we talk about topics way longer than needed? :) thoughts as to how we can speed things up?15:42
*** karthiks is now known as karthiks_afk15:43
* rloo puts down the ironic logo in meeting agenda15:43
TheJuliaJayF: :)15:43
* rloo kidding15:43
mgouldrloo: make everyone drink at least a litre of fluid before the meeting starts15:43
JayFTheJulia: I have about 4-5 things on my list, all of which can't get done this week, and as a bonus I'm out Fri-Weds15:43
TheJuliarloo: I still think my idea of handling bike shedding could be useful for meeting agendas.15:43
TheJuliaJayF: 10-4, I'll update the whiteboard with no updates15:44
JayFTheJulia: if you can help make a case for docs spec it might ascend past "start API code for faults"15:44
JayFoh, whiteboard!15:44
* JayF goes and updates his stuff15:44
* TheJulia thinks multiple shots of espresso need to be prepared now15:45
rlooTheJulia: :) We are all part of the problem, since presumably 'we' are the ones talking...?15:45
TheJuliarloo: Indeed... and hot tubs would severely restrict the length of bike shedding... or stop it in it's tracks.15:46
rlooTheJulia: i think hot tubs would prolong it. Being in a freezer would be better :)15:47
*** snecklifter has quit IRC15:47
mgouldhot tubs whose temperature gradually increases15:47
jlvillalI would think the meeting chair (dtantsur) should be allowed to order the topics as they desire.15:47
JayFrloo: that's what they did to that poor blue polar bear logo for freezer team :P15:47
mgoulddownside: all decisions get made in favour of whoever has the greatest heat tolerance15:48
*** snecklifter has joined #openstack-ironic15:48
rlooJayF: :)15:48
*** trown|brb is now known as trown15:48
*** ccamacho has quit IRC15:48
*** amarao has quit IRC15:49
*** ccamacho has joined #openstack-ironic15:49
* mgould wonders how good dtantsur's tolerance to heat and cold is15:49
mgouldon the one hand, Russian, but on the other, high surface-area-to-volume ratio15:49
rlooso for instance, looking at the agenda, it isn't clear, what and when things should be brought up at the meeting.15:49
rloojlvillal: wrt your enable-plugin-ironic topic. would it make sense to have an rfe for it? or some place where what you are proposing is documented. one thing i don't like about our agenda, is that it is in a wiki and is lost.15:52
jlvillalrloo: I don't know. Not sure it requires an RFE. In  face it might not need to be on the agenda.15:52
jlvillalrloo: Before I had a much uglier way I was proposing that was going to cause us to have to break the gate for awhile.15:53
NobodyCamGood Morning Ironic'ers15:53
jlvillalrloo: But I figured out a better way, which is documented on the wiki. So no gate-breakage.15:53
jlvillalNobodyCam: Hi!15:53
NobodyCammorning jlvillal :)15:53
openstackgerritMiles Gould proposed openstack/ironic-ui master: s/UUID/UUID or name/ in docstrings  https://review.openstack.org/44495415:53
rloojlvillal: it seems like it would be nice to document these things outside the wiki. we don't open bugs for CI related things?15:53
jlvillalrloo: So now I'm tempted to remove it, since I think we can just do it as there is no gate-breakage.15:53
JayFjlvillal: tbh for something like that, I think an email is proabbly good15:54
JayFjlvillal: because people mainly care why, not always how -- since at this point for gate stuff you're one of the prime experts for ironic on how15:54
rloojlvillal: or, since it is related to multi-node grenade, maybe put that down in the subteam report related to that?15:54
jlvillalrloo: I like that. I'll add it to the subteam status.15:55
mgouldmorning NobodyCam15:55
vdrokgood morning NobodyCam15:55
vdrokand morning JayF :)15:55
rloojlvillal: and/or what JayF said. cuz yeah, /me is no expert in that area.15:55
NobodyCammorning mgould and vdrok :)15:55
NobodyCammorning rloo :)15:55
rloohey, morning NobodyCam!15:55
NobodyCam:)15:55
jlvillalJayF: This should be transparent to people. So not sure it requires an email to the mailing list. A change to devstack/upgrade/settings and later a change to project-config. The gate will continue to chug along.15:56
jlvillalBut I can send an email if people desire.15:57
JayFjlvillal: I like mailing the list for complex coordinaed stuff because 1) Hopefully someone other than me learns what I was doing and why, 2) It's a sneaky way to ask for reviews w/o having to beg, especially for X-project stuff15:57
jlvillalJayF: Okay. I'll send it something out in the next hour.15:58
*** crushil has joined #openstack-ironic15:59
JayFTheJulia: I put a more specific update in so I could set a deadline for myself -- spec up by EOM15:59
TheJulia10-415:59
JayFJFYI: Ironic meeting is not for another hour due to attack of the DST16:02
TheJuliaAt least for those who had DST transition this past weekend16:04
*** vmud213 has joined #openstack-ironic16:04
TheJulia(this time of the year is always chaos because of Daylight savings time16:05
jlvillalJayF, rloo: FYI: Sent email16:05
TheJulia)16:05
rloothx jlvillal16:05
dtantsurmjturek, you have to join ironic-bugs group to change importance16:06
dtantsurTheJulia, np, hope you you're ok now16:06
mjturekdtantsur: yep! rloo pointed that out to me too. I joined it can set importance now16:07
mjturekthanks!16:07
* dtantsur still catches up with pings, while on a meeting...16:07
dtantsurmgould, I'm relatively tolerant to both, prefer heat actually16:08
openstackgerritMerged openstack/ironic master: grenade: Only 'enable_plugin ironic' if not already in conf  https://review.openstack.org/44433516:09
*** vmud213 has quit IRC16:09
dtantsurjlvillal, I may end up reordering topics as we go16:10
* dtantsur has PTL POWWERRZZZ16:10
jlvillaldtantsur: +1 on that :)16:10
*** zackf has quit IRC16:10
*** vmud213 has joined #openstack-ironic16:10
*** vmud213 has quit IRC16:15
*** vmud213 has joined #openstack-ironic16:15
dtantsurjlvillal, in fact, I suggest we remove your agenda item and Just Do It (tm). especially since the first patch has landed already..16:15
*** vmud213 has quit IRC16:16
jlvillaldtantsur: Works for me. I can do that. I had added it earlier because my first solution meant breaking the gate...16:16
*** vmud213 has joined #openstack-ironic16:16
jlvillalI'll remove it.16:16
dtantsurthanks!16:16
jlvillaldtantsur: And it is gone! :)16:17
dtantsurrloo, do you mind if we come to Boston summit brainstorming in the end of the meeting? I have a gut feeling that the other two topics are a bit more pressing..16:17
rloodtantsur: i am fine. honestly, i put it there cuz i saw an email about it. i don't expect us to do the brainstorming today.16:18
dtantsurack, thanks!16:18
jlvillaldtantsur: As an FYI, you may want to do something similar for ironic-inspector if you plan on having multi-node. Or just to make project-config simpler.16:18
dtantsurdunno if we plan, but we may. mind filing a bug against ironic-inspector please?16:18
jlvillaldtantsur: Sure I will do that.16:19
dtantsurthnx16:20
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362816:21
*** dprince has quit IRC16:22
openstackgerritOksana Voshchana proposed openstack/ironic master: Adds use wsgi without port  https://review.openstack.org/44433716:27
openstackgerritOksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi  https://review.openstack.org/43729916:27
openstackgerritKyrylo Romanenko proposed openstack/ironic master: Add negative functional API tests for portgroups  https://review.openstack.org/43753516:29
*** makowals has quit IRC16:30
*** vmud213 has quit IRC16:31
*** vmud213 has joined #openstack-ironic16:31
*** trown is now known as trown|lunch16:31
*** gfhellma has joined #openstack-ironic16:32
openstackgerritVladyslav Drok proposed openstack/ironic master: Fix the exception message in tempest plugin  https://review.openstack.org/44498116:33
openstackgerritVladyslav Drok proposed openstack/ironic master: Fix the exception message in tempest plugin  https://review.openstack.org/44498116:34
*** pcaruana has quit IRC16:38
*** vmud213 has quit IRC16:38
*** vmud213 has joined #openstack-ironic16:39
jkilpatrmgould, anyone BZ that cpu usage thing yet?16:40
*** dprince has joined #openstack-ironic16:41
*** hshiina has joined #openstack-ironic16:42
dtantsurjkilpatr, he's leading a meeting downstream right now, 15 more minutes to go16:44
*** makowals has joined #openstack-ironic16:44
*** zackf has joined #openstack-ironic16:44
jkilpatrdtantsur, ok thanks.16:44
*** ccamacho has quit IRC16:45
rookdtantsur lucasagomes heya!16:48
dtantsuro/16:48
rookohh jkilpatr already sent ya guys a message16:48
lucasagomesrook, hey :D16:48
rookI wanted to send ya something16:48
rookhttps://snapshot.raintank.io/dashboard/snapshot/63GIebBJWAkAyH5iI286v6BXqtm4l31B16:49
rookSo, this system was idle (no deployments over the weekend16:49
lucasagomesrook, we are on a meeting downstream right now, I will take a look soon-ish16:49
rookyou can see before i left for the weekend, I restarted ironic processes16:49
rookahh crap ok16:49
rooklucasagomes: maybe I can join that and present this finding there?16:49
rook:P16:49
jkilpatrrook, literally writing this bz now?16:50
jkilpatrwoops, not ?, I am doing that16:50
jkilpatrrook, check out ironic conductor memory usage16:50
jkilpatrsame thing.16:50
jkilpatrbut only by a few hundred megs16:50
*** vmud213 has quit IRC16:50
*** sayantan_ has joined #openstack-ironic16:51
rookrss?16:51
rookjkilpatr: i don't see that.16:51
jkilpatrrook, virtual, rss doesn't seem to follow the trend16:52
*** lee4 has joined #openstack-ironic16:53
rooknot a concern.16:53
jkilpatrrook, not for us but it may help them diagnose?16:53
rookjkilpatr: if you narrow in on the usage pattern, the spikes are 1 minute a part (60 sec)16:53
rookand it seems to be with gathering status on the nodes.16:53
jrollooooo neat16:54
rookhttps://snapshot.raintank.io/dashboard/snapshot/Sp2wuk2M5adTpqfXMJenMXcSlCav2PiZ16:55
rookso ^ that shows the 60 second interval16:55
rookit wakes up, does something then goes back to sleep16:55
rookhowever, if we trend it back when I JUST restarted ironic-conductor16:55
jrollright, so that's likely the periodic tasks (specifically I'd guess the power state sync)16:56
*** stendulker has joined #openstack-ironic16:57
*** lucasagomes has quit IRC16:59
*** rook has quit IRC16:59
*** sai has quit IRC16:59
jlvillalMeeting time in less than 1 minute :)16:59
*** sai has joined #openstack-ironic17:00
*** lucasagomes has joined #openstack-ironic17:00
*** vmud213 has joined #openstack-ironic17:00
*** rook has joined #openstack-ironic17:00
mgouldjkilpatr: not that I'm aware of, are you writing it up now?17:01
rookback... not sure what message you guys missed/saw17:01
*** rook is now known as Guest8423517:01
jkilpatrhttps://bugs.launchpad.net/ironic/+bug/167245717:01
openstackLaunchpad bug 1672457 in Ironic "Ironic Conductor performance trends down with uptime" [Undecided,New]17:01
jkilpatrboom bz'd17:01
Guest84235ugh, now I am a guest17:01
mgould\o/17:01
jkilpatrrook you got guested again?17:01
*** Guest84235 is now known as rook-17:02
jrollrook-: in case you missed it, I said "right, so that's likely the periodic tasks (specifically I'd guess the power state sync)"17:03
*** vmud213 has quit IRC17:03
rook-jroll: yup missed that.17:03
rook-jroll: that seems like such a innocent task.17:03
*** vmud213 has joined #openstack-ironic17:03
rook-jroll i wonder what you think it could be doing to cause so much CPU time growth.17:03
jrollrook-: oh I'm not saying this is normal, but17:04
rook-because increasing the interval will mask the issue... it will still grow over time.17:04
jrollit talks to BMCs, which are notoriously flaky, and does this in a subprocess (iirc)17:04
jrollthough that depends on driver17:04
rook-jroll pxe_ipmi17:04
rook-or different driver?17:04
jrollok right, so shells out to ipmitool17:04
jrollso I'm wondering if some of those ipmitool calls are hanging17:05
jkilpatrrook-, bz https://bugs.launchpad.net/ironic/+bug/167245717:05
openstackLaunchpad bug 1672457 in Ironic "Ironic Conductor performance trends down with uptime" [Undecided,New]17:05
jkilpatrmgould, tried the xinetd thing, doesn't work, breaks introspection17:05
jkilpatr:(17:05
rook-jroll right, so they aren't async17:05
rook-they sit there and wait for ipmitool to respond.17:05
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient master: Add tests for --wrap option for driver-properties output  https://review.openstack.org/29707717:06
*** dyasny has joined #openstack-ironic17:06
lucasagomesjroll, hmm indeed. rook- do we have a list of the process that are running over time logged somewhere ?17:06
jrollrook-: yep17:06
rook-and possibly subsequent calls could take longer.17:06
jrollalso this is one thread that loops through each node17:06
jrollit might be getting to the same node each time, even17:06
jrollthere should be a timeout, iirc, though17:06
rook-ironic    98469  0.0  0.0      0     0 ?        Z    13:06   0:00 [ipmitool] <defunct>17:07
rook-just cauth ^17:07
rook-caught17:07
jrollthat'll do it17:07
lucasagomesjroll, I think now we can run jobs in parallel, after the intodruction of futurist btw17:07
*** chlong has quit IRC17:07
lucasagomesand we could look at futurist to see if it has something to do with the processes hanging like that17:07
jrolllucasagomes: yeah, we parallelize the task, but each periodic task is one thread, I think17:07
jrollso one thread for power state17:08
lucasagomesjroll, oh right yeah, true that17:08
rook-right17:08
rook-i see this quite often17:08
rook-[ipmitool] <defunct>17:08
jrollrook-: sorry, in our upstream meeting at the same time, so I may be slow17:09
jrollhere's the code that runs ipmitool commands https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ipmitool.py#L36317:10
jrollmaybe there is no timeout for that command17:10
jrollhuh.17:10
rook- 4 of the 107 nodes are not in maint17:11
rook-sorry that was wrong17:12
rook-4 of the 107 are in main mode17:12
rook-the rest are good to go17:12
mgouldjkilpatr: yeah, we thought that might happen - thanks for trying!17:12
jkilpatrmgould, also interesting the ironic-inspector setting for limiting the time between introspections doesn't seem to work the way I would assumeit would.17:13
jkilpatrif you set that to 30 seconds and then issue 30 introspection operations they all go pxe at the same time still17:13
*** vmud213 has quit IRC17:14
*** vmud213 has joined #openstack-ironic17:14
mgouldhuh17:15
mgouldthat sounds wrong17:16
mgouldwhat's the name of the setting?17:16
* jkilpatr goes diving into log archive17:16
jrollrook-: yeah, looks like we don't kill these processes after some amount of time :/17:17
*** vmud213 has quit IRC17:17
jlvillalBackports that could use reviews: https://review.openstack.org/#/c/444950/ for stable/newton17:17
patchbotpatch 444950 - ironic (stable/newton) - grenade: Only 'enable_plugin ironic' if not alread...17:17
*** vmud213 has joined #openstack-ironic17:17
jlvillalAnd https://review.openstack.org/#/c/444944/ for stable/ocata17:17
patchbotpatch 444944 - ironic (stable/ocata) - grenade: Only 'enable_plugin ironic' if not alread...17:17
*** openstackgerrit has quit IRC17:18
*** e0ne has quit IRC17:19
jkilpatrmgould, inspector.conf introspection_delay, # Delay (in seconds) between two introspections. (integer value)17:19
jkilpatrdoes not do what it says on the tin as far as I can tell17:20
*** vmud213 has quit IRC17:20
*** vmud213 has joined #openstack-ironic17:21
*** openstackgerrit has joined #openstack-ironic17:23
openstackgerritMatt Riedemann proposed openstack/ironic master: DNM: testing oslo deprecation warnings  https://review.openstack.org/44502017:23
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362817:26
openstackgerritMerged openstack/ironic-python-agent master: Correct failure message output when downloading  https://review.openstack.org/44107217:33
*** mgoddard has quit IRC17:33
openstackgerritMerged openstack/ironic master: This adds a tempest test for creating a chassis with a specific UUID.  https://review.openstack.org/40919617:35
jrollJayF: dtantsur: easy stable patch win after the meeting https://review.openstack.org/#/q/I09ed78ef918eb0359045350e89d42c6df9f3e490,n,z17:35
JayFocata approved, the newton is not depends-on so going to wait to land that17:36
*** kjw3 has joined #openstack-ironic17:36
jrolloh, good point17:37
jrollthanks17:37
*** venki has quit IRC17:42
*** fragatina has quit IRC17:44
-openstackstatus- NOTICE: restarting gerrit to address performance problems17:44
*** fragatina has joined #openstack-ironic17:44
*** gutter has joined #openstack-ironic17:47
*** gutter has quit IRC17:47
*** fragatina has quit IRC17:49
*** derekh has quit IRC17:49
*** alineb has quit IRC17:49
*** jaosorior has quit IRC17:56
mgouldjkilpatr: I think there may be a thundering-herd problem with introspection delays: https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/introspect.py#L11817:57
aNuposicmat128, pas-ha, JayF setting "PYOPTIMIZE_TINYIPA=false" worked to run glean inside tinyipa. Thanks :) Looks like original *.py were removed and are required by glean17:57
JayFaNuposic: unless that doesn't bloat the image significantly, we might have to determine how t omake it work with that set to true :/17:58
jkilpatrmgould, so the lock isn't working because of contention?17:59
*** tesseract has quit IRC17:59
openstackgerritMatt Riedemann proposed openstack/ironic master: DNM: testing oslo deprecation warnings  https://review.openstack.org/44502018:00
* jroll bbiab18:00
JayFI'm on PTO for the rest of the day, if anyone needs something leave me an email or msg on IRC, will be back tomorrow o/18:00
*** ralonsoh has quit IRC18:00
rlooenjoy JayF!18:01
*** aarefiev is now known as aarefiev_afk18:01
dtantsurenjoy JayF18:01
dtantsurrloo, I remember you said you don't want to send summaries to the ML any more. is it still the case?18:02
*** wanyen has joined #openstack-ironic18:02
wanyenHi, can we discuss redfish pyton lib?18:02
nicodemoshey vdrok, could you take a look on 386014 ? :) we are done with the requested changes18:02
rloodtantsur: yes, was going to do it one last time after the subteams got updated, so today. then was going to look around to see if anyone else wanted to do it18:02
*** venki has joined #openstack-ironic18:03
dtantsurrloo, I can do it, but I'm afraid I'll forget. if you don't mind reminding me for some first time, will be awesome18:03
*** jason77 has joined #openstack-ironic18:03
dtantsurwanyen, we can, but I'm not sure what to discuss right now.18:03
wanyendtantsur, can we discuss redfish python lib ?18:03
rloodtantsur: ok, let me see if i can find a volunteer first. i am worried that you have too much on your plate.18:04
dtantsurrloo, thanks :)18:04
wanyenWhat I would like to discuss is the overall redfish python lib direction.18:04
dtantsurwanyen, do you have questions about the library itself or the ironic driver?18:04
dtantsurmost of people here do not develop python-redfish library, you'd better reach out to bcornec18:05
wanyenI think susy helps us to get basic powe and management features but we will need a lot  more modules to add to support other features likeinspect, virtual meida, etc.18:05
dtantsurwanyen, IIRC Nisha was going to work on inspection, virtual media may also be doable18:05
lucasagomesJayF, see ya enjoy18:06
vdroknicodemos: will do tomorrow, a bit late atm :)18:06
dtantsurwanyen, anyway, it all boils down to whether and when the python-redfish library will be ready for us to consume. it's not something I personally can affect. but it's something you can contribute to, if you're interested.18:06
TheJuliawanyen: the focus right now is minimal viable functionality in order to deploy.  The specification details the view towards the libraries which many of us seem to be in agreement about18:06
nicodemosty vdrok o/18:06
wanyenThe question is whether we will keep adding more python lib into sushy or we leverage those lib that already supported in redfish-python.18:06
dtantsurwanyen, my answer is above18:07
dtantsurif we need to add stuff to sushy to move forward, we will. if we find python-redfish ready, we'll consider switching to it.18:07
mgouldjkilpatr: on second thoughts, no, I think the logic there ought to work18:07
dtantsurI can't make any promises, because I don't know when and whether both events will happen.18:07
mgouldjkilpatr: could you check the logs for the string "Attempting to acquire lock on last introspection time"?18:08
wanyenDon't we want to work with Bruno to make redfsih-python lib meet ironic requirements?18:08
openstackgerritJoanna Taryma proposed openstack/ironic-specs master: Response header based retries  https://review.openstack.org/44404618:08
* mgould -> home; will check back tomorrow18:08
dtantsurwanyen, who - we?18:08
lucasagomeswanyen, please, read the spec (and the comments on it)18:08
*** jason77 has quit IRC18:08
wanyenIronic upstream18:08
lucasagomeswanyen, https://review.openstack.org/#/c/18465318:08
patchbotpatch 184653 - ironic-specs - Propose Redfish support into Ironic (MERGED)18:08
nicodemosTheJulia, we are done with https://review.openstack.org/#/c/386014 could you look to it? :)18:08
patchbotpatch 386014 - ironic - Validation before perform node deallocation18:08
jlvillaldtantsur: JayF: Maybe one of you could +2 the stable/newton patch without workflow +118:09
*** e0ne has joined #openstack-ironic18:09
jlvillalhttps://review.openstack.org/#/c/444950/18:09
patchbotpatch 444950 - ironic (stable/newton) - grenade: Only 'enable_plugin ironic' if not alread...18:09
jlvillalAnd then when the stable/ocata one is merged jroll could Approve it.18:09
*** trown|lunch is now known as trown18:09
dtantsurwanyen, I think the answer is between "no" and "not right now". please refer to lucasagomes' comments on the spec for background18:09
jlvillalThe stable/ocata one got caught in the gerrit restart and now has to go through check and gate again.18:09
dtantsurplease keep in mind that most of people here are overcommitted even without it18:10
wanyendtanstsur, I read Bruno's redfish-python and susy lib code.18:10
* jlvillal has to go to 1:1 with his manager18:10
*** bcornec2 has quit IRC18:10
*** hshiina has quit IRC18:10
*** jason77 has joined #openstack-ironic18:11
dtantsurwanyen, I need to run, sorry. may I please redirect you to lucasagomes for more context?18:11
lucasagomesdtantsur, g'night!18:11
wanyenSusy is good but a lot more redfish modules needs to be added.  redfish-python has a lot more redfish modules supported.  I think it won't take long to make it meet Ironic requirements.18:12
*** dtantsur is now known as dtantsur|afk18:12
dtantsur|afksee you all18:12
nicodemossee you dtantsur|afk18:12
TheJuliawanyen: It seems you've already looked at https://review.openstack.org/#/c/184653/, but you may want to look more at the back and forth discussion on some of the change sets.18:13
patchbotpatch 184653 - ironic-specs - Propose Redfish support into Ironic (MERGED)18:13
lucasagomeswanyen, yeah sushy only does the basic stuff for now, like power management (soft and hard), boot device management, NMI and boot mode (UEFI vs BIOS)18:13
*** jason77 has quit IRC18:14
*** mgould is now known as mgould|afk18:14
wanyenwe will need to add ethernetCollection, Storage, Chassis, manager, etc to sushy in order to support MAC and storage discovery, and virtual media18:14
lucasagomeswanyen, and, at least few weeks ago when I looked at python-redfish it contained a lot of bugs (see the bug tracker for it) and part of the things above was broken18:14
lucasagomeswanyen, right yeah, the idea is to grow the library until we have python redfish mature enough to be re-evaluated. I mean, if you think python-redfish is faster to fix18:15
lucasagomesI would say, go for it18:15
*** vmud213 has quit IRC18:15
wanyenLucasagmoes, we will either need to do work in sushy or redfish-python to support more redfish driver features.  So the question is which one we should pick to add more work.18:15
TheJuliawanyen: Basically your asking us to help you make what is a simple business decision. The cost/effort/results need to be weighed.  The community can't give you the answer in this case.18:17
lucasagomeswanyen, IMHO, sushy. The problem with python-redfish IMHO is that it has some archtetural problems and lack tests of all sorts18:17
lucasagomesso those need to be fixed before adding new features to it18:18
lucasagomesbut, it's up to you to decide where to spend ur time on18:18
lucasagomesthat's why I think you should read the comments and conclude it urself18:18
stendulkerlucasagomes: I we need to have a concerted effort that could enable most of the redfish features througha single library18:19
stendulkerlucasagomes: Else it would be duplication of code till any switchover happens, if ever that happens18:20
stendulkerswitchover of libraries..18:20
TheJuliastendulker: Then estimate the cost/effort that it will take, and plan accordingly.18:20
lucasagomesstendulker, sure, that's why I think people interested in adding those feature should evaluate it18:20
lucasagomesstendulker, I mean, I did it. I looked at the options we had and I concluded that it was easier to us to start from scratch (the comments on why is in the spec comments)18:21
*** mgoddard has joined #openstack-ironic18:21
stendulkerlucasagomes: I understand, but since we kept the option open for other libratry, it kind of creates a confusion18:22
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests  https://review.openstack.org/44362818:22
wanyenJust an example, if I chose to implement virtual media with redfish-python lib, will ironic upstream accept it?18:22
lucasagomeswanyen, if you just add that without fixing the rest of the stuff that we identified as broken I think the answer is no (btw, I'm not in anyway representing "ironic upstream" here, that's my own opinion)18:23
jrollagree, the basic architecture of python-redfish is fundamentally incompatible with ironic today18:24
wanyenlucasagomes, assumption is that we address your concerns in redfish-pyton 1st.18:25
vdrokgood night everyone!18:27
lucasagomeswanyen, right, if that's the assumption I don't see why the community shouldn't re-evaluate using it then. But, my tip is to you to look at the current problems reported and estimate how long it's going to take18:27
nicodemosnight vdrok18:27
lucasagomesvdrok, see ya g'night18:29
wanyenIt really either adding more redfish module support in susy or fixing pyton-redfish lib.  So, if ironic's longer term direction is to use redfish-pyton lib when it meets ironic's requirements, then it might make more sense to fix redfish-python lib so that same ironic features don't need to be implemented twice.18:29
*** kjw3 has quit IRC18:30
lucasagomeswanyen, ok, if that's what you have concluded go for it. I understand the situation can be a bit confusing at the moment because we don't want to discard any possibilities (or bless one library over the other)18:33
lucasagomesso it's up to people's judgment to evaluate and conclude where they want to spend their time own18:33
TheJulialucasagomes: Well said18:33
lucasagomesI don't speak for Ironic in general neither I can tell you where to put ur efforts on so, I'm sorry but you will have to figure that out by urself18:33
wanyenok.  so much redfsih python lib discussion for now. got to go.  Thanks!18:34
jkilpatrmgould|afk, no hits in the logs for the lock aqusition, I'm sure I had the option enabled, but are we sure that it gets called through the tripleo commands? I know tripleo stuff goes through stage codepaths (hence we never see the inspecting state...)18:34
*** wanyen has quit IRC18:35
openstackgerritMatt Riedemann proposed openstack/ironic master: Deprecate support for glance v1  https://review.openstack.org/44149118:35
*** sayantan_ has quit IRC18:35
*** priteau has joined #openstack-ironic18:38
*** fragatina has joined #openstack-ironic18:40
*** fragatina has quit IRC18:40
*** stendulker has quit IRC18:41
*** fragatina has joined #openstack-ironic18:41
*** joanna has joined #openstack-ironic18:41
aNuposicJayF, i think so glean requires .py, but specifically i can look what else glean requires (i don't see any thing specified on glean page) and may be on that can decide whether new flag needs to introduce to enable .py and keep other things optimized by setting PYOPTIMIZE=true18:41
*** jkilpatr has quit IRC18:42
*** joanna is now known as joanna_lunch18:42
*** jkilpatr has joined #openstack-ironic18:43
*** sayantan_ has joined #openstack-ironic18:45
lucasagomesfolks I'm calling it a day, have a great evening all!18:45
lucasagomesand as per discussion in the meeting, here's the patch to the governance repo: https://review.openstack.org/#/c/445046/18:46
patchbotpatch 445046 - governance - Add sushy under Ironic governance18:46
jrollthanks lucasagomes! have a good night :)18:46
lucasagomeso/ see ya18:46
*** lucasagomes is now known as lucas-afk18:46
*** zackf has quit IRC18:59
*** kjw3 has joined #openstack-ironic19:01
*** zackf has joined #openstack-ironic19:01
*** athomas has quit IRC19:04
*** zhugaoxiao has quit IRC19:04
*** zhugaoxiao has joined #openstack-ironic19:05
openstackgerritOksana Voshchana proposed openstack/ironic master: Adds use wsgi without port  https://review.openstack.org/44433719:06
openstackgerritOksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi  https://review.openstack.org/43729919:06
openstackgerritJim Rollenhagen proposed openstack/ironic stable/ocata: Set tempest's max microversion to 1.31  https://review.openstack.org/44506119:16
jrollJayF: dtantsur|afk: ^ easy stable review we should land before we break something19:16
*** dyasny has quit IRC19:21
*** joanna_lunch is now known as joanna19:22
openstackgerritOpenStack Proposal Bot proposed openstack/bifrost master: Updated from global requirements  https://review.openstack.org/44506419:29
openstackgerritVladyslav Drok proposed openstack/ironic master: Refactor waiters in our tempest plugin  https://review.openstack.org/41068419:30
openstackgerritOpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements  https://review.openstack.org/44508019:32
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements  https://review.openstack.org/44508119:32
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-lib master: Updated from global requirements  https://review.openstack.org/44508219:32
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent master: Updated from global requirements  https://review.openstack.org/44508319:32
*** _vsaienko has joined #openstack-ironic19:32
*** mgoddard has quit IRC19:41
*** ChubYann has joined #openstack-ironic19:42
*** ccamacho has joined #openstack-ironic19:45
*** jkilpatr has quit IRC19:51
*** rook- is now known as rook19:51
*** dyasny has joined #openstack-ironic19:51
*** jkilpatr has joined #openstack-ironic19:52
*** zackf has quit IRC19:56
*** amoralej is now known as amoralej|off19:57
*** jcoufal has quit IRC19:57
*** zackf has joined #openstack-ironic20:00
*** chlong has joined #openstack-ironic20:04
*** zackf has quit IRC20:04
*** jheroux has quit IRC20:05
*** crushil has quit IRC20:12
*** amotoki has quit IRC20:13
*** dsneddon has quit IRC20:25
*** dprince has quit IRC20:28
*** fragatin_ has joined #openstack-ironic20:30
*** Goneri has quit IRC20:30
*** fragatina has quit IRC20:33
*** _vsaienko has quit IRC20:35
*** vsaienko has joined #openstack-ironic20:37
*** mjura has quit IRC20:37
mrdaMorning Ironic20:38
openstackgerritVasyl Saienko proposed openstack/ironic master: Add Ironic standalone tests  https://review.openstack.org/42355620:39
joannamorning mrda20:41
*** dsneddon has joined #openstack-ironic20:41
mrdao/20:46
*** jrcloud has joined #openstack-ironic20:50
*** zackf has joined #openstack-ironic20:50
openstackgerritVasyl Saienko proposed openstack/ironic master: Cleanup hung iscsi session  https://review.openstack.org/38809220:52
openstackgerritMerged openstack/ironic stable/ocata: grenade: Only 'enable_plugin ironic' if not already in conf  https://review.openstack.org/44494420:53
jlvillaljroll: FYI: The stable/ocata patch landed. So I think the stable/newton patch is ready to go: https://review.openstack.org/444950   For grenade fix.20:54
patchbotpatch 444950 - ironic (stable/newton) - grenade: Only 'enable_plugin ironic' if not alread...20:54
jrolljlvillal: cool, thanks! done20:56
jlvillaljroll: Awesome. Thanks :)20:56
*** Sukhdev has joined #openstack-ironic20:57
*** bww has joined #openstack-ironic20:59
*** gfhellma has quit IRC21:00
openstackgerritRuby Loo proposed openstack/ironic master: Speed up test_touch_conductor_deadlock()  https://review.openstack.org/44518221:04
rloojoanna: might be of interest ^^21:06
*** mjturek has quit IRC21:06
*** trown is now known as trown|outtypewww21:06
joannarloo: thanks for the info! will look at that :)21:06
joannarloo: Nice!21:07
rloojoanna: i always forget to look at the timings when reviewing. i was looking at something else and noticed it.21:08
openstackgerritOksana Voshchana proposed openstack/ironic master: Adds use wsgi without port  https://review.openstack.org/44433721:08
openstackgerritOksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi  https://review.openstack.org/43729921:08
*** vsaienko has quit IRC21:09
*** vsaienko has joined #openstack-ironic21:10
*** _vsaienko has joined #openstack-ironic21:12
*** _vsaienko has quit IRC21:13
*** _vsaienko has joined #openstack-ironic21:14
*** vsaienko has quit IRC21:14
*** sayantan_ has quit IRC21:15
*** _vsaienko has quit IRC21:15
*** _vsaienko has joined #openstack-ironic21:15
*** e0ne has quit IRC21:18
*** sayantan_ has joined #openstack-ironic21:20
*** penick has joined #openstack-ironic21:21
*** chlong has quit IRC21:34
jlvillaljroll: Not sure if you already have this Nova patch on your list: https://review.openstack.org/#/c/444106/21:34
patchbotpatch 444106 - nova - Add populate_retry to schedule_and_build_instances21:34
jlvillaljroll: I think that is the fix for the multi-node failure that is sometimes seen.21:35
*** rpioso has quit IRC21:43
jrolljlvillal: neat, thanks. I did not21:44
jlvillaljroll: I only knew because vsaienk0 posted about it as a comment in a patch :)21:44
*** _vsaienko has quit IRC21:45
*** chlong has joined #openstack-ironic21:47
*** fragatin_ has quit IRC21:51
openstackgerritRamamani Yeleswarapu proposed openstack/ironic-inspector-specs master: Remove link to sample inventory from Dmidecode spec  https://review.openstack.org/44521921:52
*** fragatina has joined #openstack-ironic21:52
*** vsaienko has joined #openstack-ironic21:52
*** baoli has quit IRC22:01
*** [1]cdearborn has quit IRC22:02
openstackgerritSolio Sarabia proposed openstack/ironic master: Include a copy of tempest.scenario.manager module  https://review.openstack.org/43925222:03
openstackgerritMerged openstack/ironic stable/newton: grenade: Only 'enable_plugin ironic' if not already in conf  https://review.openstack.org/44495022:05
*** vsaienko has quit IRC22:06
*** yonglihe has quit IRC22:09
*** priteau has quit IRC22:10
*** vsaienko has joined #openstack-ironic22:10
*** priteau has joined #openstack-ironic22:10
*** jkilpatr has quit IRC22:11
openstackgerritOksana Voshchana proposed openstack/ironic master: Adds use wsgi without port  https://review.openstack.org/44433722:12
openstackgerritOksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi  https://review.openstack.org/43729922:12
*** jvgrant has quit IRC22:13
*** priteau has quit IRC22:15
*** zackf has quit IRC22:24
*** snecklifter has quit IRC22:25
*** chlong has quit IRC22:26
openstackgerritOksana Voshchana proposed openstack/ironic master: Adds use wsgi without port  https://review.openstack.org/44433722:29
openstackgerritOksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi  https://review.openstack.org/43729922:30
*** vsaienko has quit IRC22:33
*** baoli has joined #openstack-ironic22:36
*** vsaienko has joined #openstack-ironic22:37
*** baoli has quit IRC22:41
*** zackf has joined #openstack-ironic22:44
*** rbudden has quit IRC22:44
*** bfournie has quit IRC22:45
*** awidders_ has joined #openstack-ironic22:46
*** awiddersheim has quit IRC22:48
*** jkilpatr has joined #openstack-ironic22:50
*** _vsaienko has joined #openstack-ironic22:51
*** vsaienko has quit IRC22:51
*** zackf has quit IRC22:53
*** _vs_ has joined #openstack-ironic22:56
*** _vsaienko has quit IRC22:56
*** _vs_ has quit IRC22:58
*** vsaienko has joined #openstack-ironic23:04
openstackgerritVasyl Saienko proposed openstack/ironic master: Add Ironic standalone tests  https://review.openstack.org/42355623:10
*** jvgrant has joined #openstack-ironic23:13
*** awidders_ has quit IRC23:15
*** awiddersheim has joined #openstack-ironic23:16
openstackgerritOpenStack Proposal Bot proposed openstack/ironic stable/ocata: Updated from global requirements  https://review.openstack.org/44380023:25
*** mtanino has joined #openstack-ironic23:26
*** vsaienko has quit IRC23:33
*** vsaienko has joined #openstack-ironic23:34
openstackgerritRuby Loo proposed openstack/python-ironicclient master: Add DRIVER_RESOURCE to remove duplicated strings  https://review.openstack.org/44092623:35
*** pmannidi has joined #openstack-ironic23:36
*** rloo has quit IRC23:40
*** sayantan_ has quit IRC23:41
*** rbudden has joined #openstack-ironic23:43
*** sayantan_ has joined #openstack-ironic23:49
*** sayantan_ has quit IRC23:54

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