| jroll | oh, nice | 00:00 |
|---|---|---|
| JayF | jroll: is there a doc for setting up IPA with that patch? | 00:00 |
| * jroll sticks this in a shell script | 00:00 | |
| jroll | JayF: I still need to update the 'pad, but essentially just follow the ironic devstack guide, with some localrc changes | 00:00 |
| jroll | JayF: 2 secs | 00:01 |
| adam_g | jroll, that config relies on some slave scripts that come from other places. if you're trying to run it manually, i think you'd do something like: 'tox -e smoke-serial -- $REGEX' | 00:01 |
| jroll | JayF: https://etherpad.openstack.org/p/ipa-devstack | 00:02 |
| jroll | adam_g: cool, thank you! will report back | 00:02 |
| Shrews | jroll: i think you're going to be better off to only run the tests that you're going add until adam_g and myself get all of our changes merged | 00:02 |
| adam_g | jroll, a good starting point would be focus on tempest.scenario.test_baremetal_basic_ops and get that to pass provisioning, since thats the main test we rely on ATM | 00:02 |
| Shrews | it's sort of a mess right now | 00:03 |
| Shrews | although we're replacing that test :) | 00:03 |
| Shrews | err, trying to | 00:03 |
| jroll | Shrews: I want to run the same tests that we currently run in the gate (before *and* after y'all change everything) :) | 00:03 |
| Shrews | jroll: then you're in for an adventure! | 00:03 |
| jroll | :P | 00:03 |
| jroll | I'll start with testr run tempest.scenario.test_baremetal_basic_ops | 00:04 |
| Shrews | we *currently* don't run many tests. that's what we're trying to change | 00:04 |
| jroll | right | 00:04 |
| Shrews | that's a good starting point | 00:04 |
| jroll | nod | 00:04 |
| jroll | I'll likely poke y'all tomorrow or something | 00:04 |
| Shrews | sure | 00:05 |
| jroll | thanks again :) | 00:05 |
| *** mdorman has quit IRC | 00:06 | |
| jroll | romcheg: I think I might be hitting the same network issues as you, did you figure a fix out for that? | 00:37 |
| *** chuckC has quit IRC | 00:39 | |
| *** shakamunyi has quit IRC | 00:43 | |
| * jroll flips the devstack table | 00:48 | |
| *** ellenh has quit IRC | 01:05 | |
| *** christop1eraedo is now known as christopheraedo | 01:17 | |
| openstackgerrit | Yongli He proposed a change to openstack/ironic: Rewrite ironic policy to use the new changes of common policy https://review.openstack.org/97731 | 01:19 |
| jroll | Ran 1 (-1) tests in 327.867s (-370.202s) | 01:27 |
| jroll | PASSED (id=4) | 01:27 |
| jroll | \o/ | 01:27 |
| *** eghobo has quit IRC | 01:33 | |
| *** nosnos has joined #openstack-ironic | 01:50 | |
| *** shakamunyi has joined #openstack-ironic | 01:54 | |
| *** jgrimm has joined #openstack-ironic | 02:07 | |
| *** shakamunyi has quit IRC | 02:07 | |
| *** chuckC has joined #openstack-ironic | 02:15 | |
| *** killer_prince is now known as lazy_prince | 02:22 | |
| *** shakamunyi has joined #openstack-ironic | 02:22 | |
| *** Halacs has quit IRC | 03:04 | |
| *** Halacs has joined #openstack-ironic | 03:08 | |
| *** eghobo has joined #openstack-ironic | 03:37 | |
| openstackgerrit | Zhongyue Luo proposed a change to openstack/ironic: Sync oslo imageutils, strutils to Ironic https://review.openstack.org/111894 | 03:53 |
| *** pcrews has quit IRC | 03:54 | |
| *** ramineni has joined #openstack-ironic | 03:56 | |
| *** bmahalakshmi has joined #openstack-ironic | 04:05 | |
| *** shausy has joined #openstack-ironic | 04:15 | |
| *** rameshg87 has joined #openstack-ironic | 04:18 | |
| *** eghobo has quit IRC | 04:22 | |
| *** eghobo has joined #openstack-ironic | 04:23 | |
| *** Poornima has joined #openstack-ironic | 04:24 | |
| *** sabah has joined #openstack-ironic | 04:28 | |
| *** lazy_prince is now known as killer_prince | 04:29 | |
| *** bvivek has joined #openstack-ironic | 04:43 | |
| *** k4n0 has joined #openstack-ironic | 04:43 | |
| *** jgrimm has quit IRC | 04:49 | |
| openstackgerrit | Zhongyue Luo proposed a change to openstack/ironic: Sync oslo imageutils, strutils to Ironic https://review.openstack.org/111894 | 04:59 |
| *** nikunj2512 has joined #openstack-ironic | 05:01 | |
| *** eguz has joined #openstack-ironic | 05:21 | |
| *** killer_prince has quit IRC | 05:24 | |
| *** eghobo has quit IRC | 05:25 | |
| *** bvivek has quit IRC | 05:26 | |
| *** rakesh_hs has joined #openstack-ironic | 05:26 | |
| *** rameshg87 has quit IRC | 05:36 | |
| *** shakamunyi has quit IRC | 05:42 | |
| *** killer_prince has joined #openstack-ironic | 05:46 | |
| *** killer_prince is now known as lazy_prince | 05:46 | |
| *** pcrews has joined #openstack-ironic | 05:46 | |
| *** rameshg87 has joined #openstack-ironic | 05:47 | |
| *** bvivek has joined #openstack-ironic | 05:51 | |
| *** aswadr has joined #openstack-ironic | 05:58 | |
| *** pcrews has quit IRC | 05:59 | |
| *** eguz has quit IRC | 06:01 | |
| *** shakamunyi has joined #openstack-ironic | 06:09 | |
| *** dtantsur|afk is now known as dtantsur | 06:10 | |
| dtantsur | Morning Ironic! | 06:10 |
| *** shakamunyi has quit IRC | 06:24 | |
| *** shausy has quit IRC | 06:24 | |
| *** shausy has joined #openstack-ironic | 06:25 | |
| *** bvivek has quit IRC | 06:29 | |
| *** bvivek has joined #openstack-ironic | 06:35 | |
| *** radsy has quit IRC | 06:41 | |
| *** ifarkas has joined #openstack-ironic | 06:47 | |
| *** Mikhail_D_ltp has joined #openstack-ironic | 06:47 | |
| *** rakesh_hs2 has joined #openstack-ironic | 06:49 | |
| *** rakesh_hs has quit IRC | 06:49 | |
| *** shakamunyi has joined #openstack-ironic | 06:50 | |
| *** shakamunyi has quit IRC | 06:58 | |
| openstackgerrit | Anusha Ramineni proposed a change to openstack/ironic-specs: Management Interface to firmware update https://review.openstack.org/100842 | 07:00 |
| *** ndipanov has joined #openstack-ironic | 07:02 | |
| openstackgerrit | Nisha Agarwal proposed a change to openstack/ironic-specs: Discover node properties at node-create/node-update https://review.openstack.org/100951 | 07:07 |
| *** rakesh_hs has joined #openstack-ironic | 07:10 | |
| openstackgerrit | A change was merged to openstack/ironic: Sync oslo imageutils, strutils to Ironic https://review.openstack.org/111894 | 07:10 |
| *** rakesh_hs2 has quit IRC | 07:10 | |
| *** Nisha has joined #openstack-ironic | 07:18 | |
| *** rakesh_hs2 has joined #openstack-ironic | 07:23 | |
| *** rakesh_hs has quit IRC | 07:23 | |
| *** mkerrin has quit IRC | 07:25 | |
| *** sabah has quit IRC | 07:26 | |
| *** mkerrin has joined #openstack-ironic | 07:37 | |
| *** rakesh_hs2 has quit IRC | 07:42 | |
| *** rakesh_hs has joined #openstack-ironic | 07:44 | |
| *** rakesh_hs has quit IRC | 07:46 | |
| *** rameshg87 has quit IRC | 07:47 | |
| *** rakesh_hs has joined #openstack-ironic | 07:49 | |
| *** foexle has joined #openstack-ironic | 07:50 | |
| *** shakamunyi has joined #openstack-ironic | 07:55 | |
| *** Nisha has quit IRC | 07:55 | |
| *** jistr has joined #openstack-ironic | 07:56 | |
| *** romcheg1 has joined #openstack-ironic | 07:58 | |
| *** rwsu has quit IRC | 07:59 | |
| *** lucasagomes has joined #openstack-ironic | 08:05 | |
| *** shakamunyi has quit IRC | 08:10 | |
| *** romcheg1 has quit IRC | 08:18 | |
| *** romcheg1 has joined #openstack-ironic | 08:18 | |
| dtantsur | lucasagomes, morning! mind having a look: https://etherpad.openstack.org/p/Ironic-PoCDiscovery-Juno | 08:34 |
| romcheg1 | Morning folks! | 08:34 |
| *** MattMan has joined #openstack-ironic | 08:34 | |
| dtantsur | romcheg1, morning! | 08:35 |
| lucasagomes | morning! romcheg dtantsur | 08:37 |
| lucasagomes | dtantsur, will take a look | 08:37 |
| romcheg1 | lucasagomes, dtantsur: I have quite a long vacation on Aug 8 - Aug 25 so please poke yuriyz during that time about all the migration stuff | 08:38 |
| lucasagomes | romcheg1, oh nice one. Sure | 08:38 |
| dtantsur | romcheg1, ack :) enjoy! | 08:38 |
| lucasagomes | jbjohnso, ack, it's cool to include me I will take a look at the patches soon | 08:40 |
| lucasagomes | jbjohnso, thanks for that | 08:40 |
| Haomeng | lucasagomes: enjoy the vacation:) | 08:40 |
| lucasagomes | Haomeng, morning :) hey it's romcheg that is going on vacations | 08:41 |
| Haomeng | lucasagomes: :) | 08:41 |
| romcheg1 | :) | 08:41 |
| * lucasagomes wish it was me as well :) | 08:41 | |
| romcheg1 | Or even :-P | 08:41 |
| dtantsur | lol | 08:41 |
| Haomeng | click the wrong name:) | 08:41 |
| Haomeng | :) | 08:41 |
| Haomeng | jbjohnso: I have uploaded the test result with your patch, can you help to check why it not working if you are free, thank you. | 08:42 |
| openstackgerrit | Ghe Rivero proposed a change to openstack/ironic: Sync oslo.incubator modules https://review.openstack.org/110941 | 08:45 |
| *** ndipanov has quit IRC | 08:46 | |
| *** ndipanov has joined #openstack-ironic | 08:46 | |
| *** Nisha has joined #openstack-ironic | 08:48 | |
| *** Nisha has quit IRC | 08:52 | |
| lucasagomes | dtantsur, cool stuff! | 08:54 |
| dtantsur | thnx! | 08:54 |
| dtantsur | lucasagomes, now we need only network part for this PoC to work completely | 08:54 |
| lucasagomes | dtantsur, yeah, that's the hairy part. Any news on that side? | 08:55 |
| dtantsur | lucasagomes, not from me. we wanted to discuss it with marios once you're available (likely today) | 08:55 |
| lucasagomes | dtantsur, ah cool alright yeah let's do it | 08:57 |
| *** igordcard has joined #openstack-ironic | 08:57 | |
| *** Alexei_987 has joined #openstack-ironic | 09:00 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Check if boot device is persistent on ipminative https://review.openstack.org/111959 | 09:08 |
| ifarkas | yuriyz, hi, do you have time to chat about https://review.openstack.org/#/c/104850/4/ironic/drivers/modules/drac/client.py ? | 09:21 |
| yuriyz | ifarkas, sorry but not now | 09:21 |
| ifarkas | yuriyz, alright, do you know when you will be available? | 09:22 |
| openstackgerrit | Ghe Rivero proposed a change to openstack/ironic: Migration to oslo.utils library https://review.openstack.org/110596 | 09:23 |
| yuriyz | ifarkas, in two hours | 09:24 |
| ifarkas | yuriyz, thanks, works for me! | 09:24 |
| *** bmahalakshmi has quit IRC | 09:30 | |
| *** bmahalakshmi has joined #openstack-ironic | 09:34 | |
| *** lazy_prince is now known as killer_prince | 09:37 | |
| *** killer_prince is now known as lazy_prince | 09:37 | |
| *** enikanorov has joined #openstack-ironic | 09:44 | |
| *** shakamunyi has joined #openstack-ironic | 09:57 | |
| *** shakamunyi has quit IRC | 10:01 | |
| *** Nisha has joined #openstack-ironic | 10:16 | |
| *** enikanorov__ has joined #openstack-ironic | 10:21 | |
| *** Poornima has quit IRC | 10:21 | |
| *** enikanorov has quit IRC | 10:22 | |
| *** lazy_prince is now known as killer_prince | 10:32 | |
| *** killer_prince is now known as lazy_prince | 10:37 | |
| yuriyz | ifarkas, some comments added, do you have any questions? | 10:38 |
| ifarkas | yuriyz, not anymore, I wanted to discuss your comment importutils but lucasagomes already touched on that | 10:48 |
| ifarkas | yuriyz, thanks anyway! | 10:49 |
| *** ramineni has quit IRC | 11:03 | |
| *** nosnos has quit IRC | 11:10 | |
| *** bmahalakshmi has quit IRC | 11:16 | |
| *** bmahalakshmi has joined #openstack-ironic | 11:16 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add iPXE Installation Guide documentation https://review.openstack.org/106809 | 11:18 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add iPXE support for Ironic https://review.openstack.org/99318 | 11:18 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add iPXE Installation Guide documentation https://review.openstack.org/106809 | 11:20 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add iPXE support for Ironic https://review.openstack.org/99318 | 11:20 |
| *** lucasagomes is now known as lucas-hungry | 11:21 | |
| openstackgerrit | Vladyslav Drok proposed a change to openstack/ironic: Remove gettextutils _ injection https://review.openstack.org/110634 | 11:27 |
| openstackgerrit | Ilya Pekelny proposed a change to openstack/ironic: Test migrations with Alembic, using Oslo.db https://review.openstack.org/111984 | 11:29 |
| *** bmahalakshmi has quit IRC | 11:32 | |
| *** shausy2 has joined #openstack-ironic | 11:38 | |
| *** shausy has quit IRC | 11:38 | |
| dtantsur | brb | 11:46 |
| *** shakamunyi has joined #openstack-ironic | 11:48 | |
| *** shakamunyi has quit IRC | 11:58 | |
| *** nikunj2512 has quit IRC | 12:01 | |
| *** Nisha has quit IRC | 12:10 | |
| *** lucas-hungry has quit IRC | 12:25 | |
| *** lucas-hungry has joined #openstack-ironic | 12:38 | |
| *** lazy_prince is now known as killer_prince | 12:42 | |
| *** igordcard has quit IRC | 12:43 | |
| *** lucas-hungry is now known as lucasagomes | 12:43 | |
| *** k4n0 has quit IRC | 12:50 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Check if boot device is persistent on ipminative https://review.openstack.org/111959 | 12:59 |
| *** jistr has quit IRC | 13:00 | |
| *** jistr has joined #openstack-ironic | 13:03 | |
| *** jasondotstar has joined #openstack-ironic | 13:03 | |
| *** comstud has quit IRC | 13:09 | |
| *** matty_dubs|gone is now known as matty_dubs | 13:11 | |
| *** stevebaker has quit IRC | 13:11 | |
| *** mgagne has quit IRC | 13:12 | |
| *** dhellmann_ is now known as dhellmann | 13:12 | |
| *** mgagne has joined #openstack-ironic | 13:13 | |
| *** mgagne is now known as Guest27295 | 13:13 | |
| *** comstud has joined #openstack-ironic | 13:14 | |
| *** cian_ has quit IRC | 13:14 | |
| *** Madasi has quit IRC | 13:14 | |
| *** dtantsur has quit IRC | 13:14 | |
| *** yuriyz has quit IRC | 13:14 | |
| *** rainya_ has quit IRC | 13:14 | |
| *** Haomeng has quit IRC | 13:14 | |
| *** stevebaker has joined #openstack-ironic | 13:15 | |
| *** jgrimm has joined #openstack-ironic | 13:15 | |
| *** Haomeng has joined #openstack-ironic | 13:15 | |
| *** cian_ has joined #openstack-ironic | 13:15 | |
| *** Madasi has joined #openstack-ironic | 13:15 | |
| *** dtantsur has joined #openstack-ironic | 13:15 | |
| *** yuriyz has joined #openstack-ironic | 13:15 | |
| *** rainya_ has joined #openstack-ironic | 13:15 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Expose {set,get}_boot_device in the API https://review.openstack.org/90151 | 13:22 |
| *** SpamapS has quit IRC | 13:23 | |
| *** SpamapS has joined #openstack-ironic | 13:24 | |
| *** SpamapS has quit IRC | 13:24 | |
| *** SpamapS has joined #openstack-ironic | 13:24 | |
| *** adam_g has quit IRC | 13:25 | |
| *** adam_g has joined #openstack-ironic | 13:29 | |
| *** adam_g has quit IRC | 13:30 | |
| *** adam_g has joined #openstack-ironic | 13:30 | |
| *** jistr has quit IRC | 13:30 | |
| *** jistr has joined #openstack-ironic | 13:30 | |
| romcheg1 | lucasagomes: What do you think about my proposal https://review.openstack.org/#/c/97731/ ? | 13:30 |
| romcheg1 | I'm going to push a little more on it | 13:31 |
| lucasagomes | romcheg1, 1 sec in a call | 13:31 |
| romcheg1 | ok, np | 13:31 |
| *** dhellmann has quit IRC | 13:32 | |
| *** dhellmann has joined #openstack-ironic | 13:33 | |
| jbjohnso | was wondering what people might think of confluent's api, example: http://paste.openstack.org/show/90486/ | 13:34 |
| *** chuck_ has joined #openstack-ironic | 13:35 | |
| *** zul has quit IRC | 13:36 | |
| *** dhellmann has quit IRC | 13:36 | |
| *** dhellmann has joined #openstack-ironic | 13:39 | |
| *** shakamunyi has joined #openstack-ironic | 13:42 | |
| openstackgerrit | Vladyslav Drok proposed a change to openstack/ironic: Remove gettextutils _ injection https://review.openstack.org/110634 | 13:48 |
| *** shakamunyi has quit IRC | 13:49 | |
| Shrews | adam_g: grr... i think we need to go ahead and push forward with setting the rescue flag in tempest. Disabling cinder is causing some tests to fail b/c they aren't using the @services decorator, but there is a check for the rescue feature flag | 13:51 |
| Shrews | and disabling cinder has been +A'd | 13:53 |
| *** shausy2 has quit IRC | 14:00 | |
| *** pcrews has joined #openstack-ironic | 14:02 | |
| *** wendar has quit IRC | 14:04 | |
| *** ndipanov has quit IRC | 14:04 | |
| lucasagomes | romcheg1, back :D | 14:04 |
| *** wendar has joined #openstack-ironic | 14:05 | |
| lucasagomes | romcheg1, hmm yeah that code looks a bit odd indeed | 14:05 |
| romcheg1 | lucasagomes: So I proposed to do a small refactoring instead of just rewriting the same stuff | 14:05 |
| lucasagomes | romcheg1, right... you mean rewriting ironic/common/policy.py ? | 14:06 |
| romcheg1 | yes | 14:07 |
| romcheg1 | to make use of existing oslo code | 14:07 |
| lucasagomes | because mostly of the changes comes from the sync from oslo | 14:07 |
| romcheg1 | After syncing oslo it's quite easy to remove most of our local policies-related code | 14:08 |
| *** wendar has quit IRC | 14:08 | |
| *** wendar has joined #openstack-ironic | 14:08 | |
| romcheg1 | And use new features in oslo.policies | 14:09 |
| lucasagomes | right... well I don't have objections to it... in fact I thought that the patch #97731 was a first step to do it | 14:09 |
| lucasagomes | cause it sync the new oslo code and then adapt our common/policy.py to use it | 14:09 |
| romcheg1 | lucasagomes: Why not perform the refactoring at the same time? | 14:10 |
| romcheg1 | It's a very small change | 14:10 |
| lucasagomes | romcheg1, right yeah can be done as well... I wasn't sure about the size of the changes | 14:10 |
| romcheg1 | I think it's even smaller than the change that is currently there | 14:11 |
| *** shakamunyi has joined #openstack-ironic | 14:11 | |
| romcheg1 | lucasagomes, because it's mostly deleting old code :) | 14:12 |
| lucasagomes | romcheg1, right... I don't have objections really. Maybe, just one about "patches should do only one thing" , so I would argue that we should sync the new one patch and refactor the code in another | 14:12 |
| lucasagomes | but yeah if it's really small, squeezing the changes in one patch seems okish as well for me | 14:13 |
| romcheg1 | lucasagomes: Syncing already requires some refactoring, so it's identical, i think | 14:13 |
| lucasagomes | romcheg1, well yeah that's true | 14:15 |
| *** bvivek has quit IRC | 14:18 | |
| *** mdorman has joined #openstack-ironic | 14:30 | |
| *** nikunj2512 has joined #openstack-ironic | 14:31 | |
| *** shakamunyi has quit IRC | 14:32 | |
| *** rakesh_hs has quit IRC | 14:34 | |
| NobodyCam | morning ironic | 14:35 |
| *** jgrimm has quit IRC | 14:36 | |
| *** nikunj2512 has quit IRC | 14:36 | |
| *** rameshg87 has joined #openstack-ironic | 14:44 | |
| lucasagomes | NobodyCam, yo morning | 14:45 |
| NobodyCam | :) oh I feel like I've been hit by a train :( | 14:47 |
| Shrews | NobodyCam: :( | 14:52 |
| *** Poornima has joined #openstack-ironic | 14:54 | |
| *** chuck_ is now known as zul | 14:54 | |
| *** zul has quit IRC | 14:54 | |
| *** zul has joined #openstack-ironic | 14:54 | |
| *** rameshg87_ has joined #openstack-ironic | 15:00 | |
| *** rameshg87 has quit IRC | 15:01 | |
| NobodyCam | Shrews: yep I now have the travel plague | 15:11 |
| lucasagomes | NobodyCam, gosh u too? | 15:12 |
| lucasagomes | Shrews, morning | 15:12 |
| *** shakamunyi has joined #openstack-ironic | 15:12 | |
| Shrews | hey lucasagomes | 15:13 |
| NobodyCam | lucasagomes: yep... :( this is a nasty cold/flu too | 15:13 |
| lucasagomes | NobodyCam, damn... Now I'm feeling better but on wednesday I was pretty bad | 15:13 |
| lucasagomes | NobodyCam, sore throat and all, couldn't even talk much | 15:13 |
| lucasagomes | NobodyCam, s/wednesday/thursday (tho I was bad on weds as well) | 15:14 |
| NobodyCam | lucasagomes: :) | 15:18 |
| *** shausy has joined #openstack-ironic | 15:18 | |
| NobodyCam | Yea I was mostly feeling ok. I think I pushed my self on sunday, washed the rv and washed / waxed the car and that was to much... monday I felt like I was hit with a large stick | 15:19 |
| Ng | if a machine is being rebuilt and it fails and goes into ERROR state, does the machine get released back, or is it still associated and able to have rebuild called again? | 15:20 |
| Ng | (I am told that nova-bm released the machine immediately) | 15:20 |
| Shrews | Ng: I don't think you can rebuild an instance that isn't ACTIVE | 15:23 |
| jroll | morning ironic :) | 15:26 |
| jroll | Shrews: I got tempest passing :D | 15:26 |
| Shrews | jroll: awesome! | 15:26 |
| Ng | Shrews: interesting, so that would suggest that if a node goes ERROR, you've pretty much lost it permanently? | 15:26 |
| dtantsur | jroll, NobodyCam, Shrews, morning | 15:27 |
| jroll | without any patches too! | 15:27 |
| Shrews | Ng: tbh, i'm not 100% sure what happens if it goes to ERROR | 15:27 |
| Shrews | jroll: scenario test shouldn't require any patches | 15:27 |
| jroll | Shrews: unless my driver code sucks :P | 15:27 |
| Shrews | Ng: https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L436 | 15:28 |
| *** shakamunyi has quit IRC | 15:28 | |
| Shrews | so, must be ACTIVE | 15:28 |
| Ng | Shrews: thanks | 15:32 |
| *** jgrimm has joined #openstack-ironic | 15:32 | |
| jroll | dtantsur: thanks for the feedback on 108457, I responded | 15:33 |
| NobodyCam | morning dtantsur , Ng :) | 15:34 |
| Ng | hey NobodyCam :) | 15:34 |
| *** agordeev has quit IRC | 15:35 | |
| *** agordeev has joined #openstack-ironic | 15:35 | |
| NobodyCam | Ng: how goes the CQB? | 15:36 |
| Ng | NobodyCam: interesting ;) | 15:36 |
| NobodyCam | :) | 15:36 |
| devananda | morning, all | 15:37 |
| Shrews | morning devananda. how ya feelin'? | 15:37 |
| NobodyCam | morning devananda :) /me hopes your feeling better | 15:37 |
| devananda | jroll: awesome! | 15:38 |
| jroll | devananda: :) https://etherpad.openstack.org/p/ipa-devstack | 15:38 |
| devananda | so I still have pneumonia. Will be around today when I'm not sleeping. | 15:38 |
| jroll | :( get all the sleep | 15:38 |
| dtantsur | devananda, morning! get better :( | 15:39 |
| devananda | jroll: that's fantastic! great walkthrough | 15:39 |
| jroll | thanks :) | 15:39 |
| jroll | I'll post it as a docs patch later on | 15:40 |
| devananda | jroll: I'd suggest proposing a patch to devstack to do all the swift-enablement separately | 15:40 |
| jroll | devananda: yeah, I was waiting for someone to call me out on that :P | 15:40 |
| devananda | jroll: as that is generally useful now for both IPA and iLO drivers, and sholuldn't be harmful to IPMI tests | 15:40 |
| jroll | indeed | 15:40 |
| devananda | jroll: then propose the others as a job to infra, initially either experimental or non-voting, in ironic and IPA, once the devstack changes are landed | 15:41 |
| jroll | yep | 15:41 |
| devananda | jroll: feel free to create / update wiki pages for this, too | 15:41 |
| jroll | yeah, planning on it | 15:41 |
| devananda | jroll: but please don't propose doc changes that include non-committed code references | 15:41 |
| devananda | (or WIP the doc change until the code lands) | 15:42 |
| openstackgerrit | Vladyslav Drok proposed a change to openstack/ironic: Add driver name on driver load exception https://review.openstack.org/112049 | 15:42 |
| *** foexle has quit IRC | 15:43 | |
| jroll | devananda: ok, sounds good | 15:44 |
| *** ifarkas has quit IRC | 15:44 | |
| Shrews | adam_g: FYI, i *finally* duplicated your smoke test results. 0 failures | 15:45 |
| Shrews | concurrency = 2 | 15:45 |
| NobodyCam | was there a horizon patch with the starting of the ironic tab/wireframe? | 15:45 |
| jroll | NobodyCam: mrda-away knew of someone with some POC code, but idk if it's up anywhere | 15:46 |
| *** dtantsur is now known as dtantsur|afk | 15:47 | |
| NobodyCam | ack :) | 15:47 |
| devananda | NobodyCam: not afaik. jcoufal has wireframes and posted them to the ML. mrda knew of someone else who had a POC code (possibly unrelated to cjoufal's work), but I didn't see it | 15:47 |
| jroll | maybe mikal knows where that's at :) ^ | 15:47 |
| NobodyCam | yea I just did a quick look thru the horizon que and didn;t see anything | 15:48 |
| Shrews | NobodyCam: nothing from the red hat guy: https://review.openstack.org/#/q/project:openstack/horizon+owner:%22Jaromir+Coufal+%253Cjcoufal%2540redhat.com%253E%22,n,z | 15:48 |
| devananda | seatch the ML archives under [TripleO] | 15:48 |
| devananda | or [Tuskar] | 15:48 |
| *** killer_prince is now known as lazy_prince | 15:52 | |
| *** shausy has quit IRC | 15:57 | |
| *** jistr has quit IRC | 16:00 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Fix self.fields on API Chassis object https://review.openstack.org/112055 | 16:00 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Fix self.fields on API Port object https://review.openstack.org/112056 | 16:00 |
| Shrews | ALL: Just FYI, cinder is now disabled in the ironic jobs. Be on the lookout for any strange errors. Not expecting any, but you never know. | 16:00 |
| lucasagomes | devananda, jroll morning | 16:00 |
| *** matty_dubs is now known as matty_dubs|lunch | 16:01 | |
| lucasagomes | Shrews, heh I always disable it locally here | 16:01 |
| lucasagomes | Shrews, usually when I re-run stack.sh after an unstack.sh cinder fails with something weird | 16:01 |
| lucasagomes | anyway... seems unstable | 16:02 |
| *** romcheg1 has left #openstack-ironic | 16:02 | |
| Shrews | lucasagomes: i keep it disabled too | 16:04 |
| *** Mikhail_D_ltp has quit IRC | 16:10 | |
| *** rameshg87 has joined #openstack-ironic | 16:11 | |
| *** openstackstatus has quit IRC | 16:17 | |
| *** openstack has joined #openstack-ironic | 16:18 | |
| *** ellenh has joined #openstack-ironic | 16:18 | |
| openstackgerrit | Ramakrishnan G proposed a change to openstack/ironic-specs: iLO Virtual Media iSCSI Deploy Driver https://review.openstack.org/97744 | 16:34 |
| rameshg87 | NobodyCam, JayF, devananda, jroll, request you to take a look at ^^ ilo deploy driver spec https://review.openstack.org/#/c/97744/ | 16:36 |
| JayF | I'm already halfway through the diff :P | 16:36 |
| rameshg87 | i have just modified as per what we agreed upon in yesterday's meeting | 16:36 |
| rameshg87 | thanks JayF :-) | 16:36 |
| *** chuckC has quit IRC | 16:41 | |
| *** rwsu has joined #openstack-ironic | 16:43 | |
| devananda | lucasagomes: thinking about semver, i should actually tag this client as 0.2, right? | 16:44 |
| devananda | lucasagomes: i mean, we added several new features, not just bug fixes | 16:45 |
| lucasagomes | devananda, oh yeah that's true | 16:45 |
| devananda | lucasagomes: based on my release notes, it has 4 new commands or subcommands | 16:45 |
| lucasagomes | yeah we added few things on that release | 16:45 |
| lucasagomes | yup | 16:45 |
| devananda | k, bumping | 16:45 |
| devananda | i can't update the existing tag, but i can push a 0.2.0 with a new message | 16:46 |
| JayF | devananda: shouldn't you be somewhere recovering from pnemonia? :-C | 16:47 |
| devananda | JayF: I am laying under a blanket, just trying to be soewhat useful when I dont feel like sleeping | 16:47 |
| JayF | Just feel better man, that stuff is rough and I think quite a few folks who were at the mid-cycle are down | 16:48 |
| lucasagomes | heh yeah, that aircon was like siberia style man | 16:48 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Updated from global requirements https://review.openstack.org/111524 | 16:50 |
| *** Mikhail_D_ltp has joined #openstack-ironic | 16:53 | |
| openstackgerrit | Nisha Agarwal proposed a change to openstack/ironic-specs: Discover node properties for iLO driver https://review.openstack.org/103007 | 16:59 |
| *** matty_dubs|lunch is now known as matty_dubs | 17:00 | |
| *** Nisha has joined #openstack-ironic | 17:01 | |
| lucasagomes | aight I will call it a day | 17:03 |
| lucasagomes | have a good night everybody | 17:03 |
| lucasagomes | devananda, NobodyCam hope you get better! | 17:03 |
| *** rameshg87 has quit IRC | 17:11 | |
| *** rameshg87_ has quit IRC | 17:12 | |
| NobodyCam | rameshg87_: night lucasagomes | 17:12 |
| NobodyCam | gah | 17:12 |
| NobodyCam | night lucasagomes | 17:12 |
| *** lucasagomes is now known as lucas-afk | 17:14 | |
| *** chuckC has joined #openstack-ironic | 17:16 | |
| *** Alexei_987 has quit IRC | 17:19 | |
| *** Poornima has quit IRC | 17:24 | |
| NobodyCam | rameshg87 reviewed. one nit / one concern | 17:44 |
| JayF | NobodyCam: commented on your comment :P | 17:46 |
| JayF | NobodyCam: tl;dr: that limitiation only exists in the iscsi version of that driver. the IPA version will be fine, and is depedant on that spec :) | 17:46 |
| * JayF hopes everyone only uses the ilo-ipa driver version | 17:46 | |
| JayF | :P | 17:46 |
| jroll | :D | 17:46 |
| NobodyCam | :) | 17:46 |
| NobodyCam | JayF: :) very good... | 17:47 |
| NobodyCam | yea thats a severe impact for guys wanting to run windows | 17:48 |
| NobodyCam | not that we support that today | 17:48 |
| adam_g | romcheg, https://review.openstack.org/#/c/111859/ | 17:49 |
| jroll | JayF: how is that limitation resolved with IPA? | 17:50 |
| JayF | jroll: boot from disk support | 17:51 |
| JayF | jroll: pxe/iscsi has no support for boot from disk, so for iLo driver, you have to provide an attached boot media via OOB to load the kernel/ramdisk | 17:51 |
| jroll | oh, right | 17:51 |
| *** lucas-afk has quit IRC | 17:56 | |
| Shrews | adam_g: ugh, yeah, we need to push the rescue=False stuff now | 17:57 |
| Shrews | http://logs.openstack.org/49/112049/1/check/check-tempest-dsvm-ironic/0378e79/console.html#_2014-08-05_16_36_09_834 | 17:58 |
| adam_g | Shrews, did something change? | 17:58 |
| Shrews | adam_g: yeah. cinder is disabled now | 17:58 |
| jroll | why is cinder required to test rescue apis? | 17:59 |
| JayF | because I think rescue uses boot from block | 17:59 |
| jroll | grr | 17:59 |
| adam_g | Shrews is the ironic gate broken now because of it? | 17:59 |
| Shrews | adam_g: it appears so | 17:59 |
| jroll | yay | 18:00 |
| jroll | got a bug number? | 18:00 |
| Shrews | making one now | 18:00 |
| jroll | and/or the review number for the "disable cinder" change? | 18:00 |
| jroll | ok | 18:00 |
| jroll | updating the whiteboard | 18:00 |
| adam_g | Shrews, the ironic jobs were failing for every jenkins check in 105599 :| | 18:03 |
| Shrews | adam_g: yeah, part of that was the devstack bug about not having libssl-dev | 18:03 |
| Shrews | which fixed the one ironic test | 18:03 |
| Shrews | but it was already approved by the time i was looking into the new failures | 18:04 |
| Shrews | https://bugs.launchpad.net/devstack/+bug/1353027 | 18:06 |
| devananda | JoshNang: https://blueprints.launchpad.net/ironic/+spec/swift-temp-urls is complete, right? | 18:08 |
| jroll | devananda: tis | 18:09 |
| openstackgerrit | Jarrod Johnson proposed a change to stackforge/pyghmi: Change to name-only lookups in RAKP https://review.openstack.org/112096 | 18:09 |
| JoshNang | devananda: yes and no. i had direct url support mentioned in the spec, but found out it is way more complex than i originally thought (5 different formats just for swift, keystone, etc), so that wasn't in the merged patch | 18:09 |
| Shrews | https://review.openstack.org/112097 | 18:09 |
| jroll | anybody want to review some devstack patches? :) https://review.openstack.org/#/c/112095/ and https://review.openstack.org/#/c/108457/ | 18:09 |
| *** lazy_prince is now known as killer_prince | 18:09 | |
| *** Mikhail_D_ltp has quit IRC | 18:10 | |
| jroll | Shrews: maybe make a note for people not to +A that until ironic tests pass :| | 18:10 |
| JoshNang | devananda: direct url makes the conf easier (1 required conf vs 3), but glance's code to manage swift direct_urls is ~400 lines, so i'd lean towards leaving it out and if we really want it, integrate it into glance. | 18:10 |
| Shrews | jroll: ++ | 18:11 |
| Shrews | nothing like breaking the gate to make one feel worthwile | 18:13 |
| Shrews | worthwhile, too | 18:13 |
| devananda | JoshNang: ok, so it's as done as it's going to be, and it's done *enough* for our needs -- but what's outlined in the spec is nto actually finished | 18:13 |
| JoshNang | devananda: correct | 18:13 |
| jroll | Shrews: can't blame yourself for a patch getting approved too soon :P | 18:14 |
| devananda | JoshNang: I think we should reflect that somewhere. Possibly in an annotation to the spec, so that anyone looking back at it (after Juno is done) understansd what whas actually done | 18:14 |
| JoshNang | devananda: wfm. i'll get a patch up later today | 18:14 |
| JoshNang | i can update the blueprint as well | 18:14 |
| devananda | JoshNang: thanks. that should be easy for us to approve | 18:14 |
| JoshNang | devananda: np | 18:15 |
| devananda | fwiw, I think https://blueprints.launchpad.net/ironic is current now | 18:15 |
| devananda | I've tried to have it as focused on Juno as possible | 18:18 |
| * devananda returns to AFK status | 18:19 | |
| jroll | Shrews: do you know how to set things that I would normally set in localrc, when using devstack-gate? | 18:23 |
| jroll | I want to set up a job similar to https://github.com/openstack-infra/config/blob/master/modules/openstack_project/files/jenkins_job_builder/config/devstack-gate.yaml#L542 | 18:23 |
| jroll | but with e.g. IRONIC_DEPLOY_DRIVER=agent_ssh | 18:23 |
| *** bvivek has joined #openstack-ironic | 18:24 | |
| jroll | would I just set DEVSTACK_GATE_IRONIC_DEPLOY_DRIVER=agent_ssh? | 18:24 |
| Shrews | jroll: https://github.com/openstack-infra/devstack-gate/blob/master/devstack-vm-gate.sh#L119-L125 | 18:24 |
| jroll | ah | 18:25 |
| jroll | nice, thanks | 18:25 |
| Shrews | jroll: oh, wait. hrm.... | 18:25 |
| jroll | I love all the cross-project dependencies | 18:25 |
| jroll | I think that will work, I'll do: if [[ "$DEVSTACK_GATE_IRONIC_DEPLOY_DRIVER" == "...... | 18:26 |
| jroll | or something similar | 18:26 |
| Shrews | jroll: yeah, the dependencies still leave me confused at times | 18:27 |
| jroll | indeed | 18:27 |
| *** igordcard has joined #openstack-ironic | 18:27 | |
| jroll | more like... I'm going to have a 3-patch chain here... all on different projects | 18:27 |
| jroll | Shrews: one last thing to bug you about, for the agent driver, should I clone tempest-dsvm-ironic, tempest-dsvm-virtual-ironic, or both? | 18:30 |
| Shrews | jroll: adam_g is working on eliminating one of those... can't remember which | 18:30 |
| jroll | yeah, virtual looks like the newer one, it's got the giant regex of tests to skip | 18:31 |
| Shrews | jroll: we're eliminating that regex | 18:31 |
| adam_g | jroll, right now i'd say cone the virtual-ironic, thats the one that actually sets up nova-compute to use ironic. the others dont | 18:31 |
| jroll | ah, I see | 18:31 |
| jroll | adam_g: what do the others actually do, then? | 18:31 |
| *** rakesh_hs has joined #openstack-ironic | 18:31 | |
| adam_g | jroll, long term we're going to get rid of the virtual-ironic one, and have tempest-dsvm-ironic set up nova. and rename them to something like tempest-dsvm-ironic-pxe, tempest-dsvm-ironic-agent, etc. | 18:31 |
| adam_g | jroll, they deploy ironic api+conductor but nova is set to use the default libvirt/kvm driver. the ironic API gets stressed but not the compute side | 18:32 |
| jroll | hmm, ok | 18:32 |
| jroll | thanks :) | 18:32 |
| adam_g | jroll, those existed before we had support in devstack, devstack-gate, etc. for actually deploying the virt driver | 18:32 |
| jroll | I see | 18:33 |
| jroll | brb | 18:33 |
| *** igordcard has quit IRC | 18:36 | |
| *** penick_ has joined #openstack-ironic | 18:38 | |
| *** ellenh has quit IRC | 18:43 | |
| *** f13o_ has joined #openstack-ironic | 18:45 | |
| *** dguerri`afk is now known as dguerri | 18:50 | |
| devananda | JayF: it looks like none of our not-a-CMDB discussions got captured in the etherpad :( | 19:12 |
| *** aswadr has quit IRC | 19:12 | |
| devananda | adam_g: how close do you think ya'll are to consolidating the existing tempest-* things, and should jroll just wait for that before cloning? | 19:14 |
| adam_g | devananda, thats not going to happen till all of the tempest stuff lands and we're able to throw the whole smoke suite at it. hoping to have all that stuff reviewed by EOW, then we can reconfigure current jobs | 19:15 |
| NobodyCam | devananda: I was just looking for that "not a cmdb" info | 19:16 |
| adam_g | devananda, , or we could drop the old jobs, rename virtual-ironic and then reconfigure the job later to drop regex and run a standard looking tempest job | 19:16 |
| adam_g | before any of that stuff lands | 19:16 |
| devananda | adam_g: if we go that route, we should first make sure the virtual-ironic job is running all our API CRUD tests too | 19:17 |
| devananda | adam_g: so that when we drop the old jobs, we don't actually lose any coverage | 19:17 |
| devananda | adam_g: that actually sounds like the most straight forward thing to do | 19:18 |
| adam_g | devananda, we run all of tempest.api.baremetal | 19:18 |
| adam_g | devananda, ill throw a config patch up now | 19:18 |
| *** rwsu has quit IRC | 19:18 | |
| devananda | adam_g: duh. of course the api.baremetal already run. thanks | 19:22 |
| *** bvivek has quit IRC | 19:22 | |
| *** dhellmann is now known as dhellmann_ | 19:22 | |
| *** dguerri is now known as dguerri`afk | 19:29 | |
| Shrews | ugh. tests still failing | 19:29 |
| Shrews | i don't understand that :( | 19:30 |
| NobodyCam | did anyone take notes on the not a cmdb stuff? | 19:32 |
| *** igordcard has joined #openstack-ironic | 19:34 | |
| *** ellenh has joined #openstack-ironic | 19:35 | |
| *** Mikhail_D_ltp has joined #openstack-ironic | 19:41 | |
| Shrews | adam_g: so, we need a way to set the rescue feature flag when VIRT_DRIVER=ironic, but when libvirt is used as the driver. Any ideas? | 19:42 |
| adam_g | devananda, huh? | 19:42 |
| adam_g | er | 19:43 |
| adam_g | Shrews, huh? | 19:43 |
| Shrews | adam_g: ugh, bad sentence. | 19:44 |
| Shrews | adam_g: check-tempest-dsvm-ironic-nv uses VIRT_DRIVER=libvirt, so the rescue flag does not get set properly | 19:44 |
| openstackgerrit | Nisha Agarwal proposed a change to openstack/ironic-specs: Discover node properties for iLO driver https://review.openstack.org/103007 | 19:45 |
| Shrews | adam_g: so 112097 has no effect, and that test still fails. not sure how to set that flag for that test (and the postgres one) | 19:46 |
| adam_g | Shrews, im putting together a patch right now to consolidate jobs and have that job use VIRT_DRIVER=ironic. | 19:46 |
| Shrews | adam_g: ah, that would solve it | 19:46 |
| adam_g | Shrews, maybe you should revert the patch that disables cinder until thats ready? | 19:46 |
| jroll | Shrews: just, if [[ "$VIRT_DRIVER" == "ironic" ]] && [[ "$DEVSTACK_GATE_IRONIC" == "1" ]] ; might do it? | 19:47 |
| jroll | errr s/ironic/libvirt/ | 19:47 |
| NobodyCam | gah: https://www.openstack.org/vote-paris/Presentation/love-your-pets-don-t-kill-them-transitioning-enterprise-workloads-to-openstack | 19:47 |
| jroll | wat | 19:47 |
| jroll | kill the pets | 19:47 |
| jroll | or the virtual pets, anyway | 19:47 |
| NobodyCam | :-p | 19:47 |
| Shrews | jroll: good idea | 19:47 |
| adam_g | i dont think the DEVSTACK_GATE_* env vars are available to devstack when its running on the slave | 19:48 |
| adam_g | devstack-gate*.sh converts those to appropriate devstack values in the localrc thats used there | 19:48 |
| jroll | ah, right, I was thinking this patch was in devstack-gate | 19:49 |
| jroll | but like, could still do something similar no? | 19:49 |
| Shrews | jroll: but where? | 19:49 |
| Shrews | maybe the revert is the thing to do | 19:49 |
| jroll | yeah, idk | 19:50 |
| *** rwsu has joined #openstack-ironic | 19:54 | |
| Shrews | revert up: https://review.openstack.org/112122 | 19:59 |
| openstackgerrit | Jarrod Johnson proposed a change to stackforge/pyghmi: Change to name-only lookups in RAKP https://review.openstack.org/112096 | 20:00 |
| *** dhellmann_ is now known as dhellmann | 20:02 | |
| lifeless | adam_g: devstack-gate*.sh runs on the slave | 20:03 |
| lifeless | adam_g: so they should be available | 20:03 |
| *** Nisha has quit IRC | 20:03 | |
| *** f13o_ has quit IRC | 20:03 | |
| *** faizan has joined #openstack-ironic | 20:03 | |
| Shrews | well, let's just revert it for now | 20:04 |
| adam_g | lifeless, maybe they're available, but adding logic into devstack itself to inspect those DEVSTACK_GATE_* env vars isn't the right fix | 20:05 |
| lifeless | adam_g: agreed | 20:06 |
| adam_g | jroll, did you have a d-s-g patch to set the ironic deploy driver? | 20:13 |
| Shrews | so, we're still going to need 112097 to fix this *after* adam's work and when we disable it again, so i'm going to leave that one open | 20:13 |
| faizan | Hi NobodyCam | 20:14 |
| jroll | adam_g: not yet, need to land devstack changes first. might put up a wip in a bit | 20:14 |
| NobodyCam | hi faizan | 20:14 |
| jroll | adam_g: did you want to check it out or? | 20:14 |
| faizan | NobodyCam: if you get time could you please look at uefi spec https://review.openstack.org/#/c/99850 | 20:14 |
| NobodyCam | sure | 20:15 |
| adam_g | jroll, okay. im about to push a patch that reconfigs our jobs and sets that in the job config. i'm using DEVSTACK_GATE_IRONIC_DRIVER there, didnt know if you've already chosen something else | 20:15 |
| faizan | NobodyCam: jroll has already reviewed and given a +2 for this updated spec. | 20:15 |
| jroll | adam_g: wfm, I'm using IRONIC_DEPLOY_DRIVER in devstack but I'm indifferent :) | 20:16 |
| adam_g | jroll, cool. the d-s-g patch can just 'echo "IRONIC_DEPLOY_DRIVER=$DEVSTACK_GATE_IRONIC_DRIVER" >>localrc' or simialr | 20:17 |
| faizan | Hi Devananda: did you get a chance to look at uefi spec after removing dependency on nova related changes? | 20:17 |
| jroll | adam_g: yep, sounrds good | 20:17 |
| Shrews | I have to be afk for a couple of hours. Anyone want to volunteer to babysit 112122 for me? | 20:21 |
| NobodyCam | faizan: are you planing any changes to devstack or disk image builder to support building the uefi enviroment? | 20:22 |
| jroll | Shrews: I'm uninformed but can try | 20:22 |
| *** rakesh_hs has quit IRC | 20:23 | |
| Shrews | jroll: nothing to do really. that should make the ironic tests pass again. if all good, just poke for approval | 20:23 |
| faizan | NobodyCam: no | 20:23 |
| jroll | Shrews: no problem | 20:23 |
| Shrews | jroll: thx. i'll be back in a while and will check in | 20:24 |
| jroll | cool, see ya later :) | 20:24 |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/python-ironicclient: Trim trailing slash and version from endpoint https://review.openstack.org/107715 | 20:26 |
| adam_g | jroll, devananda https://review.openstack.org/#/c/112134/ | 20:26 |
| jroll | adam_g: cool, I guess I should base off of that for my patches? | 20:27 |
| adam_g | jroll, may wanna wait till someone from infra to sanity check it first :) but the approach should stay the same WRT deploy drvier | 20:28 |
| jroll | adam_g: sure, you get the idea though :) | 20:29 |
| *** penick_ has quit IRC | 20:40 | |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Add docs for agent driver with devstack https://review.openstack.org/112136 | 20:41 |
| NobodyCam | faizan: looking st lines 45/46 of http://review.openstack.org/#/c/99850/8/specs/juno/uefi-boot-for-ironic.rst .. if no boot mode is set would it not be better to assume bios mode? ie only attempt a uefi deploy if the admin as set the uefi support setting? | 20:42 |
| *** kevinbenton has quit IRC | 20:44 | |
| jroll | adam_g: found a fairly major typo in https://review.openstack.org/#/c/112134/ | 20:46 |
| jroll | :P | 20:46 |
| faizan | Nobodycam: If not boot_mode is set in nova flavor extra_specs, then scheduler will pick node with any type of "boot_mode" capability | 20:47 |
| NobodyCam | faizan: so a not configured for uefi booting CAN also bios boot with out changing settings? | 20:49 |
| NobodyCam | s/not/node/ | 20:49 |
| faizan | NobodyCam: no it will not be used for bios boot, but since a user has not specified any boot type in nova flavor, it may deploy the image on either bios or uefi capable node | 20:51 |
| faizan | NobodyCam: because the same fs image can be used for both bios and uefi boot | 20:53 |
| *** kevinbenton has joined #openstack-ironic | 20:56 | |
| wanyen | NoBodyCam, Nisha forwarded me her IRC conversation with you. You mention about CMDB, is there any openstack CMDB project? Can you pass teh pointer? | 21:07 |
| *** coolsvap has quit IRC | 21:08 | |
| NobodyCam | wanyen: the only thing I have on htat is https://www.openstack.org/vote-paris/Presentation/ironman-ironic-driver-for-ilo | 21:09 |
| NobodyCam | and that is all the info I have on it | 21:09 |
| jroll | adam_g: you don't have a ds-g patch to use DEVSTACK_GATE_IRONIC_DRIVER yet, do you? | 21:10 |
| wanyen | NoBodyCam, it sounded like Ironic wants to get node proerties from a CMDB. I was wondering whetehr there is a openstack CMDB project and how does Ironic to get access to CMDB outside of Ironic? | 21:11 |
| * devananda steps away again after the TC meeting | 21:11 | |
| *** matty_dubs is now known as matty_dubs|gone | 21:11 | |
| *** penick has joined #openstack-ironic | 21:12 | |
| *** coolsvap has joined #openstack-ironic | 21:12 | |
| NobodyCam | wanyen: I do not know of a actual cmdb for openstack now, I am hoping the talk that the link i posted turns into one | 21:13 |
| NobodyCam | mordred: do you know if there IS a openstack CMDB project? | 21:14 |
| jroll | NobodyCam: I don't think you linked the right talk :P | 21:14 |
| jroll | did you mean https://www.openstack.org/vote-paris/Presentation/cmdb-for-openstack-public-cloud-based-assets | 21:14 |
| NobodyCam | jroll: why yes I did!!! Thank you | 21:15 |
| NobodyCam | :-p | 21:15 |
| jroll | lol | 21:15 |
| wanyen | NobodyCam, if ther eis no openstack cmdb project then what's the rationale of ironic to get access to cmdb? | 21:15 |
| wanyen | s/thee eis/ there is | 21:16 |
| jroll | I don't believe ironic has plans to get node properties from a CMDB? | 21:16 |
| NobodyCam | wanyen: many folks already have a CMDB | 21:16 |
| *** openstackgerrit has quit IRC | 21:16 | |
| NobodyCam | jroll: there was talk about adding support to the cli for importing from a csv, or other cmdb output format | 21:17 |
| adam_g | jroll, no, happy to put one up. didn tknow if there were other knobs you were adding | 21:17 |
| wanyen | NobodyCam, so is the current thinking for ironic to get access to vendor specific cmdb? | 21:17 |
| jroll | adam_g: all good, just checking :) | 21:18 |
| *** openstackgerrit has joined #openstack-ironic | 21:18 | |
| NobodyCam | wanyen: the other way around, cmdb's could enroll nodes | 21:18 |
| wanyen | NobodyCam, I think node proerty discovery is not the same as CMDB though. Node proeprty discovery is primarily to discover node proerties related to Nova scheduling. It is much confied information than cmdb. | 21:19 |
| mordred | NobodyCam: there is not | 21:19 |
| mordred | NobodyCam: if openstack wanted to grow something, I think a CMDB interface project would be more useful | 21:19 |
| mordred | with pluggable backends that can talk to the different CMDBs | 21:19 |
| jroll | adam_g: https://review.openstack.org/112143 | 21:19 |
| NobodyCam | mordred: TY: oh and have you seen: https://www.openstack.org/vote-paris/Presentation/cmdb-for-openstack-public-cloud-based-assets | 21:20 |
| *** jasondotstar has quit IRC | 21:20 | |
| mordred | I'd heard about it | 21:20 |
| jroll | NobodyCam: import csv; for row in csv: ironic.create_node(...) :) | 21:21 |
| NobodyCam | jroll: yep | 21:21 |
| jroll | NobodyCam: as in, do that outside of ironic | 21:21 |
| jroll | I'm happy to open source our script, but it won't be very useful to anyone else :P | 21:22 |
| NobodyCam | jroll: a simple shell script would do the trick | 21:22 |
| wanyen | NobodyCan, what if a deployer does not have a cmdb? | 21:22 |
| jroll | NobodyCam: right | 21:22 |
| * JayF notes we have a CMDB, but it's awful | 21:22 | |
| adam_g | jroll, cool. gonna try to update the CI etherpad with everything thats going on | 21:23 |
| jroll | adam_g: link me to that? | 21:23 |
| NobodyCam | wanyen: most deployers have a way of tracking their resources | 21:23 |
| adam_g | jroll, https://etherpad.openstack.org/p/IronicCI | 21:24 |
| jroll | thanks | 21:24 |
| wanyen | NodobyCan, Are you saying ironic make the asusmption that a deployer will have a cmdb so that ironic does not need to provide node proerty discovery? | 21:25 |
| jroll | even if NobodyCam isn't saying that, I am | 21:26 |
| jroll | well | 21:26 |
| jroll | more that node property discovery is a feature for a CMDB, not for a provisioning tool | 21:26 |
| wanyen | NobodyCam, this is a fairy new asumption to me. Because node proeprty discovery was one of neded features in past ironic discussion. | 21:27 |
| wanyen | s/neded/needed | 21:27 |
| *** faizan has quit IRC | 21:29 | |
| NobodyCam | wanyen: yes in the past we did say that. However after working through things it was decided that it is really out side the scope for ironic do to discovery | 21:29 |
| *** scubacuda has quit IRC | 21:30 | |
| NobodyCam | wanyen: there are tools out side of Ironic, as just one example Hp's One view. which already dose discovery and can manage firmware and bios setting already | 21:32 |
| wanyen | NobodyCam, IMO some deployers will have cmdb some will not. I am not sure why Ironic needs to rule out node proerty discovery entirely. It would be better to workwith or without CMDB. | 21:34 |
| *** f13o_ has joined #openstack-ironic | 21:37 | |
| NobodyCam | wanyen: it was a group discussion that came up with direction, I am by no means the only one saying this | 21:42 |
| wanyen | NObodyCam, I am not entirely follow the logic. There are tools to do a lot of things incudling bare-metal provisioning. So, I don't see teh logic that there are tools to do x so we don't do x. | 21:43 |
| NobodyCam | I should have taken notes at the meetup | 21:45 |
| *** Mikhail_D_ltp has quit IRC | 21:46 | |
| jroll | if you're deploying enough servers that you need ironic to do so, and don't have a cmdb.... good luck | 21:46 |
| wanyen | NobodyCam, yes. I understand it was the outcome of mid cycle meetup. Not meant to debate with you just try to make sense out of the mid cycle outcome. | 21:46 |
| adam_g | jroll, good eye (112134) | 21:48 |
| jroll | :) | 21:49 |
| NobodyCam | wanyen: does not acutally use any of the data in node.properties, it seems out of place for a provisitioning service like ironic to discover things about a node so that nova can pick which node to use | 21:49 |
| NobodyCam | wanyen: IRONIC ... | 21:49 |
| adam_g | devananda, jroll Shrews okay ive updated https://etherpad.openstack.org/p/IronicCI with all pending work. please update /w anything i've missed | 21:51 |
| adam_g | romcheg, ^ | 21:51 |
| jroll | adam_g: nice, ty | 21:53 |
| wanyen | NobodyCam, "does not use any of the data in node.properties", what are you referring to? | 21:53 |
| NobodyCam | wanyen: the data in properties is not used by ironic | 21:54 |
| NobodyCam | it is handed to nova | 21:54 |
| NobodyCam | nova then selects the node to use | 21:55 |
| wanyen | NobodyCam, I thought node properties such as disk size, memory size,...etc are used for schduling. | 21:55 |
| NobodyCam | yes they are but nova | 21:55 |
| NobodyCam | not Ironic | 21:55 |
| NobodyCam | s/but nova/by Nova/ | 21:56 |
| wanyen | NobodyCam, there are used by the Nova host manager to do bare-metal scheduling. | 21:57 |
| wanyen | NobodyCam, my point is that those data are used for ironic bare-metal scheduling. | 21:58 |
| adam_g | Shrews, if cinder is going to be re-enabled for now, can we just merge https://review.openstack.org/#/c/109300/ instead of https://review.openstack.org/#/c/112097/ ? | 21:58 |
| wanyen | so they are related to Ironic. | 21:58 |
| wanyen | NodobyCam, I will follow up with you via hp email to discuss OneView. | 21:59 |
| NobodyCam | wanyen: Ack :) | 22:00 |
| wanyen | NobodyCam, Jroll, JayF: thanks fo the info regarding mid cycle discussion of node proerty discovery. | 22:00 |
| wanyen | s/fo/for | 22:00 |
| NobodyCam | brb | 22:00 |
| adam_g | romcheg, ping? | 22:10 |
| *** Shrews has quit IRC | 22:12 | |
| *** BadCub has quit IRC | 22:13 | |
| NobodyCam | adam_g: may be late for him | 22:14 |
| *** Shrews has joined #openstack-ironic | 22:14 | |
| *** BadCub has joined #openstack-ironic | 22:15 | |
| *** romcheg1 has joined #openstack-ironic | 22:28 | |
| openstackgerrit | Ghe Rivero proposed a change to openstack/ironic: Sync oslo.incubator modules https://review.openstack.org/110941 | 22:29 |
| NobodyCam | brb | 22:41 |
| *** romcheg1 has left #openstack-ironic | 22:46 | |
| *** jgrimm has quit IRC | 22:55 | |
| *** f13o_ has quit IRC | 23:14 | |
| NobodyCam | and again brb | 23:18 |
| *** mdorman has quit IRC | 23:21 | |
| jroll | adam_g: one thing on this pad, IPA might require more than 512MB RAM | 23:42 |
| adam_g | jroll, eek | 23:42 |
| adam_g | jroll, we lowered from 1024 to 512 specifically to allow more capacity on the devstack slaves to enable parallel tempest tests | 23:43 |
| jroll | that is to say... deva tried running it with 512 and it died in a ball of fire :| | 23:43 |
| jroll | right | 23:43 |
| jroll | I'll make a note and we'll see what we can do | 23:43 |
| jroll | I haven't investigated much | 23:43 |
| adam_g | jroll, yeah, i think he was hitting the same issue we were originally hitting with using tmpfs instead of ramfs | 23:43 |
| jroll | could | 23:43 |
| jroll | could be* | 23:43 |
| adam_g | tmpfs blows up with ENOSPC if it uses >%50 RAM | 23:44 |
| jroll | orly | 23:44 |
| jroll | interesting | 23:44 |
| adam_g | initramfs will happily let you eat all your RAM | 23:44 |
| jroll | JayF: ^^ you might be interested in this | 23:44 |
| NobodyCam | oh thats good to know | 23:44 |
| devananda | jroll: iirc, that was it, yes | 23:46 |
| jroll | devananda: that's what the patches say :P | 23:47 |
| devananda | jroll: also your deploy ramdisk's uncompressed size is quite large | 23:47 |
| jroll | I'm very aware >.> | 23:47 |
| devananda | iirc it's about 2x the current one | 23:47 |
| devananda | ? | 23:47 |
| jroll | dib element may help that | 23:47 |
| devananda | right | 23:47 |
| devananda | also, the dib element itself could be more lean | 23:47 |
| jroll | idk what the size is offhand | 23:47 |
| jroll | probably | 23:47 |
| devananda | that was something we had intended to do "in the fullness of time". which really means putting it on the backburner until it actually matters | 23:48 |
| jroll | right | 23:49 |
| jroll | I might check it out, or see if root-ramfs makes that happier | 23:50 |
| jroll | (or both) | 23:50 |
| NobodyCam | lean sure until you aad "python" to busybox | 23:51 |
| jroll | :P | 23:51 |
| NobodyCam | :p | 23:51 |
| *** chuckC has quit IRC | 23:55 | |
| JayF | Yeah plus because right now our agent stores the image locally before writing | 23:57 |
| JayF | you not only need enough free ram to run all of our image, you also need enough to store the image (as well as logs fwiw) | 23:58 |
| jroll | that should be negligible in the gate, the cirros image used is like 20mb or something | 23:58 |
| JayF | and coreos defaults to using half your ram for your root fs | 23:58 |
| JayF | so something like rootflags=size=256M might help in the 512M case | 23:58 |
| JayF | but I honestly just don't think our image is small enough to run in that space tbh | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!