*** shu-mutou-AFK is now known as shu-mutou | 00:00 | |
*** piet has joined #openstack-ironic | 00:01 | |
*** chaustin has quit IRC | 00:07 | |
mrda | Do we have a supported hardware list in Ironic? I know we have https://wiki.openstack.org/wiki/Ironic/Drivers but anything more Vendor/Model specific? | 00:14 |
---|---|---|
JayF | I know that some driver docs, like the HP, will specify what firmwares and such they work with | 00:15 |
mrda | thanks JayF | 00:16 |
*** Sukhdev has quit IRC | 00:16 | |
openstackgerrit | Stephanie Miller proposed openstack/bifrost: tftp service name can vary https://review.openstack.org/283879 | 00:16 |
*** Sukhdev has joined #openstack-ironic | 00:17 | |
openstackgerrit | Stephanie Miller proposed openstack/bifrost: Install python if it's not there https://review.openstack.org/283783 | 00:18 |
*** Sukhdev has quit IRC | 00:18 | |
*** spandhe has quit IRC | 00:19 | |
*** suro-patz1 has quit IRC | 00:22 | |
*** suro-patz has joined #openstack-ironic | 00:23 | |
*** spandhe has joined #openstack-ironic | 00:28 | |
*** thrash is now known as thrash|g0ne | 00:33 | |
ayoung | 2016-02-23 23:27:27.241 20785 INFO nova.filters [req-d2b17826-6303-4165-8df0-f26091bc7f87 789eaa25a2f3407bab33982cebc58dd0 91680b78999e484c8ccb4425eec20edb - - -] Filter ComputeCapabilitiesFilter returned 0 hosts | 00:42 |
ayoung | How do I figure out why ComputeCapabilitiesFilter fails to match the available node? | 00:43 |
ayoung | 2016-02-23 23:27:27.241 20785 DEBUG nova.scheduler.filters.compute_capabilities_filter [req-d2b17826-6303-4165-8df0-f26091bc7f87 789eaa25a2f3407bab33982cebc58dd0 91680b78999e484c8ccb4425eec20edb - - -] (instack. | 00:44 |
ayoung | localdomain, 9f936ffd-b69c-4f1b-9537-9feee5eb890e) ram:4096 disk:40960 io_ops:0 instances:0 fails instance_type extra_specs requirements host_passes /usr/lib/python2.7/site-packages/nova/scheduler/filters/comput | 00:44 |
ayoung | e_capabilities_filter.py:101 | 00:44 |
ayoung | I wonder if it is just a bit too small disk wise? | 00:45 |
ayoung | hmmm | 00:45 |
JayF | if you changed something in the sched to make it attempt before | 00:48 |
JayF | then got that error you copied in | 00:48 |
JayF | that would make perfect sense | 00:48 |
ayoung | JayF, I'm not sure I understand you | 00:49 |
JayF | So before, the error you pasted | 00:50 |
JayF | 15:17:37 <ayoung> The location 40962 is outside of the device /dev/sdb.\n' | 00:50 |
ayoung | JayF, Ah, I figured that was a size issue, yeah | 00:50 |
JayF | 50% guessing 50% actual knowledge: seems like an error that you'd get from the disk being too small | 00:50 |
ayoung | so I upped the disk size to 435 Gb | 00:50 |
ayoung | I'm past that err...I think | 00:50 |
ayoung | except that it seems to be failing fatser now | 00:50 |
ayoung | ComputeCapabilitiesFilter does not like something | 00:51 |
ayoung | and it is no longer the disk size | 00:51 |
ayoung | JayF, so I moved the disk from 40 Gb to 54 and now... | 00:52 |
ayoung | 2016-02-24 00:40:03.227 20785 DEBUG nova.filters [req-1b7e3520-e15c-4ad5-b641-448e198e87ba 789eaa25a2f3407bab33982cebc58dd0 91680b78999e484c8ccb4425eec20edb - - -] Filter RamFilter returned 1 host(s) get_filtered_objects /usr/lib/python2.7/site-packages/nova/filters.py:104 | 00:52 |
ayoung | 2016-02-24 00:40:03.227 20785 INFO nova.filters [req-1b7e3520-e15c-4ad5-b641-448e198e87ba 789eaa25a2f3407bab33982cebc58dd0 91680b78999e484c8ccb4425eec20edb - - -] Filter DiskFilter returned 0 hosts | 00:52 |
ayoung | that seems strange...is it not willing to leave the 4+Gb wasted? | 00:53 |
JayF | I'm honestly not sure; I know very little about the scheduler and how Ironic interacts with it | 00:53 |
*** hoangcx has joined #openstack-ironic | 00:58 | |
*** Goneri has joined #openstack-ironic | 00:59 | |
*** garthb_ has quit IRC | 01:02 | |
openstackgerrit | Gonéri Le Bouder proposed openstack/ironic: ipxe: add --timeout parameter to kernel and initrd https://review.openstack.org/283893 | 01:09 |
*** ppiela has quit IRC | 01:09 | |
*** suro-patz has quit IRC | 01:09 | |
*** praneshp has quit IRC | 01:09 | |
*** Goneri has quit IRC | 01:12 | |
*** Goneri has joined #openstack-ironic | 01:12 | |
*** Sukhdev has joined #openstack-ironic | 01:14 | |
*** penick has quit IRC | 01:28 | |
openstackgerrit | Stephanie Miller proposed openstack/bifrost: Improve log collection https://review.openstack.org/283897 | 01:31 |
*** spandhe has quit IRC | 01:31 | |
*** spandhe has joined #openstack-ironic | 01:33 | |
*** trown|outtypewww has quit IRC | 01:37 | |
*** ChrisAusten has joined #openstack-ironic | 01:39 | |
*** piet has quit IRC | 01:53 | |
*** suro-patz has joined #openstack-ironic | 01:56 | |
*** baoli has joined #openstack-ironic | 01:58 | |
*** Sukhdev has quit IRC | 01:59 | |
*** jaybeale has quit IRC | 01:59 | |
*** dims has quit IRC | 01:59 | |
ayoung | JayF, ACHA I was making the VM too small...2048 instead of 4096... | 02:00 |
ayoung | lets see how it fails this time | 02:00 |
*** Sukhdev has joined #openstack-ironic | 02:01 | |
*** Sukhdev has quit IRC | 02:01 | |
*** spandhe has quit IRC | 02:01 | |
*** vishwanathj has quit IRC | 02:06 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic-python-agent: Replace all the 'self.log' calls with global LOG https://review.openstack.org/282154 | 02:06 |
*** Goneri has quit IRC | 02:06 | |
*** pcaruana has quit IRC | 02:07 | |
*** chenke has joined #openstack-ironic | 02:13 | |
*** haomeng_ has joined #openstack-ironic | 02:18 | |
haomeng_ | chenke,hi | 02:18 |
*** pcaruana has joined #openstack-ironic | 02:19 | |
*** suro-patz has quit IRC | 02:24 | |
*** vishwanathj has joined #openstack-ironic | 02:26 | |
*** vishwana_ has joined #openstack-ironic | 02:28 | |
*** suro-patz has joined #openstack-ironic | 02:29 | |
*** vishwanathj has quit IRC | 02:31 | |
*** mgoddard_ has joined #openstack-ironic | 02:32 | |
*** mgoddard has quit IRC | 02:35 | |
*** suro-patz has quit IRC | 02:36 | |
*** Marga_ has quit IRC | 02:39 | |
*** chaustin has joined #openstack-ironic | 02:39 | |
*** chaustin has quit IRC | 02:44 | |
*** sivaramakrishna has joined #openstack-ironic | 02:46 | |
*** pcaruana has quit IRC | 03:01 | |
*** hoangcx has quit IRC | 03:06 | |
*** ChubYann has quit IRC | 03:08 | |
*** dims has joined #openstack-ironic | 03:13 | |
*** pcaruana has joined #openstack-ironic | 03:16 | |
*** ChubYann has joined #openstack-ironic | 03:28 | |
openstackgerrit | vinay kumar muddu proposed openstack/proliantutils: Changes to NIC info retrieving logic https://review.openstack.org/273118 | 03:30 |
*** suro-patz has joined #openstack-ironic | 03:35 | |
*** hoangcx has joined #openstack-ironic | 03:41 | |
*** suro-patz has quit IRC | 03:42 | |
*** links has joined #openstack-ironic | 03:46 | |
*** Nisha has joined #openstack-ironic | 03:50 | |
*** vaishnavi has joined #openstack-ironic | 03:52 | |
*** boris-42 has quit IRC | 03:54 | |
*** baoli has quit IRC | 03:55 | |
*** pcaruana has quit IRC | 04:01 | |
*** baoli has joined #openstack-ironic | 04:03 | |
*** pcaruana has joined #openstack-ironic | 04:15 | |
*** yolanda has joined #openstack-ironic | 04:24 | |
*** baoli has quit IRC | 04:28 | |
*** baoli has joined #openstack-ironic | 04:29 | |
*** raddaoui has joined #openstack-ironic | 04:29 | |
*** raddaoui_ has joined #openstack-ironic | 04:29 | |
*** raddaoui_ has quit IRC | 04:30 | |
*** raddaoui has quit IRC | 04:30 | |
*** raddaoui has joined #openstack-ironic | 04:31 | |
*** raddaoui_ has joined #openstack-ironic | 04:31 | |
*** hoangcx has quit IRC | 04:35 | |
*** Marga_ has joined #openstack-ironic | 04:36 | |
*** raddaoui_ has quit IRC | 04:37 | |
*** raddaoui has quit IRC | 04:37 | |
*** baoli has quit IRC | 04:39 | |
*** Marga_ has quit IRC | 04:40 | |
*** Marga_ has joined #openstack-ironic | 04:43 | |
*** kalpase has joined #openstack-ironic | 04:43 | |
*** penick has joined #openstack-ironic | 04:48 | |
*** penick_ has joined #openstack-ironic | 04:50 | |
*** praneshp has joined #openstack-ironic | 04:52 | |
*** piet has joined #openstack-ironic | 04:52 | |
*** penick has quit IRC | 04:53 | |
*** penick_ is now known as penick | 04:53 | |
*** harlowja_at_home has quit IRC | 04:53 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Fail devstack if unable to create some images in glance https://review.openstack.org/279270 | 04:54 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: WIP: Add unit tests for 'ironic/common/config_generator/generator.py' https://review.openstack.org/283937 | 04:56 |
*** Marga_ has quit IRC | 04:57 | |
*** Marga_ has joined #openstack-ironic | 04:58 | |
*** praneshp_ has joined #openstack-ironic | 04:58 | |
openstackgerrit | Vaishnavi proposed openstack/ironic: [IRONIC] Config variable to configure [glance] section https://review.openstack.org/266803 | 04:59 |
*** praneshp has quit IRC | 04:59 | |
*** praneshp_ is now known as praneshp | 04:59 | |
openstackgerrit | vinay kumar muddu proposed openstack/proliantutils: Changes to NIC info retrieving logic https://review.openstack.org/273118 | 04:59 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Fail devstack if some variables are not set https://review.openstack.org/279270 | 05:02 |
*** deray has joined #openstack-ironic | 05:04 | |
*** amotoki has joined #openstack-ironic | 05:05 | |
*** praneshp has quit IRC | 05:07 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Follow up nits of irmc oob inspection https://review.openstack.org/280060 | 05:13 |
*** raddaoui has joined #openstack-ironic | 05:18 | |
*** raddaoui_ has joined #openstack-ironic | 05:18 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Fail devstack if some variables are not set https://review.openstack.org/279270 | 05:20 |
*** jaybeale has joined #openstack-ironic | 05:22 | |
openstackgerrit | Sivaramakrishna Garimella proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 05:26 |
*** hoangcx has joined #openstack-ironic | 05:26 | |
*** pcaruana has quit IRC | 05:38 | |
*** ijw has quit IRC | 05:46 | |
*** dims has quit IRC | 05:48 | |
*** ChrisAusten has quit IRC | 05:50 | |
*** pcaruana has joined #openstack-ironic | 05:53 | |
*** piet has quit IRC | 06:00 | |
*** jaybeale has quit IRC | 06:00 | |
*** jaybeale has joined #openstack-ironic | 06:01 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Fail devstack if some variables are not set https://review.openstack.org/279270 | 06:02 |
*** Haomeng has quit IRC | 06:15 | |
*** Haomeng has joined #openstack-ironic | 06:15 | |
*** harlowja_at_home has joined #openstack-ironic | 06:17 | |
*** raddaoui_ has quit IRC | 06:17 | |
*** raddaoui has quit IRC | 06:17 | |
*** haomeng_ has quit IRC | 06:19 | |
*** openstack has joined #openstack-ironic | 13:22 | |
*** subscope has quit IRC | 13:24 | |
*** chaustin has joined #openstack-ironic | 13:24 | |
sinval | ping lucasagomes, too much work? | 13:29 |
jroll | alineb: hey, reviewed your thing, I feel like we can make this easier if we make it a top-level argument like --debug | 13:33 |
jroll | but in general I love this | 13:33 |
* jroll can't wait to pipe to jq | 13:33 | |
*** KennethWilke has joined #openstack-ironic | 13:34 | |
*** mgoddard__ has joined #openstack-ironic | 13:38 | |
alineb | thank you for your review jroll! | 13:38 |
jroll | welcome | 13:38 |
dtantsur | morning jroll, TheJulia, sinval | 13:39 |
alineb | yes, i noticed the problem while adding the tests, so i'm trying to correct it | 13:39 |
jroll | now on to the hard part... dtantsur's futurist patch | 13:39 |
sinval | morning dtantsur and jroll | 13:39 |
jroll | morning y'all | 13:39 |
alineb | and i will look at the --debug flag to make --json global, thanks for the hint :) | 13:39 |
dtantsur | jroll, it's not as huge as I was afraid initially it would be :) | 13:39 |
*** dims has joined #openstack-ironic | 13:40 | |
jroll | dtantsur: which means it's even more thought crammed into each line :D | 13:40 |
dtantsur | true :) | 13:41 |
*** mgoddard_ has quit IRC | 13:41 | |
*** amotoki has joined #openstack-ironic | 13:44 | |
deray | vdrok, https://review.openstack.org/#/c/211019/18/ironic/drivers/modules/ilo/firmware_processor.py:l45 | 13:45 |
ohamada | dtantsur, sambetts: hello guys, thank you for help last week with the ancient hp hardware | 13:46 |
deray | if I remove those then, can i use CONF.ilo.. here: https://review.openstack.org/#/c/211019/18/ironic/drivers/modules/ilo/firmware_processor.py:l250 | 13:47 |
*** Goneri has joined #openstack-ironic | 13:47 | |
deray | or do i have to do ilo_common.CONF.ilo..? | 13:47 |
lucasagomes | sinval, hello there | 13:47 |
lucasagomes | sinval, we can talk yes... | 13:48 |
sinval | cool | 13:48 |
* lucasagomes got a cold, but he's trying his best to keep doing something | 13:48 | |
dtantsur | ohamada, you're welcome, hope it went ok | 13:48 |
sinval | lucasagomes, hope you get well soon | 13:49 |
ohamada | dtansur: yeah, i had finally a reason to replace them with some less ancient ones | 13:49 |
sinval | lucasagomes, I've been thinking about the periodic task vs daemon thing, periodic task is good, but, at least, by doing synchronization of resources with the daemon, users can use multiple instances of it, and, is expected that we could make daemon listen to OneView's event bus, so, it will be pretty fast to know when a node has been taken in OneView | 13:50 |
ohamada | dtantsur: i wanted to ask - how can I reduce the keystone timeout in the overcloud? I mean how can i modify it on the ironic node - before the deployment statrs | 13:51 |
sambetts | jroll, alineb: I wonder if we should try to adopt the style for formatting that has been adopted by the openstack client? e.g. openstack server list -f json | 13:51 |
sinval | lucasagomes, the daemon the tool that I mentioned during the discussion of using tooz for shared memory lock | 13:51 |
sinval | the daemon is the tool* | 13:51 |
sambetts | ohamada: :) | 13:51 |
jroll | sambetts: hmmm, it can't hurt, but I do think --json is the most intuitive | 13:52 |
dtantsur | ohamada, sorry, that's something I don't really know. You may have better luck asking on #tripleo.. it's not something that ironic is managing. | 13:52 |
lucasagomes | sinval, gotcha... Sure I'm not against it, just thought about mentioning the periodic task cause that would may facilitate the deployment (one less component to take care of) | 13:53 |
lucasagomes | and since we will soon switch to futurist, periodic tasks will run in parallel as they should | 13:53 |
sambetts | jroll: it depends if we want to support other formatting features too, OSC supports csv, json, table (the default), value and yaml | 13:53 |
lucasagomes | sinval, about the daemon, will it talk directly to the Ironic APIs right? | 13:53 |
sinval | lucasagomes, yes | 13:53 |
lucasagomes | sinval, any concerns with security there? Since Ironic is an admin-only api atm? | 13:54 |
ohamada | dtantsur: ok, thanks | 13:54 |
jroll | sambetts: yeah, I'd like to stop at json and if people want others they can help us transition to OSC :) | 13:54 |
sambetts | jroll: :-P true true | 13:54 |
sinval | lucasagomes, well, about this I'm afraid that we don't have what to do instead of using admin-only api, do you have any suggestion? | 13:55 |
sinval | lucasagomes, I mean, once the daemon is just automates what the operator does... | 13:55 |
sinval | lucasagomes, do you think that by documenting this concern would be enough for the users? | 13:56 |
lucasagomes | sinval, yeah I don't think there's much to do in that case either... Pehaps highlight it in the security impact of the spec | 13:56 |
lucasagomes | sinval, I believe so, if we have no way around it | 13:56 |
sinval | lucasagomes, fair enough | 13:57 |
lucasagomes | until we can support a more fine grained policies in the ironic api | 13:57 |
jroll | dtantsur: so, "Configuration option "periodic_interval" is deprecated and has no effect." | 13:58 |
jroll | dtantsur: is there any point in deprecating (instead of removing) if it has no effect now? and is there any effect it should have in terms of back compat | 13:59 |
sinval | lucasagomes, about periodic task, I think that would be cool provide solutions that can use polling, but, it will not be better than have the daemon that listens the bus... | 13:59 |
jroll | ? | 13:59 |
dtantsur | jroll, 1. probably not so much.. I can follow-up with removal. 2. it was a fine-grained tuning of periodic tasks, which I think does not have much sense to change.. so I don't think it will affect anyone. | 14:00 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Fail devstack if some variables are not set https://review.openstack.org/279270 | 14:00 |
sinval | lucasagomes, maybe we can have a first version of dynalloc that works with daemon, and we can improve the ironic side with futurist as next steps | 14:00 |
jroll | dtantsur: yeah, so AIUI that option sets "the spacing to check all periodic tasks" and the other options set the spacing for them to actually run, which has an effective minimum of periodic_interval (because they will only be checked on that interval) | 14:01 |
lucasagomes | sinval, yeah, event actions is much better indeed | 14:01 |
dtantsur | jroll, now that we run all tasks in their own threads, it does not make much sense IMO | 14:01 |
jroll | dtantsur: meaning it's the spacing for any tasks that don't set spacing (and now what happens to those? do they break or?) | 14:01 |
lucasagomes | sinval, yeah, because I believe we could have a thread listen to the event bus as well | 14:02 |
* dtantsur refreshes his memory | 14:02 | |
lucasagomes | sinval, but that all said, this is a very unique behavior for a driver in Ironic. Having 2 different system sharing the pool of machines | 14:02 |
jroll | dtantsur: 60s default before https://github.com/openstack/oslo.service/blob/master/oslo_service/periodic_task.py#L46 | 14:02 |
sinval | lucasagomes, yeah, but, notice that the code for listening the bus is pretty much specific, and it needs specific authorization config, so it will lead to more configs at ironic.conf | 14:03 |
dtantsur | jroll, well, spacing is required in futurist now.. | 14:03 |
lucasagomes | sinval, right, but it's also optional right? That's just used when the resources are being shared | 14:03 |
lucasagomes | if you have only ironic managing the nodes that's not required | 14:03 |
jroll | dtantsur: right, I'm looking around, I think this is solvable | 14:03 |
sinval | lucasagomes, true | 14:04 |
*** piet has joined #openstack-ironic | 14:04 | |
lucasagomes | sinval, and most of the code could also live at python-oneviewclient you believe? | 14:04 |
*** ayoung has quit IRC | 14:04 | |
dtantsur | jroll, I'd say everybody sane should leave it just as default.. but maybe it's only me. I don't feel like reintroducing one more compatibility wrapper just because of that... but I can, if you think it's critical | 14:04 |
jroll | dtantsur: so, I'm thinking in driver_periodic_task, if spacing is not set we default to periodic_interval, and drop both of those at the same time | 14:05 |
sinval | lucasagomes, I would say yes, but I have to think more about this, once that oneviewclient is just a lib, I'm not sure if it will works well if we just put the event bus code there | 14:05 |
lucasagomes | fair enough | 14:06 |
dtantsur | jroll, yeah, makes sense.. I forgot that it's only for driver_periodic_task. I'll update it as soon as you're done reviewing | 14:06 |
jroll | dtantsur: yeah, I'll leave a comment and review the rest, thanks | 14:06 |
sinval | lucasagomes, I mean, it's just rabbitmq client code, there is no reason for not working inside it, but I have to double check it | 14:06 |
sinval | lucasagomes, but it's my first action for event bus listening as threads inside Ironic | 14:07 |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Set node last_error in TaskManager https://review.openstack.org/273992 | 14:08 |
sinval | lucasagomes, I'll update the spec, saying that we are going to close the first version of dynalloc with the daemon for resource synchronization and in a second moment we can provide things inside Ironic also | 14:08 |
lucasagomes | sinval, cool. Thanks for that | 14:08 |
dtantsur | EmilienM, hey! I forgot to tell you yesterday, but your problem with tempest plugin should be solved now | 14:08 |
sinval | lucasagomes, last topic: only maintenance mode instead of maintenance and manageable | 14:09 |
sinval | lucasagomes, IMO, using only maintenance mode to indicate that a node is in use by OneView and 'unavailable' for Ironic can be misleading, because, the can appear as 'available' state, would be awkward, don't you agree? So, we do think that moving the node back to manageable would be the best approach to leave things clear for the operator. | 14:09 |
sinval | the node can appear* | 14:09 |
lucasagomes | sinval, yeah they will be available with the maintenance flag set. Yeah maybe it's not very visible indeed, plus, we can hit the case where the data from previous tenant is still in the local HD if we only toggle the maintenance flag | 14:10 |
lucasagomes | by moving states we will make sure cleaning is triggered | 14:11 |
lucasagomes | so yeah, moving state may be the right thing to do here | 14:11 |
sinval | lucasagomes, yes | 14:11 |
lucasagomes | sinval, would be good to also check prior to cleaning if the node has a server profile assigned to it | 14:12 |
lucasagomes | or someone may start cleaning a node which is being used by someone in OneView | 14:12 |
sinval | lucasagomes, yes, when we hit cleaning, if the node does not have any SP applied, we check that and apply a SP to do the cleaning | 14:13 |
lucasagomes | oh true | 14:13 |
lucasagomes | ++ | 14:13 |
sinval | lucasagomes, cleaning is one of the main concerns for us in this implementation | 14:14 |
sinval | lucasagomes, I'm not sure if you had the chance to see the liliars replies, but regarding the security impact regarding the disk when the hardware is released | 14:15 |
*** piet has quit IRC | 14:15 | |
lucasagomes | sinval, I will take a look | 14:15 |
*** piet has joined #openstack-ironic | 14:16 | |
sinval | lucasagomes, OneView does not send its hardware to a "cleaning" process after workloads, so, it's an issue | 14:16 |
lucasagomes | yeah, but Ironic will do it right? | 14:16 |
sinval | lucasagomes, yeah | 14:16 |
lucasagomes | I mean, it still a problem for OneView-only users | 14:17 |
sinval | lucasagomes, yeah | 14:17 |
lucasagomes | but I don't think we can do anything about it | 14:17 |
sinval | lucasagomes, I'm afraid that we can't do anything | 14:18 |
sinval | lucasagomes, do you still thinking that it's good to mention it as a security impact in the spec? | 14:18 |
sinval | lucasagomes, I mean, it's bad for OneView users... but for Ironic, I don't think so... | 14:18 |
lucasagomes | sinval, I think so, I mean they are all impacts of having a shared system | 14:18 |
lucasagomes | yeah | 14:18 |
sinval | lucasagomes, hum | 14:18 |
sinval | lucasagomes, ok, will do | 14:19 |
lucasagomes | wouldn't cost have 1 small paragraph just pointing it out | 14:19 |
sinval | lucasagomes, true | 14:19 |
sinval | lucasagomes, I think that's it | 14:19 |
sinval | lucasagomes, everything clear from my side | 14:19 |
lucasagomes | thanks for that! | 14:20 |
sinval | lucasagomes, thank you for the support, it was a awesome review on that spec | 14:20 |
*** alexpilotti has joined #openstack-ironic | 14:20 | |
lucasagomes | np :D I will re-review it after the updates | 14:20 |
sinval | lucasagomes, cool | 14:21 |
*** alexpilotti has quit IRC | 14:24 | |
jroll | dtantsur: ok I'm done :) | 14:25 |
jroll | this is awesome, I'm excited | 14:25 |
dtantsur | good, updating now :) | 14:25 |
jroll | always takes me a bit to get through black magic like that | 14:25 |
vdrok | deray, correct, just CONF.ilo... | 14:26 |
dtantsur | heh | 14:26 |
deray | vdrok, okay.. | 14:26 |
* jroll steps away for a bit | 14:26 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Move _from_db_object() into base class https://review.openstack.org/276960 | 14:34 |
openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Bring OSC plugin inline with approved spec https://review.openstack.org/284160 | 14:36 |
EmilienM | dtantsur: I figured. Now I found more problems | 14:40 |
dtantsur | oh | 14:40 |
EmilienM | dtantsur: before I was running baremetal tests from tempest and they worked. Now, the tests from ironic do not work. I got 403 all the time. Let me show you | 14:40 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Switch to Futurist library for asynchronous execution and periodic tasks https://review.openstack.org/264720 | 14:45 |
dtantsur | jroll, lucasagomes, yuriyz ^^ | 14:45 |
openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Bring OSC plugin inline with approved spec https://review.openstack.org/284160 | 14:47 |
openstackgerrit | Imre Farkas proposed openstack/ironic: [WIP] DRAC: RAID configuration on vendor-passthru https://review.openstack.org/176319 | 14:52 |
EmilienM | dtantsur: http://logs.openstack.org/86/283786/1/check/gate-puppet-openstack-integration-scenario002-tempest-dsvm-centos7/d56dd17/console.html#_2016-02-23_21_31_02_255 | 14:58 |
EmilienM | so this is when I run tox with all plugins flag. | 14:58 |
* dtantsur brb, 5 mins please | 14:59 | |
EmilienM | dtantsur: if I run tests from tempest tree, baremetal tests work | 14:59 |
*** mtanino has joined #openstack-ironic | 14:59 | |
*** pas-ha has quit IRC | 15:00 | |
*** deray has quit IRC | 15:01 | |
*** krtaylor has quit IRC | 15:02 | |
*** betherly_ is now known as betherly | 15:06 | |
*** jaybeale has joined #openstack-ironic | 15:06 | |
NobodyCam | good morning Ironicer | 15:11 |
lucasagomes | NobodyCam, morning | 15:12 |
NobodyCam | good morning dtantsur lucasagomes rloo devananda jroll JayF sambetts jlvillal vdrok mgould sinval derekh and all others not listed here directly | 15:13 |
NobodyCam | :) | 15:13 |
*** krtaylor has joined #openstack-ironic | 15:14 | |
*** pas-ha has joined #openstack-ironic | 15:16 | |
jroll | \o NobodyCam | 15:22 |
dtantsur | morning NobodyCam | 15:22 |
NobodyCam | o/ | 15:22 |
NobodyCam | :) | 15:22 |
*** yolanda has quit IRC | 15:24 | |
*** Nisha has joined #openstack-ironic | 15:24 | |
dtantsur | EmilienM, is it possible you grab ironic-api logs from that run? | 15:24 |
EmilienM | dtantsur: http://logs.openstack.org/86/283786/1/check/gate-puppet-openstack-integration-scenario002-tempest-dsvm-centos7/d56dd17/logs/ironic | 15:25 |
jroll | dtantsur: hm, removing that lock scares me a bit | 15:26 |
jroll | it's probably fine but idk | 15:26 |
dtantsur | jroll, we figured out that submit() has its own lock | 15:26 |
jroll | ok | 15:26 |
dtantsur | jroll, see e.g. https://github.com/openstack/futurist/blob/master/futurist/_futures.py#L376 (a bit badly named IMO) | 15:27 |
jroll | dtantsur: yeah, I did find it in comments, thank you | 15:28 |
jroll | +2 for you sir | 15:28 |
dtantsur | thnx :) | 15:28 |
dtantsur | lucasagomes, mind taking a look at the futurist patch https://review.openstack.org/264720 again please? | 15:28 |
dtantsur | ditto yuriyz ^^ | 15:28 |
*** penick has joined #openstack-ironic | 15:29 | |
lucasagomes | dtantsur, will do in a sec | 15:29 |
*** dims has quit IRC | 15:29 | |
*** chaustin has quit IRC | 15:29 | |
dtantsur | awesome! | 15:29 |
* dtantsur wants to merge it a bit before feature freeze to give more time for testing | 15:29 | |
jroll | ++ | 15:30 |
*** dims has joined #openstack-ironic | 15:30 | |
lucasagomes | dtantsur, also, if you have a time https://review.openstack.org/#/c/273577/ :-) | 15:31 |
* lucasagomes opens the futurist patch | 15:31 | |
mgould | NobodyCam, morning! | 15:32 |
dtantsur | lucasagomes, lol, git renaming detection went crazy on that patch | 15:32 |
NobodyCam | :) | 15:32 |
lucasagomes | dtantsur, yeah :-/ | 15:32 |
mgould | EmilienM, I could really use some advice on https://review.openstack.org/#/c/276187 if you have a minute | 15:32 |
EmilienM | mgould: sure, shoot | 15:33 |
mgould | great, thanks | 15:33 |
EmilienM | mgould: I'm just asking for a commit message | 15:33 |
mgould | 1) how can I test that the dnsmasq config actually does what it's meant to? | 15:33 |
*** alexpilotti has joined #openstack-ironic | 15:34 | |
EmilienM | wait, you're asking me ironic question? | 15:34 |
EmilienM | I'm not ironic expert, just fyi | 15:34 |
mgould | 2) the corresponding patch for Bifrost has been -1ed until the code ensures the files ipxe.efi and undionly.kpxe are in the right place; is the same thing necessary here, and if so how? | 15:34 |
*** alexpilotti has quit IRC | 15:34 | |
mgould | EmilienM, no, I'm asking you a puppet question | 15:34 |
*** dims has quit IRC | 15:34 | |
*** krtaylor has quit IRC | 15:34 | |
mgould | given the available test framework, what options do I have for testing the boot process? | 15:35 |
EmilienM | functional tests? we only run ironic API tests for now | 15:35 |
EmilienM | and dtantsur is helping to figure why they are failing when using tempest plugin | 15:35 |
mgould | I've created a dnsmasq instance locally and booted machines off it, but that's not repeatable | 15:35 |
EmilienM | mgould: we don't have extended tests for that. | 15:36 |
dtantsur | yeah, and it looks really weird | 15:36 |
mgould | right, OK | 15:36 |
EmilienM | mgould: I'll +2 your patch once we have a commit message. | 15:36 |
mgould | wow, OK, that was easier than I expected | 15:36 |
EmilienM | yes | 15:36 |
*** thrash is now known as thrash|biab | 15:37 | |
EmilienM | Puppet OpenStack CI don't tests *everything* in all projects. We just run smoke + some API tests not in smoke, like Ironic. | 15:37 |
EmilienM | mgould: you can rtfm that if you want to know more: https://github.com/openstack/puppet-openstack-integration#description | 15:37 |
dtantsur | jroll, did we even change anything in our tempest plugin after moving it in-tree? I don't get what's wrong with EmilienM's run | 15:38 |
EmilienM | mgould: but this is basically what we test: https://github.com/openstack/puppet-openstack-integration/blob/master/run_tests.sh#L99-L104 | 15:38 |
EmilienM | dtantsur: the 403 thing is weird. | 15:38 |
EmilienM | I did not have it before | 15:38 |
EmilienM | and I'm running Ironic from trunk | 15:38 |
*** pas-ha has quit IRC | 15:39 | |
jroll | dtantsur: I don't believe so, maybe this is a roles vs policy thing? | 15:41 |
jroll | all I see is "getting 403s" | 15:41 |
dtantsur | yep | 15:42 |
jroll | but yeah, I'd check user roles vs what's in policy | 15:42 |
lucasagomes | dtantsur, jroll +2+a | 15:44 |
lucasagomes | since it had 2+2 already | 15:44 |
dtantsur | \o/ | 15:44 |
jroll | <3 | 15:45 |
* jroll braces for impact | 15:45 | |
dtantsur | oh yeah, I hope I won't be killed tomorrow by angry users :) | 15:45 |
*** raddaoui has joined #openstack-ironic | 15:46 | |
*** raddaoui_ has joined #openstack-ironic | 15:46 | |
*** chaustin has joined #openstack-ironic | 15:46 | |
lucasagomes | dtantsur, I wouldn't approve if it was thursday | 15:46 |
jlvillal | Good morning dtantsur jrist lucasagomes EmilienM mgould NobodyCam sambetts TheJulia vdrok and everyone else :) | 15:46 |
lucasagomes | but since it's wednesday we may be safe :D | 15:46 |
lucasagomes | jlvillal, hello there | 15:46 |
jrist | good morning jlvillal :) | 15:46 |
jlvillal | And jroll too :) | 15:47 |
* jrist loves that | 15:47 | |
mgould | jlvillal, morning! | 15:47 |
lucasagomes | jrist, a-ha! hah | 15:47 |
lucasagomes | always man | 15:47 |
jrist | I get greeted even when it's not intended :) | 15:47 |
jlvillal | jrist: My auto-complete fail :) | 15:47 |
jroll | hi jrist! :) | 15:47 |
* jrist waves | 15:47 | |
jrist | hi jroll ! | 15:47 |
*** krtaylor has joined #openstack-ironic | 15:47 | |
jrist | (almost did hi jrist. lol) | 15:47 |
*** chenke_ has joined #openstack-ironic | 15:47 | |
*** chaustin has quit IRC | 15:47 | |
jroll | lol | 15:47 |
* jlvillal grumbles that hexchat claims it auto-completes in order of who it has seen most recently... | 15:47 | |
jroll | dtantsur: I'll try to get that running at some sort of scale soon | 15:47 |
dtantsur | morning jlvillal | 15:48 |
jlvillal | :) | 15:48 |
dtantsur | lucasagomes, lol yeah, wednesday fixes everything | 15:48 |
lucasagomes | yeah, or at least it shouldn't break it :-P | 15:49 |
mgould | EmilienM, going back to my second question, "how can I get the iPXE chainload images into the right place?"... | 15:50 |
mgould | does puppet-ironic currently install ironic, inspector etc from operating system packages? | 15:50 |
EmilienM | yes | 15:50 |
*** pas-ha has joined #openstack-ironic | 15:50 | |
EmilienM | RDO / UCA | 15:50 |
mgould | cool | 15:50 |
*** chenke has quit IRC | 15:50 | |
mgould | so the Right Thing would be to add a dependency from the inspector rpm/deb to ipxe and then copy the files across? | 15:51 |
jlvillal | mgould: Did you have a chance to try the Grenade stuff? | 15:51 |
mgould | jlvillal, not since your latest changes | 15:52 |
*** Sukhdev has joined #openstack-ironic | 15:52 | |
jlvillal | mgould: Okay. I'm reaching out to my openstackclient contact to see if I can get some help with the bug. | 15:52 |
jlvillal | https://bugs.launchpad.net/bugs/1549095 | 15:52 |
openstack | Launchpad bug 1549095 in Ironic "devstack fails while running Ironic grenade job: init__() got an unexpected keyword argument 'token'" [Undecided,In progress] - Assigned to John L. Villalovos (happycamp) | 15:52 |
mgould | jlvillal, thanks | 15:52 |
*** garthb_ has joined #openstack-ironic | 15:56 | |
*** daemontool has quit IRC | 15:57 | |
*** penick has quit IRC | 15:58 | |
*** baoli has quit IRC | 15:58 | |
*** baoli has joined #openstack-ironic | 15:59 | |
*** daemontool has joined #openstack-ironic | 16:02 | |
*** dims has joined #openstack-ironic | 16:03 | |
lucasagomes | openstackgerrit, is dead?! | 16:03 |
*** Nisha has quit IRC | 16:03 | |
NobodyCam | ??? | 16:03 |
*** Nisha has joined #openstack-ironic | 16:03 | |
lucasagomes | anyway... dtantsur https://review.openstack.org/#/c/284219/ that's adding the deprecated_for_removal info to the sample file | 16:03 |
lucasagomes | NobodyCam, just upload a patch and it didn't report :-) | 16:03 |
lucasagomes | uploaded* | 16:03 |
NobodyCam | oh the bot | 16:04 |
lucasagomes | yeah | 16:05 |
openstackgerrit | Brant Knudson proposed openstack/ironic: Pass roles to oslo_context's RequestContext https://review.openstack.org/284227 | 16:08 |
openstackgerrit | Brant Knudson proposed openstack/ironic: RequestContext use oslo.context's to/from_dict https://review.openstack.org/284228 | 16:08 |
jlvillal | lucasagomes: Long live openstackgerrit ! :) | 16:11 |
lucasagomes | lol | 16:11 |
NobodyCam | lol | 16:11 |
lucasagomes | hiccup!? | 16:11 |
jlvillal | lucasagomes: Well it liked the other two patches. Maybe it just doesn't like you? ;) | 16:11 |
lucasagomes | maybe yes | 16:12 |
*** chaustin has joined #openstack-ironic | 16:12 | |
NobodyCam | oh just over worked... I have seen it drop things before | 16:12 |
dtantsur | mkovacik_, https://review.openstack.org/#/c/260246/ | 16:12 |
jlvillal | lucasagomes: In my case it is usually, I think I pushed it but then I actually go back and look and see that Gerrit had rejected me :( | 16:12 |
NobodyCam | oh=or | 16:12 |
mkovacik_ | dtantsur, /me looks | 16:12 |
dtantsur | jlvillal, yeah, or is asking for confirmation about a patch chain | 16:12 |
jlvillal | Exactly :) | 16:13 |
jlvillal | I do an <enter> and switch to some other task... | 16:13 |
*** mgoddard__ has quit IRC | 16:14 | |
*** mgoddard has joined #openstack-ironic | 16:14 | |
lucasagomes | jlvillal, not a big deal tho... I just wanted to use "^" to show the patch instead of having to put the link in my msg | 16:17 |
*** keedya has joined #openstack-ironic | 16:17 | |
jlvillal | :) | 16:17 |
lucasagomes | NobodyCam, will fix all the bots as part of the PixieBot work | 16:17 |
lucasagomes | :-P | 16:17 |
*** rpioso has joined #openstack-ironic | 16:18 | |
NobodyCam | ieek :) | 16:18 |
jlvillal | lucasagomes: NobodyCam will fix everything :D | 16:18 |
lucasagomes | ++ | 16:18 |
NobodyCam | all the bot | 16:18 |
dtantsur | oh yeah, that's a big newton priority | 16:18 |
*** spandhe has joined #openstack-ironic | 16:18 | |
NobodyCam | all your bot belong to us | 16:18 |
jlvillal | Newton Abbey :) | 16:18 |
mkovacik_ | dtantsur, cool, just the implementation is missing; I guess that will be next patch | 16:18 |
dtantsur | mkovacik_, hopefully :) | 16:19 |
*** rajinir has joined #openstack-ironic | 16:19 | |
mkovacik_ | dtantsur, but one still has to maintain the state machine | 16:19 |
vdrok | morning jlvillal ! | 16:19 |
jlvillal | vdrok: o/ | 16:19 |
lucasagomes | mkovacik_, dtantsur not sure I got that patch... it's a merge of taskflow and tooz !? | 16:21 |
* lucasagomes reads the code | 16:21 | |
dtantsur | lucasagomes, it moves one part of taskflow to tooz, and it's probably good for us | 16:21 |
openstackgerrit | Merged openstack/ironic: Switch to Futurist library for asynchronous execution and periodic tasks https://review.openstack.org/264720 | 16:22 |
dtantsur | w000t! | 16:22 |
mkovacik_ | lucasagomes, seems the jobs abstraction and job board will be moved into tooz | 16:22 |
* dtantsur goes to the beach to celebrate | 16:22 | |
*** spandhe has quit IRC | 16:22 | |
mkovacik_ | dtantsur, yay :D | 16:22 |
NobodyCam | lol /me offers dtantsur a beach towel | 16:22 |
lucasagomes | mkovacik_, I see | 16:22 |
lucasagomes | interesting | 16:22 |
dtantsur | now watch for regressions everyone :) | 16:23 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Update ironic.conf.sample https://review.openstack.org/284218 | 16:24 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add deprecated_for_removal config info in ironic.conf.sample https://review.openstack.org/284219 | 16:24 |
* lucasagomes updates the patches to include dtantsur config option | 16:24 | |
jlvillal | dtantsur: I thought you lived in a land-locked country. Where's the beach? :) | 16:27 |
dtantsur | jlvillal, don't say so... | 16:27 |
* dtantsur cries in the corner | 16:27 | |
jlvillal | ;) | 16:27 |
jlvillal | dtantsur: You can go to Greece or one of those warm places. They have beaches! | 16:28 |
dtantsur | jlvillal, I plan on Italy, but it depends on many things | 16:28 |
* jlvillal has a very cold Oregon coast nearby... | 16:28 | |
jlvillal | nearby = 90 minute drive... | 16:28 |
dtantsur | this fits into my definition of nearby :) the closest real airport is 2 hours by bus | 16:29 |
jlvillal | dtantsur: Italy sounds fun. I went to the Cinque Terre there. It was nice. | 16:29 |
dtantsur | jlvillal, my wife, however, has plans about Japan. and our budget definitely won't survive 2 of these :) | 16:30 |
* jlvillal votes for Italy. As just went to Japan. | 16:31 | |
*** harlowja_at_home has joined #openstack-ironic | 16:31 | |
dtantsur | harlowja_at_home, hey there! the switch-to-futurist patch has now merged :) | 16:32 |
dtantsur | harlowja_at_home, now I wonder, is periodics code covered by the existing tempest job on futurist? or do we need an ironic job there? | 16:32 |
harlowja_at_home | dtantsur, hey, merged, woot!! | 16:33 |
*** garthb has joined #openstack-ironic | 16:33 | |
harlowja_at_home | dtantsur, tempest job on futurist? | 16:34 |
dtantsur | harlowja_at_home, yep | 16:35 |
dtantsur | it has something neutron-based now, right? | 16:35 |
* jlvillal likes the proposal about separating the design summit from the main summit | 16:35 | |
lucasagomes | jlvillal, ++ | 16:35 |
lucasagomes | hope to get one in dublin | 16:35 |
dtantsur | jlvillal, I like it, but I don't think it will solve the stated problems | 16:35 |
dtantsur | lucasagomes, I vote for Prague :) it's deadly cheap and very cool | 16:35 |
harlowja_at_home | dtantsur, so u are thinking about a gate/job that will run ironic(stuffs) on every commit right? | 16:36 |
dtantsur | harlowja_at_home, yep. to have the periodics code covered, if it's not already (I dunno what the current job covers) | 16:36 |
*** raddaoui__ has joined #openstack-ironic | 16:36 | |
lucasagomes | dtantsur, I'm good with prague too | 16:36 |
jlvillal | I'd vote for Prague too. Since I've been there in the last two years. Dublin sounds fun. Not sure how cheap it is. | 16:36 |
lucasagomes | jlvillal, cheaper than london :-) | 16:37 |
dtantsur | lol | 16:37 |
dtantsur | anything is cheaper than london, no? | 16:37 |
harlowja_at_home | dtantsur, let's see if that's possible dims yt | 16:37 |
lucasagomes | hah | 16:37 |
lucasagomes | dtantsur, oslo is also quite expensive AFAIK | 16:37 |
* jlvillal tries to think what is more expensive than London...Keeps thinking....Still thinking.... | 16:37 | |
harlowja_at_home | i vote for going to lucasagomes house | 16:37 |
harlowja_at_home | lol | 16:38 |
harlowja_at_home | summit in lucasagomes house | 16:38 |
harlowja_at_home | lol | 16:38 |
dtantsur | lol +1 | 16:38 |
lucasagomes | harlowja_at_home, +1 from me! | 16:38 |
harlowja_at_home | :) | 16:38 |
lucasagomes | that will be convenient | 16:38 |
dtantsur | lucasagomes, it took me TIME to realize that oslo is also a city :D | 16:38 |
NobodyCam | lol | 16:38 |
harlowja_at_home | u may have to redo the whole place with this many guys coming to your place... | 16:38 |
harlowja_at_home | the place may not survive that many guys/girls... | 16:38 |
harlowja_at_home | lol | 16:38 |
harlowja_at_home | to many nerdssssss | 16:39 |
harlowja_at_home | lol | 16:39 |
harlowja_at_home | dtantsur, i'll followup with dims about that job | 16:39 |
jrist | hey lucasagomes isn't it your birthday tomorrow? | 16:39 |
jrist | happy early birthday! | 16:39 |
harlowja_at_home | dtantsur, dims has been working onhttps://etherpad.openstack.org/p/dims-periodic-jobs | 16:39 |
harlowja_at_home | but that's slightly different | 16:39 |
dtantsur | yeah | 16:40 |
lucasagomes | harlowja_at_home, yeah, well probably the $$ from foundation will be enough to allow me buying a new house | 16:40 |
dtantsur | ok thanks man :) | 16:40 |
lucasagomes | so, I'm not worried | 16:40 |
harlowja_at_home | lucasagomes, :) | 16:40 |
lucasagomes | jrist, hey, yes tomorrow :-D | 16:40 |
lucasagomes | jrist, thank you! | 16:40 |
harlowja_at_home | lucasagomes, are u turning 18? | 16:40 |
dtantsur | LOL | 16:40 |
harlowja_at_home | man, u've grown up | 16:40 |
lucasagomes | harlowja_at_home, 21 | 16:40 |
harlowja_at_home | nice nice | 16:40 |
dtantsur | he can now drink cool stuff! | 16:40 |
lucasagomes | so i can drink | 16:40 |
harlowja_at_home | lol | 16:41 |
jroll | lmao | 16:41 |
jroll | happy early birthday lucas \o/ | 16:41 |
lucasagomes | thanks :D | 16:41 |
EmilienM | dtantsur: any hint on 403? | 16:42 |
EmilienM | I'm a bit lost. | 16:42 |
dtantsur | EmilienM, man I'm lost too | 16:42 |
dtantsur | lucasagomes, wanna join the debugging party? | 16:42 |
jroll | EmilienM: did you check the roles that the user has, vs the policy.json file? | 16:42 |
harlowja_at_home | dtantsur, soo what shall we do next for futurist :-P | 16:42 |
dtantsur | harlowja_at_home, rewrite it in Go of course \o/ | 16:43 |
harlowja_at_home | i have some idears, lol | 16:43 |
* dtantsur hides | 16:43 | |
harlowja_at_home | ha | 16:43 |
lucasagomes | dtantsur, sure, where's it? | 16:43 |
jroll | dtantsur: ++ | 16:43 |
dtantsur | lucasagomes, EmilienM tries to run our tempest in their puppet gate, and our API just returns 403 on anything | 16:43 |
EmilienM | jroll: the question is, it works perfectly when running tempest from tree | 16:43 |
dtantsur | lucasagomes, http://logs.openstack.org/86/283786/1/check/gate-puppet-openstack-integration-scenario002-tempest-dsvm-centos7/d56dd17/console.html#_2016-02-23_21_31_02_255 | 16:43 |
EmilienM | dtantsur: wait, it's also worth mentionning all is working when running tempest/baremetal | 16:44 |
EmilienM | our failures are happening with ironic/tempest tests | 16:44 |
harlowja_at_home | dtantsur, i'd like to be able to add more callback points to all the executors, but that somewhat involves messing around with the stdlib :( | 16:44 |
* lucasagomes looks | 16:44 | |
EmilienM | lucasagomes: all logs are here: http://logs.openstack.org/86/283786/1/check/gate-puppet-openstack-integration-scenario002-tempest-dsvm-centos7/d56dd17/logs | 16:44 |
harlowja_at_home | like having a way to know how long a submitted function has been one the work queue | 16:44 |
dtantsur | harlowja_at_home, messing with stdlib sounds like too much for fun to put it in production | 16:44 |
*** mgoddard_ has joined #openstack-ironic | 16:44 | |
harlowja_at_home | dtantsur, ya, sadly | 16:44 |
harlowja_at_home | *has been on the work queue | 16:45 |
lucasagomes | EmilienM, dtantsur jroll I wonder if has something to do with https://review.openstack.org/#/c/236982/ | 16:45 |
dtantsur | harlowja_at_home, btw you had some cool fancy statistics collectors in futurist, right? I'm thinking how to wire that in, so that we can watch how much every periodic tasks takes etc | 16:45 |
*** jistr has quit IRC | 16:45 | |
harlowja_at_home | dtantsur, it has some limitations due to said messing around with the python stdlib :( | 16:45 |
dtantsur | lucasagomes, maybe? | 16:45 |
harlowja_at_home | but yes, the basic exists | 16:45 |
harlowja_at_home | dtantsur, the issue is the wait time on the executor work queue, i don't have access to how long that is | 16:46 |
harlowja_at_home | without doing some nasties | 16:46 |
harlowja_at_home | lol | 16:46 |
*** baoli_ has joined #openstack-ironic | 16:46 | |
EmilienM | lucasagomes: it can be. But again, I run baremetal with success when running tests from tempest itself | 16:46 |
*** raddaoui_ has quit IRC | 16:47 | |
*** raddaoui has quit IRC | 16:47 | |
harlowja_at_home | dtantsur, but perhaps periodically have a function that self-dumps the periodic stats out? | 16:47 |
harlowja_at_home | ^ tried to use the word periodic as much as i could | 16:47 |
harlowja_at_home | lol | 16:47 |
dtantsur | harlowja_at_home, you succeeded, congrats! :D | 16:47 |
harlowja_at_home | :-P | 16:47 |
dtantsur | yeah, that's what I think. e.g. every 2 minutes dump statistics to the DEBUG log | 16:47 |
*** mgoddard has quit IRC | 16:47 | |
dtantsur | ... and I'll probably hack on it right now | 16:48 |
jroll | uhhhhhh | 16:48 |
dtantsur | jroll, objections? | 16:48 |
jroll | 2016-02-23 21:18:03.293 21731 INFO keystone.common.wsgi [req-007dd5ec-7aa4-46b8-bd4d-84cf1ccbb773 fcd7cd580dfd40cbb13d1a48b643460d 0ff67256196045c3a0faa4d6e30e7888 - default default] DELETE http://127.0.0.1:35357/v3/users/aad2cb8192994712b91f4bbb546402fb | 16:48 |
*** moshele has quit IRC | 16:48 | |
dtantsur | ah, it's not about our conversation | 16:48 |
*** baoli has quit IRC | 16:48 | |
jroll | wait wtf it uses a different user | 16:48 |
jroll | (later on) | 16:49 |
*** raddaoui__ has quit IRC | 16:49 | |
harlowja_at_home | dtantsur, sounds good to me | 16:49 |
*** Goneri has quit IRC | 16:49 | |
jroll | EmilienM: hrm, all the auth stuff looks fine to me :/ | 16:50 |
lucasagomes | yeah, just checked the policy.json file... looks fine | 16:51 |
*** piet has quit IRC | 16:51 | |
*** piet has joined #openstack-ironic | 16:52 | |
harlowja_at_home | dtantsur, https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Promise | 16:52 |
harlowja_at_home | for other ideas on futurist stuff | 16:52 |
harlowja_at_home | ignore that it says javascript, lol | 16:53 |
harlowja_at_home | it'd be nice to have some of those features/apis | 16:53 |
dtantsur | hmm, yeah | 16:53 |
harlowja_at_home | but slightly made hard by inability to mess around with internals of python stdlib | 16:54 |
harlowja_at_home | but guess i should use my ability to contribute to the PSF someday | 16:54 |
harlowja_at_home | sad part is that will take about 3 years to merge code there | 16:54 |
harlowja_at_home | they are really slow, lol | 16:54 |
harlowja_at_home | 3-300years | 16:55 |
harlowja_at_home | lol | 16:55 |
dtantsur | wow | 16:55 |
lucasagomes | EmilienM, I'm not expert in keystone, but http://logs.openstack.org/86/283786/1/check/gate-puppet-openstack-integration-scenario002-tempest-dsvm-centos7/d56dd17/logs/keystone/keystone.txt.gz#_2016-02-23_21_15_48_196 | 16:55 |
lucasagomes | does it means Ironic is using v3 ? | 16:55 |
lucasagomes | I remember it having some problems with ti | 16:55 |
lucasagomes | it* | 16:55 |
dtantsur | harlowja_at_home, ugh.. the Watcher object does not give access to the periodic task name? | 16:56 |
lucasagomes | https://bugs.launchpad.net/ironic/+bug/1494776 | 16:56 |
openstack | Launchpad bug 1494776 in Ironic "Ironic API fails when keystone /v2.0 pipeline is disabled" [High,Confirmed] - Assigned to Pavlo Shchelokovskyy (pshchelo) | 16:56 |
harlowja_at_home | dtantsur, hmmm | 16:56 |
jroll | lucasagomes: oh yeah, good point: http://logs.openstack.org/86/283786/1/check/gate-puppet-openstack-integration-scenario002-tempest-dsvm-centos7/d56dd17/logs/ironic/ironic-api.txt.gz#_2016-02-23_21_18_13_712 | 16:56 |
jroll | it is v3 | 16:56 |
harlowja_at_home | dtantsur, oops, ha | 16:56 |
lucasagomes | jroll, yeah | 16:56 |
*** Guest51435 is now known as mgagne | 16:56 | |
lucasagomes | EmilienM, ^ | 16:56 |
*** mgagne has quit IRC | 16:56 | |
*** mgagne has joined #openstack-ironic | 16:56 | |
*** Sukhdev has quit IRC | 16:57 | |
lucasagomes | jroll, we probably should prioritize that problem | 16:57 |
harlowja_at_home | dtantsur, seems like a useful thing to have, ha | 16:57 |
*** trown is now known as trown|lunch | 16:57 | |
lucasagomes | and get v3/ fixed in mitaka | 16:57 |
lucasagomes | pas-ha, I think you've been working on it ? ^ | 16:58 |
jroll | lucasagomes: yeah indeed | 16:58 |
dtantsur | harlowja_at_home, yeah. maybe even a dict 'name' -> (function, watcher) | 16:58 |
dtantsur | available on the worker | 16:58 |
harlowja_at_home | dtantsur, sure | 16:58 |
harlowja_at_home | dtantsur, the data exists, its already maintained in 2 lists | 16:59 |
harlowja_at_home | self._watchers = [] self._callables = [] | 16:59 |
dtantsur | exactly | 16:59 |
*** dims has quit IRC | 17:00 | |
pas-ha | lucasagomes, yes, I believe the move to Keystoneauth and sessions where possible should solve this. is there any new issue? | 17:00 |
*** amotoki has quit IRC | 17:00 | |
EmilienM | lucasagomes, dtantsur: yes we use v3 when we can. But it used to work like a few days ago. | 17:00 |
lucasagomes | pas-ha, gotcha... I will take a look at that patch | 17:00 |
harlowja_at_home | dtantsur, so i guess just a zip() of both and make a new function | 17:01 |
harlowja_at_home | and there u go, ha | 17:01 |
harlowja_at_home | done | 17:01 |
lucasagomes | pas-ha, EmilienM was having some problems with it running the temptest tests | 17:01 |
harlowja_at_home | function/method | 17:01 |
pas-ha | lucasagomes, thnks | 17:01 |
lucasagomes | http://logs.openstack.org/86/283786/1/check/gate-puppet-openstack-integration-scenario002-tempest-dsvm-centos7/d56dd17/console.html#_2016-02-23_21_31_02_255 | 17:01 |
*** ifarkas has quit IRC | 17:02 | |
EmilienM | pas-ha, lucasagomes: let me rephrase it (again): I'm having issue with tempest tests in ironic tree, not in tempest itself. | 17:02 |
EmilienM | tests in tempest itself just work. | 17:02 |
EmilienM | but when I run tests with tox -eall-plugin, it fails. | 17:03 |
jlvillal | mgould: If interested we are meeting over in #openstack-meeting for testing stuff | 17:03 |
pas-ha | is there Keystone V3 set up/used there? | 17:03 |
*** dims has joined #openstack-ironic | 17:03 | |
pas-ha | because currently we AFAIU we manually set up the keystone middleware the old, legacy way, which is working over v2, and AFAIK v2 has no notion of token issued through v3 | 17:04 |
pas-ha | so if the token got from v3 comes in to ironic, keystonemiddleware might have troubles to validate it | 17:05 |
pas-ha | or I may be completely wrong | 17:05 |
pas-ha | may try directly with passing auth/session into ironic client | 17:06 |
EmilienM | I'm going to file a bug | 17:11 |
*** chaustin has quit IRC | 17:12 | |
lucasagomes | EmilienM, thanks | 17:14 |
*** achanda has joined #openstack-ironic | 17:15 | |
*** achanda has quit IRC | 17:15 | |
Nisha | lucasagomes, regarding the error, {u'debuginfo': None, u'faultcode': u'Client', u'faultstring': u'Not found: Extension with id iscsi not found.'} | 17:15 |
*** achanda has joined #openstack-ironic | 17:15 | |
Nisha | The ramdisk has the iscsi registered as the extensions | 17:16 |
Nisha | I reinstalled the IPA in ramdisk also | 17:16 |
Nisha | still i see the same issue | 17:16 |
lucasagomes | Nisha, :-( | 17:18 |
lucasagomes | Nisha, I probably would need to investigate it deeper to figure out what's going on | 17:18 |
lucasagomes | Nisha, how are you building the IPA ramdisk? DIB? | 17:19 |
Nisha | Yes ramdisk was initially built with DIB | 17:19 |
Nisha | i opened up the ramdisk | 17:19 |
mgould | jlvillal, thanks | 17:20 |
Nisha | and installed the IPA and ironic-li | 17:20 |
* mgould had gone out to buy tea | 17:20 | |
Nisha | ironic-lib* | 17:20 |
Nisha | and packed it again | 17:20 |
NobodyCam | night mgould | 17:20 |
*** max_lobur has quit IRC | 17:21 | |
mgould | NobodyCam, good night! | 17:22 |
mgould | TheJulia, are you around today? | 17:22 |
dims | jroll : dtantsur : identified a potential break with next set of oslo releases (oslo.context specifically) https://bugs.launchpad.net/oslo.context/+bug/1549317 - evidence is here - https://review.openstack.org/#/c/284151/ | 17:23 |
openstack | Launchpad bug 1549317 in oslo.policy "Ironic TestACL and TestTrustedCallHook fails with latest oslo.* master" [Undecided,New] | 17:23 |
lucasagomes | Nisha, gotcha, installed in a chroot fashion ? | 17:26 |
lucasagomes | python setup.py install | 17:26 |
lucasagomes | or venv or something... in any case, dunno why the entry point is not there | 17:26 |
Nisha | yes | 17:26 |
lucasagomes | and I dunno much about hw to debug setuptools entry points (I'm sure we can google it) | 17:26 |
jroll | dims: anything for us to do? | 17:27 |
dims | jroll : give me a bit to look at both the oslo.context change and the current code in ironic. will ping back :) | 17:28 |
Nisha | i can see it in the reamdisk at /usr/lib/python2.7/site-packages/ironic_python_agent-1.0.1.dev105-py2.7.egg-info | 17:28 |
jroll | sure thing dims | 17:28 |
*** blakec has joined #openstack-ironic | 17:29 | |
dtantsur | dims, thanks! | 17:29 |
dtantsur | calling it a day, see you | 17:31 |
jroll | later dtantsur :) | 17:31 |
lucasagomes | dtantsur, see ya | 17:31 |
*** dtantsur is now known as dtantsur|afk | 17:31 | |
Nisha | lucasagomes, any clue what can be the issue? | 17:33 |
*** Goneri has joined #openstack-ironic | 17:33 | |
lucasagomes | Nisha, not off the top of my head :-/ I would try to build the ramdisk from source without modifying it later tho | 17:34 |
lucasagomes | you can do it with coreos right? Add ironic-lib to the requirements.txt and run the image build urself from the modified code | 17:34 |
*** spandhe has joined #openstack-ironic | 17:34 | |
lucasagomes | see if that works | 17:34 |
Nisha | lucasagomes, no i get some basic issues while building image with coreos | 17:35 |
Nisha | i had tried up a lot, and then i gave up and opened up the working ramdisk | 17:35 |
Nisha | and replaced the IPA with my changed IPA | 17:35 |
Nisha | and packed up again | 17:35 |
Nisha | jroll, ^^^ i enabled a ramdisk with IPA and ironic-lib changes required for agent drivers to support partition images. The ramdisk works for agent_ipmitool driver with partition image | 17:36 |
jroll | it's almost impossible to debug a ramdisk that wasn't built with our tooling :/ | 17:36 |
lucasagomes | Nisha, yeah I think that's something in the process of building that image :-/ | 17:36 |
jroll | Nisha: that's great to hear :) | 17:37 |
Nisha | but pxe_ipmitool netboot fails | 17:37 |
lucasagomes | jroll, yeah | 17:37 |
Nisha | with the same ramdisk | 17:37 |
jroll | Nisha: what kind of failure? | 17:37 |
lucasagomes | jroll, {u'debuginfo': None, u'faultcode': u'Client', u'faultstring': u'Not found: Extension with id iscsi not found.'} | 17:38 |
*** max_lobur has joined #openstack-ironic | 17:38 | |
lucasagomes | I think it's the setuptools entrypoint | 17:38 |
lucasagomes | for the iscsi extension | 17:38 |
jroll | well that should fail localboot too right? | 17:38 |
jroll | oh agent_ipmitool works, right | 17:38 |
Nisha | i havent touched any code for iscsi_deploy. and i am getting error like | 17:38 |
Nisha | > /opt/stack/ironic/ironic/drivers/modules/iscsi_deploy.py(442)do_agent_iscsi_deploy() | 17:38 |
Nisha | -> import pdb | 17:38 |
Nisha | (Pdb) c | 17:38 |
Nisha | > /opt/stack/ironic/ironic/drivers/modules/iscsi_deploy.py(444)do_agent_iscsi_deploy() | 17:38 |
Nisha | -> if result['command_status'] == 'FAILED': | 17:38 |
lucasagomes | localboot uses "image" extension | 17:38 |
Nisha | (Pdb) print result | 17:38 |
Nisha | {u'debuginfo': None, u'faultcode': u'Client', u'faultstring': u'Not found: Extension with id iscsi not found.'} | 17:38 |
Nisha | (Pdb) n | 17:38 |
Nisha | KeyError: ('command_status',) | 17:38 |
Nisha | yes | 17:38 |
jroll | please use paste.openstack.org | 17:38 |
Nisha | lucasagomes, i tested netboot for agent_ipmitool also | 17:39 |
jroll | (in the future) | 17:39 |
Nisha | jroll, sure | 17:39 |
jroll | so yeah, if you used pip install to install your new IPA, it should work | 17:39 |
jroll | other than that I can't really help debug a ramdisk not built with normal tooling | 17:39 |
Nisha | agent_ipmitool for netboot in bios for partition image worked fine with the image | 17:39 |
jroll | oh you know what | 17:40 |
jroll | I wonder if the iscsi extension isn't loading properly | 17:40 |
jroll | and if we swallow that error | 17:40 |
Nisha | I installed the IPA in the ramdisk by copying my IPA in the /tmp directory of ramdisk, and from there i did sudo -E python setup.py install | 17:40 |
jroll | here: https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/agent.py#L142 | 17:41 |
sambetts | jlvillal: was there a QA meeting today? | 17:41 |
Nisha | How do you know if its loading properly or not | 17:41 |
jlvillal | sambetts: There sure was! :) | 17:41 |
jroll | Nisha: that's yet to be seen :) | 17:41 |
* jroll looking | 17:41 | |
jlvillal | sambetts: http://eavesdrop.openstack.org/meetings/ironic_qa/2016/ironic_qa.2016-02-24-17.00.log.html | 17:41 |
sambetts | damn I completely missed it ... | 17:42 |
sambetts | XD | 17:42 |
jlvillal | :( | 17:42 |
jlvillal | sambetts: It is the highlight of the week for pretty much everyone :) | 17:42 |
sambetts | haha :-P I was going to tell everyone how much fun I'm having installing my CI ;) | 17:43 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Trying latest oslo.* from master https://review.openstack.org/284151 | 17:45 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: Tolerate roles in context.RequestContext https://review.openstack.org/284300 | 17:45 |
sinval | sambetts: oh, I notice that you like strong feelings hahaha | 17:45 |
pas-ha | lucasagomes, it seems there is some new magic in keystonemiddleware | 17:46 |
pas-ha | http://logs.openstack.org/86/283786/1/check/gate-puppet-openstack-integration-scenario002-tempest-dsvm-centos7/d56dd17/logs/ironic/ironic-api.txt.gz#_2016-02-23_21_18_01_048 | 17:46 |
pas-ha | and next line | 17:46 |
dims | jroll : https://review.openstack.org/#/c/284300/1/ironic/common/context.py should do the trick | 17:46 |
pas-ha | although we configure keystonemiddleware to talk to v2.0 in ironic.conf, internally it goes to v3 nevertheless | 17:47 |
lucasagomes | pas-ha, ouch it's being redirected to v3 | 17:47 |
jroll | dims: we'll need to backport this to liberty too, right? | 17:47 |
* sambetts is calling it a evening | 17:47 | |
dims | jroll : yep | 17:47 |
jroll | urgh | 17:47 |
dims | jroll : let's wait till the jobs are green | 17:48 |
*** moshele has joined #openstack-ironic | 17:48 | |
jroll | dims: well, I +2'd anyway :) | 17:48 |
lucasagomes | dims, cool | 17:48 |
jroll | thank you for helping | 17:48 |
dims | jroll : thanks! | 17:48 |
* lucasagomes +2 waiting for the jobs | 17:49 | |
*** sambetts is now known as sambetts|afk | 17:49 | |
lucasagomes | I will call it a day, so jroll will probably push the final button on that | 17:49 |
mgould | sambetts|afk, good night! | 17:49 |
*** trown|lunch is now known as trown | 17:50 | |
jroll | lucasagomes: sambetts|afk: have a good night! :) | 17:50 |
lucasagomes | thanks | 17:51 |
lucasagomes | see y'all tomorrow | 17:51 |
*** lucasagomes is now known as lucas-hungry | 17:51 | |
*** Nisha has quit IRC | 17:51 | |
lucas-hungry | Nisha, I can try helping why coreos build image fails for you tomorrow | 17:51 |
lucas-hungry | if you don't figure it out in the meantime | 17:51 |
lucas-hungry | night all ! | 17:51 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic-python-agent: Warn if extension fails to load https://review.openstack.org/284309 | 17:52 |
EmilienM | lucas-hungry, dtantsur|afk : https://bugs.launchpad.net/ironic/+bug/1549414 | 17:52 |
openstack | Launchpad bug 1549414 in Ironic "Baremetal API tests fail with 403 error (from ironic tree)" [Undecided,New] | 17:52 |
jroll | nisha just quit | 17:52 |
jroll | wat | 17:52 |
EmilienM | please bring love in this bug | 17:52 |
jroll | nisha pls, I just wrote a patch to help :( | 17:53 |
EmilienM | jroll: I promess, I won't quit if you fix my bug :-P | 17:53 |
*** Nisha has joined #openstack-ironic | 17:53 | |
lucas-hungry | EmilienM, thanks | 17:53 |
jroll | EmilienM: I didn't fix it, just added logging, I can happily do that for you :P | 17:53 |
* jroll wishes nisha had code posted so he could test... | 17:54 | |
lucas-hungry | jroll, btw, if you have time https://review.openstack.org/#/c/284036/ (devstack-gate change to work with ipmitool) | 17:54 |
jroll | cool | 17:54 |
lucas-hungry | it's like 3LOC | 17:54 |
lucas-hungry | so, no rush :-) | 17:54 |
* lucas-hungry goes now | 17:54 | |
Nisha | I can post the code, i also wish to | 17:54 |
EmilienM | jroll: no you can't, I'm running RDO trunk packaging, not source. | 17:54 |
Nisha | but pxe_ipmi is not working | 17:54 |
Nisha | for me | 17:55 |
Nisha | atleast | 17:55 |
*** alexpilotti has joined #openstack-ironic | 17:55 | |
*** Marga_ has quit IRC | 17:55 | |
jroll | Nisha: oh, welcome back | 17:55 |
jroll | Nisha: add this patch and see if you get that warning https://review.openstack.org/284309 | 17:55 |
Nisha | jroll, yes it got disconnected in between | 17:55 |
Nisha | jroll, sure let me try. Should i post the patch without pxe_ipmi passing | 17:57 |
Nisha | ? | 17:57 |
*** moshele has quit IRC | 17:57 | |
jroll | Nisha: posting code never hurts anything :) | 17:57 |
jroll | I thought you had gone so I wanted to try it :) | 17:57 |
Nisha | It has three patches, ironic-lib, IPA and ironic | 17:57 |
Nisha | No it got disconnected in between | 17:57 |
jroll | right, I know that now | 17:58 |
Nisha | let me try with your patch and see if i get some loading error... | 17:58 |
Nisha | but if extension fails to load, how can i make that success | 17:58 |
Nisha | ? | 17:58 |
jroll | by fixing the bug that causes loading to fail | 17:59 |
*** mkovacik_ has quit IRC | 17:59 | |
*** thrash|biab is now known as thrash | 18:00 | |
*** rpioso has quit IRC | 18:00 | |
EmilienM | jroll: can you make sure the bug I reported is triaged? It looks pretty high to me. | 18:01 |
*** derekh has quit IRC | 18:01 | |
jroll | EmilienM: dtantsur|afk is our main triager, it will get done | 18:02 |
EmilienM | oh you have a triager ok. Cool! | 18:02 |
*** mgoddard_ has quit IRC | 18:02 | |
*** mgoddard has joined #openstack-ironic | 18:03 | |
*** alexpilotti has quit IRC | 18:07 | |
*** praneshp has joined #openstack-ironic | 18:07 | |
*** alexpilo_ has joined #openstack-ironic | 18:07 | |
*** suro-patz has joined #openstack-ironic | 18:07 | |
Nisha | jroll, it doesnt give any warning while installing IPA | 18:09 |
Nisha | in ramdisk | 18:09 |
jroll | Nisha: when running IPA | 18:09 |
Nisha | jroll, i used ur patch, and then did IPA installation in ramdisk | 18:10 |
Nisha | ohk, let me pack it up and see | 18:10 |
jroll | Nisha: it should happen at IPA startup | 18:10 |
Nisha | jroll, whats the priority for RAID for Mitaka | 18:11 |
Nisha | there are two patches pending for RAID | 18:11 |
Nisha | so just asking | 18:12 |
Nisha | #Link https://review.openstack.org/226330, https://review.openstack.org/226234 | 18:12 |
jroll | Nisha: yes, those are priority | 18:13 |
jroll | thank you | 18:13 |
*** chaustin has joined #openstack-ironic | 18:13 | |
*** degorenko is now known as _degorenko|afk | 18:14 | |
openstackgerrit | vinay kumar muddu proposed openstack/proliantutils: Changes to NIC info retrieving logic https://review.openstack.org/273118 | 18:18 |
*** alexpilotti has joined #openstack-ironic | 18:19 | |
*** alexpilo_ has quit IRC | 18:20 | |
Nisha | jroll, vdrok, regarding https://review.openstack.org/246036... | 18:24 |
Nisha | we could replace the RequestContext with get_admin_context() as per suggestion, but I dont know the implcation of changing it | 18:25 |
Nisha | and the issue was seen only in fail over scenario | 18:26 |
jroll | Nisha: well, it can't be any more broken than it is now, right? | 18:26 |
jroll | :) | 18:26 |
jroll | get_admin_context sounds right to me | 18:26 |
Nisha | it can be correct, if it breaks the normal deploy behaviour also | 18:27 |
vdrok | Nisha, I've tried that locally and it worked for me, but please double check :) | 18:27 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Trying latest oslo.* from master https://review.openstack.org/284151 | 18:27 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: Tolerate roles in context.RequestContext https://review.openstack.org/284300 | 18:27 |
openstackgerrit | Lilia Sampaio proposed openstack/ironic-specs: Dynamic allocation of nodes for the OneView drivers https://review.openstack.org/275726 | 18:27 |
Nisha | vdrok, the fix needs the multiconductor setup and now i dont have that handy :( | 18:27 |
jroll | I don't see how it could break deploys | 18:28 |
dims | jroll : foiled by flake8 :) the py27,34 passed. so updated the review ^^ | 18:28 |
jroll | anywhere this is used is currently broken, I believe | 18:28 |
jroll | dims: heh, such is life | 18:28 |
Nisha | I fixed and raised the patch when i had multiconductor setup | 18:28 |
Nisha | jroll, hmm i agree... | 18:28 |
Nisha | but then i will need multiconductor setup | 18:29 |
Nisha | vdrok, how did u tested it locally? | 18:29 |
jroll | you could run two conductors in devstack | 18:29 |
Nisha | in normal deploy? or with takeover? | 18:29 |
Nisha | in a single devstack? | 18:29 |
jroll | yes | 18:29 |
Nisha | how? | 18:29 |
Nisha | i used two nodes for multiconductor setup | 18:30 |
jroll | by running ironic-conductor again, with a different config file to set different hostname | 18:30 |
*** ohamada has quit IRC | 18:30 | |
*** sinval is now known as sinval-brb | 18:30 | |
*** electrofelix has quit IRC | 18:30 | |
vdrok | Nisha, yes, I was testing with devstack all-in-one | 18:30 |
Nisha | vdrok, did u test the failover also? | 18:31 |
Nisha | for pxe drivers | 18:31 |
vdrok | Nisha, nope | 18:31 |
Nisha | this issue was seen for conductor failover only | 18:31 |
Nisha | the normal deploy went thru with existing code | 18:31 |
Nisha | after taht when i did conductor fail over i could see glance fails with "unauthorize error" for downloading the image | 18:32 |
Nisha | And when i fixed it as the patch does, then i faced this issue | 18:33 |
Nisha | https://review.openstack.org/246033 | 18:33 |
Nisha | vdrok, is it possible for you to test it for failover also | 18:33 |
vdrok | Nisha, sorry, I'm leaving now so cannot setup this right now. i can try tomorrow if needed | 18:34 |
Nisha | Ok Thanks | 18:34 |
*** Marga_ has joined #openstack-ironic | 18:34 | |
Nisha | jroll, i dont see any warning from IPA | 18:34 |
Nisha | but same error | 18:34 |
jroll | Nisha: interesting | 18:35 |
Nisha | 2016-02-24 10:37:32.206 DEBUG ironic.drivers.modules.agent_client [-] Agent command iscsi.start_iscsi_target for node bed3bce8-827f-40cf-954e-3bfb99f0753b returned result None, error None, HTTP status code 404 from (pid=29721) _command /opt/stack/ironic/ironic/drivers/modules/agent_client.py:95 | 18:35 |
*** lhcheng has joined #openstack-ironic | 18:36 | |
jroll | Nisha: can you paste the full IPA logs please, from startup to end? | 18:36 |
Nisha | IPA logs from conductor? | 18:37 |
jroll | Nisha: no, from IPA | 18:37 |
openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient: Use keystoneauth instead of keystoneclient https://review.openstack.org/275722 | 18:38 |
openstackgerrit | Vladyslav Drok proposed openstack/bifrost: DO NOT MERGE https://review.openstack.org/276360 | 18:38 |
Nisha | IPA will log to the conductor only ? am i correct? | 18:38 |
vdrok | good night everyone! | 18:38 |
jroll | it will log to journald in the ramdisk | 18:38 |
*** stevemar has joined #openstack-ironic | 18:38 | |
jroll | and you can add systemd.journald.forward_to_console=yes to the kernel cmd line to make it log to console | 18:39 |
Nisha | or i shud login to the ramdisk on the server and get using journalctl? | 18:39 |
jroll | yes | 18:39 |
Nisha | but copy paste doesnt work on console | 18:39 |
Nisha | :( | 18:39 |
jroll | it does on my console | 18:39 |
jroll | or you can add your ssh key to IPA | 18:39 |
Nisha | oh ...lot of work... | 18:40 |
Nisha | IPA is difficult | 18:40 |
jroll | it really isn't | 18:40 |
JayF | Nisha: if you're using coreos, add ssh_key="ADSFGHJADFSGKLSDFHGDFKJGDFS user@host" to your kernel command line | 18:40 |
JayF | to easily inject a single key | 18:40 |
jroll | it's unpacking and repacking the image making it difficult | 18:40 |
Nisha | I can give the screenshot | 18:41 |
Nisha | if that works | 18:41 |
jroll | screenshots are fine | 18:41 |
Nisha | but where shud i paste? | 18:41 |
jroll | not ideal but better than nothing | 18:41 |
jroll | imgur.com is a decent image host | 18:41 |
*** alexpilotti has quit IRC | 18:43 | |
*** alexpilotti has joined #openstack-ironic | 18:43 | |
Nisha | it gives "Error root KeyError: u'iscsi' | 18:45 |
*** ijw has joined #openstack-ironic | 18:46 | |
Nisha | jroll, ^^^ | 18:46 |
Nisha | JayF, using Fedora ramdisk | 18:47 |
JayF | Nisha: ah, then IDK :) | 18:47 |
jroll | Nisha: I'd like the full logs, especially from startup | 18:47 |
*** jcoufal has quit IRC | 18:48 | |
jroll | that's just telling us what we already know: the extension isn't loaded | 18:48 |
Nisha | jroll, ah | 18:48 |
jroll | there should be a warning during startup that says it failed to load | 18:48 |
Nisha | jroll, i can see extensions.image and extensions.iscsi fails to load | 18:49 |
jroll | "Extension iscsi failed to load with exception ..." | 18:49 |
Nisha | No module named rtslib | 18:49 |
jroll | \o/ | 18:49 |
Nisha | oh means i need to install this in ramdisk and then it shud work | 18:49 |
jroll | https://github.com/openstack/ironic-python-agent/blob/master/requirements.txt#L21 | 18:49 |
jroll | yep | 18:49 |
Nisha | hmmm | 18:49 |
jroll | maybe just install all of requirements.txt :) | 18:49 |
Nisha | jroll thanks for ur patch | 18:50 |
jroll | welcome | 18:50 |
Nisha | it helped a lot | 18:50 |
*** Marga_ has quit IRC | 18:50 | |
*** ijw has quit IRC | 18:50 | |
*** Marga_ has joined #openstack-ironic | 18:51 | |
jroll | Nisha: btw, the build automation we provide handles these for you | 18:52 |
Nisha | yes, but there are some issues in coreos build tool ...doesnt run on my system | 18:52 |
Nisha | and anyway i had to test my chanegs so many times | 18:52 |
Nisha | so now i know the issue, i shud be able to resolve it | 18:53 |
openstackgerrit | Gonéri Le Bouder proposed openstack/ironic-python-agent: iscsi: wipe part table before starting the target https://review.openstack.org/284347 | 18:54 |
*** chenke__ has joined #openstack-ironic | 18:54 | |
*** Nisha has quit IRC | 18:55 | |
*** rajinir has quit IRC | 18:55 | |
*** daemontool has quit IRC | 18:55 | |
*** Nisha has joined #openstack-ironic | 18:55 | |
*** chenke_ has quit IRC | 18:57 | |
mgould | reviews please! https://review.openstack.org/#/c/276187 | 18:58 |
* mgould -> home; good night everyone! | 18:58 | |
*** ijw has joined #openstack-ironic | 18:59 | |
*** raddaoui has joined #openstack-ironic | 19:01 | |
*** raddaoui_ has joined #openstack-ironic | 19:01 | |
*** raddaoui__ has joined #openstack-ironic | 19:01 | |
*** mgould has quit IRC | 19:02 | |
harlowja_at_home | dtantsur|afk, sooo if u want http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/devstack-gate.yaml#n1374 is apparently the place to add-in futurist | 19:03 |
harlowja_at_home | that will make it get included (from git) for dvsm | 19:03 |
harlowja_at_home | i think that's what u want? | 19:03 |
Nisha | jroll, and others can we have some reviews on RAID patches? | 19:04 |
*** ijw has quit IRC | 19:04 | |
*** alineb has quit IRC | 19:05 | |
*** ijw has joined #openstack-ironic | 19:05 | |
*** afaranha_ has quit IRC | 19:06 | |
*** Nisha_away has joined #openstack-ironic | 19:10 | |
*** Nisha has quit IRC | 19:10 | |
*** Sukhdev has joined #openstack-ironic | 19:10 | |
*** athomas has quit IRC | 19:14 | |
*** afaranha has joined #openstack-ironic | 19:19 | |
*** pas-ha has quit IRC | 19:21 | |
*** ChubYann has joined #openstack-ironic | 19:22 | |
*** mkovacik has joined #openstack-ironic | 19:24 | |
Nisha_away | jroll, it fails to start IPA after installing all the dependencies | 19:26 |
harlowja_at_home | dtantsur|afk, https://review.openstack.org/#/c/284367/ | 19:26 |
Nisha_away | jroll, i will debug it further tomorrow | 19:27 |
Nisha_away | but it fails to start IPA giving the error as "Configuration file /usr/lib/systemd/system/ironic-python-agent.service is amrked executable. please remove executable permission bits. proceeding anyway" | 19:29 |
Nisha_away | and then in next line gives the systemd exec error | 19:29 |
*** jaybeale has quit IRC | 19:29 | |
*** Nisha_away has quit IRC | 19:34 | |
openstackgerrit | Gonéri Le Bouder proposed openstack/ironic: ipxe: add --timeout parameter to kernel and initrd https://review.openstack.org/283893 | 19:34 |
Goneri | Is it possible to have a case where the device exposed over iscsi by python-ironic-agent must not be modified? | 19:43 |
Goneri | I try to answer a question from TheJulia here https://review.openstack.org/#/c/284347/1 | 19:43 |
*** trown is now known as trown|brb | 19:45 | |
*** rloo has joined #openstack-ironic | 19:48 | |
*** trown|brb is now known as trown | 19:49 | |
*** afaranha has left #openstack-ironic | 19:58 | |
*** alexpilotti has quit IRC | 19:58 | |
*** penick has joined #openstack-ironic | 20:01 | |
*** alexpilotti has joined #openstack-ironic | 20:01 | |
*** alexpilotti has quit IRC | 20:01 | |
*** mkovacik has quit IRC | 20:05 | |
openstackgerrit | Gonéri Le Bouder proposed openstack/ironic-python-agent: iscsi: wipe part table before starting the target https://review.openstack.org/284347 | 20:07 |
*** nathan-h_ has joined #openstack-ironic | 20:07 | |
*** alexpilotti has joined #openstack-ironic | 20:08 | |
*** nathan-h_ has quit IRC | 20:11 | |
*** alexpilotti has quit IRC | 20:12 | |
jlvillal | Goneri: On that patch. Is there a reason it is iscsi specific? Can it be more generic? | 20:16 |
*** alexpilotti has joined #openstack-ironic | 20:17 | |
*** chaustin has quit IRC | 20:30 | |
*** alexpilotti has quit IRC | 20:36 | |
*** Marga_ has quit IRC | 20:48 | |
*** Sukhdev has quit IRC | 20:48 | |
*** Marga_ has joined #openstack-ironic | 20:48 | |
*** jaybeale has joined #openstack-ironic | 20:49 | |
*** mkovacik has joined #openstack-ironic | 20:51 | |
Goneri | jlvillal, hi, I believe to related to the way the iscsi driver works | 20:51 |
jlvillal | Goneri: I left a comment in the patch. | 20:51 |
Goneri | jlvillal, it assumes the kernel will by itself reload the partition table, but sometime it does happens. | 20:52 |
Goneri | jlvillal, Yep and I answered some minutes ago | 20:52 |
jlvillal | Goneri: thanks | 20:52 |
*** boris-42 has joined #openstack-ironic | 20:53 | |
*** chaustin has joined #openstack-ironic | 20:54 | |
*** penick has quit IRC | 21:00 | |
*** piet has quit IRC | 21:00 | |
sinval-brb | good night folks, see you'all tomorrow | 21:01 |
*** sinval-brb is now known as sinval-afk | 21:01 | |
*** achanda has quit IRC | 21:01 | |
NobodyCam | night sinval-afk | 21:06 |
*** pcaruana has quit IRC | 21:10 | |
openstackgerrit | Merged openstack/ironic-lib: Updated from global requirements https://review.openstack.org/283985 | 21:11 |
*** Goneri has quit IRC | 21:13 | |
*** trown is now known as trown|outtypewww | 21:14 | |
*** chaustin has quit IRC | 21:19 | |
*** chaustin has joined #openstack-ironic | 21:22 | |
jlvillal | NobodyCam: Do you know what are release schedule is? | 21:23 |
jlvillal | NobodyCam: Like when is the cut-off for things to get into Mitaka | 21:23 |
*** raddaoui__ has quit IRC | 21:23 | |
*** raddaoui has quit IRC | 21:23 | |
*** raddaoui_ has quit IRC | 21:23 | |
jlvillal | Or anyone else who knows :) | 21:23 |
NobodyCam | jlvillal: like http://releases.openstack.org/mitaka/schedule.html | 21:24 |
jlvillal | NobodyCam: So are we following milestones? | 21:24 |
jlvillal | So M-3 is next week then for us. | 21:24 |
openstackgerrit | Kurt Taylor proposed openstack/ironic-python-agent: Add DIB ironic-agent element to readme for IPA https://review.openstack.org/272774 | 21:32 |
openstackgerrit | Merged openstack/ironic: Tolerate roles in context.RequestContext https://review.openstack.org/284300 | 21:33 |
*** ppiela has joined #openstack-ironic | 21:33 | |
*** jaybeale has quit IRC | 21:33 | |
*** ekarlso- has quit IRC | 21:34 | |
*** ekarlso- has joined #openstack-ironic | 21:34 | |
NobodyCam | :) | 21:34 |
devananda | jlvillal: we are not exactly following milestones for the server, but we DO need to cut a release soon | 21:37 |
devananda | jlvillal: and that will be the basis for mitaka integrated release, yes | 21:37 |
devananda | jlvillal: also, our client and our libraries are bound by common milestones | 21:37 |
jlvillal | devananda: Thanks. I thought so. I see we are 'release' with intermediary | 21:37 |
devananda | yes | 21:37 |
mrda | Morning Ironic | 21:40 |
*** krtaylor has quit IRC | 21:41 | |
*** mtanino__ has joined #openstack-ironic | 21:41 | |
*** achanda has joined #openstack-ironic | 21:43 | |
*** achanda has quit IRC | 21:45 | |
*** ndipanov has quit IRC | 21:45 | |
*** MattMan has quit IRC | 21:45 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/283983 | 21:45 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Add comments to generator.py https://review.openstack.org/283954 | 21:45 |
*** MattMan has joined #openstack-ironic | 21:45 | |
*** raddaoui has joined #openstack-ironic | 21:54 | |
*** Sukhdev has joined #openstack-ironic | 21:54 | |
*** raddaoui__ has joined #openstack-ironic | 21:54 | |
*** raddaoui_ has joined #openstack-ironic | 21:54 | |
*** krtaylor has joined #openstack-ironic | 21:55 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: WIP: Add unit tests for 'ironic/common/config_generator/generator.py' https://review.openstack.org/283937 | 21:58 |
*** suro-patz has quit IRC | 22:01 | |
*** suro-patz has joined #openstack-ironic | 22:02 | |
*** Sukhdev has quit IRC | 22:03 | |
*** raddaoui__ has quit IRC | 22:06 | |
*** raddaoui_ has quit IRC | 22:06 | |
*** raddaoui has quit IRC | 22:06 | |
*** daemontool has joined #openstack-ironic | 22:11 | |
*** Sukhdev has joined #openstack-ironic | 22:14 | |
*** dims has quit IRC | 22:14 | |
*** Sukhdev has quit IRC | 22:17 | |
*** Sukhdev has joined #openstack-ironic | 22:17 | |
*** chaustin has quit IRC | 22:18 | |
*** raildo is now known as raildo-afk | 22:19 | |
*** daemontool has quit IRC | 22:23 | |
*** chaustin has joined #openstack-ironic | 22:25 | |
*** dprince has quit IRC | 22:26 | |
*** penick has joined #openstack-ironic | 22:27 | |
*** Sukhdev has quit IRC | 22:29 | |
*** rloo has quit IRC | 22:29 | |
*** praneshp has quit IRC | 22:34 | |
*** vishwana_ is now known as vishwanathj | 22:39 | |
*** chaustin has quit IRC | 22:40 | |
*** uunsr has quit IRC | 22:47 | |
*** harshs has joined #openstack-ironic | 22:57 | |
*** harshs has quit IRC | 22:58 | |
*** suro-patz has quit IRC | 23:00 | |
*** dims has joined #openstack-ironic | 23:08 | |
*** achanda has joined #openstack-ironic | 23:10 | |
*** suro-patz has joined #openstack-ironic | 23:10 | |
*** thrash is now known as thrash|g0ne | 23:12 | |
*** Sukhdev has joined #openstack-ironic | 23:15 | |
*** blakec has quit IRC | 23:23 | |
*** praneshp has joined #openstack-ironic | 23:31 | |
*** praneshp_ has joined #openstack-ironic | 23:32 | |
*** praneshp has quit IRC | 23:35 | |
*** praneshp_ is now known as praneshp | 23:35 | |
*** jaybeale has joined #openstack-ironic | 23:44 | |
*** raddaoui has joined #openstack-ironic | 23:48 | |
*** raddaoui__ has joined #openstack-ironic | 23:48 | |
*** raddaoui_ has joined #openstack-ironic | 23:48 | |
*** jaybeale has quit IRC | 23:49 | |
*** smoriya_ has joined #openstack-ironic | 23:50 | |
*** raddaoui has quit IRC | 23:50 | |
*** raddaoui_ has quit IRC | 23:50 | |
*** raddaoui__ has quit IRC | 23:50 | |
openstackgerrit | Merged openstack/ironic: Move _from_db_object() into base class https://review.openstack.org/276960 | 23:55 |
zer0c00l | lucas-hungry: sure. I didn't know UEFI + MBR would work. But thanks for clarifying | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!