| *** ijw has joined #openstack-ironic | 00:04 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/377446 | 00:04 |
|---|---|---|
| openstackgerrit | Bruno Cornec proposed openstack/ironic-specs: Propose Redfish support into Ironic https://review.openstack.org/184653 | 00:09 |
| *** awiddersheim has quit IRC | 00:19 | |
| *** awiddersheim has joined #openstack-ironic | 00:22 | |
| *** sdake_ has quit IRC | 00:25 | |
| *** absubram has joined #openstack-ironic | 00:29 | |
| *** Goneri has joined #openstack-ironic | 00:34 | |
| openstackgerrit | Marc Aubry proposed openstack/python-ironicclient: Make error work with client in python3 https://review.openstack.org/378050 | 00:35 |
| *** ijw_ has joined #openstack-ironic | 00:36 | |
| *** hoangcx has joined #openstack-ironic | 00:38 | |
| *** ijw has quit IRC | 00:39 | |
| *** absubram has quit IRC | 00:44 | |
| *** aNuposic has quit IRC | 00:48 | |
| *** rloo has quit IRC | 00:51 | |
| *** absubram has joined #openstack-ironic | 00:51 | |
| *** Goneri has quit IRC | 00:52 | |
| *** absubram_ has joined #openstack-ironic | 00:52 | |
| *** sdake has joined #openstack-ironic | 00:52 | |
| *** absubram has quit IRC | 00:56 | |
| *** absubram_ is now known as absubram | 00:56 | |
| *** xuao has joined #openstack-ironic | 01:08 | |
| *** absubram has quit IRC | 01:11 | |
| openstackgerrit | Michael Still proposed openstack/ironic: Add required packages that aren't in a default Ubuntu install. https://review.openstack.org/379053 | 01:12 |
| *** syed_ has quit IRC | 01:14 | |
| *** bfournie has joined #openstack-ironic | 01:15 | |
| *** phuongnh has joined #openstack-ironic | 01:15 | |
| *** mtanino has quit IRC | 01:16 | |
| *** ijw_ has quit IRC | 01:20 | |
| openstackgerrit | Tang Chen proposed openstack/python-ironicclient: Make console commands support multiple node operation https://review.openstack.org/378477 | 01:38 |
| *** ijw has joined #openstack-ironic | 01:40 | |
| *** tuanla has joined #openstack-ironic | 01:52 | |
| *** tuanla has quit IRC | 01:53 | |
| *** tuanluong has joined #openstack-ironic | 01:54 | |
| *** praneshp has quit IRC | 01:57 | |
| *** ijw has quit IRC | 02:01 | |
| tonyb | JayF, jroll: all patches +W'd | 02:19 |
| openstackgerrit | Merged openstack/ironic: [install-guide] Import "Bare Metal service overview" https://review.openstack.org/377973 | 02:20 |
| openstackgerrit | Merged openstack/ironic: [install-guide] Import Install and configure sections https://review.openstack.org/377974 | 02:23 |
| openstackgerrit | Merged openstack/ironic: [install-guide] Import "integration with other OpenStack components" https://review.openstack.org/377975 | 02:24 |
| openstackgerrit | Merged openstack/ironic: [install-guide] Import "Image requirements" https://review.openstack.org/377976 | 02:24 |
| openstackgerrit | Merged openstack/ironic: [install-guide] Import "Flavor creation" https://review.openstack.org/377977 | 02:24 |
| openstackgerrit | Merged openstack/ironic: [install-guide] Import "Local boot with partition images" https://review.openstack.org/377978 | 02:24 |
| openstackgerrit | Merged openstack/ironic: [install-guide] Import "Enrollment" and "Troubleshooting" sections https://review.openstack.org/377979 | 02:24 |
| openstackgerrit | Merged openstack/ironic: [install-guide] Import HTTPS, standalone and root device hints https://review.openstack.org/377980 | 02:24 |
| openstackgerrit | Merged openstack/ironic: [install-guide] Import configdrive https://review.openstack.org/377981 | 02:24 |
| openstackgerrit | Merged openstack/ironic: [install-guide] Import "Appending kernel parameters to boot instances" https://review.openstack.org/377982 | 02:24 |
| openstackgerrit | Merged openstack/ironic: [install-guide] Import "Building or downloading a deploy ramdisk image" https://review.openstack.org/377983 | 02:25 |
| openstackgerrit | Merged openstack/ironic: [install-guide] Import "Trusted boot with partition image" https://review.openstack.org/377984 | 02:26 |
| openstackgerrit | Merged openstack/ironic: [install-guide] Import "Setup the drivers for the Bare Metal service" https://review.openstack.org/377985 | 02:26 |
| openstackgerrit | Merged openstack/ironic: Add link to new guide in old install guide https://review.openstack.org/377986 | 02:26 |
| openstackgerrit | Merged openstack/ironic: Update developer's guide "Installation Guide" link https://review.openstack.org/377987 | 02:26 |
| openstackgerrit | Merged openstack/ironic: Fix absolute links to install-guide.rst in developer docs https://review.openstack.org/377988 | 02:26 |
| openstackgerrit | Merged openstack/ironic: Point documentation links to newton equivalents https://review.openstack.org/378037 | 02:26 |
| tonyb | Oh look it's a merge fest :) | 02:27 |
| *** thrash is now known as thrash|g0ne | 02:38 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/ironic: Trivial: Fix some typos in comments and docstring https://review.openstack.org/379081 | 02:42 |
| *** gcb has joined #openstack-ironic | 02:43 | |
| *** yuanying_ has quit IRC | 02:50 | |
| *** sdake has quit IRC | 02:52 | |
| *** david-lyle has quit IRC | 03:04 | |
| *** bdeschenes has quit IRC | 03:05 | |
| *** yuanying has joined #openstack-ironic | 03:05 | |
| *** wajdi has joined #openstack-ironic | 03:08 | |
| *** ijw has joined #openstack-ironic | 03:15 | |
| *** sdake has joined #openstack-ironic | 03:16 | |
| *** sdake has quit IRC | 03:19 | |
| *** ijw has quit IRC | 03:20 | |
| *** ijw has joined #openstack-ironic | 03:22 | |
| *** appu103_ has joined #openstack-ironic | 03:27 | |
| *** appu103_ is now known as aparna | 03:28 | |
| *** sdake has joined #openstack-ironic | 03:31 | |
| *** links has joined #openstack-ironic | 03:33 | |
| *** yuanying has quit IRC | 03:33 | |
| *** yuanying has joined #openstack-ironic | 03:34 | |
| openstackgerrit | Nam Nguyen Hoai proposed openstack/bifrost: TrivialFix: Fix typo in the RST file https://review.openstack.org/379105 | 03:36 |
| *** vikrant has joined #openstack-ironic | 03:41 | |
| *** aswadr_ has joined #openstack-ironic | 03:43 | |
| *** anish has quit IRC | 03:46 | |
| *** hoonetorg has quit IRC | 03:47 | |
| *** pcaruana has quit IRC | 03:47 | |
| *** hoangcx has quit IRC | 03:47 | |
| *** xek__ has quit IRC | 03:47 | |
| *** dtantsur has quit IRC | 03:47 | |
| *** d0ugal has quit IRC | 03:48 | |
| *** hoangcx has joined #openstack-ironic | 03:48 | |
| *** yuanying has quit IRC | 03:49 | |
| *** keekz has quit IRC | 03:51 | |
| *** anish has joined #openstack-ironic | 03:51 | |
| *** alineb has quit IRC | 03:51 | |
| xuao | Hello ironic, it seems that the jenkins have crashed for several days. :''-( | 03:51 |
| openstackgerrit | Nam Nguyen Hoai proposed openstack/ironic-inspector: TrivialFix: Fix typo in the configuration file https://review.openstack.org/379108 | 03:52 |
| *** yuanying has joined #openstack-ironic | 03:52 | |
| *** hoonetorg has joined #openstack-ironic | 03:53 | |
| *** dtantsur has joined #openstack-ironic | 03:54 | |
| *** keekz has joined #openstack-ironic | 03:55 | |
| *** d0ugal has joined #openstack-ironic | 03:57 | |
| *** pcaruana has joined #openstack-ironic | 03:59 | |
| *** lifeless has quit IRC | 04:02 | |
| *** aparna has quit IRC | 04:03 | |
| *** aparna has joined #openstack-ironic | 04:04 | |
| *** lifeless has joined #openstack-ironic | 04:05 | |
| *** gcb has quit IRC | 04:06 | |
| openstackgerrit | Nam Nguyen Hoai proposed openstack/ironic-python-agent: TrivialFix: Fix typo https://review.openstack.org/379112 | 04:07 |
| *** gcb has joined #openstack-ironic | 04:08 | |
| *** alineb has joined #openstack-ironic | 04:12 | |
| *** vmud213 has joined #openstack-ironic | 04:14 | |
| *** ijw has quit IRC | 04:25 | |
| *** bathri-s has joined #openstack-ironic | 04:29 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/377443 | 04:37 |
| jlvillal | xuao, Yes: https://etherpad.openstack.org/p/IronicWhiteBoard | 04:47 |
| jlvillal | xuao, We aren't sure what is the problem :( | 04:47 |
| *** sabeen1 has quit IRC | 04:55 | |
| *** amotoki has joined #openstack-ironic | 04:57 | |
| *** sdake has quit IRC | 05:00 | |
| *** amotoki has quit IRC | 05:02 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/377443 | 05:06 |
| xuao | jlvillal, get√ | 05:08 |
| *** amotoki has joined #openstack-ironic | 05:17 | |
| *** vsaienko has joined #openstack-ironic | 05:24 | |
| *** appu103_ has joined #openstack-ironic | 05:28 | |
| *** vsaienko has quit IRC | 05:32 | |
| *** aparna has quit IRC | 05:32 | |
| *** e0ne has joined #openstack-ironic | 05:41 | |
| *** aparnav has joined #openstack-ironic | 05:51 | |
| *** appu103_ has quit IRC | 05:51 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/ironic: Use method delete_if_exists from oslo.utils https://review.openstack.org/379159 | 05:55 |
| openstackgerrit | Merged openstack/ironic: iLO drivers documentation update https://review.openstack.org/356989 | 05:57 |
| *** xek has joined #openstack-ironic | 05:59 | |
| *** mjura has joined #openstack-ironic | 06:08 | |
| *** mjura has quit IRC | 06:17 | |
| *** mjura has joined #openstack-ironic | 06:17 | |
| *** rcernin has joined #openstack-ironic | 06:22 | |
| *** vishwanathj has joined #openstack-ironic | 06:22 | |
| *** rbartal has joined #openstack-ironic | 06:31 | |
| *** aNuposic has joined #openstack-ironic | 06:40 | |
| *** hoangcx_ has joined #openstack-ironic | 06:56 | |
| *** jaosorior has joined #openstack-ironic | 06:58 | |
| *** hoangcx has quit IRC | 06:59 | |
| *** hoangcx has joined #openstack-ironic | 07:25 | |
| *** hoangcx_ has quit IRC | 07:26 | |
| pas-ha | morning Ironic | 07:28 |
| *** aNuposic has quit IRC | 07:31 | |
| tangchen_ | morning guys :) | 07:41 |
| *** hshiina has joined #openstack-ironic | 07:41 | |
| tuanluong | morning ironic | 07:42 |
| *** vsaienko has joined #openstack-ironic | 07:45 | |
| gcb | good afternoon ironic :-) | 07:45 |
| *** jpich has joined #openstack-ironic | 07:48 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing broken grenade job https://review.openstack.org/377653 | 07:55 |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:00 | |
| *** aswadr_ has quit IRC | 08:12 | |
| aarefiev | morning! | 08:18 |
| *** vsaienko has quit IRC | 08:18 | |
| *** bathri-s has quit IRC | 08:18 | |
| gcb | Is the grenade job fixed now? | 08:18 |
| *** vsaienko has joined #openstack-ironic | 08:18 | |
| *** bathri-s has joined #openstack-ironic | 08:19 | |
| tangchen_ | Hi aarefiev, morning :) | 08:19 |
| aarefiev | morning | 08:19 |
| betherly | jroll: https://review.openstack.org/#/c/378779/ | 08:25 |
| patchbot | patch 378779 - releases - Release ironic-ui 2.1.0 | 08:25 |
| *** strigazi_AFK is now known as strigazi | 08:29 | |
| *** vsaienko has quit IRC | 08:29 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing broken grenade job https://review.openstack.org/377653 | 08:29 |
| *** vsaienko has joined #openstack-ironic | 08:37 | |
| *** derekh has joined #openstack-ironic | 08:38 | |
| *** Romanenko_K has joined #openstack-ironic | 08:38 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session glance and swift https://review.openstack.org/352289 | 08:52 |
| *** vsaienko has quit IRC | 08:54 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic: Invalid URL and Typo in enrollment.rst https://review.openstack.org/379289 | 09:02 |
| *** jaosorior is now known as jaosorior_lunch | 09:05 | |
| *** dmellado has joined #openstack-ironic | 09:05 | |
| dmellado | vsaienk0: ping re https://review.openstack.org/#/c/379256 | 09:05 |
| patchbot | patch 379256 - ironic - Skip tempest tests if ironic isn't available | 09:05 |
| vsaienk0 | dmellado: I replied there | 09:06 |
| dmellado | I was thinking about for now remove the original options as you said | 09:06 |
| dmellado | and put up a follow-up patch with the new tempest-conf option | 09:06 |
| dmellado | would that be acceptable for you? | 09:06 |
| *** wendar has quit IRC | 09:10 | |
| vsaienk0 | dmellado: I prefer to do not add follow-up when current patch is not merged | 09:12 |
| dmellado | vsaienk0: ack, then I add the newwer option to the config.py within the ironic tempest plugin and push another patchset | 09:14 |
| vsaienk0 | dmellado: do not forget to set correct enabled_drivers in tempest: https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L1514 | 09:15 |
| dmellado | vsaienk0: I see, I was thinking about having it live on the ironic plugin, but I'll check | 09:16 |
| dmellado | thanks! | 09:16 |
| *** skazi has quit IRC | 09:17 | |
| *** sambetts|afk is now known as sambetts | 09:18 | |
| sambetts | Morning all | 09:18 |
| *** electrofelix has joined #openstack-ironic | 09:19 | |
| sergek | Morning all | 09:25 |
| *** hshiina has quit IRC | 09:30 | |
| *** skazi has joined #openstack-ironic | 09:30 | |
| *** sdake has joined #openstack-ironic | 09:47 | |
| mgould | morning aarefiev gcb tangchen_ pas-ha betherly dmellado vsaienk0 sambetts sergek and everyone else! | 09:49 |
| dmellado | morning mgould ;) | 09:50 |
| * mgould now has an espresso machine in the office; this may be dangerous | 09:50 | |
| sambetts | mgould: hahaha it certainly is | 09:50 |
| betherly | mgould: oh my days my caffeine intake has gone up so much since i got my coffee machine | 09:50 |
| sambetts | I use to be caffine free... then I got mine... | 09:51 |
| tangchen_ | mgould: Morning. :) | 09:52 |
| tangchen_ | mgould: https://review.openstack.org/#/c/379265/ A very simple trivial fix. Please help to check it. :) | 09:53 |
| patchbot | patch 379265 - ironic - Add missing error to log in do_node_deploy() | 09:53 |
| dmellado | actually I'm trying to lower out my caffeine intake xD | 09:59 |
| * dmellado lowered the rate of his nesspresso | 10:00 | |
| dmellado | xD | 10:00 |
| aarefiev | morning mgould sambetts | 10:00 |
| mgould | tangchen_: LGTM | 10:02 |
| *** amotoki has quit IRC | 10:06 | |
| sergek | mgould: :) | 10:09 |
| tangchen_ | mgould: Thanks a lot. :) | 10:13 |
| *** hoangcx has quit IRC | 10:13 | |
| mgould | tangchen_: np | 10:15 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session glance and swift https://review.openstack.org/352289 | 10:15 |
| *** e0ne has quit IRC | 10:16 | |
| *** milan has joined #openstack-ironic | 10:22 | |
| pas-ha | guys, is install-guide meant to be version-specific (a separate version is published for stable/... and master) or it is always published from master and for new options/behavior I have to mention from which Ironic version onwards they are available? | 10:23 |
| openstackgerrit | Tang Chen proposed openstack/ironic: Cleanup: Remove useless function data() https://review.openstack.org/379345 | 10:23 |
| *** thrash|g0ne is now known as thrash | 10:28 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing broken grenade job https://review.openstack.org/377653 | 10:33 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session glance and swift https://review.openstack.org/352289 | 10:33 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: [install-guide] describe pxe.ipxe_swift_tempurl https://review.openstack.org/379358 | 10:37 |
| *** phuongnh has quit IRC | 10:38 | |
| openstackgerrit | Tang Chen proposed openstack/ironic: Fail test if excepted error was not raised https://review.openstack.org/379360 | 10:41 |
| tangchen_ | jlvillal: Hi, John, would you please help to check these two simple fix patches when you are free ? https://review.openstack.org/#/c/379265/ and https://review.openstack.org/#/c/379360/ | 10:47 |
| patchbot | patch 379265 - ironic - Add missing error to log in do_node_deploy() | 10:47 |
| patchbot | patch 379360 - ironic - Fail test if excepted error was not raised | 10:47 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/ironic: Remove unused methods https://review.openstack.org/379364 | 10:50 |
| *** nicodemos has quit IRC | 10:51 | |
| *** xilet_ has quit IRC | 10:52 | |
| *** leduardo_ has quit IRC | 10:55 | |
| *** vnogin has joined #openstack-ironic | 10:56 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing broken grenade job https://review.openstack.org/377653 | 10:57 |
| *** bdeschenes has joined #openstack-ironic | 10:59 | |
| *** xilet has joined #openstack-ironic | 11:00 | |
| zigo | jroll: betherly: I've find a *VERY* easy way to address my issue: https://review.openstack.org/gitweb?p=openstack/deb-horizon.git;a=blob;f=debian/patches/stores-SECRET_KEY-in-tmp-folder-for-tests.patch;h=75775dc107659a29a0262f62085136a4bc72ab36;hb=7a5780bc83b631faa418d1605ead1aad13fb8cc1 | 11:01 |
| zigo | :) | 11:01 |
| zigo | I'll submit this to horizon upstream. | 11:01 |
| betherly | zigo: oh awesome!!! | 11:02 |
| betherly | thats a horizon patch ye? do you need to add anything to the ironic-ui before we release? | 11:02 |
| *** jaosorior_lunch is now known as jaosorior | 11:02 | |
| betherly | zigo: ^ | 11:03 |
| zigo | Yes, horizon patch. | 11:03 |
| openstackgerrit | Yushiro FURUKAWA proposed openstack/python-ironicclient: Add prefix "$" for command examples https://review.openstack.org/379371 | 11:03 |
| zigo | Which I'll be carying in the Debian package until that's approved upstream. | 11:03 |
| zigo | Nothing to add in ironic-ui. | 11:03 |
| zigo | I've restarted a build of ironic-ui, let's see how it goes. | 11:03 |
| betherly | zigo: fingers and toes crossed :p | 11:04 |
| mgould | hmmm, apparently we've only bought an espresso machine so we can use up our stash of knockoff Nespresso capsules, then we're planning to sell the machine on | 11:05 |
| mgould | I think we may skip the "use up knockoff capsules" step | 11:05 |
| *** awiddersheim has quit IRC | 11:06 | |
| *** dprince has joined #openstack-ironic | 11:06 | |
| sambetts | mgould: haha, thats random, why do you have nespresso caps without a coffee machine to use them? | 11:08 |
| mgould | from the previous nespresso machine which broke before I started | 11:10 |
| sambetts | ahhh :-P | 11:10 |
| mgould | so they're at least a year old, which probably isn't helping | 11:10 |
| sambetts | mgould: I get my caps from https://www.pactcoffee.com/ they are soo much better than the offical ones | 11:10 |
| * mgould bookmarks - thanks! | 11:11 | |
| *** hshiina has joined #openstack-ironic | 11:11 | |
| sambetts | mgould: they do really great ground coffee mailed to your door too, I just like the convience of the pods | 11:12 |
| mgould | ah, nice | 11:13 |
| mgould | fun fact: both the London Stock Exchange and Lloyd's of London started as coffee houses | 11:13 |
| sambetts | yeah, I love that story! My brother has a book that goes into all the history of the coffee in the UK as well as how it goes from plant to our cups its a really interesting read | 11:14 |
| *** mat128|afk is now known as mat128 | 11:14 | |
| *** vnogin has left #openstack-ironic | 11:15 | |
| mat128 | morning ironic! | 11:15 |
| sambetts | hey mat128! | 11:15 |
| mat128 | s/ironic/coffee drinkers/ | 11:15 |
| mat128 | =) | 11:15 |
| zigo | betherly: Is it normal that there's only a single test?!? | 11:16 |
| zigo | betherly: http://logs.openstack.org/94/378694/6/check/gate-deb-ironic-ui-dpkg-debian-jessie/83879e2/console.html#_2016-09-29_11_07_35_817238 | 11:16 |
| *** bdeschenes has quit IRC | 11:16 | |
| mgould | morning mat128 | 11:17 |
| mgould | sambetts: sounds very interesting! I don't suppose you can remember the title? | 11:17 |
| mat128 | zigo: did you see https://review.openstack.org/#/c/378744/ | 11:17 |
| patchbot | patch 378744 - ironic-ui - Mock openstack_dashboard secret key in unit tests | 11:17 |
| mgould | zigo: that certainly looks suboptimal | 11:17 |
| *** vsaienko has joined #openstack-ironic | 11:18 | |
| mgould | probably depressingly common, though... | 11:18 |
| *** ccamacho is now known as ccamacho|lunch | 11:18 | |
| zigo | mat128: mgould: Yup, it didn't work for me. Moreover, such a simple patch for Horizon seems a way better than a complicated mock hack. | 11:18 |
| mat128 | zigo: ok, weird. | 11:18 |
| *** openstackgerrit has quit IRC | 11:19 | |
| mat128 | zigo: yes, there's only one patch in the tree: https://github.com/openstack/ironic-ui/tree/master/ironic_ui/test/tests | 11:19 |
| mat128 | weird :( | 11:19 |
| *** openstackgerrit has joined #openstack-ironic | 11:19 | |
| sambetts | mgould: I believe it was this one https://www.amazon.co.uk/Curious-Baristas-Guide-Coffee/dp/1849755639 | 11:20 |
| sambetts | mgould: I'm like 90% sure anyway | 11:20 |
| mgould | sambetts: thanks! | 11:21 |
| sambetts | after all this coffee talk brb going to make one :-P | 11:22 |
| mgould | zigo: yeah, complex mocking is usually a Bad Thing | 11:22 |
| sambetts | it often means your test is touching to much code too | 11:25 |
| *** jrist has joined #openstack-ironic | 11:26 | |
| *** bks has joined #openstack-ironic | 11:26 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Add read and execute perms to httpboot and tftpboot folder https://review.openstack.org/374015 | 11:28 |
| *** dtantsur has quit IRC | 11:29 | |
| *** lucas-afk is now known as lucasagomes | 11:33 | |
| *** sturivnyi__ has joined #openstack-ironic | 11:37 | |
| sturivnyi__ | devananda, Hi =) Would be grateful if you could review my patch https://review.openstack.org/#/c/293634/ | 11:41 |
| patchbot | patch 293634 - python-ironicclient - Tests for testing chassis-create command | 11:41 |
| *** bdeschenes has joined #openstack-ironic | 11:46 | |
| lucasagomes | good morning all! | 11:49 |
| sambetts | hey lucasagomes! | 11:50 |
| openstackgerrit | Daniel Mellado proposed openstack/ironic: Skip tempest tests if ironic isn't available https://review.openstack.org/379256 | 11:51 |
| *** bathris has joined #openstack-ironic | 11:53 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/ironic: Remove unused methods https://review.openstack.org/379364 | 11:54 |
| *** bathris has quit IRC | 11:55 | |
| *** jrist has quit IRC | 11:56 | |
| *** bathri-s has quit IRC | 11:57 | |
| *** stevebak` is now known as stevebaker | 11:57 | |
| *** aparnav has quit IRC | 12:02 | |
| *** vmud213 has quit IRC | 12:07 | |
| jroll | morning y'all | 12:08 |
| sambetts | Morning jroll | 12:08 |
| lucasagomes | jroll, sambetts morning! | 12:08 |
| jroll | zigo: neat :P | 12:08 |
| *** tuanluong has quit IRC | 12:13 | |
| *** ccamacho|lunch is now known as ccamacho | 12:15 | |
| vsaienk0 | lucasagomes, jroll: seems I found root cause of grenade failure | 12:16 |
| lucasagomes | vsaienk0, o/ what's up ? | 12:16 |
| jroll | \o/ | 12:16 |
| sambetts | nice! | 12:16 |
| jroll | sorry I haven't been terribly helpful on that :( | 12:16 |
| vsaienk0 | after neutron upgrade/restart it uses new internal tag for network since we using vxlan network | 12:17 |
| *** jrist has joined #openstack-ironic | 12:17 | |
| sambetts | as in the OVS tag? | 12:17 |
| vsaienk0 | sambetts right! | 12:17 |
| sambetts | damn ... so we need to redetect the tag and switch our fake VM interface thing over to the new tag? | 12:18 |
| *** catintheroof has joined #openstack-ironic | 12:18 | |
| jroll | -.- | 12:18 |
| vsaienk0 | yeah, I playing with it https://review.openstack.org/#/c/377653/ | 12:18 |
| patchbot | patch 377653 - ironic - Testing broken grenade job | 12:18 |
| *** trown|outtypewww is now known as trown | 12:18 | |
| vsaienk0 | sambetts, jroll: what do you think if we switch gates to use flat network in future? | 12:19 |
| jroll | I don't have a good reason to be opposed | 12:20 |
| sambetts | Make sense to me, then we'll be testing closer to the real deployment case anyway | 12:20 |
| openstackgerrit | Tang Chen proposed openstack/ironic: Do not use mutable object as func default param https://review.openstack.org/379415 | 12:21 |
| vsaienk0 | jroll, sambetts: I will rebase code than https://review.openstack.org/#/c/340695 | 12:21 |
| patchbot | patch 340695 - ironic - Switch to Neutron Flat network. | 12:21 |
| jroll | sure | 12:21 |
| * lucasagomes looks | 12:22 | |
| vsaienk0 | jroll, sambetts: thanks! | 12:22 |
| lucasagomes | but yeah, also makes sense to me | 12:22 |
| jroll | vsaienk0: np, let me know if I can help with the grenade thing | 12:22 |
| sambetts | at least we know now thats its a test environment issue not a real issue | 12:23 |
| lucasagomes | sambetts, ++ | 12:24 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing broken grenade job https://review.openstack.org/377653 | 12:27 |
| *** jrist has quit IRC | 12:28 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: [install-guide] add section on Glance+Swift config https://review.openstack.org/379422 | 12:31 |
| *** bks has quit IRC | 12:38 | |
| *** dtantsur has joined #openstack-ironic | 12:38 | |
| openstackgerrit | Marc Aubry proposed openstack/python-ironicclient: Make error work with client in python3 https://review.openstack.org/378050 | 12:40 |
| *** jheroux has joined #openstack-ironic | 12:41 | |
| *** gcb has quit IRC | 12:41 | |
| openstackgerrit | M V P Nitesh proposed openstack/python-ironicclient: Fixed wrap from taking negative values https://review.openstack.org/379436 | 12:45 |
| *** rbartal has quit IRC | 12:47 | |
| mgould | morning jroll | 12:48 |
| *** e0ne has joined #openstack-ironic | 12:50 | |
| *** rloo has joined #openstack-ironic | 12:55 | |
| *** david-lyle has joined #openstack-ironic | 12:57 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic-ui: Update .gitreview for stable/newton https://review.openstack.org/379444 | 12:57 |
| *** vikrant has quit IRC | 12:57 | |
| jroll | betherly: the last thing to do for ironic-ui newton branch: https://review.openstack.org/#/c/379444/1 | 13:00 |
| patchbot | patch 379444 - ironic-ui (stable/newton) - Update .gitreview for stable/newton | 13:00 |
| milan | Morning Ironic! :) | 13:00 |
| betherly | jroll: ok cool | 13:01 |
| mgould | morning milan | 13:01 |
| jroll | betherly: for context, that's automatically made by the release automation | 13:02 |
| milan | morning aarefiev betherly dtantsur jroll lucasagomes mgould pas-ha sambetts vsaienk0 zigo ! :) | 13:02 |
| betherly | ah right ok! do we need to re-release once thats through to change tags? | 13:02 |
| betherly | jroll: ^ | 13:02 |
| jroll | betherly: nope! | 13:02 |
| jroll | betherly: the only thing that change does is make it so running 'git review' on that branch proposes it to the right branch | 13:02 |
| betherly | ah right gotcha | 13:03 |
| betherly | jroll: thanks!! | 13:03 |
| vsaienk0 | morning milan | 13:03 |
| milan | vdrok I missed ;) morning vdrok! | 13:03 |
| *** Goneri has joined #openstack-ironic | 13:05 | |
| rloo | hi and morning milan, vsaienk0, betherly, jroll, mgould | 13:07 |
| rloo | jroll: this can be abandoned: https://review.openstack.org/#/c/374864/ | 13:07 |
| patchbot | patch 374864 - ironic (stable/newton) - Link to newton version of old install guide | 13:07 |
| milan | morning rloo! :) | 13:07 |
| rloo | and Yay, we've backported the install guide to newton! | 13:07 |
| jroll | rloo: done | 13:07 |
| jroll | and \o/ | 13:07 |
| rloo | jroll: thx :) | 13:08 |
| sambetts | Hey milan | 13:08 |
| *** cdearborn has joined #openstack-ironic | 13:08 | |
| openstackgerrit | Merged openstack/ironic-ui: Update .gitreview for stable/newton https://review.openstack.org/379444 | 13:08 |
| milan | hey sambetts! :D /me just about to update the introspection statuses spec (was public holiday here yesterday) | 13:09 |
| sambetts | milan: oh awesome, I look forward to the new revision, what did we end up deciding about the date/inf thing? | 13:09 |
| milan | sambetts, no-one likes the inf | 13:10 |
| vsaienk0 | good morning rloo | 13:10 |
| milan | so we could use interval | 13:10 |
| milan | ? | 13:10 |
| mgould | morning rloo | 13:10 |
| milan | ?finished_at=<date>/ | 13:10 |
| milan | sambetts, ^ mind the slash | 13:10 |
| milan | or? | 13:11 |
| sambetts | milan: I like that idea, I guess we'll have to implement it ourselves though, but I don't think that is a big issues, we could even push to get it in an oslo utils lib if it works well | 13:11 |
| sergek | rloo: good morning | 13:11 |
| rloo | hi vsaienk0, sergek! | 13:11 |
| milan | sambetts, ack, I gave it a thought and what you suggested --- just splitting the slash --- makes most sense; we don't actually need duration spec | 13:12 |
| milan | two ISO dates or no date | 13:12 |
| milan | later we could enhance with duration, if needed | 13:12 |
| sambetts | exactly :D | 13:12 |
| *** baoli has joined #openstack-ironic | 13:13 | |
| milan | cool :) | 13:13 |
| * milan writes | 13:13 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Skip tempest tests if ironic isn't available https://review.openstack.org/379256 | 13:13 |
| *** baoli_ has joined #openstack-ironic | 13:14 | |
| vsaienk0 | sambetts, jroll: I'm wondered if you know why we decided to have vnic_type='baremetal' for neutron ports for Ironic case? | 13:15 |
| sambetts | vsaienk0: because the ml2 drivers need to know that its a baremetal port they are binding | 13:16 |
| sambetts | so they can handle it correctly | 13:16 |
| *** links has quit IRC | 13:17 | |
| sambetts | other than that I don't really know | 13:17 |
| vsaienk0 | sambetts: why not use device_owner option for this? It looks like more suitable https://github.com/openstack/ironic/blob/master/ironic/common/neutron.py#L96 | 13:17 |
| *** baoli has quit IRC | 13:17 | |
| sambetts | vsaienk0: in the tenant case the device owner is compute because its "owned" by the nova instance | 13:19 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing broken grenade job https://review.openstack.org/377653 | 13:20 |
| vsaienk0 | sambetts: right, but in case baremetal it is baremetal compute always | 13:20 |
| vsaienk0 | sambetts: I'm trying to pass somehow information about port type, eg: bonding, and I don't see other options like use vnic_type=bond | 13:21 |
| *** jaosorior has quit IRC | 13:21 | |
| sambetts | vsaienk0: you don't need to do that, bonds can be determined by the fact that the binding_profile has the LLC for 2 ports in it | 13:21 |
| sambetts | vsaienk0: e.g. 1 neutron port being requested to bind to 2 (or more) physical ports | 13:22 |
| vsaienk0 | sambetts: during nova boot we should pass vnic_type | 13:22 |
| *** jaosorior has joined #openstack-ironic | 13:22 | |
| sambetts | vsaienk0: why? the port isn't bound until we update it in Ironic later | 13:22 |
| vsaienk0 | sambetts: when user request boot instance it should somehow pass it instance should be connected via bond or not | 13:23 |
| sambetts | vsaienk0: bonds aren't user configurable | 13:23 |
| *** wajdi_ has joined #openstack-ironic | 13:23 | |
| mat128 | ^ | 13:23 |
| mat128 | deployers preference / chosen by flavor | 13:23 |
| vsaienk0 | sambetts: I want to give opportunity for user to build instances with bond and with single nics | 13:23 |
| mat128 | vsaienk0: we do bonding downstream (not with Neutron atm) and have built the following "system": | 13:24 |
| mat128 | nodes have capabilities (can do lacp or not) which is derived from TOR switch | 13:24 |
| mat128 | flavors require a capability | 13:24 |
| vsaienk0 | mat128: mostly all devices supports lacp fallback | 13:24 |
| mat128 | vsaienk0: nodes yes, but not TOR switches | 13:25 |
| mat128 | most low level / old are limited to 4 or some limit | 13:25 |
| vsaienk0 | mat128: I meant TOR switches, | 13:25 |
| mat128 | vsaienk0: we have plenty of switches for previous gen hardware that can't do more than 4 port channels | 13:26 |
| *** wajdi has quit IRC | 13:26 | |
| mat128 | given how those TOR switches are connected to old gen hardware, we sell those at a discount price | 13:26 |
| mat128 | (or rent? anyway, this is a public cloud) | 13:27 |
| mat128 | vsaienk0: so, long story short, nodes have caps, flavors have requirements and images have requirements | 13:27 |
| sambetts | the architechture we have right now is designed to enforce the bonds, until we have composable hardware (there is a spec somewhere) that allows capabilites to switch different features, e.g. bonds on/off, RAID levels etc then I don't see us implmenting it | 13:27 |
| mat128 | sambetts: thats correct | 13:27 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/377443 | 13:27 |
| mat128 | im just drawing the picture | 13:28 |
| vsaienk0 | mat128, sambetts: do you have strong opinion why not give user opportunity to do not allow specify vnic type (bond/single nic)? | 13:28 |
| sambetts | there is no way for a user to specify that information through nova | 13:28 |
| sambetts | without flavors | 13:28 |
| sambetts | which we don't support yet | 13:28 |
| mat128 | vsaienk0: downstream, the user chooses it via metadata, but this is completely our own | 13:28 |
| vsaienk0 | sambetts: I'm thinking to use vnic_type for it | 13:28 |
| mat128 | afaik, nova hasnt designed the metadata for this, but we found it more practical that way | 13:28 |
| vsaienk0 | neutron allows to set vnic_type for user, we can update nova to allow pass vnic_type also | 13:29 |
| mat128 | vsaienk0: how would you see the call to nova boot in both cases? | 13:29 |
| sambetts | vsaienk0: vnic_types are traditionally determined by the type of neutron network you are connecting to and the capablities of that network, we are kind of hacking it with the baremetal type | 13:30 |
| vsaienk0 | for single nic: nova --flavor baremetal --image $image --nic net-id=$net_id,vnic-type=bond | 13:30 |
| vsaienk0 | for bond case: nova --flavor baremetal --image $image --nic net-id=$net_id,vnic-type=bond | 13:30 |
| vsaienk0 | for single nic: nova --flavor baremetal --image $image --nic net-id=$net_id | 13:30 |
| mat128 | vsaienk0: can we attach more "nic" and they end up being tagged vlans? if so, what happens if I provide one of each? | 13:30 |
| mat128 | nova --flavor baremetal --image $image --nic net-id=$net_id,vnic-type=bond --nic net-id=$net_id2 | 13:31 |
| sambetts | mat128: thats what I'm working on ;) | 13:31 |
| vsaienk0 | sambetts: neutron already have a vnic_type=mcvtap, and it is in the same category with bond https://github.com/openstack/neutron/blob/dd4f1253c951d78a5b497680dfb31317ba469a58/neutron/tests/unit/agent/linux/test_ip_link_support.py#L58 | 13:31 |
| mat128 | sambetts: just hinting that vnic_type per-nic might not be the best, or there has to be logic to do the decision | 13:31 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Functional tests for portgroups in ironicclient https://review.openstack.org/297727 | 13:31 |
| mat128 | vlan-aware instances | 13:31 |
| sambetts | ++ | 13:31 |
| vsaienk0 | mat128: it is another use case, sambetts is working on it :) | 13:31 |
| *** mtanino has joined #openstack-ironic | 13:32 | |
| mat128 | vsaienk0: ok. what happens if I have more than 1 nic without sambetts' work? | 13:32 |
| mat128 | vsaienk0: oh, bond is already part of that list | 13:32 |
| mat128 | interesting :S | 13:32 |
| vsaienk0 | mat128: yah, and I don't see there baremetal :) | 13:33 |
| mat128 | baremetal shouldnt be any different | 13:33 |
| mat128 | I see gre | 13:34 |
| mat128 | you *could* do that | 13:34 |
| mat128 | on baremetal | 13:34 |
| mat128 | same with bond | 13:34 |
| vsaienk0 | yes, but baremetal according to this logic is not valid vnic_type | 13:35 |
| sambetts | thats because its wizardry and special :-P | 13:35 |
| mat128 | vsaienk0: not sure I'm following you correctly | 13:35 |
| mat128 | vsaienk0: you want vnic-type=baremetal ?? | 13:35 |
| sambetts | mat128: thats what we have now | 13:35 |
| mat128 | ah shit | 13:35 |
| mat128 | lol | 13:35 |
| sambetts | mat128: its a special vnic_type for showing this port is going to be directly connected to a BM | 13:36 |
| vsaienk0 | mat128: I'm looking for the best option that will allow user to pick server with single nic or bond | 13:36 |
| sambetts | I wonder though why we didn't just use the "direct" vnic type | 13:36 |
| mat128 | we really ought to port our network stuff over to https://github.com/openstack/networking-generic-switch/ | 13:36 |
| mat128 | https://github.com/internap/netman | 13:36 |
| mat128 | sambetts: direct sounds good, but we cant change the past | 13:37 |
| vsaienk0 | with proper deprecation it is possible | 13:37 |
| sambetts | yeah, I'm now just questioning why we made that decision in the first place :/ I can't remember the orignal arch discussions | 13:37 |
| jroll | so that ML2 mechanisms know it's for baremetal | 13:37 |
| vsaienk0 | sambetts: that was the original question I was asked for :) | 13:38 |
| jroll | imagine a neutron install for both VM and baremetal - they'll have OVS and arista-ml2 or whatever installed | 13:38 |
| jroll | how do the plugins know whether to handle it or not? | 13:38 |
| mat128 | whats the difference between baremetal and a virtual instance that completely owns a NIC on an hypervisor? | 13:38 |
| *** hshiina has quit IRC | 13:38 | |
| sambetts | jroll: binding_profile and device_owner? | 13:38 |
| vsaienk0 | jroll: yes, but looks like vnic_type is not appropriate option for it | 13:38 |
| sambetts | because its technicaly a vlan vnic right/ | 13:39 |
| sambetts | ? | 13:39 |
| jroll | it seems like the most appropriate thing at the time | 13:39 |
| sambetts | yeah :/ | 13:39 |
| jroll | (and still does, to me) | 13:39 |
| jroll | I don't think we should work on allowing nova users to choose single nic or bond, yet | 13:39 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add a possibility early determine if notification is needed https://review.openstack.org/379469 | 13:39 |
| jroll | let's get bonding working first | 13:39 |
| * sambetts tries to remember how they do it for SRIOV | 13:39 | |
| jroll | get vlan aware vms working | 13:39 |
| vsaienk0 | jroll: why not? | 13:40 |
| jroll | and then start talking about further user customization | 13:40 |
| vsaienk0 | jroll: I have working PoC :) | 13:40 |
| sambetts | we don't have any other user customisation on the nodes | 13:40 |
| jroll | vsaienk0: because one thing at a time - when we start trying to solve everything at once, we go down a rabbit hole and it takes three cycles to do anything | 13:40 |
| mat128 | ++ | 13:40 |
| mat128 | right now it should be operator decision | 13:40 |
| vsaienk0 | jroll: ok I see, so for now we want to proceed with static portgroups only | 13:41 |
| jroll | vsaienk0: yes, let's get that done first :) | 13:41 |
| *** joprovost has joined #openstack-ironic | 13:41 | |
| jroll | I'm surprised that someone might choose single nic when they can have a bond instead, anyway | 13:41 |
| mat128 | jroll: the image might not support bonding | 13:42 |
| mat128 | jroll: VMware ESXi is a good example, unless you have the right license :P | 13:42 |
| jroll | seriously? sighhhhhh | 13:42 |
| jroll | I would have assumed any modern OS supports it | 13:42 |
| mat128 | jroll: yup | 13:42 |
| mat128 | jroll: it's artificially crippled | 13:42 |
| mat128 | to make you buy a bigger license | 13:43 |
| jroll | right | 13:43 |
| vsaienk0 | jroll: might be you know who can help to add bonding support to cirros? https://bugs.launchpad.net/cirros/+bug/1625210 | 13:43 |
| openstack | Launchpad bug 1625210 in CirrOS "bonding module is missed on cirros-0.3.4-x86_64-disk.img " [Undecided,New] | 13:43 |
| sambetts | even then you can have some nodes with bonds and some without and use flavor keys to land on the non-bonded node | 13:43 |
| mat128 | vsaienk0: I remember lucasagomes doing something with this | 13:43 |
| * lucasagomes reads | 13:43 | |
| mat128 | or it was EFI? | 13:43 |
| mat128 | lucasagomes: latest cirros has bonding support, right? | 13:43 |
| lucasagomes | mat128, it was about EFI | 13:43 |
| mat128 | just unpublished | 13:44 |
| mat128 | aww | 13:44 |
| jroll | vsaienk0: smoser maintains that | 13:44 |
| jroll | vsaienk0: he's in #openstack, #cloud-init, #launchpad, maybe more | 13:44 |
| jroll | it just needs a release, yeah? | 13:44 |
| lucasagomes | mat128, the fix might be the same then... I was talking to <smoser> about releasing a new version of cirros which will solve both problems | 13:44 |
| *** athomas has joined #openstack-ironic | 13:44 | |
| lucasagomes | mat128, vsaienk0 have you guys tried to use the daily images ? | 13:44 |
| mat128 | lucasagomes: im not sure latest CirrOS supports bonding | 13:44 |
| vsaienk0 | lucasagomes: nope | 13:45 |
| lucasagomes | mat128, vsaienk0 https://review.openstack.org/375095 | 13:45 |
| patchbot | patch 375095 - openstack-dev/devstack - Ironic: Set default images for UEFI deployment | 13:45 |
| lucasagomes | may worth trying... tho this is for UEFI ^ | 13:45 |
| mat128 | http://download.cirros-cloud.net/daily/ | 13:45 |
| lucasagomes | mat128, ++ | 13:45 |
| lucasagomes | mat128, vsaienk0 if you guys can confirm it, we can make a even stronger case to get a new release | 13:46 |
| vsaienk0 | lucasagomes: let me check if bodning is supported in daily builds | 13:46 |
| lucasagomes | vsaienk0, thanks! | 13:47 |
| vsaienk0 | mat128, jroll, lucasagomes, sambetts: thanks for for help and active participation | 13:48 |
| lucasagomes | vsaienk0, we should thank YOU for looking into it | 13:48 |
| jroll | vsaienk0: of course, thanks for the good discussion and all the work you do :D | 13:48 |
| mat128 | I just found out CirrOS just grabs vanilla kernels: https://git.launchpad.net/cirros/tree/bin/build-release#n32 | 13:50 |
| sambetts | vsaienk0: totally :D, I am super into composable hardware, so this sort of thing is exactly where I'd like to go eventually, as well as for things like different RAID configurations, bios settings and more | 13:50 |
| mat128 | and changelog states they moved to 16.04 kernels now in 0.4.0: https://git.launchpad.net/cirros/tree/ChangeLog#n3 | 13:50 |
| yuriyz | hi ironic mariojv if you have some time please review a small patch https://review.openstack.org/#/c/379469/ | 13:51 |
| patchbot | patch 379469 - ironic - Add a possibility early determine if notification ... | 13:51 |
| mariojv | morning yuriyz - sure thing, i'll review it shortly | 13:52 |
| *** vsaienko has quit IRC | 13:52 | |
| *** xavierr|afk is now known as xavierr | 13:54 | |
| xavierr | good morning Ironic | 13:54 |
| *** vsaienko has joined #openstack-ironic | 13:55 | |
| lucasagomes | xavierr, morning | 13:55 |
| xavierr | morning sambetts mariojv lucasagomes vsaienk0 jroll yuriyz mariojv | 13:55 |
| mariojv | morning xavierr | 13:56 |
| mgould | morning xavierr yuriyz | 13:56 |
| xavierr | morning mgould :) | 13:57 |
| xavierr | lucasagomes: I did the refactor for that != and == in inspection interface patch :) | 13:58 |
| lucasagomes | xavierr, cool, ill re-review it today | 13:58 |
| xavierr | lucasagomes: I didn't know that operator lib | 13:58 |
| xavierr | lucasagomes: awesome ty! :) | 13:58 |
| lucasagomes | xavierr, it comes handy, sometimes :-) | 13:58 |
| xavierr | haha yeap | 13:58 |
| *** xavierr is now known as xavierr|mtg | 13:59 | |
| *** joprovost1 has joined #openstack-ironic | 14:01 | |
| *** joprovost has quit IRC | 14:02 | |
| *** joprovost1 is now known as joprovost | 14:02 | |
| *** sabeen has joined #openstack-ironic | 14:03 | |
| *** sabeen1 has joined #openstack-ironic | 14:05 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing broken grenade job https://review.openstack.org/377653 | 14:07 |
| mariojv | yuriyz: i'm having a hard time seeing how this would help performance by itself. are you planning to use should_notify outside the module for expensive notifications? | 14:08 |
| *** sabeen has quit IRC | 14:08 | |
| mariojv | i.e. using it before any real constructors are done | 14:09 |
| yuriyz | mariojv yes you are right for example we can skip chassis_id -> chassis_uuid conversion and do not execute db SELECT if notifications is disabled | 14:11 |
| mariojv | ok, sounds good | 14:11 |
| *** athomas has quit IRC | 14:11 | |
| *** rama_y has joined #openstack-ironic | 14:12 | |
| yuriyz | and we can redefine it for subclasses, but only as staticmethod | 14:13 |
| yuriyz | currently we have only static logic, but if some dependencies from class will be needed we can convert it to classmethod | 14:15 |
| jroll | devananda: dtantsur: care to look at https://review.openstack.org/#/c/374863/ ? | 14:21 |
| patchbot | patch 374863 - ironic (stable/newton) - Update devstack IPA_DOWNLOAD_BRANCH to stable/newton | 14:21 |
| mariojv | yuriyz: reviewed, i think it's fine as staticmethod | 14:21 |
| mariojv | yuriyz: code looks good, main thing is i think there should be 1 more unit test | 14:22 |
| yuriyz | mariojv thanks | 14:22 |
| mariojv | np | 14:22 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Functional tests for portgroups in ironicclient https://review.openstack.org/297727 | 14:23 |
| jroll | sambetts: we need a task framework :( https://bugs.launchpad.net/ironic/+bug/1623556 | 14:23 |
| openstack | Launchpad bug 1623556 in Ironic "[RFE] Extend send_raw vendor pass-through command to return collected data" [Wishlist,Confirmed] - Assigned to Grzegorz Grasza (xek) | 14:23 |
| * jroll runs off to find food | 14:24 | |
| *** sdake has quit IRC | 14:24 | |
| *** sdake has joined #openstack-ironic | 14:26 | |
| *** syed_ has joined #openstack-ironic | 14:30 | |
| *** sdake_ has joined #openstack-ironic | 14:34 | |
| *** sdake has quit IRC | 14:36 | |
| *** mkam has joined #openstack-ironic | 14:36 | |
| sambetts | jroll: hmm interesting, like a framework for starting a background task and then retreiving the data async? | 14:36 |
| *** amotoki has joined #openstack-ironic | 14:38 | |
| openstackgerrit | Michael Turek proposed openstack/python-ironicclient: Refactor provision state so all actions can use inherited take_action https://review.openstack.org/364552 | 14:43 |
| *** amotoki has quit IRC | 14:48 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Skip tempest tests if ironic isn't available https://review.openstack.org/379256 | 14:50 |
| *** aswadr_ has joined #openstack-ironic | 14:53 | |
| *** sdake_ is now known as sdake_dnd | 14:53 | |
| *** sdake has joined #openstack-ironic | 14:59 | |
| *** sdake_dnd has quit IRC | 15:00 | |
| *** mjura has quit IRC | 15:00 | |
| jroll | sambetts: yeah, something like that, probably could include most of what we do into it | 15:02 |
| jroll | retrieving data/status/etc | 15:02 |
| sambetts | totally! it would make a ton of sense to be able to reuse it for get power state etc | 15:03 |
| lucasagomes | vsaienk0, apparently bonding is not present :-/ | 15:04 |
| lucasagomes | vsaienk0, I've deployed a daily image here to test uefi and tried to load it | 15:05 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add a possibility early to determine if notification is needed https://review.openstack.org/379469 | 15:06 |
| vsaienk0 | lucasagomes: where I can find instructions how to build and cirros image? | 15:06 |
| lucasagomes | vsaienk0, not sure, lemme try to find something | 15:06 |
| * lucasagomes just tested the daily images | 15:06 | |
| lucasagomes | vsaienk0, https://git.launchpad.net/cirros/tree/bin/ | 15:08 |
| * lucasagomes haven't tried | 15:08 | |
| lucasagomes | vsaienk0, apparently all you need is to add "bonding" to src/etc/modules | 15:11 |
| vsaienk0 | lucasagomes: thanks, I will try | 15:11 |
| lucasagomes | np, I will leave one image buiding while I go get some food as well | 15:12 |
| *** lucasagomes is now known as lucas-hungry | 15:12 | |
| *** amotoki has joined #openstack-ironic | 15:14 | |
| *** nicodemos has joined #openstack-ironic | 15:15 | |
| *** amotoki has quit IRC | 15:20 | |
| *** sdake is now known as sdake_dnd | 15:26 | |
| *** amotoki has joined #openstack-ironic | 15:27 | |
| sturivnyi__ | dtantsur Hi, would be grateful if you could review my patch https://review.openstack.org/#/c/293634/ | 15:27 |
| patchbot | patch 293634 - python-ironicclient - Tests for testing chassis-create command | 15:27 |
| *** amotoki has quit IRC | 15:28 | |
| *** rbrud has joined #openstack-ironic | 15:29 | |
| *** amotoki has joined #openstack-ironic | 15:34 | |
| *** baoli_ has quit IRC | 15:34 | |
| *** Romanenko_K has quit IRC | 15:35 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 15:36 |
| *** amotoki has quit IRC | 15:36 | |
| xek | jroll, Hi, in regards to https://bugs.launchpad.net/ironic/+bug/1623556 I saw that there is a similar feature already in the get_sensors_data, but it runs ipmitool command directly | 15:40 |
| openstack | Launchpad bug 1623556 in Ironic "[RFE] Extend send_raw vendor pass-through command to return collected data" [Wishlist,Confirmed] - Assigned to Grzegorz Grasza (xek) | 15:40 |
| jroll | xek: yeah, sounds like they want it on demand | 15:40 |
| xek | jroll, do you think we should maybe implement this call in pure python then? | 15:42 |
| xek | after we have a framework for background tasks | 15:42 |
| jroll | xek: I mean, I don't have a problem with it using ipmitool, why would we need to do it in python? | 15:43 |
| jroll | also, just realized you submitted the bug, heh :D | 15:43 |
| *** krtaylor has quit IRC | 15:44 | |
| xek | jroll, as perr sambetts comment, it looks like it could take too long, and may be needing a background task to not take too much resources | 15:44 |
| jroll | xek: well, we need to have it happen in the conductor, I don't think sambetts disagrees with that, only with holding up the API for so long | 15:45 |
| xek | but maybe this task should run ipmitool then... | 15:45 |
| jroll | yeah | 15:45 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add power state change notifications https://review.openstack.org/321865 | 15:47 |
| bfournie | hi sambetts, I've been trying to run the link_local_connection hook and it does execute, but I keep getting a KeyError in it during introspection. I was wondering if you can take a look at notes here: http://paste.openstack.org/show/583510/ to see if the sequence to enable it was correct. Thanks. | 15:49 |
| *** openstackgerrit has quit IRC | 15:49 | |
| *** openstackgerrit has joined #openstack-ironic | 15:50 | |
| bfournie | s/link_local_connection/local_link_connection | 15:50 |
| *** karthiks has joined #openstack-ironic | 15:57 | |
| *** pcaruana has quit IRC | 16:03 | |
| *** tangchen_ has quit IRC | 16:03 | |
| *** absubram has joined #openstack-ironic | 16:03 | |
| *** rcernin has quit IRC | 16:05 | |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient: If no resource, don't call Resource.to_dict() https://review.openstack.org/379567 | 16:13 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector-specs: Endpoint to list introspection statuses https://review.openstack.org/375045 | 16:14 |
| zackf | Hi all, i've followed the ironic install guide and configured neutron according to the guide. When i try to boot a baremetal node, i'm getting neutron port not found... I've ran the ironic port-create command, is there anything i need to do on the neutron side to assign a port? | 16:14 |
| sambetts | bfournie: taking a look now, thats really odd behaivor | 16:14 |
| *** tangchen_ has joined #openstack-ironic | 16:15 | |
| *** rbrud has quit IRC | 16:15 | |
| sambetts | bfournie: so those mac address that are 1 bit to high are coming straight from the inventory discovered by inspector | 16:15 |
| bfournie | sambetts: I'm wondering if I need to change the order and put local_link_connection before extra_hardware | 16:15 |
| *** lucas-hungry is now known as lucasagomes | 16:15 | |
| sambetts | bfournie: nothing in inspector should be editing the physcial mac addresses as part of the processing | 16:17 |
| sambetts | bfournie: do you have the inspecotr logs I can look at?? | 16:18 |
| sambetts | bfournie: / more of the inspector logs | 16:19 |
| bfournie | sambetts: yes, what is the best way to post them for you? | 16:19 |
| sambetts | bfournie: https://bugs.launchpad.net/ironic-inspector/+bug/1628991 you can add an attachment to a comment on this bug report | 16:21 |
| openstack | Launchpad bug 1628991 in Ironic Inspector "Ironic-Inspector LLDP processing seems to be handling the wrong mac address" [Undecided,New] | 16:21 |
| bfournie | sambetts: I've also updated http://paste.openstack.org/show/583519/, I think extra_hardware is deleting the data key, which is why the mac has been removed | 16:22 |
| sambetts | bfournie: Looking at the code the iface['mac_address'] comes from the inventory we've inspected, so that data still exists, and the ironic_ports list is the result of doing an ironic port-list so it seems we are inspecting a mac address that isn't configured as an Ironic port | 16:25 |
| *** lifeless has quit IRC | 16:25 | |
| *** ijw has joined #openstack-ironic | 16:26 | |
| *** sdake_dnd is now known as sdake | 16:27 | |
| *** baoli has joined #openstack-ironic | 16:28 | |
| sambetts | bfournie: I think the problem is the LLC hook works on the assumption that inspector will add all the ports in finds on the node into Ironic | 16:29 |
| bfournie | sambetts: this is the mac for eth1. Don't we just add ironic ports for the nic attached to provisioning network (eth0 in this case). so we wouldn't have an ironic port for eth1. | 16:29 |
| bfournie | sambetts: yes, I agree | 16:29 |
| sambetts | bfournie: so I might need to add a conditional into the LLC hook to check the CONF.processing.keep_ports | 16:29 |
| *** e0ne has quit IRC | 16:30 | |
| bfournie | sambetts: yep. btw, I have added the log to the bug report | 16:30 |
| sambetts | bfournie: for reference what do you set CONF.processing.keep_ports too ? | 16:30 |
| *** ijw has quit IRC | 16:30 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/ironic: Enable DeprecationWarning in test environments https://review.openstack.org/379581 | 16:31 |
| bfournie | sambetts: I didn't change it, but it was set to 'keep_ports = added' | 16:32 |
| *** joprovost1 has joined #openstack-ironic | 16:32 | |
| *** joprovost has quit IRC | 16:32 | |
| *** joprovost1 is now known as joprovost | 16:32 | |
| *** lifeless has joined #openstack-ironic | 16:33 | |
| bfournie | sambetts: maybe it should be set to 'keep_ports = all'? | 16:33 |
| sambetts | bfournie: what about the CONF.processing.add_ports setting? | 16:33 |
| sambetts | bfournie: keep_ports is about deleteing ports that we don't find rather than adding them | 16:34 |
| *** derekh has quit IRC | 16:34 | |
| bfournie | sambetts: ok, add_ports is not set in conf, I guess it defaults to pxe? "#add_ports = pxe" | 16:35 |
| rama_y | Hi, question on IPA: to test IPA code, what is the best approach to build ramdisk - CoreOS or Diskimagebuilder or TinyIPA? | 16:35 |
| rama_y | JayF: question on IPA? | 16:37 |
| sambetts | bfournie: oooo, quick question have you got the validate_interfaces hook running? | 16:37 |
| sambetts | bfournie: having that add_ports setting set should mean that inspector adds the port it PXE booted on into Ironic | 16:38 |
| sambetts | bfournie: but it obviously isn't adding eth0 into Ironic | 16:38 |
| JayF | rama_y: for testing, tinyipa is generally accepted as the preferred -- it's smaller and better suited for test environments | 16:41 |
| *** rajinir has joined #openstack-ironic | 16:41 | |
| bfournie | sambetts: it looks like validate_interfaces is one of the default hooks, but I have not verified that it ran. I think eth0 was added as its mac matches the Ironic port - b0:83:fe:c6:63:86 | 16:42 |
| rama_y | JayF: ok, thanks. | 16:42 |
| sambetts | bfournie: sorry my bad, I got eth0 and eth1 confused | 16:43 |
| *** priteau has joined #openstack-ironic | 16:44 | |
| sambetts | bfournie: yeah so things are working as expected then, *wipes forehead*, I think its a bug in the LLC hook then which I can address, I never saw this happen because I always ran with add_ports=all | 16:44 |
| bfournie | sambetts: np, from the log I posted I can see that validate_interfaces ran and skipped all interfaces but eth0 | 16:45 |
| sambetts | ++ yup I can see that too :) I'll get a patch up to address it ASAP | 16:45 |
| sambetts | bfournie: thanks for testing!!! | 16:45 |
| bfournie | sambetts: Great, thank you!!! | 16:46 |
| bfournie | sambetts: one more question about this hook - is it planned that local_link_connection will be a default hook or will it always need to be added? | 16:47 |
| sambetts | bfournie: currently discovering LLDP information as part of IPA is optional, i.e it requires you to add an additional flag to the kernal boot options, so currently its an optional hook. The reason its option in IPA is because some people said it causes the ramdisk to be slow, I think that might have been a bug which we have addresses. I would like it to be a default option, because if it | 16:50 |
| sambetts | doesn't affect performance then there is no harm in running it in an evironment that doesn't support LLDP | 16:50 |
| NobodyCam | Good Morning Ironic'ers | 16:53 |
| milan | morning NobodyCam! :) | 16:53 |
| NobodyCam | morning milan :) | 16:53 |
| *** trown is now known as trown|lunch | 16:53 | |
| bfournie | sambetts: ok, thanks for the explanation | 16:54 |
| *** amotoki has joined #openstack-ironic | 16:57 | |
| *** amotoki has quit IRC | 17:03 | |
| milan | bfournie, btw, you could possibly update the config and call the reprocess command, maybe it will show no error | 17:03 |
| milan | or more errors :D | 17:03 |
| *** krtaylor has joined #openstack-ironic | 17:04 | |
| *** xavierr|mtg is now known as xavierr | 17:05 | |
| xavierr | morning NobodyCam :D | 17:06 |
| NobodyCam | morning xavierr :) | 17:06 |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector: LLC Hook: Do not assume interfaces are added to Ironic https://review.openstack.org/379614 | 17:15 |
| sambetts | bfournie: ^^^^^ | 17:15 |
| sambetts | bfournie: Please try with this patch, I hope it solves your problem | 17:16 |
| bfournie | sambetts: great, thanks. I will try this now. | 17:16 |
| sambetts | oh wow ... when did it become 18:16... | 17:16 |
| bfournie | sambetts: sorry, late night for you. thanks again for looking at this | 17:18 |
| sambetts | thats ok, time seems to fly for me atm | 17:18 |
| *** jaosorior has quit IRC | 17:20 | |
| *** aNuposic has joined #openstack-ironic | 17:20 | |
| *** MattMan has quit IRC | 17:21 | |
| *** MattMan has joined #openstack-ironic | 17:21 | |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic: Timing metrics: OneView drivers https://review.openstack.org/379619 | 17:22 |
| *** aswadr_ has quit IRC | 17:22 | |
| jlvillal | jroll: Do you know what the plan is for the experimental job: tempest-dsvm-ironic-pxe_ssh-full ? | 17:23 |
| *** mgould is now known as mgould|afk | 17:24 | |
| jlvillal | Seems like it has been experimental for over a year now. | 17:24 |
| jroll | jlvillal: hopefully find time to get it working one day | 17:24 |
| * jlvillal is only complaining because it slows down the experimental job he cares about testing :) | 17:24 | |
| jlvillal | jroll: Okay. thanks | 17:24 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 17:33 |
| mat128 | anyone here to review this py3 bug on ironicclient? https://review.openstack.org/#/c/378050/ | 17:44 |
| patchbot | patch 378050 - python-ironicclient - Make error work with client in python3 | 17:44 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Update external links to developer documentation https://review.openstack.org/378990 | 17:47 |
| *** e0ne has joined #openstack-ironic | 17:48 | |
| jlvillal | mat128: I wouldn't mind a better commit message on the py3 bug. | 17:49 |
| * jlvillal is not a fan of one line commit messages | 17:50 | |
| mat128 | Marx314: ^ | 17:50 |
| Marx314 | good, I'll try to be more explicit | 17:50 |
| *** lukl1 has quit IRC | 17:51 | |
| *** lukl has joined #openstack-ironic | 17:52 | |
| rloo | ditto mat128, Marx314 :) | 17:52 |
| * milan gone | 17:52 | |
| milan | good night Ironic! :) | 17:52 |
| rloo | bye milan | 17:52 |
| jlvillal | milan: Ciao | 17:53 |
| milan | o/ :) | 17:53 |
| *** milan has quit IRC | 17:54 | |
| *** electrofelix has quit IRC | 17:55 | |
| *** trown|lunch is now known as trown | 17:56 | |
| NobodyCam | night mikal | 17:56 |
| NobodyCam | gah | 17:56 |
| NobodyCam | tab fail | 17:56 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Update configuration reference link to latest draft https://review.openstack.org/378991 | 17:57 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Update external links to developer documentation https://review.openstack.org/378990 | 17:57 |
| mat128 | damn ;@ | 17:57 |
| mat128 | it resubmitted an old revision | 17:57 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Update configuration reference link to latest draft https://review.openstack.org/378991 | 18:01 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Update external links to developer documentation https://review.openstack.org/378990 | 18:01 |
| mat128 | good ^ | 18:01 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing broken grenade job https://review.openstack.org/377653 | 18:03 |
| jlvillal | vsaienk0: Hopefully you saw clarkb's message: <clarkb> vsaienko: jlvillal btw multinode is currently broken if using neutron + devstack on osic. This may impact the work you are doing if you see ssh failures | 18:03 |
| openstackgerrit | Marc Aubry proposed openstack/python-ironicclient: Fix python3 compatibility when HTTP Error are returned https://review.openstack.org/378050 | 18:04 |
| mat128 | jlvillal, rloo: sounds good now ^ thanks Marx314 | 18:05 |
| rloo | mat128: thx, will look later. | 18:06 |
| *** jpich has quit IRC | 18:07 | |
| openstackgerrit | Xavier proposed openstack/ironic: Avoid name errors in oneview periodics https://review.openstack.org/374989 | 18:11 |
| xavierr | rloo, vdrok ^^^ :) | 18:12 |
| *** Sukhdev has joined #openstack-ironic | 18:12 | |
| *** karthiks has quit IRC | 18:12 | |
| *** xavierr is now known as xavierr| | 18:14 | |
| *** xavierr| is now known as xavierr|class | 18:14 | |
| *** baoli has quit IRC | 18:16 | |
| *** baoli has joined #openstack-ironic | 18:17 | |
| *** joprovost has quit IRC | 18:20 | |
| *** aNuposic has quit IRC | 18:24 | |
| *** karthiks has joined #openstack-ironic | 18:25 | |
| *** aNuposic has joined #openstack-ironic | 18:26 | |
| openstackgerrit | Marc Aubry proposed openstack/python-ironicclient: Fix python3 compatibility when HTTP Error are returned https://review.openstack.org/378050 | 18:42 |
| *** sdake has quit IRC | 18:50 | |
| *** sdake has joined #openstack-ironic | 18:50 | |
| *** krtaylor has quit IRC | 18:54 | |
| *** Mr_T has left #openstack-ironic | 18:55 | |
| *** Goneri has quit IRC | 19:03 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing broken grenade job https://review.openstack.org/377653 | 19:06 |
| *** baoli has quit IRC | 19:08 | |
| *** baoli has joined #openstack-ironic | 19:08 | |
| *** rama_y has quit IRC | 19:30 | |
| *** rama_y has joined #openstack-ironic | 19:31 | |
| openstackgerrit | Andre Aranha proposed openstack/ironic: Added validation for local_link_connection https://review.openstack.org/377106 | 19:34 |
| *** vsaienko has quit IRC | 19:34 | |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Added validation for local_link_connection https://review.openstack.org/377103 | 19:34 |
| jlvillal | Marx314: How come the unit tests didn't fail before the patch you proposed? | 19:34 |
| jlvillal | Marx314: We do test with Python 3. So slightly confused. | 19:34 |
| *** vsaienko has joined #openstack-ironic | 19:34 | |
| Marx314 | the unittest didn't test any bytes object | 19:35 |
| jlvillal | Marx314: Is there a test now which does that? | 19:35 |
| *** aNuposic has quit IRC | 19:37 | |
| Marx314 | now that the code use the text function of the lib requests, no need for that anymore | 19:37 |
| Marx314 | unittest can't make sure everything will work together, functional test do... it's the merge with the implementation of requests here the issue | 19:38 |
| openstackgerrit | Stenio Araujo proposed openstack/python-oneviewclient: Adds attribute mac_type to a SPT and a validation https://review.openstack.org/377054 | 19:38 |
| openstackgerrit | Stenio Araujo proposed openstack/ironic: Adds another validation step when using dynamic allocation https://review.openstack.org/377073 | 19:38 |
| jlvillal | Marx314: Not sure I fully understand. But I have go talk to someone so I don't have time to look into it. Thanks. | 19:39 |
| *** yarkot has quit IRC | 19:39 | |
| *** purp_too has quit IRC | 19:39 | |
| *** purp has joined #openstack-ironic | 19:40 | |
| *** baoli has quit IRC | 19:40 | |
| *** baoli has joined #openstack-ironic | 19:41 | |
| *** Goneri has joined #openstack-ironic | 19:42 | |
| *** yarkot has joined #openstack-ironic | 19:42 | |
| *** romcheg has quit IRC | 19:43 | |
| *** manjeets has quit IRC | 19:43 | |
| *** lekha has quit IRC | 19:43 | |
| *** zhenguo has quit IRC | 19:43 | |
| *** tdurakov has quit IRC | 19:43 | |
| *** aNuposic has joined #openstack-ironic | 19:43 | |
| *** romcheg has joined #openstack-ironic | 19:45 | |
| *** francoblanco_ has joined #openstack-ironic | 19:45 | |
| *** tdurakov has joined #openstack-ironic | 19:45 | |
| *** hj-hpe has joined #openstack-ironic | 19:45 | |
| *** manjeets has joined #openstack-ironic | 19:46 | |
| *** lekha has joined #openstack-ironic | 19:46 | |
| *** vsaienko has quit IRC | 19:47 | |
| *** NobodyCam_ has joined #openstack-ironic | 19:48 | |
| *** francoblanco has quit IRC | 19:50 | |
| *** hpe-hj has quit IRC | 19:50 | |
| *** NobodyCam has quit IRC | 19:50 | |
| *** marlinc has quit IRC | 19:50 | |
| *** francoblanco_ is now known as francoblanco | 19:50 | |
| *** NobodyCam_ is now known as NobodyCam | 19:50 | |
| *** marlinc has joined #openstack-ironic | 19:51 | |
| *** zhenguo has joined #openstack-ironic | 19:53 | |
| lucasagomes | folks calling it a day, have a great evening all! | 19:53 |
| *** lucasagomes is now known as lucas-afk | 19:54 | |
| mat128 | o/ lucas-afk | 19:54 |
| xavierr|class | good evening lucas-afk o/ | 19:54 |
| *** xavierr|class is now known as xavierr | 19:55 | |
| *** vsaienko has joined #openstack-ironic | 19:58 | |
| *** sdake has quit IRC | 20:08 | |
| *** bdeschenes has quit IRC | 20:13 | |
| *** rbrud has joined #openstack-ironic | 20:19 | |
| *** baoli has quit IRC | 20:21 | |
| *** baoli has joined #openstack-ironic | 20:22 | |
| *** catintheroof has quit IRC | 20:32 | |
| *** baoli has quit IRC | 20:42 | |
| *** baoli has joined #openstack-ironic | 20:44 | |
| *** melwitt has quit IRC | 20:44 | |
| vsaienko | I'm wondered if jroll, jlvillal are here? | 20:46 |
| jroll | vsaienko: kind of, what's up | 20:46 |
| jlvillal | vsaienko: I'm here. Though on a phone call. | 20:46 |
| *** trown is now known as trown|outtypewww | 20:47 | |
| vsaienko | check this https://review.openstack.org/#/c/377653/ | 20:47 |
| patchbot | patch 377653 - ironic - Testing broken grenade job | 20:47 |
| vsaienko | I spend 2 days debugging this issue... | 20:47 |
| vsaienko | I have grenade :) | 20:47 |
| jlvillal | vsaienko: Woot! | 20:47 |
| vsaienko | I hate grenade :) | 20:47 |
| jroll | vsaienko: thank you for this <3 | 20:48 |
| jlvillal | vsaienko: Oh I thought you were saying you "have grenade" | 20:48 |
| jroll | vsaienko: so we just need the grenade patch? | 20:49 |
| jroll | also O_O I can't believe that was the bug | 20:49 |
| jroll | holy cow | 20:49 |
| vsaienko | It was auto spelling on mac | 20:50 |
| vsaienko | jroll, It is to late here, I will update patch tomorrow, and add a lot of NOTES | 20:50 |
| jroll | vsaienko: but, https://review.openstack.org/#/c/379192/ is the fix, right? | 20:50 |
| patchbot | patch 379192 - openstack-dev/grenade - Fix neutron resource cleanup | 20:50 |
| jroll | vsaienko: ah, we need the ironic patch too? | 20:51 |
| *** melwitt has joined #openstack-ironic | 20:51 | |
| vsaienko | jroll, right we need to update tag on ovs port after neutron restart also | 20:51 |
| *** melwitt is now known as Guest99228 | 20:52 | |
| JayF | so can you clean up the ironic-side patch to remove the debugging stuff | 20:52 |
| JayF | so we can vote on that and get folks to vote on the grenade bits? | 20:52 |
| jroll | vsaienko: cool, thank you | 20:52 |
| jlvillal | vsaienko: Great work! Thank you very much. | 20:53 |
| vsaienko | JayF patch to grenade is not required, but desired https://review.openstack.org/#/c/379192/ | 20:53 |
| patchbot | patch 379192 - openstack-dev/grenade - Fix neutron resource cleanup | 20:53 |
| *** baoli has quit IRC | 20:53 | |
| vsaienko | If you do not mind I will upload fresh version of ironic patch tomorrow | 20:53 |
| *** baoli has joined #openstack-ironic | 20:54 | |
| jroll | vsaienko: I'll bug grenade folks to merge that, clean up the ironic patch, and get that merged | 20:55 |
| vsaienko | jlvillal, seems I need to update presentation with new networking in grenade :) | 20:55 |
| jroll | vsaienko: and then you can add notes in a follow up? | 20:55 |
| jlvillal | vsaienko: heh | 20:55 |
| vsaienko | jroll, ok sure | 20:55 |
| jroll | cool | 20:55 |
| * jroll does it | 20:55 | |
| jroll | dang, sdague out | 20:55 |
| *** mkam has quit IRC | 20:56 | |
| *** vsaienko has quit IRC | 20:56 | |
| *** mkam has joined #openstack-ironic | 20:57 | |
| *** baoli has quit IRC | 20:57 | |
| *** ijw has joined #openstack-ironic | 20:58 | |
| *** baoli has joined #openstack-ironic | 20:58 | |
| jroll | I find it odd that we're creating ovs taps in destroy | 20:58 |
| jroll | but let's roll with it and find something better later | 20:58 |
| *** amotoki has joined #openstack-ironic | 21:00 | |
| *** vishwanathj has quit IRC | 21:01 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Fix grenade jobs https://review.openstack.org/377653 | 21:03 |
| jroll | alright, hope that passes :) | 21:03 |
| jroll | removed depends-on and such | 21:03 |
| *** e0ne has quit IRC | 21:05 | |
| *** anteaya has quit IRC | 21:05 | |
| *** amotoki has quit IRC | 21:05 | |
| *** mkam has quit IRC | 21:05 | |
| jlvillal | jroll: There is a file "A" in there | 21:06 |
| jroll | ah boo | 21:07 |
| jroll | hang on | 21:07 |
| jlvillal | jroll: It was there before. | 21:07 |
| jroll | I focused on the one file | 21:07 |
| jroll | yeah I know | 21:07 |
| *** ijw has quit IRC | 21:08 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Fix grenade jobs https://review.openstack.org/377653 | 21:08 |
| jroll | alright, that should do it - removed commented code too | 21:08 |
| *** e0ne has joined #openstack-ironic | 21:08 | |
| jlvillal | thanks | 21:10 |
| *** vsaienko has joined #openstack-ironic | 21:12 | |
| *** Goneri has quit IRC | 21:13 | |
| *** sdake has joined #openstack-ironic | 21:13 | |
| JayF | looking | 21:13 |
| * jlvillal hopes it works :) | 21:14 | |
| *** mkam has joined #openstack-ironic | 21:18 | |
| *** jheroux has quit IRC | 21:24 | |
| *** baoli has quit IRC | 21:26 | |
| *** baoli has joined #openstack-ironic | 21:27 | |
| xavierr | good evening all | 21:27 |
| *** xavierr is now known as xavierr|afk | 21:28 | |
| *** anteaya has joined #openstack-ironic | 21:28 | |
| *** mkam has quit IRC | 21:40 | |
| JayF | zigo: Do you all package up any of our documentation? Or is it code-only? | 21:40 |
| *** e0ne has quit IRC | 21:45 | |
| *** e0ne has joined #openstack-ironic | 21:45 | |
| *** e0ne has quit IRC | 21:45 | |
| *** e0ne has joined #openstack-ironic | 21:46 | |
| *** e0ne has quit IRC | 21:46 | |
| *** e0ne has joined #openstack-ironic | 21:47 | |
| *** e0ne has quit IRC | 21:47 | |
| *** e0ne has joined #openstack-ironic | 21:47 | |
| *** e0ne has quit IRC | 21:48 | |
| *** dtantsur has quit IRC | 21:48 | |
| *** e0ne has joined #openstack-ironic | 21:48 | |
| *** e0ne has quit IRC | 21:49 | |
| *** e0ne has joined #openstack-ironic | 21:49 | |
| *** e0ne has quit IRC | 21:49 | |
| *** Goneri has joined #openstack-ironic | 21:53 | |
| *** baoli has quit IRC | 21:53 | |
| *** baoli has joined #openstack-ironic | 21:54 | |
| mrda | Morning Ironic | 21:58 |
| *** Goneri has quit IRC | 22:05 | |
| *** baoli has quit IRC | 22:06 | |
| *** baoli has joined #openstack-ironic | 22:07 | |
| *** ijw has joined #openstack-ironic | 22:11 | |
| *** absubram has quit IRC | 22:12 | |
| *** causten has joined #openstack-ironic | 22:14 | |
| jlvillal | jroll: JayF: https://review.openstack.org/#/c/377653/ seems to fix Grenade. It got past the point where it was failing before. I did +2 and +A. So if it passes all the jobs it should head to the merge gate. | 22:20 |
| patchbot | patch 377653 - ironic - Fix grenade jobs | 22:20 |
| JayF | sure, works for me | 22:20 |
| *** causten has quit IRC | 22:21 | |
| *** causten has joined #openstack-ironic | 22:22 | |
| *** Guest99228 is now known as melwitt | 22:23 | |
| *** wajdi_ has quit IRC | 22:25 | |
| *** cdearborn has quit IRC | 22:26 | |
| *** baoli has quit IRC | 22:41 | |
| *** baoli has joined #openstack-ironic | 22:41 | |
| *** baoli has quit IRC | 22:42 | |
| *** baoli has joined #openstack-ironic | 22:42 | |
| jlvillal | Woot! Grenade test passed :) | 22:46 |
| jlvillal | https://review.openstack.org/#/c/377653/ | 22:46 |
| patchbot | patch 377653 - ironic - Fix grenade jobs | 22:46 |
| *** dprince has quit IRC | 23:05 | |
| *** awiddersheim has joined #openstack-ironic | 23:06 | |
| *** sdake has quit IRC | 23:12 | |
| *** baoli has quit IRC | 23:13 | |
| *** priteau has quit IRC | 23:16 | |
| jlvillal | sambetts: vsaienk0: https://bugs.launchpad.net/neutron/+bug/1629133 FYI | 23:25 |
| openstack | Launchpad bug 1629133 in neutron "New neutron subnet pool support breaks multinode testing." [Undecided,New] | 23:25 |
| *** rajinir has quit IRC | 23:25 | |
| *** rbrud has quit IRC | 23:29 | |
| *** vsaienko has quit IRC | 23:33 | |
| *** ijw has quit IRC | 23:33 | |
| *** ijw has joined #openstack-ironic | 23:34 | |
| *** ijw has quit IRC | 23:38 | |
| *** ijw has joined #openstack-ironic | 23:38 | |
| *** bdeschenes has joined #openstack-ironic | 23:46 | |
| *** ijw has quit IRC | 23:48 | |
| *** ijw has joined #openstack-ironic | 23:49 | |
| *** ijw has quit IRC | 23:50 | |
| *** sdake has joined #openstack-ironic | 23:55 | |
| *** rama_y has quit IRC | 23:57 | |
| *** rama_y has joined #openstack-ironic | 23:57 | |
| *** vsaienko has joined #openstack-ironic | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!