dkehn | NobodyCam: thats were the conflict was be fixed it, I hand removed the issue and all is good , hmm don't like that | 00:00 |
---|---|---|
dkehn | NobodyCam: ok something is horked, now the py27 is blowing, oslo.rootwrap errors | 00:01 |
NobodyCam | when was the last update. several patches landed yesterday | 00:02 |
dkehn | rebase 10 mins ago | 00:02 |
NobodyCam | https://review.openstack.org/#/c/66400/ | 00:02 |
dkehn | hmm, eerythiong was working before the rebase, of course | 00:02 |
NobodyCam | wrong link | 00:03 |
NobodyCam | https://review.openstack.org/#/c/66389/ | 00:03 |
NobodyCam | that is the rootwrap patch that landed | 00:03 |
dkehn | NobodyCam: so the whole ironic/openstack/common/rootwrap dir is gone? | 00:05 |
dkehn | NobodyCam: looks like i've got that update | 00:06 |
dkehn | NobodyCam: have time for a call | 00:06 |
NobodyCam | yes | 00:07 |
NobodyCam | ok | 00:07 |
NobodyCam | sure | 00:07 |
*** jbjohnso has joined #openstack-ironic | 00:08 | |
*** ifarkas has quit IRC | 00:12 | |
dkehn | NobodyCam: yepper, that was it, thx | 00:33 |
NobodyCam | :) | 00:34 |
*** matsuhashi has joined #openstack-ironic | 00:34 | |
*** hemna is now known as hemnafk | 00:39 | |
openstackgerrit | A change was merged to openstack/ironic: PXE instance_name is no longer mandatory https://review.openstack.org/67974 | 00:57 |
* devananda packs up for the evening | 01:00 | |
devananda | g'night everyone! | 01:01 |
NobodyCam | night devananda | 01:01 |
NobodyCam | :) | 01:01 |
*** mdurnosvistov has quit IRC | 01:02 | |
*** nosnos has joined #openstack-ironic | 01:34 | |
*** epim has quit IRC | 01:36 | |
*** jbjohnso has quit IRC | 01:40 | |
Haomeng | devananda: night:) | 01:42 |
*** datajerk has joined #openstack-ironic | 01:49 | |
*** rongze has joined #openstack-ironic | 02:13 | |
*** coolsvap has quit IRC | 02:21 | |
*** vkozhukalov has joined #openstack-ironic | 02:23 | |
*** coolsvap has joined #openstack-ironic | 02:24 | |
openstackgerrit | Lee Li proposed a change to openstack/python-ironicclient: Remove unused method 'string_to_bool' from utils https://review.openstack.org/68543 | 02:33 |
*** coolsvap has quit IRC | 02:37 | |
*** jbjohnso has joined #openstack-ironic | 02:38 | |
openstackgerrit | Ruby Loo proposed a change to openstack/ironic: ipmitool's _power_status raises IPMIFailure https://review.openstack.org/66894 | 02:49 |
openstackgerrit | dekehn proposed a change to openstack/ironic: Adds Neutron support to Ironic https://review.openstack.org/66071 | 02:54 |
*** jbjohnso has quit IRC | 02:57 | |
*** rloo has quit IRC | 03:06 | |
*** jbjohnso has joined #openstack-ironic | 03:30 | |
*** jbjohnso has quit IRC | 03:48 | |
*** jbjohnso has joined #openstack-ironic | 03:49 | |
*** jbjohnso has quit IRC | 04:00 | |
*** harlowja is now known as harlowja_away | 04:23 | |
*** aignatov_ is now known as aignatov | 04:29 | |
*** rongze has quit IRC | 04:47 | |
*** rongze has joined #openstack-ironic | 04:48 | |
*** rongze has quit IRC | 04:53 | |
*** aignatov is now known as aignatov_ | 04:56 | |
*** datajerk has quit IRC | 04:59 | |
*** jbjohnso has joined #openstack-ironic | 05:01 | |
*** jbjohnso has quit IRC | 05:09 | |
*** harlowja_away is now known as harlowja | 05:10 | |
*** rongze has joined #openstack-ironic | 05:22 | |
*** rongze has quit IRC | 05:27 | |
*** coolsvap has joined #openstack-ironic | 05:41 | |
*** rongze has joined #openstack-ironic | 05:54 | |
*** rongze has quit IRC | 06:01 | |
openstackgerrit | Jenkins proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/68024 | 06:04 |
*** rongze has joined #openstack-ironic | 06:35 | |
*** rongze_ has joined #openstack-ironic | 06:39 | |
*** rongze has quit IRC | 06:40 | |
*** harlowja is now known as harlowja_away | 06:45 | |
*** matsuhashi has quit IRC | 06:50 | |
*** matsuhashi has joined #openstack-ironic | 06:54 | |
*** marty_ has joined #openstack-ironic | 06:55 | |
*** marty_ is now known as Guest28628 | 06:56 | |
*** hstimer has quit IRC | 06:59 | |
*** Guest28628 has quit IRC | 07:00 | |
*** mrda is now known as mrda_away | 07:16 | |
*** romcheg has joined #openstack-ironic | 07:25 | |
*** vetalll has joined #openstack-ironic | 07:48 | |
*** ifarkas has joined #openstack-ironic | 07:54 | |
*** mdurnosvistov has joined #openstack-ironic | 08:06 | |
*** yuriyz has joined #openstack-ironic | 08:29 | |
*** jistr has joined #openstack-ironic | 08:32 | |
GheRivero | morning Ironic | 08:53 |
*** aignatov_ is now known as aignatov | 08:54 | |
*** aignatov is now known as aignatov_ | 09:00 | |
*** rongze_ has quit IRC | 09:06 | |
*** vkozhukalov has quit IRC | 09:06 | |
*** john-n-seattle has quit IRC | 09:06 | |
*** Alexei_987 has joined #openstack-ironic | 09:07 | |
*** rongze_ has joined #openstack-ironic | 09:11 | |
*** vkozhukalov has joined #openstack-ironic | 09:11 | |
*** john-n-seattle has joined #openstack-ironic | 09:11 | |
*** rongze_ has quit IRC | 09:13 | |
*** vkozhukalov has quit IRC | 09:13 | |
*** john-n-seattle has quit IRC | 09:13 | |
*** vkozhukalov has joined #openstack-ironic | 09:14 | |
*** rongze has joined #openstack-ironic | 09:14 | |
*** derekh has joined #openstack-ironic | 09:14 | |
*** Haomeng|2 has joined #openstack-ironic | 09:24 | |
*** Haomeng has quit IRC | 09:24 | |
openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Add parameter for filtering nodes by maintenance mode https://review.openstack.org/63937 | 09:24 |
*** athomas has joined #openstack-ironic | 09:27 | |
*** dshulyak has joined #openstack-ironic | 09:28 | |
*** aignatov_ is now known as aignatov | 09:30 | |
openstackgerrit | Dmitry Shulyak proposed a change to openstack/ironic: Fix duplicate uuid error fo pxe tests https://review.openstack.org/67130 | 09:33 |
*** aignatov is now known as aignatov_ | 09:42 | |
*** coolsvap has quit IRC | 09:43 | |
*** coolsvap has joined #openstack-ironic | 09:45 | |
*** aignatov_ is now known as aignatov | 09:46 | |
*** john-n-seattle has joined #openstack-ironic | 09:48 | |
*** vetalll has quit IRC | 10:07 | |
*** max_lobur_afk is now known as max_lobur | 10:07 | |
*** athomas has quit IRC | 10:12 | |
*** athomas has joined #openstack-ironic | 10:12 | |
*** ColinTaylor has quit IRC | 10:15 | |
ifarkas | lsmola_, I think I have found the reason why ironic driver in nova is failing for us: https://review.openstack.org/#/c/51328/9/nova/virt/ironic/volume_driver.py | 10:19 |
*** martyntaylor has joined #openstack-ironic | 10:20 | |
lsmola_ | ifarkas: hmm for me it is failing here http://paste.openstack.org/show/61705/ | 10:22 |
ifarkas | lsmola_, your nova-compute is up? | 10:26 |
lsmola_ | ifarkas: when I restart it, yes | 10:27 |
lsmola_ | ifarkas: but it is killed after this exception | 10:27 |
lsmola_ | ifarkas: it happens in heat stack-create of overcloud | 10:27 |
lsmola_ | ifarkas: before that it runs fine in undercloud | 10:28 |
ifarkas | lsmola_, you have debug=true and verbose=true in nova.conf? | 10:28 |
lsmola_ | ifarkas: no :-) | 10:28 |
lsmola_ | ifarkas: will do | 10:28 |
*** rongze has quit IRC | 10:45 | |
*** rongze has joined #openstack-ironic | 10:45 | |
*** rongze has quit IRC | 10:50 | |
*** lucasagomes has joined #openstack-ironic | 10:57 | |
*** matsuhashi has quit IRC | 11:05 | |
*** ndipanov_ has joined #openstack-ironic | 11:09 | |
*** matsuhashi has joined #openstack-ironic | 11:14 | |
*** rongze has joined #openstack-ironic | 11:16 | |
*** rongze has quit IRC | 11:20 | |
*** ndipanov has quit IRC | 11:24 | |
*** vetalll has joined #openstack-ironic | 11:33 | |
*** michchap has quit IRC | 11:40 | |
openstackgerrit | Dmitry Shulyak proposed a change to openstack/ironic: Fix duplicate uuid error fo pxe tests https://review.openstack.org/67130 | 11:54 |
openstackgerrit | Dmitry Shulyak proposed a change to openstack/ironic: alembic with initial migration and tests https://review.openstack.org/67415 | 11:54 |
*** rongze has joined #openstack-ironic | 11:57 | |
*** Haomeng|3 has joined #openstack-ironic | 11:58 | |
*** Haomeng|2 has quit IRC | 11:59 | |
*** Haomeng has joined #openstack-ironic | 12:02 | |
*** Haomeng|3 has quit IRC | 12:06 | |
*** nosnos has quit IRC | 12:36 | |
*** ndipanov_ has quit IRC | 12:36 | |
*** ndipanov has joined #openstack-ironic | 12:37 | |
*** matsuhashi has quit IRC | 12:39 | |
*** anniec has joined #openstack-ironic | 12:48 | |
*** anniec_ has joined #openstack-ironic | 12:51 | |
*** anniec has quit IRC | 12:53 | |
*** anniec_ is now known as anniec | 12:53 | |
*** coolsvap has quit IRC | 12:58 | |
*** jdob has joined #openstack-ironic | 13:00 | |
*** linggao has joined #openstack-ironic | 13:16 | |
*** aignatov is now known as aignatov_ | 13:18 | |
*** aignatov_ is now known as aignatov | 13:20 | |
*** aignatov is now known as aignatov_ | 13:29 | |
max_lobur | morning Ironic! | 13:30 |
max_lobur | morning lucasagomes | 13:30 |
max_lobur | wanted to askw | 13:30 |
max_lobur | we're not exposing node reservation property via REST | 13:30 |
max_lobur | is that intentional right? | 13:30 |
max_lobur | https://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/node.py#L225-L274 | 13:30 |
yuriyz | morning max_lobur | 13:32 |
max_lobur | morning yuriyz | 13:32 |
yuriyz | there is my patch on review with this https://review.openstack.org/#/c/55549/20/ironic/api/controllers/v1/node.py | 13:32 |
lucasagomes | max_lobur, morning :D | 13:33 |
lucasagomes | yup as yuriyz, pointed out, his patch to break the lock is adding it to the node's response request | 13:33 |
lucasagomes | actually, that's a very good information to have on nodes, maybe even fits in a patch for it's own change | 13:34 |
max_lobur | ah I see | 13:34 |
max_lobur | so we were waiting for readonly support in wsme | 13:35 |
*** aignatov_ is now known as aignatov | 13:35 | |
max_lobur | which yuriyz has implemented :) | 13:35 |
max_lobur | k, thanks :) | 13:37 |
yuriyz | max_lobur, please review my patch if you have some time, code in node controller rewritten | 13:39 |
max_lobur | yuriyz, yep, I will! | 13:40 |
lucasagomes | yea, :D readonly would be a great feature | 13:41 |
max_lobur | once I finish with race | 13:41 |
openstackgerrit | Dmitry Shulyak proposed a change to openstack/ironic: alembic with initial migration and tests https://review.openstack.org/67415 | 13:48 |
*** vetalll has left #openstack-ironic | 14:00 | |
*** rloo has joined #openstack-ironic | 14:02 | |
*** rloo has quit IRC | 14:03 | |
*** yfujioka has joined #openstack-ironic | 14:03 | |
*** rloo has joined #openstack-ironic | 14:03 | |
*** rloo has quit IRC | 14:04 | |
*** rloo has joined #openstack-ironic | 14:04 | |
*** rloo has quit IRC | 14:04 | |
*** rloo has joined #openstack-ironic | 14:05 | |
*** rloo has quit IRC | 14:06 | |
*** rloo has joined #openstack-ironic | 14:06 | |
*** aignatov is now known as aignatov_ | 14:08 | |
*** rloo has quit IRC | 14:09 | |
*** rloo has joined #openstack-ironic | 14:10 | |
*** rloo has quit IRC | 14:10 | |
*** rloo has joined #openstack-ironic | 14:10 | |
openstackgerrit | Dmitry Shulyak proposed a change to openstack/ironic: alembic with initial migration and tests https://review.openstack.org/67415 | 14:11 |
*** aignatov_ is now known as aignatov | 14:12 | |
*** rloo has quit IRC | 14:13 | |
*** rloo has joined #openstack-ironic | 14:14 | |
*** rloo has quit IRC | 14:14 | |
*** rloo has joined #openstack-ironic | 14:14 | |
*** rloo has quit IRC | 14:17 | |
GheRivero | morning | 14:17 |
lucasagomes | morning GheRivero | 14:18 |
*** rloo has joined #openstack-ironic | 14:18 | |
*** coolsvap has joined #openstack-ironic | 14:18 | |
*** rloo_ has joined #openstack-ironic | 14:34 | |
*** matty_dubs|gone is now known as matty_dubs | 14:36 | |
*** datajerk has joined #openstack-ironic | 14:42 | |
rloo | hi lucasagomes | 14:43 |
*** datajerk1 has joined #openstack-ironic | 14:43 | |
lucasagomes | rloo, morning :) | 14:44 |
rloo | lucasagomes, wanted to ask you about https://review.openstack.org/66894 | 14:45 |
*** max_lobur has quit IRC | 14:45 | |
rloo | are you serious, you want me to separate the docstrings into a diff review? | 14:45 |
lucasagomes | rloo, right, yea I think you maybe substantial changes on the docstrings, so that would fit another review | 14:45 |
lucasagomes | rloo, hmm | 14:45 |
lucasagomes | I actually was heh | 14:45 |
rloo | lucasagomes: there's only one real code change there. I threw in the docstrings "for free" | 14:46 |
*** max_lobur has joined #openstack-ironic | 14:46 | |
lucasagomes | rloo, I know that why | 14:46 |
rloo | lucasagomes: i thought I saw deva do a similar thing in some other review but maybe i was wrong. | 14:46 |
lucasagomes | rloo, docstrings are real code | 14:46 |
lucasagomes | they are not merely "comments" on the functions in python | 14:46 |
rloo | lucasagomes. ha ha, then they should have already been there ;) | 14:46 |
*** datajerk has quit IRC | 14:47 | |
rloo | lucasagomes. ok, if you're serious. just seems like extra work for everyone involved (reviewers, etc). | 14:47 |
lucasagomes | rloo, right, yea I thought about it as well. I usually don't mind when it's just like improving one docstring related to the method you're going to modify on that patch set | 14:48 |
lucasagomes | but on that case it's adding docstring to all the other methods that weren't touched by that change | 14:49 |
lucasagomes | so that's why I think that we should split it in two patches | 14:49 |
lucasagomes | it's a nice practice to have very focused commits | 14:49 |
rloo | lucasagomes - should i open a bug against it or is it ok to just submit a review w/o a bug? | 14:49 |
lucasagomes | rloo, but rloo, if you don't think it should be done. Like there's no harm... that's my opnion only | 14:49 |
lucasagomes | the rest of the guys can vote and give their opnion about the change as well | 14:50 |
rloo | lucasagomes. i know it's your opinion and i respect it. just want to see if i can change your mind ;) | 14:50 |
lucasagomes | rloo, I would just submit another review | 14:50 |
lucasagomes | rloo, heh | 14:50 |
rloo | ok thx lucasagomes. | 14:50 |
lucasagomes | rloo, but just one thing about docstrings, they are real code in python | 14:51 |
*** rongze has quit IRC | 14:51 | |
lucasagomes | the docstrings are not comments on the functions, they are understood as the __doc__ attribute of an object | 14:51 |
lucasagomes | so it's def real code | 14:52 |
rloo | lucasagomes: yes, that's true. but they're "just strings" :-) | 14:52 |
lucasagomes | rloo, and thanks for adding the docstrings to that module, they much needed wanted :) | 14:53 |
lucasagomes | rloo, heh well yea, that's correct heh | 14:53 |
NobodyCam | morning says the man slowly making coffee | 14:58 |
rloo | wakey wakey NobodyCam | 14:59 |
NobodyCam | :) | 14:59 |
*** jbjohnso has joined #openstack-ironic | 14:59 | |
lucasagomes | NobodyCam, morning :D | 15:02 |
openstackgerrit | Ruby Loo proposed a change to openstack/ironic: ipmitool's _power_status raises IPMIFailure https://review.openstack.org/66894 | 15:02 |
NobodyCam | morning lucasagomes L) | 15:05 |
NobodyCam | lsmola_: got things working? | 15:05 |
lsmola_ | NobodyCam: nope | 15:07 |
NobodyCam | :( | 15:07 |
lsmola_ | NobodyCam: I have the ironic nodes working | 15:07 |
lsmola_ | NobodyCam: but i couldn;t go pass the not authorized | 15:07 |
NobodyCam | can you paste the error | 15:08 |
* lsmola_ looking | 15:09 | |
lsmola_ | NobodyCam: http://paste.openstack.org/show/61705/ | 15:11 |
* NobodyCam looks. | 15:12 | |
lsmola_ | NobodyCam: and this is not being set for some reason https://github.com/NoBodyCam/ironic-element/blob/master/nova/os-apply-config/etc/nova/nova.conf#L146 | 15:13 |
NobodyCam | give me just a minute here | 15:14 |
ifarkas | NobodyCam, morning! | 15:15 |
ifarkas | NobodyCam, I also managed to debug my issue with the ironic driver. Could you please take a look at the comments at https://review.openstack.org/#/c/51328/9/nova/virt/ironic/volume_driver.py ? | 15:16 |
NobodyCam | morning ifarkas | 15:16 |
NobodyCam | ifarkas: yes | 15:17 |
NobodyCam | one sec | 15:18 |
openstackgerrit | dekehn proposed a change to openstack/ironic: Adds Neutron support to Ironic https://review.openstack.org/66071 | 15:18 |
ifarkas | NobodyCam, ahh, okay. I wasn't sure if I am not missing something more fundamental :-) | 15:18 |
ifarkas | NobodyCam, I built the undercloud images following the commit msg of https://review.openstack.org/#/c/66461/ As a workaround, I will just create my image without DIB_REPOLOCATION_nova and patch nova after Heat deployed it | 15:21 |
NobodyCam | ifarkas: that vol driver was quite old I think | 15:21 |
ifarkas | NobodyCam, do you mean that the patch should work? | 15:24 |
NobodyCam | no I mean I need to push up a new version | 15:26 |
NobodyCam | [remote rejected] HEAD -> refs/publish/master/bp/deprecate-baremetal-driver (change 51328 closed) | 15:26 |
NobodyCam | the vol driver in the review is still using bmdb | 15:27 |
NobodyCam | lol need to get deva to click the restore button | 15:27 |
openstackgerrit | Ruby Loo proposed a change to openstack/ironic: Docstrings for ipmitool https://review.openstack.org/68665 | 15:28 |
ifarkas | NobodyCam, hehe | 15:28 |
ifarkas | NobodyCam, and thanks! I am going to rebuild my images then | 15:29 |
NobodyCam | ifarkas: do you have a test env setup | 15:29 |
ifarkas | NobodyCam, yes | 15:29 |
NobodyCam | can you test with this: http://paste.openstack.org/show/mI4IA7NaMkaB7Ke2LwOi/ | 15:29 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Support preserve_ephemeral https://review.openstack.org/68236 | 15:30 |
ifarkas | NobodyCam, sure, I will do it | 15:32 |
GheRivero | dshulyak: ping | 15:33 |
*** yuriyz has quit IRC | 15:33 | |
NobodyCam | morning GheRivero :) | 15:34 |
dshulyak | GheRivero: pong | 15:34 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Support preserve_ephemeral https://review.openstack.org/68236 | 15:34 |
GheRivero | morning NobodyCam | 15:35 |
*** datajerk1 has quit IRC | 15:35 | |
*** lucasagomes is now known as lucas-hungry | 15:36 | |
*** datajerk has joined #openstack-ironic | 15:36 | |
NobodyCam | :) | 15:36 |
GheRivero | dshulyak: about the Duplicate uuid patch, if is it fixed by the one that Yuriy comments, what's the reason for it now? | 15:36 |
dshulyak | GheRivero: i will abandon that patch. i've already rebased patch with alembic on new master | 15:37 |
GheRivero | cool :) | 15:38 |
*** yuriyz has joined #openstack-ironic | 15:40 | |
NobodyCam | ahh I'm not getting a correct $SEED_IP | 15:40 |
*** jdob has quit IRC | 15:41 | |
NobodyCam | actually I'm not getting a SEED_IP at all | 15:41 |
ifarkas | NobodyCam, SEED_IP=192.0.2.1 doesn't help? | 15:45 |
ifarkas | I suppose you followed devtest | 15:45 |
NobodyCam | how did you guess | 15:45 |
NobodyCam | The IP address of the VM is printed out at the end of boot-seed-vm, or you can query the testenv json which is updated by boot-seed-vm: | 15:46 |
NobodyCam | it didn't update the file | 15:46 |
NobodyCam | {"host-ip":"192.168.122.1", "seed-ip":"", "node-macs":"52:54:00:09:f8:7c "} | 15:46 |
NobodyCam | ifarkas: devtest is broken then? | 15:47 |
ifarkas | NobodyCam, not sure if it's broken, it updated for me | 15:47 |
NobodyCam | ack... /me tryies agiain | 15:48 |
NobodyCam | tries even | 15:48 |
ifarkas | NobodyCam, check: dib-seed.log | 15:48 |
ifarkas | the ip should be at the end | 15:48 |
NobodyCam | ya.. it didn't | 15:50 |
NobodyCam | oh wait | 15:50 |
NobodyCam | fatal: unable to access 'https://git.openstack.org/openstack/nova.git/': Could not resolve host: git.openstack.org | 15:50 |
NobodyCam | there we go | 15:50 |
ifarkas | NobodyCam, did you exported DIB_REPOLOCATION_nova and DIB_REPOREF_nova? | 15:51 |
NobodyCam | if prob not LOL... :( | 15:52 |
ifarkas | NobodyCam, afaik, there's an issue with the workflow describe in the commit message. I think I had the same error message and changing DIB_REPOREF_nova=refs/changes/28/51328 to DIB_REPOREF_nova=refs/changes/28/51328/9 fixed it | 15:53 |
*** datajerk has quit IRC | 15:54 | |
*** jdob has joined #openstack-ironic | 15:55 | |
*** jcooley_ has joined #openstack-ironic | 15:57 | |
NobodyCam | brb | 15:57 |
*** datajerk has joined #openstack-ironic | 16:00 | |
openstackgerrit | Dmitry Shulyak proposed a change to openstack/ironic: alembic with initial migration and tests https://review.openstack.org/67415 | 16:06 |
* ifarkas back in an hour or so | 16:06 | |
*** ifarkas has quit IRC | 16:07 | |
NobodyCam | :) | 16:08 |
*** datajerk has quit IRC | 16:10 | |
devananda | morning, all | 16:11 |
NobodyCam | morning devananda :) | 16:12 |
NobodyCam | brb | 16:12 |
*** anniec has quit IRC | 16:12 | |
*** aignatov is now known as aignatov_ | 16:13 | |
openstackgerrit | Ruby Loo proposed a change to openstack/ironic: SSHPower driver raises IronicExceptions https://review.openstack.org/66990 | 16:14 |
*** datajerk has joined #openstack-ironic | 16:21 | |
yuriyz | morning devananda | 16:24 |
yuriyz | devananda, what you think about deploy timeouts managetment by periodic task (for https://bugs.launchpad.net/ironic/+bug/1270986) ? | 16:26 |
*** lucas-hungry is now known as lucasagomes | 16:29 | |
max_lobur | ping lucasagomes | 16:30 |
lucasagomes | max_lobur, pong | 16:30 |
max_lobur | https://review.openstack.org/#/c/67506/1/ironic/api/controllers/v1/node.py I've been thinking about this | 16:30 |
devananda | yuriyz: i think there's two problems with it :) | 16:30 |
max_lobur | so the current strategy is to let client receive 200 | 16:30 |
max_lobur | and then let it polling the api to know the new state | 16:31 |
max_lobur | right& | 16:31 |
devananda | yuriyz: how does periodic_task interrupt a separate thread which is running deploy_utils? | 16:31 |
max_lobur | I meant 202, not 200 :) | 16:31 |
lucasagomes | max_lobur, yup | 16:31 |
max_lobur | ok, cool | 16:31 |
max_lobur | because I just hit a bug in old implementation | 16:32 |
devananda | yuriyz: and what happens if the hashring changes and periodic_task fires on a different conductor? | 16:32 |
lucasagomes | the client can use the /states to track the current state of his request | 16:32 |
devananda | while first conductor still has a lock | 16:32 |
devananda | yuriyz: so it is somewhat related to the work to break locks that you did -- but goes a step further. instead of "break a lock held by a dead conductor" timeout needs to tell the process which is alive and healthy, holding a lock, to stop, cleanup, and release it | 16:33 |
max_lobur | lucasagomes, the rpc_node has not been updated before sending to the client, so target power state was always wrong | 16:33 |
*** coolsvap is now known as coolsvap_away | 16:33 | |
max_lobur | ok, glad we removed it | 16:33 |
lucasagomes | yea, we can't rely on that rpc_node, it can change between the time we return it | 16:34 |
max_lobur | yes | 16:34 |
max_lobur | ok, cool :D | 16:34 |
lucasagomes | :) | 16:34 |
*** hemna has joined #openstack-ironic | 16:35 | |
*** coolsvap_away is now known as coolsvap | 16:35 | |
*** datajerk has quit IRC | 16:36 | |
*** coolsvap is now known as coolsvap_away | 16:37 | |
openstackgerrit | A change was merged to openstack/ironic: ipmitool's _power_status raises IPMIFailure https://review.openstack.org/66894 | 16:39 |
*** rloo has quit IRC | 16:39 | |
*** rloo has joined #openstack-ironic | 16:40 | |
*** rloo has quit IRC | 16:40 | |
*** rloo has joined #openstack-ironic | 16:40 | |
*** rloo has quit IRC | 16:45 | |
*** rloo has joined #openstack-ironic | 16:46 | |
*** datajerk has joined #openstack-ironic | 16:48 | |
*** matty_dubs is now known as matty_dubs|lunch | 16:55 | |
openstackgerrit | Dmitry Shulyak proposed a change to openstack/ironic: alembic with initial migration and tests https://review.openstack.org/67415 | 16:55 |
devananda | max_lobur: hi! | 16:57 |
max_lobur | devananda, morning | 16:57 |
*** yfujioka has quit IRC | 16:58 | |
devananda | max_lobur: see pm | 16:58 |
*** coolsvap_away is now known as coolsvap | 16:59 | |
*** martyntaylor has left #openstack-ironic | 17:04 | |
NobodyCam | GheRivero: lucasagomes did you test eview.openstack.org/#/c/68243? | 17:06 |
*** ifarkas has joined #openstack-ironic | 17:06 | |
lucasagomes | NobodyCam, I pulled it and ran the test | 17:08 |
lucasagomes | tests* | 17:08 |
openstackgerrit | A change was merged to openstack/ironic: Add comment about node.instance_uuid https://review.openstack.org/68220 | 17:08 |
NobodyCam | :) | 17:08 |
lucasagomes | also checked if it was really part of the global requirements | 17:09 |
NobodyCam | :) | 17:09 |
openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Check BMC availability in ipmitool 'validate' method https://review.openstack.org/68697 | 17:10 |
openstackgerrit | A change was merged to openstack/ironic: Updated from global requirements https://review.openstack.org/68243 | 17:13 |
NobodyCam | gate is so fast now | 17:13 |
*** lsmola_ has quit IRC | 17:13 | |
lucasagomes | NobodyCam, yup | 17:15 |
lucasagomes | the gate is actually pretty cool | 17:15 |
lucasagomes | was pointed by markmc to this blog post today:https://dague.net/2013/02/21/the-openstack-gate/ | 17:16 |
lucasagomes | good reading | 17:16 |
NobodyCam | :) | 17:16 |
NobodyCam | got seed booted now getting Unable to establish connection to http://192.0.2.2:35357/v2.0/tenants | 17:18 |
NobodyCam | humm | 17:18 |
NobodyCam | anyone seen this error before?: http://paste.openstack.org/show/HSLPA47PhSgiz5jkrpqv/ | 17:21 |
*** vetalll has joined #openstack-ironic | 17:23 | |
devananda | max_lobur: ping | 17:23 |
lucasagomes | NobodyCam, which version of the six is installed? | 17:24 |
vetalll | Hi everyone | 17:26 |
lucasagomes | NobodyCam, I remember that the six add_metaclass was added not long time ago, lemme check the six changelog | 17:26 |
*** yuriyz has quit IRC | 17:26 | |
lucasagomes | NobodyCam, maybe we need a minimum version in the global requirements | 17:26 |
vetalll | Could anyone explain is it really a bug ? https://bugs.launchpad.net/ironic/+bug/1259346 | 17:26 |
NobodyCam | lucasagomes: Downloading six-1.5.2.tar.gz | 17:28 |
vetalll | Ruby Loo is not convinced tha it is a bug | 17:28 |
*** jistr has quit IRC | 17:28 | |
devananda | vetalll: see the email to openstack-dev | 17:29 |
max_lobur | devananda, pong, I'm here | 17:29 |
devananda | vetalll: it's somewhat enmeshed in other aspects of the project and i'm trying to tease it apart | 17:29 |
devananda | max_lobur: are you really? :) | 17:30 |
NobodyCam | vetalll: title of that email is Node groups and multi-node operations | 17:30 |
devananda | max_lobur: see private message pls | 17:30 |
max_lobur | oh, sorry :) | 17:30 |
lucasagomes | http://lists.openstack.org/pipermail/openstack-dev/2014-January/025170.html | 17:30 |
* max_lobur looking | 17:30 | |
lucasagomes | vetalll, ^ | 17:31 |
vetalll | thx for link | 17:33 |
devananda | lucasagomes: going to push another release of the client today. any critical patches that should land first? | 17:37 |
rloo | vetalll: there was a bit of a discussion on IRC yesterday about that bug/your changes. | 17:38 |
lucasagomes | devananda, no critical patches | 17:39 |
lucasagomes | I will approve 68543 | 17:39 |
lucasagomes | but that's only removing an unused method | 17:39 |
lucasagomes | so you can go ahead with the release | 17:39 |
NobodyCam | bbt...brb | 17:40 |
devananda | lucasagomes: ack, thanks | 17:40 |
*** max_lobur is now known as max_lobur_afk | 17:45 | |
*** datajerk has quit IRC | 17:47 | |
openstackgerrit | A change was merged to openstack/python-ironicclient: Remove unused method 'string_to_bool' from utils https://review.openstack.org/68543 | 17:47 |
*** vetalll has left #openstack-ironic | 17:47 | |
*** Alexei_987 has quit IRC | 17:48 | |
*** vetalll has joined #openstack-ironic | 17:48 | |
NobodyCam | got to something up with my heat template | 17:49 |
NobodyCam | post bbt walkies... bbiafm | 17:50 |
*** matty_dubs|lunch is now known as matty_dubs | 17:51 | |
*** ndipanov is now known as ndipanov_gone | 17:53 | |
*** vetalll has quit IRC | 17:58 | |
*** derekh has quit IRC | 18:00 | |
*** datajerk has joined #openstack-ironic | 18:01 | |
*** aignatov_ is now known as aignatov | 18:02 | |
NobodyCam | back | 18:04 |
*** datajerk has quit IRC | 18:06 | |
*** datajerk has joined #openstack-ironic | 18:10 | |
*** harlowja_away is now known as harlowja | 18:10 | |
ifarkas | NobodyCam, I wasn't really successful with using: http://paste.openstack.org/show/mI4IA7NaMkaB7Ke2LwOi/ | 18:12 |
ifarkas | here's the error: http://paste.openstack.org/show/61776/ | 18:12 |
NobodyCam | ifarkas: ya I having several issues my self | 18:14 |
NobodyCam | ifarkas: your using dib correct? | 18:14 |
ifarkas | NobodyCam, yep | 18:15 |
NobodyCam | can you paste your stackrc from the /root/ of the undercloud | 18:15 |
devananda | lucasagomes: I've cloned the preserve-ephemeral BP: https://blueprints.launchpad.net/ironic/+spec/preserve-ephemeral | 18:15 |
devananda | lucasagomes: i see you've assigne dthe bug to yourself. shall i assign the bp to you? | 18:16 |
lucasagomes | devananda, oh, nice | 18:16 |
lucasagomes | sure yea | 18:16 |
lucasagomes | I will add the tags to the commits I already have in gerrit | 18:16 |
lucasagomes | the bp tag I mean | 18:16 |
ifarkas | NobodyCam, sure: http://paste.openstack.org/show/61777/ | 18:16 |
NobodyCam | yep export OS_PASSWORD= | 18:17 |
*** rloo has quit IRC | 18:17 | |
lucasagomes | devananda, thanks :) | 18:17 |
NobodyCam | somethings wrong | 18:17 |
NobodyCam | I'm looking at it ATM | 18:17 |
*** rloo has joined #openstack-ironic | 18:17 | |
ifarkas | NobodyCam, I usually use openstack-tools on the host machine and sshing to undercloud only for debugging | 18:18 |
ifarkas | you have the same issue with your stackrc? | 18:18 |
NobodyCam | yes | 18:18 |
*** datajerk has quit IRC | 18:19 | |
*** datajerk has joined #openstack-ironic | 18:21 | |
*** coolsvap has quit IRC | 18:23 | |
*** coolsvap has joined #openstack-ironic | 18:24 | |
NobodyCam | ifarkas: also my nova.conf is missing lots of things | 18:26 |
devananda | oh hey guys, look! http://lists.openstack.org/pipermail/openstack-dev/2014-January/025267.html | 18:27 |
devananda | were official and stuff :) | 18:27 |
ifarkas | NobodyCam, for example? | 18:27 |
*** datajerk has quit IRC | 18:27 | |
NobodyCam | compute_driver = | 18:28 |
*** datajerk has joined #openstack-ironic | 18:28 | |
ifarkas | devananda, \o/ | 18:28 |
ifarkas | NobodyCam, that one is correct for me: compute_driver = ironic.driver.IronicDriver | 18:29 |
lucasagomes | good stuff in i2 :) | 18:30 |
*** jcooley_ has quit IRC | 18:31 | |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Make the PXE driver understand ephemeral disks https://review.openstack.org/65750 | 18:35 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Support preserve_ephemeral https://review.openstack.org/68236 | 18:36 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Support preserve_ephemeral https://review.openstack.org/68236 | 18:37 |
*** hstimer has joined #openstack-ironic | 18:44 | |
*** jcooley_ has joined #openstack-ironic | 18:45 | |
*** rloo has quit IRC | 18:48 | |
*** marty_ has joined #openstack-ironic | 18:48 | |
*** rloo has joined #openstack-ironic | 18:48 | |
*** marty_ is now known as Guest62853 | 18:48 | |
*** hstimer has quit IRC | 18:49 | |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Removed duplicated function to create a swap fs https://review.openstack.org/68722 | 18:52 |
*** datajerk has quit IRC | 18:53 | |
NobodyCam | brb mak'n mo coffee | 18:54 |
*** datajerk has joined #openstack-ironic | 18:54 | |
*** jcooley_ has quit IRC | 19:01 | |
*** aignatov is now known as aignatov_ | 19:20 | |
*** vkozhukalov has quit IRC | 19:22 | |
NobodyCam | attempting to boot undercloud again | 19:25 |
*** harlowja is now known as harlowja_away | 19:31 | |
NobodyCam | brb | 19:47 |
*** amrit has joined #openstack-ironic | 19:50 | |
*** tvakhitova has joined #openstack-ironic | 19:50 | |
*** harlowja_away is now known as harlowja | 19:59 | |
*** tvakhitova has quit IRC | 20:05 | |
*** datajerk has quit IRC | 20:23 | |
*** datajerk has joined #openstack-ironic | 20:24 | |
lucasagomes | ok I'm done for today, g'night everybody! | 20:28 |
*** mdurnosvistov has quit IRC | 20:28 | |
*** ifarkas has quit IRC | 20:28 | |
*** aignatov_ is now known as aignatov | 20:29 | |
lucasagomes | night rloo devananda NobodyCam :) | 20:30 |
rloo | night lucasagomes! | 20:30 |
devananda | night, lucasagomes | 20:31 |
jbjohnso | off to lenovo with me | 20:32 |
NobodyCam | night lucasagomes | 20:34 |
NobodyCam | night jbjohnso | 20:35 |
*** mrda_away is now known as mrda | 20:42 | |
*** coolsvap is now known as coolsvap_away | 20:44 | |
*** ifarkas has joined #openstack-ironic | 20:54 | |
ifarkas | NobodyCam, have you figured out why your nova.conf is empty? it reminds me of this heat bug: https://review.openstack.org/#/c/68135 | 20:55 |
*** coolsvap_away has quit IRC | 20:56 | |
NobodyCam | actually it does kinda look like that | 20:57 |
NobodyCam | lol and only 4 hours befor the gate starts to test that patch | 20:58 |
ifarkas | yep, I built my image with that heat patch included | 20:58 |
* NobodyCam thinks he'll do the same... | 20:59 | |
ifarkas | the issue is caused by heat not setting up the user role properly | 21:00 |
ifarkas | and then when os-refresh-config wants to retrieve the heat metadata it just get an unauthorized error | 21:00 |
NobodyCam | that would explain Failed to add user undercloud-CompletionHandle-lcgo6yuuhyep to role heat_stack_user, check role exists! | 21:00 |
NobodyCam | ifarkas: you applied that patch to both seed and undercloud? | 21:02 |
ifarkas | NobodyCam, before building the seed image, I did: export DIB_REPOLOCATION_heat="https://review.openstack.org/openstack/heat" | 21:03 |
ifarkas | export DIB_REPOREF_heat="refs/changes/35/68135/1" | 21:03 |
ifarkas | so it applied for both image | 21:03 |
NobodyCam | yep | 21:03 |
ifarkas | but I am only using heat on the seed for now | 21:03 |
NobodyCam | :) | 21:03 |
ifarkas | I want Ironic to deploy not Heat :-) | 21:03 |
NobodyCam | re boot seed... bbaib after looking for some food | 21:07 |
ifarkas | i am also leaving. good night everyone! | 21:13 |
NobodyCam | ifarkas: Thank you and have a good night | 21:14 |
ifarkas | NobodyCam, welcome ;-) | 21:15 |
*** ifarkas has quit IRC | 21:18 | |
* NobodyCam re deploys his undercloud | 21:33 | |
* devananda files expeses from LCA | 21:38 | |
NobodyCam | :) | 21:38 |
* NobodyCam notes that https://review.openstack.org/#/c/68135 did seem to be the root of most of the evil he was encountering | 21:52 | |
*** jdob has quit IRC | 21:56 | |
NobodyCam | quick walkies brb | 22:02 |
*** jbjohnso has quit IRC | 22:05 | |
*** linggao has quit IRC | 22:05 | |
*** rloo_ has quit IRC | 22:10 | |
*** max_lobur has joined #openstack-ironic | 22:19 | |
*** rloo has quit IRC | 22:25 | |
*** rloo has joined #openstack-ironic | 22:26 | |
*** rloo_ has joined #openstack-ironic | 22:28 | |
*** rloo has quit IRC | 22:42 | |
*** rloo has joined #openstack-ironic | 22:43 | |
*** lucasagomes has quit IRC | 22:49 | |
*** epim has joined #openstack-ironic | 22:55 | |
NobodyCam | brb | 22:56 |
romcheg | Have a good day/evening guys. See you tomorrow! | 23:01 |
*** rloo has quit IRC | 23:01 | |
*** rloo has joined #openstack-ironic | 23:02 | |
max_lobur | g'night romcheg :) | 23:02 |
NobodyCam | good night romcheg | 23:03 |
romcheg | max_lobur: I'd recommend you to follow my decision :) | 23:03 |
max_lobur | hehe | 23:04 |
max_lobur | I'm almost done | 23:04 |
NobodyCam | humm | 23:04 |
NobodyCam | error: invalid command 'egg_info' | 23:04 |
*** rloo has quit IRC | 23:05 | |
*** rloo has joined #openstack-ironic | 23:06 | |
*** rloo has quit IRC | 23:10 | |
*** rloo has joined #openstack-ironic | 23:10 | |
*** aignatov is now known as aignatov_ | 23:12 | |
*** matty_dubs is now known as matty_dubs|gone | 23:14 | |
*** anniec has joined #openstack-ironic | 23:17 | |
*** rloo has quit IRC | 23:18 | |
*** rloo has joined #openstack-ironic | 23:19 | |
*** rloo has quit IRC | 23:20 | |
*** rloo has joined #openstack-ironic | 23:20 | |
*** rloo has quit IRC | 23:21 | |
*** rloo has joined #openstack-ironic | 23:21 | |
*** rloo has quit IRC | 23:22 | |
*** rloo has joined #openstack-ironic | 23:23 | |
*** rloo has quit IRC | 23:49 | |
*** rloo has joined #openstack-ironic | 23:49 | |
*** rloo has quit IRC | 23:50 | |
*** romcheg has quit IRC | 23:50 | |
*** rloo has joined #openstack-ironic | 23:50 | |
*** rloo has quit IRC | 23:51 | |
*** rloo has joined #openstack-ironic | 23:51 | |
*** rloo has quit IRC | 23:51 | |
*** rloo has joined #openstack-ironic | 23:51 | |
*** rloo has quit IRC | 23:52 | |
*** rloo has joined #openstack-ironic | 23:53 | |
*** rloo has quit IRC | 23:53 | |
*** rloo has joined #openstack-ironic | 23:53 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!