| *** ijw has quit IRC | 00:01 | |
| *** achanda has quit IRC | 00:03 | |
| rloo | g'night everyone. I'm off; will be back on Thursday. | 00:03 |
|---|---|---|
| *** rloo has quit IRC | 00:03 | |
| NobodyCam | night rloo | 00:05 |
| NobodyCam | JayF: :( | 00:05 |
| NobodyCam | uggh | 00:05 |
| JayF | NobodyCam: talking to them now, doesn't seem like we're likely to be bitten by the exact race he was talking about | 00:06 |
| JayF | NobodyCam: he said it impacts /dev/disk/by-uuid/* links more than anything else | 00:06 |
| JayF | i.e. it says it's done but that link isn't there yet | 00:06 |
| *** Marga_ has quit IRC | 00:08 | |
| *** Marga_ has joined #openstack-ironic | 00:08 | |
| *** Marga__ has joined #openstack-ironic | 00:10 | |
| *** Marga_ has quit IRC | 00:10 | |
| TheJulia | BadCub_Hiding: gray goose would work | 00:11 |
| mrda | Why are you North Americans away tomorrow? | 00:12 |
| JayF | I'm not away tomorrow | 00:12 |
| JayF | afaik there aren't any holidays | 00:13 |
| TheJulia | jroll: so I guess the concern I would have is if its a concern, then initalizaiton of other tests should also likely be a concern | 00:13 |
| *** igordcard_ has quit IRC | 00:13 | |
| TheJulia | but I'm not going to think about it anymore today because I want to actually sleep tonight | 00:13 |
| egon | Sleeping is overrated. | 00:13 |
| NobodyCam | mrda: just BadCub_Hiding as he is going to pick up his car | 00:13 |
| TheJulia | it is, sadly, required | 00:13 |
| mrda | oh | 00:14 |
| * TheJulia actually has to take meds to fall asleep | 00:14 | |
| NobodyCam | everyone else will be here | 00:14 |
| egon | :( | 00:14 |
| * NobodyCam goes back to other things | 00:15 | |
| *** Marga__ has quit IRC | 00:28 | |
| *** naohirot has joined #openstack-ironic | 00:30 | |
| *** jcoufal has joined #openstack-ironic | 00:38 | |
| *** coolsvap|afk is now known as coolsvap | 00:50 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Ensure configdrive isn't mounted in CoreOS ramdisks https://review.openstack.org/167063 | 01:00 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Ensure configdrive isn't mounted in CoreOS ramdisks https://review.openstack.org/167063 | 01:02 |
| *** mtanino has joined #openstack-ironic | 01:09 | |
| *** jerryz has joined #openstack-ironic | 01:19 | |
| *** mtanino has quit IRC | 01:32 | |
| *** korekhov has quit IRC | 01:59 | |
| *** r-daneel has quit IRC | 02:02 | |
| *** korekhov has joined #openstack-ironic | 02:08 | |
| *** ijw_ has quit IRC | 02:19 | |
| *** jcoufal has quit IRC | 02:29 | |
| *** korekhov has quit IRC | 02:29 | |
| *** korekhov has joined #openstack-ironic | 02:30 | |
| *** spandhe has quit IRC | 02:32 | |
| *** kkoski has joined #openstack-ironic | 02:34 | |
| *** korekhov has quit IRC | 02:34 | |
| *** achanda has joined #openstack-ironic | 02:37 | |
| *** killer_prince has quit IRC | 02:39 | |
| *** absubram has joined #openstack-ironic | 02:40 | |
| *** hj-hp has quit IRC | 02:41 | |
| *** ramineni has joined #openstack-ironic | 02:42 | |
| *** killer_prince has joined #openstack-ironic | 02:46 | |
| *** killer_prince is now known as lazy_prince | 02:46 | |
| *** achanda has quit IRC | 02:53 | |
| *** achanda has joined #openstack-ironic | 02:56 | |
| naohirot | BadCub_Hiding: Hi, I updated https://wiki.openstack.org/wiki/Ironic/Drivers#Drivers | 03:01 |
| naohirot | BadCub_Hiding: thanks :) | 03:01 |
| *** alex_xu has quit IRC | 03:12 | |
| *** kkoski has quit IRC | 03:16 | |
| *** achanda has quit IRC | 03:24 | |
| *** achanda has joined #openstack-ironic | 03:32 | |
| *** Nisha has joined #openstack-ironic | 03:34 | |
| *** achanda has quit IRC | 03:41 | |
| *** achanda has joined #openstack-ironic | 03:43 | |
| *** spandhe has joined #openstack-ironic | 03:52 | |
| *** achanda has quit IRC | 03:53 | |
| *** achanda has joined #openstack-ironic | 03:59 | |
| *** Marga_ has joined #openstack-ironic | 04:06 | |
| *** Marga_ has quit IRC | 04:06 | |
| *** Marga_ has joined #openstack-ironic | 04:07 | |
| *** spandhe_ has joined #openstack-ironic | 04:12 | |
| *** zhidong has joined #openstack-ironic | 04:13 | |
| *** spandhe has quit IRC | 04:14 | |
| *** spandhe_ is now known as spandhe | 04:14 | |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: ILO: (PART-1) Test cases for RIS Operations https://review.openstack.org/166779 | 04:28 |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: ILO: (PART-1) Test cases for RIS Operations https://review.openstack.org/166779 | 04:31 |
| ramineni | Nisha: ^^ | 04:32 |
| *** coolsvap is now known as coolsvap|afk | 04:44 | |
| *** ChuckC has joined #openstack-ironic | 04:51 | |
| *** sandhya has joined #openstack-ironic | 05:04 | |
| openstackgerrit | Nisha Agarwal proposed stackforge/proliantutils: Adding functionality required by ilo-inspection https://review.openstack.org/164142 | 05:08 |
| Nisha | ramineni ^^^^ | 05:09 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: grub2 bootloader support for uefi boot mode https://review.openstack.org/166192 | 05:09 |
| Nisha | stendulker ^^^^ | 05:09 |
| *** Marga_ has quit IRC | 05:10 | |
| *** harlowja is now known as harlowja_away | 05:22 | |
| *** Nisha_away has joined #openstack-ironic | 05:28 | |
| *** Nisha has quit IRC | 05:28 | |
| *** chenglch has joined #openstack-ironic | 05:33 | |
| *** lazy_prince has quit IRC | 05:37 | |
| *** rameshg87 has joined #openstack-ironic | 05:38 | |
| rameshg87 | good morning ironic | 05:38 |
| *** killer_prince has joined #openstack-ironic | 05:41 | |
| *** killer_prince is now known as lazy_prince | 05:41 | |
| Nisha_away | rameshg87, https://review.openstack.org/164142 | 05:43 |
| rameshg87 | Nisha_away: ack | 05:45 |
| rameshg87 | Nisha_away: i hope you are not really away ;-) | 05:46 |
| Nisha_away | rameshg87, hehe | 05:46 |
| openstackgerrit | PSPradhan proposed stackforge/proliantutils: Update RIS library https://review.openstack.org/163290 | 05:47 |
| mrda | Morning rameshg87 | 05:59 |
| *** ukalifon1 has joined #openstack-ironic | 06:00 | |
| rameshg87 | good afternoon (or rather evening) mrda :) | 06:00 |
| mrda | :) | 06:03 |
| openstackgerrit | PSPradhan proposed stackforge/proliantutils: Update RIS library https://review.openstack.org/163290 | 06:04 |
| *** achanda has quit IRC | 06:15 | |
| *** Nisha_away is now known as Nisha | 06:17 | |
| * Nisha nisha is back | 06:18 | |
| *** ramineni is now known as ramineni-away | 06:18 | |
| *** ramineni-away is now known as ramineni | 06:19 | |
| *** yog__ has joined #openstack-ironic | 06:36 | |
| *** pcaruana has quit IRC | 06:41 | |
| *** chlong has quit IRC | 06:42 | |
| *** Marga_ has joined #openstack-ironic | 06:52 | |
| *** dlpartain has joined #openstack-ironic | 07:00 | |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: Refactor code in RIS https://review.openstack.org/167117 | 07:01 |
| *** chlong has joined #openstack-ironic | 07:03 | |
| *** dlpartain has left #openstack-ironic | 07:09 | |
| *** killer_prince has joined #openstack-ironic | 07:13 | |
| openstackgerrit | Yuiko Takada proposed stackforge/ironic-discoverd: Add supporting generate config files https://review.openstack.org/163281 | 07:13 |
| *** lazy_prince has quit IRC | 07:14 | |
| *** killer_prince is now known as lazy_prince | 07:14 | |
| *** lazy_prince has quit IRC | 07:18 | |
| *** killer_prince has joined #openstack-ironic | 07:20 | |
| *** killer_prince is now known as lazy_prince | 07:20 | |
| openstackgerrit | Yuiko Takada proposed stackforge/ironic-discoverd: Add supporting generate config files https://review.openstack.org/163281 | 07:22 |
| *** lazy_prince has quit IRC | 07:25 | |
| *** lazy_prince has joined #openstack-ironic | 07:30 | |
| *** Marga_ has quit IRC | 07:33 | |
| *** Marga_ has joined #openstack-ironic | 07:33 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add attempt of max retries to power state sync logs https://review.openstack.org/167122 | 07:37 |
| *** pas-ha has joined #openstack-ironic | 07:41 | |
| *** yog__ has quit IRC | 07:41 | |
| *** yog__ has joined #openstack-ironic | 07:41 | |
| openstackgerrit | Ghe Rivero proposed openstack/ironic-python-agent: Use oslo_log lib https://review.openstack.org/162790 | 07:44 |
| openstackgerrit | Ghe Rivero proposed openstack/ironic-python-agent: Sync from oslo.incubator https://review.openstack.org/162791 | 07:44 |
| Haomeng | GheRivero: ping | 07:48 |
| GheRivero | pong | 07:48 |
| Haomeng | GheRivero: I have a question about i18n lib | 07:48 |
| GheRivero | Haomeng: sure. tell me | 07:49 |
| Haomeng | GheRivero: do you know if the new i18n lib from oslo, which support lazy translation or not? | 07:49 |
| GheRivero | yes it does | 07:49 |
| Haomeng | GheRivero: means if it can handle the 'accept-language' header parameters in the request | 07:49 |
| *** Marga_ has quit IRC | 07:49 | |
| Haomeng | GheRivero: that means we can do nothing in our ironic code, i18n lib can hanndle the lazy translation, right? | 07:50 |
| GheRivero | shpuld be possible | 07:50 |
| GheRivero | doc is not very verbose about that, but I guess it will be possible | 07:51 |
| GheRivero | http://docs.openstack.org/developer/oslo.i18n/usage.html#lazy-translation | 07:51 |
| Haomeng | GheRivero: ok, let me check the doc, thank you:) | 07:52 |
| Haomeng | GheRivero: :) | 07:52 |
| Haomeng | GheRivero: but I can not find the enable_lazy() method call in our ironic code | 07:53 |
| Haomeng | GheRivero: so should we add enable_lazy() call to enable it? | 07:53 |
| GheRivero | I was just checking that. No we don't call it :/ but we should | 07:53 |
| Haomeng | GheRivero: agree with you | 07:54 |
| Haomeng | GheRivero: but not sure if "trans_msg = oslo_i18n.translate(msg, desired_locale=my_locale)" should be add also | 07:54 |
| Haomeng | GheRivero: maybe it should be in the restapi handling logic | 07:54 |
| GheRivero | Haomeng: agree | 07:55 |
| Haomeng | GheRivero: :) | 07:56 |
| Haomeng | GheRivero: GheRivero: how do you think if this is feature or not, if yes, can not add in such k3 release | 07:56 |
| Haomeng | GheRivero: maybe a lot of code should be changed to support lazy translate | 07:56 |
| *** Nisha has quit IRC | 07:56 | |
| *** pas-ha has quit IRC | 07:56 | |
| GheRivero | Haomeng: I think is not too much code, but some people will consider it a feature :( | 07:57 |
| *** hyakuhei has joined #openstack-ironic | 07:57 | |
| *** Nisha has joined #openstack-ironic | 07:57 | |
| GheRivero | Haomeng: we can propose a patch and wait to someone too complain... or not | 07:58 |
| *** pas-ha has joined #openstack-ironic | 07:58 | |
| Haomeng | GheRivero: yes, but so far, I have no idea which code should be changed, let me check with other projects | 07:58 |
| openstackgerrit | Ghe Rivero proposed openstack/ironic: Use oslo_log lib https://review.openstack.org/157602 | 07:58 |
| openstackgerrit | Ghe Rivero proposed openstack/ironic: Sync with oslo.incubator https://review.openstack.org/165940 | 07:58 |
| Haomeng | GheRivero: depens on our ironic existing code | 07:58 |
| Haomeng | GheRivero: this is nova code - https://github.com/openstack/nova/blob/f4e3effb72e9bba2dd4c5471447e45e4ee9b529f/nova/api/openstack/wsgi.py#L1168 | 08:00 |
| Haomeng | GheRivero: but we have no such wsgi python code in our ironic api | 08:02 |
| GheRivero | Haomeng: too bad. We must look were to put it in our rest code then | 08:03 |
| Haomeng | GheRivero: yes, so I think this is not a bug level change | 08:04 |
| Haomeng | GheRivero: https://github.com/openstack/ironic/blob/b4f1db88683f5bb9ff0af7159e34c3117aa0beff/ironic/cmd/api.py#L48 | 08:05 |
| Haomeng | GheRivero: we just create a simple wsgi server - wsgi = simple_server.make_server | 08:05 |
| Haomeng | GheRivero: any idea? :) | 08:08 |
| GheRivero | Haomeng: not now. But I'll check on this later and be sure to find one | 08:10 |
| Haomeng | GheRivero: strange, nova code did not call enable_lazy() also | 08:10 |
| Haomeng | GheRivero: ok, thank you | 08:10 |
| Haomeng | GheRivero: no rush:) | 08:10 |
| Haomeng | GheRivero: keystone called - https://github.com/openstack/keystone/blob/ce701f27710c1000d5d7babd3a7bd3fe7e1af083/keystone/server/wsgi.py#L25 | 08:10 |
| *** chlong has quit IRC | 08:12 | |
| Haomeng | GheRivero: another question here | 08:12 |
| GheRivero | Haomeng: truth be told, I give up worrying about i18n a long time ago. It's mostly error prone and almost impossible to find any error/solution in google/internet if is not in English | 08:13 |
| Haomeng | GheRivero: I understand the wsgi should have common code in oslo, but not sure, do you know? if yes, we can run with same common code to support lazy translation | 08:13 |
| Haomeng | GheRivero: yes, i18n is our gap:) | 08:13 |
| Haomeng | GheRivero: :) | 08:15 |
| GheRivero | Haomeng: I remember a couple of cycles ago there was something about a common wsgi, but lost track of it, so I guess they didn't make it | 08:15 |
| Haomeng | GheRivero: ok, got it:) | 08:15 |
| Haomeng | GheRivero: so dont worry:) | 08:15 |
| Haomeng | GheRivero: :) | 08:15 |
| *** ifarkas has joined #openstack-ironic | 08:21 | |
| *** ndipanov has joined #openstack-ironic | 08:28 | |
| *** lazy_prince has quit IRC | 08:28 | |
| *** killer_prince has joined #openstack-ironic | 08:28 | |
| *** killer_prince is now known as lazy_prince | 08:28 | |
| *** dtantsur|afk is now known as dtantsur | 08:33 | |
| dtantsur | Morning! | 08:33 |
| Nisha | dtantsur, morning! | 08:34 |
| dtantsur | o/ | 08:35 |
| *** Nisha_away has joined #openstack-ironic | 08:36 | |
| *** Nisha has quit IRC | 08:36 | |
| *** hyakuhei has quit IRC | 08:38 | |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: Fix exceptions https://review.openstack.org/167139 | 08:45 |
| *** killer_prince has joined #openstack-ironic | 08:47 | |
| *** lazy_prince has quit IRC | 08:47 | |
| *** killer_prince is now known as lazy_prince | 08:47 | |
| *** andreykurilin_ has joined #openstack-ironic | 08:53 | |
| *** Marga_ has joined #openstack-ironic | 08:53 | |
| *** Nisha has joined #openstack-ironic | 08:55 | |
| *** Nisha_away has quit IRC | 08:55 | |
| *** andreykurilin_ has quit IRC | 08:59 | |
| *** zhidong has quit IRC | 09:00 | |
| *** andreykurilin_ has joined #openstack-ironic | 09:00 | |
| *** jistr has joined #openstack-ironic | 09:04 | |
| *** romcheg has joined #openstack-ironic | 09:06 | |
| *** lucasagomes has joined #openstack-ironic | 09:11 | |
| *** derekh has joined #openstack-ironic | 09:15 | |
| *** egon has quit IRC | 09:19 | |
| *** lazy_prince has quit IRC | 09:20 | |
| *** tiagogomes_ has joined #openstack-ironic | 09:21 | |
| *** Marga_ has quit IRC | 09:21 | |
| *** ukalifon1 has quit IRC | 09:21 | |
| *** killer_prince has joined #openstack-ironic | 09:22 | |
| *** killer_prince is now known as lazy_prince | 09:22 | |
| *** egon has joined #openstack-ironic | 09:24 | |
| *** hyakuhei has joined #openstack-ironic | 09:39 | |
| *** killer_prince has joined #openstack-ironic | 09:47 | |
| *** lazy_prince has quit IRC | 09:48 | |
| *** killer_prince is now known as lazy_prince | 09:48 | |
| *** ukalifon has joined #openstack-ironic | 09:48 | |
| *** naohirot has quit IRC | 09:50 | |
| *** achanda has joined #openstack-ironic | 09:51 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add config drive documentation and script https://review.openstack.org/166201 | 09:53 |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: Fallback to RIBCL https://review.openstack.org/167151 | 09:54 |
| ramineni | rameshg87: ^^^ | 09:54 |
| *** achanda has quit IRC | 09:56 | |
| *** lazy_prince has quit IRC | 09:57 | |
| *** killer_prince has joined #openstack-ironic | 09:58 | |
| *** killer_prince is now known as lazy_prince | 09:58 | |
| *** andreykurilin_ has quit IRC | 09:59 | |
| *** andreykurilin_ has joined #openstack-ironic | 09:59 | |
| rameshg87 | ramineni: ack | 10:04 |
| *** chenglch has quit IRC | 10:07 | |
| *** alex_xu has joined #openstack-ironic | 10:07 | |
| devananda | morning, all | 10:08 |
| rameshg87 | devananda: morning | 10:09 |
| rameshg87 | devananda: back to france again ? :) | 10:09 |
| devananda | rameshg87: yup | 10:10 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add root device hints documentation https://review.openstack.org/166801 | 10:11 |
| lucasagomes | devananda, morning, that's very early :) | 10:11 |
| devananda | lucasagomes: nope. it's 11am :) | 10:11 |
| lucasagomes | oh france | 10:11 |
| lucasagomes | just read it now | 10:11 |
| devananda | generating the state diagram still doesn't work for me. dont suppose anyone else has tried tools/states_to_dot.py ? | 10:12 |
| * lucasagomes haven't | 10:14 | |
| devananda | oh. it's misisng a quotation mark :-/ | 10:14 |
| lucasagomes | devananda, btw, once you have a time mind taking a look at the comments at https://review.openstack.org/166201 ? There's some discussion whether the script to generate the configdrive should live under tools (now tools/contrib) or should be in the ironic-client | 10:15 |
| *** pelix has joined #openstack-ironic | 10:15 | |
| lucasagomes | oh :/ it would be good to have unittests for such tools | 10:15 |
| devananda | client | 10:15 |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Fix formatting error in states_to_dot https://review.openstack.org/167159 | 10:16 |
| *** andreykurilin__ has joined #openstack-ironic | 10:17 | |
| *** andreykurilin_ has quit IRC | 10:17 | |
| *** vdrok_afk is now known as vdrok | 10:23 | |
| vdrok | morning ironic | 10:23 |
| *** Nisha has quit IRC | 10:23 | |
| *** spandhe has quit IRC | 10:26 | |
| openstackgerrit | Merged stackforge/ironic-discoverd: More generic option for tuning adding ports https://review.openstack.org/166908 | 10:30 |
| *** andreykurilin_ has joined #openstack-ironic | 10:37 | |
| *** andreykurilin__ has quit IRC | 10:37 | |
| lintan | hi devananda | 10:41 |
| devananda | jroll: how close to accurate is this? http://www.websequencediagrams.com/cgi-bin/cdraw?lz=Tm92YS0-QVBJOiBzZXQgaW5zdGFuY2VfaW5mbwoADg9wcm92aXNpb25fc3RhdGUKQVBJLT5Db25kdWN0b3I6IGRvX25vZGVfZGVwbG95KCkKABMJABoNdXBkYXRlIHB4ZSwgdGZ0cCBjb25maWdzACMMTmV1dHJvbgAkCURIQ1BCT09UABcNb2RlOiBwb3dlciBvbgpOb2RlAC4LREhDUCByZXF1ZXN0CgBIBwAsCG5leHQtc2VydmVyID0gAIE5CQA8BwCBRwthdHRlbXB0cwCBIwVib290AG4Sc2VuZCBhZ2VudCBrZXJuZWwsIHJhbWRpc2ssICYAgVQHAIEUCACBLAVydW5zACoHA | 10:43 |
| devananda | argh. sorry - didn't realize the link was *that* long :( | 10:44 |
| *** andreykurilin_ has quit IRC | 10:45 | |
| *** hyakuhei has quit IRC | 10:47 | |
| *** yuanying has quit IRC | 10:51 | |
| lintan | devananda, do we have a plan to replace ironic's object code by using oslo.versionedobjets in L? | 11:00 |
| devananda | lintan: it has'nt been discussed yet | 11:00 |
| devananda | lintan: but I believe that is the direction we want to go, once oslo.versionedobjects is ready for us to migrate to | 11:01 |
| *** sandhya has quit IRC | 11:03 | |
| lintan | devananda, OK, it will be great if Ironic can make it | 11:04 |
| devananda | lintan: are you volunteering to work on that? :) | 11:05 |
| *** ramineni has quit IRC | 11:05 | |
| lintan | devananda, Yes, for sure. | 11:05 |
| devananda | lintan: great! | 11:05 |
| devananda | lintan: feel free to add it here with any details you'd like -- https://etherpad.openstack.org/p/ironic-liberty-priorities | 11:06 |
| devananda | that is where we're sketching ideas about what to do in Liberty | 11:06 |
| lintan | OK, I get it | 11:06 |
| *** dtantsur is now known as dtantsur|bbl | 11:18 | |
| * rameshg87 goes home | 11:18 | |
| *** rameshg87 has left #openstack-ironic | 11:18 | |
| *** jamielennox is now known as jamielennox|away | 11:45 | |
| * lucasagomes spent 1 hour to find out that it was a cache problem, the new image was in glance but ironic was referring to an old image that only existed in the cache :( :( :( | 11:56 | |
| lucasagomes | not sure, should we check if deploy_{kernel,ramdisk} still exist in glance ? And only them get it from the cache? | 11:57 |
| lucasagomes | :/ | 11:57 |
| Shrews | Good morning Ironicers | 12:03 |
| *** Haomeng|2 has joined #openstack-ironic | 12:05 | |
| TheJulia | good morning | 12:06 |
| *** Haomeng has quit IRC | 12:08 | |
| lucasagomes | TheJulia, Shrews good morning | 12:08 |
| *** chlong has joined #openstack-ironic | 12:12 | |
| *** chlong has quit IRC | 12:15 | |
| *** lucasagomes is now known as lucas-hungry | 12:15 | |
| *** kkoski has joined #openstack-ironic | 12:17 | |
| *** kkoski has quit IRC | 12:21 | |
| *** jcoufal has joined #openstack-ironic | 12:22 | |
| *** athomas has joined #openstack-ironic | 12:23 | |
| *** dtantsur|bbl is now known as dtantsur | 12:26 | |
| *** thiagop has quit IRC | 12:28 | |
| *** jistr is now known as jistr|class | 12:30 | |
| *** chlong has joined #openstack-ironic | 12:32 | |
| *** chlong has quit IRC | 12:35 | |
| *** chlong has joined #openstack-ironic | 12:36 | |
| *** ChuckC has quit IRC | 12:37 | |
| *** chlong has quit IRC | 12:39 | |
| *** Marga_ has joined #openstack-ironic | 12:42 | |
| *** dlpartain has joined #openstack-ironic | 12:43 | |
| *** dlpartain has left #openstack-ironic | 12:43 | |
| *** dprince has joined #openstack-ironic | 12:44 | |
| *** chlong has joined #openstack-ironic | 12:56 | |
| *** kkoski has joined #openstack-ironic | 12:56 | |
| *** subscope has quit IRC | 12:58 | |
| *** ChuckC has joined #openstack-ironic | 13:01 | |
| devananda | lucas-hungry: did glance image have new uuid, or was it new file but same image uuid? | 13:08 |
| devananda | Shrews, TheJulia, g'morning | 13:09 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add a return after saving node power state https://review.openstack.org/167208 | 13:19 |
| *** pas-ha has quit IRC | 13:20 | |
| *** Marga_ has quit IRC | 13:21 | |
| *** rameshg87 has joined #openstack-ironic | 13:27 | |
| lucas-hungry | devananda, glance had a new UUID the old UUID was deleted from glance | 13:29 |
| lucas-hungry | but it was still registered in the node as pxe_deploy_ramdisk/kernel | 13:29 |
| *** lucas-hungry is now known as lucasagomes | 13:29 | |
| lucasagomes | since the old uuid was in the cache | 13:29 |
| lucasagomes | it was using it | 13:29 |
| devananda | lucas-hungry: gotcha. but node instance_info was updated? | 13:29 |
| lucasagomes | the pxe_deploy options are in the driver_info | 13:30 |
| devananda | er, right | 13:30 |
| lucasagomes | so it wasn't updated, after I deleted the cache | 13:30 |
| lucasagomes | it started failing because the image wasn't found | 13:30 |
| lucasagomes | so I updated the references and it works | 13:30 |
| lucasagomes | cache invalidation is hard I know heh | 13:30 |
| devananda | seems like a bug, heh | 13:30 |
| lucasagomes | it was my bad, but took me some time to figure that out | 13:30 |
| lucasagomes | yeah | 13:30 |
| lucasagomes | probably, if it's a glance UUID we should check that out | 13:31 |
| lucasagomes | I mean, if pxe_deploy options point to a glance uuid (now it's possible to point to any http url, or file in the fs) we maybe should check out if that UUID is still available in glance before getting it from the cache | 13:32 |
| *** ChuckC has quit IRC | 13:32 | |
| devananda | TheJulia: random question - wdyt of bifrost's ironic-deploy using dynamically generated inventory (ie, fetch from ironic, rather than from csv) ? | 13:32 |
| *** pas-ha has joined #openstack-ironic | 13:35 | |
| TheJulia | devananda: I like the idea, although I'm not sure the use case would prove to be very useful in that case. If I'm managing a fleet of machines, I could see myself wanting to provision a known set of them, deploy to a known set of them, but not all unprovisioned or all machines in inventory. Naturally, with the right requirements and use cases though, it could be very useful | 13:37 |
| NobodyCam | good morning Ironic-y folk | 13:37 |
| TheJulia | Good morning NobodyCam | 13:37 |
| devananda | TheJulia: indeed. name matching comes to mind :) | 13:38 |
| NobodyCam | morning TheJulia devananda lucasagomes | 13:38 |
| jroll | devananda: flowchart lgtm, just missing create port and "plug vifs" | 13:38 |
| devananda | g'morning, NobodyCam | 13:38 |
| lucasagomes | NobodyCam, good morning :) | 13:38 |
| NobodyCam | :) | 13:38 |
| lucasagomes | jroll, good morning | 13:38 |
| jroll | morning devananda NobodyCam TheJulia and lucasagomes :) | 13:38 |
| devananda | jroll: meh. good enough then | 13:38 |
| NobodyCam | morning jroll | 13:38 |
| TheJulia | devananda: true... search="row5cabinet4" | 13:38 |
| TheJulia | (if that was in the name) | 13:39 |
| jroll | TheJulia: so yeah... parallel tests. since dhcp_factory.DHCPFactory._dhcp_provider is a global-ish, singleton-ish sort of thing... that's going to get racy | 13:40 |
| jroll | would love someone else to sanity check me on that though | 13:40 |
| jroll | https://review.openstack.org/#/c/166903/4/ironic/tests/dhcp/test_factory.py | 13:40 |
| *** romcheg has quit IRC | 13:40 | |
| TheJulia | jroll: that makes sense, I think. I need more coffee to spend cycles on it. | 13:41 |
| *** lazy_prince is now known as killer_prince | 13:41 | |
| jroll | indeed :) | 13:41 |
| NobodyCam | rameshg87: happen to be around? | 13:43 |
| TheJulia | devananda: realistically, it would have to be a dynamic inventory addition... and I just thought of two modules to add. | 13:44 |
| * TheJulia adds another card to her wall | 13:44 | |
| *** r-daneel has joined #openstack-ironic | 13:44 | |
| rameshg87 | NobodyCam: hello | 13:45 |
| rameshg87 | morning | 13:45 |
| NobodyCam | morning rameshg87 :) I just had a question you */might/* know the answer to | 13:46 |
| *** mtanino has joined #openstack-ironic | 13:46 | |
| rameshg87 | NobodyCam: sure .. | 13:46 |
| NobodyCam | do you if anyone is activly workong on https://bugs.launchpad.net/ironic/+bug/1432498 ? | 13:47 |
| openstack | Launchpad bug 1432498 in Ironic "Whole disk image feature is broken in UEFI environment for pxe drivers" [High,Confirmed] | 13:47 |
| rameshg87 | NobodyCam: stendulker from my team had found this is happening | 13:47 |
| NobodyCam | rameshg87: ya I was looking at https://launchpad.net/ironic/+milestone/kilo-rc1 | 13:48 |
| NobodyCam | and that one of the bugs that didn't have a assignee | 13:48 |
| rameshg87 | NobodyCam: he found the whole disk image support had broken both partition-image and whole-disk-image | 13:48 |
| rameshg87 | NobodyCam: but he has fixed the partition-image one | 13:48 |
| rameshg87 | NobodyCam: but he is still investigating, and havn't found out the right fix for whole-disk-image thing | 13:48 |
| NobodyCam | can I tag him as the assignee for the bug? | 13:49 |
| rameshg87 | NobodyCam: yeah i think you can .. | 13:49 |
| rameshg87 | NobodyCam: if he isn't taking it, one of us from the ilo driver team will try to take it up from him | 13:49 |
| NobodyCam | :) I'll do that and dbl check with him when I see him online :) | 13:50 |
| TheJulia | devananda: thinking about it, we would need to store enough metadata in ironic to build configdrives, else we would have to build them in advance.... many possibilities and issues. | 13:50 |
| rameshg87 | NobodyCam: sure, thanks | 13:51 |
| NobodyCam | :) thank you rameshg87 :) | 13:51 |
| rameshg87 | and NobodyCam thanks for the udevsettle thing, it worked :) (i hope you saw that) | 13:52 |
| *** jistr|class is now known as jistr | 13:53 | |
| jroll | I like this page (the full book is great too) https://github.com/csabapalfi/release-it/blob/master/2-stability_antipatterns.md | 13:53 |
| NobodyCam | rameshg87: awesome... thou I saw something from JayF late lastnight about that | 13:53 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add local boot with partition images documentation https://review.openstack.org/166788 | 13:53 |
| NobodyCam | rameshg87: see the bottum of http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2015-03-23.log | 13:55 |
| *** achanda has joined #openstack-ironic | 13:55 | |
| NobodyCam | bottom even :-p | 13:56 |
| * rameshg87 checks | 13:56 | |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Switch to oslo.config https://review.openstack.org/163281 | 13:57 |
| rameshg87 | NobodyCam: oh | 13:57 |
| rameshg87 | NobodyCam: so introduce back sleep again :) | 13:58 |
| rameshg87 | NobodyCam: for the kernel events to get in ? | 13:58 |
| rameshg87 | NobodyCam: but i tried 2-3 times on that hardware in which it failed earlier - it was going through | 13:58 |
| rameshg87 | NobodyCam: yeah may be it depends upon how fast the kernel recieves those events (may be it happened 3-4 times properly on my system) | 13:59 |
| NobodyCam | rameshg87: ya | 13:59 |
| jroll | jay followed up on that and said we probably won't hit that race | 13:59 |
| rameshg87 | jroll: was it to us ? :) | 13:59 |
| NobodyCam | but maybe best to check with JayF if you see him, as he was still looking in to things lastnight | 13:59 |
| rameshg87 | NobodyCam: okay | 13:59 |
| jroll | rameshg87: yes | 14:00 |
| NobodyCam | jroll: ++ very true he did say that | 14:00 |
| jroll | 00:06:31 JayF | NobodyCam: talking to them now, doesn't seem like we're likely to be bitten by the exact race he was talking about | 14:00 |
| jroll | 00:06:46 JayF | NobodyCam: he said it impacts /dev/disk/by-uuid/* links more than anything else | 14:00 |
| jroll | 00:06:54 JayF | i.e. it says it's done but that link isn't there yet | 14:00 |
| NobodyCam | oh that was in todays log | 14:00 |
| NobodyCam | lol | 14:00 |
| *** achanda has quit IRC | 14:00 | |
| NobodyCam | jroll: no word on WhiteBoard? | 14:03 |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Switch to oslo.config https://review.openstack.org/163281 | 14:04 |
| rameshg87 | anyone has some opinion on this - http://lists.openstack.org/pipermail/openstack-dev/2015-March/059738.html ? | 14:07 |
| rameshg87 | i think we never discussed actively on what needs to be done - except in one of the irc meetings | 14:08 |
| rameshg87 | it didn't get anywhere at that time, and we were running after kilo items | 14:08 |
| rameshg87 | since kilo is almost settled - i think we have got to think of what can be done for this - perhaps worth taking in summit if it goes that far .. | 14:09 |
| jroll | NobodyCam: I haven't heard anything, no | 14:09 |
| NobodyCam | I'm checking now | 14:09 |
| lucasagomes | rameshg87, chassis capabilities? | 14:09 |
| rameshg87 | lucasagomes: yeah | 14:09 |
| lucasagomes | rameshg87, afaict, we all agree that chassis can offer some capabilities and that's because it was first introduced in our API | 14:09 |
| lucasagomes | we just haven't used it yet | 14:09 |
| lucasagomes | I mean, coded | 14:10 |
| rameshg87 | lucasagomes: yeah so can we use it ? | 14:10 |
| rameshg87 | lucasagomes: i know many hardware have so many things that can be done through chassis | 14:10 |
| lucasagomes | yeah, well I think so | 14:10 |
| lucasagomes | I don't see why we wouldn't use it | 14:10 |
| rameshg87 | lucasagomes: if people are ready to do it - why no allow chassis operations atleast through a driver-vendor-passthru | 14:10 |
| lucasagomes | it's already in the API and won't be deleted on v1/ | 14:10 |
| lucasagomes | so | 14:10 |
| lucasagomes | rameshg87, but we do, no? | 14:11 |
| lucasagomes | I mean I don't anything against anyone using it | 14:11 |
| rameshg87 | lucasagomes: okay | 14:11 |
| lucasagomes | I even remember dtantsur suggesting to use it when someone wanted to use a script that was doing some chassis operations | 14:11 |
| rameshg87 | lucasagomes: i assume we don't have anyone tried to use chassis in that way, right ? | 14:11 |
| lucasagomes | suggested* | 14:11 |
| rameshg87 | lucasagomes: yes | 14:12 |
| lucasagomes | rameshg87, yes, that's what I think as well | 14:12 |
| rameshg87 | lucasagomes: okay | 14:12 |
| lucasagomes | we just didn't have someone to code anything yet | 14:12 |
| rameshg87 | lucasagomes: have you looked at https://review.openstack.org/#/c/158577/ ? | 14:12 |
| lucasagomes | rameshg87, I haven't, but that was what dtantsur suggested to use driver passthru on chassis no? | 14:13 |
| rameshg87 | lucasagomes: as per the spec, they have come up with a script (which uses a sort-of-pluggable-api) | 14:13 |
| dtantsur | morning to everyone are missed today :) | 14:13 |
| NobodyCam | morning dtantsur :) | 14:13 |
| lucasagomes | dtantsur, morning | 14:13 |
| rameshg87 | lucasagomes: yeah, may be a switch to driver-level-vendor-passthru might be better idea ? | 14:14 |
| dtantsur | lucasagomes, I think it was me, because I pretty much hate adding functionality as scripts (especially with my Red Hat on :) | 14:14 |
| rameshg87 | dtantsur: o/ | 14:14 |
| jroll | we have a driver vendor passthru :) | 14:14 |
| lucasagomes | dtantsur, +1 | 14:14 |
| lucasagomes | rameshg87, +1 | 14:15 |
| lucasagomes | and being on driver level passthru we can even add tests to it | 14:15 |
| jroll | dtantsur: you mean your red fedora? | 14:15 |
| lucasagomes | unittests* | 14:15 |
| rameshg87 | lucasagomes: yeah | 14:15 |
| dtantsur | jroll, definitely :) | 14:15 |
| * lucasagomes looks at his fedora on the wall :) | 14:15 | |
| lucasagomes | with all that being said about scripts, the config drive script (https://review.openstack.org/#/c/166201/) do you guys think it would be better to be part of the ironic CLI? | 14:17 |
| * dtantsur shouts YES | 14:17 | |
| lucasagomes | ok... I will put a patch up for that to see how it looks like | 14:19 |
| dtantsur | lucasagomes, I think we already have a copy-paste of the same thingy in rdo-manager, right? I'd wish we could just rely on the client... | 14:19 |
| lucasagomes | dtantsur, do we? | 14:19 |
| dtantsur | not sure | 14:19 |
| lucasagomes | idk about rdo-manager. We use nova there | 14:19 |
| rameshg87 | lucasagomes: if it was in cli, the node-set-provision-state could accept the directory containing config drive files, no ? | 14:19 |
| rameshg87 | :) | 14:19 |
| lucasagomes | so we don't generate it standalone like that | 14:19 |
| dtantsur | ah maybe | 14:19 |
| *** BadCub_Hiding is now known as BadCub | 14:19 | |
| dtantsur | rameshg87, that's the idea | 14:19 |
| lucasagomes | rameshg87, yeah that's the idea | 14:19 |
| BadCub | Mornin folks | 14:20 |
| NobodyCam | good mornign BadCub | 14:20 |
| dtantsur | :) | 14:20 |
| rameshg87 | okay | 14:20 |
| dtantsur | BadCub, o/ | 14:20 |
| lucasagomes | BadCub, good morning | 14:20 |
| lucasagomes | dtantsur, rameshg87 you guys think that using the same --configdrive option but pointing to a dir would do it? | 14:20 |
| lucasagomes | or would be better to have a separated one? | 14:20 |
| lucasagomes | --generate-configdrive <dir> | 14:20 |
| lucasagomes | something like that | 14:20 |
| * lucasagomes votes for a separated one | 14:20 | |
| rameshg87 | lucasagomes: --configdrive-dir | 14:21 |
| rameshg87 | yeah | 14:21 |
| rameshg87 | a separate one | 14:21 |
| lucasagomes | cool | 14:21 |
| lucasagomes | okie | 14:21 |
| dtantsur | lucasagomes, I don't have a strong opinion on this | 14:21 |
| NobodyCam | brb | 14:24 |
| *** achanda has joined #openstack-ironic | 14:32 | |
| * NobodyCam is back | 14:32 | |
| rameshg87 | i save the state of my devstack vm and restore it back when i want to work again - sometimes all the services are terribly slow | 14:35 |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Switch to oslo.config https://review.openstack.org/163281 | 14:35 |
| rameshg87 | anyone faced the issue ? any suggestions to try ? | 14:35 |
| rameshg87 | do you guys save the state of your devstack vm while going off-work ? | 14:36 |
| rameshg87 | *if you run stack on your laptop* | 14:36 |
| dtantsur | rameshg87, I did some time ago. Usually slowness disappears after some time. | 14:36 |
| rameshg87 | dtantsur: exactly | 14:36 |
| lucasagomes | rameshg87, I have some local env but I shutdown my vms | 14:36 |
| devananda | jroll: the ascii for that graphic: https://github.com/devananda/talks/blob/master/ascii/agent-flow.txt | 14:37 |
| rameshg87 | dtantsur: for the first one hour or so, it's terribly slow | 14:37 |
| dtantsur | ifarkas, lucasagomes, may I ask you both to review this change to discoverd: https://review.openstack.org/#/c/163281/ ? It's pretty big and dangerous | 14:37 |
| lucasagomes | then later I just did a ./rejoin-stack (or something likt that) | 14:37 |
| lucasagomes | you gotta set script /dev/null first too | 14:37 |
| jroll | devananda: neat | 14:37 |
| ifarkas | dtantsur, sure thing | 14:37 |
| devananda | lucasagomes: any chance you'll happen to be in London on Sunday? :) | 14:37 |
| dtantsur | rameshg87, an hour? hmm... I think it's less for me. I however never run stuff on the laptop directly, usually in the vm | 14:37 |
| BadCub | g'mornin devananda | 14:37 |
| lucasagomes | dtantsur, sure will take a look | 14:38 |
| rameshg87 | dtantsur: okay | 14:38 |
| devananda | morning, BadCub | 14:38 |
| lucasagomes | devananda, :( not really r you going to be there? | 14:38 |
| devananda | lucasagomes: yah. in Bristol mon-tue, but going to spend sunday hanging out in london | 14:38 |
| pshige | morning | 14:38 |
| dtantsur | o/ | 14:39 |
| NobodyCam | morning pshige | 14:39 |
| BadCub | g'mornin pshige | 14:39 |
| lucasagomes | devananda, oh sounds pretty good. I would need to catch a flight to go there tho :/ | 14:39 |
| devananda | ugh :( | 14:40 |
| lucasagomes | yeah different islands :( | 14:40 |
| jroll | expense it, it's a business meeting right? | 14:40 |
| jroll | :) | 14:40 |
| *** pas-ha has quit IRC | 14:40 | |
| dtantsur | lol | 14:40 |
| lucasagomes | hah | 14:40 |
| lucasagomes | london is a great city anyway, I should visit it again. I don't even remember the last time I've been there, maybe 2012 | 14:42 |
| *** achanda has quit IRC | 14:42 | |
| *** stendulker has joined #openstack-ironic | 14:42 | |
| dtantsur | lucasagomes, you should visit Brno at least once this year :) | 14:43 |
| *** mjturek1 has quit IRC | 14:43 | |
| lucasagomes | dtantsur, +1 man, it's been a year already | 14:43 |
| lucasagomes | march 17 I remember because of the st patricks | 14:43 |
| dtantsur | kind of, yeah. I just started at RH :) | 14:43 |
| lucasagomes | dtantsur, nice! yeah, brno is great! zebras + beers :) | 14:44 |
| dtantsur | ++ | 14:44 |
| lucasagomes | zebras == ribs in czech | 14:44 |
| dtantsur | actually žebras (if IIRC understands this) | 14:44 |
| dtantsur | * IRC | 14:45 |
| lucasagomes | yeah, well I don't know how to put that thing hah | 14:45 |
| * BadCub heads off to get ready to go get the beloved Bumble Bee from it's long vacation from home | 14:45 | |
| lucasagomes | dtantsur, btw all the config options for discoverd are on that conf.py file right? | 14:46 |
| dtantsur | lucasagomes, except for edeploy plugin | 14:46 |
| lucasagomes | cause that was the reason why we didn't port ironic to olso_config yet, since the config options are all around the code (and it's good cause it's self contained on each module) | 14:46 |
| lucasagomes | we didn;t want to create one file where all the references should live | 14:46 |
| lucasagomes | dtantsur, hmm I see | 14:46 |
| dtantsur | lucasagomes, well, discoverd is MUCH smaller and has much less plugins, so we can afford having all options in our place | 14:47 |
| lucasagomes | yeah | 14:47 |
| *** mjturek1 has joined #openstack-ironic | 14:48 | |
| jroll | lucasagomes: wait, does oslo_config require all the options to be declared in one place? | 14:48 |
| dtantsur | jroll, not necessary, but it requires a way to list them all | 14:48 |
| lucasagomes | jroll, it requires you to return a list of all options | 14:48 |
| dtantsur | jroll, discoverd has always had it's options in one place | 14:48 |
| jroll | oh. ugh. | 14:48 |
| lucasagomes | differently from the old config generation that would look at the code | 14:49 |
| jroll | right | 14:49 |
| lucasagomes | jroll, ceilometer is using oslo_config take a look | 14:49 |
| lucasagomes | jroll, https://github.com/openstack/ceilometer/blob/master/ceilometer/opts.py | 14:49 |
| lucasagomes | x.x | 14:49 |
| dtantsur | jroll, better take a look at discoverd patch :D | 14:49 |
| lucasagomes | yeah or that ^ | 14:49 |
| * dtantsur is good at shamelessly make people review code | 14:49 | |
| jroll | oh my god | 14:49 |
| * jroll has a sad | 14:50 | |
| jroll | dtantsur: you clearly aren't very good or you would have provided a link :) | 14:50 |
| lucasagomes | jroll, one benefit of oslo_config is that you can have different config files for different services | 14:50 |
| NobodyCam | lol | 14:50 |
| lucasagomes | say ironic-api and ironic-conductor for e.g | 14:50 |
| * dtantsur has failed :( | 14:50 | |
| * BadCub is laughing | 14:51 | |
| *** EmilienM|PTO is now known as EmilienM | 14:51 | |
| dtantsur | jroll, https://review.openstack.org/#/c/163281/ | 14:51 |
| jroll | lucasagomes: meh | 14:51 |
| * jroll /ignore dtantsur | 14:51 | |
| lucasagomes | jroll, :) yeah just saying heh | 14:51 |
| lucasagomes | that's why we don't use in ironic | 14:51 |
| lucasagomes | maintaining a hard-coded list of opts is a bit too much | 14:52 |
| *** pas-ha has joined #openstack-ironic | 14:52 | |
| dtantsur | especially with a lot of drivers | 14:52 |
| jroll | well, it's more of a hardcoded list of modules | 14:53 |
| jroll | but yeah. | 14:53 |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Switch to oslo.config https://review.openstack.org/163281 | 14:57 |
| dtantsur | lucasagomes, ifarkas, quick fix, sorry ^^^ | 14:57 |
| jlvillal | Good morning Ironic. I saw far too many of you online at 6:30am Pacific Time when I woke up ;) | 14:57 |
| jlvillal | lucasagomes: Gblame? :) | 14:58 |
| dtantsur | jlvillal, morning :) | 14:58 |
| jlvillal | dtantsur: dobré odpoledne | 15:00 |
| dtantsur | :) | 15:00 |
| lucasagomes | jlvillal, awesome Gwrite for git add everytime now | 15:00 |
| *** ChuckC has joined #openstack-ironic | 15:00 | |
| NobodyCam | morning jlvillal | 15:01 |
| jlvillal | lucasagomes: Cool. I should use that more (Gwrite). I mostly just use Gblame with the going back in time feature. | 15:01 |
| jlvillal | NobodyCam: Good morning. You were one of the early risers I was talking about :) | 15:01 |
| BadCub | g'mornin jlvillal | 15:01 |
| NobodyCam | jlvillal: lol yeppers | 15:02 |
| jlvillal | BadCub: :) Thanks | 15:02 |
| lucasagomes | jlvillal, yeah Gblame is very good too, I just played a bit with it. Didn't need to go back in history yet | 15:03 |
| lucasagomes | but I'm sure I will use it soon | 15:03 |
| NobodyCam | anyone happen to know why Rakesh abandoned https://review.openstack.org/#/c/151268 ? | 15:03 |
| jlvillal | lucasagomes: You can use '~' or '^' for going back in history | 15:03 |
| jlvillal | lucasagomes: and 'o' is nice to actually see the git commit of the line you are on. | 15:04 |
| BadCub | okay, now I am really leaving. Have a good day folks. :-) | 15:04 |
| jlvillal | BadCub: Bye. Are you outside the US today? | 15:04 |
| NobodyCam | jlvillal: nope | 15:05 |
| lucasagomes | dtantsur, why you need to import conf on the main.py? | 15:05 |
| lucasagomes | jlvillal, +1 | 15:05 |
| NobodyCam | BadCub's car has been in the shop for like 6 keeks | 15:05 |
| NobodyCam | weeks even | 15:05 |
| dtantsur | lucasagomes, otherwise the options are now know on startup | 15:06 |
| dtantsur | lucasagomes, e.g. trying to check CONF.discoverd.debug fails | 15:06 |
| lucasagomes | oh | 15:06 |
| jlvillal | NobodyCam: BadCub :Ouch! That is a long time. | 15:06 |
| lucasagomes | gotcha | 15:06 |
| *** achanda has joined #openstack-ironic | 15:06 | |
| NobodyCam | :-p ya | 15:06 |
| *** achanda has quit IRC | 15:06 | |
| *** hj-hp has joined #openstack-ironic | 15:07 | |
| jroll | NobodyCam: what car? | 15:07 |
| NobodyCam | the focus, Ford had to replace the motor | 15:08 |
| jroll | oh yeah, didn't a tune blow it or something? | 15:08 |
| NobodyCam | one of the connectiong rods gave out :( | 15:09 |
| NobodyCam | (while getting a new exaust) | 15:10 |
| jroll | oh huh | 15:10 |
| * jroll has a bad memory then | 15:10 | |
| NobodyCam | it will be nice to rid our selfs of the rental thing | 15:11 |
| NobodyCam | lol then we just have to get my car down from WA | 15:13 |
| jroll | heh | 15:13 |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Update from global requirements https://review.openstack.org/167266 | 15:14 |
| *** ijw has joined #openstack-ironic | 15:19 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient: Clean openstack-common module list https://review.openstack.org/162799 | 15:19 |
| *** ijw has quit IRC | 15:20 | |
| *** Marga_ has joined #openstack-ironic | 15:22 | |
| NobodyCam | oh TheJulia have you seen: https://github.com/dominis/ansible-shell | 15:22 |
| *** romcheg has joined #openstack-ironic | 15:22 | |
| *** ijw has joined #openstack-ironic | 15:22 | |
| jroll | whoa | 15:23 |
| jroll | neat. | 15:23 |
| *** viktors has quit IRC | 15:23 | |
| jroll | I like it | 15:23 |
| NobodyCam | :) | 15:23 |
| *** ijw has quit IRC | 15:23 | |
| *** ijw has joined #openstack-ironic | 15:24 | |
| *** ijw has quit IRC | 15:25 | |
| *** ijw has joined #openstack-ironic | 15:26 | |
| *** yog__ has quit IRC | 15:29 | |
| rameshg87 | folks any idea if we can use logs.openstack.org for posting temporary results | 15:34 |
| rameshg87 | i mean for a third-party-ci | 15:34 |
| * rameshg87 should rather check in infra | 15:34 | |
| NobodyCam | ya I'd infra is the best place to ask that one :) | 15:35 |
| NobodyCam | i'd THINK infra ... | 15:35 |
| NobodyCam | :-p | 15:35 |
| *** BertieFulton has joined #openstack-ironic | 15:35 | |
| rameshg87 | okay | 15:38 |
| NobodyCam | brb | 15:38 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Ensure configdrive isn't mounted in CoreOS ramdisks https://review.openstack.org/167063 | 15:40 |
| *** absubram has quit IRC | 15:40 | |
| * JayF fixed tests and ^ is RFR | 15:42 | |
| *** anderbubble has joined #openstack-ironic | 15:42 | |
| dtantsur | jroll, JayF, mind having a look at https://review.openstack.org/#/c/164440/ ? touches agent deploy | 15:44 |
| JayF | I will right meow | 15:44 |
| dtantsur | meow-meow :) | 15:45 |
| JayF | I was thinking we landed support in the agent for partition images? | 15:45 |
| JayF | Or am I just conflating that with the iscsi+agent support for partition images | 15:45 |
| dtantsur | I think it was bumped, at least the Ironic part... | 15:45 |
| JayF | ah, okay | 15:45 |
| dtantsur | the spec has landed https://review.openstack.org/#/c/137363/ | 15:46 |
| JayF | dtantsur: how do we know if an image is whole disk or not? | 15:46 |
| dtantsur | JayF, there's some capability/property for that IIRC | 15:46 |
| JayF | It seems like that attribute is already set somehow in driver_internal_info | 15:46 |
| dtantsur | aha the patch got bumped: https://review.openstack.org/#/c/162008/ | 15:47 |
| JayF | I'm mainly curious about how the driver_internal_info value gets set | 15:48 |
| JayF | making sure we aren't adding a dependency on a formerly optional property to an image or something similar | 15:48 |
| * dtantsur does not remember... | 15:48 | |
| jroll | I assume it looks at glance metadata? | 15:50 |
| jroll | or actually iirc | 15:50 |
| jroll | it's the lack of kernel/ramdisk on the glance image | 15:50 |
| JayF | +1, assuming confidence in where that data comes from | 15:50 |
| *** hj-hp has quit IRC | 15:51 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add root device hints documentation https://review.openstack.org/166801 | 15:51 |
| *** bradjones has quit IRC | 15:51 | |
| dtantsur | jroll, leaving https://review.openstack.org/#/c/164440/ for you to approve? | 15:51 |
| lucasagomes | jroll, +1 yeah I would assume that as well | 15:51 |
| *** bradjones has joined #openstack-ironic | 15:51 | |
| *** bradjones has quit IRC | 15:51 | |
| *** bradjones has joined #openstack-ironic | 15:51 | |
| JayF | dtantsur: fwiw jroll is the only one of the j's who can approve a thing in Ironic :P | 15:53 |
| jroll | dtantsur: not sure why you need my approval >.> | 15:53 |
| jroll | but I'll look | 15:53 |
| dtantsur | JayF, aha, but you're a core on -specs, aren't you? | 15:54 |
| * jroll +A | 15:54 | |
| JayF | dtantsur: I'm core on specs/IPA but not Ironic | 15:54 |
| JayF | dtantsur: JoshNang is core on IPA but not specs/ironic | 15:54 |
| dtantsur | oh | 15:55 |
| JayF | Ugh, my ability to find etherpads | 16:01 |
| JayF | is greatly diminished with the death of the whiteboard | 16:01 |
| jroll | recovery is in progress | 16:01 |
| jroll | please hold | 16:02 |
| JayF | I've listened to this same 2 minute elevator music song | 16:02 |
| JayF | for two days | 16:02 |
| JayF | :P | 16:02 |
| jroll | well | 16:02 |
| jroll | fungi found it | 16:02 |
| lucasagomes | https://wiki.openstack.org/wiki/Ironic#Etherpads (we should really add the etherpads there) | 16:02 |
| jroll | just needs to restore | 16:02 |
| NobodyCam | ya fungi is working on it | 16:02 |
| *** EmilienM is now known as EmilienM|PTO | 16:02 | |
| JayF | BadCub / Others https://review.openstack.org/#/c/167063/ + https://bugs.launchpad.net/ironic/+bug/1433812 need to get on the review day etherpad | 16:02 |
| openstack | Launchpad bug 1433812 in Ironic "ironic-python-agent silently fails to write a configdrive if a previous drive is found" [High,In progress] - Assigned to Jay Faulkner (jason-oldos) | 16:03 |
| JayF | can I just add it or should someone be prodded? | 16:03 |
| jroll | jfdi | 16:03 |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Add option to delete ports after introspection https://review.openstack.org/166915 | 16:03 |
| jroll | man, I really want to put a module called requirements.txt on pypi | 16:03 |
| jroll | I always forget the -4 | 16:03 |
| NobodyCam | jroll: +++ jdi! | 16:03 |
| jroll | -r | 16:03 |
| jroll | and make it just download a script that executes pip install -r requirements.txt | 16:04 |
| *** romcheg has quit IRC | 16:05 | |
| *** Marga_ has quit IRC | 16:05 | |
| *** jistr has quit IRC | 16:05 | |
| *** romcheg has joined #openstack-ironic | 16:06 | |
| lucasagomes | jroll, create an alias? | 16:06 |
| jroll | lucasagomes: nah, this way everyone else gets the benefits :) | 16:06 |
| lucasagomes | heh fair enuff | 16:06 |
| jroll | also. way too many servers for an alias. :P | 16:06 |
| NobodyCam | JayF: +2'd | 16:08 |
| tiagogomes_ | hi, can someone clarify what are the network requirements for deploying with Ironic? I have installed Ironic in a VM, and the VM is bridged with the host network, which correspods to the office network. I am trying to deploy to a baremetal server that is on the same office network, but I think that the baremetal server is trying to DHCP from the office DHCP server, instead of Neutron's DHCP server. It is mandatory t | 16:11 |
| tiagogomes_ | hat the VM and the baremetal server are in a network without an existing DHCP server? | 16:11 |
| *** penick has joined #openstack-ironic | 16:11 | |
| jroll | tiagogomes_: in general, if there's more than one DHCP server on a network, you're going to have problems | 16:12 |
| NobodyCam | ya | 16:12 |
| NobodyCam | you could exclude the nodes mac from the existing dhcp server but thats not the path to start down | 16:13 |
| tiagogomes_ | ok, then Neutron doesn't create any virtual network or vlan with the baremetal server? | 16:14 |
| JayF | You'd have to have neutron configured to control a physical switch for that to even be possible (not many folks do) | 16:16 |
| *** chlong has quit IRC | 16:16 | |
| JayF | but support for that didn't land in Ironic for K, I believe | 16:16 |
| rameshg87 | folks 1 question about our basic tempest test | 16:16 |
| rameshg87 | https://github.com/openstack/tempest/blob/master/tempest/scenario/test_baremetal_basic_ops.py#L129-L130 | 16:17 |
| rameshg87 | i think we should make it adding floating ip and testing connectivity with that should really be a configurable option | 16:17 |
| rameshg87 | wdyt ? | 16:17 |
| NobodyCam | rameshg87: that seems like a basic operation, what is the use case behind making it optional | 16:18 |
| rameshg87 | NobodyCam: can we do configuration of floating ips on a provider flat network ? | 16:19 |
| * rameshg87 might start asking stupid questions | 16:20 | |
| rameshg87 | NobodyCam: in our ci, i have the hardware lying around in a flat network | 16:20 |
| *** stendulker has quit IRC | 16:20 | |
| rameshg87 | NobodyCam: i just configured devstack for a single provider network, stacked and then ran the test_baremetal_server_ops test | 16:20 |
| tiagogomes_ | JayF do you have any suggestion of how I can try the deployment then? | 16:22 |
| rameshg87 | NobodyCam: i think i should run with an l3 agent if i want to have floating ips | 16:22 |
| JayF | tiagogomes_: you can use the agent deployment method with DHCP provider of none | 16:22 |
| rameshg87 | NobodyCam: that's really a neutron thing - i would like to make it optional :( | 16:22 |
| JayF | tiagogomes_: and configure your DHCP server to boot the deploy ramdisk | 16:22 |
| rameshg87 | NobodyCam: does that make sense ? | 16:22 |
| * JayF bbl | 16:23 | |
| tiagogomes_ | JayF I can't do that, I don't want to mess with the office DHCP server | 16:23 |
| NobodyCam | rameshg87: is this for something like 3rd party testing? | 16:23 |
| rameshg87 | NobodyCam: yes | 16:23 |
| JayF | tiagogomes_: You can't run Ironic without being able to control the DHCP server yourself or without letting Ironic control it | 16:23 |
| JayF | tiagogomes_: the only exception is the iLo drivers that use Virtual Media to transmit the deploy ramdisk, but that only works with some HP hardware | 16:24 |
| NobodyCam | ack then i would okay with it. because in that case your just looking for the drivers results | 16:24 |
| rameshg87 | NobodyCam: yes | 16:24 |
| NobodyCam | rameshg87: ++ | 16:24 |
| NobodyCam | makes sense to me | 16:24 |
| rameshg87 | NobodyCam: if i just verify the connectivity with my private ip, the test runs fine | 16:24 |
| rameshg87 | NobodyCam: just like we do after rebuild - https://github.com/openstack/tempest/blob/master/tempest/scenario/test_baremetal_basic_ops.py#L148 | 16:25 |
| NobodyCam | oh brb... bbt | 16:26 |
| jlvillal | I just saw this: https://bugs.launchpad.net/bugs/1435948 | 16:27 |
| openstack | Launchpad bug 1435948 in Ironic "Incorrect assertEqual statements in unit tests" [Low,Triaged] | 16:27 |
| lucasagomes | devananda, ping re deploy kernel/ramdisk on the flavor | 16:27 |
| lucasagomes | devananda, https://github.com/openstack/nova/blob/master/nova/virt/ironic/patcher.py#L158-L160 before we were thinking about removing this functionality, but now, after the ipa-as-default-ramdisk I think it makes sense to keep it | 16:27 |
| jlvillal | kkoski: Are you going to work on https://bugs.launchpad.net/bugs/1435948 ? If not, I could work on it. | 16:28 |
| lucasagomes | since we can use the same ramdisk for multiple drivers, it would make sense to be able to specify the deploy_{kernel, ramdisk} in the flavor | 16:28 |
| lucasagomes | and we normalized the parameter names too, so you don't need to specify the "pxe_" namespace anymore | 16:28 |
| *** absubram has joined #openstack-ironic | 16:29 | |
| lucasagomes | can pass deploy_{kernel, ramdisk}=<id> for both agent and pxe drivers | 16:29 |
| jroll | why not just specify on the node? | 16:30 |
| lucasagomes | jroll, you could do both | 16:30 |
| lucasagomes | the driver respects it | 16:30 |
| jroll | yeah | 16:30 |
| lucasagomes | I mean, if it's in the flavor the driver would write it to the node | 16:30 |
| lucasagomes | if it's not in the flavor the driver just don't do anything | 16:31 |
| lucasagomes | jroll, makes sense? | 16:31 |
| jroll | yeah, just think it's odd to have two ways to configure the same thing | 16:31 |
| jroll | slash confusing | 16:31 |
| lucasagomes | right | 16:31 |
| NobodyCam | jlvillal: thats been open for a while I beleieve | 16:31 |
| jroll | update node.driver_info, why isn't it using my new ramdisk | 16:32 |
| lucasagomes | yeah I see that, that's why I'm asking. If it's confusing we should remove it then | 16:32 |
| jroll | oh it's on the flavor | 16:32 |
| lucasagomes | I just found it a bit useful to have in the flavor | 16:32 |
| lucasagomes | since it's a "global" way to set a deploy ramdisk | 16:32 |
| jroll | idk, it would be confusing to me | 16:32 |
| jroll | I guess, idk, that's what automation is for :) | 16:32 |
| rameshg87 | dtantsur: please have a look at this bug: https://bugs.launchpad.net/ironic/+bug/1435959 | 16:32 |
| openstack | Launchpad bug 1435959 in Ironic "rebuild fails with ilo* drivers" [Undecided,New] | 16:32 |
| lucasagomes | jroll, right | 16:32 |
| lucasagomes | ok better remove from flavor then | 16:32 |
| rameshg87 | dtantsur: please prioritize as high and for rc1. a critical bug in ilo driver :( | 16:33 |
| lucasagomes | we should have removed it this cycle | 16:33 |
| rameshg87 | fix rolling right-away | 16:33 |
| lucasagomes | that was kept to keep compability with icehouse | 16:33 |
| jroll | well | 16:33 |
| dtantsur | rameshg87, I can bump it, but please do the triaging: I can't confirm bug in ilo | 16:33 |
| dtantsur | rameshg87, "triaging" meaning "finding the cause" here | 16:33 |
| jroll | lucasagomes: I think I tried to, but we didn't deprecate it properly. that may have been juno though. | 16:33 |
| *** ukalifon has quit IRC | 16:34 | |
| rameshg87 | dtantsur: i have tested the fix and patch rolling in a minute | 16:34 |
| lucasagomes | jroll, heh yeah | 16:34 |
| lucasagomes | np we remove it from nova in L | 16:34 |
| dtantsur | rameshg87, please update the bug description too, so that I can mark it as triaged | 16:34 |
| jroll | lucasagomes: we still could remove it this cycle | 16:34 |
| lucasagomes | jroll, will nova accept a patch for that? | 16:34 |
| rameshg87 | updating | 16:34 |
| lucasagomes | if so I can put a patch up removing it | 16:34 |
| lucasagomes | well I put a patch up and then they can merge it this cycle or the next | 16:34 |
| jroll | lucasagomes: idk, if they don't then the patch waits til L | 16:35 |
| lucasagomes | doesn't matter really | 16:35 |
| jroll | yeah | 16:35 |
| lucasagomes | cool ok will do that | 16:35 |
| lucasagomes | jroll, thanks :0 | 16:35 |
| lucasagomes | :) | 16:35 |
| jroll | np :) | 16:35 |
| jlvillal | NobodyCam: The email I got said "NEW", maybe that is the state? | 16:36 |
| rameshg87 | dtantsur: done | 16:36 |
| dtantsur | thnx | 16:36 |
| TheJulia | JayF: fyi, I've been using a fairly recent coreos ipa image from yesterday evening and have my confgdrives generally seem to be writing out without any issues, although I'm still passing the coreos.configdrive=0 setting by default. | 16:41 |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Attach ilo_boot_iso only if node is active https://review.openstack.org/167317 | 16:42 |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Attach ilo_boot_iso only if node is active https://review.openstack.org/167317 | 16:43 |
| NobodyCam | jlvillal: oh ya that one is new | 16:44 |
| NobodyCam | :-p | 16:44 |
| jlvillal | NobodyCam: :) | 16:45 |
| *** ijw has quit IRC | 16:46 | |
| NobodyCam | TheJulia: JayF also put up this https://review.openstack.org/167063 | 16:46 |
| TheJulia | I spotted that earlier this morning :) | 16:47 |
| TheJulia | or was it last night | 16:47 |
| TheJulia | heh | 16:47 |
| NobodyCam | :) | 16:48 |
| rameshg87 | folks please have a look at https://review.openstack.org/167317 when you get some time - a small fix but a pretty important one :) | 16:49 |
| rameshg87 | good night ironic, see you tomorrow | 16:49 |
| NobodyCam | night rameshg87 | 16:49 |
| TheJulia | goodnight | 16:49 |
| *** rameshg87 has quit IRC | 16:49 | |
| *** spandhe has joined #openstack-ironic | 16:49 | |
| *** bandicot has joined #openstack-ironic | 16:50 | |
| * TheJulia crosses her fingers because she just turned on cleaning | 16:50 | |
| NobodyCam | TheJulia: woo ... /me holds breath | 16:50 |
| *** vdrok is now known as vdrok_afk | 16:51 | |
| JoshNang | :O | 16:51 |
| *** hyakuhei has joined #openstack-ironic | 16:52 | |
| * TheJulia taps her foot waiting for the node to finish initial deployment JUST so she can delete the node | 16:52 | |
| *** Nisha has joined #openstack-ironic | 16:57 | |
| TheJulia | well, so far so good | 16:57 |
| jlvillal | NobodyCam: Darn, fixing the tests found one failed test :( | 17:00 |
| TheJulia | An ipa.debug option would rock :) | 17:00 |
| kkoski | jvillal: Are you talking about https://bugs.launchpad.net/ironic/+bug/1435948? | 17:00 |
| openstack | Launchpad bug 1435948 in Ironic "Incorrect assertEqual statements in unit tests" [Low,Triaged] - Assigned to John L. Villalovos (happycamp) | 17:00 |
| jlvillal | kkoski: Yep | 17:00 |
| kkoski | kk | 17:00 |
| jlvillal | kkoski: I commented in the bug about failure. Investigating now. | 17:00 |
| kkoski | I was just about to take a look at fixing it, but it sounds like you've got it handled | 17:01 |
| dtantsur | g'night folks, see you tomorrow | 17:01 |
| *** dtantsur is now known as dtantsur|afk | 17:01 | |
| NobodyCam | night dtantsur|afk | 17:01 |
| jlvillal | kkoski: I pinged you earlier :) | 17:01 |
| kkoski | jlvillal: Yeah sorry, I reported it, then grabbed some lunch :P | 17:01 |
| JoshNang | TheJulia: agreed. after fighting with devstack and ipa last week, i have a couple things i want to fix | 17:02 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP: Fix broken unittests usage of sort() https://review.openstack.org/167325 | 17:02 |
| jlvillal | kkoski: ^^^ but failing one test. In case you wanted to see patch so far | 17:02 |
| TheJulia | wow, I think my test VM's disk is only 10G and it is taking a really long time :( | 17:02 |
| *** bandicot has quit IRC | 17:03 | |
| *** korekhov has joined #openstack-ironic | 17:05 | |
| JoshNang | yeahhh it's not quick at all :/ | 17:05 |
| NobodyCam | TheJulia: https://www.youtube.com/watch?v=uWvhS2nvqJA | 17:05 |
| *** bandicot has joined #openstack-ironic | 17:06 | |
| TheJulia | NobodyCam: does it include a blowtorch? | 17:07 |
| NobodyCam | lol nope just a sldge hammer | 17:07 |
| TheJulia | melting hard disks sound more fun | 17:07 |
| *** kevinbenton has quit IRC | 17:07 | |
| *** Marga_ has joined #openstack-ironic | 17:08 | |
| NobodyCam | TheJulia: but that may not get rid of all the data? :-p http://www.extremetech.com/computing/140975-scientists-accidentally-create-quantum-dots-out-of-lava | 17:09 |
| *** jmccrory has joined #openstack-ironic | 17:10 | |
| NobodyCam | seems we still need a few years of development to be able to read the data but... | 17:11 |
| NobodyCam | lol | 17:11 |
| TheJulia | Fine then. First we melt, then we drop the ingots into a volcano... or launch them into the daystar. | 17:11 |
| NobodyCam | hehhehe | 17:12 |
| kkoski | jlvillal: I'm taking a look now | 17:16 |
| jlvillal | kkoski: Cool :) | 17:16 |
| *** romcheg has quit IRC | 17:17 | |
| *** harlowja_away is now known as harlowja | 17:18 | |
| TheJulia | Cleaning is now enabled by default in bifrost | 17:22 |
| jroll | \o/ | 17:24 |
| lucasagomes | TheJulia, hi, bifrost uses IPA with pxe_ drivers? | 17:25 |
| lucasagomes | and ipxe too right? | 17:25 |
| jroll | boooooooooo pxe | 17:25 |
| jroll | :P | 17:25 |
| lucasagomes | jroll, heh no swift man! :P | 17:25 |
| jroll | don't need swift for IPA :) | 17:25 |
| lucasagomes | it should work without any other openstack component, I think that's the reason | 17:25 |
| lucasagomes | jrist, with agent? | 17:26 |
| jroll | only if you use glance | 17:26 |
| lucasagomes | oh | 17:26 |
| TheJulia | lucasagomes: ipxe, agent_ssh for testing and agent_ipmitool for 'normal' use | 17:26 |
| *** ijw has joined #openstack-ironic | 17:26 | |
| lucasagomes | yeah true | 17:26 |
| lucasagomes | TheJulia, oh gotcha | 17:26 |
| lucasagomes | cool | 17:26 |
| lucasagomes | jroll, ^ so it's agent_ see :D | 17:26 |
| *** mjturek1 has quit IRC | 17:26 | |
| jroll | yep I know :) | 17:26 |
| jroll | so uh, who wants to paste our whiteboard back in and format it? | 17:26 |
| jroll | http://paste.openstack.org/show/196032/ | 17:26 |
| jlvillal | kkoski: That one test seems totally broken | 17:27 |
| *** hyakuhei has quit IRC | 17:27 | |
| jlvillal | kkoski: Maybe lucasgomes would know. | 17:27 |
| TheJulia | I also put a playbook that is essentially a single playbook test, very simple and basic, needs work, but it kind of just works for testing in VM mode. A little later, I'm going to fire it off on a machine with 8GB of ram and see how it goes. :) | 17:27 |
| *** achanda has joined #openstack-ironic | 17:28 | |
| NobodyCam | Last WhiteBOard text restored | 17:28 |
| NobodyCam | history does appear to be lost | 17:28 |
| TheJulia | In any event, sufficent coffee has been ingested, time to look at the issue from yesterday | 17:28 |
| NobodyCam | but thanks to fungi in -infra we have the text from the last rev back | 17:29 |
| JoshNang | TheJulia: yay for cleaning! | 17:29 |
| NobodyCam | *may require some reformatting | 17:29 |
| lucasagomes | TheJulia, :) good stuff | 17:30 |
| jlvillal | lucasagomes: https://review.openstack.org/167325 gets this error: http://paste.openstack.org/show/196091/ | 17:30 |
| jlvillal | lucasagomes: I noticed you worked on that code. Not sure if you have any ideas. | 17:30 |
| jroll | jlvillal: that's ramineni's code | 17:31 |
| lucasagomes | jlvillal, oh... hmmm .sort() is wrong indeed | 17:31 |
| lucasagomes | jlvillal, I would thing you should do sorted(list) | 17:31 |
| lucasagomes | like | 17:32 |
| lucasagomes | self.assertEqual(sorted(list1), sorted(list2)) | 17:32 |
| * JayF cleaned up a section of the etherpad | 17:32 | |
| jlvillal | lucasagomes: Yes that would also work. I was using the unittest builtin method to compare lists via six. | 17:32 |
| lucasagomes | jlvillal, right, not sure what is best | 17:33 |
| jlvillal | lucasagomes: But the test is broken. I'll keep investigating to see if I can figure out why. | 17:33 |
| lucasagomes | jlvillal, fair enuff, try sorted() | 17:33 |
| jlvillal | lucasagomes: Also fails: http://paste.openstack.org/show/196094/ | 17:35 |
| lucasagomes | right it doesn't fail on the sort tho | 17:36 |
| lucasagomes | it fails because there's one uuid which is not being passed | 17:36 |
| lucasagomes | u'1be26c0b-03f2-4d2e-ae87-c02d7f33c781', | 17:36 |
| jlvillal | lucasagomes: Yeah | 17:36 |
| lucasagomes | that uuid is not present on the first dict | 17:36 |
| lucasagomes | so I assume it's something with the test itself | 17:36 |
| lucasagomes | or code that might be wrong | 17:36 |
| jlvillal | lucasagomes: Thanks. I'll keep looking :) | 17:36 |
| lucasagomes | jlvillal, cool! thank you for fixing it :) | 17:37 |
| jlvillal | lucasagomes: It isn't fixed yet ;) | 17:37 |
| yuriyz | hello Ironic jlvillal I commented this already https://review.openstack.org/#/c/167325/1/ironic/tests/db/test_ports.py | 17:37 |
| lucasagomes | will be soon heh :D | 17:37 |
| lucasagomes | yuriyz, morning | 17:37 |
| lucasagomes | or evening :) | 17:37 |
| yuriyz | maybe evening :) | 17:37 |
| jlvillal | yuriyz: Thanks. Either Gerrit is slow sending emails or our mail system has a problem. So I hadn't seen the email notification. | 17:38 |
| NobodyCam | hi yuriyz :) | 17:38 |
| jlvillal | yuriyz: Thanks! | 17:38 |
| yuriyz | morning NobodyCam | 17:38 |
| NobodyCam | :) | 17:38 |
| *** BertieFulton has quit IRC | 17:41 | |
| NobodyCam | brb | 17:43 |
| kkoski | yuriyz: So the test was forgetting about the existing port, and was assuming that it would get back only the 5 ports that it just created? | 17:43 |
| *** derekh has quit IRC | 17:45 | |
| yuriyz | kkoski yes you are right we create one extra before in setUp() | 17:45 |
| kkoski | cool, thanks | 17:45 |
| kkoski | jlvillal: does that take care of the broken test for you? | 17:45 |
| jlvillal | kkoski: Yes. | 17:46 |
| openstackgerrit | Merged openstack/python-ironicclient: Use oslo.i18n lib https://review.openstack.org/162797 | 17:46 |
| kkoski | jlvillal: Are you just going to submit a fix then? | 17:47 |
| kkoski | I'm still trying to set up a dev environment, so that would be fastest :P | 17:47 |
| jlvillal | jlvillal: Yes I will push new patch soon. | 17:48 |
| jlvillal | kkoski: Working on commit message | 17:48 |
| *** zz_jgrimm is now known as jgrimm | 17:48 | |
| openstackgerrit | Ghe Rivero proposed openstack/python-ironicclient: Clean openstack-common module list https://review.openstack.org/162799 | 17:49 |
| openstackgerrit | Ghe Rivero proposed openstack/python-ironicclient: Sync oslo.incubator https://review.openstack.org/162806 | 17:49 |
| *** ijw has quit IRC | 17:50 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Fix broken unittests usage of sort() https://review.openstack.org/167325 | 17:50 |
| jlvillal | kkoski: Pushed ^^^^ | 17:52 |
| NobodyCam | jroll: :( re: https://review.openstack.org/#/c/165430/ | 17:53 |
| jroll | yep. | 17:53 |
| kkoski | jlvillal: thanks! | 17:53 |
| NobodyCam | TheJulia: hows that clenning doing? still going? | 17:54 |
| jlvillal | kkoski: Thank you for spotting the issue! :) | 17:54 |
| JayF | jroll: or any other Ironic core around; this needs +2A to resolve #1433812 enough for Kilo (even though we should fix it "better" long term) https://review.openstack.org/#/c/167063/ | 17:55 |
| *** tiagogomes_ has quit IRC | 17:56 | |
| jroll | +A | 17:57 |
| lucasagomes | jroll, JayF I was too late :( but it would be good to add to ipxe template and elilo template as well | 17:58 |
| * lucasagomes puts a follow up patch | 17:58 | |
| TheJulia | NobodyCam: oh, it completed fairly quickly | 17:58 |
| jroll | lucasagomes: oh ha, sorry | 17:58 |
| NobodyCam | TheJulia: nice :) | 17:58 |
| JayF | lucasagomes: want me to write the follow up or are you already on it? | 17:59 |
| lucasagomes | JayF, I would start, but if you have it handy there | 17:59 |
| lucasagomes | I don't mind, I know we have too many templates and it sucks | 17:59 |
| lucasagomes | I hope we can make it better in the future | 17:59 |
| lucasagomes | (specially if we start using only 1 ramdisk for deployments) | 18:00 |
| *** ijw has joined #openstack-ironic | 18:00 | |
| openstackgerrit | Merged openstack/ironic: Raise exception for Agent Deploy driver when using partition images https://review.openstack.org/164440 | 18:02 |
| * lucasagomes notices there's no unittests for ipxe and elilo templates (y) | 18:02 | |
| lucasagomes | sigh | 18:02 |
| *** Haomeng has joined #openstack-ironic | 18:02 | |
| JayF | lucasagomes: heh, that means I'm /extra/ willing to do it | 18:02 |
| JayF | lucasagomes: actually, let me do it | 18:02 |
| JayF | lucasagomes: I can't +2 a thing, and you can | 18:02 |
| lucasagomes | JayF, cool, yeah go ahead | 18:02 |
| JayF | ...should I add the tests too then? | 18:02 |
| JayF | :( | 18:03 |
| lucasagomes | :) | 18:03 |
| lucasagomes | please? | 18:03 |
| *** bandicot has quit IRC | 18:03 | |
| lucasagomes | and I will have to go now :( | 18:03 |
| NobodyCam | night lucasagomes | 18:03 |
| NobodyCam | JayF: awesome! | 18:04 |
| lucasagomes | I will take a look at the patch later, JayF add me as a reviewer please | 18:04 |
| lucasagomes | have a good night everyone | 18:04 |
| JayF | I will, goodnight! | 18:04 |
| *** jcoufal has quit IRC | 18:04 | |
| *** jamielennox|away is now known as jamielennox | 18:04 | |
| *** lucasagomes is now known as lucas-dinner | 18:04 | |
| *** Haomeng|2 has quit IRC | 18:05 | |
| NobodyCam | TheJulia: have you seen jroll's comment on 166903? | 18:05 |
| TheJulia | NobodyCam: yes, working on it right now | 18:08 |
| *** jmccrory has quit IRC | 18:09 | |
| NobodyCam | woo hoo :) | 18:09 |
| *** jmccrory has joined #openstack-ironic | 18:09 | |
| clif_h | adam_g: JayF said you might know why I can't attach to a devstack's screen instance | 18:17 |
| adam_g | clif_h, hmm. not sure what happens when you try? | 18:17 |
| JayF | man, you don't have to point a big giant arrow at me | 18:17 |
| adam_g | :) | 18:17 |
| JayF | no need to tell adam_g I'm driving the bus :) | 18:17 |
| clif_h | lol | 18:17 |
| adam_g | imag | 18:18 |
| adam_g | er | 18:18 |
| clif_h | screen -r gives ' | 18:18 |
| clif_h | 'cannot open terminal - please check' | 18:18 |
| clif_h | the terminal in question only has root permissions | 18:18 |
| clif_h | and screen -r as root says no session to reattach to | 18:18 |
| JayF | clif_h: part of me wants to suggest you open the perms on that terminal, but I think that'd be gross :) | 18:18 |
| clif_h | idc its just a devstacked machine | 18:18 |
| adam_g | clif_h, huh. no ive never seen that | 18:19 |
| NobodyCam | brb | 18:19 |
| clif_h | chmod did it | 18:20 |
| *** Marga_ has quit IRC | 18:24 | |
| *** achanda has quit IRC | 18:28 | |
| *** ifarkas has quit IRC | 18:38 | |
| openstackgerrit | Josh Gachnang proposed openstack/ironic: Allow instance_uuid to be removed during cleaning https://review.openstack.org/167033 | 18:39 |
| *** mgoddard1 has joined #openstack-ironic | 18:43 | |
| *** Nisha has quit IRC | 18:45 | |
| *** mgoddard has quit IRC | 18:45 | |
| *** hyakuhei has joined #openstack-ironic | 18:48 | |
| *** hyakuhei has quit IRC | 18:48 | |
| *** mgoddard1 has quit IRC | 18:50 | |
| *** achanda has joined #openstack-ironic | 18:57 | |
| *** pas-ha has quit IRC | 18:57 | |
| *** achanda has quit IRC | 19:00 | |
| *** achanda has joined #openstack-ironic | 19:00 | |
| *** pelix has quit IRC | 19:00 | |
| *** jistr has joined #openstack-ironic | 19:02 | |
| *** mgoddard has joined #openstack-ironic | 19:02 | |
| *** ijw has quit IRC | 19:08 | |
| NobodyCam | JoshNang: should we test that other operations do fail in cleaning as we expect them to? on ^^^^ | 19:08 |
| JoshNang | NobodyCam: oh, duh. very much yes. | 19:09 |
| NobodyCam | :) | 19:09 |
| *** jcoufal has joined #openstack-ironic | 19:11 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Fix broken unittests usage of sort() https://review.openstack.org/167325 | 19:13 |
| openstackgerrit | Merged openstack/ironic: Fix formatting error in states_to_dot https://review.openstack.org/167159 | 19:18 |
| openstackgerrit | Merged openstack/ironic: Ensure configdrive isn't mounted in CoreOS ramdisks https://review.openstack.org/167063 | 19:18 |
| *** Marga_ has joined #openstack-ironic | 19:21 | |
| NobodyCam | jlvillal: instresting use of six in test_chassis | 19:23 |
| NobodyCam | can I ask why not just self.assertEqual(len(x), len(y)) | 19:24 |
| jlvillal | NobodyCam: The unittest framework has a function to compare lists irregardless of order. But they changed the name from Py2 vs Py3. Thus six. | 19:24 |
| jlvillal | NobodyCam: Not trying to compare length. Trying to compare contents | 19:24 |
| NobodyCam | ahh so assertCountEqual is a missleading name | 19:26 |
| jlvillal | NobodyCam: I would not disagree. But that is what unittest in Python3 chose as the name :( | 19:27 |
| NobodyCam | lol ack :) | 19:27 |
| jlvillal | NobodyCam: a and b have the same elements in the same number, regardless of their order | 19:28 |
| openstackgerrit | Josh Gachnang proposed openstack/ironic: Allow instance_uuid to be removed during cleaning https://review.openstack.org/167033 | 19:30 |
| *** jistr has quit IRC | 19:31 | |
| *** spandhe has quit IRC | 19:38 | |
| *** morganfainberg is now known as needslesscoffee | 19:39 | |
| *** spandhe has joined #openstack-ironic | 19:40 | |
| *** mgoddard has quit IRC | 19:41 | |
| *** achanda has quit IRC | 19:42 | |
| *** mgoddard has joined #openstack-ironic | 19:47 | |
| *** needslesscoffee is now known as morganfainberg | 19:50 | |
| *** jerryz has quit IRC | 19:52 | |
| *** jcoufal has quit IRC | 19:58 | |
| *** mgoddard has quit IRC | 20:04 | |
| *** ijw has joined #openstack-ironic | 20:18 | |
| *** mgoddard has joined #openstack-ironic | 20:24 | |
| *** hj-hp has joined #openstack-ironic | 20:24 | |
| *** romcheg has joined #openstack-ironic | 20:25 | |
| *** ijw_ has joined #openstack-ironic | 20:30 | |
| *** ijw has quit IRC | 20:33 | |
| *** jmccrory1 has joined #openstack-ironic | 20:34 | |
| *** jmccrory has quit IRC | 20:34 | |
| *** achanda has joined #openstack-ironic | 20:35 | |
| *** jmccrory has joined #openstack-ironic | 20:39 | |
| *** jmccrory1 has quit IRC | 20:42 | |
| mrda | Morning Ironic | 20:44 |
| NobodyCam | morning mrda | 20:44 |
| mrda | o/ | 20:44 |
| NobodyCam | ok I have a dumb git question. can I branch from a branch or better to create new branch just rebase on top of the orginial branch? | 20:45 |
| TheJulia | NobodyCam: you can branch from a branch | 20:45 |
| NobodyCam | :) | 20:46 |
| NobodyCam | Ty | 20:46 |
| TheJulia | if your wanting to rebase, might not be a bad idea to create a new branch and do the rebase in that branch so you can go back if you have to | 20:47 |
| mrda | ...especially if it's a nasty rebase | 20:48 |
| TheJulia | +1 | 20:48 |
| NobodyCam | shouldn't be but I'll do that... just sounds safer | 20:49 |
| TheJulia | just remember the branch name you used | 20:49 |
| *** mgoddard has quit IRC | 20:52 | |
| NobodyCam | brb | 20:55 |
| *** spandhe has quit IRC | 21:04 | |
| *** spandhe has joined #openstack-ironic | 21:06 | |
| *** openstackgerrit has quit IRC | 21:07 | |
| *** openstackgerrit has joined #openstack-ironic | 21:07 | |
| *** trown is now known as trown|outttypeww | 21:09 | |
| * BadCub is back | 21:09 | |
| TheJulia | wb | 21:10 |
| BadCub | ty :-) | 21:13 |
| openstackgerrit | Merged openstack/ironic: Fix broken unittests usage of sort() https://review.openstack.org/167325 | 21:16 |
| *** ndipanov has quit IRC | 21:22 | |
| *** andreykurilin_ has joined #openstack-ironic | 21:31 | |
| *** harlowja is now known as harlowja_away | 21:32 | |
| *** jmccrory has quit IRC | 21:34 | |
| *** andreykurilin__ has joined #openstack-ironic | 21:36 | |
| *** andreykurilin_ has quit IRC | 21:36 | |
| *** dprince has quit IRC | 21:38 | |
| *** kkoski has quit IRC | 21:40 | |
| *** mdbooth has quit IRC | 21:48 | |
| *** mdbooth has joined #openstack-ironic | 21:49 | |
| *** jmccrory has joined #openstack-ironic | 21:53 | |
| *** jmccrory has quit IRC | 21:53 | |
| *** ijw has joined #openstack-ironic | 21:54 | |
| *** ijw_ has quit IRC | 21:57 | |
| *** Marga_ has quit IRC | 21:59 | |
| *** lucas-dinner has quit IRC | 22:04 | |
| *** achanda_ has joined #openstack-ironic | 22:04 | |
| *** spandhe has quit IRC | 22:05 | |
| *** harlowja_away is now known as harlowja | 22:05 | |
| *** hj-hp has quit IRC | 22:06 | |
| *** hj-hp has joined #openstack-ironic | 22:07 | |
| *** achanda has quit IRC | 22:08 | |
| *** spandhe has joined #openstack-ironic | 22:08 | |
| TheJulia | jroll: so your suggestion re: 166903 seems to work out better in the long run, it found more discrepencies so \o/ and :( | 22:11 |
| NobodyCam | uggh but still \o/ | 22:12 |
| NobodyCam | ya for US finding them and not a user !!!! | 22:13 |
| jroll | TheJulia: awesome | 22:13 |
| * TheJulia goes and finds dinner | 22:15 | |
| NobodyCam | :) enjoy the dinnering | 22:16 |
| *** Marga_ has joined #openstack-ironic | 22:18 | |
| *** andreykurilin__ has quit IRC | 22:23 | |
| *** andreykurilin_ has joined #openstack-ironic | 22:23 | |
| jlvillal | JayF: Question on: https://review.openstack.org/#/c/162790/5/ironic_python_agent/cmd/agent.py On line 108 on the right hand side. Do you know what a call to CONF() does? | 22:23 |
| *** jcoufal has joined #openstack-ironic | 22:23 | |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic: Added vagrant VM for developer use https://review.openstack.org/167415 | 22:25 |
| JayF | There's something about how we load conf | 22:25 |
| JayF | because we inject configs there for metrics that can be sent from conductor | 22:25 |
| *** kkoski has joined #openstack-ironic | 22:26 | |
| JayF | jroll: ^ do you remember the details on that? | 22:26 |
| *** jrist has quit IRC | 22:28 | |
| *** anderbubble has quit IRC | 22:28 | |
| *** jrist has joined #openstack-ironic | 22:29 | |
| *** Marga_ has quit IRC | 22:30 | |
| *** athomas has quit IRC | 22:31 | |
| NobodyCam | ugg | 22:33 |
| NobodyCam | brb | 22:33 |
| jroll | JayF: jlvillal: https://github.com/openstack/oslo.config/blob/master/oslo_config/cfg.py#L1802-1858 | 22:33 |
| *** kkoski has quit IRC | 22:33 | |
| jlvillal | jroll: Thanks! | 22:33 |
| jroll | jlvillal: that said, I don't remember offhand what it does :) | 22:34 |
| jlvillal | jroll: JayF: Though makes me wonder in what order should https://review.openstack.org/#/c/162790/5/ironic_python_agent/cmd/agent.py happen. Before CONF() or after it? | 22:34 |
| jlvillal | Or maybe it doesn't matter. | 22:35 |
| *** romcheg has quit IRC | 22:35 | |
| *** Marga_ has joined #openstack-ironic | 22:36 | |
| *** kkoski has joined #openstack-ironic | 22:37 | |
| jroll | jlvillal: seems right at a glance | 22:39 |
| jroll | jlvillal: the only way to know is to run the code | 22:39 |
| jlvillal | :) | 22:39 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Correct update_dhcp_opts methods https://review.openstack.org/166903 | 22:39 |
| TheJulia | and now I really go to dinner | 22:40 |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Check temp dir is usable for ipmitool driver https://review.openstack.org/160383 | 22:46 |
| *** absubram has quit IRC | 22:53 | |
| *** kkoski has quit IRC | 22:53 | |
| *** kkoski has joined #openstack-ironic | 22:54 | |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Check temp dir is usable for ipmitool driver https://review.openstack.org/160383 | 22:58 |
| *** kkoski has quit IRC | 22:59 | |
| *** kkoski has joined #openstack-ironic | 23:00 | |
| *** penick has quit IRC | 23:02 | |
| *** penick has joined #openstack-ironic | 23:07 | |
| *** andreykurilin_ has quit IRC | 23:07 | |
| *** andreykurilin__ has joined #openstack-ironic | 23:07 | |
| *** hj-hp has quit IRC | 23:12 | |
| *** chlong has joined #openstack-ironic | 23:20 | |
| *** penick has quit IRC | 23:22 | |
| *** jmccrory has joined #openstack-ironic | 23:27 | |
| *** andreykurilin__ has quit IRC | 23:34 | |
| *** chlong has quit IRC | 23:35 | |
| *** chlong has joined #openstack-ironic | 23:35 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Ensure configdrive isn't mounted for ipxe/elilo https://review.openstack.org/167449 | 23:35 |
| *** ChuckC has quit IRC | 23:35 | |
| *** ijw_ has joined #openstack-ironic | 23:37 | |
| *** ChuckC has joined #openstack-ironic | 23:39 | |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Add check for usable tftp directory to PXE driver https://review.openstack.org/167451 | 23:39 |
| *** ijw has quit IRC | 23:40 | |
| JayF | I'm like 99% sure I did something wrong on https://review.openstack.org/167449 | 23:43 |
| JayF | I put a comment on it about where | 23:43 |
| JayF | but someone with knowledge of these things giving me a review would be super helpful | 23:43 |
| *** kkoski has quit IRC | 23:46 | |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Add check for usable tftp directory to PXE driver https://review.openstack.org/167451 | 23:49 |
| *** penick has joined #openstack-ironic | 23:52 | |
| *** Haomeng|2 has joined #openstack-ironic | 23:56 | |
| *** Haomeng has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!