Wednesday, 2016-06-22

TheJuliahttps://bugs.launchpad.net/ironic/+bug/159319400:05
openstackLaunchpad bug 1593194 in Ironic "[RFE]Enable hardware disk erase in ironic" [Undecided,Incomplete]00:05
TheJuliatriaging as incomplete, pointing to secure erase and sking for clarification00:05
JayFsounds more like invalid to me, lol00:06
TheJuliaI think they are meaning scsi disk to be honest00:06
TheJuliascsi/sas00:07
*** mbound has quit IRC00:07
TheJuliahttps://bugs.launchpad.net/ironic/+bug/159466800:12
openstackLaunchpad bug 1594668 in Ironic "Create a port using node_name" [Undecided,New] - Assigned to Sharat Sharma (sharat-sharma)00:12
openstackgerritMerged openstack/ironic-python-agent: Fix unwanted unmounting of /usr/share/oem  https://review.openstack.org/33247200:12
mat128https://bugs.launchpad.net/ironic/+bugs?orderby=-datecreated&start=000:14
TheJuliayup, looking at the same list00:14
*** praneshp has joined #openstack-ironic00:15
tonybJayF, jroll: Both are merged release at will00:19
jlvillalIf you aren't talking please mute :)00:25
jlvillalWe hear some background noise00:25
jlvillalhttps://bugs.launchpad.net/ironic/+bug/159302100:26
openstackLaunchpad bug 1593021 in Ironic "[RFE] Make devstack fail if insufficient RAM assigned for ramdisk" [Undecided,New] - Assigned to Bharath kumar (bharath-kumar3)00:26
TheJuliasomebody's phone is ringing...00:26
mat128JayF: got anything in flight about bug ^00:27
mat128?00:27
mat128I remember something about 768mb or something00:27
jlvillalhttps://review.openstack.org/#/q/owner:%22Jay+Faulkner+%253Cjay%2540jvf.cc%253E%2200:28
*** Nisha has quit IRC00:28
jlvillalhttps://launchpad.net/~ironic-bugs00:30
TheJuliahttps://bugs.launchpad.net/ironic/+bug/159233500:33
openstackLaunchpad bug 1592335 in Ironic "ironic node validate fails to show the relevant message for the missing ilo parameter(agent_ilo)" [Low,In progress] - Assigned to Shivanand Tendulker (stendulker)00:33
TheJulia^^^ caught my attention00:33
openstackgerritMerged openstack/ironic: Updated from global requirements  https://review.openstack.org/33229500:37
mat128TheJulia: http://docs-draft.openstack.org/37/329837/1/check/gate-ironic-releasenotes/c5e2bb3//releasenotes/build/html/current-series.html00:42
TheJuliaYeah :(00:43
*** catintheroof has joined #openstack-ironic00:47
*** YiboCai is now known as yibo00:48
*** penick has quit IRC00:48
openstackgerritMoshe Levi proposed openstack/ironic-python-agent: Replace assertRaisesRegexp with assertRaisesRegex  https://review.openstack.org/33225000:48
*** phuongnh has joined #openstack-ironic00:56
jlvillalWow! I think mrda will be sad he wasn't in the call...01:02
jlvillal$(type -P disk-image-builder > /dev/null); echo $?01:06
jlvillal$(type -P bash > /dev/null); echo $?01:06
*** fragatina has joined #openstack-ironic01:06
*** Sukhdev_ has quit IRC01:06
*** catintheroof has quit IRC01:07
*** mbound has joined #openstack-ironic01:08
jlvillalhttps://review.openstack.org/#/c/330294/1/devstack/lib/ironic01:08
mat128https://github.com/openstack/ironic/commit/259a4922437288e9988aa7c201830792b5a0ad45#diff-fb50fd952d51d577ae0be49196b05e5aL111101:08
mat128disk-image-create according to http://docs.openstack.org/developer/ironic/deploy/install-guide.html#buildingdibbaseddeployramdisk01:09
openstackgerritJohn L. Villalovos proposed openstack/ironic: Restore diskimage-builder install  https://review.openstack.org/33029401:11
jlvillalTheJulia, mat128 Done: https://review.openstack.org/#/c/330294/1..2/devstack/lib/ironic01:12
*** mbound has quit IRC01:13
jlvillalMikroTik01:19
mrdajlvillal: huh?01:25
TheJuliamrda: side discussion :)01:25
mat128discussing routers and whatnot ;)01:25
jlvillalmrda, Just trying to make you feel guilty for not chatting with us.01:25
mrdaoh, so fun things, but not super important.01:26
* mrda is sooo tired this morning01:26
mrdaIt's not that I don't like y'all :)01:26
jlvillalmrda, It's that you don't like us :P01:26
jlvillalmrda, Remember to mute before you start snoring in the conference call :)01:27
mrdaHardly.  There's not many things that would get me on a call from midnight to 5am :)01:27
jlvillalmat128, I'm going to take off and go do some grocery shopping, and maybe a trip to Home Depot01:29
TheJuliagoodnight jlvillal01:29
jlvillalThanks TheJulia01:30
TheJuliaWe're calling it a night on the call, goodnight everyone!01:33
*** rloo has quit IRC01:38
*** edand has joined #openstack-ironic01:45
*** sacharya has joined #openstack-ironic01:46
*** cdearborn has quit IRC01:49
*** edand has quit IRC01:50
*** sacharya has quit IRC01:50
*** hoangcx has joined #openstack-ironic01:56
*** spandhe has quit IRC02:03
openstackgerritwangjianhe proposed openstack/ironic: KeystoneFailure: Could not authorize in Keystone: A username and password or token is required.  https://review.openstack.org/33260002:09
*** PollyZ has joined #openstack-ironic02:10
openstackgerritwangjianhe proposed openstack/ironic: KeystoneFailure: Could not authorize in Keystone: A username and password or token is required.  https://review.openstack.org/33260002:13
openstackgerritMerged openstack/ironic-python-agent: Add configuration options for DISK_WAIT  https://review.openstack.org/32490902:29
*** PollyZ_ has joined #openstack-ironic02:30
*** PollyZ has quit IRC02:33
*** PollyZ_ has quit IRC02:35
*** PollyZ has joined #openstack-ironic02:37
*** yuanying has quit IRC02:46
*** PollyZ has quit IRC02:50
*** PollyZ has joined #openstack-ironic02:50
*** PollyZ has quit IRC03:04
*** Nisha has joined #openstack-ironic03:10
*** PollyZ has joined #openstack-ironic03:11
*** Nisha_away has joined #openstack-ironic03:17
*** Nisha_away is now known as Nisha103:17
*** Nisha has quit IRC03:20
*** PollyZ has quit IRC03:21
*** PollyZ has joined #openstack-ironic03:26
*** links has joined #openstack-ironic03:29
openstackgerritNisha Agarwal proposed openstack/proliantutils: Fix the logic for fetching the gpu device count  https://review.openstack.org/31540203:36
*** PollyZ has quit IRC03:40
*** Nisha1 has quit IRC03:40
*** spandhe has joined #openstack-ironic03:41
*** spandhe_ has joined #openstack-ironic03:44
*** catintheroof has joined #openstack-ironic03:44
*** spandhe has quit IRC03:45
*** spandhe_ is now known as spandhe03:45
*** yuanying has joined #openstack-ironic03:47
openstackgerritMathieu Mitchell proposed openstack/ironic: Use PRIVATE_NETWORK_NAME for devstack plugin  https://review.openstack.org/33261403:49
*** yogi has joined #openstack-ironic03:59
*** sacharya has joined #openstack-ironic03:59
*** catintheroof has quit IRC04:02
*** jaybeale has joined #openstack-ironic04:04
*** jvgrant has joined #openstack-ironic04:04
*** jvgrant has quit IRC04:09
*** appu103_ has joined #openstack-ironic04:14
*** appu103_ is now known as aparnav04:16
openstackgerritShivanand Tendulker proposed openstack/ironic: Add validation of 'ilo_deploy_iso' in deploy.validate()  https://review.openstack.org/32983704:17
*** smoriya has joined #openstack-ironic04:19
*** sacharya has quit IRC04:31
*** praneshp has quit IRC04:34
*** praneshp has joined #openstack-ironic04:36
openstackgerritDao Cong Tien proposed openstack/ironic: [WIP] Nova-compatible serial console: socat console_utils  https://review.openstack.org/32816804:42
*** Sukhdev has joined #openstack-ironic04:49
*** Sukhdev has quit IRC04:49
*** Sukhdev has joined #openstack-ironic04:50
*** skramaja has joined #openstack-ironic04:51
*** rcernin has joined #openstack-ironic04:56
*** ijw has quit IRC05:04
*** rajinir has quit IRC05:04
*** rcernin has quit IRC05:16
*** joprovost has quit IRC05:17
*** shirly has joined #openstack-ironic05:19
*** jaybeale has quit IRC05:26
*** ChubYann has quit IRC05:32
*** chlong has quit IRC05:34
*** penick has joined #openstack-ironic05:36
*** Sukhdev_ has joined #openstack-ironic05:40
*** Sukhdev has quit IRC05:41
*** praneshp has quit IRC05:42
*** praneshp has joined #openstack-ironic05:44
*** chlong has joined #openstack-ironic05:47
*** rbartal has joined #openstack-ironic05:54
*** rcernin has joined #openstack-ironic05:55
*** penick has quit IRC05:57
*** moshele has joined #openstack-ironic05:58
*** amotoki has joined #openstack-ironic06:01
openstackgerritYuiko Takada Mori proposed openstack/ironic-specs: Nova compatible serial console support  https://review.openstack.org/31950506:02
*** ccamacho has joined #openstack-ironic06:06
*** yolanda has quit IRC06:29
openstackgerritMoshe Levi proposed openstack/ironic-python-agent: tox: Update flake8 to ignore tinyipa imagebuild folders  https://review.openstack.org/33224706:33
*** praneshp has quit IRC06:39
openstackgerritAparna proposed openstack/proliantutils: Sort values of raid_config for 'share_physical_disks'  https://review.openstack.org/32631906:44
*** david-lyle_ has joined #openstack-ironic06:46
*** tesseract- has joined #openstack-ironic06:47
*** slagle_ has joined #openstack-ironic06:50
*** athomas has joined #openstack-ironic06:53
*** ptx0_ has joined #openstack-ironic06:54
*** intr1nsic_ has joined #openstack-ironic06:55
*** sacharya has joined #openstack-ironic06:59
*** chihhsin_ has joined #openstack-ironic06:59
openstackgerritSwapnil Kulkarni (coolsvap) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844007:00
openstackgerritSwapnil Kulkarni (coolsvap) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844007:00
*** hamzy_ has joined #openstack-ironic07:01
*** ccamacho1 has joined #openstack-ironic07:04
*** yolanda has joined #openstack-ironic07:04
*** ccamacho has quit IRC07:04
*** david-lyle has quit IRC07:04
*** hamzy has quit IRC07:04
*** alineb has quit IRC07:04
*** intr1nsic has quit IRC07:04
*** slagle has quit IRC07:04
*** chihhsin has quit IRC07:04
*** rm_work has quit IRC07:04
*** ptx0 has quit IRC07:04
*** intr1nsic_ is now known as intr1nsic07:04
*** fragatina has quit IRC07:07
*** fragatina has joined #openstack-ironic07:07
*** ccamacho1 is now known as ccamacho|afk07:10
*** yolanda has quit IRC07:10
*** itamarl has joined #openstack-ironic07:13
*** spandhe has quit IRC07:13
*** rm_work has joined #openstack-ironic07:14
*** aarefiev_ has joined #openstack-ironic07:15
*** spandhe has joined #openstack-ironic07:16
pas-hamorning Ironic07:18
*** sacharya has quit IRC07:18
*** _vsaienko has joined #openstack-ironic07:22
*** pcaruana has joined #openstack-ironic07:22
*** chlong has quit IRC07:25
*** _vsaienko has quit IRC07:27
*** ccamacho|afk is now known as ccamacho07:28
*** aparnav has quit IRC07:28
*** _vsaienko has joined #openstack-ironic07:29
*** _vsaienko has quit IRC07:41
*** jtomasek has quit IRC07:45
*** jtomasek has joined #openstack-ironic07:47
*** alineb has joined #openstack-ironic07:53
lucasagomesmorning all07:55
zigoGood morning everyone. Does version 5.1.1 fixes CVE-2016-4985?07:55
*** ohamada has joined #openstack-ironic07:55
zigohttps://bugs.launchpad.net/ironic/+bug/157279607:56
openstackLaunchpad bug 1572796 in Ironic "Node information including credentials exposed to unathenticated users (CVE-2016-4985)" [High,Fix released] - Assigned to Devananda van der Veen (devananda)07:56
*** electrofelix has joined #openstack-ironic07:57
*** Sukhdev_ has quit IRC07:58
*** Sukhdev has joined #openstack-ironic07:58
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:02
zigoLooks like it doesn't to me.08:04
_milan_morning lucasagomes pas-ha zigo :)08:05
*** Sukhdev has quit IRC08:05
_milan_#pixiesay morning Ironic! -m flexing08:06
*** e0ne has joined #openstack-ironic08:06
lucasagomes_milan_, oh seems it's down, I will bring the bot up again later08:10
_milan_lucasagomes, yeah, poor pixie08:11
*** aarefiev_ has quit IRC08:11
*** lucasagomes is now known as lucas-brb08:11
*** hoangcx has quit IRC08:11
*** M00nr41n has joined #openstack-ironic08:17
*** yuikotakadamori has quit IRC08:18
*** yolanda has joined #openstack-ironic08:18
*** spandhe has quit IRC08:19
*** mbound has joined #openstack-ironic08:23
*** dmk0202 has joined #openstack-ironic08:23
*** amotoki has quit IRC08:28
*** sacharya has joined #openstack-ironic08:28
openstackgerritMerged openstack/python-ironic-inspector-client: Updated from global requirements  https://review.openstack.org/33236708:31
openstackgerritNisha Agarwal proposed openstack/proliantutils: Fix the logic for fetching the gpu device count  https://review.openstack.org/31540208:33
*** sacharya has quit IRC08:33
*** rbartal has quit IRC08:34
sambetts|afkMorning all08:37
*** sambetts|afk is now known as sambetts08:37
*** bapalm has quit IRC08:37
openstackgerritMerged openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/33229608:38
*** derekh has joined #openstack-ironic08:38
*** bapalm has joined #openstack-ironic08:42
sergekmorning, Ironic!08:43
vdrokmorning ironic, lucas-brb _milan_ sambetts and sergek08:51
sergekvdrok: o/08:53
*** amotoki has joined #openstack-ironic08:53
_milan_morning sergek vdrok sambetts :)08:53
sergek_milan_: o/08:54
*** ccamacho has quit IRC09:10
*** snecklifter has quit IRC09:11
pas-hazigo: yes, the fix is in 5.1.2, tag pushed out just yesterday (when CVE was publicly disclosed)09:13
zigoFYI: I pushed 5.1.2 in Sid, and 4.2.5 to Jessie-backports.09:14
pas-hazigo: thanks09:15
openstackgerritGerard Braad proposed openstack/ironic-webclient: Set IP and port using environment variables  https://review.openstack.org/33270609:20
openstackgerritVladyslav Drok proposed openstack/ironic: Added operator documentation for ironic portgroups  https://review.openstack.org/22849609:21
openstackgerritVladyslav Drok proposed openstack/ironic: Add multitenancy-related fields to port API object  https://review.openstack.org/20624409:21
openstackgerritVladyslav Drok proposed openstack/ironic: Create common neutron module  https://review.openstack.org/31739009:21
openstackgerritVladyslav Drok proposed openstack/ironic: Add network_interface node field DB, OBJ  https://review.openstack.org/31739109:21
openstackgerritVladyslav Drok proposed openstack/ironic: Add 'neutron' network driver  https://review.openstack.org/31739309:21
openstackgerritVladyslav Drok proposed openstack/ironic: Update the deploy drivers with network flipping logic  https://review.openstack.org/21326209:21
openstackgerritVladyslav Drok proposed openstack/ironic: Add network_interface API  https://review.openstack.org/31739209:21
openstackgerritVladyslav Drok proposed openstack/ironic: Add network_drivers  https://review.openstack.org/28585209:21
openstackgerritVladyslav Drok proposed openstack/ironic: Add portgroups to support LAG interfaces - API  https://review.openstack.org/33217709:21
openstackgerritGerard Braad proposed openstack/ironic-webclient: Correct markup of README.md  https://review.openstack.org/33270709:24
*** ccamacho has joined #openstack-ironic09:27
openstackgerritMoshe Levi proposed openstack/ironic-python-agent: Add vendor id, vendor name and product_id to interface information  https://review.openstack.org/27994009:37
*** ifarkas has joined #openstack-ironic09:44
openstackgerritSam Betts proposed openstack/ironic-python-agent: Support LLDP data as part of interfaces in inventory  https://review.openstack.org/32058409:46
*** bapalm has quit IRC09:48
openstackgerritMoshe Levi proposed openstack/ironic-python-agent: Add vendor id, vendor name and product_id to interface information  https://review.openstack.org/27994009:50
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: WIP: Discovery tempest test  https://review.openstack.org/32376909:51
openstackgerrityogananth subramanian proposed openstack/ironic-python-agent: Exporting Network Interface Product and Vendor ID to Inspector  https://review.openstack.org/33149509:51
openstackgerritVladyslav Drok proposed openstack/ironic: Properly set ephemeral size in agent drivers  https://review.openstack.org/33008909:53
ifarkasmorning all!09:53
*** bapalm has joined #openstack-ironic09:54
sambettsJayF, rloo: https://review.openstack.org/#/c/324909/ has broken the IPA functional tests, I've only discovered it once I rebased my LLDP patch because apprently we don't have a job to catch it in the gate09:55
*** mgould|afk is now known as mgould09:55
mgouldmorning Ironic09:56
vdrokmorning ifarkas and mgould09:56
mgouldmorning vdrok ifarkas sambetts10:01
*** thrash|g0ne is now known as thrash10:01
*** fragatina has quit IRC10:01
*** yibo has quit IRC10:06
*** daemontool has joined #openstack-ironic10:07
*** appu103_ has joined #openstack-ironic10:07
*** yolanda has quit IRC10:10
*** fragatina has joined #openstack-ironic10:12
openstackgerritVladyslav Drok proposed openstack/ironic: Change the logic of selecting image for tests  https://review.openstack.org/32962510:16
*** daemontool has quit IRC10:19
*** ccamacho has quit IRC10:20
*** cdearborn has joined #openstack-ironic10:21
*** fragatina has quit IRC10:31
*** lucas-brb is now known as lucasagomes10:32
*** ccamacho has joined #openstack-ironic10:36
*** derekh has quit IRC10:42
*** dtantsur|afk is now known as dtantsur10:48
dtantsurMorning Ironic10:48
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Warn on legacy auth options on conductor start  https://review.openstack.org/32666010:49
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Update devstack plugin with new auth options  https://review.openstack.org/29731210:49
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Migrate to using keystoneauth Sessions  https://review.openstack.org/23698210:49
*** ohamada has quit IRC10:57
*** ohamada has joined #openstack-ironic10:57
*** hkominos has joined #openstack-ironic11:00
jrollzigo: 5.1.2 does, see the notes section here http://www.openwall.com/lists/oss-security/2016/06/21/611:00
jrolland now I see you pushed new packages11:01
jrollthanks <311:01
jrollmorning everyone11:01
zigo:)11:01
zigoYup, all done.11:01
* jroll finds the driver comp spec11:01
hkominosmorning people11:01
jroll48 comments O_O11:02
jrollhi hkominos :)11:02
dtantsurmorning jroll11:03
dtantsuroh yeah, much comments not wow dtantsur will bark11:04
*** snecklifter has joined #openstack-ironic11:04
jrolllol11:04
* jroll wants to make a creepy owlet version of that meme11:05
dtantsurlemme know what you think and I'll start updating11:05
dtantsurLOL just do it, I'll put it on my social networks :D11:05
sambettsmorning dtantsur, for some reason my client has decided your green today :-P11:06
sambettso/ jroll11:06
dtantsursambetts, your client is correct :D11:08
sambettsdtantsur: normally your orange, but recently it keeps changing it, you were pink 2 days ago11:09
hkominoshello openstackers. Is there any way/or log where I can actually see why i cant deploy a node? I mean that i am getting this generic error no host available but i want to know why the scheduler does not choose my hosts11:09
dtantsurhkominos, did you see http://docs.openstack.org/developer/ironic/deploy/troubleshooting.html#nova-returns-no-valid-host-was-found-error already?11:09
hkominosnope. I will take a look. (i should not the yesterday the same machine was selected for deployment!)11:10
lucasagomessambetts, ping error functional tests in IPA11:12
lucasagomesdid you open a bug about it?11:12
lucasagomessambetts, I'm opening one btw11:14
sambettslucasagomes: Thanks, I hadn't got round to that yet11:15
lucasagomessambetts, yeah I will put a fix up to it11:15
snecklifterdtantsur: morning, sorry I missed midcycle yesterday, I did the blueprint for software RAID which was discussed11:16
lucasagomesfound out what's broken already11:16
snecklifterPlease let me know if there is anything I can help with on that11:16
sambettslucasagomes: Oh nice! I think its probably that the disk wait thing is waiting longer than the tests are for the agent to start11:16
sambettslucasagomes: am I right?11:16
snecklifterBut reading etherpad it looks like there is some direction :)11:17
hkominosdtantsur : I am getting this error in my node-show11:17
hkominos| maintenance_reason     | During sync_power_state, max retries exceeded for node                   | |                        | 84b41f97-f01c-4496-8662-d8bc6db8647c, node state power on does not match | |                        | expected state 'power off'. Updating DB state to 'power on' Switching    | |                        | node to maintenance mode.11:17
hkominoshttp://paste.openstack.org/show/521143/11:17
*** phuongnh has quit IRC11:17
hkominosthis will be better. However what does it mean that i dont have power credentials? I can control the node through Ipmi11:18
dtantsursnecklifter, hi! a couple of folks volunteered to continue this effort, please stay tuned11:18
snecklifterdtantsur: will do, thanks11:18
dtantsurhkominos, seems like the power credentials are wrong or there is something wrong with your BMC11:19
sambettshkominos: that means that Ironic has failed to set the power state of the node to the one stored in the DB, and therefore has put the node in maintenance removing it from normal use11:19
hkominosBMC = conductor?11:20
*** chlong has joined #openstack-ironic11:21
sambettshkominos: https://en.wikipedia.org/wiki/Intelligent_Platform_Management_Interface#Baseboard_management_controller11:21
sambettshkominos: BMC is the part of the server your controlling that received IPMI requests and handles them11:22
openstackgerritLucas Alvares Gomes proposed openstack/ironic-python-agent: Fix functional tests  https://review.openstack.org/33275511:24
lucasagomessambetts, ^11:25
jrolldtantsur: okay, I'm done writing comments, I don't think I said anything crazy11:25
jrollmorning sambetts, lucasagomes :D11:25
* jroll releases IPA stable11:26
mgouldmorning jroll11:27
dtantsurlong term support is great, until you have to backport a CVE to all branches >_<11:27
lucasagomesjroll, hey there :-) morning11:28
jrollhey mgould \o11:28
jrolldtantsur: heh11:28
dtantsurwe support ironic since Juno..11:28
jrolloh ouch11:29
jrollglhf with that11:29
jrollspeaking of which, I still see kilo tags in git.o.o everywhere11:30
jrolldunno what's up with that11:30
sambettsdtantsur, jroll: my idea of using validation to verify the interfaces would help in the case of operator deciding they don't want to support a particular interface any more? or if on upgrade a vendor removes support for a particular interface from their hardware type? WDYT?11:30
dtantsuryou mean branches?11:30
jrolldtantsur: er, yeah brances11:31
jrollbranches, too11:31
dtantsurtags will stay, I guess. dunno why they didn't remove branches. tonyb?11:31
dtantsur(that's re stable/kilo)11:31
jrolldtantsur: I see it on nova too, so I assume there's still work to do11:31
jrollsambetts: oh, that's a good question, what do we do when a combination becomes invalid?11:32
dtantsurmmmmmm11:32
jrollthis is all too hard, let's all go fishing11:32
dtantsurgood question, sambetts11:32
dtantsur+111:32
jrollso yeah, I think node-validate should also handle these11:32
jrollmake sure all interfaces are compatible and enabled11:33
jrollbut, I don't think we should allow an invalid combination as proposed before11:33
dtantsurI think we need both: a check on update and node-validate to handle it11:33
jroll+111:33
sambetts+111:33
dtantsurplease leave the comments, I'll start bulk-addressing them in a few11:34
sambettsA problem that exists today is that you can have a node configured for a certain driver and then you can remove all conductors that support that driver and the node is then orphaned, and you can even do a node update to change its driver as far as I know because of locks etc and not being able to find a conductor, I wonder if we can fix that with this new system or not :/11:35
sambettsdtantsur: not sure where to leave my comment :-P11:35
dtantsurEVERYWHERE!!111:35
sambettshahaha11:35
jrolldtantsur: turns out I can go fishing right here, strange fish I've caught though http://i.imgur.com/KiuAWQI.jpg11:37
dtantsurlol, good catch :D11:38
sambettsheh11:38
lucasagomessambetts, jroll not very familiar with the project-config, but here's the proposed fix for the tox-func https://review.openstack.org/#/c/332766/11:39
jrolllucasagomes: what does that do, tox -e func?11:40
lucasagomesjroll, run functional tests against ipa11:40
jrollright11:40
jrollTIL that IPA has functional tests11:40
lucasagomesjroll, apparently josh added few11:40
lucasagomesvery few but there are some11:41
jrolllucasagomes: it's done with tox -e func, though, right?11:41
lucasagomesyuo11:41
lucasagomesyup11:41
jrollsoooo11:41
lucasagomesand mario as well11:41
lucasagomeshttps://review.openstack.org/#/c/224247/11:41
jrollit fails for me because IPA doesn't start11:41
lucasagomescorrect11:41
sambettsjroll: thats why we need a job, a patch landed last night that broke it11:41
jrollaha11:41
lucasagomesjroll, check the ML11:41
lucasagomesjroll, http://lists.openstack.org/pipermail/openstack-dev/2016-June/097940.html11:42
jrollugh I have to read the ML?11:42
jroll:P11:42
sambettsonly the Ironic part ;)11:42
* jroll has not done email yet this morning11:42
lucasagomesjroll, heh tl;dr, sambetts found out that it was broken11:42
jrollooo that's fun11:42
tonybdtantsur: I think we're part way through the process.  we tried somethign new and it kinda-sorta works now we're fixing that11:42
lucasagomesI've opened the bugs and added two patches to fix the thing and to add a job11:42
jrollsambetts: re https://review.openstack.org/#/c/332755/111:44
dtantsurtonyb, got it, thanks11:44
sambettslucasagomes: probably could just call the job gate-ironic-python-agent-func11:44
jrollsambetts: do we want to add a job that breaks our gate?11:44
lucasagomessambetts, maybe? I'm following the pattern there11:45
dtantsursambetts, this is a standard name11:45
dtantsurif he does not include "tox" he'll have to define this job himself11:45
lucasagomessambetts, and I think we should apply the fix as well before adding the job11:45
lucasagomesno harm on that11:45
dtantsurthe pattern is gate-{project}-tox-{env}11:45
lucasagomesyou can verify it manually11:45
sambettslucasagomes, jroll; yeah got it, I'll test and land11:46
sambettsdtantsur, lucasagomes: oh I though it would be like the the other jobs: gate-ironic-python-agent-python27 <- no tox there, but I guess its a different kettle of fish11:46
dtantsursambetts, see the func job on inspector, it's defined in a similar fashion11:47
*** sdake has quit IRC11:48
*** rbudden has joined #openstack-ironic11:49
sambettsdtantsur, lucasagomes: yeah, my mistake I'd only quickly skimed that patch, and didn't register that we were using an existing template11:50
* sambetts goes to get more coffee 11:50
*** chlong has quit IRC11:51
_milan_morning dtantsur jroll11:52
dtantsurhey _milan_11:52
_milan_o/ :)11:52
_milan_and morning mgould11:52
openstackgerritMerged openstack/ironic-python-agent: Fix functional tests  https://review.openstack.org/33275511:54
*** ccamacho has quit IRC11:55
*** ccamacho has joined #openstack-ironic11:56
hkominossambetts,dtantsur. This issue is really confusing. Why would ironic complain apout the BMC if i can use ironic CLI and control the power state?11:58
jrollhkominos: the CLI (and the API) for power state just sends the request, the API accepts it and does it in the background12:01
jrollas BMCs are often slow, we don't tie up the API waiting for the bmc12:01
*** jtomasek_ has joined #openstack-ironic12:02
*** sdake has joined #openstack-ironic12:03
*** dprince has joined #openstack-ironic12:03
*** jjohnson2 has joined #openstack-ironic12:05
sambettshkominos: I would try to ensure you can power on and off the server using ipmitool manaually, ipmitool —H 1.2.3.4 —v —I lanplus —U username —P userpassword chassis power on/off12:05
hkominosi can12:06
sambettsok, the node in Ironic is currently power off, and it will try to ensure that the real hardware is synced to that state, is any other software you have running trying to turn it back on?12:07
sambettsand therefore is fighting with Ironic12:07
hkominosI will get back on that. So it seems that the machine is trying to do some provisioning . However i get in a login-bootstrap login and get an error. Unable to find ext4 VFS (simithng like that)12:08
*** raildo-afk is now known as raildo12:09
hkominos(no there is no otherhardware trying to contact that node)12:09
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: WIP: Discovery tempest test  https://review.openstack.org/32376912:11
*** smoriya has quit IRC12:11
openstackgerritVladyslav Drok proposed openstack/ironic: Change the logic of selecting image for tests  https://review.openstack.org/32962512:12
*** e0ne has quit IRC12:13
*** e0ne has joined #openstack-ironic12:14
openstackgerritDmitry Tantsur proposed openstack/ironic-specs: The driver composition reform  https://review.openstack.org/18837012:17
dtantsurjroll, sambetts, next iteration ^^^12:17
jrollwoot12:18
jrolldtantsur: what'd you think about moving deprecation things to a separate spec?12:18
dtantsurjroll, I did that12:18
jrollawesome12:18
jrollthanks!12:18
dtantsurwell, I didn't write the actual spec yet ;)12:18
dtantsurbut I've removed all specifics about the deprecation12:19
*** jtomasek_ has quit IRC12:19
jrollya12:20
dtantsurI hope we merge this today, or I'll escape to the forest to live with other owls12:20
sambettshahahaha12:20
hkominoshahahhahahah12:22
*** makowals has joined #openstack-ironic12:23
*** jjohnson2 has quit IRC12:25
*** d0ugal has quit IRC12:25
*** d0ugal has joined #openstack-ironic12:25
*** jjohnson2 has joined #openstack-ironic12:25
*** thiagop has joined #openstack-ironic12:27
*** jjohnson2_ has joined #openstack-ironic12:29
thiagopGood morning, Ironic12:30
dtantsurmorning thiagop12:30
thiagophow's the midcycle going? I was unable to attend to the meetings so far (personal issues)12:30
*** jjohnson2 has quit IRC12:31
dtantsurthiagop, pretty productive, I would say12:35
jroll+112:35
dtantsurhad some RAID-related discussions; went over driver composition and boot from volume specs12:35
thiagopWas the review day on the network stuff done?12:39
thiagopI'd really like to follow that to try to get some glue on how it works...12:39
dtantsurwe didn't have a review day iirc, just had a brief discussion and then moved to future work on this topic12:40
dtantsurthe consensus on the current network patches: split away the portgroups, then start landing them12:40
thiagopuhmm12:40
*** sdake has quit IRC12:43
sambettsthiagop: the patches needed work and rebasing before we could review them, I think vdrok has processed them now, and they are open for reviewing again now12:45
thiagopnice to know sambetts, I'll try to get to them later today12:45
vdrokyeah, they seem to be ready for review, tho the multitenancy job seems to not work, there were some changes to tempest, i'm looking at it now locally12:46
jrolldtantsur: I'd like to think more about task.driver.deploy==None, idk how I feel about that, but I've +2'd again12:47
dtantsurjroll, I also have mixed feelings about it, but I also don't see any options. Well, we can refuse to work with this node completely, but that's even worse..12:48
thiagopThe zuul changes to stop using jenkins are taking some hairs off my head. Now that I'm almost done with the tempest job for our CI, infra is changing the jobs to use ansible12:48
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: WIP: Discovery tempest test  https://review.openstack.org/32376912:48
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add tests for openstackclient plugin  https://review.openstack.org/30168712:48
thiagopI'll try not to tackle it now and make the job works first12:48
jrolldtantsur: we can't, because node-update requires it :(12:48
sambettsthiagop: :(12:48
jrollthiagop: they've recommended third parties not move to just zuul yet12:48
sambettsthiagop: they recomned that third party CIs CD there test infra12:49
sambettsjroll: ^12:49
dtantsurjroll, correct. so this is kind of an edge case, which we should recommend operators to avoid.12:49
sambettsjroll: as a note I don't :-P12:49
thiagopjroll: really? That's really nice to know12:49
jrollsambetts: thiagop: http://lists.openstack.org/pipermail/openstack-dev/2016-June/097739.html12:49
jrolldtantsur: yeah, I'd just not like the entire codebase to explode :)12:50
sambettsjroll, thiagop: thats cool, that might simplfiy the test infra quite a bit12:50
jrolldtantsur: e.g. validate will need to check if each interface is None as well12:50
jrolloh, it does already12:51
jroll\o/12:51
thiagopjroll: cool, thanks12:51
dtantsur:)12:51
jrollbtw, we have a 6.0.0 tag now \o/12:52
dtantsurwoohoo12:52
jrollhttp://docs.openstack.org/releasenotes/ironic/newton.html#id112:52
thiagopFolks, if you have some slots on your review lists, our dynamic allocation code seems good enough for reviews as of now: https://review.openstack.org/#/c/286192/12:54
dtantsurwow, that's a huge list of changes12:54
dtantsurthiagop, probably after the midcycle, sorry :(12:54
thiagopdtantsur: don't be sorry, after the midcycle is more than fine12:55
jrolldtantsur: yeah, turns out we're doing lots of good work :D12:55
dtantsur\o/12:55
dtantsurlets celebrate12:55
jroll\o/12:56
thiagopI have a beer back at home. Will make a toast to Ironic later tonight.12:56
* jroll raises his coffee mug o/'12:57
*** ohamada has quit IRC12:57
* dtantsur wants to get beer for tonight as well12:57
thiagopholy cow, my coffee mug is empty12:57
* thiagop .fetch(coffee)12:58
* dtantsur gets the 3rd cup for today, which is something he really rarely does12:58
*** trown|outtypewww is now known as trown12:58
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add test for ironic driver-list command  https://review.openstack.org/33215012:59
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: WIP:Tempest: basic test: remove ports  https://review.openstack.org/33279412:59
*** thiagop is now known as thiagop-afk13:01
*** wagiel has joined #openstack-ironic13:03
*** lucasagomes is now known as lucas-hungry13:03
* lucas-hungry brb quick lunch13:03
wagielHello openstackers13:03
mgouldhi wagiel13:03
hkominoshallå13:04
hkominoshelloooo13:05
*** chlong has joined #openstack-ironic13:06
wagielI am trying to setup dhcp agent in ironic w/ dhcp rely thru vyatta. it that possible?13:07
wagiel*relay13:07
*** links has quit IRC13:09
*** slagle_ is now known as slagle13:10
*** rloo has joined #openstack-ironic13:11
_milan_wagiel, I don't know about Ironic but mind Inspector has it's own DHCP that it manages access to through iptables. You might have to workaround that if you need Inspector.13:13
sambettswagiel: are you using Ironic with Neutron?13:14
wagielsambetts: yes, neutron in use13:20
sambettswagiel: Ironic uses DHCP provided by neutron, I would explore if neutron has vyatta support if you want to use it13:24
*** appu103_ has quit IRC13:27
wagielwhen using neutron DHCP is it required to have baremetals in the same vlan/subnet as the ironic?13:28
sambettsits possible for the TFTP and http connection to Ironic to go through a neutron router13:29
*** Mr_Broke_ has joined #openstack-ironic13:29
hkominoswagiel. MY understating is this. Openstack has its own networks which can be Vlan tagged so lets say that for the BM network has a 100 VLAN tag. The only thing that you must provide is the baremetal machine gets access to the BM network only . However the machine must receive only untagged traffic. then the conductor will do the rest13:31
hkominossambetts:I am getting a weird error on the physical machine after provisioning . Cant find ext4 FS and then the deployment fails. Could that mean that it is an image problem?13:35
hkominosor an image metadata problem?13:35
sambettsI'd expect its a problem with the image itself, are you using whole disk images or partition images?13:36
*** sdake has joined #openstack-ironic13:37
openstackgerritSam Betts proposed openstack/ironic: Fail early if ramdisk type is dib, and not building  https://review.openstack.org/32824613:38
hkominosI have no idea. how do i find out?13:38
sambettshkominos: the image that your using, does it have multiple parts in glance, ie. image + kernel + initrd, or is it just a single thing13:40
wagiellet say i have bm network 10.10.10.x and ironic in network 20.10.10.x, dhcp relay thru vyatta, i need  neutron provided dhcp to serve bm network block 10.10.10.x13:40
hkominosmutliple parts13:41
*** Mr_Broke_ has quit IRC13:41
hkominosbut these were provided by fuel ie -->initramfs, linux, squashfs13:42
sambettsneutron if configured correctly will serve DHCP to that 10.10.10.x network, I don't know why you need vyatta, the DHCP server isn't running in the same network as Ironic13:42
wagielno13:42
wagieltwo separate networks/vlans13:43
sambettsyes, ok, so you need a neutron router between the two to allow the node to talk to Ironic, but you don't need to proxy DHCP becuase that should already be in the 10.10.10.x network13:44
sambettsbecuase neutron has put it there13:44
*** joprovost has joined #openstack-ironic13:45
*** lucas-hungry is now known as lucasagomes13:45
*** xhku has joined #openstack-ironic13:45
wagielok thank i will test some options13:47
rloohi and good morning on this last day of the midcycle :)13:48
sambettsGoood morning rloo, I forgot to ask did you see the red (Strawberry) moon on Monday? It was quite impressive here13:49
*** chlong has quit IRC13:49
rloosambetts: unfortunately no. I was disappointed. it was cloudy :-(13:50
sambetts:(13:50
rloosambetts: glad you got to see it!13:50
openstackgerritVladyslav Drok proposed openstack/ironic: Change the logic of selecting image for tests  https://review.openstack.org/32962513:50
dtantsurmorning rloo13:51
rloovdrok: what's with the reno? https://review.openstack.org/#/c/317390/7/releasenotes/notes/dhcp-providers-interface-change-53a2f48a292c9a6e.yaml13:51
rloohi dtantsur13:51
dtantsurrloo, I guess you know what I'm going to ask you, don't you? ;)13:51
rloodtantsur: driver composition is next, in a few seconds :)13:51
vdrokmorning rloo13:51
dtantsurawesome :) sorry for bothering you, I want to finish it so much..13:51
*** PollyZ has joined #openstack-ironic13:51
*** wagiel has quit IRC13:52
rloodtantsur: (thought I'd get an easy one done first to make me feel good, ha ha.)13:52
dtantsursure sure13:52
vdrokso IIRC that was something introduced in earlier patches and the intention was to mention it in one of the following patches13:52
rloovdrok: OH. Could you just add a separate patch for that reno?13:52
rloovdrok: it doesn't have to be in the chain.13:52
vdrokrloo: yeah, sure13:52
rloovdrok: it just seems silly to slip it in a patch that has nothing to do with it.13:52
vdrokrloo: otoh, if we merge it today, we can leave it there? :)13:53
vdrokyep, ok, it may be confusing for others13:53
rloovdrok: sigh. at a minimum, there shoudl be something in the commit. but really, it shouldn't be there.13:53
rloovdrok: should have been done as a followup to whatever patch.13:54
rloovdrok: are you sure that reno refers to something that already landed?13:54
vdrokrloo: let me check to be sure13:54
vdrokrloo: yes, it was merged, change-id I7f1cba65275078da750aa49ae83ba7345d6fd5e013:55
rloovdrok: ok, then do a follow up patch for that, and mention that change-id. it'll make it easier to review.13:56
rloovdrok: sorry about all this 'extra' work.13:56
*** derekh has joined #openstack-ironic13:57
vdrokrloo: np :)13:57
vdrokrloo: another thing is - we've merged that commit a long time ago, and I think we already released at least 1 major version (maybe 2) so we'll have to backport it? or just drop it completely and don't bother? jroll, wdyt?13:59
rloovdrok: OH. Who asked you to add a reno?13:59
*** baoli has joined #openstack-ironic13:59
vdrokrloo: I think you did :)14:00
*** baoli_ has joined #openstack-ironic14:00
vdroksee the last comment here https://review.openstack.org/#/c/206245/14:00
rloovdrok: I did? recently? (I mean, I could have, just don't remember)14:00
vdroknope, not recently14:01
vdrokin february :)14:01
*** ametts has joined #openstack-ironic14:01
*** baoli__ has joined #openstack-ironic14:01
rloovdrok: uhhhh, what took so long?14:01
vdrokrloo: so, I'm trying to remember the history. we started to merge multitenancy patches. this reno was added to the next in chain after that one ^^ but never merged14:02
rloovdrok: oh. that was when we thought we were going to get the/more code landed i think.14:02
vdrok++14:02
*** joprovost1 has joined #openstack-ironic14:03
rloovdrok: I say just forget about it. but maybe check with jroll. the email was sent out and no one has complained yet and it doesn't belong in the current set of renos. if anything, it should only be added to stable/mitaka.14:04
*** joprovost has quit IRC14:04
*** joprovost1 is now known as joprovost14:04
*** baoli has quit IRC14:04
*** baoli_ has quit IRC14:05
vdrokrloo: yep, OK. I will update the chain in a few14:05
*** chlong has joined #openstack-ironic14:06
*** ohamada has joined #openstack-ironic14:06
jrollrloo: agree with 'forget about it', I suspect nobody uses out of tree dhcp providers14:08
jrollafter a release it's too late to add a reno anyway, because it depends on git history14:08
rloojroll, vdrok: ok, we'll forget about it!14:08
jroll:)14:09
sambettsI guess this is why we should ensure we inlcude the renos in the original patches14:09
rloosambetts: ++, or a followup patch. No assuming that the next one in the chain will get in.14:09
rloos/get in/get in, in a timely manner/14:10
sambetts+114:10
sambetts(I do technically have an out of tree dhcp driver, but the in-tree networking stuff should superseed it anyway)14:11
sambettsbut then again I'm so involved with everything I don't need a reno to tell me :)14:12
vdroksambetts: I'm struggling with understanding why multitenancy job fails right now. It seems that a couple of variables are just not set correctly. I see IRONIC_NETWORK_INTERFACE=neutron in http://logs.openstack.org/32/296432/50/experimental/gate-tempest-dsvm-ironic-multitenant-network-nv/b77e769/logs/devstack-early.txt.gz and local.conf14:14
rloosambetts: that's right, no renos, we want to encourage everyone to be involved upstream in order to know what's going on!14:14
vdrokbut here it is empty - http://logs.openstack.org/32/296432/50/experimental/gate-tempest-dsvm-ironic-multitenant-network-nv/b77e769/logs/devstacklog.txt.gz14:14
vdroksambetts: ohhh, one more silly mistake :)14:15
vdroknevermind14:15
vdrokit's changed here for some reason https://review.openstack.org/#/c/296432/50/devstack/lib/ironic14:16
openstackgerritMerged openstack/python-ironicclient: Move methods to utils.py  https://review.openstack.org/33248914:17
sambettsvdrok: ah14:18
openstackgerritVladyslav Drok proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643214:19
*** rajinir has joined #openstack-ironic14:21
*** ptx0_ has quit IRC14:21
*** ptx0 has joined #openstack-ironic14:21
*** ptx0 is now known as Guest2313514:21
* vdrok goes afk for an hour14:23
*** praneshp has joined #openstack-ironic14:24
*** joprovost1 has joined #openstack-ironic14:24
*** joprovost has quit IRC14:25
*** joprovost1 is now known as joprovost14:25
rloodtantsur: let me know if you have any questions.14:29
rloodtantsur: wrt my comments wrt your driver composition14:29
*** PollyZ has quit IRC14:29
*** PollyZ has joined #openstack-ironic14:30
dtantsursure, didn't read yet14:30
*** yolanda has joined #openstack-ironic14:32
*** amotoki has quit IRC14:36
dtantsurrloo, replied. I think there is some misunderstanding here14:44
rloodtantsur: ok, looking.14:46
*** aarefiev_ has joined #openstack-ironic14:46
*** david-lyle_ has quit IRC14:47
*** naohirot has joined #openstack-ironic14:47
rloodtantsur: so wrt an interface becoming invalid (line 226), are we only talking about the case where it is disabled (ie, not a value in the enabled_*_interfaces config option)?14:49
dtantsurright14:49
rloodtantsur: the default interface will always be enabled even if not in the enabled_*_interfaces?14:49
dtantsuryes14:49
rloodtantsur: ok.14:49
dtantsurso if a user has a non-default interface which becomes disabled14:50
dtantsurwhich should be a very rare case, and we'll advise against doing that14:50
dtantsur(in the docs)14:50
cineramaoh crap, the dial in for midcycle is busy14:53
mat128cinerama: the PSTN provider has issues14:54
mat128-infra confirmed this with me yesterday14:54
dtantsuroops..14:54
mat128they suspect the voip.ms account ran out of money (yeah..)14:54
mat128apparently Foundation doesnt pay for it directly and people have to pay it and expanse it14:54
mat128they suggested using the SIP line directly14:54
*** hshiina has joined #openstack-ironic14:55
jrollheh, this happened last time14:55
jroll:(14:55
cineramaoh ok14:55
cineramawe might be able to get it sorted14:56
*** izumi777 has joined #openstack-ironic14:57
cineramaiirc some folks were having call quality issues w/sip client so it'd be nice if we can get it fixed14:57
mat128cinerama: I was having issues Monday, which is why I switched to the PSTN line using Hangouts14:57
*** jistr is now known as jistr|mtg14:58
jrollmidcycle time \o/14:59
rloodoes it mean that number doesn't work at all? i'm not set up to use sip :-(14:59
mat128rloo: thats right14:59
mat128:(14:59
rlooSIGH14:59
jroll:|14:59
mat128cinerama is in -infra trying to get it solved15:00
jrollrloo: they recommend installing jitsi, if that's possible for you15:00
mat128apparently the account balance is -8$15:00
*** rpioso has joined #openstack-ironic15:00
jrollthey are working on it though15:00
cineramashould be back in a few minutes, stay tuned15:02
mat128[11:01:34]  <jeblair> cinerama: it should be restored in a few mins15:02
mat128oops15:02
mat128rloo: ^15:02
jrollhshiina: naohirot: are you on the call? or will you be soon?15:02
*** rloo has quit IRC15:02
sambettsrloo, dtantsur: I'm also thinking there is a case of what happens if we upgrade Ironic and a vendor removes an interface from their supported interfaces on a hardware type15:03
hshiinajroll, i'm on call now.15:03
jrollgreat15:03
lucasagomesJayF, https://review.openstack.org/#/c/332766/15:04
*** yuriyz|2 has joined #openstack-ironic15:04
izumi777hshiina, sorry I can't connect to SIP...15:04
hshiinaizumi777, i see.15:05
sambettsjroll, lucasagomes, JayF: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/python-jobs.yaml#L30815:05
mrdaMorning15:06
jrollizumi777: what sort of problems are you having?15:06
jrollhshiina: will naohirot be joining?15:06
dtantsurlucasagomes, https://review.openstack.org/#/c/146092/15:06
hshiinajroll, he cannot call now, but he is in irc now.15:07
jrollhshiina: ok15:07
izumi777I use Jitsi, and try to conenct to sip:conference@pbx.openstack.org, "connected" message is shown, but I can't here any voice...15:07
mat128sip:7777@pbx.openstack.org15:07
mat128izumi777:15:07
jrollizumi777: ^15:07
*** moshele has quit IRC15:08
sambettsdtantsur: func3 would be cool to get running15:08
*** sacharya has joined #openstack-ironic15:08
dtantsursambetts, we're a project-config change away from it :)15:08
sambettstotally15:09
*** rcernin has quit IRC15:09
*** rloo has joined #openstack-ironic15:09
mat128lucasagomes, sambetts: running tox, if it works I can push that changeset, would you mind Lucas?15:09
izumi777Hmm, I changed the right address (sip:7777@pbx.openstack.org), but I can't listen any voice....15:10
lucasagomesmat128, don't mind no, I'm also running the tox -e zuul here locally15:10
naohirotHello jroll15:10
naohirotI didn't install sip client15:10
naohirotjroll: I tried PSTN but it doesn't work15:10
*** alineb has quit IRC15:10
*** yolanda has quit IRC15:10
*** sdake has quit IRC15:10
*** bapalm has quit IRC15:10
*** robcresswell_ has joined #openstack-ironic15:10
*** bapalm has joined #openstack-ironic15:10
*** yarkot1 has quit IRC15:10
*** robcresswell has quit IRC15:10
*** yolanda has joined #openstack-ironic15:10
*** robcresswell_ is now known as robcresswell15:10
jrollnaohirot: the PTSN is down, it will be back soon, infra is working on it15:10
*** tesseract- has quit IRC15:10
mat128izumi777: does it look anything like this: https://www.dropbox.com/s/xnzt0vf30gdvwzr/Screen%20Shot%202016-06-22%20at%2011.10.32%20AM.png?dl=015:10
jrollnaohirot: actually, I guess it is back now15:10
cineramajroll, naohirot: it's back up now15:10
*** yarkot1 has joined #openstack-ironic15:11
*** sdake_ has joined #openstack-ironic15:11
naohirotjroll: I just joined 7777 room15:11
izumi777Yes > mat12815:11
*** rloo_ has joined #openstack-ironic15:11
naohirotizumi777: you can try PSTN15:12
naohirotizumi777: PSTN is now working15:12
jroll"spray and pray" comes to mind15:13
mat128nmi and pray (TM)15:13
TheJuliayeah...15:13
lucasagomesmat128,   zuul: commands succeeded15:13
lucasagomes  congratulations :)15:13
mat128go :)15:13
izumi777is it toll-free? > naohirot15:13
mat128mine was taking longer to run15:13
lucasagomesmat128, pushing a new patch-set then15:13
mat128Number of jobs generated:  998515:13
mat128god15:13
naohirotizumi777: you need to call from inside company15:14
lucasagomesmat128, thanks a lot for testing it anyway!15:14
lucasagomesappreciate15:14
mat128my tiny MacBookAir can't cope with it15:14
mat128np, ever since we started using zuul and the whole setup downstream I learned a lot15:14
izumi777Oh, If so, I can't try PSTN, because I have no company PC and I don't know how to do it.15:15
sambettsNova NMI spec: https://specs.openstack.org/openstack/nova-specs/specs/liberty/approved/inject-nmi.html15:15
mat128izumi777: I personally used Google Hangouts for free US calls15:16
mat128making it a free call for me15:16
mat128izumi777: hangouts.google.com15:16
*** alineb has joined #openstack-ironic15:16
dtantsurnaohirot, that's me speaking15:16
mat128maybe we can switch to text?15:17
mat128I'm concerned kernels not configured to dump and reboot on NMI will not behave correctly if we expect power off after sending NMI15:17
izumi777Thank you for your kindness, but I have no experience Google Hangouts, so just a moment> mat12815:18
mat128naohirot: dtantsur was suggesting making 2 specs, one for soft power on/off and one for NMI15:18
*** sdake has joined #openstack-ironic15:18
dtantsurnaohirot, I'm asking what you think about splitting 2 specs like sambetts suggested15:18
izumi777text chatting is good for me too15:18
*** sdake_ has quit IRC15:19
*** rloo has quit IRC15:20
*** penick has joined #openstack-ironic15:22
*** pcaruana has quit IRC15:22
*** penick has quit IRC15:24
*** rcernin has joined #openstack-ironic15:24
mat128flash thought: I still want to be able to use NMI injection if I'm using SNMP power driver15:24
mat128it's similar to changing boot device15:24
openstackgerritZhenguo Niu proposed openstack/ironic-lib: Include wipefs --force option  https://review.openstack.org/31991815:26
*** mbound has quit IRC15:27
dtantsurFIY the notes are starting on line 33615:27
mat128https://review.openstack.org/#/c/186700/15:29
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Tests for CLI help message  https://review.openstack.org/26227315:30
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: tox: Update flake8 to ignore tinyipa imagebuild folders  https://review.openstack.org/33224715:31
*** jistr|mtg is now known as jistr15:32
jlvillalIs etherpad being troublesome for anyone else?15:32
mat128jlvillal: a1 here15:32
jrollyeah, all good for me15:32
jlvillalOkay. I keep getting force reload messages. Thanks15:32
jrolljlvillal: you might have it open in two tabs, and they're fighting between each other15:33
jlvillalI'll look15:33
*** praneshp has quit IRC15:33
* JayF brb15:35
*** itamarl has quit IRC15:36
*** pcaruana has joined #openstack-ironic15:36
jlvillalEtherpad issue: Only seems to get triggered when I go to: https://etherpad.openstack.org/p/ironic-newton-midcycle  and try to make Line 21 "Normal" style15:37
mat128jlvillal: easy fix for you, don't ;)15:37
mat128jlvillal: I had that "reconnect" popup when doing it15:38
jlvillalmat128, :P15:38
mat128but it seems to have worked15:38
mat128jlvillal: it didnt apply tho :(15:38
mat128weird15:38
mat128jlvillal: changing any line to normal does it15:38
mat128(that isnt already)15:39
jlvillalOh15:39
*** Guest23135 has quit IRC15:40
*** ptx0 has joined #openstack-ironic15:40
*** ptx0 has quit IRC15:40
*** ptx0 has joined #openstack-ironic15:40
devanandamorning, folks15:40
* devananda dials into the bridge15:40
mrdahey devananda15:41
dtantsurmorning devananda15:41
sambettshey devananda15:41
lucasagomesdevananda, morning15:41
*** spandhe has joined #openstack-ironic15:42
*** jaybeale has joined #openstack-ironic15:43
JayFjlvillal: re: the six vs oslotest thing, my general resolution is to use the library "closest" to us. oslotest is openstack specific, so we should use it. If there was no openstack lib, I'm all for using six.15:43
*** spandhe_ has joined #openstack-ironic15:43
JayFjlvillal: it's sorta like, we can farm out this problem "forever" to oslotest15:43
sambettshttps://specs.openstack.org/openstack/nova-specs/specs/liberty/approved/inject-nmi.html15:43
jrollhttps://specs.openstack.org/openstack/nova-specs/specs/liberty/approved/inject-nmi.html#proposed-change15:43
sambettsnova spec ^15:43
jlvillalJayF, Okay. I don't mind being over-rulled :)15:43
openstackgerritMerged openstack/ironic-python-agent: tox: Update flake8 to ignore tinyipa imagebuild folders  https://review.openstack.org/33224715:43
JayFwell there's no such thing as overruled15:44
JayFthere's consensus building15:44
* JayF entices jlvillal to join his consensus with candy and bribery15:44
jlvillalheh15:44
JayF*candy not included. **bribes not included.15:44
jrollbtw, welcome to the party devananda :D15:44
jlvillalJayF, I guess for some strange reason it bugs me to change the underlying library. Maybe I have no good reason :)15:44
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add test for 'baremetal port create' command  https://review.openstack.org/33288015:45
jlvillalSo pleasant music ;)15:45
JayFjlvillal: https://github.com/openstack/oslotest/blob/master/oslotest/base.py#L35 it's just a subclass of the class we were using15:45
jlvillalJayF, We we were using 'unittest'. I'd be happier to switch to 'testtools'15:46
jlvillalJayF, But likely I just need to be willing to change my mind...15:46
JayFI mean, again, I go back to my usual resolution: there's someone who cares about openstack writing testing libraries. I'd rather have that code helping us than to be different15:47
*** spandhe has quit IRC15:47
*** spandhe_ is now known as spandhe15:47
_milan_devananda, +1 on the logging of it (NMI); but should it be possible to reset that log/counter?15:47
*** Sukhdev has joined #openstack-ironic15:47
mat128_milan_: I suggest we implement something more generic if we want to throttle/rate-limit any kind of commands15:47
rloo_can we make it a vendor passthru until (if?) it becomes more apparent that it belongs in power or management interface?15:47
mat128_milan_: maybe for anything getting "into" the driver?15:48
jrollrloo_: it was originally proposed as vendor passthru for irmc, and we decided it should be implemented in the interface for all drivers to implement15:48
*** catintheroof has joined #openstack-ironic15:49
rloo_jroll: but if we can't decide (although the votes seem to go to management), why not temporarily put it as vendor passthru.15:49
jrollrloo_: management passthru or power passthru? :)15:49
rloo_jroll: i don't care, it is a passthru!15:49
jrollheh15:49
*** sabeen has joined #openstack-ironic15:49
_milan_mat128, yeah, like a generic log, +2 on that if it's accessible through API +200 ;)15:49
rloo_honestly, except for some issues (i know big issues) it seems like it makes more sense for power interface.15:49
mat128naohirot, devananda: is the behavior defined in redfish when you change the power state to "nmi"? does it come back to on? does it even have an effect on power state?15:50
*** chlong has quit IRC15:50
izumi777hshiina, naohirot: Did we reach the consensus about I/F?15:50
*** harlowja_at_home has joined #openstack-ironic15:50
rloo_izumi777: no consensus on interface. i think it was 5? for management, 1 for power... so far.15:50
rloo_suggested plan: anyone that has an opinion, go vote on the spec.15:51
*** adu has joined #openstack-ironic15:51
izumi777Could you tell me the reason why management is better ?15:51
rloo_izumi777: i would hope that the folks for management, would mention it in the spec.15:52
rloo_izumi777: mostly because with power interface, the actual power state is *known* after the request. with nmi, it is unknown.15:52
rloo_izumi777: naohirot just agreed that management is ok.15:53
izumi777rloo_: "unknown" means "it depends on user's configuration" ?15:53
*** makowals has quit IRC15:53
rloo_izumi777: no, don't think so. it depends on the hw. whether the nmi 'succeeds' or not.15:54
izumi777rloo_: NMI is "non-maskable" interrupts, so it always suceeds, I think.15:55
rloo_izumi777: sorry, I don't know much about this and i dialled in late. i haven't even read the spec.15:55
rloo_izumi777: the reasoning *should* be mentioned as a comment/review in the spec.15:56
izumi777rloo_: no problem, thanks  :)15:56
rloo_izumi777: i see that folks are commenting on the spec, as to why they want management, so best to read them.15:56
devanandahttp://redfish.dmtf.org/schemas/Resource.json15:56
izumi777Inject-NMI is also important feature for users, let's merge the spec, shall we?15:57
jrollizumi777: when the spec is updated to have this in the management interface, we will be :)15:58
izumi777jroll: okay :)15:59
*** daemontool has joined #openstack-ironic16:00
openstackgerritDmitry Tantsur proposed openstack/ironic: Add dbapi and objects functions to get a node by associated MAC addresses  https://review.openstack.org/33093816:00
openstackgerritDmitry Tantsur proposed openstack/ironic: [WIP] Introduce new driver call and RPC for heartbeat  https://review.openstack.org/33289016:00
*** dmk0202 has quit IRC16:02
*** _vsaienko has joined #openstack-ironic16:04
rloo_18 minute break.16:06
rloo_v2 API is next :)16:07
jrollback at 1625 utc16:07
devanandawhether we put this on the management or power interfaces, it will need to be optional -- some drivers will not be able to implement it16:08
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add test for 'baremetal port create' command  https://review.openstack.org/33288016:08
jrolldevananda++16:09
* jroll steps away16:09
devanandaalso, that will influence where we think of exposing it inthe REST API16:10
sambettsdevananda: totally, I think perhaps a do management action X API might be required, and the driver can response to it as such, ironic node-do-management-action <uuid> <action> ?16:11
devanandathe spec proposal is clear: both soft off and inject nmi are being added to the existing /states/power endpoint16:11
*** naohirot has quit IRC16:11
devanandasambetts: for generic management actions? maybe. but in this case ,the spec is clear: no new management actions are being added to the rEST API here16:12
devanandait's just a new target power state16:12
*** harlowja_at_home has quit IRC16:12
sambettsdevananda: yeah, I disbuted that because I don't believe it fits into Ironics understanding of a power action/state16:12
devanandasambetts: yea16:12
devanandaI agree - it does not16:12
devanandathat was one of my original issues with this proposal, about a year ago I think16:12
*** trown is now known as trown|lunch16:13
devanandasoft_off and soft_reboot make sense within our power API16:14
*** izumi777 has quit IRC16:14
devanandabut inject_nmi and push_power_button do not16:14
* devananda comments on spec real quick16:14
jlvillallucasagomes, https://review.openstack.org/#/c/321014/5/nova/virt/ironic/driver.py  How ugly is my suggestion? :)16:14
*** links has joined #openstack-ironic16:16
lucasagomesjlvillal, seems alright16:16
*** baoli__ has quit IRC16:16
lucasagomesmaybe we should not break, but def ur suggestion makes it easier to read16:17
jlvillallucasagomes, Not so elegant, but probably easier to read than that return statement ;)16:17
lucasagomesexactly16:17
*** baoli has joined #openstack-ironic16:17
*** Romanenko_K has quit IRC16:17
*** kromanenko_ has quit IRC16:18
*** kromanenko__ has quit IRC16:18
*** _vsaienko has quit IRC16:18
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Deny some driver_info updates when console is on  https://review.openstack.org/33000316:22
dtantsurdevananda, when you have a second, could you please also check the driver composition spec?16:23
dtantsurit does not look like we can merge it during the midcycle, but maybe at least this week...16:23
* jlvillal enjoyed not wearing a headset for 18 minutes :)16:25
* jroll hugs his super comfy headphones16:25
jlvillalheh16:25
mat128jroll: whats your setup?16:25
jrollmat128: I just got new plushy velour earpads, I use ATH-M50X headphones16:26
jrolland a blue mic16:26
jlvillalmat128, I think this is jroll's setup: http://gizmochunk.com/wp-content/uploads/2013/07/022112-Headphones.jpg  ;)16:26
jrollO_o16:26
mat128I'm personally using office-provided http://www.jabra.ca/products/wired_headsets/jabra_biz__1900_series/jabra_biz_1900_usb_duo It's not too bad but not great either16:26
*** spandhe has quit IRC16:26
mat128jroll: snowball blue?16:26
mat128erm16:26
mat128the yeti thing?16:26
sambettsI'm using a snowball16:26
jrolluh, yeah I think so16:26
jrollI forget exactly the name16:27
mat128thought you all had headsets16:27
jrollmat128: earpads: https://www.amazon.com/ATH-M50-PADS-VELVET-ATH-M50RD-ATH-M50WH-ATH-M50xBL-ATH-M50xWH/dp/B00F2ZM6CS16:27
jrolllook how soft16:27
mat128was about to get this to make my regular headphones a headset: http://www.modmic.com/16:27
jrollfriend got one of those, he likes it16:27
*** mbound has joined #openstack-ironic16:27
mat128https://www.shure.com/americas/products/headphones/srh840-professional-monitoring-headphones16:27
* mrda thinks jlvillal is jealous of his headphones16:27
jrollI use these as headphones much more than as a headset16:27
sambettsI use these headphones https://www.google.co.uk/webhp?sourceid=chrome-instant&ion=1&espv=2&ie=UTF-8#safe=off&q=sennheiser+17516:28
rajinirsambetts , thiagop-afk, is there a sample jenkins job script I can get that uses hardware for CI? Basically I'm trying to switch the gate to use our hardware nodes and looking for a sample16:28
jlvillalmat128, Jabra 930 Pro here. Or is it 920? I never remember16:28
mat128jlvillal: 1 ear?16:28
jlvillalmat128, Yep16:28
sambettsrajinir: I can provide a local.conf, but I don't have a gate style jenkins job16:29
sambettsrajinir: are you trying to provide hardware nodes to the openstack-infra CI?16:29
sambettsrajinir: or third party CI?16:29
rajinirsambetts: yes16:29
mat128I hear echo, anyone not talking could you please mute?16:30
rajinirsambetts: Should I be looking using the pre_hook_test to register the hardware nodes? How do I make the gate use the hardware instead o the virtual Vms16:30
thiagop-afkrajinir: mine is not yet ready, but you can take a look at it here: https://git.lsd.ufcg.edu.br/ironicdrivers/ci-project-config/blob/master/jenkins/jobs/ironic.yaml16:30
openstackgerritYossi Ovadia proposed openstack/ironic-python-agent: Handle diskless hardware connected to remote iscsi  https://review.openstack.org/32780716:31
*** mbound has quit IRC16:32
*** baoli has quit IRC16:33
*** baoli has joined #openstack-ironic16:33
openstackgerritDmitry Tantsur proposed openstack/ironic: Introduce new driver call and RPC for heartbeat  https://review.openstack.org/33289016:34
rajinirthiagop-afk: thanks will take a look16:34
rajinirsambetts: sure, anything will help16:34
openstackgerritMerged openstack/python-oneviewclient: Added functions to translate between uuid and uri  https://review.openstack.org/30187716:35
*** e0ne has quit IRC16:37
openstackgerritDmitry Tantsur proposed openstack/ironic: [WIP] Promote agent vendor passthru to core API  https://review.openstack.org/33001816:37
rajinirsambetts: Thirdparty CI16:37
*** daemontool has quit IRC16:37
*** athomas has quit IRC16:38
sambettsrajinir: http://paste.openstack.org/show/521216/ <- this is the local.conf that runs in my thirdparty CI16:39
*** links has quit IRC16:40
*** rcernin has quit IRC16:40
dtantsurinstance_uuid also provides a simple locking for us16:41
rajinirsambetts: thank you16:42
*** nicodemos has quit IRC16:43
*** baoli has quit IRC16:43
*** pcaruana has quit IRC16:43
*** jaybeale has quit IRC16:43
*** baoli has joined #openstack-ironic16:44
*** nicodemos has joined #openstack-ironic16:44
*** sacharya_ has joined #openstack-ironic16:47
mat128| properties             | {u'memory_mb': u'65536', u'cpu_arch': u'x86_64', u'local_gb': u'480',    |16:48
mat128|                        | u'cpus': 24, u'capabilities': u'class:metal,network_model:flat           |16:48
mat128|                        | active_passive bond lacp,hardware_cpu_model:Intel(R) Xeon(R) CPU E5-2620 |16:48
mat128|                        | v3 @ 2.40GHz,hardware_cpu_threads:24,hardware_cpu_count:2,hardware_netwo |16:48
mat128|                        | rk_link_speed_mbit:10000,hardware_storage_raid_controller:None,hardware_ |16:48
mat128|                        | storage_count:1,hardware_storage_size_gb:480,hardware_storage_1_size_gb: |16:48
mat128|                        | 480,hardware_storage_1_type:SSD,hardware_storage_1_interface:SATA'}      |16:48
jrollneat16:49
mat128thats a small one :S16:49
mat128one disk, etc.16:49
mat128staging :)16:49
dtantsurwow16:49
*** sacharya has quit IRC16:50
*** penick has joined #openstack-ironic16:50
jrollwhoa, speak of the devil16:50
lucasagomesmat128, hah16:50
jrollhi penick :)16:50
rloo_penick must have heard 'yahoo' mentioned16:50
penickrut roh16:51
*** xavierr has quit IRC16:51
rloo_we've always wanted to move capabilities out of properties16:51
penick(I don’t have backscroll)16:52
rloo_penick: wouldn't help. we're having our midcycle. you have to dial in16:52
jrollpenick: in midcycle, deva said "what if we got rid of arbitrary properties/capabilities" :)16:52
jrollwe decided we need it, no worries16:52
*** PollyZ has quit IRC16:53
*** derekh has quit IRC16:55
mat128Just got an interesting email titled: "Virtualization - Let's Go All The Way"16:55
mat128lmao16:55
sambettsBareMetal - Let's go further than all the way16:56
mat128"Baremetal - What powers your virtualization"16:56
mat128;)16:56
sambetts;)16:56
*** ohamada has quit IRC16:56
*** spandhe has joined #openstack-ironic16:56
openstackgerritImre Farkas proposed openstack/ironic: [WIP] DRAC RAID configuration  https://review.openstack.org/17631916:57
mat128https://wiki.openstack.org/wiki/API_Working_Group16:57
*** moshele has joined #openstack-ironic16:58
openstackgerritAparna proposed openstack/proliantutils: Sort values of raid_config for 'share_physical_disks'  https://review.openstack.org/32631916:58
*** PollyZ has joined #openstack-ironic16:58
*** jcoufal has joined #openstack-ironic16:59
mat128https://github.com/openstack/api-wg/tree/master/guidelines16:59
jrollor more readable, http://specs.openstack.org/openstack/api-wg/17:00
mat128AH17:00
mat128was looking for the rendered link17:01
mat128it isnt under docs.openstack.org17:01
*** mgoddard has quit IRC17:01
*** Jabadia has joined #openstack-ironic17:03
*** PollyZ has quit IRC17:03
Sukhdevjroll sambetts : any update on the networking patches? I am waiting for a ping from you guys :-)17:05
jrollSukhdev: no, we've been discussing other things, I don't expect we'll get to them17:05
JayFSukhdev: We're havnig a virtual-mid-cycle this week17:05
jrollSukhdev: at least in our call17:05
jrollJayF: you missed monday, we said we'd ping Sukhdev to jump in the call if we discussed networking more :)17:06
JayFI did miss monday17:06
sambettsSukhdev: they have been rebased but we've not got back to them in the midcycle, so they should be review/testable again17:06
*** Jabadia has quit IRC17:07
Sukhdevjroll sambetts : sure - I will try to update my testbed with the updated patches to make sure everything still works good -17:08
sambettsawesome :D thanks Sukhdev17:08
Sukhdevjroll sambetts : if you get to these during mid-cycle meeting, just ping me and I will jump on the call17:08
sambettsSukhdev: totally :D17:08
jrollsure17:08
SukhdevJayF : no worries :-)17:09
*** sdake_ has joined #openstack-ironic17:09
sambettsSukhdev: did you see my LLDP patches? I belive the inspector one has already merged, the IPA one needs a new patch set to update the commit message but is basically ready to go17:09
* jlvillal always wonders when it gets so quiet...17:10
mat128haha same17:10
*** electrofelix has quit IRC17:10
sambettsmy headphones cancel so much background noise, I wonder if I've gone deft17:10
*** sdake has quit IRC17:10
jlvillalsambetts, deft punk? ;)17:10
Sukhdevsambetts : I saw them, but, did not really review them - will look them over17:10
sambettsjlvillal: XD deaf...17:11
jlvillalsambetts, I thought you meant 'daft'.  heh17:11
sambetts;)17:11
*** watanabe_isao has joined #openstack-ironic17:12
jlvillalhttps://wiki.openstack.org/wiki/API_Working_Group#Members17:14
*** watanabe_isao has quit IRC17:14
jlvillalOh lucasagomes is on the API WG ;)17:14
lucasagomesjlvillal, yeah17:14
*** moshele has quit IRC17:15
dtantsurSHIGEMATSU Mitsuhiro is there as well17:15
dtantsurI think he participated in ironic stuff :)17:15
jlvillalMeeting logs. http://eavesdrop.openstack.org/meetings/api_wg/2016/  So people who actually attend the meetings might be a good place to start.17:16
sambettsDo we need to involve the working group that produces those user stories to ensure that we cover all the things that they want?17:16
jrollsambetts: I think not17:17
rloo_sambetts: how many user stories are there?17:17
jrollthose folks produce feature requests17:17
*** Sukhdev has quit IRC17:17
sambettsrloo_: I can't remember exactly, I just remember seeing that baremetal was like in the top 5 requests or something, I've tracked it since then17:18
sambettsi've not tracked it*17:18
jlvillalsambetts, Oh the group that surveys the users to see what they are using.17:18
jlvillalsambetts, There is also a "user story" group. Where they say, I am a user of Ironic and I want it to do ....17:19
*** snecklifter has quit IRC17:19
mat128break for 15 mins17:19
sambettsjlvillal: https://wiki.openstack.org/wiki/ProductTeam#OpenStack_Product_Working_Group17:19
sambettsjlvillal: I think thats it17:20
*** rcernin has joined #openstack-ironic17:20
sambettsjlvillal: they produced this https://github.com/openstack/openstack-user-stories/blob/master/user-stories/proposed/baremetal-service.rst17:21
jrollback at 1735 utc, next topic is client version negotiation17:21
jlvillalsambetts, Yep!17:21
* jroll steps away17:21
jlvillalsambetts, https://review.openstack.org/289627  :)17:21
jlvillalsambetts, That was the "user story" group I was thinking of17:21
sambettsjlvillal: ah right17:22
*** PollyZ has joined #openstack-ironic17:25
openstackgerritSam Betts proposed openstack/ironic-python-agent: Support LLDP data as part of interfaces in inventory  https://review.openstack.org/32058417:26
openstackgerritDmitry Tantsur proposed openstack/ironic: [WIP] Promote agent vendor passthru to core API  https://review.openstack.org/33001817:26
*** Jabadia has joined #openstack-ironic17:26
openstackgerritSam Betts proposed openstack/ironic-inspector: Add generic LLDP processing hook  https://review.openstack.org/32108217:27
*** sabeen has quit IRC17:28
JayFsambetts: I don't neccessarily think we should, but jroll suggested we override getattr/setattr in that NetworkInterface class to log deprecation17:28
JayFsambetts: seems like overkill to me but I want him to be +2 to doing it without that before landing that17:28
* JayF goes off to +2 to the pathc17:28
jlvillalsambetts, Would it make any sense for 'self.switch_port_descr' to become a setter method? And then log a warning about deprecation? Or is that overkill?17:28
* jlvillal is leaning towards overkill17:29
*** sdake_ has quit IRC17:29
* dtantsur is heavily leaning towards overkill17:29
jlvillalheh.17:30
*** Jabadia has quit IRC17:30
*** jlvillal has left #openstack-ironic17:30
*** jlvillal has joined #openstack-ironic17:30
*** PollyZ has quit IRC17:31
rloo_sambetts, jlvillal: yeah, thats the only user story i am aware of.17:31
openstackgerritYossi Ovadia proposed openstack/ironic-python-agent: Handle diskless hardware connected to remote iscsi  https://review.openstack.org/32780717:32
sambettsjlvillal: overkill I think because only people that are watching the IPA logs will see it17:33
jlvillalsambetts, Thanks17:33
rloo_sambetts, jlvillal, etc. I'm going to +A it17:33
sambettsthanks rloo_17:34
rloo_thank YOU sambetts!17:34
JayFCode merging always makes me happy \17:34
JayF\o/17:34
rloo_JayF: have you +A17:35
rloo_'d an ironic patch yet?17:35
JayFI think I did a requirements patch last night17:35
jlvillalsambetts, So what does TLV mean?17:35
jrollJayF: sambetts: I'm fine with just leaving it nulled out17:35
sambettsjlvillal: https://en.wikipedia.org/wiki/Type-length-value17:35
rloo_JayF: good. initiation is over for you :)17:35
JayFI mean, I had +A in IPA and Ironic specs for a while17:36
JayFI still generally prioritize IPA reviews over other things, just because my knowledge base usually makes the reviews more quality / faster17:36
rloo_JayF: yeah, I know. so not big deal for you! or sambetts actually17:36
mat128everyone back on the call?17:36
JayFI have to figure out the best way to take a bite of Ironic reviews, I think just using the dash and going down the list isn't the most effective use of time17:36
*** rloo_ is now known as rloo17:37
JayFrloo: the funny bit is, my very non-technical little brother was visiting, and asked what I did, and I told him I just got voted in as one of the 13 core "experts" on the software I work on17:38
sambettsjlvillal: basically its like a tuple kinda17:38
JayFrloo: big-brother-bragging-rights++ :P17:38
sambettsBTW people we're on L295+ in the etherpad17:38
rlooJayF: heh. I hope he was impressed!17:38
JayFhe got all the artistry in the family, I got all the engineer-y17:38
jlvillalsambetts, Thanks, +2 from me.17:39
sambettsjlvillal: sweet :D17:40
JayFIs there a reason we wouldn't want to explicitly add a "supported versions" endpoint to our public API?17:41
*** sabeen has joined #openstack-ironic17:41
JayFIt seems hacky to purposefully fail a request to gather information17:41
*** daemontool has joined #openstack-ironic17:41
dtantsurJayF, we have such17:42
jrollJayF: yeah, we have one, it's at /17:42
dtantsuror /v117:42
rlooJayF: i thought we already provided the supported versions if you do something like , yeah, that ^^17:42
JayFjroll: that doesn't return microversion afaict17:43
mat128the / endpoint reports "v1"17:43
mat128not 1.10 or wtv17:43
jrollJayF: yeah, I was wrong17:43
* lucasagomes brb for 1 min17:43
*** daemontool has quit IRC17:44
mat128I feel our client is not following the meta links in /v117:44
dtantsurmat128, we do have one of endpoints reporting microversions17:44
dtantsurI remember the patch adding it :)17:45
rloowe have documentation: http://docs.openstack.org/developer/ironic/webapi/v1.html17:47
mat128why isnt our client sending the initial request with the max it supports?17:47
rlooit sez, min version of 1.1 if no version requested by client. Wow, it is correct too :)17:47
rloomat128: cuz when we introduced new features/changes, we didn't want to break users.17:47
mat128I think we are confusing microversions with behavioral changes17:48
mat128if you are using microversion 1.1417:48
sambettsrloo: but unless they upgrade their client then their behaviour won't change17:48
mat128negociation should make it work as before17:48
rloomat128: i don't know if we are confusing. or what do you mean by 'behavioral' change?17:48
mat128between 1.1 and 1.wtv, nodes begin in enroll17:48
mat128thats a change in behavior17:49
mat128not like, we changed a fields name17:49
rloomat128: so microversions were added to deal with behavioral changes.17:49
rloomat128: what do you mean by 'confusing'. true they are not the same thing, but they are meant to deal with each other.17:49
mat128rloo: interesting, thats not what I would expect by a microversion change and I was indeed surprised when I have first seen this in Ironic17:50
mat128rloo: maybe it's just the wording, idk17:50
rloomat128: that 1.1 / enroll change, changed the value that was used for node.driver.17:50
rloomat128: so anyone writing/looking for node.driver=<iforgotwhatitwas>, it wouldn't work when we changed it to node.driver=enroll17:51
mat128rloo: in other words, I wouldn't expect my nodes to be created in enroll instead of active simply because I upgraded my python-ironicclient17:52
*** PollyZ has joined #openstack-ironic17:53
mat128which is why I think there is a mismatch in the meaning of microversions17:53
mat128when implementing new features17:53
rloomat128: can't have this discussion with you and listen to the talking. not sure we're in sync.17:53
mat128sry17:53
*** trown|lunch is now known as trown17:54
*** alineb has quit IRC17:57
*** PollyZ has quit IRC17:57
JayFI don't understand why the fact it's changing matters if we also bump major version of the client?17:58
dtantsurI feel like we're reinventing "latest" with this whole thing, tbh17:59
JayFI mean, not really. All my apps will still use 'latest' because I want to break early :)17:59
mat128JayF: version autonegociation means nodes are created in "enroll" instead of available. This isn't an API structure change, but a behavior change.17:59
mat128unless you pin to 1.1017:59
JayFmat128: you should expect change if you upgrade your client from X.y.z to X+1.0.018:00
dtantsurJayF, I don't see a difference between devananda's proposal and just always passing "latest"..18:00
mat128JayF: but signalling this through a major version works18:00
*** mgould is now known as mgould|afk18:00
*** ijw has joined #openstack-ironic18:01
jrolldtantsur: I understand it as "find the latest version and lock the client to that for subsequent requests"18:01
dtantsurstill, how is it different? :)18:01
mat128JayF: I think the microversion should have been bumped when we introduced 1.11 because it was a breaking change18:02
JayFmat128: I don't really have strong opinions about that, honestly18:03
mat128default --ironic-api-version to "latest" and it all works18:04
*** PollyZ has joined #openstack-ironic18:06
*** yogi has quit IRC18:07
sambettsI think it would be quite nice if I initalise an Ironic python client and pass in a particular version, then if I try to call ironic.thing_that_doesn't_exist_in_that_version you get an NotImplemented or TypeError or something18:08
dtantsurI personally want to pass a minimum possible version, then adjust features based on whether the actual version is higher18:09
sambettsthe same could happen if I request latest and it negoitates down, then you get a TypeError18:09
sambettsif you call an unsupported function18:09
dtantsurso e.g. inspector would pass minimum_version='1.2', then if actual version >= 1.11, allow discovery18:10
sambettsI'd also prefer if the client did the hiding/translating/compatibility of stuff, e.g. requests the version from the server, and turns things on and off based on if version >= version: and the server just always serves latest it supports18:12
sambettsif the version is 1.X it should only ever be adding new things18:13
* JayF brb quickly18:13
sambettsso old clients should always work with new Ironic18:13
mat128sambetts: but 1.11 :(18:13
sambettsmat128: :'(18:14
*** ChubYann has joined #openstack-ironic18:15
mrdarloo: 300 seconds18:19
rloomrda: ok. so not too long. i think when i reviewed, we might have discussed some way for the user to change that value with eg an environment var but don't think that happened.18:19
mrdahttps://github.com/openstack/python-ironicclient/blob/master/ironicclient/common/filecache.py#L2918:19
mrdarloo: that wouldn't be hard to add in18:20
rloomrda: right, which is why i didn't insist i think :)18:20
*** sacharya_ has quit IRC18:20
* mrda remembers that I think the dogpile library might have an env that you can override it anyway18:21
*** david-lyle has joined #openstack-ironic18:21
*** sacharya has joined #openstack-ironic18:21
rloomrda: it has occurred to me several times, that we need to be better about documenting :)18:21
mrdarloo: would you like that added?  A ironicclient env variable to allow cache override?18:22
rloomrda: i want to wait for someone to ask for it18:22
rloomrda: or someone could just add it18:22
* mrda is happy to throw a patch up18:22
* rloo is not ready to know/vote now18:23
* rloo hates microversioning. can't keep it straight cuz we used to have cli use the latest known version.18:28
*** e0ne has joined #openstack-ironic18:30
jlvillalSo, uh, can the client be used to request what is the maximum version supported? By both the client and the server?18:31
jlvillalAs in, print that info out to the user.18:32
mrda--debug18:32
rloojlvillal: i don't see why not. did you want a new command for it?18:32
rlooyes, --debug can be used but seems ugly.18:32
jlvillalI was just wondering if it did it.18:32
lucasagomesfolks I might need to drop a bit earlier today18:33
rlooi don't think there is anything that will explicitly return what the max version supported by client is.18:33
jrolljlvillal: there's not a "ironic show-me-the-versions" command, no18:33
lucasagomesI will check the etherpad later on18:33
jrollcheers, thanks for joining lucasagomes, have a good night :)18:33
jlvillalthanks lucasagomes  Ciao18:33
lucasagomesthanks a lot!18:33
rloojlvillal: you'd have to guess that based on behaviour i think. or look at code. the min/max from server is returned by ironic in response header. i think.18:33
rloobye lucasagomes18:33
mrdathanks lucasagomes!18:34
jrollyes, response headers18:34
jlvillalI wasn't sure if that would be a useful thing or not. Leaning towards not useful since it hasn't been done before :)18:34
sambettscya tomorrow lucasagomes o/18:34
rloojlvillal: i actually think it might be useful. so you don't need to do --debug :)18:34
lucasagomessee y'all tomorrow18:34
*** lucasagomes is now known as lucas-afk18:34
jrollwhat ruby said :)18:34
*** ijw has quit IRC18:37
*** ijw has joined #openstack-ironic18:38
rloojenkins debugging18:38
JayFdevananda: I'm locking myself in a dark place tomorrow until that spec gets done, regardless of if the gate breaks or anything else18:38
*** penick has quit IRC18:39
*** mbound has joined #openstack-ironic18:39
rloook, seemed like less brainpower needed, than for keystone policy stuff :)18:39
jrollbreak until 1900 UTC (18 minutes)18:41
rloobreak for 18ish minutes18:41
jrollkeystone policy discussion after that18:41
dtantsurfolks, I have to drop for today, sorry18:41
devanandaJayF: LOL18:42
*** ifarkas has quit IRC18:42
dtantsursee you tomorrow18:42
sambettso/ dtantsur, have a good evening18:42
*** dtantsur is now known as dtantsur|afk18:42
JayFrloo: devananda: monday 8a pst possible for that presentation?18:42
rloobye dtantsur|afk will look at your driver comp later today18:42
dtantsur|afkthanks!18:42
rlooJayF: that works for me. not sure about devananda or others.18:43
devanandaJayF: not really18:43
devanandaJayF: monday at 8am, renovation starts in my apartment -- I'll be working from a cafe until about noon, when I head to the airport18:44
jrollJayF: isn't that when our meeting is?18:44
devanandaactually yea, I have meetings from 8-11am on Monday already18:44
*** sdake has joined #openstack-ironic18:47
JayFoh valid.18:48
JayFdevananda: what days are you in next week? I'm out next week starting thursday.18:48
devanandatue-thu I'm offsite18:48
JayFI could also do this friday as well18:48
devanandaand I might get sucked into internal things on fri18:48
devananda:(18:48
devananda*on next fri18:48
JayFI'm out like Jun 29 - July 11 and maybe more18:48
devanandathis fri could work for me18:48
JayFrloo: ^ this fri? 8am pst?18:48
*** ijw has joined #openstack-ironic18:49
*** Sukhdev has joined #openstack-ironic18:52
openstackgerritMerged openstack/ironic: Properly set ephemeral size in agent drivers  https://review.openstack.org/33008918:53
*** hshiina has left #openstack-ironic18:54
*** sdake_ has joined #openstack-ironic18:55
*** sdake has quit IRC18:58
rlooJayF: Fri doesn't work for me. But I suppose it could work for me.18:58
rlooJayF: I'll make that sacrifice if needed :)18:58
*** achanda has joined #openstack-ironic18:58
JayFwell basically it sounds like either fri, or deva misses it, or it doesn't happen until late july18:58
rlooJayF: I don't think it is urgent. Just that jroll (at summit) offered to do it. heh.18:59
jrollrloo: shh. he might put it back on me.18:59
jroll:D18:59
* jroll runs away18:59
JayFjroll: you know me better than that, I like doing this sort of thing19:00
jrollindeed19:00
JayFknowledge sharing / presentations are by far the best part of the job19:00
devanandaafk a few minutes more19:00
JayFPartition image jobs for agent driver are landing now19:00
mat128yay19:01
sambettsL293/4  on the Etherpad19:01
*** achanda has quit IRC19:02
mat128JayF: this will have to be removed/fixed: https://github.com/openstack-infra/devstack-gate/blob/master/devstack-vm-gate.sh#L35619:03
JayFhttps://review.openstack.org/#/c/327437/19:04
*** adu has quit IRC19:05
jrollbrb, dog wants in19:05
jrollback19:06
mat128did I hear a cat?19:08
mat128^^19:08
jrollmine :D19:09
rlooso this will remain the same? https://review.openstack.org/#/c/325599/6/etc/ironic/policy.json19:09
openstackgerritMerged openstack/ironic-python-agent: Support LLDP data as part of interfaces in inventory  https://review.openstack.org/32058419:10
*** penick has joined #openstack-ironic19:10
*** penick has quit IRC19:10
rloosambetts: ^^ :D19:11
sambetts\o/ Sukhdev ^^19:11
Sukhdevsambetts : hi19:11
jrollhttp://specs.openstack.org/openstack/oslo-specs/specs/newton/policy-in-code.html#proposed-change19:11
mrdaSee y'all laters, /me heading back to bed.19:11
*** penick has joined #openstack-ironic19:12
sambettsSukhdev: just pointing out that the IPA patch for LLDP just landed :)19:12
rloonight (or morning) mrda19:12
*** e0ne has quit IRC19:12
sambettsSukhdev: the inspector part Depends-On the Ironic API patch to all local-link19:12
sambettsto add *19:12
Sukhdevsambetts : got it - thanks19:12
sambettsSukhdev: my brain is constantly mixing up our new local link info field and link local info from IPv6 :-P19:13
Sukhdev:-)19:14
jrollnova spec: http://specs.openstack.org/openstack/nova-specs/specs/newton/approved/policy-in-code.html19:14
jrollhttp://specs.openstack.org/openstack/nova-specs/specs/newton/approved/policy-in-code.html#work-items19:15
*** liliars has joined #openstack-ironic19:16
sambettsSukhdev: it doesn't look like we're going to make it to a group review of the networking patches, perhaps we should organise one of the networking subteam meetings on audio to do a group review instead of just saying hey heres the patches to review19:17
*** baoli has quit IRC19:17
Sukhdevsambetts : that sounds like a good idea - these are way overdue to be merged19:18
sambettsSukhdev: +100019:18
*** baoli has joined #openstack-ironic19:18
devanandahttps://review.openstack.org/#/c/325672/2/lib/nova_plugins/hypervisor-ironic19:19
sambettsjroll: are we taking notes anywhere?19:19
jrollsambetts: I have not been19:19
jrollcould someone?19:19
sambettsreason I ask is because I'm falling behind19:20
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/33236819:21
openstackgerritOpenStack Proposal Bot proposed openstack/python-oneviewclient: Updated from global requirements  https://review.openstack.org/33237419:21
*** livelace has joined #openstack-ironic19:21
Sukhdevdevananda : did you see the comment on the patch - do you mean "nova" or "ironic" for the user?19:22
pas-hahttps://github.com/openstack/nova/blob/master/nova/conf/ironic.py19:23
openstackgerritThiago Paiva Brito proposed openstack/ironic: [WIP] Custom power sync  https://review.openstack.org/33297819:26
*** livelace has quit IRC19:26
*** sdake has joined #openstack-ironic19:31
jrollrloo: just occurred to me I won't be here monday, and the next monday, so maybe weekly meeting isn't great for thinking on priorities19:31
jrollunless we wait for july 1119:31
rloojroll: july 11 is probably fine. we all know what the first priority is, don't we? :)19:31
jrollrloo: indeed!19:32
rloojroll: i am guessing we have the same priorities as decided at summit. it is the secondary ones i was interested in. i don't think we're going to get everything done, so i'd like us to focus to get at least some done.19:32
jrollrloo: indeed, I agree19:32
*** sdake_ has quit IRC19:33
*** MattMan has quit IRC19:33
*** MattMan has joined #openstack-ironic19:33
*** baoli has quit IRC19:35
*** baoli has joined #openstack-ironic19:35
* JayF not quite as good at following along on audio only19:36
*** joprovost has quit IRC19:40
*** fragatina has joined #openstack-ironic19:41
*** fragatina has quit IRC19:42
*** fragatina has joined #openstack-ironic19:42
*** sabeen has quit IRC19:42
*** sabeen has joined #openstack-ironic19:42
*** sabeen has quit IRC19:43
*** sabeen has joined #openstack-ironic19:43
pas-hahttps://github.com/openstack/heat/blob/master/etc/heat/policy.json#L319:46
*** yuriyz|2 has quit IRC19:53
*** baoli has quit IRC19:53
*** baoli has joined #openstack-ironic19:54
rloomat128: did you still have questions about the microversioning?19:58
mat128rloo: nah, I was just understanding it differently, but looking at our "log" of versions I guess it makes sense19:58
mat128http://docs.openstack.org/developer/ironic/webapi/v1.html19:58
jroll\o/ thanks everyone19:58
rloomat128: good. you've been brainwashed!19:58
mat128rloo: following semver there maybe would have helped, but what is done is done :)19:59
mat128haha19:59
rloothanks jroll for organizing etc19:59
rloomat128: hindsight etc...19:59
vdrokthank you all and good night!19:59
mat128jroll: want me to come up with a draft for the summary?19:59
rloonight vdrok19:59
jrollmat128: that would be super helpful, yes!20:00
mat128jroll: will do20:00
jrollmat128: feel free to email it to just me, or paste.o.o or whatever20:01
mat128I'll find something that works :)20:01
jrollthanks!20:01
mat128it's just text, dont worry20:01
mat128:P20:01
*** sdake_ has joined #openstack-ironic20:01
*** aarefiev_ has quit IRC20:02
sambettsmat128 is better at words that /me is20:02
mat128^was than intentional? (that/than) :)20:02
mat128will do my best to come up with something we can share with the rest of the group that couldnt assist20:03
pas-hatanks all again and g'night :)20:03
mat128was fun spending this time getting to know the group20:03
mat128thanks everyone20:03
sambettsmat128: ;) its 21:00 and I'm 3/4 through a beer20:03
mat128hah20:03
* JayF apologizes for being unmutted on bridge during opening of other meeting20:03
mat128JayF: I think you were alone at that point20:04
mat128;)20:04
sambettsother meeting?20:04
JayFdownstream.20:04
sambettsah :-P20:04
*** sdake has quit IRC20:04
* mat128 steps away until 0000 UTC for the next session20:05
*** baoli has quit IRC20:14
*** baoli_ has joined #openstack-ironic20:14
sambettsanyone know what the plan is for the 0000 session?20:20
jrollsambetts: my plan is whiskey20:21
sambettshaha, /me is on a late one today, I have a 22:00 meeting downstream because timezones :'(20:22
jrollouchhhh20:22
*** adu has joined #openstack-ironic20:22
*** sdake_ has quit IRC20:34
*** dprince has quit IRC20:35
Sukhdevjroll : hey I am down for some whisky - some good single malt :-)20:36
*** baoli_ has quit IRC20:39
*** fragatina has quit IRC20:39
*** hamzy_ is now known as hamzy20:40
*** jjohnson2_ has quit IRC20:46
*** spandhe has quit IRC20:48
*** spandhe has joined #openstack-ironic20:50
*** spandhe has quit IRC20:55
*** spandhe has joined #openstack-ironic21:01
*** trown is now known as trown|outtypewww21:04
*** yolanda has quit IRC21:06
*** PollyZ has quit IRC21:14
*** PollyZ has joined #openstack-ironic21:16
*** penick has quit IRC21:21
*** Sukhdev has quit IRC21:21
*** _Fdaisuke_ has joined #openstack-ironic21:25
*** Fdaisuke has quit IRC21:26
*** thiagop-afk has quit IRC21:28
*** sdake has joined #openstack-ironic21:28
*** yolanda has joined #openstack-ironic21:29
*** cdearborn has quit IRC21:35
*** yolanda has quit IRC21:36
*** sdake_ has joined #openstack-ironic21:38
*** sdake has quit IRC21:39
*** derekh has joined #openstack-ironic21:40
* sambetts -> bed 21:46
sambettsnight all o/21:46
*** sambetts is now known as sambetts|afk21:46
*** ametts has quit IRC21:50
*** Sukhdev has joined #openstack-ironic22:00
*** adu has quit IRC22:02
*** Sukhdev has quit IRC22:05
*** rbudden has quit IRC22:12
*** Sukhdev has joined #openstack-ironic22:15
*** sdake has joined #openstack-ironic22:21
*** ijw has quit IRC22:23
*** ijw has joined #openstack-ironic22:24
*** sdake_ has quit IRC22:24
*** rpioso has quit IRC22:28
*** achanda_ has joined #openstack-ironic22:29
*** jcoufal has quit IRC22:29
*** sdake_ has joined #openstack-ironic22:31
*** sdake has quit IRC22:34
*** sdake_ has quit IRC22:49
*** derekh has quit IRC22:54
*** penick has joined #openstack-ironic22:56
*** rcernin has quit IRC23:00
*** Sukhdev has quit IRC23:01
*** PollyZ has quit IRC23:04
*** zhenguo_ has quit IRC23:11
*** sdake has joined #openstack-ironic23:24
*** catintheroof has quit IRC23:35
*** Nisha_away has joined #openstack-ironic23:37
openstackgerritMerged openstack/ironic: remove neutron stuff from devstack deb packages  https://review.openstack.org/32625023:38
*** sdake has quit IRC23:42
*** sabeen has quit IRC23:47
*** baoli has joined #openstack-ironic23:47
*** sacharya has quit IRC23:51
*** baoli has quit IRC23:54
*** baoli has joined #openstack-ironic23:57
*** baoli has quit IRC23:58

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