| devananda | NobodyCam: that's the one. mrda - if you want to clobber sjing's patch, that's it ^ | 00:02 |
|---|---|---|
| *** max_lobur has quit IRC | 00:02 | |
| *** max_lobur has joined #openstack-ironic | 00:02 | |
| devananda | one of yuriyz' patches is almost ready, too. https://review.openstack.org/#/c/72395/ | 00:02 |
| devananda | just needs a bit more work | 00:02 |
| mrda | ok, shall look | 00:03 |
| mrda | thncx | 00:03 |
| NobodyCam | have an hour is omg thats either a really slow network or really BIG image | 00:06 |
| NobodyCam | or both | 00:06 |
| NobodyCam | s/have/half/ | 00:06 |
| devananda | mrda: looks ilke https://review.openstack.org/#/c/76614/ conflicts with yours and needs to be rebased | 00:09 |
| devananda | mrda: i doubt lucas would mind :) | 00:09 |
| devananda | NobodyCam: call it conservative, lol | 00:09 |
| devananda | NobodyCam: also to note: PXE driver will POST back right after first boot. iLO driver will post back after image is compied. | 00:10 |
| NobodyCam | ok | 00:11 |
| mrda | devananda: and that's the patch you asked me to write yesterday, which lucas beat me to overnight :) | 00:11 |
| devananda | haha | 00:11 |
| *** max_lobur has quit IRC | 00:11 | |
| NobodyCam | question about line 469 of https://review.openstack.org/#/c/72395/4/ironic/conductor/manager.py | 00:11 |
| NobodyCam | if this was a takeover situation ie. orginal master node doing the deploy dies and is taken over would we not hit the node is locked? | 00:13 |
| NobodyCam | s/master node/master conductor/ | 00:13 |
| NobodyCam | the node is in deploywait so it should be locked | 00:14 |
| devananda | mrda: another one with small fixups that can be done in isolation and then easily landed -- https://review.openstack.org/#/c/72495/5 | 00:21 |
| devananda | NobodyCam: no -- deploywait state is set then the node is unlocked | 00:21 |
| mrda | devananda: ta | 00:21 |
| devananda | NobodyCam: http://git.openstack.org/cgit/openstack/ironic/tree/ironic/drivers/modules/pxe.py#n550 | 00:22 |
| devananda | NobodyCam: http://git.openstack.org/cgit/openstack/ironic/tree/ironic/conductor/manager.py#n313 | 00:23 |
| devananda | oooh | 00:23 |
| devananda | did max get a patch up to refactor do_node_deploy to use a worker thread? | 00:23 |
| *** matsuhashi has joined #openstack-ironic | 00:23 | |
| devananda | i dont see one | 00:24 |
| devananda | mrda: there's a big new patch you could work on :) | 00:24 |
| devananda | mrda: take a look at how change_node_power_state uses a worker thread -- http://git.openstack.org/cgit/openstack/ironic/tree/ironic/conductor/manager.py#n216 | 00:24 |
| devananda | mrda: but do_node_deploy does not -- http://git.openstack.org/cgit/openstack/ironic/tree/ironic/conductor/manager.py#n271 | 00:25 |
| devananda | well, it should :) | 00:25 |
| devananda | same for do_node_teardown | 00:25 |
| devananda | mrda: off hand, I think both those methods should: take a task_manager lock; validate the request; save the state to DB to advertise that work is starting; start a worker thread to do the actual work | 00:27 |
| devananda | this is similar to the work max is doing in https://review.openstack.org/#/c/74063/2/ironic/conductor/manager.py | 00:27 |
| devananda | but not overlapping | 00:27 |
| openstackgerrit | Michael Davies proposed a change to openstack/ironic: Allow client to set the maintenance mode https://review.openstack.org/76614 | 00:28 |
| NobodyCam | ahh ok Thank you deva | 00:30 |
| * NobodyCam thought for some reason the node was locked | 00:30 | |
| *** lnxnut has joined #openstack-ironic | 00:32 | |
| devananda | NobodyCam: it probably used to be :) | 00:33 |
| NobodyCam | lol | 00:40 |
| NobodyCam | :) | 00:40 |
| NobodyCam | brb | 00:40 |
| mrda | devananda: Just regarding Ghe's "Move ipminative _set_boot_device to VendorPassthru" (https://review.openstack.org/#/c/72495) | 00:44 |
| mrda | Did you want the suggestions done as an amendment to that patch | 00:45 |
| mrda | or as a dependent new patch? | 00:45 |
| devananda | mrda: a follow-on is best unless GheRivero doesn't mind you clobbering his patch :) | 00:48 |
| devananda | especially since those are minor issues, follow-on seems simpilest | 00:49 |
| mrda | np, I'll make it a new patch. Stay tuned :) Then I'll look at your other suggestion aka "the bigger new patch" | 00:49 |
| devananda | mrda: i'm heading out in a little bit -- if you want to look at / talk about my othe rsuggestion, now's your chance | 00:50 |
| devananda | there's always a chance i'll be back online late(r) tonight, too | 00:51 |
| * mrda reads scrollback | 00:52 | |
| devananda | Haomeng: hi! i am taking a quick look at https://review.openstack.org/#/c/72538/9/ironic/conductor/manager.py and have a question | 00:53 |
| devananda | Haomeng: it seems like ceilometer integration depends on the nova driver? | 00:54 |
| mrda | devananda: I think I understand what you're asking regarding worker thread & locks. I think I'll be fine to get a patch up for this | 00:56 |
| devananda | mrda: fantastic :) | 00:56 |
| mrda | :w | 00:57 |
| NobodyCam | back | 00:58 |
| *** harlowja_away is now known as harlowja | 01:05 | |
| openstackgerrit | Michael Davies proposed a change to openstack/ironic: Suggested improvements to _set_boot_device https://review.openstack.org/76721 | 01:10 |
| openstackgerrit | Michael Davies proposed a change to openstack/ironic: Move ipminative _set_boot_device to VendorPassthru https://review.openstack.org/72495 | 01:10 |
| *** lnxnut has quit IRC | 01:27 | |
| *** lnxnut has joined #openstack-ironic | 01:27 | |
| devananda | review etherpad updated -- https://etherpad.openstack.org/p/IronicReviewDay | 01:31 |
| *** lnxnut has quit IRC | 01:31 | |
| devananda | and with that, i'm going to head out for a bit and get some much-needed air :) | 01:32 |
| NobodyCam | enjoy devananda | 01:33 |
| NobodyCam | Think I'll make a rum and diet coke! | 01:34 |
| Haomeng | devananda: yes ceilometer depends on nova client to collect nova compute meters, but our ironic just send impi data message to notification bus and ceilometer will listen and handle the message | 01:37 |
| Haomeng | devananda: you mean our ironic nova driver? | 01:40 |
| Haomeng | devananda: yes, some fields like user_id and tenant_id are required for ceilometer, so I think it can be populated by our ironic nova driver, that is same as the existing nova instance_uuid | 01:41 |
| devananda | Haomeng: ok. so two things | 01:42 |
| devananda | Haomeng: firstly, the nova "ironic" driver is blocked and very unlikely to land in icehouse. not that that stops us from landing this code, but it should be considered | 01:42 |
| Haomeng | devananda: so far our patch can work to send ipmi data out to ceilometer, ceilometer guys will help to implement the handler to consum our impi data message into ceilometer resource/meter/samples | 01:42 |
| Haomeng | devananda: yes, i see | 01:42 |
| devananda | Haomeng: secondly, right now, we're only stashing the instance_uuid. so this patch also depends on getting the tenant_id and user_id somehow -- it could either ask nova for them, since we have the instance_uuid, or we need to have the nova "ironic" driver stash them in Ironic when associating it -- that's a small change to that driver, but I think a reasonable one | 01:43 |
| Haomeng | devananda: so for such user_id and tenant_id fields, how do you think if we have othere way to get these nova instance data | 01:43 |
| Haomeng | devananda: yes | 01:43 |
| Haomeng | devananda: i will check with nova guys to see if we can get tenant_id and user_id from nova instance properties | 01:44 |
| Haomeng | devananda: and these are not confirmed by ceilometer guys, the orig ceilometer ipmi inspector owner is working with me for this ipmi data collection now, he is trying to design ceilometer part | 01:45 |
| Haomeng | devananda: so I will co-work with nova and ceilometer guys to complete this bp | 01:45 |
| Haomeng | devananda: thanks for your valuable comments, I will commit new patch:) | 01:45 |
| *** nosnos has joined #openstack-ironic | 01:46 | |
| Haomeng | devananda: :) | 01:46 |
| devananda | Haomeng: great work, btw | 01:49 |
| devananda | Haomeng: please keep at it. We probably won't land it in the next few days since it's dependent on several other features, but I suspect it'll be good to land as soon as Juno development opens | 01:50 |
| Haomeng | devananda: sure, that is fine with me:) | 01:52 |
| Haomeng | devananda: I will working on this bp for long time:) | 01:53 |
| Haomeng | devananda: another question with a defect confirmation | 01:54 |
| Haomeng | devananda: this one - https://bugs.launchpad.net/ironic/+bug/1284781 | 01:54 |
| Haomeng | devananda: I understand we should remove the definition from line L45 and the validation mandatory_attrs logic L128-L134 https://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/port.py#L45 L128-L134 | 01:56 |
| Haomeng | devananda: wsme has such "readonly" and "mandatory" functionality, so our code logic can be removed | 01:58 |
| devananda | Haomeng: i think mrda is working on that? | 02:04 |
| Haomeng | devananda: you mean mrda is working on https://bugs.launchpad.net/ironic/+bug/1284781 ? I will check with him, but no any comments from him in the bug | 02:06 |
| Haomeng | mrda:ping | 02:06 |
| *** jcooley_ has quit IRC | 02:37 | |
| *** derekh is now known as derekh_afk | 02:57 | |
| *** rloo has quit IRC | 03:01 | |
| *** dkehn_ has quit IRC | 03:13 | |
| *** sjing has joined #openstack-ironic | 03:28 | |
| sjing | Deva, are you there? | 03:29 |
| *** coolsvap has quit IRC | 03:42 | |
| *** sjing has quit IRC | 03:55 | |
| *** jcooley_ has joined #openstack-ironic | 04:17 | |
| *** lnxnut has joined #openstack-ironic | 04:29 | |
| *** jcooley_ has quit IRC | 04:46 | |
| *** jcooley_ has joined #openstack-ironic | 04:52 | |
| mrda | sorry Haomeng - been AFK | 04:59 |
| *** jcooley_ has quit IRC | 04:59 | |
| *** matsuhashi has quit IRC | 05:00 | |
| *** lnxnut has quit IRC | 05:02 | |
| *** coolsvap has joined #openstack-ironic | 05:04 | |
| mrda | Haomeng: I | 05:06 |
| mrda | 'm not working on https://bugs.launchpad.net/ironic/+bug/1284781 | 05:06 |
| mrda | but I am looking at https://bugs.launchpad.net/ironic/+bug/1271317 (removing XML support from i-api) which is a little related | 05:08 |
| Haomeng | mrda: :) | 05:16 |
| Haomeng | mrda: no worries:) | 05:16 |
| *** coolsvap has quit IRC | 05:16 | |
| Haomeng | mrda: deva mentioned you:) | 05:16 |
| *** coolsvap has joined #openstack-ironic | 05:17 | |
| Haomeng | mrda: I think he is busy, so ...:) | 05:17 |
| Haomeng | mrda: and I am interesting with your defect 1271317 (removing XML support from i-api) also:) | 05:18 |
| Haomeng | mrda: I investigated it but no solution, do you have any idea now? | 05:19 |
| openstackgerrit | Michael Davies proposed a change to openstack/ironic: Prevent GET /v1/nodes returning maintenance field https://review.openstack.org/76094 | 05:22 |
| mrda | Haomeng: I'm going to try a temporary solution of hacking content_types, and then try and work out the correct way of doing it | 05:24 |
| Haomeng | mrda: cool | 05:24 |
| mrda | but it may mean modifying the wsme or pecan | 05:24 |
| Haomeng | mrda: I tried to find out some wsme context_type confirguations, but no findings:) | 05:24 |
| mrda | I think other openstack projects may want this too, if they move to pecan/wsme and don't want XML | 05:24 |
| Haomeng | Haomeng: I do the difference with other procect | 05:25 |
| Haomeng | mrda: such Heat, they are different with us | 05:25 |
| *** rameshg87 has joined #openstack-ironic | 05:25 | |
| Haomeng | mrda: in heat, the context-type for api supportting are configurationable | 05:25 |
| Haomeng | mrda: let me show you the code, hope can help you | 05:25 |
| mrda | that would be good if you can | 05:29 |
| Haomeng | mrda: https://github.com/openstack/heat/blob/master/heat/common/wsgi.py#L714 | 05:29 |
| Haomeng | mrda: this is the heat code | 05:29 |
| Haomeng | mrda: for your reference:) | 05:30 |
| Haomeng | mrda: heat is using XMLResponseSerializer, but our Ironic did not use this for xml | 05:30 |
| mrda | thanks Haomeng | 05:30 |
| Haomeng | mrda: not sure if we can port | 05:30 |
| Haomeng | mrda: if we can not find out the wsme configurations to disable the xml content-type supporting | 05:31 |
| Haomeng | mrda: welcome:) | 05:31 |
| mrda | I might also email openstack-dev about this as there are some pecan/wsme authors there, to see if they have any ideas | 05:32 |
| *** yongli has joined #openstack-ironic | 05:33 | |
| Haomeng | mrda: great idea | 05:36 |
| *** vkozhukalov has joined #openstack-ironic | 05:36 | |
| *** killer_prince has quit IRC | 05:54 | |
| *** KanagarajM_ has joined #openstack-ironic | 05:57 | |
| *** killer_prince has joined #openstack-ironic | 06:03 | |
| openstackgerrit | Jenkins proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/71192 | 06:07 |
| *** lazy_prince has joined #openstack-ironic | 06:11 | |
| openstackgerrit | lokesh s proposed a change to openstack/ironic: Ironic deployment docs https://review.openstack.org/74282 | 06:11 |
| *** loki_ has joined #openstack-ironic | 06:12 | |
| *** pradipta` is now known as pradipta | 06:14 | |
| *** vkozhukalov has quit IRC | 06:18 | |
| *** saju_m has joined #openstack-ironic | 06:48 | |
| * devananda is back and semi-around | 06:52 | |
| *** harlowja is now known as harlowja_away | 07:23 | |
| *** rameshg87 has quit IRC | 07:23 | |
| *** rameshg87 has joined #openstack-ironic | 07:24 | |
| *** sirushti is now known as shortstop | 07:32 | |
| *** pbrooko has joined #openstack-ironic | 07:48 | |
| *** pbrooko has quit IRC | 07:49 | |
| *** vkozhukalov has joined #openstack-ironic | 07:53 | |
| *** jcooley_ has joined #openstack-ironic | 07:56 | |
| *** pbrooko has joined #openstack-ironic | 08:01 | |
| *** saju_m has quit IRC | 08:10 | |
| *** ndipanov has joined #openstack-ironic | 08:10 | |
| *** saju_m has joined #openstack-ironic | 08:10 | |
| *** aignatov_ is now known as aignatov | 08:17 | |
| *** saju_m has quit IRC | 08:25 | |
| *** saju_m has joined #openstack-ironic | 08:26 | |
| *** romcheg1 has joined #openstack-ironic | 08:29 | |
| *** jistr has joined #openstack-ironic | 08:30 | |
| *** romcheg has quit IRC | 08:31 | |
| *** jcooley_ has quit IRC | 08:39 | |
| *** jrist has quit IRC | 08:41 | |
| *** aignatov is now known as aignatov_ | 08:48 | |
| *** pbrooko has quit IRC | 08:50 | |
| *** saju_m has quit IRC | 08:52 | |
| *** yuriyz has joined #openstack-ironic | 08:54 | |
| *** jrist has joined #openstack-ironic | 08:55 | |
| *** aignatov_ is now known as aignatov | 08:55 | |
| *** aignatov is now known as aignatov_ | 08:56 | |
| *** aignatov_ is now known as aignatov | 08:57 | |
| *** ifarkas has joined #openstack-ironic | 09:01 | |
| *** martyntaylor has joined #openstack-ironic | 09:04 | |
| *** martyntaylor has quit IRC | 09:09 | |
| *** martyntaylor has joined #openstack-ironic | 09:10 | |
| openstackgerrit | Yongli He proposed a change to openstack/python-ironicclient: Exception message should not be localize https://review.openstack.org/76803 | 09:11 |
| *** dshulyak has joined #openstack-ironic | 09:13 | |
| *** mrda is now known as mrda_away | 09:33 | |
| *** derekh_afk is now known as derekh | 09:34 | |
| *** mdurnosvistov_lt has joined #openstack-ironic | 09:36 | |
| *** mdurnosvistov_lt has quit IRC | 09:36 | |
| *** mdurnosvistov_lt has joined #openstack-ironic | 09:36 | |
| *** mdurnosvistov_lt has quit IRC | 09:36 | |
| *** mdurnosvistov_lt has joined #openstack-ironic | 09:37 | |
| *** lucasagomes has joined #openstack-ironic | 09:38 | |
| *** max_lobur has joined #openstack-ironic | 09:52 | |
| *** tatyana has joined #openstack-ironic | 09:58 | |
| *** bashok has joined #openstack-ironic | 10:00 | |
| *** romcheg1 has quit IRC | 10:01 | |
| *** athomas has quit IRC | 10:09 | |
| *** Hefeweizen has quit IRC | 10:13 | |
| *** aignatov is now known as aignatov_ | 10:17 | |
| *** romcheg has joined #openstack-ironic | 10:19 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: API: Expose a way to start/stop the console https://review.openstack.org/72998 | 10:26 |
| *** athomas has joined #openstack-ironic | 10:33 | |
| *** saju_m has joined #openstack-ironic | 10:35 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Make the PXE driver understand ephemeral disks https://review.openstack.org/65750 | 10:41 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Support preserve_ephemeral https://review.openstack.org/68236 | 10:46 |
| *** jcooley_ has joined #openstack-ironic | 11:01 | |
| *** aignatov_ is now known as aignatov | 11:03 | |
| *** aignatov is now known as aignatov_ | 11:05 | |
| *** aignatov_ is now known as aignatov | 11:07 | |
| *** jcooley_ has quit IRC | 11:11 | |
| *** nosnos has quit IRC | 11:14 | |
| *** KanagarajM_ has quit IRC | 11:17 | |
| openstackgerrit | ChenZheng proposed a change to openstack/python-ironicclient: Sort requirement files in alphabetical order https://review.openstack.org/76827 | 11:36 |
| *** loki_ has quit IRC | 11:39 | |
| *** rameshg87 has left #openstack-ironic | 11:52 | |
| *** jcooley_ has joined #openstack-ironic | 11:53 | |
| *** jcooley_ has quit IRC | 11:59 | |
| *** tatyana has quit IRC | 12:23 | |
| *** tatyana_ has joined #openstack-ironic | 12:23 | |
| *** bashok has quit IRC | 12:36 | |
| *** saju_m has quit IRC | 12:53 | |
| *** linggao has joined #openstack-ironic | 13:02 | |
| *** jdob has joined #openstack-ironic | 13:25 | |
| *** lucasagomes is now known as lucas-hungry | 13:28 | |
| *** rloo has joined #openstack-ironic | 13:37 | |
| *** jcooley_ has joined #openstack-ironic | 13:40 | |
| *** rloo has quit IRC | 13:41 | |
| *** rloo has joined #openstack-ironic | 13:42 | |
| *** rloo has quit IRC | 13:44 | |
| *** rloo has joined #openstack-ironic | 13:44 | |
| *** jcooley_ has quit IRC | 13:46 | |
| *** aignatov is now known as aignatov_ | 13:58 | |
| *** aignatov_ is now known as aignatov | 14:06 | |
| devananda | g'mornng, all | 14:10 |
| romcheg | Morning devananda | 14:10 |
| agordeev | devananda, romcheg: good morning! | 14:11 |
| rloo | wow, morning devananda. Hi everyone else ;) | 14:15 |
| * devananda starts hitting the reviews early | 14:19 | |
| agordeev | rloo: hi | 14:23 |
| devananda | romcheg: want to give https://review.openstack.org/#/c/76293/ a review? :) | 14:23 |
| openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Add timeout for waiting callback from deploy ramdisk https://review.openstack.org/72395 | 14:26 |
| *** lucas-hungry is now known as lucasagomes | 14:30 | |
| romcheg | *looks* | 14:33 |
| *** jbjohnso_ has joined #openstack-ironic | 14:35 | |
| *** jcooley_ has joined #openstack-ironic | 14:35 | |
| yuriyz | morning Ironic | 14:35 |
| lucasagomes | morning all | 14:37 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Make the PXE driver understand ephemeral disks https://review.openstack.org/65750 | 14:37 |
| agordeev | yuriyz, lucasagomes: morning! | 14:39 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Support preserve_ephemeral https://review.openstack.org/68236 | 14:41 |
| *** jcooley_ has quit IRC | 14:41 | |
| devananda | yuriyz: posting comments on 72395 -- almost there! | 14:44 |
| devananda | yuriyz: are you around now and able to fix a few small issues with the patch? | 14:50 |
| yuriyz | devananda, ok | 14:50 |
| yuriyz | but I dont see any posted comments yet | 14:51 |
| devananda | just hit Save :) | 14:53 |
| * devananda is multitasking | 14:53 | |
| *** matty_dubs|gone is now known as matty_dubs | 15:00 | |
| *** morgabra has quit IRC | 15:01 | |
| *** morgabra has joined #openstack-ironic | 15:03 | |
| NobodyCam | good morning devananda :) | 15:08 |
| NobodyCam | and Ironic :) | 15:11 |
| * NobodyCam makes coffee | 15:12 | |
| *** ndipanov has quit IRC | 15:12 | |
| agordeev | NobodyCam: morning | 15:12 |
| *** lnxnut has joined #openstack-ironic | 15:14 | |
| *** mdurnosvistov has joined #openstack-ironic | 15:14 | |
| *** blamar_ has joined #openstack-ironic | 15:16 | |
| *** SpamapS_ has joined #openstack-ironic | 15:16 | |
| NobodyCam | mornng agordeev :) | 15:17 |
| NobodyCam | devananda: you redo the review day pad? | 15:19 |
| devananda | yep | 15:19 |
| devananda | just a lot of cleanup | 15:19 |
| NobodyCam | cool :) | 15:20 |
| *** martyntaylor has quit IRC | 15:20 | |
| *** athomas has quit IRC | 15:21 | |
| *** yuriyz has quit IRC | 15:21 | |
| *** mdurnosvistov_ has quit IRC | 15:21 | |
| *** blamar has quit IRC | 15:21 | |
| *** SpamapS has quit IRC | 15:21 | |
| *** blamar_ is now known as blamar | 15:21 | |
| *** aignatov is now known as aignatov_ | 15:22 | |
| *** aignatov_ is now known as aignatov | 15:24 | |
| *** yuriyz has joined #openstack-ironic | 15:28 | |
| *** athomas has joined #openstack-ironic | 15:28 | |
| NobodyCam | lucasagomes: I'm not sure what up with the volume driver | 15:31 |
| lucasagomes | NobodyCam, what's up? | 15:31 |
| NobodyCam | stuff like: http://logs.openstack.org/26/71026/14/check/gate-nova-python27/2c6ff28/console.html#_2014-02-26_19_46_38_063 | 15:36 |
| lucasagomes | hmm | 15:38 |
| lucasagomes | yeah i'll have to take a look, cause off the top of my head idk | 15:38 |
| NobodyCam | yea, oh side question ..are comments like : NOTE(lucasagomes): When creating a node via API (POST) we have to use chassis_uuid? we changed that didn't we? | 15:39 |
| NobodyCam | I think I know why its failing. but need to look in to it. | 15:40 |
| *** killer_prince has quit IRC | 15:41 | |
| *** lazy_prince is now known as killer_prince | 15:41 | |
| lucasagomes | NobodyCam, we did | 15:41 |
| lucasagomes | NobodyCam, before we used chassis_id, but then we agreed that we are not exposing numeral id's on the API | 15:41 |
| lucasagomes | so we now use chassis_uuid | 15:42 |
| lucasagomes | NobodyCam, but that comment is because, on the tests when ur creating a node using the dblayer instead of POST via API | 15:42 |
| lucasagomes | we have to use chassis_id | 15:42 |
| *** martyntaylor has joined #openstack-ironic | 15:42 | |
| lucasagomes | cause chassis_uuid just exist in the API | 15:42 |
| NobodyCam | ahh | 15:47 |
| NobodyCam | :) | 15:47 |
| mdurnosvistov | Morning all! :) | 15:47 |
| NobodyCam | lucasagomes: I think this is whay tests are failing : http://paste.openstack.org/show/SWSwefTpay8Oe9ATiWs4/ | 15:47 |
| lucasagomes | NobodyCam, yeah, that part is a bit confusing :( | 15:47 |
| lucasagomes | mdurnosvistov, morning | 15:47 |
| NobodyCam | but I'm not sure | 15:47 |
| NobodyCam | morning mdurnosvistov :) | 15:47 |
| devananda | mrda_away: have answers for you re: XML | 15:48 |
| lucasagomes | NobodyCam, hmmm | 15:50 |
| lucasagomes | yeah that missing parameter might cause some problems indeed | 15:50 |
| lucasagomes | devananda, bout hw to disable it? | 15:52 |
| lucasagomes | about* | 15:52 |
| devananda | yep | 15:53 |
| devananda | lucasagomes: check pecan.request.content_type and reject with 415 Unsupported | 15:54 |
| devananda | lucasagomes: we could do that globally, or we could decorate all the patch() methods with it | 15:54 |
| lucasagomes | devananda, ah! awesome | 15:55 |
| lucasagomes | maybe we should make a hook for it? | 15:55 |
| lucasagomes | pecan hook* (in case we want it globally disabled) | 15:56 |
| * NobodyCam gets fresh cup of coffee | 16:00 | |
| devananda | ooh! coffee! I should go make some :) | 16:01 |
| * lucasagomes is sad because he's only have tea at home :( (coffee <3) | 16:02 | |
| NobodyCam | no coffee? | 16:03 |
| NobodyCam | omg... I'd die | 16:04 |
| NobodyCam | or at least wouldn't wake up | 16:04 |
| lucasagomes | heh yeah | 16:04 |
| lucasagomes | I gotta buy a coffee maker | 16:04 |
| NobodyCam | :-p | 16:04 |
| romcheg | lucasagomes: buy gaggia :) | 16:04 |
| NobodyCam | lol, /me know what to get lucasagomes | 16:04 |
| lucasagomes | hahah | 16:04 |
| NobodyCam | morning romcheg :) | 16:04 |
| rloo | tea drinkers unite! Tea is BETTER than coffee :-) | 16:05 |
| lucasagomes | romcheg, gaggia? | 16:05 |
| lucasagomes | lemme check | 16:05 |
| romcheg | Morning folks | 16:05 |
| lucasagomes | I like filtered coffee | 16:05 |
| NobodyCam | morning rloo :) something about a party in boston | 16:05 |
| romcheg | Gaggia is one of the best vendors of espresso machines | 16:05 |
| rloo | NobodyCam, and I'm not even an American. | 16:05 |
| NobodyCam | lol :-p | 16:05 |
| rloo | romcheg: the Italians apparently think you're not a real man or something if you don't have one of those manual gaggias. | 16:06 |
| lucasagomes | romcheg, o wow! ahaha looks awesome | 16:06 |
| romcheg | lucasagomes: Now I have Saeco Poemia but seriously considering to change it to Gaggia Classic | 16:07 |
| romcheg | Sorry for advertising :D | 16:08 |
| rloo | romcheg: apparently vibiemme is really good. | 16:09 |
| lucasagomes | romcheg, hehe it's grand, it's good to have references :P | 16:09 |
| *** coolsvap has quit IRC | 16:10 | |
| romcheg | rloo: you've just said that tea is better than coffee, so how can we trust this recommendation? :-P | 16:10 |
| *** aignatov is now known as aignatov_ | 16:11 | |
| lucasagomes | rloo, I drink tea, maybe more than coffee but I still prefer coffee | 16:11 |
| rloo | romcheg. good question ;) I don't like coffee but my spouse does, and he's moved up from gaggias, etc, to a vibiemme. | 16:11 |
| rloo | romcheg: he throws out coffee beans if they are X days old. I think he's a snob, but he loves his espresso. Illy beans are no longer good enough. | 16:12 |
| romcheg | rloo: Actually I tried coffe made on vibiemme and it was really good. But I think there's no official distributor here in Ukraine :( | 16:14 |
| rloo | romcheg: stick with the gaggia. it is good enough. and less $$ :-) | 16:15 |
| romcheg | So it's hard to find them and they are extremely expensive | 16:15 |
| rloo | romcheg. Exactly! He wouldn't tell me how $$$... At least he isn't into expensive cars ;) | 16:17 |
| *** max_lobur has quit IRC | 16:18 | |
| NobodyCam | *core call in 5?? | 16:20 |
| NobodyCam | max you here | 16:28 |
| *** jcooley_ has joined #openstack-ironic | 16:29 | |
| NobodyCam | romcheg: you get invite | 16:29 |
| romcheg | NobodyCam: Not yet | 16:30 |
| romcheg | romcheg.prihod@gmail.com ? | 16:30 |
| openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Add timeout for waiting callback from deploy ramdisk https://review.openstack.org/72395 | 16:35 |
| *** vkozhukalov has quit IRC | 16:36 | |
| *** jcooley_ has quit IRC | 16:36 | |
| *** jcooley_ has joined #openstack-ironic | 16:38 | |
| *** vkozhukalov has joined #openstack-ironic | 16:39 | |
| *** aignatov_ is now known as aignatov | 16:43 | |
| openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Add timeout for waiting callback from deploy ramdisk https://review.openstack.org/72395 | 16:44 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Make the PXE driver understand ephemeral disks https://review.openstack.org/65750 | 16:47 |
| *** jcooley_ has quit IRC | 16:59 | |
| *** jistr has quit IRC | 17:00 | |
| *** killer_prince is now known as lazy_prince | 17:01 | |
| *** digambar has joined #openstack-ironic | 17:01 | |
| *** vkozhukalov has quit IRC | 17:05 | |
| *** yuriyz has quit IRC | 17:05 | |
| *** digambar has quit IRC | 17:09 | |
| *** martyntaylor has quit IRC | 17:13 | |
| devananda | rloo: some of your comments on 76293 -- are you expecting hardware to be in states besides ON | OFF ? | 17:13 |
| rloo | devananda: I wasn't actually thinking of what the possible values might be. | 17:14 |
| rloo | devananda. Mostly to handle/help deal with any bugs or whatever that might creep into the code. | 17:14 |
| *** jcooley_ has joined #openstack-ironic | 17:17 | |
| *** dhellmann is now known as dhellmann_ | 17:17 | |
| *** viktors is now known as viktors_away | 17:22 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Removing get_node_power_state, bumping RPC version https://review.openstack.org/73097 | 17:23 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Removing get_node_power_state, bumping RPC version https://review.openstack.org/73097 | 17:24 |
| *** digambar has joined #openstack-ironic | 17:25 | |
| *** matty_dubs is now known as matty_dubs|lunch | 17:26 | |
| jbjohnso_ | devananda, fwiw, the four possible states that I have returned are 'on', 'off', 'suspended', and 'unknown' | 17:26 |
| jbjohnso_ | devananda, in other software stack. | 17:26 |
| *** tatyana_ has quit IRC | 17:27 | |
| *** dshulyak has quit IRC | 17:27 | |
| devananda | romcheg: http://logs.openstack.org/95/72395/7/check/check-tempest-dsvm-ironic/4ccfda2/logs/devstacklog.txt.gz#_2014-02-27_17_12_23_692 | 17:31 |
| *** digambar has quit IRC | 17:33 | |
| *** aignatov is now known as aignatov_ | 17:35 | |
| NobodyCam | https://bugs.launchpad.net/ironic/+bug/1285793 | 17:37 |
| *** dhellmann_ is now known as dhellmann | 17:39 | |
| *** rameshg87 has joined #openstack-ironic | 17:43 | |
| *** jcooley_ has quit IRC | 17:45 | |
| *** dtalton has joined #openstack-ironic | 17:45 | |
| *** derekh has quit IRC | 17:46 | |
| *** dtalton has left #openstack-ironic | 17:46 | |
| *** SpamapS_ is now known as SpamapS | 17:48 | |
| *** SpamapS has quit IRC | 17:48 | |
| *** SpamapS has joined #openstack-ironic | 17:48 | |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Add option to sync node power state from DB https://review.openstack.org/76293 | 17:50 |
| *** aignatov_ is now known as aignatov | 17:57 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: API: Expose a way to start/stop the console https://review.openstack.org/72998 | 18:03 |
| rameshg87 | good morning devananda: | 18:05 |
| *** harlowja_away is now known as harlowja | 18:12 | |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Add option to sync node power state from DB https://review.openstack.org/76293 | 18:18 |
| *** jcooley_ has joined #openstack-ironic | 18:18 | |
| *** athomas has quit IRC | 18:25 | |
| NobodyCam | devananda: romcheg https://review.openstack.org/#/c/76094/ | 18:30 |
| *** matty_dubs|lunch is now known as matty_dubs | 18:34 | |
| *** romcheg has quit IRC | 18:35 | |
| *** saju_m has joined #openstack-ironic | 18:37 | |
| *** athomas has joined #openstack-ironic | 18:39 | |
| *** saju_m has quit IRC | 18:41 | |
| *** jgrimm has joined #openstack-ironic | 18:41 | |
| *** dshulyak has joined #openstack-ironic | 18:43 | |
| *** max_lobur has joined #openstack-ironic | 18:49 | |
| *** saju_m has joined #openstack-ironic | 18:51 | |
| shortstop | Hi devananda, have you got a chance to look at 73054? :) | 18:53 |
| devananda | shortstop: trying to fix the gate tests right now | 18:53 |
| shortstop | devananda, Ah! No problem :) | 18:55 |
| *** vkozhukalov has joined #openstack-ironic | 18:55 | |
| rameshg87 | devananda: just wanted to have a talk a few things about ilo driver. is it okay to talk now ? | 18:55 |
| *** dtalton has joined #openstack-ironic | 18:56 | |
| shortstop | devananda, will nag for review later then :-p | 18:57 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: API: Expose a way to start/stop the console https://review.openstack.org/72998 | 18:57 |
| *** dtalton has quit IRC | 19:00 | |
| *** rameshg87 has left #openstack-ironic | 19:17 | |
| NobodyCam | brb | 19:20 |
| *** aignatov is now known as aignatov_ | 19:23 | |
| *** saju_m has quit IRC | 19:35 | |
| *** ChanServ sets mode: +o devananda | 19:36 | |
| *** devananda changes topic to "Jenkins check & gate is broken right now, pending this fix: https://review.openstack.org/#/c/76943" | 19:37 | |
| * devananda takes a break | 19:38 | |
| rloo | thx devananda! | 19:39 |
| NobodyCam | ok late bubbie breakfast served :-p | 19:40 |
| matty_dubs | I'm not sure what I did, but testr is suddenly hemorrhaging alembic errors to the effect of my script directory having multiple heads (due to branching). | 19:41 |
| matty_dubs | Anyone have any idea how to resolve this? | 19:42 |
| matty_dubs | I didn't (intentionally, at least) do anything to Ironic, so I'm not sure how to fix it. | 19:42 |
| NobodyCam | matty_dubs: did you try recreating the test env? not sure it help but... | 19:45 |
| NobodyCam | I have not run into that error | 19:45 |
| matty_dubs | NobodyCam: Probably can't hurt! I'll try that. | 19:45 |
| matty_dubs | NobodyCam: You might be onto something. Rebuilding that fails because libffi is not found. Maybe a new dep came in. | 19:48 |
| NobodyCam | ahh :) | 19:48 |
| matty_dubs | Coming from Ruby, I feel right at home with my development environment randomly needing to be reinstalled every now and then due to bizarre errors. ;) | 19:49 |
| NobodyCam | matty_dubs: LOL ++ | 19:49 |
| * NobodyCam has worked at java shops where production system would just fail for no reason (other then they where running java in production) | 19:50 | |
| matty_dubs | Ha, I used to be one of 2 admins on some Solr servers. They'd periodically run out of heap space (we gave it 12GB!) and just fall over. | 19:52 |
| matty_dubs | And then take so long to restart that Nagios would try to 'restart' them before they came back up. | 19:53 |
| matty_dubs | Oh the nightmares. | 19:53 |
| NobodyCam | matty_dubs: :) yeppers ... activemq cluster for me but same issues!!! :-p | 19:53 |
| NobodyCam | devananda: your devstack patch is about to pass jenkins checks, just fyi: the gate checks look to backed up over 5 1/2 hours ... happy happy joy joy | 19:57 |
| matty_dubs | Hrm, this is an alembic issue. 'cd ironic/db/sqlalchemy; alembic branches' shows some sort of branching. | 19:57 |
| devananda | NobodyCam: yep, i know | 19:58 |
| NobodyCam | :( | 19:58 |
| devananda | NobodyCam: sdague just got back and +2/+A'd it ahead of jenkins. so it'll land as soon as possible ... | 19:58 |
| devananda | but our gate will probably still be down all day | 19:58 |
| NobodyCam | :( | 19:59 |
| NobodyCam | brb | 20:00 |
| *** dshulyak has quit IRC | 20:09 | |
| mrda_away | morning all | 20:29 |
| *** mrda_away is now known as mrda | 20:29 | |
| mrda | devananda: thanks, I saw the email from dhellman | 20:29 |
| NobodyCam | devananda: if you have a minute: https://review.openstack.org/#/c/72998/ | 20:29 |
| mrda | devananda: Looks like I need to file a bp for removing XML support from wsme | 20:30 |
| *** epim has joined #openstack-ironic | 20:31 | |
| NobodyCam | oh and mornig mrda :) | 20:32 |
| mrda | morning NobodyCam! | 20:34 |
| lifeless | lucasagomes: hi | 20:43 |
| lifeless | lucasagomes: don't use nova bm as the reference point | 20:43 |
| lifeless | lucasagomes: look at novas flavor definitions instead | 20:44 |
| *** hstimer has joined #openstack-ironic | 20:48 | |
| *** ifarkas has quit IRC | 20:56 | |
| devananda | lifeless: flavor def is stored in the DB with _gb | 20:57 |
| devananda | lifeless: https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/models.py#L210 | 20:58 |
| *** rloo has quit IRC | 21:04 | |
| *** rloo has joined #openstack-ironic | 21:05 | |
| lifeless | devananda: I know :) | 21:22 |
| devananda | lifeless: i thought you were suggesting we store things in mb? | 21:22 |
| lifeless | no | 21:25 |
| lifeless | I'masking lucas to not perpetuate that mistake | 21:25 |
| devananda | ahh | 21:25 |
| devananda | lifeless: as an aside, swap is still in mb in nova :( | 21:29 |
| devananda | the db column isn't suffixed, but the raw value is MB | 21:29 |
| *** athomas has quit IRC | 21:32 | |
| lifeless | argh, well I think rounding it up to GB would be fine for Ironic. what do you think? | 21:35 |
| devananda | ++ | 21:39 |
| *** hemna has quit IRC | 21:50 | |
| NobodyCam | brb | 21:51 |
| *** lnxnut has quit IRC | 21:51 | |
| *** hemna has joined #openstack-ironic | 21:52 | |
| *** lnxnut has joined #openstack-ironic | 21:53 | |
| *** lnxnut has quit IRC | 21:54 | |
| *** lnxnut has joined #openstack-ironic | 21:54 | |
| *** lnxnut_ has joined #openstack-ironic | 21:58 | |
| *** lnxnut has quit IRC | 21:58 | |
| *** linggao has quit IRC | 22:06 | |
| *** jdob has quit IRC | 22:07 | |
| *** jbjohnso_ has quit IRC | 22:10 | |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Add option to sync node power state from DB https://review.openstack.org/76293 | 22:14 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Allow client to set the maintenance mode https://review.openstack.org/76614 | 22:19 |
| openstackgerrit | Rohan Kanade proposed a change to openstack/ironic: Partially remove hard-coded ID's from testdata https://review.openstack.org/76990 | 22:20 |
| *** lnxnut_ has quit IRC | 22:24 | |
| *** lnxnut has joined #openstack-ironic | 22:29 | |
| *** lnxnut has quit IRC | 22:33 | |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Add option to sync node power state from DB https://review.openstack.org/76293 | 22:39 |
| devananda | NobodyCam: there ^ :) | 22:39 |
| devananda | rloo: that also hopefully addresses your comments as well | 22:41 |
| rloo | devananda: I'll take a look. | 22:41 |
| NobodyCam | lol | 22:42 |
| rloo | (even if Jenkins doesn't like it) | 22:42 |
| * NobodyCam clicks | 22:42 | |
| NobodyCam | Thank you devananda :) LGTM , will wait for mr J | 22:42 |
| devananda | NobodyCam: jenins will -1 it, i guarantee | 22:43 |
| * devananda points to room title | 22:43 | |
| NobodyCam | ya but I'll ignore the tempest results | 22:43 |
| devananda | ah :) | 22:43 |
| *** lnxnut has joined #openstack-ironic | 22:44 | |
| *** vkozhukalov has quit IRC | 22:47 | |
| *** lnxnut has quit IRC | 22:48 | |
| mrda | so the overnight tempest failures in ironic have a common source? (as per devananda's email?) | 22:49 |
| devananda | yep | 22:49 |
| devananda | mrda: see channel topic | 22:49 |
| *** rloo has quit IRC | 22:50 | |
| *** rloo has joined #openstack-ironic | 22:50 | |
| mrda | devananda: thanks, my client doesn't display that automagically | 22:50 |
| devananda | queue is 7hr long, it's been in the queue for 2.5. | 22:50 |
| devananda | assuming its fixed tomorrow morning PST, I'l go through and re+A a bunch of things | 22:51 |
| *** max_lobur has quit IRC | 22:51 | |
| devananda | ya'll dont need to not-review or not-code because of it :) | 22:51 |
| rloo | I wasn't looking at the reviews cuz of jenkins. Hmm... | 22:54 |
| NobodyCam | devananda: wanta take a look over https://review.openstack.org/#/c/74282/ (deployer docs) | 22:59 |
| NobodyCam | see if I missed anything large? | 22:59 |
| devananda | k | 22:59 |
| *** matty_dubs is now known as matty_dubs|gone | 23:09 | |
| *** mrda is now known as pseudomikal | 23:15 | |
| *** pseudomikal is now known as mrda | 23:15 | |
| NobodyCam | devananda: htttp://paste.openstack.org/show/SWSwefTpay8Oe9ATiWs4/ | 23:23 |
| NobodyCam | http://paste.openstack.org/show/SWSwefTpay8Oe9ATiWs4/ | 23:24 |
| *** SpamapS_ has joined #openstack-ironic | 23:43 | |
| *** SpamapS has quit IRC | 23:45 | |
| *** openstackgerrit has quit IRC | 23:45 | |
| *** romcheg has joined #openstack-ironic | 23:49 | |
| *** SpamapS_ is now known as SpamapS | 23:53 | |
| *** SpamapS has quit IRC | 23:53 | |
| *** SpamapS has joined #openstack-ironic | 23:53 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!