| *** hemna is now known as hemnafk | 00:01 | |
| *** ijw_ has joined #openstack-ironic | 00:02 | |
| *** ijw has quit IRC | 00:06 | |
| *** BadCub is now known as BadCub_Hiding | 00:10 | |
| *** shakamunyi has joined #openstack-ironic | 00:13 | |
| *** barra204 has joined #openstack-ironic | 00:13 | |
| *** jlk has joined #openstack-ironic | 00:13 | |
| *** jlk has quit IRC | 00:13 | |
| *** jlk has joined #openstack-ironic | 00:13 | |
| *** ijw_ has quit IRC | 00:27 | |
| *** ijw has joined #openstack-ironic | 00:27 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Lock Bifrost to Ansible stable-1.9 https://review.openstack.org/187532 | 00:29 |
|---|---|---|
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Preserve logs in logs/ folder https://review.openstack.org/186581 | 00:31 |
| *** zhenguo has joined #openstack-ironic | 00:33 | |
| *** Sukhdev has quit IRC | 00:35 | |
| *** hblixt has quit IRC | 00:40 | |
| dhellmann | NobodyCam: I have a patch up to exclude the bad version | 00:44 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Update network provider spec https://review.openstack.org/187829 | 00:52 |
| jroll | devananda: ^ network spec thing | 00:52 |
| *** bethelwell has quit IRC | 00:54 | |
| *** mtanino has quit IRC | 00:56 | |
| TheJulia | goodnight folks | 01:00 |
| *** jrist has quit IRC | 01:08 | |
| *** superfly_ has joined #openstack-ironic | 01:12 | |
| *** spandhe has quit IRC | 01:13 | |
| *** superfly_ is now known as superflyy | 01:14 | |
| *** thrash is now known as thrash|g0ne | 01:14 | |
| *** jlk has left #openstack-ironic | 01:17 | |
| *** smoriya has joined #openstack-ironic | 01:18 | |
| *** jrist has joined #openstack-ironic | 01:20 | |
| *** barra204 has quit IRC | 01:24 | |
| *** shakamunyi has quit IRC | 01:24 | |
| *** superflyy has quit IRC | 01:24 | |
| *** eghobo has quit IRC | 01:33 | |
| *** ijw has quit IRC | 01:34 | |
| *** ijw has joined #openstack-ironic | 01:35 | |
| *** kan_ has joined #openstack-ironic | 01:35 | |
| openstackgerrit | Merged openstack/ironic: Update the config drive doc to replace deprecated value https://review.openstack.org/187534 | 01:37 |
| Haomeng|2 | TheJulia: night:) | 01:42 |
| *** chenglch has joined #openstack-ironic | 01:53 | |
| *** davideagnello has quit IRC | 01:58 | |
| *** yuanying has joined #openstack-ironic | 02:03 | |
| heyongli | Haomeng|2, hello | 02:14 |
| *** Haomeng|2 is now known as Haomeng | 02:15 | |
| Haomeng | heyongli: morning:) | 02:15 |
| heyongli | morning, | 02:15 |
| heyongli | Haomeng, when i play with the bifrost, in clean machine, sudo passwd for localhost is needed, it's ansible side i know, but mention this would be nice in wiki, or just tell usr, use '-K', what do you think about this. | 02:17 |
| openstackgerrit | baiyuan proposed openstack/ironic: Add documentation for getting a node's console https://review.openstack.org/176642 | 02:19 |
| Haomeng | heyongli: yes, agree with you, so can you help to update bifrost's readme? | 02:19 |
| Haomeng | heyongli: thank you | 02:19 |
| heyongli | sure | 02:19 |
| openstackgerrit | Yongli He proposed openstack/bifrost: local instll with -K is nice to user https://review.openstack.org/187841 | 02:24 |
| *** yuanying_ has joined #openstack-ironic | 02:28 | |
| *** yuanying has quit IRC | 02:31 | |
| *** achanda has joined #openstack-ironic | 02:32 | |
| Haomeng | heyongli: thanks for your patch, will help to review | 02:34 |
| heyongli | thanks, Haomeng | 02:35 |
| Haomeng | heyongli: yw:) | 02:35 |
| heyongli | play with the bifrost recently | 02:35 |
| Haomeng | heyongli: yes, it is useful to install ironic standalone to provision the bare metal, and i want to integrate with ironic-inspector and ironic web-client, then we have all-in-one standalone ironic components:) | 02:37 |
| *** dumbhead is now known as balajin | 02:37 | |
| heyongli | Haomeng, good idea, actually, i also think it value able, bm machine management, why not ironic | 02:38 |
| openstackgerrit | baiyuan proposed openstack/ironic: fix node-get-console returns url always start with http https://review.openstack.org/185523 | 02:38 |
| Haomeng | heyongli: yes, if we have more baremetal monitoring tool to be integrated into all-in-one standalone, and it is more useful and cool to manage/monitoring/using the baremetal with one-step solution | 02:39 |
| heyongli | Haomeng, we need a 'devstack' support | 02:40 |
| heyongli | i have not set it up 2 days ... -:) | 02:40 |
| heyongli | readme bypass several things, a new user run into problem, i think , could we have a bash scripts setup a demo env for developer and experience only user? | 02:41 |
| heyongli | i would like capture my steps from a fresh system, composite to a single bash for demo developer .. Haomeng | 02:45 |
| *** saripurigopi has quit IRC | 02:47 | |
| Haomeng | heyongli: yes, it is new project, and have some patch to be applied, for example the package dependency missing issue | 02:49 |
| *** ijw has quit IRC | 02:50 | |
| heyongli | i apply them already, always want a single click, -:) | 02:52 |
| Haomeng | heyongli: :) | 02:52 |
| heyongli | Haomeng, this project help a lot already, i still seek a lazy way to do all stuff, i burn a Unattended usb stick for install this fist wake up machine in stack. | 02:56 |
| openstackgerrit | Zhai, Edwin proposed openstack/ironic-specs: New meter for Node Manager https://review.openstack.org/176109 | 02:57 |
| Haomeng | heyongli: yes, we can build these into usb/iso which can run on live mode without installation, these are pre-installed by biforst already:) | 02:57 |
| openstackgerrit | Tan Lin proposed openstack/ironic-specs: Bare Metal Trust Using Intel TXT https://review.openstack.org/133902 | 02:58 |
| heyongli | Haomeng, yeah, both way are awesome, are we going to do this, i like it | 02:58 |
| Haomeng | heyongli: :) | 02:58 |
| *** achanda has quit IRC | 03:02 | |
| *** achanda has joined #openstack-ironic | 03:03 | |
| heyongli | Haomeng, do we have a way to check if we need update apt-cache? every time i try to re apply role, this is take too long time, i don't thing it needed each time apply role | 03:03 |
| openstackgerrit | Merged openstack/ironic: Do not save auth token on TFTP server in PXE driver https://review.openstack.org/159819 | 03:03 |
| Haomeng | heyongli: I understand it is working on safe mode, you can disable the task as workaround to save your time:) | 03:04 |
| heyongli | ok, reasonable | 03:04 |
| heyongli | why not we add a safe mode option? | 03:05 |
| heyongli | worth? maybe not, i don't know | 03:05 |
| Haomeng | heyongli: I understand it is better to update cache first to avoid some package version issue | 03:06 |
| *** achanda has quit IRC | 03:08 | |
| heyongli | safemode by default, it is. | 03:08 |
| Haomeng | yes | 03:08 |
| *** yuanying has joined #openstack-ironic | 03:15 | |
| Haomeng | heyongli: I think no need to add *safe mode* option:) | 03:15 |
| *** yuanying_ has quit IRC | 03:16 | |
| *** yuanying has quit IRC | 03:22 | |
| *** [1]cdearborn has quit IRC | 03:31 | |
| *** achanda has joined #openstack-ironic | 03:34 | |
| *** spandhe has joined #openstack-ironic | 03:36 | |
| *** spandhe_ has joined #openstack-ironic | 03:42 | |
| *** spandhe has quit IRC | 03:43 | |
| *** spandhe_ is now known as spandhe | 03:43 | |
| *** yuanying has joined #openstack-ironic | 03:52 | |
| *** alexpilotti has joined #openstack-ironic | 03:55 | |
| *** achanda has quit IRC | 03:58 | |
| *** achanda has joined #openstack-ironic | 03:58 | |
| *** achanda has quit IRC | 04:03 | |
| *** saripurigopi has joined #openstack-ironic | 04:04 | |
| *** yuanying has quit IRC | 04:10 | |
| *** yuanying has joined #openstack-ironic | 04:12 | |
| *** yuanying has quit IRC | 04:18 | |
| *** yuanying has joined #openstack-ironic | 04:19 | |
| *** yuanying has quit IRC | 04:20 | |
| *** yuanying_ has joined #openstack-ironic | 04:24 | |
| *** yuanying_ has quit IRC | 04:25 | |
| *** yuanying has joined #openstack-ironic | 04:27 | |
| *** coolsvap|afk is now known as coolsvap | 04:30 | |
| *** coolsvap is now known as coolsvap|afk | 04:31 | |
| *** coolsvap|afk is now known as coolsvap | 04:32 | |
| *** krtaylor has quit IRC | 04:40 | |
| *** spandhe has quit IRC | 04:41 | |
| *** spandhe has joined #openstack-ironic | 04:45 | |
| *** yuanying has quit IRC | 04:45 | |
| *** yuanying has joined #openstack-ironic | 04:48 | |
| *** saripurigopi has quit IRC | 04:52 | |
| *** saripurigopi has joined #openstack-ironic | 04:52 | |
| openstackgerrit | baiyuan proposed openstack/ironic: fix node-get-console returns url always start with http https://review.openstack.org/185523 | 04:59 |
| heyongli | Haomeng, yeah, kind of overshooting | 05:14 |
| heyongli | Haomeng, downloading OS image failed 'time out' , but wget directly is definitely ok, why? any clue? | 05:18 |
| *** saripurigopi has quit IRC | 05:23 | |
| *** krtaylor has joined #openstack-ironic | 05:27 | |
| Haomeng | heyongli: did you access Internet via http proxy? | 05:42 |
| openstackgerrit | Feng Xi Yan proposed openstack/ironic: Add documentation for getting a node's console https://review.openstack.org/176642 | 05:42 |
| heyongli | global , redsocks, may be it no stable, will timeout sometime, i try downloading to /httpboot/ directly. | 05:43 |
| Haomeng | heyongli: bifrost does not support proxy now | 05:44 |
| heyongli | redsocks is transparent to any app | 05:44 |
| heyongli | maybe just unstable, -:) | 05:44 |
| Haomeng | heyongli: yes, it is not proxy, should work | 05:44 |
| heyongli | it's ok now, thanks | 05:49 |
| Haomeng | heyongli: welcome:) | 05:50 |
| *** yuanying has quit IRC | 05:51 | |
| *** Shrews has quit IRC | 05:51 | |
| *** rameshg87 has joined #openstack-ironic | 05:52 | |
| rameshg87 | good morning ironic | 05:52 |
| heyongli | those ansible will hide some error, i see a proposal to keep log, right? | 05:52 |
| *** dguerri`away has quit IRC | 05:55 | |
| *** yuikotakada has joined #openstack-ironic | 05:55 | |
| *** Shrews has joined #openstack-ironic | 05:56 | |
| *** dguerri`away has joined #openstack-ironic | 05:57 | |
| *** dguerri`away is now known as dguerri | 05:57 | |
| *** kan_ has quit IRC | 06:05 | |
| *** radek_ has joined #openstack-ironic | 06:07 | |
| Haomeng | rameshg87: morning Ramesh:) | 06:10 |
| *** kan_ has joined #openstack-ironic | 06:10 | |
| Haomeng | heyongli: yes, should redirect the output message including both stand output and error to log file | 06:11 |
| *** romainh has joined #openstack-ironic | 06:11 | |
| rameshg87 | Haomeng: o/ | 06:14 |
| Haomeng | rameshg87: :) | 06:14 |
| *** shakamunyi has joined #openstack-ironic | 06:16 | |
| *** barra204 has joined #openstack-ironic | 06:17 | |
| *** ukalifon1 has joined #openstack-ironic | 06:23 | |
| openstackgerrit | Haomeng,Wang proposed stackforge/ironic-discoverd: Supports standalone ironic https://review.openstack.org/187878 | 06:31 |
| openstackgerrit | Haomeng,Wang proposed stackforge/ironic-discoverd: Supports standalone ironic https://review.openstack.org/187878 | 06:32 |
| openstackgerrit | Haomeng,Wang proposed stackforge/ironic-discoverd: Supports standalone ironic https://review.openstack.org/187878 | 06:35 |
| *** yuanying has joined #openstack-ironic | 06:37 | |
| yuikotakada | Haomeng: Hi :) thank you for posting the patch. You can generate .conf file by executing "tox -egenconfig" | 06:38 |
| Haomeng | yuikotakada: got, thanks for your infomation:) | 06:38 |
| Haomeng | yuikotakada: this can generate example.conf, right? | 06:38 |
| yuikotakada | Haomeng: right :) | 06:39 |
| Haomeng | yuikotakada: ok, will try, thanks:) | 06:39 |
| openstackgerrit | Tan Lin proposed openstack/ironic-specs: Refactor Ironic's versioned objects to oslo.versionedobjects https://review.openstack.org/187879 | 06:42 |
| yuikotakada | Haomeng: your're welcome. I'm looking forward your new patch set. | 06:45 |
| Haomeng | yuikotakada: you are working on standalone ironic recently, right? | 06:45 |
| Haomeng | yuikotakada: :) | 06:45 |
| yuikotakada | Haomeng: No. But supporting standalone ironic is fine ;) And you? | 06:47 |
| Haomeng | yuikotakada: yes, it can run out-of-openstack, can do some deployment on *under-cloud* level | 06:48 |
| openstackgerrit | Haomeng,Wang proposed stackforge/ironic-discoverd: Supports standalone ironic https://review.openstack.org/187878 | 06:48 |
| Haomeng | yuikotakada: regenerated the example.conf, welcome your comments:) | 06:48 |
| yuikotakada | Haomeng: I don't know about standalone ironic well, but isn't it inconvenient without via-nova? If I remember correctly, we cannot know the IP address of baremetal node just using ironic, right? | 06:52 |
| yuikotakada | Haomeng: thank you for updating patch. I'll check. | 06:52 |
| Haomeng | yuikotakada: yes, standalone ironic will kickoff the baremetal provisioning without nova boot, can be used in non-openstack env | 06:53 |
| Haomeng | yuikotakada: ok, thank you | 06:55 |
| *** pradipta has joined #openstack-ironic | 06:55 | |
| *** yuikotakada has quit IRC | 06:56 | |
| *** spandhe has quit IRC | 06:57 | |
| *** yuikotakada has joined #openstack-ironic | 06:59 | |
| *** ifarkas has joined #openstack-ironic | 07:00 | |
| yuikotakada | Haomeng: so that just installing OS is your purpose? | 07:01 |
| yuikotakada | like just creating Undercloud. | 07:01 |
| Haomeng | yuikotakada: yes, install os and manage the baremetal | 07:01 |
| *** radek_ has quit IRC | 07:02 | |
| Haomeng | yuikotakada: we can run all-in-one standalone ironic with ironic+inspector+webclient and more hardware management/monitoring tool to manage and use baremetal, maybe we can create usb drive or iso live media to run them directly without installation | 07:02 |
| yuikotakada | Haomeng: gotcha. Interesting use case. | 07:04 |
| Haomeng | yuikotakada: :) | 07:04 |
| *** radek_ has joined #openstack-ironic | 07:05 | |
| *** athomas has joined #openstack-ironic | 07:14 | |
| *** radek__ has joined #openstack-ironic | 07:18 | |
| *** radek_ has quit IRC | 07:20 | |
| *** bradjones has quit IRC | 07:20 | |
| *** dtantsur|afk is now known as dtantsur | 07:21 | |
| dtantsur | Morning! | 07:21 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: When boot option is not persisted, set boot on next power on https://review.openstack.org/177642 | 07:22 |
| *** bradjones has joined #openstack-ironic | 07:22 | |
| *** yuanying has quit IRC | 07:22 | |
| dtantsur | Haomeng, oh, sounds interesting | 07:23 |
| * dtantsur will look at the patch soon | 07:23 | |
| yuikotakada | dtantsur: o/ | 07:25 |
| dtantsur | yuikotakada, o/ | 07:25 |
| openstackgerrit | Zhai, Edwin proposed openstack/ironic-specs: New meter for Node Manager https://review.openstack.org/176109 | 07:33 |
| openstackgerrit | Michael Davies proposed openstack/python-ironicclient: Cache negotiated api microversion for server https://review.openstack.org/173674 | 07:33 |
| *** radek_ has joined #openstack-ironic | 07:35 | |
| *** chlong has quit IRC | 07:37 | |
| *** radek__ has quit IRC | 07:37 | |
| *** coolsvap is now known as coolsvap|afk | 07:41 | |
| *** ifarkas has quit IRC | 07:43 | |
| *** jcoufal has joined #openstack-ironic | 07:45 | |
| *** coolsvap|afk is now known as coolsvap | 07:46 | |
| dtantsur | yuikotakada, maybe you already know, but our rename discoverd -> inspector was approved by the TC yesterday | 07:50 |
| dtantsur | \o/ | 07:50 |
| dtantsur | so I'm going to create infra patches today | 07:50 |
| dtantsur | and ask for launchpad renaming | 07:50 |
| *** rameshg87 is now known as rameshg87-lunch | 07:50 | |
| yuikotakada | dtantsur: Wao! I didn't know, Congrats!! | 07:50 |
| *** jistr has joined #openstack-ironic | 07:51 | |
| yuikotakada | dtantsur: there are some tasks we need to do... 1 is as you mentioned, launchpad renaming, and changing diskimage-builder's element name is also necessary, right? and...what? We also need to move files under ironic_inspector_ramdisk, but it seems enough later | 07:56 |
| dtantsur | yuikotakada, I guess with move to IPA we'll just deprecated ironic-discoverd-ramdisk element, so I don't think we should rename it.. | 07:56 |
| dtantsur | and (hopefully) files under ironic_inspector_ramdisk will move to IPA directly | 07:57 |
| *** romcheg has joined #openstack-ironic | 07:59 | |
| yuikotakada | dtantsur: will ironic-discoverd-ramdisk element be deprecated? although the shapes are different? so that, diskimage-builder's is shell script, but ironic_inspector_ramdisk is python script. Isn't there any person who want to use simple script not python? It's just a question, I don't understand which kind of usecases there are. | 08:00 |
| *** ndipanov has joined #openstack-ironic | 08:00 | |
| dtantsur | yuikotakada, that goes in line with Ironic willing to deprecate old bash-based ramdisk. while there may be use cases, it's too much a burden to support 2 ramdisks | 08:01 |
| yuikotakada | dtantsur: OK, I didn't know that. I understand. | 08:02 |
| *** ifarkas has joined #openstack-ironic | 08:03 | |
| *** pas-ha has joined #openstack-ironic | 08:03 | |
| openstackgerrit | Tan Lin proposed openstack/ironic-specs: Bare Metal Trust Using Intel TXT https://review.openstack.org/133902 | 08:04 |
| *** MattMan has joined #openstack-ironic | 08:13 | |
| *** lucasagomes has joined #openstack-ironic | 08:14 | |
| *** derekh has joined #openstack-ironic | 08:20 | |
| openstackgerrit | Yuiko Takada proposed stackforge/ironic-discoverd: Use keystonemiddleware auth credentials https://review.openstack.org/170782 | 08:23 |
| *** spandhe has joined #openstack-ironic | 08:31 | |
| *** heyongli has quit IRC | 08:34 | |
| *** e0ne has joined #openstack-ironic | 08:46 | |
| dtantsur | yuikotakada, launchpad rename is done. I will propose a patch to fix links | 08:47 |
| *** heyongli has joined #openstack-ironic | 08:49 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Validate capability in properties and instance_info https://review.openstack.org/174785 | 08:49 |
| yuikotakada | dtantsur: Thanks! I've checked it. Do we need rename github also? or will be changed automatically? | 08:50 |
| dtantsur | yuikotakada, I will propose patch to infra for that. It's a manual action and it will require some time | 08:50 |
| *** heyongli has quit IRC | 08:51 | |
| *** pelix has joined #openstack-ironic | 08:52 | |
| yuikotakada | dtantsur, OK, I see. Are there something I can help you? | 08:52 |
| dtantsur | yuikotakada, thanks, I will ping you for various reviews | 08:52 |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Launchpad project was renamed https://review.openstack.org/187914 | 08:53 |
| yuikotakada | dtantsur, sure :) | 08:53 |
| dtantsur | e.g. this one ^^^ | 08:53 |
| *** edcragg has joined #openstack-ironic | 08:54 | |
| yuikotakada | will pypi also be renamed? | 08:55 |
| dtantsur | yuikotakada, I will create a new project later on, so that people can download old versions under old name | 08:56 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic-specs: Add nodes tagging support https://review.openstack.org/183971 | 08:57 |
| yuikotakada | dtantsur, OK , pypi.python.org/pypi/ironic-inspector, right? So that we need to write new & old URLs into README later. thank you. I'm learning a lot of things about project :) | 08:59 |
| *** e0ne is now known as e0ne_ | 09:01 | |
| dtantsur | yvw :) | 09:01 |
| *** e0ne_ is now known as e0ne | 09:01 | |
| openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/187679 | 09:03 |
| * dtantsur brb | 09:03 | |
| sambetts | Good (UGT) Morning o/ | 09:03 |
| openstackgerrit | Merged openstack/ironic: Update iLO documentation for UEFI secure boot https://review.openstack.org/178582 | 09:03 |
| pshige | sambetts: morining :) | 09:04 |
| *** heyongli has joined #openstack-ironic | 09:06 | |
| sambetts | pshige: o/ | 09:07 |
| *** heyongli has quit IRC | 09:07 | |
| *** ifarkas has quit IRC | 09:16 | |
| *** yuikotakada has quit IRC | 09:17 | |
| *** yuikotakada has joined #openstack-ironic | 09:20 | |
| openstackgerrit | Merged openstack/ironic: Revert "Add simplegeneric to py34 requirements" https://review.openstack.org/186208 | 09:24 |
| yuikotakada | sambetts: o/ | 09:25 |
| sambetts | yuikotakada: Just replying to your comments on my ironic-inspector patch :) | 09:26 |
| yuikotakada | sambetts: Thanks :D | 09:27 |
| *** lazy_prince has joined #openstack-ironic | 09:28 | |
| yuikotakada | I'll leave here for dinner a couple of hours ;) | 09:29 |
| *** yuikotakada has quit IRC | 09:29 | |
| *** ifarkas has joined #openstack-ironic | 09:35 | |
| openstackgerrit | Merged openstack/ironic: fix node-get-console returns url always start with http https://review.openstack.org/185523 | 09:38 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Remove auth token saving from iLO driver https://review.openstack.org/187924 | 09:48 |
| *** pradipta has quit IRC | 09:48 | |
| *** erelwell has joined #openstack-ironic | 09:52 | |
| *** erelwell is now known as bethelwell | 09:52 | |
| *** e0ne is now known as e0ne_ | 09:55 | |
| *** e0ne_ is now known as e0ne | 09:57 | |
| *** lazy_prince has quit IRC | 10:22 | |
| *** spandhe has quit IRC | 10:23 | |
| *** romcheg has quit IRC | 10:35 | |
| *** romcheg has joined #openstack-ironic | 10:36 | |
| dtantsur | sambetts, I will try to find where this assertEqual ordering tradition comes from... | 10:40 |
| dtantsur | I'm pretty sure we follow it in Ironic itself | 10:40 |
| sambetts | dtantsur: I've seen it throughout OpenStack and I really don't know why, to be personally it seems backwards :-P | 10:41 |
| sambetts | s/to be/to me/ | 10:41 |
| dtantsur | I think we should follow it at least for consistency, but yeah, it's interesting, why | 10:42 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E123/6/7/8 in ironic/api https://review.openstack.org/186458 | 10:43 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E123/6/7/8 in ironic/common https://review.openstack.org/186456 | 10:43 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E123/6/7/8 in ironic /cmd /db and /dhcp https://review.openstack.org/186457 | 10:43 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E123/6/7/8 for ironic/tests/drivers https://review.openstack.org/186450 | 10:43 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E123/6/7/8 in non-driver ironic/tests https://review.openstack.org/186451 | 10:43 |
| openstackgerrit | Merged stackforge/ironic-discoverd: Launchpad project was renamed https://review.openstack.org/187914 | 10:43 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Fixes flake8 E123/6/7/8 for ironic/drivers https://review.openstack.org/186454 | 10:43 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Remove E123/6/7/8 from flake8 ignore list https://review.openstack.org/186021 | 10:43 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Fixes flake8 E123/6/7/8 in ironic/conductor https://review.openstack.org/186455 | 10:43 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E123/6/7/8 for ironic/tests/api https://review.openstack.org/186452 | 10:43 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Fixes flake8 E123/6/7/8 for ironic/tests/conductor https://review.openstack.org/186453 | 10:43 |
| sambetts | Sorry for the spam | 10:43 |
| sambetts | dtantsur: Consistancy in the tests in OpenStack yes, but not with the rest of programming :-P ah well | 10:45 |
| dtantsur | right :) but we're part of openstack now | 10:45 |
| sambetts | dtantsur: Yeah, interestingly after doing a quick search for the reasoning behind it, the expected, actual order is the xUnit/sUnit frameworks used, and it seems to have stuck | 10:50 |
| dtantsur | interesting | 10:51 |
| sambetts | dtantsur: theres a lot of people online asking the question why that order, its weird that the python doc examples are "wrong" and they don't put a preference on which should be first or second :-P | 10:53 |
| dtantsur | yeah :) any fixed order will be good, because it makes tests so easier to read | 10:53 |
| pshige | :) | 10:54 |
| sambetts | dtantsur: what do you think about Yuiko's comment about combining the tests? Thats all I've got left to do before pushing a new patch set for that | 10:57 |
| *** romcheg has quit IRC | 10:57 | |
| * dtantsur is looking again | 10:58 | |
| openstackgerrit | chenglch proposed openstack/ironic-specs: Add petitboot iscsi deploy driver support https://review.openstack.org/177641 | 11:00 |
| *** chenglch has quit IRC | 11:00 | |
| pshige | Now a tempest patch for Ironic got +A :) | 11:01 |
| dtantsur | sambetts, left a comment + found one bug in tests | 11:02 |
| dtantsur | pshige, cool, which one? | 11:02 |
| pshige | it took about two months ... | 11:02 |
| pshige | https://review.openstack.org/#/c/173061/ | 11:03 |
| sambetts | dtantsur: Thanks :D | 11:04 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Use oslo config import methods for Keystone options https://review.openstack.org/187944 | 11:09 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: WIP: Support for disabling DHCP server configuration https://review.openstack.org/187362 | 11:13 |
| *** lucasagomes has quit IRC | 11:18 | |
| openstackgerrit | Peter Martini proposed openstack/bifrost: Typo fix (disk imae builder) https://review.openstack.org/187949 | 11:25 |
| *** e0ne is now known as e0ne_ | 11:32 | |
| *** Haomeng|2 has joined #openstack-ironic | 11:35 | |
| *** Haomeng has quit IRC | 11:38 | |
| *** trown|outttypeww is now known as trown | 11:40 | |
| *** e0ne_ has quit IRC | 11:42 | |
| * rameshg87-lunch goes home | 11:42 | |
| *** rameshg87-lunch has quit IRC | 11:42 | |
| *** romcheg has joined #openstack-ironic | 11:43 | |
| openstackgerrit | Merged openstack/ironic: Prevent node delete based on provision, not power, state https://review.openstack.org/186361 | 11:44 |
| sambetts | dtantsur: I added autospec=True to the mock.patch in my tests as you suggested, but having autospec=True breaks the py34 tests for some reason :( | 11:46 |
| TheJulia | good morning! | 11:48 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Update test-birfrost to limit log lines https://review.openstack.org/187703 | 11:49 |
| dtantsur | sambetts, probably this "some reason" should be fixed ;) what's the failure? | 11:50 |
| dtantsur | TheJulia, morning | 11:50 |
| sambetts | dtantsur: its an exception from mock itself when trying to patch it with autospec, py27 passes fine, http://paste.openstack.org/show/259168/ | 11:53 |
| *** dprince has joined #openstack-ironic | 11:54 | |
| dtantsur | oh yeah, mock is broken under Python 3.4 | 11:54 |
| dtantsur | sambetts, there's a workaround: https://github.com/stackforge/ironic-discoverd/blob/master/ironic_inspector_ramdisk/test/test_discover.py#L24-L28 | 11:55 |
| sambetts | dtantsur: ah, nice! | 11:57 |
| sambetts | dtantsur: well sort of XD | 11:58 |
| sambetts | dtantsur: as nice as a hack can be :-P | 11:58 |
| *** kan_ has quit IRC | 11:58 | |
| *** e0ne has joined #openstack-ironic | 11:58 | |
| *** lucasagomes has joined #openstack-ironic | 11:59 | |
| *** yuikotakada has joined #openstack-ironic | 12:03 | |
| *** lucasagomes is now known as lucas-hungry | 12:05 | |
| *** thrash|g0ne is now known as thrash | 12:05 | |
| *** derekh has quit IRC | 12:10 | |
| openstackgerrit | Sam Betts proposed stackforge/ironic-discoverd: Make the number of green threads configurable https://review.openstack.org/187723 | 12:11 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Support for disabling DHCP server configuration https://review.openstack.org/187362 | 12:21 |
| *** [1]cdearborn has joined #openstack-ironic | 12:24 | |
| *** athomas has quit IRC | 12:36 | |
| openstackgerrit | Pratyusha proposed stackforge/proliantutils: Add iSCSI boot support in RIS https://review.openstack.org/187162 | 12:36 |
| *** yuanying has joined #openstack-ironic | 12:38 | |
| openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Implement OEM Inventory components https://review.openstack.org/187592 | 12:39 |
| *** yuanying has quit IRC | 12:46 | |
| *** athomas has joined #openstack-ironic | 12:47 | |
| *** jcoufal has quit IRC | 12:52 | |
| *** rwsu has joined #openstack-ironic | 12:54 | |
| *** smoriya has quit IRC | 12:57 | |
| *** jistr is now known as jistr|mtg | 12:57 | |
| *** derekh has joined #openstack-ironic | 12:58 | |
| dtantsur | gertty is doing loooooong database migration after update... going for some tea, brb | 12:58 |
| MattMan | Can one call the inspection interface on a driver via ironicclient CLI ? | 13:01 |
| *** romcheg has quit IRC | 13:02 | |
| *** lucas-hungry is now known as lucasagomes | 13:08 | |
| lucasagomes | MattMan, I think that is done by setting the provision state to inspect? /me checks | 13:08 |
| *** rloo has joined #openstack-ironic | 13:10 | |
| lucasagomes | MattMan, yeah, ironic node-set-provision-state $node_uuid inspect | 13:10 |
| lucasagomes | now, the node should be moved to the "manageable" state first, so it's not exposed to the Nova scheduler while instrospecting | 13:11 |
| lucasagomes | ironic node-set-provision-state $node_uuid manage | 13:11 |
| lucasagomes | ironic node-set-provision-state $node_uuid inspect | 13:11 |
| lucasagomes | ironic node-set-provision-state $node_uuid provide (so it goes back to a state where it can be seen by the nova scheduler) | 13:12 |
| MattMan | lucasagomes: thanks... will give it a whirl... just testing my kilo port of solaris driver... next step port to git tip and then will attempt to write up a spec | 13:13 |
| lucasagomes | MattMan, cool | 13:13 |
| lucasagomes | MattMan, solaris driver? sparc? | 13:14 |
| MattMan | sparc & x86 | 13:14 |
| MattMan | Solaris specific in that it uses Solaris deployment tools for actual provisioning | 13:14 |
| lucasagomes | nice | 13:15 |
| MattMan | In both cases utilizes ipmitool to configure for network boot (pxe for x86 and wanboot for sparc) | 13:15 |
| MattMan | Hopefully when I get to writing up the spec all will be clear... or maybe not :) | 13:16 |
| lucasagomes | heh you will | 13:16 |
| lucasagomes | this still gotta be fixed right? https://bugs.launchpad.net/ironic/+bug/1391598 | 13:16 |
| openstack | Launchpad bug 1391598 in Ironic "get_supported_boot_devices() returns static device list" [Wishlist,Confirmed] | 13:16 |
| MattMan | yes please | 13:16 |
| MattMan | optional task=None I hope would be OK | 13:17 |
| lucasagomes | +1 | 13:19 |
| *** cdearborn has joined #openstack-ironic | 13:21 | |
| MattMan | I'll attempt to get a gerrit review up and going | 13:23 |
| MattMan | Will be my first one :) | 13:24 |
| lucasagomes | MattMan, cool, if u want mark me as a reviwer I will glad review it | 13:24 |
| MattMan | thanks will do | 13:24 |
| MattMan | For kilo is 0.5.1 recommended version of ironicclient ? | 13:26 |
| *** jrist has quit IRC | 13:26 | |
| openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Implement OEM Inventory components https://review.openstack.org/187592 | 13:27 |
| lucasagomes | I think you can use the last version that was tagged recently | 13:28 |
| lucasagomes | I don't think we have added anything incompatible since the kilo release | 13:28 |
| MattMan | So 0.7.0 | 13:29 |
| openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Implement OEM Inventory components https://review.openstack.org/187592 | 13:29 |
| rloo | hello ironickers, lucasagomes, MattMan | 13:30 |
| rloo | hey dtantsur, let me know when you want to discuss enroll | 13:31 |
| *** cui has joined #openstack-ironic | 13:32 | |
| dtantsur | rloo, hey, morning! right now will work :) | 13:32 |
| dtantsur | otherwise I will probably by away in 1.5 hrs | 13:32 |
| rloo | dtantsur: ok, give me 30 sec to pull up your spec | 13:32 |
| dtantsur | sure thing | 13:32 |
| rloo | dtantsur: wrt the ordering (your comment) | 13:33 |
| openstackgerrit | Sam Betts proposed stackforge/ironic-discoverd: Make the number of green threads configurable https://review.openstack.org/187723 | 13:33 |
| *** jcoufal has joined #openstack-ironic | 13:33 | |
| rloo | dtantsur: 1. the minor version of client only has warning, or we've up'd the version to the new version? | 13:33 |
| dtantsur | rloo, only warning | 13:34 |
| dtantsur | the whole point of warning is to warn (hmm) people before we break them, not after | 13:34 |
| dtantsur | btw that was proposal by sambetts IIRC | 13:34 |
| rloo | dtantsur: so the client doesn't know about the new version. the only way someone can get it is to explicitly specify the new version or higher | 13:34 |
| dtantsur | rloo, yep | 13:34 |
| rloo | dtantsur: what happens if a new feature/version is added to ironic | 13:34 |
| rloo | dtantsur: the idea was that whenever a microversion was added to server, we up the client so it is supported | 13:35 |
| dtantsur | rloo, right, one day we up it | 13:35 |
| rloo | dtantsur: it means that 'latest' doesn't represent the latest on the server | 13:35 |
| dtantsur | do we have any "latest" support on client | 13:35 |
| * dtantsur does not remember | 13:36 | |
| *** [1]cdearborn has quit IRC | 13:36 | |
| rloo | dtantsur: so if we land your enroll stuff, and the next day we land featureX and clientX is up'd to have featureXversion, that is OK? | 13:36 |
| dtantsur | rloo, well, with major version bump and after some time with warning | 13:36 |
| rloo | dtantsur: yeah, version 'latest' == client uses the version that it knows about | 13:36 |
| rloo | dtantsur: my concern is that we 'delay' or wahtever other features cuz of the enroll feature | 13:37 |
| dtantsur | rloo, tbh I don't see how we can avoid it | 13:37 |
| rloo | dtantsur: 'delay' -- in the client, which is inconsistent with what I *thought*/how it was meant to work | 13:37 |
| dtantsur | but yeah, we will. if people want them, they should bump microversion right now | 13:38 |
| dtantsur | I think that's how microversions are supposed to work at all | 13:38 |
| rloo | dtantsur: you lost me there | 13:38 |
| rloo | dtantsur: if people want what? | 13:38 |
| rloo | dtantsur: you mean people should explicitly specify the version they want? | 13:39 |
| dtantsur | enroll change? well, people do | 13:39 |
| *** jrist has joined #openstack-ironic | 13:39 | |
| dtantsur | rloo, the same thing should happen every time we bump a microversion | 13:39 |
| openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Implement OEM Inventory components https://review.openstack.org/187592 | 13:39 |
| rloo | dtantsur: what 'same thing'? | 13:39 |
| dtantsur | rloo, everything we discuss right now | 13:39 |
| dtantsur | it didn't happen only because only once we actually bumped microversion due to a breaking change | 13:40 |
| dtantsur | other "bumps" were due to us wanting for every change, even non-breaking | 13:40 |
| lucasagomes | rloo, hi there! | 13:40 |
| rloo | dtantsur: ahh. hmm. I don't know that we've documented how microversioning should work so voila, confusion reigns. | 13:41 |
| sambetts | rloo, dtantsur: could you release an rc/beta version of the client that does understand the new features, so if they want to use them they have to explictly upgrade their client? the client stable would stay as is, until after a deprecation period then the new one becomes the "stable" | 13:41 |
| dtantsur | sambetts, every client understands the feature | 13:41 |
| *** e0ne is now known as e0ne_ | 13:41 | |
| dtantsur | IIRC we don't have notion of "supported feature" for a client | 13:41 |
| rloo | sambetts: yes and no. i mean yes we could do what we want but no, i think we need to step back | 13:41 |
| *** e0ne_ is now known as e0ne | 13:41 | |
| rloo | dtantsur, sambetts: I think we need to think about/document how microversioning will/should work, so we don't go through this with every feature that isn't backwards compatible | 13:42 |
| cui | Hi,all: can i enable bm's VLAN Networking feature by using any neutron-xxx-plugins? | 13:42 |
| dtantsur | rloo, I think we did it last cycle | 13:42 |
| dtantsur | rloo, anyway, I'm open to better ideas how to implement ENROLL | 13:43 |
| rloo | dtantsur: uhhh, well, let's say it certainly isn't clear that we three have the same idea on how it should work. so something is wrong (not communicating?) | 13:43 |
| *** dan_ has joined #openstack-ironic | 13:43 | |
| dtantsur | I'm open to other ideas ;) | 13:43 |
| *** dan_ is now known as Guest83679 | 13:43 | |
| rloo | dtantsur, sambetts: let me see if we have anything documented, might have been irc chats... | 13:44 |
| openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Implement OEM Inventory components https://review.openstack.org/187592 | 13:44 |
| openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Implement OEM Inventory components https://review.openstack.org/187592 | 13:45 |
| rloo | dtantsur: basically, I think your proposal is overly complicated. but i need to step back to think about what we discussed/decided about microversions. | 13:45 |
| dtantsur | rloo, your suggestions? | 13:45 |
| rloo | dtantsur: based on my understanding of the microversioning, I would have thought: 1. make change to server; 2. make change to client to set version to new version | 13:46 |
| dtantsur | rloo, so the only difference is whether to put a warning? | 13:46 |
| rloo | dtantsur: but again, that's based on my understanding on how it was supposed to work. my understanding could be wrong cuz it doesn't seem to jive with yours or sambetts | 13:47 |
| dtantsur | I'm fine with not putting a warning at all, anyway not everyone will see it | 13:47 |
| rloo | dtantsur: not just the warning but the whole major/minor stuff | 13:47 |
| openstackgerrit | Merged openstack/ironic: Remove bogus conditional from node_update https://review.openstack.org/186386 | 13:47 |
| *** romcheg has joined #openstack-ironic | 13:47 | |
| dtantsur | rloo, your suggest us stopping following semantic versioning? | 13:47 |
| dtantsur | otherwise we have to bump major version with every breaking change | 13:47 |
| rloo | dtantsur: so i haven't grok'd the diff between the microversions and semantic versioning. | 13:48 |
| dtantsur | they're orthogonal | 13:48 |
| dtantsur | when you break your clients/API, you bump the major version - that's the rule of semantic versioning | 13:48 |
| rloo | dtantsur: let me get back in 30 min or so. I need to refresh my memory on this microversioning stuff. | 13:49 |
| dtantsur | so if python-ironicclient 1.0.0 worked for people, and then we break it, we should issue python-ironicclient 2.0.0 | 13:49 |
| dtantsur | not 1.1.0 or 1.0.1 | 13:49 |
| dtantsur | actually, it's kind of technical detail here | 13:49 |
| rloo | dtantsur: did we break it when we went from nostate -> available? | 13:50 |
| dtantsur | rloo, we did break people, and we didn't bump the major version, mostly because of 0.x (versions starting with 0 are considered all unstable) | 13:50 |
| dtantsur | but having version 0.x for a mature client is at best weird | 13:51 |
| dtantsur | so we can have bump to 1.0.0 with this enroll change, for example | 13:51 |
| dtantsur | tl;dr: the only reason I ask for bumping a major version is because it tells people that we broke something seriously | 13:52 |
| NobodyCam | Good morning (UGT) Ironic | 13:52 |
| rloo | dtantsur: I'm not against a major version -- I'd like us (or me) to understand when we do minor/major versions so that going forward we're all on the same train | 13:52 |
| BadCub_Hiding | morning folks | 13:52 |
| dtantsur | morning, NobodyCam, morning, BadCub_Hiding, stop hiding :) | 13:53 |
| sambetts | Good UGT morning NobodyCam :) | 13:53 |
| dtantsur | rloo, for client versions can't we just follow semantic versioning? | 13:53 |
| dtantsur | (I think we actually do, just as other openstack clients) | 13:53 |
| NobodyCam | mornign dtantsur sambetts rloo lucasagomes and BadCub_Hiding | 13:54 |
| rloo | dtantsur: I am wondering what nova does | 13:54 |
| rloo | dtantsur: not that I like copying nova all the time but they started this | 13:54 |
| *** BadCub_Hiding is now known as BadCub | 13:54 | |
| rloo | morning NobodyCam, BadCub_Hiding, sambetts | 13:55 |
| BadCub | morning dtantsur NobodyCam | 13:55 |
| BadCub | morning rloo | 13:55 |
| sambetts | rloo, dtantsur: swift might be one to look at too | 13:55 |
| trown | morning NobodyCam BadCub sambetts dtantsur rloo | 13:55 |
| rloo | sambetts: does swift use microversions? | 13:56 |
| dtantsur | microversions are unrelated to the current question | 13:56 |
| BadCub | morning trown | 13:56 |
| sambetts | they use SemVer for both client and server | 13:56 |
| rloo | dtantsur: disagree, not totally unrelated | 13:56 |
| dtantsur | no, totally | 13:56 |
| dtantsur | it's our implementation detail | 13:56 |
| pshige | trown: morning :) | 13:57 |
| dtantsur | trown, morning | 13:57 |
| dtantsur | rloo, from the user's point of view, it does not matter how we implement compatibility. what does matter if a specific release breaks it or not | 13:58 |
| dtantsur | rloo, some people use semvers for their software to signal it | 13:58 |
| *** jistr|mtg is now known as jistr | 13:59 | |
| rloo | dtantsur: so I don't really care that much about what we do, as long as we agree on how we do it. | 13:59 |
| rloo | dtantsur: well, not sure if that made sense | 13:59 |
| dtantsur | that's what we're trying to do now - to agree :) | 13:59 |
| rloo | dtantsur: but i don't think it is up to the two of us to agree | 14:00 |
| *** coolsvap is now known as coolsvap|afk | 14:00 | |
| dtantsur | rloo, a couple of days ago I was assuming we agreed on it on the summit ;) | 14:00 |
| rloo | dtantsur: I don't recall that discussion :-( | 14:00 |
| rloo | dtantsur: was I there? | 14:00 |
| dtantsur | rloo, I believe so. State machine working session | 14:00 |
| rloo | dtantsur: we didn't have a state machine working session? | 14:01 |
| *** cui has quit IRC | 14:01 | |
| sambetts | dtantsur, rloo | 14:01 |
| rloo | dtantsur: or maybe it was hijacked | 14:01 |
| NobodyCam | good (UGT) morning trown pshige | 14:01 |
| dtantsur | rloo, https://etherpad.openstack.org/p/liberty-ironic-state-machine-next-steps | 14:01 |
| pshige | NobodyCam: morning :) | 14:01 |
| sambetts | :-P was about to link to that | 14:01 |
| rloo | dtantsur: hard to know how much of this was at the summit vs people adding notes but regardless -- i don't see much under enroll :-( | 14:02 |
| dtantsur | rloo, "Change default node provision state" section. it's kind of brief | 14:02 |
| sambetts | rloo: its lines 34 - 42 | 14:03 |
| *** dprince has quit IRC | 14:03 | |
| *** dprince has joined #openstack-ironic | 14:03 | |
| sambetts | its doesn't look like there was much more decided past ensure devstack doesn't implode | 14:03 |
| jlvillal | lucasagomes, I added https://review.openstack.org/#/c/176890/ to the Nova review priority list. Hopefully they will review it :) | 14:04 |
| dtantsur | sambetts, that's why we're discussing the spec right now :) | 14:04 |
| rloo | dtantsur, sambetts: guess i don't recall this cuz I am thinking now that the etherpad wasn't being projected so I didn't actually see the etherpad. | 14:04 |
| dtantsur | yeah, lack of projectors was a failure | 14:04 |
| rloo | dtantsur, sambetts: ok, so there is nothing here about a major release of the client | 14:04 |
| dtantsur | rloo, if we follow semver, it's obvious | 14:05 |
| rloo | dtantsur, sambetts: my interpretation of 'version' here is the microversion | 14:05 |
| rloo | dtantsur, sambetts: I think your interpretation of 'version' is different | 14:05 |
| dtantsur | I don't think so | 14:05 |
| dtantsur | this etherpad does not speak about client version; assuming we do use semver, it's an obvious question | 14:06 |
| cdearborn | dtantsur, trown, wanted to let you know that the discovery workaround you gave me yesterday worked great! Thx again! | 14:06 |
| dtantsur | cdearborn, np) | 14:06 |
| rloo | dtantsur: so 'bump API version when default state changes' --> ironic server microversion up'd | 14:06 |
| rloo | dtantsur: 'release new client with new version header' -> python-ironicclient with version == new version | 14:07 |
| rloo | dtantsur: nothing there about major/minor versions of client | 14:07 |
| *** lazy_prince has joined #openstack-ironic | 14:07 | |
| dtantsur | <dtantsur> this etherpad does not speak about client version; assuming we do use semver, it's an obvious question | 14:07 |
| rloo | dtantsur: so we didn't discuss that part at the summit. I'm not going crazy | 14:07 |
| dtantsur | honestly I can't believe we seriously discuss it now | 14:08 |
| rloo | dtantsur: well, i'm fine stopping the discussion. i'd still question your spec though. | 14:08 |
| dtantsur | I can just drop it and let devananda decide new version | 14:08 |
| dtantsur | rloo, could you please finally explain why? | 14:08 |
| rloo | dtantsur: yeah, it seems to me the question of major/minor client versions is separate | 14:08 |
| rloo | dtantsur: explain your spec or stopping the discussion? | 14:09 |
| trown | http://semver.org/ | 14:09 |
| openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Implement OEM Inventory components https://review.openstack.org/187592 | 14:09 |
| dtantsur | rloo, I'd like to see the reasons you question the spec, putting aside client versioning (which is none of my business actually) | 14:10 |
| rloo | dtantsur: oh, w/o the client versioning stuff in your spec, i think i'm fine with it. i just want clarification on the ordering stuff so I understand. | 14:10 |
| lucasagomes | NobodyCam, morning | 14:10 |
| lucasagomes | jlvillal, nice! thank you | 14:10 |
| rloo | dtantsur: sec, let me get back to your spec | 14:11 |
| lucasagomes | jlvillal, and morning :-) | 14:11 |
| dtantsur | rloo, so you prefer me dropping statement about client versioning, and let devananda do the same thing, but implicitly? | 14:11 |
| rloo | dtantsur: wrt your comment in the spec (again...) | 14:11 |
| jlvillal | lucasagomes, Good morning. | 14:11 |
| dtantsur | I'm fine with it | 14:11 |
| jlvillal | Good morning Ironic | 14:11 |
| dtantsur | jlvillal, morning | 14:11 |
| rloo | dtantsur: 1. release new version of ironic. ok | 14:11 |
| rloo | dtantsur: ok, let's just forget about 'releases' (Unless we need to) | 14:12 |
| rloo | dtantsur: you make your enroll changes to the server | 14:12 |
| rloo | dtantsur: you make <what> changes to the client? | 14:12 |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add iRMC Virtual Media Deploy module for iRMC Driver https://review.openstack.org/151958 | 14:13 |
| dtantsur | 2. fix devstack by defaulting to new version | 14:13 |
| dtantsur | 3. release new client version defaulting to this version | 14:13 |
| dtantsur | finished | 14:13 |
| dtantsur | rloo, ^^ | 14:13 |
| rloo | dtantsur: seems like the client needs to support 'enroll' in provision state, and up the version | 14:13 |
| dtantsur | rloo, what support do you expect? | 14:13 |
| dtantsur | "enroll" is not an action, it's just a state | 14:14 |
| dtantsur | "manage" is an action, but we already have it | 14:14 |
| rloo | dtantsur: ok, so 2. we have hardcoded in devstack, the microversion? | 14:14 |
| dtantsur | yep, so that we can fix it before doing 3. | 14:14 |
| dtantsur | otherwise 3. will break it and all our gates | 14:14 |
| rloo | dtantsur: oh right, was trying to remember how to get from enroll -> manageable | 14:14 |
| rloo | so 3 is what? 'release' -> new release of client? | 14:15 |
| dtantsur | rloo, yes | 14:15 |
| dtantsur | with bumping the default microversion | 14:15 |
| rloo | dtantsur: ok, so 3 is a release of changes to client in 1 | 14:15 |
| rloo | dtantsur: ahh, that's what I don't understand | 14:16 |
| dtantsur | yeah | 14:16 |
| rloo | dtantsur: so 1. changes to client are just the warning still? | 14:16 |
| dtantsur | didn't you object to having a warning? | 14:16 |
| dtantsur | if we drop it, step 1 is changing only Ironic itself | 14:16 |
| rloo | dtantsur: yeah, i don't see the need for the warning. | 14:16 |
| rloo | dtantsur: ok thx. no warning. | 14:16 |
| dtantsur | step 2 is changing devstack | 14:16 |
| dtantsur | step 3 is changing client | 14:17 |
| rloo | dtantsur: yup. i am good with that then. 1. ironic; 2. devstack; 3.client | 14:17 |
| dtantsur | rloo, ack, thanks. will update the spec | 14:17 |
| rloo | dtantsur: maybe I should ask for +2 powers on specs now. just for your spec :) | 14:17 |
| dtantsur | rloo, I think you should, you're doing a lot of reviews on specs anyway | 14:18 |
| jlvillal | rloo, lucasagomes: FYI: There is a proposed spec on oslo versioned objects. | 14:18 |
| dtantsur | :) | 14:18 |
| jlvillal | https://review.openstack.org/#/c/187879/ | 14:18 |
| *** romcheg has quit IRC | 14:18 | |
| rloo | jlvillal: yeah, i saw that. haven't had a chance to look. do we need a spec for it? | 14:18 |
| dtantsur | jlvillal, I don't think that needs a spec tbh | 14:18 |
| lucasagomes | jlvillal, oh nice | 14:18 |
| dtantsur | left a comment | 14:18 |
| lucasagomes | will look... | 14:18 |
| * jlvillal doesn't know if a spec is needed. I think yesterday there was discussion that a bug is enough. | 14:18 | |
| dtantsur | ++ for a bug | 14:19 |
| jlvillal | dtantsur, lucasagomes: rloo created a bug yesterday for it. | 14:19 |
| jlvillal | https://bugs.launchpad.net/ironic/+bug/1461239 | 14:20 |
| openstack | Launchpad bug 1461239 in Ironic "replace objects with oslo.versionedobjects" [Low,New] | 14:20 |
| *** mtanino has joined #openstack-ironic | 14:20 | |
| openstackgerrit | Yuiko Takada proposed stackforge/ironic-discoverd: Use keystonemiddleware auth credentials https://review.openstack.org/170782 | 14:20 |
| lucasagomes | yeah it's just internal, bug seems good | 14:21 |
| *** mdbooth has quit IRC | 14:21 | |
| rloo | ok, just commented in that spec | 14:21 |
| rloo | tan is the oslo liason, right? | 14:22 |
| jlvillal | rloo, Yes | 14:22 |
| openstackgerrit | Merged openstack/ironic: Add documentation for getting a node's console https://review.openstack.org/176642 | 14:22 |
| NobodyCam | morning jlvillal lucasagomes :) | 14:24 |
| NobodyCam | (UGT) ofc | 14:24 |
| jlvillal | NobodyCam, Good morning | 14:24 |
| *** zz_jgrimm is now known as jgrimm | 14:25 | |
| *** david-lyle has quit IRC | 14:25 | |
| NobodyCam | :) | 14:25 |
| NobodyCam | lucasagomes: have I told you I really like Pixie on the ironic.rocks page... because ... I really do :) | 14:26 |
| lucasagomes | NobodyCam, lol | 14:26 |
| dtantsur | problem is that a lot of software does not recognize ironic.rocks as a link (even with http://) | 14:27 |
| lucasagomes | NobodyCam, he's still provisioning the website, needs more time | 14:27 |
| NobodyCam | ya | 14:27 |
| *** Haomeng has joined #openstack-ironic | 14:27 | |
| NobodyCam | hehehehe | 14:27 |
| trown | dtantsur: really? | 14:27 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-specs: Add "enroll" state to the state machine https://review.openstack.org/179151 | 14:27 |
| lucasagomes | dtantsur, www.ironic.rocks?! | 14:27 |
| * BadCub heads off to early meeting | 14:27 | |
| dtantsur | rloo, updated the spec | 14:27 |
| dtantsur | lucasagomes, works here, does not work in skype | 14:27 |
| dtantsur | trown, ^^ | 14:28 |
| rloo | thx dtantsur. looking... | 14:28 |
| * dtantsur dives into project-config | 14:28 | |
| lucasagomes | dtantsur, even with the http://www ? | 14:28 |
| trown | also +1 to construction Pixie | 14:28 |
| dtantsur | lucasagomes, yep | 14:28 |
| openstackgerrit | Merged stackforge/pyghmi: Implement OEM Inventory components https://review.openstack.org/187592 | 14:28 |
| lucasagomes | fill out a bug for skype! | 14:28 |
| lucasagomes | heh | 14:28 |
| *** derekh has quit IRC | 14:28 | |
| trown | ya, skype needs to add the new TLDs | 14:28 |
| dtantsur | lol, bug for skype :D | 14:28 |
| * trown pictures geodes with hipster outfits everytime he reads "ironic.rocks" | 14:30 | |
| *** Haomeng|2 has quit IRC | 14:30 | |
| dtantsur | lol | 14:30 |
| *** rwsu has quit IRC | 14:31 | |
| *** barra204 has quit IRC | 14:36 | |
| *** shakamunyi has quit IRC | 14:36 | |
| *** killer_prince has joined #openstack-ironic | 14:48 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Initial inventory module https://review.openstack.org/184084 | 14:49 |
| *** lazy_prince has quit IRC | 14:49 | |
| TheJulia | NobodyCam: Do me a favor and take a look at 187532 please | 14:51 |
| *** lazy_prince has joined #openstack-ironic | 14:53 | |
| *** killer_prince has quit IRC | 14:53 | |
| *** naohirot has quit IRC | 14:59 | |
| * NobodyCam looks | 14:59 | |
| *** yuikotakada has quit IRC | 15:06 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: When boot option is not persisted, set boot on next power on https://review.openstack.org/177642 | 15:09 |
| *** jgrimm is now known as zz_jgrimm | 15:10 | |
| *** zz_natorious is now known as natorious | 15:11 | |
| NobodyCam | TheJulia: 187532 looks like a patch... lol | 15:13 |
| * NobodyCam is assuming look == review | 15:14 | |
| * TheJulia pours NobodyCam more coffee | 15:15 | |
| *** natorious is now known as zz_natorious | 15:16 | |
| NobodyCam | TheJulia: TY :) | 15:16 |
| *** killer_prince has joined #openstack-ironic | 15:17 | |
| NobodyCam | TheJulia: +A'd :) | 15:17 |
| TheJulia | ty | 15:18 |
| *** rwsu has joined #openstack-ironic | 15:19 | |
| *** lazy_prince has quit IRC | 15:20 | |
| *** mdbooth has joined #openstack-ironic | 15:20 | |
| *** alexpilotti has quit IRC | 15:24 | |
| *** gridinv_ has joined #openstack-ironic | 15:26 | |
| *** jcoufal has quit IRC | 15:27 | |
| lucasagomes | dtantsur, re 187678, lemme know if my answer makes sense | 15:29 |
| lucasagomes | with WOL there's no way if not ignore power off | 15:29 |
| dtantsur | I'm fine with any answer, actually. I just wanted it to be explicitly stated in a spec | 15:30 |
| lucasagomes | unless we do things like ssh'ing into the image and pwoering that off, but I want to avoid that because it's just a testing driver and powering it off is not a big deal to do manuall (at least for me) | 15:30 |
| lucasagomes | dtantsur, oh sure yeah, ok I will update it | 15:30 |
| *** e0ne is now known as e0ne_ | 15:32 | |
| *** killer_prince has quit IRC | 15:34 | |
| *** killer_prince has joined #openstack-ironic | 15:34 | |
| *** killer_prince has quit IRC | 15:35 | |
| *** e0ne_ is now known as e0ne | 15:35 | |
| *** zz_natorious is now known as natorious | 15:36 | |
| openstackgerrit | Merged openstack/bifrost: Lock Bifrost to Ansible stable-1.9 https://review.openstack.org/187532 | 15:37 |
| NobodyCam | TheJulia: ^^^^ | 15:37 |
| * TheJulia dances | 15:38 | |
| *** david-lyle has joined #openstack-ironic | 15:39 | |
| *** Sukhdev has joined #openstack-ironic | 15:39 | |
| *** zz_jgrimm is now known as jgrimm | 15:40 | |
| *** romainh has left #openstack-ironic | 15:40 | |
| *** dtantsur is now known as dtantsur|afk | 15:41 | |
| dtantsur|afk | IKEA time for me :) see you tomorrow | 15:41 |
| NobodyCam | have a good night dtantsur|afk | 15:41 |
| trown | lol, dont get lost dtantsur|afk | 15:41 |
| *** pas-ha has quit IRC | 15:41 | |
| *** hemnafk is now known as hemna | 15:41 | |
| NobodyCam | follow the arrows on the floor | 15:42 |
| *** david-lyle has quit IRC | 15:47 | |
| *** alexpilotti has joined #openstack-ironic | 15:48 | |
| sambetts | night dtantsur|afk | 15:49 |
| *** mdbooth has quit IRC | 15:49 | |
| *** mdbooth has joined #openstack-ironic | 15:50 | |
| *** thrash has quit IRC | 15:50 | |
| *** bradbeam has joined #openstack-ironic | 15:52 | |
| *** david-lyle has joined #openstack-ironic | 15:53 | |
| *** jhova has joined #openstack-ironic | 15:53 | |
| *** thrash has joined #openstack-ironic | 15:54 | |
| jhova | Hey guys - Using bifrost to deploy standalone ironic and we're hitting issues with - cat {{ baremetal_csv_file }} not being able to path the csv file | 15:55 |
| jhova | I inserted a log statement and the variable is being initialized correctly | 15:55 |
| jhova | has anyone seen this issue? | 15:55 |
| NobodyCam | hey jhova .. I have not seen that, but I know we just landed a patch that pins us to version 1.9 | 15:58 |
| NobodyCam | maybe TheJulia has seen this ^^^^ | 15:58 |
| jhova | OK, maybe that's the issue. We just used the setup script and got Ansible 2.0 | 15:58 |
| NobodyCam | https://review.openstack.org/187532 just landed | 15:59 |
| openstackgerrit | Thiago Paiva Brito proposed openstack/ironic-specs: OneView Driver for Ironic https://review.openstack.org/187762 | 15:59 |
| TheJulia | jhova: I've not seen that | 15:59 |
| TheJulia | Yeah, we're finding all sorts of issues with Ansible 2.0 right now | 15:59 |
| jhova | TheJulia: NobodyCam: we'll downgrade and see if that resolves the issue. | 15:59 |
| TheJulia | To the point we're locking on stable-1.9 for now. | 15:59 |
| jhova | thanks | 16:00 |
| TheJulia | no problem, if that doesn't fix things let us know :) | 16:00 |
| NobodyCam | jhova: if that fixes you... would you mind filling a bug on what you saw with 2.0 | 16:00 |
| * jlvillal has procured his first cup of coffee for the day :) | 16:01 | |
| BadCub | hiya jlvillal | 16:01 |
| jlvillal | BadCub: Good morning | 16:01 |
| * TheJulia is jealous of this first cup of coffee | 16:01 | |
| * BadCub goes out for a breather after 1.5 hours of calls | 16:01 | |
| * BadCub is on 4th cup of coffee | 16:02 | |
| jlvillal | TheJulia: I would assume you had yours hours ago :) | 16:02 |
| TheJulia | NobodyCam: good thinking, the more bugs people file the less bugs I have to file :) | 16:02 |
| * NobodyCam is on 3rd cup | 16:02 | |
| TheJulia | jlvillal: can't have any today :( | 16:02 |
| jlvillal | TheJulia: The horror! ;) | 16:02 |
| TheJulia | I know :( | 16:02 |
| NobodyCam | TheJulia: how are you moving | 16:03 |
| TheJulia | NobodyCam: magic | 16:03 |
| NobodyCam | :-p | 16:03 |
| *** eghobo has joined #openstack-ironic | 16:03 | |
| TheJulia | Anyone happen to know what model NUC's support AMT? | 16:04 |
| jhova | TheJulia: will do | 16:05 |
| *** jgrimm is now known as zz_jgrimm | 16:07 | |
| *** zz_jgrimm is now known as jgrimm | 16:09 | |
| *** jistr has quit IRC | 16:11 | |
| TheJulia | NUC5i5MYHE and NUC5i5MYBE | 16:12 |
| *** gridinv_ has quit IRC | 16:13 | |
| *** ifarkas has quit IRC | 16:14 | |
| openstackgerrit | Merged openstack/bifrost: Add PyYAML to the list of requirements https://review.openstack.org/187051 | 16:18 |
| jlvillal | TheJulia: The ones with an E at the end usually. | 16:20 |
| jlvillal | TheJulia: And yes, what you posted :) | 16:20 |
| *** jamielennox is now known as jamielennox|away | 16:21 | |
| *** barra204 has joined #openstack-ironic | 16:21 | |
| *** david-lyle has quit IRC | 16:22 | |
| *** spandhe has joined #openstack-ironic | 16:22 | |
| *** spandhe has quit IRC | 16:27 | |
| *** barra204 has quit IRC | 16:28 | |
| *** athomas has quit IRC | 16:35 | |
| *** jhova has quit IRC | 16:37 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add disk layout check on re-provisioning https://review.openstack.org/180014 | 16:38 |
| *** intr1nsic has joined #openstack-ironic | 16:38 | |
| *** marzif_ has joined #openstack-ironic | 16:39 | |
| sambetts | Calling it a day for today, Good UGT night ironic o/ | 16:39 |
| *** trown is now known as trown|lunch | 16:39 | |
| NobodyCam | have a good night sambetts | 16:39 |
| *** jeh has joined #openstack-ironic | 16:40 | |
| jlvillal | Has anyone already created some scripts to do a virt-install of an image useful for Ironic testing? Before I go re-invent the wheel :) | 16:45 |
| lucasagomes | jlvillal, devstack? | 16:47 |
| jlvillal | lucasagomes: It does a 'virt-install'? I was actually trying to create something to run dev-stack in. | 16:48 |
| jlvillal | lucasagomes: So I don't accidentally destroy my system when running devstack :) | 16:48 |
| lucasagomes | you can use a VM and deploy with Ironic on nested VMs | 16:48 |
| lucasagomes | I mean deploy nested VMs with Ironic | 16:49 |
| lucasagomes | so yeah, you can/should run devstack on a VM | 16:49 |
| jlvillal | lucasagomes: Right. I was just wanting to automate creating that first VM with virt-install. So I can reproduce it as needed. | 16:49 |
| lucasagomes | jlvillal, I see, hmm I don't know anything that does it... I mean you could always snapshot a VM | 16:52 |
| lucasagomes | after you run devstack and so on | 16:52 |
| lucasagomes | but yeah... it's not that nice | 16:52 |
| jlvillal | lucasagomes: Okay. I just wanted to automate creating a simple VM, that then is all ready to run devstack in. | 16:52 |
| lucasagomes | there's the docker thing that Shrews created as well, but I haven't tried yet | 16:53 |
| jlvillal | lucasagomes: I will plug away at it. Thanks! :) | 16:53 |
| lucasagomes | https://github.com/Shrews/dockstack/ | 16:53 |
| * jlvillal will need to spin up things more often with the functional testing work he has to do. | 16:53 | |
| jlvillal | lucasagomes: Cool. Thanks. | 16:53 |
| rloo | JayF (or other core-specs), the RAID spec has a +2 and is waiting for you :-) https://review.openstack.org/#/c/173214/ | 16:58 |
| JayF | RAID spec is 2x+2, 5x+1 if others want to take a final look and land it -> https://review.openstack.org/#/c/173214 | 16:58 |
| JayF | Well, it can't land without zapping | 16:59 |
| JayF | rloo: ^ | 16:59 |
| JayF | it deps on zapping which is -1 | 16:59 |
| rloo | JayF: oh geez. I -1'd zapping I think :-( | 16:59 |
| JayF | Yep | 16:59 |
| JayF | rloo: I +2 | 16:59 |
| JayF | the raid spec just now though, fwiw | 16:59 |
| NobodyCam | morning JayF | 17:00 |
| rloo | JayF: is JoshNang around/available to work on zapping spec? | 17:00 |
| JoshNang | rloo: o/ | 17:00 |
| JoshNang | sorry i was eto yesterday | 17:00 |
| rloo | JayF: or err can you make him available :D | 17:00 |
| *** bethelwell has quit IRC | 17:00 | |
| NobodyCam | morning JoshNang | 17:00 |
| JoshNang | NobodyCam: good morning! | 17:00 |
| NobodyCam | :) | 17:00 |
| rloo | no worries JoshNang. Let me know if I can help you ith the zapping spec. | 17:00 |
| JoshNang | rloo: i'll take a look at it today | 17:01 |
| JoshNang | rloo: thanks for the review! | 17:01 |
| *** jhova has joined #openstack-ironic | 17:01 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-specs: Wake-On-Lan (WOL) power driver https://review.openstack.org/187678 | 17:02 |
| *** spandhe has joined #openstack-ironic | 17:03 | |
| lucasagomes | folks I will calle it a day | 17:04 |
| lucasagomes | have a good night everyone | 17:04 |
| *** lucasagomes is now known as lucas-dinner | 17:04 | |
| TheJulia | goodnight | 17:04 |
| clif_h | jlvillal: thanks for the recheck on my review | 17:05 |
| * TheJulia ponders lunch | 17:05 | |
| openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Add serial field from SPD https://review.openstack.org/188086 | 17:06 |
| NobodyCam | night lucas-dinner | 17:07 |
| TheJulia | win 30 | 17:07 |
| TheJulia | doh | 17:07 |
| NobodyCam | lucas-dinner: I may have a question about WOL driver for you tommorrow | 17:07 |
| lucas-dinner | NobodyCam, shoot | 17:07 |
| lucas-dinner | still here :-) | 17:07 |
| NobodyCam | lol | 17:08 |
| jlvillal | clif_h: You're welcome. With the gate being broken all day I tried to find all the broken jobs yesterday and do a recheck in the evening. | 17:08 |
| NobodyCam | could we mix in the ipmi listener in to the deploy ram disk and user image then send wol for power on and ipmi for power off? | 17:09 |
| NobodyCam | I am assuming that this is a testing only driver | 17:09 |
| lucas-dinner | yea that is a testing driver... wait trying to grasp the idea here | 17:10 |
| lucas-dinner | the part I don't get is ipmi listener in the deploy ramdisk? | 17:11 |
| NobodyCam | for reboot aafter deploy | 17:11 |
| jhova | What's the best way to reset the status of a node stuck in "cleaning"? Do I need to manually set it in the database? | 17:11 |
| NobodyCam | but that may not be needed | 17:11 |
| JayF | NobodyCam: Wouldn't it be better to not emulate such a crappy protocol | 17:11 |
| JayF | jhova: Stuck how? | 17:12 |
| lucas-dinner | NobodyCam, hmm we could but the dpeloy ramdisk won't be running after the image is deployed | 17:12 |
| NobodyCam | JayF: my thought is we have all the tooling to do that now basicly | 17:12 |
| jhova | jayf: I set the node to deleted status to reclaim, it rebooted into the dicovery image and has been for the last 3 hours or so. | 17:12 |
| NobodyCam | lucas-dinner: yes not needed in deploy rammdisk | 17:12 |
| jhova | and the current status is cleaning | 17:12 |
| lucas-dinner | NobodyCam, a simple way would be to make sure the user name & deploy ramdisk has a ssh key, so we could SSH into it and power off | 17:12 |
| lucas-dinner | but, I don't wanna do it currently, cause the machines I use to deploy is just under my table | 17:13 |
| JayF | jhova: it's possible it could be taking that long to erase your drive. If you look at the node output it should indicate the cleaning step it's on | 17:13 |
| *** david-lyle has joined #openstack-ironic | 17:13 | |
| lucas-dinner | I can power them off | 17:13 |
| lucas-dinner | manually I mean | 17:13 |
| jroll | BadCub: hey fyi I might be available for the proposed midcycle dates after all | 17:13 |
| NobodyCam | jroll: w00 h00 | 17:13 |
| *** david-lyle has quit IRC | 17:13 | |
| jroll | NobodyCam: working on planning a northern michigan trip, cabin isn't available til the 16th :) | 17:14 |
| NobodyCam | lucas-dinner: ahh I expect my test system will be in other locations | 17:14 |
| NobodyCam | :) | 17:14 |
| NobodyCam | jroll: :) | 17:14 |
| NobodyCam | party (mid-cycle) in seattle | 17:14 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/188093 | 17:15 |
| lucas-dinner | NobodyCam, right, well WOL just works on the local network anyway | 17:15 |
| *** david-lyle has joined #openstack-ironic | 17:15 | |
| *** dguerri is now known as dguerri`away | 17:15 | |
| lucas-dinner | NobodyCam, well yeah the SSH'ing thing can be incorporated, but not sure I want to do it as part of that work | 17:16 |
| lucas-dinner | that needs to change image building and stuff, I'm good just powering my machines off by hand | 17:16 |
| lucas-dinner | NobodyCam, you ok with that? | 17:16 |
| NobodyCam | lucas-dinner: by other location I mean my garage | 17:16 |
| NobodyCam | lucas-dinner: ya | 17:17 |
| *** e0ne has quit IRC | 17:17 | |
| NobodyCam | I need to put more though be hind what I'm thinking | 17:17 |
| NobodyCam | go and have a pint for /me and I'll try and think about it more later today | 17:18 |
| NobodyCam | see if even I can make sense of what I think I want | 17:18 |
| NobodyCam | lol | 17:18 |
| lucas-dinner | NobodyCam, :D hah yeah thanks | 17:20 |
| lucas-dinner | NobodyCam, you can buy a wireless power switch | 17:20 |
| lucas-dinner | so you can turn it off remotly :-) | 17:21 |
| lucas-dinner | hard power off | 17:21 |
| NobodyCam | :) | 17:21 |
| lucas-dinner | and then turn it on and WOL it again :-) | 17:21 |
| lucas-dinner | http://www.amazon.com/Ankuoo-Controlling-Electronics-Monitoring-Smartphones/dp/B00NAX31WI/ref=sr_1_30?s=electronics&ie=UTF8&qid=1433352108&sr=1-30&keywords=wireless+power+switch | 17:22 |
| lucas-dinner | heh | 17:22 |
| *** spandhe has quit IRC | 17:22 | |
| lucas-dinner | aight heading, see y'all later good night! | 17:22 |
| NobodyCam | niht lucas-dinner :) enjoy | 17:23 |
| jhova | jayf: node-show doesn't show the current clean step. Is there a default password to the discovery image? | 17:23 |
| JayF | jhova: node-show --detail | 17:23 |
| jhova | jayf: running in standalone | 17:23 |
| openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Add serial field from SPD https://review.openstack.org/188086 | 17:24 |
| *** ukalifon1 has quit IRC | 17:24 | |
| JayF | I might not be the best person to help you from here on :) I'm literally going to be in meetings for hours following, lots of other smart folks here who might be able to help though | 17:24 |
| jhova | jayf: thanks man | 17:24 |
| *** david-lyle has quit IRC | 17:28 | |
| JayF | BadCub: jroll: What are the proposed dates for the mid-cycle? They aren't on the openstack wiki | 17:28 |
| jroll | JayF: aug 13-16 or so? | 17:28 |
| jroll | they're on the mtg agenda | 17:28 |
| jroll | devananda was going to send a poll | 17:29 |
| NobodyCam | https://wiki.openstack.org/w/index.php?title=Meetings/Ironic&diff=prev&oldid=82248 | 17:33 |
| NobodyCam | ya, I don't believe devananda has had the time to send out the poll.. discover and all :-p | 17:34 |
| *** Sukhdev has quit IRC | 17:34 | |
| rloo | BadCub: please chase devananda ^^ wrt midcycle :) | 17:35 |
| *** trown|lunch is now known as trown | 17:38 | |
| openstackgerrit | Merged stackforge/pyghmi: Add serial field from SPD https://review.openstack.org/188086 | 17:49 |
| *** dguerri`away is now known as dguerri | 17:59 | |
| *** edcragg has quit IRC | 17:59 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Initial inventory module https://review.openstack.org/184084 | 18:01 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Initial inventory module https://review.openstack.org/184084 | 18:02 |
| *** spandhe has joined #openstack-ironic | 18:07 | |
| *** eghobo_ has joined #openstack-ironic | 18:18 | |
| * NobodyCam has lost his coffee cup... :( | 18:18 | |
| *** eghobo has quit IRC | 18:18 | |
| *** marzif_ has quit IRC | 18:19 | |
| *** marzif_ has joined #openstack-ironic | 18:19 | |
| TheJulia | lost? did it run away? | 18:31 |
| rloo | hey jlvillal, this is fixed, right? https://bugs.launchpad.net/ironic/+bug/1461182 | 18:31 |
| openstack | Launchpad bug 1461182 in oslo.messaging "After oslo.messaging release Ironic gate is broken" [Undecided,In progress] - Assigned to Doug Hellmann (doug-hellmann) | 18:31 |
| jlvillal | rloo: Yes. | 18:31 |
| NobodyCam | TheJulia: I left it at the coffee maker :-p | 18:32 |
| jlvillal | rloo: Looks like we weren't the only ones affected by it | 18:32 |
| rloo | jlvillal: yeah | 18:32 |
| *** david-lyle has joined #openstack-ironic | 18:33 | |
| TheJulia | :( | 18:35 |
| *** spandhe has left #openstack-ironic | 18:35 | |
| *** ijw has joined #openstack-ironic | 18:35 | |
| openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Provide access to get/set community string https://review.openstack.org/188135 | 18:36 |
| openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Remove libvirt from requirements https://review.openstack.org/188136 | 18:38 |
| *** [1]cdearborn has joined #openstack-ironic | 18:38 | |
| greghaynes | Hello ironicers, I am wondering what the implications of https://review.openstack.org/#/c/186655/1/elements/deploy-ironic/init.d/80-deploy-ironic are for backwards compat | 18:44 |
| greghaynes | specifically, do older versions of ironic support that path (with the api version) | 18:45 |
| greghaynes | and/or are we ok with breaking backwards compat with versions who do not | 18:45 |
| *** spandhe has joined #openstack-ironic | 18:49 | |
| jeh | we're using bifrost on a host with a raid card (LSI) that has a single virtual disk raid array and some jbods. It looks like it might be installing the OS to a different disk then what it's booting. Are there any gotchas or tips on working with this type of configuration? | 18:51 |
| NobodyCam | jeh: you may need to use disk hints | 18:52 |
| NobodyCam | let me see if I can grap a link for ya | 18:52 |
| *** pradipta has joined #openstack-ironic | 18:53 | |
| *** cdearborn has quit IRC | 18:54 | |
| openstackgerrit | Merged openstack/bifrost: Tiny documentation fix https://review.openstack.org/186940 | 18:54 |
| *** mordred has quit IRC | 18:55 | |
| *** mordred has joined #openstack-ironic | 18:55 | |
| NobodyCam | jeh: http://docs.openstack.org/developer/ironic/deploy/install-guide.html#specifying-the-disk-for-deployment | 18:55 |
| BadCub | Hey everyone here is the link to the poll for Mid-Cycle: http://goo.gl/forms/s4pGM8mLyb | 18:55 |
| openstackgerrit | Merged openstack/bifrost: Typo fix (disk imae builder) https://review.openstack.org/187949 | 18:55 |
| BadCub | Ack.... Try this link instead... My bad: http://goo.gl/forms/RJvq0uqfSD | 18:56 |
| krotscheck | Ironic doesn't use pastedeploy. does it. | 18:57 |
| krotscheck | ? | 18:57 |
| jeh | NobodyCam: thanks that's helpful | 18:57 |
| NobodyCam | krotscheck: ??? | 18:57 |
| NobodyCam | not that I know | 18:57 |
| NobodyCam | jeh: :) | 18:57 |
| *** marzif_ has quit IRC | 18:57 | |
| krotscheck | Everyone else seem to. | 18:57 |
| *** barra204 has joined #openstack-ironic | 18:57 | |
| NobodyCam | greghaynes: humm i'd need to test that | 18:58 |
| krotscheck | I don't really have an opinion on using it or no, but it's very likely to impact the middleware discussion. | 18:58 |
| openstackgerrit | Merged stackforge/pyghmi: Provide access to get/set community string https://review.openstack.org/188135 | 18:58 |
| krotscheck | Because paste-supporting middleware looks WAY different. | 18:58 |
| *** ijw has quit IRC | 19:00 | |
| TheJulia | NobodyCam: You mentioned some issues on supermicro chassis using ipmi. Specifically were they issues along the lines of the stupid thing doesn't want to netboot? | 19:02 |
| *** barra204 has quit IRC | 19:02 | |
| NobodyCam | TheJulia: its been a while sense I've had SM boxes.. but I *_think_* thats was one of them.. | 19:04 |
| NobodyCam | brb | 19:05 |
| TheJulia | That is what my machine seems to be doing. Everything ipmi wise has been great, but even if configured in the bios, it just doesn't actually really try to boot to the network :( | 19:05 |
| *** cdearborn has joined #openstack-ironic | 19:06 | |
| openstackgerrit | Merged stackforge/pyghmi: Remove libvirt from requirements https://review.openstack.org/188136 | 19:09 |
| BadCub | I also posted the link to the poll on the meeting agenda for next week. | 19:12 |
| *** e0ne has joined #openstack-ironic | 19:13 | |
| jroll | BadCub: can you send the poll to the list so everyone sees it? | 19:15 |
| BadCub | Yep | 19:15 |
| rloo | greghaynes: wrt your question about 186655; it is backwards compatible. the original url goes to /v1/... so the change is making it explicit. | 19:16 |
| rloo | BadCub: thx for the link. Did you send email out about it? | 19:17 |
| BadCub | rloo: doing that now | 19:17 |
| rloo | BadCub: oh, I see that jroll just asked that :) | 19:17 |
| jroll | :P | 19:18 |
| BadCub | sent | 19:19 |
| *** pradipta has quit IRC | 19:20 | |
| *** [1]cdearborn has quit IRC | 19:21 | |
| greghaynes | rloo: So if I hit /v1/nodes/foo on an old version of ironic that will work? | 19:23 |
| rloo | greghaynes: yes. cuz ironic only has /v1, and if not specified, it goes to v1. | 19:24 |
| rloo | greghaynes: caveat, unless we have a bug :) | 19:24 |
| greghaynes | hah | 19:25 |
| greghaynes | ok, awesome | 19:25 |
| greghaynes | ty | 19:25 |
| rloo | greghaynes: in general, all requests have /v1/... It was noticed that the original here didn't, which is the reason for the patch. | 19:25 |
| greghaynes | gotcha | 19:25 |
| * NobodyCam is back | 19:25 | |
| *** jgrimm is now known as zz_jgrimm | 19:26 | |
| jroll | greghaynes: rloo: so where does $IRONIC_API_URL come from? | 19:37 |
| jroll | it's not a config option, it comes from ironic itself | 19:38 |
| jroll | kinda surprised this passed ci | 19:38 |
| rloo | jroll: dunno, you trying to get me to look? | 19:39 |
| jroll | rloo: no, just thinking out loud :) | 19:39 |
| rloo | jroll: I'm not familiar with diskimage-builder. Maybe NobodyCam knows | 19:40 |
| greghaynes | jroll: boot param I think | 19:40 |
| jroll | greghaynes: yeah, and looks like it comes across correct | 19:40 |
| jroll | so I'm wondering how this worked before | 19:40 |
| *** igordcard_ has quit IRC | 19:40 | |
| greghaynes | jroll: I assume what rloo was saying - youall default to /v1 if not specified? | 19:40 |
| jroll | where was the /v1 coming from before this change? | 19:40 |
| jroll | I guess? | 19:41 |
| rloo | jroll: you mean how did the $IRONIC_API_URL/nodes/... request work? | 19:41 |
| jroll | wow, TIL | 19:41 |
| jroll | I didn't realize version was optional there | 19:41 |
| rloo | I guess the commit msg could have been a bit clearer about that. | 19:42 |
| jroll | yeah, or like, a bug | 19:43 |
| rloo | jroll: yeah, that was noticed due to an Ironic patch that yuriy submitted recently | 19:43 |
| jroll | greghaynes: left a +1 there | 19:43 |
| rloo | jroll: I'm actually fine leaving it as is cuz it works but whatever :) | 19:43 |
| greghaynes | jroll: awesome | 19:43 |
| jroll | rloo: until we default to /v2 and it breaks :P | 19:44 |
| jroll | but I think we'll deprecate the bash stuff before that happens | 19:44 |
| jroll | so either way | 19:44 |
| jroll | ¯\_(ツ)_/¯ | 19:44 |
| rloo | jroll: yeah, but that goes back to my question (on the ironic patch). if we promote heartbeat and passthru etc to DeployInterface, would we change the endpoint. | 19:45 |
| rloo | jroll: and you just mentioned another reason :) | 19:45 |
| jroll | rloo: sure, that's a great question, but a separate (and solvable) one | 19:45 |
| rloo | jroll: yeah, separate. just wondering whether it is worth doing work if it ain't needed but it doesn't hurt so... | 19:46 |
| jroll | well, the work is done :P | 19:46 |
| rloo | jroll: almost, but yeah :) | 19:48 |
| *** pas-ha has joined #openstack-ironic | 19:52 | |
| *** natorious is now known as zz_natorious | 19:53 | |
| *** barra204 has joined #openstack-ironic | 19:58 | |
| *** Haomeng has quit IRC | 19:59 | |
| *** Haomeng has joined #openstack-ironic | 19:59 | |
| *** barra204 has quit IRC | 20:03 | |
| *** mtanino has quit IRC | 20:06 | |
| *** e0ne is now known as e0ne_ | 20:09 | |
| *** e0ne_ is now known as e0ne | 20:14 | |
| *** e0ne has quit IRC | 20:14 | |
| *** lucas-dinner has quit IRC | 20:15 | |
| *** alexpilotti has quit IRC | 20:17 | |
| *** gridinv_ has joined #openstack-ironic | 20:19 | |
| *** gridinv_ has quit IRC | 20:24 | |
| *** Sukhdev has joined #openstack-ironic | 20:29 | |
| *** radek_ has quit IRC | 20:44 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Initial inventory module https://review.openstack.org/184084 | 20:51 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Initial dynamic enrollment role https://review.openstack.org/188176 | 20:51 |
| martini | TheJulia: I won't have time to look at the changes til tonight, but I'm looking forward to where this is going | 20:57 |
| martini | Oh, btw, I've been on for days but haven't really checked back log; is there a place for specs for bifrost? | 20:58 |
| TheJulia | martini: no rush, it was an idea I started working on before the summit and wanted to put it out there | 20:58 |
| TheJulia | martini: we haven't identified a specific place yet. We should do that. | 20:58 |
| *** bradbeam_ has joined #openstack-ironic | 20:59 | |
| *** pas-ha has quit IRC | 20:59 | |
| * TheJulia thinks it might be time to consider dinner | 21:01 | |
| *** bradbeam has quit IRC | 21:02 | |
| *** trown is now known as trown|outttypeww | 21:03 | |
| jhova | We are trying to setup ironic standalone. We have the hosts provisioning but it goes through the install, reboots, and then loads back to the discovery ramdisk instead of the provided image. Here are the node details: http://pastebin.com/gtDVKSqY | 21:05 |
| jhova | No errors in logs during the provisioning process | 21:06 |
| * TheJulia looks | 21:06 | |
| *** cdearborn has quit IRC | 21:08 | |
| *** bradbeam has joined #openstack-ironic | 21:08 | |
| TheJulia | so after the agent has pushed the image to disk, it reboots, and ends back up on the discovery/inspector image? | 21:08 |
| NobodyCam | jhova: can you paste the conductor log for this deploy attempt? | 21:08 |
| TheJulia | I need to step away, I'll try and check back on IRC periodicially through the evening | 21:12 |
| BadCub | go get some food TheJulia :) | 21:12 |
| NobodyCam | night TheJulia | 21:12 |
| jhova | Thanks TheJulia: http://pastebin.com/9XWh67Js | 21:12 |
| jhova | alot of spam in there from other IPMI calls | 21:13 |
| *** [1]cdearborn has joined #openstack-ironic | 21:15 | |
| *** jeh has quit IRC | 21:18 | |
| mrda | Morning Ironic | 21:28 |
| BadCub | mornin mrda | 21:29 |
| NobodyCam | morning mrda | 21:29 |
| mrda | o/ | 21:30 |
| NobodyCam | jhova: just checking that your sure CentOS-7-x86_64-GenericCloud-1503.qcow2 is a valid image? | 21:39 |
| NobodyCam | looks like it should have worked | 21:39 |
| jhova | it's just a straight centos7 cloud image | 21:39 |
| *** dontalton has joined #openstack-ironic | 21:40 | |
| jhova | nobodycam: When I try to boot from local disk, it reverts back to network boot like the image isn't getting written to disk. | 21:40 |
| jhova | we also have set {u'boot_option': u'local'} in the instance options | 21:42 |
| jhova | We're trying to install to a LSI RAID 1 virtual disk | 21:43 |
| jroll | I wonder if the bios can't boot from that disk | 21:44 |
| jroll | and falls back to pxe | 21:44 |
| jroll | jhova: ^^ | 21:44 |
| jhova | jroll: We were successfully deploying images via triple0 with this setup previously | 21:45 |
| jroll | jhova: booting from that virtual disk? | 21:45 |
| openstackgerrit | Josh Gachnang proposed openstack/ironic-specs: Implement Zapping States https://review.openstack.org/185122 | 21:45 |
| jhova | yeah | 21:45 |
| jroll | hrm | 21:45 |
| jroll | so what I would do is get in there and manually use ipmitool to set boot from disk | 21:46 |
| jroll | and reboot and whatnot, try to see what's up | 21:46 |
| jroll | follow the console | 21:46 |
| jhova | It looks like when we set boot-option local in the instance properties that ironic is setting that via IPMITOOL | 21:46 |
| jroll | right | 21:46 |
| *** alexpilotti has joined #openstack-ironic | 21:46 | |
| jhova | but we can give it a whirl | 21:47 |
| jroll | so I think at a minimum you need to watch the console | 21:47 |
| jroll | and see what's going on | 21:47 |
| NobodyCam | jhova: how did you add the local boot option? | 21:48 |
| NobodyCam | ironic node-update <node-uuid> add instance_info/capabilities='{"boot_option": "local"}' | 21:48 |
| jhova | ironic node-update uuid add instance_info/capabilities='{"boot_option": "local"}' | 21:49 |
| jhova | yessir | 21:49 |
| jroll | yeah, that part is working | 21:49 |
| jroll | or seems to be | 21:49 |
| jroll | just not the actual boot | 21:49 |
| jhova | going to try and set the root device to the LSI device in the instance_info | 21:50 |
| *** dprince has quit IRC | 21:53 | |
| *** zz_natorious is now known as natorious | 21:53 | |
| jhova | jroll NobodyCam: No dice. Cleaner logs: http://pastebin.com/D4KsJyt9 | 21:58 |
| jhova | troubleshooting as jroll suggested now | 21:59 |
| NobodyCam | ++ and i'll brb | 22:00 |
| jroll | ditto | 22:00 |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Don't raise exception from set_failed_state() https://review.openstack.org/188188 | 22:00 |
| * TheJulia tries to remember if we ever landed things for TripleO so the overcloud nodes could reboot with the undercloud down | 22:10 | |
| jhova | disabled network boot and it got no valid media error | 22:14 |
| TheJulia | jhova: does that image have a partition table? | 22:15 |
| jhova | so it's not getting written to disk, going to try a different image | 22:15 |
| TheJulia | try cirros | 22:15 |
| jhova | It should, it's just he base centos cloud image | 22:15 |
| TheJulia | well, it could be a partition image and lack any portion of a boot loader | 22:15 |
| jhova | trying now | 22:18 |
| *** jamielennox|away is now known as jamielennox | 22:20 | |
| *** Sukhdev has quit IRC | 22:29 | |
| *** dguerri is now known as dguerri`away | 22:29 | |
| *** [1]cdearborn has quit IRC | 22:30 | |
| *** bradbeam has quit IRC | 22:38 | |
| jhova | TheJulia: deployed cirros and same issue. | 22:38 |
| TheJulia | :( | 22:39 |
| jhova | is there a default password on the discovery image? | 22:40 |
| TheJulia | that i don't know :( | 22:41 |
| TheJulia | it seems really odd that cirros did the same thing | 22:42 |
| jhova | agreed | 22:43 |
| jhova | banging my head here :) | 22:43 |
| TheJulia | are you sure that the tripleo machines were bootingg from disk and not network? glancing at the build scripts, it looks like overcloud nodes dont have support to boot from disk | 22:48 |
| TheJulia | but that was a qui k glance while im being distracted | 22:48 |
| *** barra204 has joined #openstack-ironic | 22:48 | |
| jhova | RDO manager was setting the capabilities flag in ironic and pulling it I believe | 22:52 |
| jhova | I'm pretty sure it was from disk, because the overcloud-full image was still on the virtual disk when we switched to the ironic standalone setup | 22:53 |
| *** barra204 has quit IRC | 22:53 | |
| TheJulia | what kind of hardware are you running? | 22:54 |
| jhova | Supermicro, LSI RAID cards. | 22:55 |
| openstackgerrit | Jamie Lennox proposed openstack/python-ironicclient: Cleanup session creation https://review.openstack.org/185850 | 22:57 |
| TheJulia | hmmm | 22:58 |
| *** Sukhdev has joined #openstack-ironic | 22:59 | |
| TheJulia | jhova: seems like the opposite problem of the supermicro i put in my home lab, it refuses to oot from the network | 23:01 |
| TheJulia | s/oot/boot/ | 23:01 |
| openstackgerrit | Haomeng,Wang proposed openstack/bifrost: supports redhat yum installation https://review.openstack.org/185294 | 23:07 |
| TheJulia | jhova: out of curiosity did you double check the bios settings? | 23:16 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: Add 'pbr' to the requirements https://review.openstack.org/188204 | 23:16 |
| jhova | TheJulia: In regards to the boot device? | 23:17 |
| TheJulia | and order | 23:18 |
| jhova | TheJulia: yeah, I even went as far as disabling all other boot devices | 23:18 |
| jhova | TheJulia: The raid device is set as the primary boot device | 23:18 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: Add 'pbr' to the requirements and update 'WSME' version https://review.openstack.org/188204 | 23:18 |
| *** chlong has joined #openstack-ironic | 23:19 | |
| TheJulia | can you paste your node info again? | 23:21 |
| jroll | jhova: if tripleo was booting from pxe, there would still be an image on the disk, just no bootloader or kernel or anything | 23:22 |
| openstackgerrit | Merged openstack/bifrost: Add pip to requirements list https://review.openstack.org/187068 | 23:22 |
| jlvillal | JoshNang: jroll: Have you noticed that 'tox -epep8' is failing for ironic-python-agent? | 23:29 |
| jroll | uh, no | 23:30 |
| jlvillal | For example: https://review.openstack.org/#/c/187680/ | 23:30 |
| jroll | so the gate is broken? | 23:30 |
| jlvillal | jroll: I can reproduce it on my system. tox -v -v -r -epep8 | 23:30 |
| jroll | everyone should keep in mind at this point that we don't work on the agent any more than the rest of the community :) | 23:30 |
| jroll | 2015-06-03 17:42:33.405 | pkg_resources.VersionConflict: (pbr 1.0.1 (/home/jenkins/workspace/gate-ironic-python-agent-pep8/.tox/pep8/lib/python2.7/site-packages), Requirement.parse('pbr<1.0,>=0.5.21')) | 23:30 |
| jroll | hmm | 23:30 |
| jhova | TheJulia: sorry, was eating dinner | 23:31 |
| jlvillal | jroll: Not sure what is causing that.... | 23:31 |
| jlvillal | jroll: Is that inside the CoreOS image? | 23:31 |
| jroll | jlvillal: interesting, we don't have pbr in requirements.txt | 23:32 |
| jroll | no, coreos image has nothign to do with pep8 or unit tests | 23:32 |
| jroll | and coreos doesn't ship python, so :P | 23:32 |
| jlvillal | jroll: I noticed that and submitted a patch. But I still get failure. | 23:32 |
| jlvillal | jroll: I saw 'imagebuild' in the command line. So I wasn't sure... | 23:32 |
| jlvillal | jroll: https://review.openstack.org/188204 | 23:32 |
| jroll | jlvillal: we flake8 the imagebuild code | 23:33 |
| jlvillal | jroll: Okay | 23:33 |
| jroll | I'll dig on it | 23:33 |
| jroll | we shouldn't need to manually update requirements, btw | 23:34 |
| jlvillal | jroll: Thanks! | 23:34 |
| *** jrist has quit IRC | 23:34 | |
| jlvillal | jroll: True, if it already exists. But if pbr should be there then doesn't someone have to add it initially? | 23:34 |
| jroll | yeah | 23:34 |
| *** rloo has quit IRC | 23:36 | |
| *** dontalton has quit IRC | 23:37 | |
| jhova | TheJulia: http://pastebin.com/DkYCkfsp | 23:38 |
| jroll | Collecting pbr<1.0,>=0.5.21 (from hacking<0.9,>=0.8.0->-r test-requirements.txt (line 4)) | 23:38 |
| jroll | jlvillal: I guess the bot doesn't update test-reqs | 23:39 |
| * jroll fixes | 23:40 | |
| jlvillal | jroll: Thanks. I was still wandering around in .tox/pep8/lib/ | 23:40 |
| jlvillal | :) | 23:40 |
| *** rwsu has quit IRC | 23:41 | |
| jroll | jlvillal: of course upgrading hacking finds a bunch of errors :) | 23:42 |
| jroll | I feel like I've been down this road | 23:42 |
| jlvillal | jroll: Sorry to send you on a 'yak shaving' mission :( | 23:43 |
| jroll | no worries | 23:43 |
| jroll | I just feel like I've done this work before | 23:43 |
| jroll | jlvillal: mind filing a bug while I hit this stuff? | 23:44 |
| jlvillal | jroll: Sure! | 23:44 |
| jroll | ty! | 23:44 |
| jlvillal | jroll: https://bugs.launchpad.net/ironic/+bug/1461717 | 23:48 |
| openstack | Launchpad bug 1461717 in Ironic "ironic-python-agent failing PEP8 tests" [Undecided,New] | 23:48 |
| *** jrist has joined #openstack-ironic | 23:48 | |
| * jlvillal needs to go home before next meeting.... | 23:48 | |
| *** smoriya has joined #openstack-ironic | 23:48 | |
| jroll | jlvillal: awesome, ty | 23:48 |
| jroll | should be done with this shortly | 23:48 |
| openstackgerrit | Michael Davies proposed openstack/python-ironicclient: Cache negotiated api microversion for server https://review.openstack.org/173674 | 23:52 |
| openstackgerrit | Joshua Harlow proposed openstack/ironic: Use svg as it looks better/scales better than png https://review.openstack.org/188211 | 23:57 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-python-agent: Update hacking and fix errors https://review.openstack.org/188212 | 23:58 |
| jroll | jlvillal: ^^ | 23:58 |
| jroll | cores that last patch fixes IPA gate | 23:58 |
| jroll | JayF: JoshNang NobodyCam whoever else is in the mood for reviewing pep8 fixes | 23:58 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-python-agent: Update hacking and fix hacking violations https://review.openstack.org/188212 | 23:59 |
| jroll | updated commit message | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!