Tuesday, 2016-10-18

*** srobert has quit IRC00:00
*** amotoki has quit IRC00:00
JayFmtanino: good luck; I'm calling it a day for now. I am around usually 1500-2400 UTC most days if you are having trouble after trying our reference config. Good luck!00:02
*** hogepodge has quit IRC00:02
mtaninoJayF: Thank you for your information. I'll try your proposed multitenant configration!00:03
mtaninoJayF: Seems that config using flat network for PROVIDERNET00:04
mtaninoThis seems work for me.00:04
*** hoangcx has quit IRC00:04
*** amotoki has joined #openstack-ironic00:05
*** hogepodge has joined #openstack-ironic00:08
*** david-lyle has quit IRC00:09
*** getvasanth has joined #openstack-ironic00:15
*** shobhar_ has quit IRC00:16
*** hogepodge has quit IRC00:20
*** hogepodge has joined #openstack-ironic00:21
*** david-lyle has joined #openstack-ironic00:21
*** mtanino__ has joined #openstack-ironic00:29
*** david-lyle has quit IRC00:29
*** Guest95387 has quit IRC00:30
*** mtanino has quit IRC00:30
*** amotoki has quit IRC00:30
*** david-lyle has joined #openstack-ironic00:31
*** david-lyle has quit IRC00:34
*** david-lyle has joined #openstack-ironic00:35
*** amotoki has joined #openstack-ironic00:40
*** amotoki has quit IRC00:44
*** anish__ is now known as anish00:44
*** david-lyle has quit IRC00:47
*** david-lyle has joined #openstack-ironic00:48
*** amotoki has joined #openstack-ironic00:48
*** david-lyle has quit IRC00:51
*** david-lyle has joined #openstack-ironic00:52
*** hoangcx has joined #openstack-ironic00:54
*** phuongnh has joined #openstack-ironic00:59
*** david-lyle has quit IRC00:59
*** david-lyle has joined #openstack-ironic01:00
*** lifeless has quit IRC01:02
*** lifeless has joined #openstack-ironic01:02
*** xuao has joined #openstack-ironic01:04
*** david-lyle has quit IRC01:05
*** david-lyle has joined #openstack-ironic01:06
*** david-lyle has quit IRC01:07
*** rpioso has quit IRC01:08
*** david-lyle has joined #openstack-ironic01:13
*** david-lyle has quit IRC01:14
*** david-lyle has joined #openstack-ironic01:15
*** hogepodge has quit IRC01:16
*** bdeschenes has quit IRC01:18
*** david-lyle has quit IRC01:19
*** david-lyle has joined #openstack-ironic01:20
*** baoli has joined #openstack-ironic01:21
*** david-lyle has quit IRC01:21
openstackgerritgengchc2 proposed openstack/ironic: Fix a typo in policy.py and firmware_processor.py  https://review.openstack.org/37848701:22
*** david-lyle has joined #openstack-ironic01:23
*** david-lyle has quit IRC01:24
*** david-lyle has joined #openstack-ironic01:25
*** david-lyle has quit IRC01:32
*** david-lyle has joined #openstack-ironic01:34
*** hogepodge has joined #openstack-ironic01:35
*** david-lyle has quit IRC01:37
*** yee3796 has joined #openstack-ironic01:38
*** david-lyle has joined #openstack-ironic01:38
*** david-lyle has quit IRC01:38
*** yee379 has quit IRC01:39
*** NobodyCam has quit IRC01:42
*** tdurakov has quit IRC01:42
*** tdurakov has joined #openstack-ironic01:42
*** NobodyCam has joined #openstack-ironic01:42
*** zhangyufei has joined #openstack-ironic01:43
*** david-lyle has joined #openstack-ironic01:44
*** david-lyle has quit IRC01:44
*** david-lyle has joined #openstack-ironic01:45
*** vdrok has quit IRC01:45
*** david-lyle has quit IRC01:46
*** david-lyle has joined #openstack-ironic01:46
*** david-lyle has quit IRC01:47
*** romcheg has quit IRC01:47
*** vdrok has joined #openstack-ironic01:48
*** romcheg_ has joined #openstack-ironic01:50
*** evgenyl has quit IRC01:51
*** evgenyl has joined #openstack-ironic01:52
*** david-lyle has joined #openstack-ironic01:56
*** david-lyle has quit IRC01:57
*** david-lyle has joined #openstack-ironic01:58
*** mtanino has joined #openstack-ironic01:59
*** baoli has quit IRC01:59
*** david-lyle has quit IRC02:00
*** mtanino__ has quit IRC02:00
*** david-lyle has joined #openstack-ironic02:01
*** david-lyle has quit IRC02:02
*** david-lyle has joined #openstack-ironic02:02
*** david-lyle has quit IRC02:03
*** david-lyle has joined #openstack-ironic02:04
*** david-lyle has quit IRC02:04
*** mtanino has quit IRC02:04
*** david-lyle has joined #openstack-ironic02:05
*** david-lyle has quit IRC02:05
*** snozhka has quit IRC02:06
*** rama_y has joined #openstack-ironic02:06
*** david-lyle has joined #openstack-ironic02:07
*** snozhka has joined #openstack-ironic02:08
*** david-lyle has quit IRC02:08
openstackgerritNaohiro Tamura proposed openstack/ironic: Add a missing error check in ipmitool driver's reboot  https://review.openstack.org/38718102:08
*** david-lyle has joined #openstack-ironic02:10
*** david-lyle has quit IRC02:11
*** david-lyle has joined #openstack-ironic02:12
*** rama_y has quit IRC02:13
*** tiendc has joined #openstack-ironic02:14
*** david-lyle has quit IRC02:14
*** tuanluong has joined #openstack-ironic02:14
*** david-lyle has joined #openstack-ironic02:17
*** david-lyle has quit IRC02:18
*** david-lyle has joined #openstack-ironic02:18
*** david-lyle has quit IRC02:19
*** david-lyle has joined #openstack-ironic02:20
*** david-lyle has quit IRC02:21
*** baoli has joined #openstack-ironic02:25
*** david-lyle has joined #openstack-ironic02:26
*** baoli_ has joined #openstack-ironic02:28
*** yuanying has quit IRC02:29
*** baoli has quit IRC02:31
*** syed_ has quit IRC02:34
*** baoli_ has quit IRC02:39
*** hogepodge has quit IRC02:40
*** thrash is now known as thrash|g0ne02:48
*** MattMan has quit IRC02:55
*** MattMan has joined #openstack-ironic02:55
*** gcb has joined #openstack-ironic02:57
*** hogepodge has joined #openstack-ironic03:04
*** portdirect has quit IRC03:05
*** portdirect_ has joined #openstack-ironic03:08
*** portdirect_ is now known as portdirect03:08
*** pester has joined #openstack-ironic03:09
*** vikrant has joined #openstack-ironic03:21
*** david-lyle has quit IRC03:23
*** david-lyle has joined #openstack-ironic03:24
*** david-lyle has quit IRC03:25
*** david-lyle has joined #openstack-ironic03:26
*** Sukhdev has joined #openstack-ironic03:26
*** david-lyle has quit IRC03:27
*** david-lyle has joined #openstack-ironic03:28
*** david-lyle has quit IRC03:29
*** deray has joined #openstack-ironic03:29
*** david-lyle has joined #openstack-ironic03:30
*** david-lyle has quit IRC03:33
*** david-lyle has joined #openstack-ironic03:34
*** links has joined #openstack-ironic03:35
*** david-lyle has quit IRC03:36
*** amotoki_ has joined #openstack-ironic03:40
*** amotoki has quit IRC03:43
*** david-lyle has joined #openstack-ironic03:46
*** david-lyle has quit IRC03:46
*** yuanying has joined #openstack-ironic03:49
*** david-lyle has joined #openstack-ironic03:50
*** appu103_ has joined #openstack-ironic03:50
*** ChubYann has quit IRC03:50
*** appu103_ is now known as aparnav03:51
*** david-lyle has quit IRC03:51
*** david-lyle has joined #openstack-ironic03:53
*** e0ne has joined #openstack-ironic03:54
*** david-lyle has quit IRC03:54
*** david-lyle has joined #openstack-ironic03:55
*** david-lyle has quit IRC03:55
*** david-lyle has joined #openstack-ironic03:57
*** david-lyle has quit IRC03:57
*** hj-hpe has quit IRC03:58
*** e0ne has quit IRC03:58
*** david-lyle has joined #openstack-ironic03:58
*** david-lyle has quit IRC03:59
*** e0ne has joined #openstack-ironic04:00
*** david-lyle has joined #openstack-ironic04:00
*** david-lyle has quit IRC04:01
*** david-lyle has joined #openstack-ironic04:01
*** david-lyle has quit IRC04:04
*** david-lyle has joined #openstack-ironic04:04
*** david-lyle has quit IRC04:05
*** ChubYann has joined #openstack-ironic04:05
*** david-lyle has joined #openstack-ironic04:06
openstackgerritMerged openstack/ironic: Remove 'agent_last_heartbeat' from node.driver_internal_info  https://review.openstack.org/38765204:06
*** david-lyle has quit IRC04:08
*** david-lyle has joined #openstack-ironic04:10
*** david-lyle has quit IRC04:11
*** david-lyle has joined #openstack-ironic04:12
*** david-lyle has quit IRC04:13
*** david-lyle has joined #openstack-ironic04:14
*** skazi has joined #openstack-ironic04:14
*** david-lyle has quit IRC04:15
*** david-lyle has joined #openstack-ironic04:15
*** david-lyle has quit IRC04:16
*** david-lyle has joined #openstack-ironic04:17
*** david-lyle has quit IRC04:18
*** david-lyle has joined #openstack-ironic04:19
*** david-lyle has quit IRC04:20
*** david-lyle has joined #openstack-ironic04:21
*** amotoki_ has quit IRC04:28
*** rama_y has joined #openstack-ironic04:40
*** skazi has quit IRC04:48
*** yuanying has quit IRC04:54
*** zhangyufei has quit IRC04:55
*** yuanying has joined #openstack-ironic05:00
*** yuanying has quit IRC05:02
*** zhangyufei has joined #openstack-ironic05:03
*** zhangyufei has quit IRC05:04
*** jaosorior has joined #openstack-ironic05:18
*** penick has joined #openstack-ironic05:26
*** ChubYann has quit IRC05:28
*** amotoki has joined #openstack-ironic05:36
*** bdeschenes has joined #openstack-ironic05:42
*** skazi has joined #openstack-ironic05:45
*** chlong has quit IRC05:47
*** skazi has quit IRC05:52
*** amotoki has quit IRC05:52
*** skazi has joined #openstack-ironic05:53
*** amotoki has joined #openstack-ironic05:57
*** rcernin has joined #openstack-ironic06:01
*** yuanying has joined #openstack-ironic06:01
*** links has quit IRC06:01
*** chlong has joined #openstack-ironic06:01
*** ccamacho has quit IRC06:04
*** amotoki has quit IRC06:04
*** amotoki has joined #openstack-ironic06:05
*** nicodemos has quit IRC06:09
*** nicodemos has joined #openstack-ironic06:10
*** vsaienko has joined #openstack-ironic06:12
*** jtomasek_ has joined #openstack-ironic06:13
*** mjura has joined #openstack-ironic06:15
*** zhangyufei has joined #openstack-ironic06:16
*** vsaienko has quit IRC06:17
*** charlledaniel has quit IRC06:17
*** charlledaniel has joined #openstack-ironic06:18
*** pcaruana has joined #openstack-ironic06:18
pas-hamorning Ironic :)06:21
*** mjura has quit IRC06:22
*** mjura has joined #openstack-ironic06:22
*** penick has quit IRC06:24
*** vsaienko has joined #openstack-ironic06:27
*** Sukhdev has quit IRC06:27
*** hogepodge has quit IRC06:30
*** jtomasek_ has quit IRC06:32
zhugaoxiaohi,everyone,how to build an centos6 ironic image?06:41
*** e0ne has quit IRC06:42
*** skazi has quit IRC06:44
*** mnasiadka has joined #openstack-ironic06:45
*** fxpester has joined #openstack-ironic06:56
*** skazi has joined #openstack-ironic07:02
*** tesseract has joined #openstack-ironic07:03
*** tesseract is now known as Guest8585507:03
*** ccamacho has joined #openstack-ironic07:05
openstackgerritNaohiro Tamura proposed openstack/ironic: Add a missing error check in ipmitool driver's reboot  https://review.openstack.org/38718107:08
*** amotoki has quit IRC07:09
*** amotoki has joined #openstack-ironic07:10
*** bdeschenes has quit IRC07:11
*** odyssey4me has quit IRC07:16
*** odyssey4me has joined #openstack-ironic07:18
*** vsaienko has quit IRC07:24
*** milan-afk is now known as milan07:24
milanmorning Ironic! :)07:24
milanmorning pas-ha, zhugaoxiao! :)07:24
zhugaoxiaomilan morning07:25
zhugaoxiaodo you know how to build centos6 image?07:26
cmysterzhugaoxiao: as in openstack build --all ?07:26
milanzhugaoxiao, unfortunately I've not tried this yet, or any other image, I'd forward you to the docs07:27
*** ohamada has joined #openstack-ironic07:30
milanzhugaoxiao, I've just hit this site with some docs: http://docs.openstack.org/developer/tripleo-docs/post_deployment/build_single_image.html07:30
*** ralonsoh has joined #openstack-ironic07:32
vsaienk0Good morning milan, pas-ha, vdrok, dtantsur, sambetts, lucasagomes and all Ironic'ers!07:34
milanmorning vsaienk0! :)07:35
*** hshiina has joined #openstack-ironic07:36
*** vsaienko has joined #openstack-ironic07:39
*** jpich has joined #openstack-ironic07:39
*** hogepodge has joined #openstack-ironic07:40
tuanluongmorning ironic07:45
zhugaoxiaomilan, https://bugs.launchpad.net/diskimage-builder/+bug/147717907:47
openstackLaunchpad bug 1477179 in diskimage-builder "centos 6 image doesn't boot after creation" [Medium,Triaged]07:47
milanmorning tuanluong!07:53
tuanluongmorning milan,07:53
milanzhugaoxiao, I see :-/07:53
*** fxpester has quit IRC07:57
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:01
*** fxpester has joined #openstack-ironic08:03
*** makowals_ has joined #openstack-ironic08:04
*** openstackgerrit has quit IRC08:04
*** openstackgerrit has joined #openstack-ironic08:05
*** makowals has quit IRC08:05
*** tuanluong has quit IRC08:11
*** makowals_ has quit IRC08:18
*** makowals has joined #openstack-ironic08:19
*** bdeschenes has joined #openstack-ironic08:22
*** vsaienko has quit IRC08:30
pas-hamorning vsaienk0 milan08:31
vdrokmorning milan vsaienk0 pas-ha and everyone else!08:32
milanmorning vdrok! :)08:32
openstackgerritNaohiro Tamura proposed openstack/ironic: Generic power interface for soft reboot and soft power off  https://review.openstack.org/21673008:36
aarefievmorning all!08:37
openstackgerritVasyl Saienko proposed openstack/ironic: Trivial: fix typo in docstring  https://review.openstack.org/38786008:40
*** electrofelix has joined #openstack-ironic08:42
vdrokgood morning aarefiev08:43
aarefievmorning vdrok08:43
*** athomas has joined #openstack-ironic08:48
*** derekh has joined #openstack-ironic08:48
*** Romanenko_K has joined #openstack-ironic08:55
*** aparnav has quit IRC09:01
*** appu103_ has joined #openstack-ironic09:01
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui: Imported Translations from Zanata  https://review.openstack.org/38787209:02
*** openstackgerrit has quit IRC09:04
*** openstackgerrit has joined #openstack-ironic09:04
milanmorning aarefiev! :)09:08
aarefievmorning milan09:08
milanaarefiev, so 2 vars or 1 var ;)09:08
aarefievI'd prefer one09:08
milanaarefiev, max_limit then? the current state?09:11
*** bdeschenes has quit IRC09:12
milanI still have to move it to the [service] section from [api] I'm afraid?09:12
aarefievmax_limit or default_limit09:12
aarefievno matter for me09:12
aarefievyeah09:12
milancool09:12
aarefievbtw api-wg says something about this09:13
aarefiev?09:13
milanlet's stick with max, if in the future it sucks, we can introduce a default09:13
milanas a second variable09:13
* milan looks09:13
*** mjura has quit IRC09:14
aarefievmilan: https://review.openstack.org/#/c/190743/2109:16
patchbotpatch 190743 - api-wg - Add description of pagination parameters (ABANDONED)09:16
aarefievit says only limit09:17
*** vmud213 has joined #openstack-ironic09:17
milanyeah09:17
milanaarefiev, i've just git grepped https://github.com/openstack/api-wg/blob/master/guidelines/http.rst09:18
* milan reads09:18
milanaarefiev, so there is https://github.com/openstack/api-wg/blob/master/guidelines/http.rst#failure-code-clarifications in addition09:19
milanif the user requests a limit over, we should return an 40009:19
*** pmannidi has quit IRC09:19
milanI think the code atm handles that09:19
milanwell, no, we'd silently set the max in that case :-/09:20
milanaarefiev, https://review.openstack.org/#/c/344921/28/ironic_inspector/api_tools.py@95 will have to assert the limit isn't over09:21
patchbotpatch 344921 - ironic-inspector - Add API for listing all introspection statuses09:21
milanwdyt?09:21
sambetts|afkSukhdev: hi? whats up?09:22
*** sambetts|afk is now known as sambetts09:22
sambettsMorning all09:22
milanmorning sambetts! :)09:22
sambettso/ milan09:22
aarefievmilan: I guess it talks about failure when limit too big or missing09:22
aarefievmorning sambetts09:23
milanyeah, exactly, if it's too big, we should give a 400 response09:23
milanif absent maybe too?09:23
milanif marker is present and limit absent?09:24
milansambetts, FYI, we've been discussing https://github.com/openstack/api-wg/blob/master/guidelines/http.rst#failure-code-clarifications < limit handling for https://review.openstack.org/#/c/344921/09:24
patchbotpatch 344921 - ironic-inspector - Add API for listing all introspection statuses09:24
aarefievhttps://bugs.launchpad.net/openstack-api-wg/+bug/159330409:25
openstackLaunchpad bug 1593304 in openstack-api-wg "There is not yet an accepted guideline for pagination" [Undecided,New]09:25
milanhmmm09:25
milanOK, so let's stick with what we have: eg min of max and user's input?09:25
milanno error if marker present but limit absent?09:26
aarefievin proposal: "And if 'limit' is not set, the number of items returned will be the maximum allowed, and this is generally controlled via config options in each project."09:26
aarefievI guess it's how all projects work09:26
* aarefiev on meeting09:27
milanaarefiev, ack, let's leave it as it is then09:27
milanenjoy the meeting09:27
aarefievheh )09:27
sambettsyeah it seems like we're doing it the right way here09:27
*** hshiina has quit IRC09:28
sambettsalthough I think we should 400 if the requested is greater than the configuration09:28
milansambetts, please, check https://review.openstack.org/#/c/344921/25/ironic_inspector/conf.py@28 if you've got opinion on the conf var name, thx09:29
patchbotpatch 344921 - ironic-inspector - Add API for listing all introspection statuses09:29
milansambetts, that's how I understand https://github.com/openstack/api-wg/blob/master/guidelines/http.rst#failure-code-clarifications09:29
*** e0ne_ has joined #openstack-ironic09:31
milansambetts, w/r the var name, current vote is in favour of max_limit as a single var that'd control the limit09:31
milanwe could add default_limit in the future if we find it necessary one day09:32
sambettsmilan: posted my thoughts on the patch09:34
*** mjura has joined #openstack-ironic09:34
milansambetts, thanks!09:35
milancool, so if I'm not mistaken, we've got a conclusion and I've got a TODO list \o/ :D09:37
milansambetts, tiny question, no objections for the max_limit to be moved from [api] to [service_opts]? line @3209:38
sambettsmilan: makes sense to me09:40
milanack09:40
*** mgould|afk is now known as mgould09:41
mgouldmorning Ironic!09:41
mgouldmorning milan aarefiev sambetts09:41
milanmorning mgould! :)09:41
*** jaosorior has quit IRC09:42
*** jaosorior has joined #openstack-ironic09:42
*** wznoinsk has quit IRC09:47
*** pmannidi has joined #openstack-ironic09:48
*** dtantsur|afk is now known as dtantsur09:50
dtantsurMorning Ironic09:50
*** wznoinsk_ has joined #openstack-ironic09:50
dtantsurJayF, https://bugs.launchpad.net/python-ironic-inspector-client09:51
*** wznoinsk_ is now known as wznoinsk09:51
milanmorning dtantsur! :)09:55
vdrokmorning sambetts mgould and dtantsur09:56
mgouldmorning vdrok09:57
openstackgerritMerged openstack/ironic: Add a missing error check in ipmitool driver's reboot  https://review.openstack.org/38718109:57
*** moshele has joined #openstack-ironic09:58
*** electrofelix has left #openstack-ironic10:02
*** amotoki has quit IRC10:03
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Catch error when wrong temp_url were generated  https://review.openstack.org/37087510:04
aarefievmorning mgould, dtantsur10:05
*** getvasanth has quit IRC10:07
*** tuanluong has joined #openstack-ironic10:12
*** xuao has quit IRC10:16
*** pmannidi has quit IRC10:16
*** zhangyufei has quit IRC10:30
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: PoC Serve iPXE configs from Ironic API  https://review.openstack.org/36943810:33
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: PoC Dynamic iPXE with core+vendor API  https://review.openstack.org/38791610:33
*** hoangcx has quit IRC10:33
*** tuanluong has quit IRC10:35
openstackgerritMerged openstack/ironic: Fixes parameters validation in SSH power manager.  https://review.openstack.org/36615010:36
*** thrash|g0ne is now known as thrash10:36
openstackgerritMerged openstack/ironic: Updated from global requirements  https://review.openstack.org/38686010:36
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Set workaround for test_network_basic_ops in grenade setting  https://review.openstack.org/38740310:36
openstackgerritSam Betts proposed openstack/ironic: Allow to use FLAT network setup  https://review.openstack.org/33438210:36
openstackgerritSam Betts proposed openstack/ironic: Add IRONIC_TEST_NETWORK_NAME devstack plugin var  https://review.openstack.org/33811710:36
sambettsvsaienk0: I rebased those patches ^10:37
*** tiendc has quit IRC10:40
*** jtomasek is now known as jtomasek|biab10:40
openstackgerritShivanand Tendulker proposed openstack/ironic: Support to perform only out-of-band cleaning for iLO drivers  https://review.openstack.org/38792110:42
*** chlong has quit IRC10:46
*** phuongnh has quit IRC10:47
*** hshiina has joined #openstack-ironic10:48
openstackgerritMerged openstack/ironic: Trivial: fix typo in docstring  https://review.openstack.org/38786010:53
*** sborkows has joined #openstack-ironic10:54
*** portdirect has quit IRC10:59
*** awiddersheim has quit IRC11:00
*** portdirect has joined #openstack-ironic11:00
*** aswadr_ has joined #openstack-ironic11:03
*** caiobo has joined #openstack-ironic11:11
*** lucas-afk is now known as lucasagomes11:12
lucasagomesmorning all11:12
*** dprince has joined #openstack-ironic11:14
*** dprince has quit IRC11:14
caiobogood morning everybody11:15
caiobogood morning lucasagomes :)11:15
dtantsurmorning lucasagomes, caiobo11:15
caiobomorning dtantsur11:16
*** amotoki has joined #openstack-ironic11:17
mat128morning everyone11:21
*** ccamacho is now known as ccamacho|lunch11:33
openstackgerritAnton Arefiev proposed openstack/ironic-specs: Torrent-based provisioning  https://review.openstack.org/31109111:35
charlledanielmorning all11:41
sambettsMorning charlledaniel11:42
dtantsursambetts, milan, aarefiev, can one of you represent ironic-inspector on https://www.openstack.org/summit/barcelona-2016/summit-schedule/events/16984/qa-tbd? this will be about QA plugins (grenade, tempest). it's a good chance to raise e.g. our grenade problems, but I have two conflicts for this slot already11:43
dtantsurdmellado, ^^^11:43
dmelladodtantsur: ack11:43
dmelladothanks!11:43
milandtantsur, I can, let's share points we'd like to present there?11:46
sambettsdtantsur: I have a conflict with the ironic neutron events session, so ideally I want to be in that one11:46
milan*s/present/highlight/complain about ;)11:47
aarefievyeah, ironic neutron events session conflict11:47
dtantsurmilan, I'm not strictly sure about the agenda, but we're suffering from the fact that we're dictated which tests to run in grenade11:47
dtantsurmilan, I'd like grenade to be much, much more versatile. currently it's a great example of nova-centric thinking of openstack11:48
dtantsurdmellado, fyi ^^^11:48
dmelladodtantsur: tbh I'm not involved into grenade, at all11:48
dmelladobut we could try to discuss that too11:49
sambettsdtantsur: its tempest smoke thats the issue right?11:49
*** moshele has quit IRC11:49
milandtantsur, nova-centric is very true and generic, that might be indeed a good topic :)11:49
*** alex_xu_ has quit IRC11:50
sambetts++ customisable tempest tests for grenade, instead of just tempest smoke would be interesting11:50
*** heyongli has quit IRC11:51
charlledanielmay someone review my patch https://review.openstack.org/#/c/372762/?11:51
patchbotpatch 372762 - ironic - Ensures that OneView nodes are free for use by Ironic11:51
*** alex_xu has joined #openstack-ironic11:51
dtantsursambetts, yes11:52
aarefievtest_network_basic_ops fails on grenade11:52
*** heyongli has joined #openstack-ironic11:53
*** jtomasek|biab is now known as jtomasek11:54
*** dprince has joined #openstack-ironic11:55
milanaarefiev, dtantsur, sambetts ack, I've created myself a calendar so I hope to be there :)11:55
sambettsawesome thanks milan!11:55
dtantsurthanks!11:55
openstackgerritVasyl Saienko proposed openstack/ironic: Switch to Neutron Flat network.  https://review.openstack.org/34069511:57
dtantsurdmellado, omfg https://review.openstack.org/#/c/369749 is horrible. just pure evil.11:58
patchbotpatch 369749 - governance - Add Ocata goal split out tempest plugins (ABANDONED)11:58
*** fxpester has quit IRC12:00
aarefievthanks milan12:01
TheJuliaGood morning12:01
dmelladodtantsur: heh, it ain't that bad12:01
milanmorning TheJulia! :)12:01
dtantsurno, it is. if it lands, I'll consider dropping tempest12:01
dtantsurdmellado, ^^^12:01
dtantsurmorning TheJulia12:01
dmelladootherwise how could you solve the issue of the main project being branched and the tests being branchless?12:01
dmelladodtantsur: that's why I also would love to have some feedback and input12:02
dmelladomtreinish: ^^12:02
dtantsurmilan, if you agree with my take on https://review.openstack.org/#/c/369749 please make sure to voice it12:02
patchbotpatch 369749 - governance - Add Ocata goal split out tempest plugins (ABANDONED)12:02
* milan looks12:02
*** vsaienko has joined #openstack-ironic12:06
TheJuliacharlledaniel: Any reason why the OneView CI is failing that revision?12:13
nicodemosmorning, ironic.12:15
vdrokgood morning TheJulia and nicodemos12:16
sambettsdtantsur: I kind of agree with the idea of splitting them out, I'm probably +0.5 right now, with the reason being that tempest tests should be black box, so changes into inspector (unless its a public API change) won't include tempest plugin changes unlike with UTs/functional where I expect every patch to include the those tests with the code changes12:17
*** appu103_ has quit IRC12:17
mrtenioMorning all.12:17
milandtantsur, it's a complex issue :-/ but I agree with some parts: 2) A packager tries to use a stable version of your project with a newer tempest 3) ronic which we hacked together a mechanism to install the project repo from master in the tempest venv while the point of a single-commit test&API change make a lot of sense :-/12:17
*** vsaienko has quit IRC12:18
*** trown|outtypewww is now known as trown12:19
*** portdirect has quit IRC12:20
*** portdirect has joined #openstack-ironic12:20
*** krtaylor has joined #openstack-ironic12:23
*** bfournie has quit IRC12:25
*** vsaienko has joined #openstack-ironic12:35
lucasagomesvdrok, did you test https://review.openstack.org/#/c/385957/ locally ?12:38
patchbotpatch 385957 - ironic-python-agent - Fix config drive writing script12:38
vdroklucasagomes: morning, yup, I did12:39
lucasagomescool :-)12:39
lucasagomesvdrok, morning12:39
vdrokI can link you a gpt cirros image if you want to check12:39
lucasagomesvdrok, +A'd12:39
lucasagomesno worries, we trust you12:40
lucasagomesjust wanted to double check before approval12:40
vdroklucasagomes: heh, thanks :)12:40
*** krtaylor has quit IRC12:41
*** dougs has joined #openstack-ironic12:41
mgouldmorning TheJulia nicodemos12:45
dtantsursambetts, yes, but... practically speaking, it will slow done adding new tests substantially. e.g. for ironic we had a hell lot of gate work ahead. being branchless means we'll have a code full of "if"s12:47
*** portdirect has quit IRC12:47
*** portdirect has joined #openstack-ironic12:48
sambettsdtantsur: tempest is built of turning things on and off based on avaible features, so I don't think that being branchless is a problem12:48
dtantsursambetts, current tempest configuration is a nightmare fwiw. you have to do a lot of implicit things to make it pass for ironic, for example12:50
dtantsuralso, I'm going to rewrite the whole approach we take to testing (launching many instances, etc)12:50
dtantsuralso, we'll have to make the devstack plugin branchless too, which might prevent us from removing deprecated things12:51
dtantsur(or maybe not, I'm already confused)12:51
sambettsdtantsur: tempest shouldn't (ideally) be aware of ironic's internals IMO only API level stuff which shouldn't change that often12:52
milansambetts, ++12:52
dtantsurwelllll... do you count things like "do we support whole disk images with pxe_* drivers" as internal?12:52
sambettsdtantsur: I think that shouldn't be in tempest, that should be in devstack12:52
*** dnikishov has quit IRC12:52
sambettsdtantsur: tempest shouldn't be adding the nodes into ironic IMO12:52
sambettsdtantsur: now tests like the ssh into the instances and check the partitions, thats a bit more complex12:53
dtantsurright, yes12:53
sambettsdtantsur: but that could be flagged on quite easily12:53
dtantsurand for inspector: do we want to update the tests with more data as we add it?12:53
dtantsur* more introspected data12:53
milansambetts, I'd say if my test case depends on that functionality and there's an API for it, ironic tempest client should "export" that and my test case should be able to reuse it so I don't have to reinvent the wheel each time ironic bumps minor API version12:54
sambettsmilan: ++12:54
sambettsdtantsur: should that be in tempest though or functional tests?12:55
milantempest should be something like oslo IMO12:55
dtantsursambetts, functional tests work on fake data12:55
dtantsurthe same problems as with unit tests: we can't validate our assumptions about the actual data12:55
dtantsur(well, we don't really do it now for advances things..)12:55
*** bdeschenes has joined #openstack-ironic12:56
*** penick has joined #openstack-ironic12:57
*** vsaienko has quit IRC12:57
dtantsuranother thing: I'm not sure how anything "branchless" benefits packagers. as often as I see branchless stuff downstream, it's always a pain12:57
* dtantsur shakes fist at DIB12:57
sambettsI think that from a tempest perspective, the test shouldn't be calling to inspector to make sure that the inspection has collected the right information, rather should be doing a nova boot with a flavor and making sure that it boots12:58
dtantsurso he we are: compromising test scope to make it fit into the proposed model :)12:58
*** strigazi1AFK is now known as strigazi12:58
*** ccamacho|lunch is now known as ccamacho12:58
*** bfournie has joined #openstack-ironic12:58
* mgould votes strenuously against making anything branchless, ever12:58
dtantsurthen to check capabilities discovery, we have to update nova flavors with the capabilities we discover12:59
dtantsurwhich requires knowing whether we discover them in a particular release/branch12:59
mgouldtests, build scripts, etc, all need to evolve with the codebase they affect12:59
sambettsdtantsur: which is in the devstack plugin12:59
*** gcb has quit IRC12:59
mgouldso if that's branched, so should they be12:59
* jroll almost feels like we should be using tempest directly against ironic for all of our features things, and then only using the nova scenarios with one driver or whatever to test our various nova features12:59
mgouldjroll: that increases specificity of the tests while reducing end-to-endness13:00
mgouldwhich is usually a good trade-off13:00
mgouldso +1 to doing most tests like that13:00
*** bdeschenes has quit IRC13:00
dtantsurjroll, what do you mean by "nova scenarios" here?13:00
mgouldwith possibly a couple of more end-to-endy ones13:01
sambettstempest scenarios I assume13:01
mgouldand on that cheery note, /me -> dentist...13:01
jrollmgould: yeah, we'd still use the nova scenarios for end-to-end (because nova's behavior shouldn't change depending on driver/uefi/ipxe/etc)13:01
jrolldtantsur: the nova tests in tempest that do a nova boot13:01
mgouldjroll: +113:01
sambettso/ mgould13:01
mgouldnothing too serious, fortunately :-)13:02
dtantsurjroll, ah, I think I get it. not sure if it changes something within the branched/branchless and single/multiple repo discussion13:02
jrolldtantsur: probably not :)13:02
*** rloo has joined #openstack-ironic13:03
dtantsurjroll, and what is your take on it?13:03
*** jaosorior is now known as jaosorior_mtg13:04
jrolldtantsur: I do see the benefits but disagree that it's a useful thing to do13:04
jrolldtantsur: basically it makes QA's life easier at the expense of the dev teams13:04
sambettsI find the point of tempest testing is to both test the project end to end and to help enforce that the project public API doesn't break between versions, if the tests change with the versions then you lose the second part13:04
jrollwhich is why we run the master version of the plugin in CI :)13:05
*** krtaylor has joined #openstack-ironic13:05
*** vsaienko has joined #openstack-ironic13:05
dtantsursambetts, I'd love to see a job running Release-1 tempest against master13:05
jrolldtantsur: tempest isn't branched...13:06
dtantsurjroll, s/tempest/Ironic tempest plugin/13:06
dtantsurjust as well as an IPA job running ironic Release-1 against IPA master13:06
*** jcoufal has joined #openstack-ironic13:06
sambettsdtantsur: its master tests against oldest supported ironic release is what should work13:06
sambettsdtantsur: which is what we test in the CI already13:07
jrolldtantsur: well, tempest isn't meant to be used branched, including the plugin13:07
dtantsurjroll, which is something I never agreed with13:07
dtantsurand I'm not sure why they're referring to packagers, as we had problems with branchless nature of tempest already13:07
jrolldtantsur: well, that's QA's stance since I've been involved in openstack13:07
dtantsuryeah, I don't expect it to be changed13:08
jrollthis is the logic to always use the master plugin, btw https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic.yaml#L200-L20813:08
dtantsuroh, interesting. I wonder if we should do the same for inspector.13:09
jrollprobably :)13:10
dtantsurso, we essentially have a branchless tempest plugin with a benefit of co-testing plugin changes with code, right?13:11
*** Marx314 has joined #openstack-ironic13:11
jrolldtantsur: yeah, the one place where this is a problem is:13:14
jrollinstalling some stable version of ironic on a box, installing master tempest on the same box, both not in virtualenvs13:14
jrollit will use the stable plugin, not master, and installing the master plugin would cause master ironic to be installed13:14
dtantsurjroll, why install master tempest? also, it won't work anyway, thanks to upper-constraints13:14
dtantsur* global-requirements actually13:15
sambettstempest doesn't have branches13:15
jrolldtantsur: because that's what openstack tells a person they should do if they want to run tempest against their cloud13:15
dtantsurthat's what we've hit. we're simulating tempest branches by using the latest tag of the release13:15
dtantsursambetts, this does not change my point13:15
jrolldtantsur: elaborate on latest tag?13:15
sambettstempest does releases off master, so they do have tagged versions13:16
dtantsurjroll, tempest has "releases" or whatever. we package the latest that was released in the given time frame13:16
jrollah13:16
vdrokgood morning jroll13:16
dtantsurjroll, dmellado can tell you more, but he's probably on the same meeting as I am13:16
jrolldtantsur: right, so qa team would probably tell packagers not to do that :)13:16
jrollbut I'm not sure13:16
jrollanyway, I'm not supposed to be here, but I mostly agree with your take on that goal13:17
dtantsurjroll, then we'll tell qa to be compatible with global-requirements from all supported releases :)13:17
*** srobert has joined #openstack-ironic13:17
jrollheh13:17
vdrokjroll: if you're still around, a question on core updates, is there a plan to add more people? :)13:17
dmelladodtantsur: I'm a meeting, maybe not the same as you but what would you need to know? xD13:17
*** Goneri has joined #openstack-ironic13:17
jrollvdrok: of course, all in good time :)13:18
dtantsurdmellado, re how we package tempest - you said from the latest tag in the given release time frame?13:18
*** vsaienko has quit IRC13:18
dmelladodtantsur: yep, basically tempest doesn't have branches but tags13:18
dmelladoso *on red hat* we do package the latest tag that was released alongside the openstack release13:19
dmelladoi.e. 13.0.0 as of now13:19
vdrokjroll: ok, was looking at the stats and wanted to ask :)13:19
*** vsaienko has joined #openstack-ironic13:20
*** jrist has quit IRC13:22
*** jrist has joined #openstack-ironic13:22
jroll:)13:22
* jroll actually goes now, see you all in a week :D13:23
dtantsurjroll, o/13:23
sambettscya in spain jroll o/13:23
rloohi and morning jroll, dmellado, dtantsur, vdrok, sambetts and everyone else13:23
sambettshey rloo13:23
dtantsurmorning rloo13:23
rloobye jroll :)13:23
vdrokgood morning rloo13:23
vdrokjroll: \o/13:23
*** jrist has quit IRC13:24
*** jcoufal has quit IRC13:26
sambettsdtantsur: do you know where the code that handles hiding passwords etc in the logs is from?13:26
*** baoli_ has joined #openstack-ironic13:27
milanmorning rloo, bye jroll! :)13:28
dtantsursambetts, not sure, I suspect in oslo.log13:28
rloosambetts, dtantsur: i would have thought oslo.utils13:29
rloosambetts: dtantsur: yes, i think oslo.utils, strutils.py13:30
rloohi milan13:30
sambettsrloo, dtantsur: oh awsome thanks!13:30
*** jrist has joined #openstack-ironic13:30
rlooyuriyz: are you interested in handling the TODO? I can do it but wanted to give you first dibs on it :) https://review.openstack.org/#/c/387652/1/ironic/drivers/modules/agent_base_vendor.py13:31
patchbotpatch 387652 - ironic - Remove 'agent_last_heartbeat' from node.driver_int... (MERGED)13:31
openstackgerritMerged openstack/ironic-python-agent: Fix config drive writing script  https://review.openstack.org/38595713:32
*** deray has quit IRC13:32
yuriyzhi ironic rloo I only left the comment13:33
*** penick has quit IRC13:33
yuriyznow I'm reworking notifications13:33
*** jtomasek is now known as jtomasek|afk13:33
rlooyuriyz: I know. I was going to fix it but thought I'd check with you to see if you wanted to. I'll do it then. thx for the comment!13:33
rlooyuriyz: it seems like either update the comment, or just fix it :)13:34
yuriyzthanks rloo13:34
*** vikrant has quit IRC13:35
*** mjturek has joined #openstack-ironic13:36
rooklucasagomes question about adding the ping to make sure when a instance goes from BUILD->ACTIVE. I am looking at adding some code for this, however, to get the IP address of the provisioning network, I would need to add the nova client... Not sure if that would be a problem? I don't see how else I could get the provisoning network ip13:36
*** jcoufal has joined #openstack-ironic13:36
sambettsrook: where are you adding this code?13:37
*** vsaienko has quit IRC13:38
lucasagomesrook, hi there! Isn't possible to get it from Neutron ? (Usually these design ideas should go to a RFE/Spec for others to throw in suggestions/alternatives)13:38
rooksambetts: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent_base_vendor.py#L589 right here :)13:39
rooklucasagomes: oh, I have a RFE :)13:39
rooki thought13:39
lucasagomessambetts, rook wants to make provisioning more reliable, currently there's no verification whatsoever if the active instance is actually booted or not so people usually babysit their deployments13:39
rookhttps://bugs.launchpad.net/ironic/+bug/16332913:40
openstackLaunchpad bug 163329 in glibc (Ubuntu) "libc6 realloc alters memory block" [Undecided,Invalid]13:40
rookugh13:40
rookhttps://bugs.launchpad.net/ironic/+bug/163329913:40
openstackLaunchpad bug 1633299 in tripleo "[RFE] Overcloud deploy resiliency " [Undecided,New]13:40
sambettsrook, lucasagomes: Ironic pinging an active won't work in the multitenant scenario because the machine once deployed is switched to the tenant network which Ironic can't access before being booted back up13:40
rooksambetts so the provisoning network is totally removed?13:41
mat128from the Ironic POV, once the image is deployed and a "reboot" has been issued, the node is provisioned13:41
sambettsrook: yes13:41
rooksambetts couldn't that happen _after_ the reboot?13:42
*** vmud213 has quit IRC13:42
rookie cloudinit?13:42
sambettsno, it occurs at the switch level13:42
rooksambetts at the ToR?13:42
sambettsyes13:42
rookjesus13:42
* rook mind blown13:42
sambettsthat was a major feature we merged in Newton13:43
rooksure13:43
rookhow can ironic ensure the desired state then?13:43
rookAnd I am not stating the right answer is, Ironic doesn't care.13:45
rookI just want to know where should we add logic to make deployments through OOO better.13:45
*** jaosorior_mtg is now known as jaosorior13:46
dtantsurrook, OOO can ignore the multitenant scenario *for now*13:46
rookdtantsur sure - however does that mean this code should reside in OOO somewhere vs Ironic.13:47
sambettsso once a node has left the provisioning network, ironic's only access to the node in question is through out of band connections, e.g. IPMI etc, I'm not sure if there is a way to check if the nodev13:47
rookthe "for now" scares me.13:47
sambettshas booted successfully13:47
lucasagomessambetts, right yeah, the multitenant scenario is something we need to account for13:47
rooksambetts ipmi is a joke13:47
dtantsur.. and a pretty bad one13:47
*** mtanino has joined #openstack-ironic13:48
sambettsyes13:48
rooksambetts and each vendor is different. I much prefer to not rely on ipmi13:48
rookDells lovely single queue and supermicro, oh supermicro13:48
rookAnyway, I just want to help make things better, not confuse shit.13:48
rookIf I am barking up the wrong tree, I can be redirected.13:48
sambettsit would have to be a new driver interface function, e.g. is_node_booted(self)13:49
sambettsbut I still don't know if thats even possible13:49
*** hamzy has quit IRC13:50
rooksambetts or could we reorder how things are done? Once the node is rebooted, you guys make switch level changes, etc... Instead, we reboot, ensure the instance is in the desired state (booted_into_os), then have a task to switch the networking>?13:50
rooknetworking is plug-n-play, we could just have something unprovision the provisioning networking from the instnace.13:50
sambettsthe point of the multitenancy changes is to prevent the teants image ever being booted in the provisioning network13:51
rookinstance*13:51
sambettsbecause its a security risk13:51
rookNot saying to leave in the provisioning network, but to have the instance boot into it, then have some cloud init process kill the connection13:51
rookifdown(provisioning net)13:51
rookthen kill it on the switch.13:52
dtantsurmaybe we can have cloud init somehow signal back to the metadata server?13:52
sambettse.g. if we deploy tenantA and tenantBs nodes at the same time then if even for a second they are in the same network then there is a securty risk13:52
dtantsuror we don't support metadata server in this case?13:52
rookThe 30 seconds it is on the provisioning network to do this will add so much resiliency when deploying large clouds (in the OOO case)13:52
rooksambetts if security is really a concern, enable ACLs13:53
rookmac based ACLs.13:53
rookAgain, I am not trying to add pain here... if this is the wrong spot, I get it.13:53
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Add node provision state change notification  https://review.openstack.org/34843713:54
*** ralonsoh_ has joined #openstack-ironic13:54
*** ralonsoh has quit IRC13:55
rookdtantsur yeah, that is what I was suggesting.13:55
openstackgerritRuby Loo proposed openstack/ironic: Remove ..agent.build_instance_info_for_deploy() in Pike  https://review.openstack.org/38766113:55
*** cdearborn has joined #openstack-ironic13:56
sambettswhat does nova offer in terms of being able to check if your image hasn't derped at boot time?13:57
sambettsfor VMs13:57
rooknot a damn thing13:57
rookcould I write something - sure.13:57
rookthe other idea dtantsur lucasagomes sambetts is to use the nova console-log.13:57
sambettsyeah, I was just thinking about that too13:58
TheJuliathen the question would be is it hypervisor agnostic13:58
rookadd some sort of completed hash a user-file, and check the console-log for the hash.13:58
rookif no hash, not complete.13:58
rloodtantsur: wrt the summit sessions, are you leading the RAID/advanced partitioning session?13:58
rookTheJulia right - I am not sure if this would be hypervisor agnostic... To be sure they have some sense of a console-log13:59
sambettsand we've just added the Socat console driver for IPMI, but then it would depend on whether the Ironic driver supported it or not13:59
dtantsurrloo, I do, provided that mat128 won't be there14:00
rloomat128: you're not going to be at the summit, right? :-(14:00
mat128dtantsur: I confirm, I won't be at the summit14:00
mat128:(14:00
rloomat128: who do you think will/should lead the vnc console session? jroll?14:00
mat128im updating both RAID and VNC console specs once im done with that production issue14:01
lucasagomesrook, sambetts yeah currently there's few drivers with console support trough nova (only 2 I think)14:01
* lucasagomes thinks about another alternative solution14:01
rloomat128: thx. i'm going to put down jroll's name for that session.14:01
mat128I think Jim can lead it and/or possibly delegate, I remember from the initial spec submission that a few members were interested14:02
mat128I remember JayF being interested too14:02
rloomat128: ok, will check with Jay. I have jroll's 'permission' to put him down for everything :)14:02
rooklucasagomes anything else would be utilize the network :/14:02
jlvillalGood morning Ironic :)14:02
rloomorning jlvillal14:02
dtantsurmorning jlvillal14:02
jlvillalHi rloo dtantsur :)14:03
rloojlvillal: were you going to lead the qa & gate improvements session? https://etherpad.openstack.org/p/ironic-ocata-summit-qa14:03
jlvillalrloo: For sure the QA portion. Not sure if someone else wants to do the gate section?14:03
rloovdrok, sambetts: do you know who will be leading the 'synchronizing events with neutron' session? https://etherpad.openstack.org/p/ironic-ocata-summit-neutron-events14:04
lucasagomesjlvillal, rloo morning14:04
rloojlvillal: ok, will put you down for QA. Who knows/wants to lead the gate part?14:04
sambettsrloo: vsaienk0 I think14:04
rloohi lucasagomes!14:04
rloovsaienk0: you OK with leading the 'synchronizing events with neutron' session? https://etherpad.openstack.org/p/ironic-ocata-summit-neutron-events14:05
jlvillalrloo: I am more than willing if nobody else wants to.14:05
vdrokRloo yup, either vsaienk0 or me, more likely vasyl14:05
-openstackstatus- NOTICE: We are away of pycparser failures in the gate and working to address the issue.14:05
rloovdrok: for a second, I thought, what, there are 3 of you? :)14:05
vdrokMorning jlvillal and mat12814:05
sambettsrook: it is possible to have the user image ping Ironic once up, but thats then requiring something to run in the user image to do that, and that the tenants network has a connection to the OS control plane via a router or something14:06
jlvillalvdrok: Privyet!14:06
vdrokrloo: :)14:06
jlvillalvsaienk0: We got multi-node to run locally yesterday :)14:06
sambetts\o/14:06
jlvillalvsaienk0: Did some bash scripts and ansible to prep the nodes.14:07
rloolucasagomes, dtantsur: do either of you want to lead the Gate Improvements part of the session? Or I can put jroll down. https://etherpad.openstack.org/p/ironic-ocata-summit-qa14:07
jlvillalvsaienk0: And then used reproduce.sh14:07
vsaienk0jlvillal: I prefer to run it on the gates, local network and network at the gates are different14:07
jlvillalvsaienk0: Ah okay. Different how?14:08
vsaienk0rloo: yes I do not mind driving 'synchronizing events with neutron' session', but It will be first time :)14:08
jlvillalvsaienk0: I wanted to be able to do that so that for adding grenade we can SSH into the box and look around.14:08
rloovsaienk0: there's always a first time, but the good thing is that it only happens once :)14:09
vsaienk0jlvillal: mtu and other stuff14:09
rooksambetts: right, so again, assumptions on the networking14:09
jlvillalvsaienk0: Oh yeah. that. Thanks for reminding me14:09
NobodyCamGood Morning Ironicers14:09
jlvillalHi NobodyCam !14:09
lucasagomesrloo, sure no problem14:10
NobodyCammorning jlvillal :)14:10
lucasagomesNobodyCam, morning14:10
dtantsurrloo, I can lead it if lucasagomes and jlvillal don't want to14:10
* lucasagomes have no problem leading it either14:10
dtantsurrloo, just put us both :)14:11
lucasagomesunless someone wants, I don't really mind14:11
lucasagomeshah14:11
*** hj-hpe has joined #openstack-ironic14:11
rloolucasagomes, dtantsur: i was going to say i would put down lucasagomes since dtantsur is leading the raid one. but i'll put you both down :)14:11
mgouldmorning NobodyCam jlvillal rloo14:12
lucasagomesrloo, ++14:12
NobodyCammorning mgould14:12
lucasagomesthanks14:12
jlvillalHey mgould :)14:12
rloomorning NobodyCam, mgould14:12
* lucasagomes goes grab some lunch14:12
NobodyCammorning rloo14:12
*** lucasagomes is now known as lucas-hungry14:12
* rloo waits to see whether the leads for the sessions changes or not :)14:13
rloothe gate is broken, i'm guessing due to the pycparser failures ^^. Should we update our gate status?14:14
jlvillalrloo: Yeah it went kind of wonky with things like: http://mirror.dfw.rax.openstack.org/pypi/simple/1-844-291-6706-dell-printer-technical-support-phone-number/14:15
jlvillaland: https://pypi.python.org/pypi/1-844-291-6706_DELL_Printer_Technical_Support_Phone_Number/84414:15
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: UUID, started_at, finished_at in the status API  https://review.openstack.org/38670314:15
jlvillalSome strange looking packages: https://pypi.python.org/pypi14:15
rloojlvillal: doesn't that mean that we should phone? :)14:16
*** jaosorior has quit IRC14:16
*** mjura has quit IRC14:16
jlvillalOh wow. Looks like it might be a PyPi thing.14:16
jlvillalrloo: heh14:16
mgouldoh dear14:16
jlvillalOr just some wonky package names.14:17
mgouldsomeone ran an upload-to-pypi script on their address book, I guess...14:17
jlvillalheh14:17
mgouldgoodness knows how14:17
jlvillalrloo: Could update the whiteboard. Global gate failure issue.14:18
rloojlvillal: i updated the whiteboard. someone remember to clear it when it is fixed14:18
bfourniemorning ironic14:18
mgouldmorning bfournie14:18
bfourniesambetts: when you have a second, I just have a quick question on lldp review - https://review.openstack.org/#/c/374381/20/specs/lldp-reporting.rst@334.  Trying to nail down “introspection list —vlan” command. Thx.14:18
patchbotpatch 374381 - ironic-inspector-specs - Add LLDP processing hook and new CLI commands14:18
jlvillalGood morning bfournie14:18
sambettsbfournie: sure, I'll take a look14:18
bfourniemorning jlvillal14:19
bfourniesambetts: thanks14:19
mgouldjlvillal: the guilty party appears to be mikey807, whoever that is14:19
*** hshiina has quit IRC14:20
jlvillalmgould: Ah14:20
jlvillalmgould: Well this one is Kevin24by7: https://pypi.python.org/pypi/1-8OO-681-7208_Bellsouth_Mail_Customer_Support_Phone_Number_440/44014:20
jlvillalSeems like a few different account :(14:21
mgouldhuh14:21
mgouldattempted DoS, perhaps14:21
openstackgerritMerged openstack/ironic: Skip create_ovs_taps() for multitenancy case  https://review.openstack.org/38236014:23
mat128jlvillal, mgould: IMO it's more of a SEO thing / attempt14:23
sambettsrook: right, out of band communications is our only static form of communication, I'm wondering if there is a way for the node to indicate to Ironic via that that its booted14:23
*** hamzy has joined #openstack-ironic14:23
mat128even if it doesnt make much sense14:23
sambettsrook: but again vendor specifc and user image specific14:24
mat128rook: there used to be an old product line where we did that14:24
mat128code inside "images" (we were installing via netboot but whatever) would call our infra to signal it was done14:24
mat128back then we did a lot via post-install, like cpanel, etc.14:25
*** wajdi has joined #openstack-ironic14:25
*** mjura has joined #openstack-ironic14:28
*** hoonetorg has quit IRC14:28
*** amotoki has quit IRC14:30
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Help a user to enable console redirection  https://review.openstack.org/38457814:31
openstackgerritKyrylo Romanenko proposed openstack/ironic: Test ML2 Neutron Events feature  https://review.openstack.org/37001614:31
*** vsaienko has joined #openstack-ironic14:32
*** vishwanathj has joined #openstack-ironic14:37
*** ppiela has joined #openstack-ironic14:39
*** michauds has joined #openstack-ironic14:40
*** rajinir has joined #openstack-ironic14:45
*** vsaienko has quit IRC14:48
*** rbrud has joined #openstack-ironic14:49
openstackgerritMario Villaplana proposed openstack/ironic: Make agent driver always PXE before cleaning  https://review.openstack.org/26384214:51
mariojvvsaienk0: i updated the above to make the deprecation warning appear only once at service startup, let me know what you think ^14:52
*** cppforlife_ has quit IRC14:55
*** kun_huang has quit IRC14:56
*** sborkows has quit IRC14:56
*** vishwanathj has quit IRC14:57
*** amcguign has joined #openstack-ironic14:58
*** amcguign has left #openstack-ironic14:59
*** kun_huang has joined #openstack-ironic14:59
*** cppforlife_ has joined #openstack-ironic14:59
*** hamzy has quit IRC15:00
rooksambetts: wrt tenant a and tenant b, does ironic not do these in order? meaning until tenant a is done, tenant b won't be served15:00
*** hamzy has joined #openstack-ironic15:02
*** mnasiadka has quit IRC15:03
*** mjura has quit IRC15:03
*** kun_huang has quit IRC15:04
sambettsrook: no, all deployments are handled in parallel, just like with a VM15:05
JayFmat128: rloo: If you put me down for leading any summit sessions you'll end up disappointed, lol15:05
mat128oh wait15:05
mat128thats true15:05
mat128lol15:05
rooksambetts: hm ok15:05
mat128rloo: yeah, I think Jim can handle that or delegate it as he see fit15:05
rlooJayF: oh yeah, I forgot!15:06
JayFon the bright side, you all have long flights ahead of you15:06
JayFand I get a week in my own bed15:06
JayFlol15:06
*** rbudden has joined #openstack-ironic15:07
*** cppforlife_ has quit IRC15:08
JayFSo I asked a question yesterday that didn't get answered: are we still accepting RFEs for deprecated-for-removal-because-no-ci drivers? Mainly the SNMP driver/15:09
JayFjroll: if you're around ^15:09
openstackgerritGalyna Zholtkevych proposed openstack/ironic: IPMI command should depend on console type  https://review.openstack.org/36364715:10
*** penick has joined #openstack-ironic15:11
*** cppforlife_ has joined #openstack-ironic15:11
*** anush has joined #openstack-ironic15:11
*** vishwanathj has joined #openstack-ironic15:12
*** lucas-hungry is now known as lucasagomes15:12
mat128JayF: I think he mentioned leaving15:13
milanguys, http://logs.openstack.org/03/386703/3/check/gate-grenade-dsvm-ironic-inspector/b5162b3/logs/grenade.sh.txt.gz#_2016-10-18_14_27_49_96115:13
JayFmat128: that's why I said if he's around, he's on vaca until summit15:13
milanhttps://github.com/eliben/pycparser/issues/151 seems affecting the inspector grenade tests15:13
mat128JayF: We have VirtualPDU coming soon and I would understand if we wanted the CI tests in place before introducing more code to the snmp driver15:13
milandtantsur, ^^15:13
JayFmilan: that's crazy15:13
milan:'(15:13
JayFmat128: IMO, if we're going to likely keep driver around, I'd just accept the RFE15:14
*** kun_huang has joined #openstack-ironic15:14
JayFmat128: just one of the things as a specs core I'm supposed to do is make sure I know this stuff before approving rfes :P15:14
openstackgerritMerged openstack/ironic-lib: Use 'code-block' for pieces of code  https://review.openstack.org/37864315:14
dtantsurmilan, I'm expecting it to affect the whole openstack15:14
mat128JayF: https://github.com/internap/virtualpdu15:14
mat128JayF: missing the integration in Ironics devstack plugin15:14
JayFnice15:14
JayFyeah, I saw that submission was from internap15:15
mat128I can push my WIP for it, but a colleague of mine is workong on it15:15
JayFupstreaming code \o/15:15
mat128yes15:15
mat128:D15:15
mat128xhku: ^15:15
sambettsmilan, dtantsur: it is, there was a message from infra not so long ago :)15:15
mat128milan, dtantsur: "[10:05:10] -openstackstatus- NOTICE: We are away of pycparser failures in the gate and working to address the issue."15:15
*** anush has quit IRC15:16
milanlol15:16
milandtantsur, I still sort of blame my Mac for that :D15:16
mat128JayF: g2g, back in ~2h15:17
JayFo/15:17
dtantsurmilan, I'm pretty sure it's somehow involved15:17
dtantsurmay I also blame UEFI slowness on it?15:18
milanlol15:18
milan:D15:18
milango ahead :D15:18
dtantsurApple was involved in UEFI, so I feel pretty ok doing that15:18
milanI feel like I'll just grab some snack till my Mac figures out how to address these issues on my behalf ;)15:19
milanmaybe I shall ask Siri15:19
dtantsurlucasagomes, do you remember that switch option (fast-something) to enable on a switch to stop iPXE timeouts?15:22
lucasagomesdtantsur, portfast15:23
dtantsurlucasagomes, aha. did it really help with http timeouts or am I confusing something?15:23
lucasagomesdtantsur, it did yeah, actually enabling it solved that problem we were investigating15:23
JayFoh man, in the real world, not having portfast on switches w/ironic+network switching15:24
JayFcauses a world of hurt15:24
JayFbecause half the time the port won't come up in time for the pxe rom to catch15:24
dtantsurJayF, so, it makes sense to put it to documentation as a general advice?15:25
JayFI would +2 such a patch15:25
lucasagomesJayF, yeah that was the case, without it enabled it worked sometimes15:25
dtantsurJayF, could you maybe write it? I'm afraid I won't come up with a good explanation15:26
JayFhttp://logs.openstack.org/57/377957/10/gate/gate-tempest-dsvm-ironic-multitenant-network/0649016/logs/devstacklog.txt.gz#_2016-10-18_15_18_55_54515:26
*** baoli_ has quit IRC15:26
JayFthe same thing that breaks inspector grenade15:26
JayFis breaking real-ironic-gate15:26
JayFdoes someone have the bug # infra is using to track? I'll put it in the whiteboard15:27
lucasagomesJayF, jaysus, I think we recently saw something like that15:27
* lucasagomes thought t has been fixed15:27
dtantsurlucasagomes, it was fixed, now it's broken again15:27
JayFdtantsur: to be blunt, I've never logged into a switch and flipped the switch, so idk if I'm any more good at it than you would be :P15:27
JayFlol, flipped the switch15:27
JayFdon't actually flip switches, it's probably not good for the internals15:27
lucasagomesJayF, https://bugs.launchpad.net/cinder/+bug/162972615:28
openstackLaunchpad bug 1629726 in networking-cisco "recompiled pycparser 2.14 breaks Cinder db sync and Nova UTs" [Critical,Confirmed]15:28
dtantsurlol15:28
sambettsI wonder if that is something we should be including in the neutron intergration stuff15:28
dtantsurJayF, me neither, I just remembered that lucasagomes said that. lucasagomes mind providing a documentation update?15:28
* dtantsur tries to find someone who can explain *why* we do it15:28
lucasagomesdtantsur, ack, I can put a patch soon-ish to the install-guide ?!15:29
JayFdtantsur: I mean, the "real world" answer for why is without it, it can take 30-60s for a switchport to come up after a connect/disconnect (Which would be caused in some machines by a power off/on, and in some switches by a switch reconfiguration)15:29
JayFdtantsur: IDK the technical reaosns behind it15:29
JayFdtantsur: why'd you mark that bug fix released for ironic? it's not fixed for us? or just b/c we have no action?15:29
*** syed_ has joined #openstack-ironic15:29
lucasagomesJayF, that's why I said I've seem it before15:29
dtantsurJayF, https://bugs.launchpad.net/cinder/+bug/1629726? because it  was fixed once15:29
openstackLaunchpad bug 1629726 in networking-cisco "recompiled pycparser 2.14 breaks Cinder db sync and Nova UTs" [Critical,Confirmed]15:29
JayFah15:29
dtantsurand now it appeared again15:29
JayFdtantsur: should I mark it confirmed agani?15:30
lucasagomesapparently it was fixed but somehow is broken again now15:30
dtantsurI guess so15:30
JayFdone15:31
sambettsso weird the global reqirements repo hasn't had a patch to change the upper contraints15:31
openstackgerritBob Fournier proposed openstack/ironic-inspector-specs: Add LLDP processing hook and new CLI commands  https://review.openstack.org/37438115:33
*** krtaylor has quit IRC15:34
rlooJayF: wrt rfe's for deprecated drivers. No, we should not accept those. they are deprecated. i think at most, it means we only fix bugs.15:38
*** vsaienko has joined #openstack-ironic15:38
rlooJayF: my take on it anyway.15:38
JayFrloo: if there was any gray area; this is it: it's like a 20 line patch to add support for another PDU to the snmp driver. That's part of why I'm struggling with just denying it :/15:38
rlooJayF: 20 lines is 20 lines more. there is overhead in doing even a one-line change.15:39
rlooJayF: exceptions are always possible though, but you'll have to talk to the PTL about that.15:39
JayFhttps://bugs.launchpad.net/ironic/+bug/1634303 + https://review.openstack.org/#/c/387718 are the pieces in question15:40
openstackLaunchpad bug 1634303 in Ironic "[RFE] Baytech MRP27 PDU support in SNMP driver" [Wishlist,In progress] - Assigned to StephaneR (srobert)15:40
patchbotpatch 387718 - ironic - Baytech MRP27 PDU support15:40
JayFit looks like something internap (mat128 and friends) are using downstream and trying to upstream15:40
JayFso I have a little more empathy than usual :)15:40
rloosambetts: for attach/detach API and vlan-aware instances, are there rfes (or whatever) for nova side? (I'm updating ocata priorities etherpad: https://etherpad.openstack.org/p/ironic-ocata-summit-priorities)15:40
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Catch error when wrong temp_url were generated  https://review.openstack.org/37087515:40
rlooJayF: I have empathy too but rules are rules. Gotta ask for an exception... :)15:41
rlooJayF: https://governance.openstack.org/reference/tags/assert_follows-standard-deprecation.html15:42
rlooJayF: look at Requiremenst, #1.15:42
JayFyeo15:42
JayFyep15:42
sambettsrloo: Not yet, I still need to propose them15:42
JayFI'm commentingon the bug now with a no15:42
JayFwe don't have an rfe-rejected tag though15:43
JayFlol15:43
rloosambetts: ok, will indicate that. thx.15:43
openstackgerritKyrylo Romanenko proposed openstack/ironic: Test ML2 Neutron Events feature  https://review.openstack.org/37001615:44
JayFvdrok: A++ comment on the ipa versioning spec about cleaning mismatch w/ IPA API version number15:46
vdrok:)15:46
rlooTheJulia: wrt boot from volume, just want to double check, this is the nova-related one: https://review.openstack.org/#/c/311696/ ?15:46
patchbotpatch 311696 - nova-specs - Add support booting baremetals from Cinder volume15:46
rloovsaienk0, vdrok: for the portgroups work, what are the related bug/rfe/spec?15:48
rloovsaienk0, vdrok: and nova changes for portgroups?15:48
vsaienk0rloo: https://bugs.launchpad.net/ironic/+bug/161875415:49
openstackLaunchpad bug 1618754 in Ironic "[RFE] Add portgroups support to Ironic" [Wishlist,In progress] - Assigned to Vasyl Saienko (vsaienko)15:49
*** ChubYann has joined #openstack-ironic15:49
vsaienk0rloo: nova side https://review.openstack.org/#/c/206163/15:49
patchbotpatch 206163 - nova - Ironic: use portgroups15:49
rloothx vsaienk015:50
srobertjayF rloo: I understand and agree with the workflow. We will push the CI work and poke you when this is ready.15:51
JayFsrobert: especially if you all are pushing to get ci and un-deprecate the driver, you should seriously hit the ML and ask for an exception. I suspect you have a better-than-average chance of getting it.15:51
rlooJayF: why can't we move the driver to the other project so folks can continue to work on it?15:52
srobertyeah! we are working hard to get a solid solution to un-deprecate. Thanks for your time.15:52
*** ohamada has quit IRC15:52
rloovsaienk0: the nova blueprint still needs to be approved! (for portgroups)15:53
vsaienk0rloo: blueprint and spec15:54
JayFrloo: I think we can't remove any of the drivers that are deprecated for at least a cycle without breaking deprecation promises15:55
*** e0ne_ has quit IRC15:55
JayFrloo: I guess it could be copied? IDK.15:55
*** e0ne has joined #openstack-ironic15:55
rloovsaienk0: OH... :-(15:55
rlooJayF: yes, I think if we copy it, and put a note in ironic code, that any changes should be done in the other project, blah blah, that would work.15:56
rlooJayF: maybe send out email about this, and that can be the process we take for any/all deprecated drivers?15:56
*** anush has joined #openstack-ironic15:56
rlooJayF: if 'we' all agree?15:56
sambettsvdrok: did vsaienk0 say you were working on the cloudinity parts of the portgroups stuff?15:56
JayFI'm just wondering if this is more of an isolated problem15:57
rlooJayF: i did wonder when we deprecated them, but figured jroll and others knew what it meant. jroll may still know what it means :)15:57
JayFi.e. How likely is it that other drivers will need/want changes15:57
vdroksambetts: yes, I do15:57
JayFwe already know SNMP is a special case b/c we have contributors actively working on getting it officially supported before it's removed15:57
rlooJayF: dunno, but hey, I love to generalize things :)15:57
vdrokcurrently trying to make it work on ubuntu15:57
sambettsvdrok: are you also working on the nova side changes required for passing in the network configuration information?15:57
vdrokyup15:57
rlooJayF: OH, I didn't know that I don't think. Hmm...15:57
JayFrloo: yeah, that's what mat128 and srobert were saying; they've got a virtualpdu project along the same lines as virtualbmc to allow us to test it, they just haven't completed the devstack integration15:58
*** penick has quit IRC15:59
sambettsvdrok: I wondered if you might want this: https://review.openstack.org/#/c/289412/1/nova/virt/ironic/driver.py@656 as reference, it is part of one of my POC patches a while ago where I had it working for trunks and portgroups15:59
patchbotpatch 289412 - nova - [WIP] Allow mapping multiple Neutron ports to Iron... (ABANDONED)15:59
jlvillalsambetts: bring it back to life! :)16:00
vdroksambetts: I did see this :) yeah, I used some things from there, thanks!16:00
sambettsvdrok: aweome :D16:00
openstackgerritGalyna Zholtkevych proposed openstack/ironic: IPMI command should depend on console type  https://review.openstack.org/36364716:00
sambettsjlvillal: just restored it :)16:00
jlvillal:)16:01
*** skazi has quit IRC16:04
rlooJayF: I think a discussion on ML would be useful then.16:04
*** sambetts has quit IRC16:05
openstackgerritStephane Robert proposed openstack/ironic: Baytech MRP27 PDU support  https://review.openstack.org/38771816:05
*** rama_y has quit IRC16:08
openstackgerritVasyl Saienko proposed openstack/ironic: Cleanup hung iscsi session  https://review.openstack.org/38809216:08
-openstackstatus- NOTICE: pycparser 2.16 released to fix assertion error from today.16:11
rookhttps://bugs.launchpad.net/ironic/+bug/1633299 lucasagomes dtantsur mat128 tried to capture the lively discussion here.16:12
openstackLaunchpad bug 1633299 in Ironic "[RFE] Overcloud deploy resiliency " [Undecided,New]16:12
lucasagomesrook, thanks, will take a look16:12
*** penick has joined #openstack-ironic16:12
*** clenimar has joined #openstack-ironic16:13
*** sambetts_ has joined #openstack-ironic16:15
*** sambetts_ is now known as sambetts16:16
sambettswell I just had a power outage... that was fun...16:16
* sambetts reminds himself to get a UPS16:16
*** srobert has quit IRC16:17
*** ElCoyote_ has quit IRC16:20
*** jtomasek|afk is now known as jtomasek16:22
rloo^^ did anyone do a recheck to verify that the gate is working for us?16:22
*** krtaylor has joined #openstack-ironic16:22
*** hamzy has quit IRC16:23
*** ElCoyote_ has joined #openstack-ironic16:24
rookoh sambetts i updated the rfe bug16:24
rooksambetts: if you wanted to add more details : https://bugs.launchpad.net/ironic/+bug/163329916:24
openstackLaunchpad bug 1633299 in Ironic "[RFE] Overcloud deploy resiliency " [Undecided,New]16:24
* rloo rechecks a patch to see if gate is working.16:25
*** Guest85855 has quit IRC16:25
vsaienk0jroll, sambetts: https://bugs.launchpad.net/ironic/+bug/1634573 bug about build instances in provisioning network, looking for your thoughts, thanks16:25
openstackLaunchpad bug 1634573 in Ironic "Allow to spawn instances in provisioning network" [Undecided,New] - Assigned to Vasyl Saienko (vsaienko)16:25
*** Romanenko_K has quit IRC16:26
sambettsrook: ok cool, hopefully that'll give some of the other developers a chance to chime in, it might be worth finding our conversation from earlier here http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/ and linking to it so that people can read through what we've already discussed16:26
sambettsvsaienk0: I'll add it to my TODO :)16:28
* xavierr says, what a day what a lovely day16:30
xavierrhey Ironic :)16:30
*** e0ne_ has joined #openstack-ironic16:32
*** e0ne has quit IRC16:32
*** baoli has joined #openstack-ironic16:34
*** manjeets has quit IRC16:35
*** hamzy has joined #openstack-ironic16:36
mgouldhi xavierr16:39
*** rpioso has joined #openstack-ironic16:39
*** jpich has quit IRC16:44
sambettso/ xavierr16:44
*** mtanino has quit IRC16:47
*** skazi has joined #openstack-ironic16:47
*** rcernin has quit IRC16:51
dtantsurfolks, does anybody know/remember why we have this line: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ipxe_config.template#L316:52
dtantsurit seems no-op, as iPXE always DHCPs, and actually breaks me, as it tries DHCPing on another NIC the second time around16:52
openstackgerritStephanie Miller proposed openstack/bifrost: Test timeout extension for inspection  https://review.openstack.org/37505016:54
JayFdtantsur: IDK; but I do know we run ipxe with a similar "dhcp" at the top in our static downstream config for onmetal16:55
sambettsdtantsur: lucasagomes added that line way back in kilo, lucasagomes any ideas?16:55
dtantsurinteresting...16:55
*** srobert has joined #openstack-ironic16:56
* lucasagomes reads16:56
*** srobert has quit IRC16:56
lucasagomessambetts, so, dhcp with no parameters will try to configure all nics afact16:56
lucasagomesbut the nic that was configured as part of the chainload should already have an IP address16:57
lucasagomesso, we probably could omit that dhcp assuming we always want to boot from that nic16:57
*** rbrud has quit IRC16:58
dtantsurlucasagomes, " will try to configure all nics" only the first one for me (not even the PXE one)16:58
sambettsdtantsur: http://ipxe.org/cmd16:59
sambettsdtantsur: it seems to have been superceeded by ifconf16:59
sambettshttp://ipxe.org/cmd/dhcp16:59
sambettsbut ifconf with no params picks the first available nic17:00
lucasagomesdtantsur, oh you are right, yeah it says the first17:00
*** rama_y has joined #openstack-ironic17:01
*** ralonsoh_ has quit IRC17:01
dtantsurok, I'll think about it tomorrow, see yo17:01
*** vsaienko has quit IRC17:01
*** dtantsur is now known as dtantsur|afk17:01
*** derekh has quit IRC17:01
lucasagomessambetts, dtantsur maybe we should instruct people to configure the nic1 to be the provisioning nic17:01
lucasagomesor if not, we should document how to modify the ipxe script/template17:02
lucasagomesto adapt to their nic order17:02
dtantsur|afkmaybe we should just drop this line.. ok, I'll investigate17:02
sambettslucasagomes: it might be different per machine though if not all your servers are the same17:02
xavierrironic'ers, I've a doubt. what's the difference between web console and serial interface. web console via horizon, right? what about serial console?17:02
lucasagomessambetts, yeah, if so we need that information in the ports17:03
lucasagomesso we could set it correctly for each node booting17:03
sambettsxavierr: serial console is accessible via a command line17:03
lucasagomesthe ipxe script will be tricky do17:03
lucasagomeswe could also loop through the nics and try to configure it all17:03
sambettslucasagomes: we have that info in the DB now, ports have pxe_enabled flag on that17:03
sambettsthem*17:03
lucasagomessambetts, right, but I mean the nic order17:03
sambettsyeah, we'd need to be able to pick by mac address in iPXE17:04
lucasagomesyeah17:04
lucasagomeswe could loop it through the nics, find the one with the mac address registered in ironic and configure it17:04
lucasagomes(boot.ipxe will be trickier tho)17:05
sambettsyeah :/17:05
xavierrsambetts: does it opens a new terminal window? I'm curious about that, I've never used... haha17:05
* xavierr will try to use some driver that has console interface17:06
sambettsxavierr: no, its like using telnet or ssh17:06
sambettstbh more like using telnet than ssh17:06
xavierrsambetts: sam, so when I do 'ironic node-get-console' it goes directly to the machine console, ok?17:08
xavierrok :)17:08
*** anush has quit IRC17:08
sambettsxavierr: it should yes17:08
xavierrty17:08
*** e0ne_ has quit IRC17:09
*** anup has joined #openstack-ironic17:10
*** anup is now known as Guest9692117:10
sambettsxavierr: either that or it'll give you a protocol URL to use like: ws://127.0.0.1:6083/?token=5f7854b7-bf3a-41eb-857a-43fc33f0b1ec17:13
sambettsxavierr: this guide might help http://docs.openstack.org/developer/nova/testing/serial-console.html17:13
*** vsaienko has joined #openstack-ironic17:14
*** trown is now known as trown|lunch17:15
* sambetts -> home 17:19
sambettsnight all17:23
*** sambetts is now known as sambetts|afk17:23
jlvillalsambetts|afk: Good night17:23
jlvillallucasagomes: Are you still in Brazil?17:24
lucasagomesjlvillal, I am yes17:26
lucasagomeswill go back to dublin after the summit17:26
rloohey lucasagomes, JayF, others: i just changed a bug to be an RFE: https://bugs.launchpad.net/ironic/+bug/1439901. I'm fine with it being approved w/o a spec.17:29
openstackLaunchpad bug 1439901 in Ironic "[RFE] node name isn't supported in the REST API for creating a port" [Wishlist,Triaged]17:29
rloolucasagomes, JayF, dtantsur|afk, jlvillal: what do you think?17:29
lucasagomesrloo, hmm the work seems simple, but I'm a bit concern that names can change17:30
lucasagomeswhere uuid is the canonical identifier of that node17:30
rloolucasagomes: is it different from node names/uuids?17:31
JayFlucasagomes: I mean, that's true for anywhere we let them use name in place of uuid? Seems like we can't protect operators from that error...17:31
JayFrloo: At a minimum, it's a simple RFE we should approve ... I sorta agree with the original filer that it seems like a bug to me17:31
JayFrloo: I thought anywere you could use a node uuid you could use a node name :/17:31
*** penick has quit IRC17:32
*** mgould is now known as mgould|afk17:32
rlooJayF: yes, that's mostly true. anywhere you could use a node uuid, you should be able to use a node name. but i haven't verified and it is possible we missed one :)17:32
rlooJayF: bugs and rfe's .. similar. addressing something that is missing. in this case, is it a bug? seems like a feature.17:32
lucasagomesrloo, I think we don't allow changing the resources UUID. I'm just thinking the case someone changes the node name suddenly we either have ports belong to other nodes or belonging to no nodes at all17:32
lucasagomesunless we also update the name in the ports after the node name update17:33
rloolucasagomes: under the hood, the port is associated with a node id/uuid, not the node name.17:33
*** lucasagomes has quit IRC17:33
*** jistr has quit IRC17:33
*** sergek has quit IRC17:33
*** pas-ha has quit IRC17:33
*** igordcard has quit IRC17:33
*** izaakk has quit IRC17:33
*** fyxim has quit IRC17:33
*** ionutbalutoiu has quit IRC17:33
*** Guest66666 has quit IRC17:33
*** EmilienM has quit IRC17:33
*** SpamapS has quit IRC17:33
*** Madasi has quit IRC17:33
*** zer0c00l has quit IRC17:33
*** kencjohnston has quit IRC17:33
*** boris-42 has quit IRC17:33
*** patrickeast has quit IRC17:33
*** mgkwill has quit IRC17:33
*** betherly has quit IRC17:33
*** greghaynes has quit IRC17:33
*** dims has quit IRC17:33
*** Ng has quit IRC17:33
*** TheJulia has quit IRC17:33
*** jroll has quit IRC17:33
*** johnthetubaguy has quit IRC17:33
*** tonyb has quit IRC17:33
*** Guest66666 has joined #openstack-ironic17:33
*** zer0c00l has joined #openstack-ironic17:33
*** dims has joined #openstack-ironic17:33
*** tonyb has joined #openstack-ironic17:33
*** Madasi has joined #openstack-ironic17:33
*** kencjohnston has joined #openstack-ironic17:33
*** igordcard has joined #openstack-ironic17:33
*** greghaynes has joined #openstack-ironic17:33
*** johnthetubaguy_ has joined #openstack-ironic17:33
*** SpamapS has joined #openstack-ironic17:34
JayFrloo: I could go either way on RFE vs bug, the difference is minimal, I'd just like to approve the RFE :D17:34
*** johnthetubaguy_ is now known as johnthetubaguy17:34
*** adas2 has joined #openstack-ironic17:34
*** lucasagomes has joined #openstack-ironic17:34
*** EmilienM has joined #openstack-ironic17:34
*** jistr has joined #openstack-ironic17:34
rlooJayF: that's what i'm looking for. I just want to make sure lucasagomes is on board.17:34
*** jroll has joined #openstack-ironic17:34
*** ionutbalutoiu has joined #openstack-ironic17:35
*** michauds has quit IRC17:35
*** EmilienM has quit IRC17:35
*** EmilienM has joined #openstack-ironic17:35
* lucasagomes got disconnected17:35
rloolucasagomes: the API allows specifying a node name. we're not changing the db or port object, it should still be associated with node uuid.17:35
lucasagomesrloo, oh right17:35
lucasagomesok so I'm good with it17:35
rlooJayF: btw, i was the one that reported that bug, eons ago, before we had rfe's :)17:35
lucasagomesI thought we would register the name in that field, if it gets converted I'm fine with it17:35
JayFrloo: well your rfe is approved. wooooo.17:35
rloolucasagomes: thx. I'll clarify in the bug.17:36
rloothx JayF and lucasagomes. (See, we can make decisions w/o our ptl :))17:36
JayFI never look at who filed a bug17:36
lucasagomeso/17:36
*** adas2 has quit IRC17:37
*** athomas has quit IRC17:38
*** SpamapS has quit IRC17:39
*** SpamapS has joined #openstack-ironic17:39
*** vsaienko has quit IRC17:39
xavierrsambetts|afk: ty :)17:39
*** patrickeast has joined #openstack-ironic17:39
*** boris-42 has joined #openstack-ironic17:40
*** srobert has joined #openstack-ironic17:41
*** pas-ha has joined #openstack-ironic17:41
*** TheJulia has joined #openstack-ironic17:41
*** mgkwill has joined #openstack-ironic17:42
*** sergek_ has joined #openstack-ironic17:42
*** Guest96921 has quit IRC17:43
*** betherly has joined #openstack-ironic17:43
*** izaakk has joined #openstack-ironic17:45
*** Ng has joined #openstack-ironic17:45
*** fyxim has joined #openstack-ironic17:45
*** rbrud has joined #openstack-ironic17:45
*** dprince has quit IRC17:52
*** anup has joined #openstack-ironic17:52
*** anup has quit IRC17:52
mat128back17:54
*** aNuposic has left #openstack-ironic17:54
vdroksambetts|afk: wooo, got the configdrive working with cloud-init on xenial17:54
vdrokwith the portgroup17:54
vdrokwill update vsaienk0's spec tomorrow17:55
*** penick has joined #openstack-ironic17:55
*** aNuposic has joined #openstack-ironic17:56
openstackgerritFélix Bouliane proposed openstack/ironic: WIP- Add virtualpdu to ironic devstack plugin  https://review.openstack.org/38815417:56
xhkumat128 ^^ I will try to complete for next week17:59
mat128xhku: make sure you Workflow -117:59
xhkuoops I CR-1 instead of W-117:59
xhkuadressed thanks17:59
*** athomas has joined #openstack-ironic18:00
openstackgerritMario Villaplana proposed openstack/ironic-lib: Add more tests for work_on_disk  https://review.openstack.org/38815618:00
mat128xhku: looks fairly complete, is there anything major missing?18:02
JayFOne thing I'd do18:05
JayFis add a Do-not-merge patch on top of that one18:05
JayFforcing IRONIC_DEPLOY_DRIVER to something_snmp18:05
rloodevananda: i'm assuming no API meeting; I'm reading the API specs now18:05
JayFso you can see if it works in the gate18:05
JayFmat128: xhku ^18:06
mat128JayF: good idea :)18:06
mat128can't say I havent done that in the past to force some parameters in18:06
JayFI mean, it's pretty much the only way to validate it's going to work in the gate18:06
JayFoutside of adding another job18:06
mat128JayF: yup18:06
xhkuwill do18:06
*** dprince has joined #openstack-ironic18:07
xhkuJayF: Thanks !18:07
JayFxhku: thanks for upstreaming code and working to save your driver that's on death row18:07
JayFlol18:07
lucasagomesdtantsur|afk, JayF re: https://review.openstack.org/#/c/387559/ there are plenty of places which we use "MAC" in ironic18:07
patchbotpatch 387559 - ironic - Account for Infiniband GID when validating an address18:07
mat128haha18:08
lucasagomesI wonder if we should just leave it as "mac" but change the is_valid_mac to actually account both, inifiniband GID and MAC addresses18:08
xhkulol18:08
lucasagomesit's a bit misleading, but renaming it all seems a lot of work18:08
lucasagomes(or we could do in parts)18:08
vdrokgood night everyone!18:09
lucasagomesvdrok, night18:09
JayFo/18:11
openstackgerritMario Villaplana proposed openstack/ironic: Mock ironic-lib properly in test_deploy_utils  https://review.openstack.org/38625518:11
*** jcoufal_ has joined #openstack-ironic18:20
*** anush has joined #openstack-ironic18:20
*** jcoufal has quit IRC18:21
*** rama_y has quit IRC18:22
*** rama_y has joined #openstack-ironic18:23
*** trown|lunch is now known as trown18:23
openstackgerritFélix Bouliane proposed openstack/ironic: WIP- Add virtualpdu to ironic devstack plugin  https://review.openstack.org/38815418:26
openstackgerritFélix Bouliane proposed openstack/ironic: DONOTMERGE-change IRONIC_DEPLOY_DRIVER to pxe_snmp  https://review.openstack.org/38816818:26
*** mtanino has joined #openstack-ironic18:29
*** milan has quit IRC18:32
ricardoasTheJulia charlledaniel: oneview ci has been acting weird lately... i'm currently working on a new deployment. It should be up tomorrow18:35
*** e0ne has joined #openstack-ironic18:36
TheJuliaricardoas: This may or may not relate https://bugs.launchpad.net/cinder/+bug/162972618:37
openstackLaunchpad bug 1629726 in networking-cisco "recompiled pycparser 2.14 breaks Cinder db sync and Nova UTs" [Critical,Confirmed]18:37
*** e0ne has quit IRC18:39
ricardoasTheJulia: I'll take a look, thanks!18:39
*** anush has quit IRC18:39
*** aswadr_ has quit IRC18:39
*** e0ne has joined #openstack-ironic18:39
*** e0ne has quit IRC18:41
xavierrvdrok: night! o/18:46
*** e0ne has joined #openstack-ironic18:51
*** shobhar_ has joined #openstack-ironic18:55
*** jcoufal_ has quit IRC18:55
*** jcoufal has joined #openstack-ironic18:57
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: UUID, started_at, finished_at in the status API  https://review.openstack.org/38670319:01
*** jcoufal has quit IRC19:12
*** jcoufal has joined #openstack-ironic19:13
*** wajdi_ has joined #openstack-ironic19:17
*** baoli has quit IRC19:18
*** wajdi__ has joined #openstack-ironic19:19
*** pester has quit IRC19:19
*** pester has joined #openstack-ironic19:19
*** e0ne has quit IRC19:20
*** e0ne has joined #openstack-ironic19:21
*** wajdi has quit IRC19:21
*** wajdi_ has quit IRC19:22
*** shobhar_ has quit IRC19:28
mariojvaNuposic: hey, do you want to change yourself to be the assignee of this bug? i know we were discussing it and a solution yesterday https://bugs.launchpad.net/ironic/+bug/143990119:29
openstackLaunchpad bug 1439901 in Ironic "[RFE] node name isn't supported in the REST API for creating a port" [Wishlist,Triaged]19:29
lucasagomesI'm calling it a day, have a great evening all!19:30
mat128o/19:31
*** lucasagomes is now known as lucas-afk19:32
openstackgerritMerged openstack/ironic: Add entry_point for oslo policy scripts  https://review.openstack.org/38096219:33
JayFmariojv: nice, I was going to suggest that as a good one to pick up19:33
*** baoli has joined #openstack-ironic19:33
*** shobhar_ has joined #openstack-ironic19:34
*** Goneri has quit IRC19:38
*** baoli has quit IRC19:38
*** vsaienko has joined #openstack-ironic19:39
*** baoli has joined #openstack-ironic19:41
*** krtaylor has quit IRC19:41
*** mmedvede has quit IRC19:44
*** baoli_ has joined #openstack-ironic19:45
*** baoli has quit IRC19:45
rlooaNuposic:  i went ahead and assigned 1439901 to you19:45
*** baoli has joined #openstack-ironic19:46
*** baoli_ has quit IRC19:46
jlvillalLooks like the gate is working again :)  Based on the merge message above.19:46
mariojv\o/19:46
*** baoli_ has joined #openstack-ironic19:47
*** jcoufal has quit IRC19:47
*** jcoufal has joined #openstack-ironic19:48
*** snecklifter has joined #openstack-ironic19:49
*** baoli has quit IRC19:51
openstackgerritMerged openstack/ironic: Adding Timing metrics for DRAC drivers.  https://review.openstack.org/37795719:52
*** kun_huang has quit IRC19:52
*** kun_huang has joined #openstack-ironic19:55
*** ricardoas has left #openstack-ironic20:00
*** portdirect has quit IRC20:00
*** baoli_ has quit IRC20:00
*** baoli has joined #openstack-ironic20:01
*** e0ne has quit IRC20:01
*** cdearborn has quit IRC20:02
*** cdearborn has joined #openstack-ironic20:05
*** portdirect has joined #openstack-ironic20:05
*** mmedvede has joined #openstack-ironic20:07
*** romcheg_ has quit IRC20:07
*** romcheg_ has joined #openstack-ironic20:09
*** dougs has quit IRC20:09
aNuposicrloo: sure. I was going to do that20:10
*** dougs has joined #openstack-ironic20:10
rlooaNuposic: :)20:10
*** dougs has quit IRC20:10
*** dougs1 has joined #openstack-ironic20:11
*** Sukhdev has joined #openstack-ironic20:11
mrdaMorning all20:14
jlvillalHi mrda :)20:15
mrdaJusr a quick question, agent_ipmitool still requires swift tempurls for user image download, right? I know there was talk about adding a new way to get images, but I can't remember if it got done...20:15
*** Sukhdev has quit IRC20:15
JayFmrda: well, we also support ceph tempurls20:17
JayFmrda: and if you're using standalone ironic, any http url in image_url will do the trick20:18
mrdathanks JayF20:18
mrdaJayF: but you can't use http urls in agent_ipmitool though, right?20:18
JayFyes, absolutely20:18
JayFI don't think you can use it /with nova/ though20:18
mrdaok, cool20:19
JayFmrda: I believe bifrost uses that, so that would be a template to go by20:20
mrdathanks20:20
JayFhttps://github.com/openstack/bifrost/blob/master/playbooks/roles/bifrost-deploy-nodes-dynamic/tasks/main.yml#L7620:21
JayFmrda: ^^20:21
mrdaJayF: Thanks20:21
*** Goneri has joined #openstack-ironic20:27
*** ricardoas has joined #openstack-ironic20:30
*** hafe has joined #openstack-ironic20:31
*** jheroux has joined #openstack-ironic20:32
openstackgerritboden proposed openstack/ironic: Replace retrying with tenacity  https://review.openstack.org/37657420:39
*** raghusab_ has joined #openstack-ironic20:43
*** trown is now known as trown|outtypewww20:46
*** hafe has quit IRC20:49
*** jheroux has quit IRC20:51
*** raghusab__ has joined #openstack-ironic21:01
*** raghusab__ has quit IRC21:01
openstackgerritMichael Turek proposed openstack/ironic-specs: Add spec for OpenBMC power module and driver.  https://review.openstack.org/38821821:11
openstackgerritMichael Turek proposed openstack/ironic-specs: Add spec for OpenBMC power module and driver.  https://review.openstack.org/38821821:12
*** skazi has quit IRC21:13
*** syed_ has quit IRC21:15
*** kun_huang has quit IRC21:15
*** kun_huang has joined #openstack-ironic21:16
*** syed_ has joined #openstack-ironic21:16
*** jmccrory has quit IRC21:17
*** kencjohnston has quit IRC21:17
*** clif_h has quit IRC21:18
*** snecklifter has quit IRC21:18
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: UUID, started_at, finished_at in the status API  https://review.openstack.org/38670321:18
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Add API for listing all introspection statuses  https://review.openstack.org/34492121:18
*** kencjohnston has joined #openstack-ironic21:19
*** clif_h has joined #openstack-ironic21:19
*** jmccrory has joined #openstack-ironic21:20
*** nicodemos has quit IRC21:25
*** wajdi__ has quit IRC21:25
*** nicodemos has joined #openstack-ironic21:25
*** Goneri has quit IRC21:25
*** dprince has quit IRC21:28
*** jcoufal_ has joined #openstack-ironic21:32
*** jcoufal has quit IRC21:33
*** jcoufal has joined #openstack-ironic21:37
*** jcoufal_ has quit IRC21:38
*** ccamacho is now known as ccamacho|out21:39
*** baoli has quit IRC21:40
*** hoonetorg has joined #openstack-ironic21:40
*** baoli has joined #openstack-ironic21:40
*** thrash is now known as thrash|g0ne21:41
openstackgerritMario Villaplana proposed openstack/ironic: Mock ironic-lib properly in test_deploy_utils  https://review.openstack.org/38625521:44
*** srobert has quit IRC21:45
*** portdirect has quit IRC21:48
*** portdirect has joined #openstack-ironic21:51
*** rbudden has quit IRC21:59
*** jcoufal has quit IRC22:00
*** cdearborn has quit IRC22:03
*** hj-hpe has quit IRC22:03
*** penick has quit IRC22:25
*** rbrud has quit IRC22:30
*** bfournie has quit IRC22:33
*** baoli has quit IRC22:34
NobodyCammorning mrda :)22:37
mrdahey NobodyCam22:42
NobodyCam:)22:42
zackfAnyone know what my ironic.conf [glance] section should look like? I keep getting a generic "can't connect to glance " error.22:53
*** pmannidi has joined #openstack-ironic22:54
*** awiddersheim has joined #openstack-ironic23:01
*** bfournie has joined #openstack-ironic23:05
*** rajinir has quit IRC23:16
*** vsaienko has quit IRC23:35
*** vishwanathj has quit IRC23:53
openstackgerritJay Faulkner proposed openstack/ironic-specs: Add spec for specific faults support  https://review.openstack.org/33411323:54
cineramazackf, if you haven't looked yet you can find some information in etc/ironic/ironic.conf.sample in the source tree, otherwise you might want to pastebin your configuration and people might be able to have a look23:59
zackfYeah, i looked in there, i have a self signed cert on all my api endpoints, wondering if that's my issue.23:59

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