*** aNupoisc has quit IRC | 00:03 | |
*** wznoinsk has joined #openstack-ironic | 00:05 | |
*** praneshp has joined #openstack-ironic | 00:10 | |
*** wajdi has joined #openstack-ironic | 00:13 | |
*** Sukhdev has quit IRC | 00:13 | |
*** Sukhdev has joined #openstack-ironic | 00:13 | |
wznoinsk | hi all, which deploy driver would you recommend as the least problematic? | 00:14 |
---|---|---|
*** ksumit has quit IRC | 00:15 | |
*** ksumit has joined #openstack-ironic | 00:15 | |
*** ksumit has quit IRC | 00:23 | |
*** ijw has joined #openstack-ironic | 00:32 | |
*** tuanla_ has joined #openstack-ironic | 00:37 | |
*** ksumit has joined #openstack-ironic | 00:39 | |
*** ijw has quit IRC | 00:40 | |
*** ijw has joined #openstack-ironic | 00:40 | |
*** joprovost has joined #openstack-ironic | 00:41 | |
*** ksumit has quit IRC | 00:48 | |
*** praneshp has quit IRC | 00:49 | |
*** rbudden has quit IRC | 00:52 | |
zer0c00l_ | https://review.openstack.org/326768 | 00:57 |
zer0c00l_ | oops | 00:58 |
*** sabeen has joined #openstack-ironic | 00:59 | |
*** xuao has joined #openstack-ironic | 01:01 | |
*** mtanino has joined #openstack-ironic | 01:02 | |
*** tangchen has quit IRC | 01:04 | |
*** penick has quit IRC | 01:07 | |
*** hoangcx has joined #openstack-ironic | 01:09 | |
*** praneshp has joined #openstack-ironic | 01:24 | |
openstackgerrit | Teng Fei proposed openstack/ironic-inspector: Modify the SQL's string from double quote to single quote. https://review.openstack.org/347795 | 01:27 |
*** Sukhdev has quit IRC | 01:28 | |
*** Sukhdev has joined #openstack-ironic | 01:30 | |
*** mtanino has quit IRC | 01:41 | |
*** rama_y has joined #openstack-ironic | 01:41 | |
*** mtanino has joined #openstack-ironic | 01:51 | |
*** rloo_ has joined #openstack-ironic | 01:53 | |
openstackgerrit | Merged openstack/ironic: Clean up release notes for 6.1.0 https://review.openstack.org/352660 | 01:53 |
*** rloo has quit IRC | 01:53 | |
*** Sukhdev has quit IRC | 01:56 | |
*** mtanino has quit IRC | 01:57 | |
*** chlong|mtg has joined #openstack-ironic | 01:57 | |
*** rloo_ has quit IRC | 02:00 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic-inspector: Add API for listing all introspection statuses https://review.openstack.org/344921 | 02:02 |
*** bfournie has joined #openstack-ironic | 02:04 | |
openstackgerrit | Merged openstack/ironic-specs: Fix Nova-compatible Serial Console's Other deployer impact https://review.openstack.org/335378 | 02:04 |
*** ijw has quit IRC | 02:05 | |
*** adu has joined #openstack-ironic | 02:07 | |
openstackgerrit | Merged openstack/ironic: Refactor OneView dynamic allocation release notes https://review.openstack.org/353049 | 02:17 |
*** jcoufal has joined #openstack-ironic | 02:17 | |
*** xuao has quit IRC | 02:22 | |
*** penick has joined #openstack-ironic | 02:35 | |
*** harlowja_at_home has joined #openstack-ironic | 02:39 | |
*** penick_ has joined #openstack-ironic | 02:40 | |
*** penick has quit IRC | 02:41 | |
*** penick_ is now known as penick | 02:41 | |
*** baoli has joined #openstack-ironic | 02:52 | |
*** harlowja_at_home has quit IRC | 02:54 | |
*** jcoufal has quit IRC | 02:55 | |
*** aNupoisc has joined #openstack-ironic | 02:58 | |
*** aNupoisc has quit IRC | 03:02 | |
*** baoli has quit IRC | 03:03 | |
*** zhangjl1 has joined #openstack-ironic | 03:05 | |
*** baoli has joined #openstack-ironic | 03:09 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic-inspector: Allow using introspection data in condition's 'value' field https://review.openstack.org/345535 | 03:12 |
*** krotscheck_ has joined #openstack-ironic | 03:12 | |
*** krotscheck has quit IRC | 03:13 | |
*** baoli has quit IRC | 03:13 | |
*** krotscheck_ is now known as krotscheck | 03:14 | |
*** BarbaraLove16 has joined #openstack-ironic | 03:20 | |
*** sabeen has quit IRC | 03:20 | |
*** sabeen has joined #openstack-ironic | 03:23 | |
*** BarbaraLove16 has quit IRC | 03:23 | |
*** links has joined #openstack-ironic | 03:37 | |
*** appu103_ has joined #openstack-ironic | 03:38 | |
*** appu103_ is now known as aparnav | 03:38 | |
*** amotoki has quit IRC | 03:39 | |
*** vikrant has joined #openstack-ironic | 03:40 | |
*** ljjjustin has quit IRC | 03:46 | |
*** mgkwill has quit IRC | 03:47 | |
*** mgkwill has joined #openstack-ironic | 03:48 | |
*** rama_y has quit IRC | 03:48 | |
*** jtomasek has quit IRC | 03:48 | |
*** ljjjustin has joined #openstack-ironic | 03:49 | |
*** praneshp has quit IRC | 03:50 | |
*** praneshp has joined #openstack-ironic | 03:51 | |
*** fragatina has quit IRC | 03:57 | |
openstackgerrit | Aparna proposed openstack/ironic: Update proliantutils version required for Newton release https://review.openstack.org/353301 | 03:58 |
*** fragatina has joined #openstack-ironic | 03:58 | |
*** fragatina has quit IRC | 03:58 | |
*** jtomasek has joined #openstack-ironic | 03:59 | |
*** hj-hpe has quit IRC | 04:02 | |
*** hjanssen-hpe has quit IRC | 04:02 | |
*** vishwanathj has quit IRC | 04:20 | |
*** vishwanathj has joined #openstack-ironic | 04:21 | |
*** aparnav has quit IRC | 04:30 | |
*** joprovost has quit IRC | 04:31 | |
*** vmud213 has joined #openstack-ironic | 04:34 | |
*** appu103_ has joined #openstack-ironic | 04:35 | |
*** psachin has joined #openstack-ironic | 04:36 | |
*** sabeen has quit IRC | 04:42 | |
*** ijw has joined #openstack-ironic | 04:54 | |
*** appu103_ is now known as aparnav | 04:57 | |
*** ijw has quit IRC | 04:59 | |
*** e0ne has joined #openstack-ironic | 05:17 | |
*** sdake has quit IRC | 05:20 | |
*** e0ne has quit IRC | 05:23 | |
*** rcernin has joined #openstack-ironic | 05:29 | |
*** ChubYann has quit IRC | 05:36 | |
*** fxpester|2 has joined #openstack-ironic | 05:41 | |
*** david-lyle_ has joined #openstack-ironic | 05:43 | |
*** TheJulia_ has joined #openstack-ironic | 05:43 | |
*** agordeev_ has joined #openstack-ironic | 05:43 | |
*** serverascode_ has joined #openstack-ironic | 05:43 | |
*** evgenyl_ has joined #openstack-ironic | 05:44 | |
*** beekneemech has joined #openstack-ironic | 05:45 | |
*** TheJulia has quit IRC | 05:48 | |
*** fxpester has quit IRC | 05:48 | |
*** evgenyl has quit IRC | 05:48 | |
*** david-lyle has quit IRC | 05:48 | |
*** agordeev has quit IRC | 05:48 | |
*** bapalm has quit IRC | 05:48 | |
*** serverascode has quit IRC | 05:48 | |
*** bnemec has quit IRC | 05:48 | |
*** bapalm_ has joined #openstack-ironic | 05:48 | |
*** TheJulia_ is now known as TheJulia | 05:48 | |
*** agordeev_ is now known as agordeev | 05:49 | |
*** evgenyl_ is now known as evgenyl | 05:49 | |
*** serverascode_ is now known as serverascode | 05:51 | |
openstackgerrit | Moshe Levi proposed openstack/ironic-python-agent: Add vendor id, vendor name and product_id to interface information https://review.openstack.org/279940 | 05:55 |
*** vishwanathj has quit IRC | 06:15 | |
*** vishwanathj has joined #openstack-ironic | 06:15 | |
*** adu has quit IRC | 06:17 | |
openstackgerrit | lokesh s proposed openstack/ironic: Adding follow-up patch for audit middleware https://review.openstack.org/350399 | 06:30 |
*** pcaruana has joined #openstack-ironic | 06:45 | |
*** moshele has joined #openstack-ironic | 06:47 | |
*** xuao has joined #openstack-ironic | 06:49 | |
*** rbartal has joined #openstack-ironic | 06:51 | |
*** tesseract- has joined #openstack-ironic | 06:53 | |
openstackgerrit | Ankit Kumar proposed openstack/proliantutils: Fix log message for exception in Gen7 https://review.openstack.org/353860 | 06:59 |
*** Goneri has joined #openstack-ironic | 07:01 | |
*** praneshp has quit IRC | 07:04 | |
*** ijw has joined #openstack-ironic | 07:11 | |
*** milan has joined #openstack-ironic | 07:11 | |
*** xuao has quit IRC | 07:14 | |
*** ijw has quit IRC | 07:15 | |
*** xuao has joined #openstack-ironic | 07:17 | |
*** ifarkas_afk is now known as ifarkas | 07:23 | |
*** Lord_Phoenix has joined #openstack-ironic | 07:29 | |
*** mbound has joined #openstack-ironic | 07:31 | |
*** ohamada has joined #openstack-ironic | 07:33 | |
*** aparnav has quit IRC | 07:36 | |
pas-ha | morning all | 07:36 |
*** xuao has quit IRC | 07:42 | |
*** jpich has joined #openstack-ironic | 07:43 | |
*** e0ne has joined #openstack-ironic | 07:45 | |
*** chlong|mtg has quit IRC | 07:46 | |
*** moshele has quit IRC | 07:55 | |
*** xuao has joined #openstack-ironic | 07:59 | |
xuao | morning ~~~~ | 07:59 |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-ironic | 08:00 | |
*** jtomasek_ has joined #openstack-ironic | 08:02 | |
*** moshele has joined #openstack-ironic | 08:07 | |
*** appu103_ has joined #openstack-ironic | 08:09 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 08:10 |
*** amoralej|off is now known as amoralej | 08:12 | |
*** rcarrillocruz has joined #openstack-ironic | 08:14 | |
*** lucas-dinner is now known as lucasagomes | 08:16 | |
lucasagomes | morning all | 08:16 |
xuao | morning~~~~~~ | 08:16 |
zhangjl1 | hey all | 08:16 |
xuao | hi lucasagomes, I am so sorry to bother you. But zhangjl1 and me have got into a new trouble..... : ( | 08:18 |
xuao | After we `nova boot` the `user-image`. We found our physical machine boot into the `deploy-system` requiring the username/password.Meanwhile, the ironic node was waiting for call back and then error. We wonder what can we do next, or if we can sign in the system to have a debug~ | 08:19 |
*** moshele has quit IRC | 08:21 | |
lucasagomes | zhangjl1, xuao 1 sec... I need to get my coffee first :D | 08:22 |
lucasagomes | s/get/drink | 08:22 |
xuao | Oh I think I need one cup too~~~ | 08:23 |
*** sdake has joined #openstack-ironic | 08:23 | |
*** moshele has joined #openstack-ironic | 08:24 | |
*** alineb has joined #openstack-ironic | 08:25 | |
*** appu103_ has quit IRC | 08:25 | |
*** appu103_ has joined #openstack-ironic | 08:25 | |
alineb | morning all | 08:25 |
xuao | morning~ | 08:26 |
moshele | lucasagomes: kindly reminder for review the infiniband patches https://review.openstack.org/#/c/264263/ and https://review.openstack.org/#/c/264257/ | 08:28 |
*** dtantsur|afk is now known as dtantsur | 08:29 | |
dtantsur | morning Ironic | 08:32 |
*** dtantsur is now known as dtantsur|bbl | 08:32 | |
*** appu103_ is now known as aparnav | 08:35 | |
lucasagomes | xuao, rigth so, it seems that the deployment ramdisk can't contact the Ironic node back (to move it from wait call-back to deploying state) | 08:36 |
lucasagomes | xuao, that's tricky to debug, depends how you've built the deploy ramdisk | 08:36 |
*** e0ne has quit IRC | 08:37 | |
lucasagomes | xuao, did you download one upstream (coreos one ?) or did you build one with DIB ? | 08:37 |
*** bvandewa has joined #openstack-ironic | 08:37 | |
zhangjl1 | we build deploy ramdisk with DIB | 08:37 |
zhangjl1 | with the command : disk-image-create ironic-agent fedora -o ironic-deploy | 08:38 |
lucasagomes | zhangjl1, right, so by default it doesn't come with any user... You can build another one and use elements such as "devuser" (https://github.com/openstack/diskimage-builder/tree/master/elements/devuser) or "dynamic-login" (https://github.com/openstack/diskimage-builder/tree/master/elements/dynamic-login) | 08:39 |
lucasagomes | zhangjl1, that will create a default user in the image so you will be able to log in the ramdisk | 08:39 |
lucasagomes | e.g, disk-image-create devuser ironic-agent fedora -o ironic-deploy | 08:40 |
lucasagomes | see the options of that element in the README as well https://github.com/openstack/diskimage-builder/tree/master/elements/devuser#environment-variables | 08:40 |
xuao | get√ we'll have a try following it ~~ | 08:41 |
*** electrofelix has joined #openstack-ironic | 08:41 | |
zhangjl1 | your means, we need to login to the ramdisk, then , then ironic deploying would be continue? | 08:41 |
*** mbound has quit IRC | 08:43 | |
*** e0ne has joined #openstack-ironic | 08:43 | |
openstackgerrit | Dao Cong Tien proposed openstack/ironic: Remove incorrect CONF.import_opt() from test_ipmitool.py https://review.openstack.org/353907 | 08:46 |
lucasagomes | zhangjl1, no I mean, you need to login to debug it | 08:48 |
lucasagomes | zhangjl1, usually the reason that happens is because the deploy ramdisk can't talk to the ironic api | 08:48 |
lucasagomes | due some network configuration issues | 08:48 |
zhangjl1 | ok!!! thanks for your advice | 08:49 |
lucasagomes | zhangjl1, once you log in, see the /proc/cmdline it will contain the URL for the ironic api | 08:49 |
lucasagomes | try to reach it | 08:49 |
zhangjl1 | i would try it now | 08:49 |
openstackgerrit | Imre Farkas proposed openstack/ironic: DRAC RAID configuration https://review.openstack.org/176319 | 08:52 |
ifarkas | morning all! | 08:53 |
xuao | morning ~~~~ | 08:53 |
zhangjl1 | hey all | 08:53 |
*** phuongnh has joined #openstack-ironic | 09:01 | |
*** permalac has quit IRC | 09:02 | |
openstackgerrit | lokesh s proposed openstack/ironic: Adding follow-up patch for audit middleware https://review.openstack.org/350399 | 09:02 |
openstackgerrit | Dao Cong Tien proposed openstack/ironic: Rename some variables in test_ipminative.py https://review.openstack.org/353914 | 09:02 |
*** Romanenko_K has joined #openstack-ironic | 09:03 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 09:04 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Source openrc on subnode in multinode topology https://review.openstack.org/353302 | 09:04 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Skip db configuration on subnodes https://review.openstack.org/353303 | 09:04 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Ignore required_services for multinode topology https://review.openstack.org/352793 | 09:04 |
openstackgerrit | Dao Cong Tien proposed openstack/ironic: Rename some variables in test_ipminative.py https://review.openstack.org/353914 | 09:08 |
*** bvandewa has quit IRC | 09:13 | |
milan | morning Ironic! :) | 09:16 |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Fix release notes formatting https://review.openstack.org/353927 | 09:19 |
aarefiev | morning! | 09:19 |
*** xuao has quit IRC | 09:22 | |
sambetts|afk | Mooooooorning everyone! | 09:23 |
*** sambetts|afk is now known as sambetts | 09:23 | |
*** xuao has joined #openstack-ironic | 09:25 | |
*** amitkqed has quit IRC | 09:27 | |
*** amitkqed has joined #openstack-ironic | 09:27 | |
xuao | morning ~~ | 09:29 |
*** xuao has quit IRC | 09:37 | |
*** xuao has joined #openstack-ironic | 09:37 | |
lucasagomes | sambetts, morning | 09:38 |
sambetts | Hey lucasagomes | 09:38 |
openstackgerrit | Dao Cong Tien proposed openstack/ironic: Remove incorrect CONF.import_opt() from test_ipmitool.py https://review.openstack.org/353907 | 09:38 |
openstackgerrit | Merged openstack/ironic: Updated tests for db migration scripts https://review.openstack.org/344246 | 09:40 |
*** xuao has quit IRC | 09:41 | |
*** xuao has joined #openstack-ironic | 09:44 | |
*** e0ne has quit IRC | 09:52 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic-inspector: Add API for listing all introspection statuses https://review.openstack.org/344921 | 09:52 |
*** e0ne has joined #openstack-ironic | 09:56 | |
*** dtantsur|bbl is now known as dtantsur | 10:04 | |
*** xuao has quit IRC | 10:06 | |
*** xuao has joined #openstack-ironic | 10:09 | |
*** hoangcx has quit IRC | 10:10 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 10:10 |
xuao | lucasagomes, it seems that the ironic_python_agent raise an error 401. And zhangjl1 is debuging now~ maybe difficult~ | 10:16 |
lucasagomes | xuao, oh right, that's good news actually. So you can ping the API | 10:17 |
lucasagomes | xuao, can you check if the ironic-python-agent process is running in the ramdisk? | 10:17 |
*** phuongnh has quit IRC | 10:17 | |
zhangjl1 | yes, i am sure is running | 10:17 |
lucasagomes | zhangjl1, there's any logs you can see ? | 10:18 |
zhangjl1 | wait minutes | 10:18 |
*** xuao has quit IRC | 10:21 | |
*** xuao has joined #openstack-ironic | 10:22 | |
milan | sambetts, morning, I could use some more opinions on https://review.openstack.org/#/c/348943 would you mind giving it a review? ;) | 10:22 |
*** aparnav has quit IRC | 10:22 | |
sambetts | milan: sure I'll try to take a look at some point today :) | 10:22 |
milan | dtantsur, morning, could you please take a look as well? ^^ | 10:23 |
milan | sambetts, thanks! | 10:23 |
zhangjl1 | lucasagomes: the error log is here, http://paste.openstack.org/show/554108/ | 10:23 |
*** eset has joined #openstack-ironic | 10:23 | |
milan | ifarkas, hey, same begging I'm afraid ;) ^^^ | 10:24 |
*** xuao has quit IRC | 10:25 | |
*** xuao has joined #openstack-ironic | 10:26 | |
*** mgould|afk is now known as mgould | 10:26 | |
mgould | morning Ironic! | 10:26 |
milan | morning mgould! :) | 10:26 |
mgould | morning milan sambetts dtantsur zhangjl1 lucasagomes xuao | 10:26 |
xuao | morning~~~ | 10:27 |
milan | morning aarefiev! :) thanks for the review yesterday, took me a bit to reply, though | 10:27 |
aarefiev | milan: hey, I'm looking your replies, about sqlalchemy, it's bug fixed only in 1.1.0 | 10:29 |
lucasagomes | zhangjl1, hmm are you deploying master right? | 10:29 |
lucasagomes | dtantsur, I wonder if we need to update the dib element to use the new endpoints for lookup/heartbeat ? | 10:29 |
milan | aarefiev, then it may get more worth using the enum.Enum | 10:30 |
ifarkas | milan, sure, on my list ;-) | 10:30 |
milan | ifarkas, thanks :) | 10:30 |
*** fragatina has joined #openstack-ironic | 10:31 | |
aarefiev | milan: it doesn't make sense to use value in enums, you should compare enum' members, not values | 10:32 |
zhangjl1 | my iroinc-api and ironic-conductor version is mitaka , and the ironic-python-agent is master branch with commit dfaf92b611d7c4abb98345fbcf21ef3ed4bb9d3b | 10:32 |
zhangjl1 | lucasagomes, is the version mismatch? | 10:32 |
lucasagomes | zhangjl1, oh that might be it then, cause it's trying to talk to v1/lookup | 10:32 |
lucasagomes | I don't think that endpoint existed in mitaka | 10:32 |
lucasagomes | it used the vendor passthru before | 10:33 |
lucasagomes | zhangjl1, apparently yes | 10:33 |
milan | aarefiev, interesting | 10:33 |
lucasagomes | zhangjl1, if you can build a ramdisk using the IPA from mitaka that would help | 10:33 |
* milan spins automaton with Enums | 10:33 | |
*** fragatina has quit IRC | 10:33 | |
*** Goneri has quit IRC | 10:33 | |
*** mbound has joined #openstack-ironic | 10:33 | |
*** fragatina has joined #openstack-ironic | 10:34 | |
aarefiev | milan: just trying with 1.1.0 version, it works fine, sqlalchemy problems ( | 10:34 |
milan | aarefiev, it didn't cross my mind I could use the attributes straight honestly | 10:35 |
zhangjl1 | lucasagomes: thanks a lot !!! | 10:35 |
*** amotoki has joined #openstack-ironic | 10:35 | |
milan | it just feels so odd you create a class with attributes of string values and once you access those they're changed behind the scenes | 10:37 |
aarefiev | milan: maybe create our own class for states or use global var here, interesting how it will looks like | 10:38 |
aarefiev | yep | 10:38 |
milan | aarefiev, that would be a way as well | 10:38 |
milan | the globals | 10:38 |
*** mbound has quit IRC | 10:38 | |
milan | since currently it's just the class | 10:38 |
aarefiev | tbt it's looks hacky now | 10:40 |
aarefiev | it looks | 10:40 |
milan | aarefiev, I know | 10:40 |
*** amotoki has quit IRC | 10:40 | |
xuao | 😂lucasagomes,Still feel puzzled about how how to build the ramdisk with version specified😂 | 10:42 |
lucasagomes | xuao, I don't know much DIB to actually help with it :-/ | 10:43 |
lucasagomes | xuao, maybe asking at #tripleo ? | 10:43 |
xuao | Thanks for your advice~~~~~~ | 10:43 |
openstackgerrit | Sam Betts proposed openstack/ironic-specs: Add spec for VLAN Aware Baremetal Instances https://review.openstack.org/277853 | 10:43 |
xuao | Help a lotttttttttttttttttttttttttttttttttttttttttttttt~ | 10:43 |
dtantsur | lucasagomes, the DIB element is deprecated, why should we? | 10:44 |
lucasagomes | xuao, you welcome | 10:44 |
lucasagomes | dtantsur, yeah, forget what I said... I think we found the problem already | 10:45 |
xuao | memeda~ | 10:45 |
dtantsur | ok :) | 10:45 |
openstackgerrit | Sam Betts proposed openstack/ironic-specs: Add spec for VLAN Aware Baremetal Instances https://review.openstack.org/277853 | 10:47 |
*** Goneri has joined #openstack-ironic | 10:47 | |
openstackgerrit | Sam Betts proposed openstack/ironic-specs: Add spec for interface attach and detach API https://review.openstack.org/317636 | 10:48 |
openstackgerrit | Sam Betts proposed openstack/ironic-specs: Add spec for interface attach and detach API https://review.openstack.org/317636 | 10:50 |
*** fragatin_ has joined #openstack-ironic | 10:50 | |
milan | aarefiev, yeah, the automaton doesn't mind using enum.Enum attributes as states and events: http://paste.openstack.org/show/554109/ | 10:52 |
*** athomas has quit IRC | 10:53 | |
aarefiev | milan: yeah, that make sense | 10:54 |
*** fragatina has quit IRC | 10:54 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Fix updating port MAC address for active nodes https://review.openstack.org/353450 | 10:56 |
milan | aarefiev, on the plus side of the current hacky way, the change to Enum won't be too bad. But simpler still would be to use plain class (no need to remove the utils code&unit tests) | 10:58 |
milan | just the SA db column will be more verbose | 10:59 |
milan | aarefiev, I'd vote for plain class for now rather than GLOBALs | 11:00 |
aarefiev | yep, I thought about some wrap to switch on enums later | 11:00 |
*** athomas has joined #openstack-ironic | 11:01 | |
*** ijw has joined #openstack-ironic | 11:02 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Document retrieving logs from the deploy ramdisk https://review.openstack.org/352483 | 11:04 |
*** xuao has quit IRC | 11:06 | |
*** ijw has quit IRC | 11:07 | |
vsaienk0 | good day Ironic'ers, dtantsur, sambetts, jroll, milan, aarefiev, vdrok, lucasagomes | 11:15 |
lucasagomes | vsaienk0, hello, good morning | 11:15 |
aarefiev | hey vsaienk0 | 11:15 |
milan | vsaienk0, morning! :) | 11:16 |
sambetts | o/ vsaienk0 | 11:16 |
mat128 | good morning vsaienk0 and everyone else :) | 11:16 |
jroll | morning everyone! | 11:16 |
* jroll too many names to list :) | 11:16 | |
mat128 | =) | 11:16 |
sambetts | hey jroll | 11:17 |
jroll | lucasagomes: dtantsur: sambetts: have you seen anything in the last 12 hours that tells us we aren't ready to release? | 11:17 |
lucasagomes | jroll, not that I can think of | 11:17 |
dtantsur | morning jroll, vsaienk0 | 11:17 |
dtantsur | lemme think | 11:17 |
sambetts | only thing might be that, the IPA image we're testing with in the gate is super out of date, because of the post job breakages | 11:18 |
dtantsur | oh yeah, this is bad | 11:18 |
lucasagomes | jroll, there's a cool patch from devananda using the new policy mechanism to hide the configdrive content | 11:18 |
sambetts | I'd like to see a couple of test runs with the latest IPA to make sure nothing is busted | 11:19 |
lucasagomes | I've +2 it today, it may be a cool thing to include in the release cause it touches security | 11:19 |
lucasagomes | for ref: https://review.openstack.org/#/c/326768/ | 11:19 |
jroll | hm, we merged something in IPA very recently | 11:19 |
dtantsur | pas-ha, hey! do you still plan on https://review.openstack.org/#/c/253054/? | 11:20 |
jroll | lucasagomes: I'm ready to cut off new things, I wanted to release this tuesday but the reno updates took longer than I though | 11:20 |
jroll | t | 11:20 |
sambetts | jroll: I guess the src job on there should have caught most things right? | 11:20 |
lucasagomes | jroll, fair enough, so I would say: go ahead and release it :D | 11:20 |
jroll | sambetts: the one on IPA, yeah | 11:20 |
lucasagomes | we can include the rest in the next one | 11:21 |
dtantsur | we're going to release ironic-inspector too, but we need to fix our release notes | 11:21 |
dtantsur | aarefiev is working on it | 11:21 |
*** appu103_ has joined #openstack-ironic | 11:21 | |
jroll | awesome | 11:22 |
sambetts | jroll: unforunatly there hasn't been anything merged into IPA to trigger the post job since my fix for it merged ysterday | 11:22 |
dtantsur | let's merge something :) any typos? | 11:22 |
jroll | ^^ | 11:22 |
jroll | doc updates always welcome ;) | 11:22 |
aarefiev | yep, I'm updating it now | 11:22 |
milan | morning jroll, mat128! :) | 11:23 |
jroll | sambetts: I've got a thing | 11:23 |
openstackgerrit | Ricardo Carrillo Cruz proposed openstack/bifrost: Default auth to empty dict instead of None https://review.openstack.org/353990 | 11:23 |
*** moshele has quit IRC | 11:24 | |
sambetts | jroll: oh nice! I was just looking at the docs and thinking we need to update the APIs in there based on dtantsur promotion patch | 11:24 |
lucasagomes | jroll, we can merge https://review.openstack.org/#/c/284309/1 after you fix it | 11:24 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic-python-agent: Make code blocks real code blocks in metrics docs https://review.openstack.org/353992 | 11:24 |
jroll | lucasagomes: oh, forgot about that one | 11:25 |
* jroll does it | 11:25 | |
jroll | that docs patch will push through the gate quicker :) | 11:25 |
rcarrillocruz | TheJulia: heya, i filed a bug and pushed change https://review.openstack.org/353990 | 11:25 |
rcarrillocruz | let me know what you think when you're around | 11:25 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic-python-agent: Warn if extension fails to load https://review.openstack.org/284309 | 11:26 |
jroll | lucasagomes: there you are, thanks for the reminder :) | 11:26 |
jroll | should I have tests for that, though? | 11:26 |
dtantsur | jroll, sambetts, mind checking https://review.openstack.org/#/c/353450/ please? seems to work locally | 11:26 |
lucasagomes | jroll, hmm would be good indeed :-/ | 11:27 |
jroll | wow, post queue at 4 hours today :/ | 11:27 |
sambetts | wow... | 11:27 |
jroll | sambetts: I say we just ship it, and have a 6.0.1 if something unexpected comes up | 11:27 |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Fix release notes formatting https://review.openstack.org/353927 | 11:28 |
jroll | and I'll wait to ship the agent until we get a proper build | 11:28 |
*** amitkqed has quit IRC | 11:28 | |
jroll | dtantsur: on my list | 11:28 |
sambetts | makes sense, to me, most things should have been covered by the -src job run on the IPA merge that happened ~43hrs ago | 11:28 |
*** amitkqed has joined #openstack-ironic | 11:28 | |
jroll | sambetts: indeed, I think we're fine | 11:28 |
jroll | https://review.openstack.org/#/c/353995/ \o/ | 11:30 |
*** moshele has joined #openstack-ironic | 11:30 | |
aarefiev | dtantsur: would be nice to have this one https://review.openstack.org/#/c/347795/ | 11:31 |
vdrok | morning | 11:31 |
aarefiev | morning vdrok | 11:31 |
vdrok | morning aarefiev | 11:32 |
dtantsur | aarefiev, yeah, if the author updates it today | 11:32 |
dtantsur | morning vdrok | 11:32 |
vdrok | dtantsur: good morning | 11:32 |
moshele | vdrok: hi, can you review again https://review.openstack.org/#/c/264263/ | 11:35 |
vdrok | moshele: hello, will try to | 11:36 |
moshele | dtantsur: I upload new patch for the vendor-id like JayF proposed https://review.openstack.org/#/c/279940/ | 11:36 |
TheJulia | Good morning | 11:36 |
moshele | dtantsur: also I hope I understood you comments on https://review.openstack.org/#/c/264257/ | 11:37 |
dtantsur | moshele, now you need your RFE https://bugs.launchpad.net/ironic-python-agent/+bug/1611856 approved | 11:37 |
openstack | Launchpad bug 1611856 in ironic-python-agent "[RFE] expose port vendor id and product_id" [Wishlist,In progress] - Assigned to Moshe Levi (moshele) | 11:37 |
dtantsur | jroll, ^^^ | 11:37 |
jroll | was this not in the spec? | 11:37 |
* jroll approves | 11:38 | |
jroll | ¯\_(ツ)_/¯ | 11:38 |
dtantsur | sambetts, you can remove your -2 now ^^^ | 11:38 |
moshele | dtantsur: so currently there is no usage for this, but maybe TripleO in the future | 11:38 |
jroll | much process, wow | 11:38 |
TheJulia | rcarrillocruz: I truly don't remember the set_fact bit, monty's review is right imho. I think we split the roles and added omit after setting the fact stuffs | 11:38 |
sambetts | dtantsur: done :) | 11:39 |
dtantsur | thnx | 11:39 |
openstackgerrit | Szymon Borkowski proposed openstack/ironic-inspector: Add PCI devices plugin to inspector https://review.openstack.org/352295 | 11:40 |
sambetts | dtantsur: code for the mac address update bug fix looks good for me, just waiting for Jenkins :) | 11:41 |
dtantsur | cool | 11:42 |
*** chlong|mtg has joined #openstack-ironic | 11:43 | |
lucasagomes | dtantsur, can you take a look at https://review.openstack.org/#/c/354004/ ? ironic-staging-drivers gate is broken due that | 11:45 |
lucasagomes | I mean, due one patch that landed in Ironic. That's fixing the tests in ironic-staging-drivers | 11:45 |
dtantsur | LGTM, assuming it passes | 11:45 |
lucasagomes | ty | 11:52 |
*** lucasagomes is now known as lucas-hungry | 11:52 | |
dtantsur | sambetts, could you please review https://review.openstack.org/#/c/353927/ ? | 11:53 |
dtantsur | we need it for the releaser | 11:53 |
dtantsur | * release | 11:53 |
*** narasimha_SV has joined #openstack-ironic | 11:53 | |
narasimha_SV | SSHPowerDriver requires one and only one of password, key_contents and key_filename to be set. | 11:54 |
narasimha_SV | I am using pxe_ssh driver for KVM | 11:54 |
*** xuao has joined #openstack-ironic | 11:54 | |
narasimha_SV | http://paste.openstack.org/show/554121/ | 11:54 |
narasimha_SV | when I try to validate the node I am getting this information | 11:55 |
narasimha_SV | when I give ssh key path in the driver_info | 11:55 |
narasimha_SV | then http://paste.openstack.org/show/554122/ | 11:55 |
narasimha_SV | this is what I am getting the issue | 11:56 |
*** xuao has quit IRC | 11:56 | |
narasimha_SV | please help me in understanding what other values I missed here to get validation properly | 11:56 |
*** xuao has joined #openstack-ironic | 11:56 | |
*** milan has quit IRC | 11:56 | |
jroll | narasimha_SV: you had this problem earlier, it looks like a permissions issue, the user running the ironic-conductor service needs to be able to read the key | 11:57 |
narasimha_SV | as part of connection i have ssh_address of the host , username and password of the host to connect | 11:57 |
sambetts | dtantsur: sure looking now | 11:57 |
narasimha_SV | jroll: is there any other way to do this | 11:58 |
narasimha_SV | rather than giving the key file name | 11:58 |
jroll | narasimha_SV: username and password should work fine without setting key_filename | 11:58 |
jroll | it's ssh_address, ssh_username, ssh_password | 11:58 |
narasimha_SV | when I use username and password node is not getting validated properly | 11:58 |
narasimha_SV | http://paste.openstack.org/show/554121/ | 11:59 |
jroll | are you using password or ssh_password? username or ssh_username? | 11:59 |
narasimha_SV | yes ssh_username and ssh_password | 11:59 |
jroll | are you sure there's no typos? please paste a node-show and censor the password | 12:00 |
narasimha_SV | ironic node-create -d pxe_ssh -i ssh_address='192.168.56.101' -i ssh_username='root' -i ssh_pasword='password' -i ssh_virt_type='virsh' -n $NAME | 12:00 |
jroll | ssh_password, not ssh_pasword | 12:00 |
narasimha_SV | ok sorry | 12:00 |
rcarrillocruz | TheJulia: yup, I think removing those facts will be functionally equivalent. I will test and push patchset if It works too | 12:00 |
jroll | narasimha_SV: no worries, you helped me find a bug :D | 12:01 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Fix ssh credential validation message https://review.openstack.org/354008 | 12:01 |
jroll | super easy review there :) | 12:01 |
jroll | one liner | 12:01 |
narasimha_SV | yes | 12:01 |
narasimha_SV | thanks it worked | 12:01 |
jroll | you're welcome | 12:01 |
*** milan has joined #openstack-ironic | 12:03 | |
TheJulia | rcarrillocruz: awesome, thank you | 12:04 |
*** baoli has joined #openstack-ironic | 12:10 | |
*** tuanla_ has quit IRC | 12:12 | |
openstackgerrit | Dao Cong Tien proposed openstack/ironic: Remove CONF.import_opt() from api/controllers/v1/node.py https://review.openstack.org/354012 | 12:15 |
*** hshiina has joined #openstack-ironic | 12:16 | |
*** xuao has quit IRC | 12:16 | |
*** xuao has joined #openstack-ironic | 12:18 | |
openstackgerrit | Merged openstack/ironic-inspector: Fix release notes formatting https://review.openstack.org/353927 | 12:20 |
*** xuao has quit IRC | 12:20 | |
*** xuao has joined #openstack-ironic | 12:21 | |
*** MattMan_ has quit IRC | 12:22 | |
*** MattMan has joined #openstack-ironic | 12:22 | |
dtantsur | jroll, as this ^^ landed, mind also proposing ironic-inspector release? | 12:24 |
*** rbartal has quit IRC | 12:24 | |
*** dprince has joined #openstack-ironic | 12:27 | |
jroll | dtantsur: sure | 12:27 |
*** xuao has quit IRC | 12:27 | |
jroll | dtantsur: 4.1.0? | 12:27 |
* jroll looks at relnotes | 12:27 | |
dtantsur | jroll, yeah, nothing major I think | 12:28 |
* jroll is glad there's a formatting fix heh | 12:28 | |
dtantsur | wow folks, we've got an amazing Ironic release out this time. thanks all! | 12:29 |
* dtantsur has just taken a look at the Ironic release notes | 12:29 | |
jroll | +1, so much good stuff in this release | 12:29 |
sambetts | \o/ | 12:29 |
jroll | thanks so much to everyone that contributed :) | 12:29 |
*** rbudden has joined #openstack-ironic | 12:30 | |
*** aparnav has joined #openstack-ironic | 12:32 | |
*** appu103_ has quit IRC | 12:32 | |
*** amoralej is now known as amoralej|lunch | 12:35 | |
*** ohamada has quit IRC | 12:35 | |
*** ohamada has joined #openstack-ironic | 12:36 | |
*** [1]cdearborn has joined #openstack-ironic | 12:39 | |
xavierr | Good morning Ironic 6.1.0 ;) | 12:40 |
nicodemos | =D | 12:40 |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Refactoring code of generating temp_url https://review.openstack.org/352289 | 12:41 |
jroll | xavierr: :D | 12:42 |
*** sdake has quit IRC | 12:43 | |
jroll | dtantsur: oh, I forgot to link you to the release request https://review.openstack.org/#/c/354026/ | 12:46 |
dtantsur | yeah, but I've seen it :) thanks | 12:46 |
jroll | ah, cool | 12:46 |
*** baoli_ has joined #openstack-ironic | 12:47 | |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Tempest: increase ironic sync timeout https://review.openstack.org/350499 | 12:48 |
*** hamzy has quit IRC | 12:48 | |
*** baoli has quit IRC | 12:50 | |
*** aparnav has quit IRC | 12:51 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Fix ssh credential validation message https://review.openstack.org/354008 | 12:51 |
*** zhangjl1 has left #openstack-ironic | 12:51 | |
openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add tests for testing node-delete command https://review.openstack.org/251458 | 12:52 |
*** lucas-hungry is now known as lucasagomes | 12:55 | |
*** amoralej|lunch is now known as amoralej | 12:58 | |
*** rloo has joined #openstack-ironic | 13:00 | |
*** vmud213 has quit IRC | 13:00 | |
*** appu103_ has joined #openstack-ironic | 13:06 | |
lucasagomes | vdrok, thanks for merging the gate fix ;D | 13:07 |
*** wajdi has quit IRC | 13:12 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add docs for node.resource_class and flavor creation https://review.openstack.org/345081 | 13:12 |
jroll | rloo: I feel like we'll have a couple back-and-forths there :) | 13:13 |
jroll | no rush on that patch though | 13:14 |
*** cdearborn has joined #openstack-ironic | 13:14 | |
rloo | jroll: really? I wonder why :) | 13:14 |
jroll | hehe | 13:14 |
rloo | jroll: I was just thinking that we really should get the docs done before we push a release (or say a feature is done) | 13:14 |
jroll | jaypipes: ^ would like your review on that one too | 13:14 |
jroll | rloo: if we want to do the former, we'll need to require docs with the code | 13:14 |
rloo | jroll: other random thought yesterday. given the amount of time/revisions it took to get the release notes done (I know, most of the delay was zuul), maybe we really ought to spend more time reviewing/cleaning up the release notes when they are originally proposed. (I really did want to punt that for the release person at the end but it does affect others too like me. ha ha.) | 13:16 |
*** z- has quit IRC | 13:16 | |
jroll | rloo: I completely agree, while we were going back and forth I was wishing the same thing, the reviews were really good | 13:17 |
rloo | jroll: i will keep that in mind for the future. I had let some things slide before. | 13:18 |
*** [1]cdearborn has quit IRC | 13:18 | |
jroll | rloo: I'm here to serve the team, so I don't mind doing the work to get them in order, but I think the reviews are the harder part of it so it doesn't really take any load off of people | 13:18 |
jroll | rloo: just takes some iteration load off of the committer | 13:18 |
rloo | jroll: right. | 13:18 |
rloo | jroll: i mean, right, you're here to serve the team! :D | 13:18 |
jroll | :) | 13:19 |
*** jcoufal has joined #openstack-ironic | 13:23 | |
*** z- has joined #openstack-ironic | 13:24 | |
jaypipes | jroll: done. | 13:24 |
jroll | thanks jaypipes | 13:25 |
jroll | vsaienk0: our grenade job upgrades everything, right? e.g. both nova and ironic are updated? | 13:26 |
vsaienk0 | jroll: yes, it upgrades everything | 13:26 |
*** narasimha_SV has quit IRC | 13:26 | |
jroll | ok thanks | 13:26 |
* jroll writing upgrade docs | 13:26 | |
*** jistr has quit IRC | 13:28 | |
lucasagomes | hi all, can I get some eyes on https://review.openstack.org/#/c/352820/ ? It's fixing a hi-prio bug and is +2 already | 13:29 |
lucasagomes | I also want to backport it to the mitaka branch once merged | 13:29 |
*** appu103_ has quit IRC | 13:29 | |
*** ppiela has joined #openstack-ironic | 13:30 | |
*** moshele has quit IRC | 13:30 | |
openstackgerrit | Cornelio Hopmann proposed openstack/ironic-specs: [RFE] Support network switches provisioning https://review.openstack.org/350570 | 13:30 |
*** jistr has joined #openstack-ironic | 13:30 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Bring upgrade documentation up to date https://review.openstack.org/354075 | 13:33 |
jroll | whee | 13:33 |
*** ohamada has quit IRC | 13:33 | |
*** ohamada has joined #openstack-ironic | 13:33 | |
sambetts | lucasagomes: +A, see my comment though | 13:37 |
openstackgerrit | Ricardo Carrillo Cruz proposed openstack/bifrost: Default auth to empty dict instead of None https://review.openstack.org/353990 | 13:40 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Clean up release note ipmitool-bootdev-persistent-uefi https://review.openstack.org/354079 | 13:40 |
jroll | lucasagomes: and a follow-up to clean up the release note ^ | 13:40 |
jroll | sambetts: if it's a problem in pyghmi, we should just fix it in pyghmi | 13:41 |
jroll | like later versions of ipmitool fixed it :) | 13:41 |
jroll | jaypipes: oh wow, I got all that right?! | 13:41 |
* jroll mind blown | 13:42 | |
sambetts | jroll: I meant more like adding the logic to actually include the efiboot option based on the nodes boot mode into the IPMINative power interface | 13:42 |
sambetts | no idea if pyghmi supported it :/ | 13:42 |
lucasagomes | sambetts, jroll I was thinking, it would be better to fix it all in 1 commit | 13:43 |
lucasagomes | sambetts, jroll to facilitate the backport | 13:43 |
lucasagomes | sambetts, can you revoke that +a and I will update the patch? | 13:43 |
jroll | sambetts: oh I see, you're right | 13:43 |
jroll | lucasagomes: no, please fix separate drivers in separate backports | 13:43 |
lucasagomes | re ipminative... I've no idea if that works, I have to check | 13:43 |
jroll | unless you mean the release note, +1 on that | 13:43 |
*** daemontool has joined #openstack-ironic | 13:43 | |
lucasagomes | jroll, I mean, the followups | 13:43 |
jroll | ++, let's squash it | 13:43 |
jroll | lucasagomes: you can -2 to stop the merge :) | 13:43 |
*** mgoddard_ has joined #openstack-ironic | 13:44 | |
lucasagomes | jroll, done :D | 13:44 |
jroll | cool | 13:44 |
lucasagomes | and I will test the ipminative driver in the env I have | 13:45 |
jroll | let me know when you squash it and I can approve and abandon my patch | 13:45 |
dtantsur | jroll, lucasagomes, removing W+1 is enough | 13:45 |
sambetts | lucasagomes: awesome :), I've never used IPMINative | 13:45 |
jroll | dtantsur: yeah, just telling lucas how to do it if sam disappeared or for the future | 13:45 |
dtantsur | oh yeah | 13:45 |
* sambetts vanishes into the night like batman | 13:46 | |
*** sborkows has joined #openstack-ironic | 13:46 | |
sambetts | :-P | 13:46 |
* jroll always forgets this part | 13:47 | |
*** sdake has joined #openstack-ironic | 13:47 | |
jroll | #success ironic 6.1.0 is released with multitenant networking, keystone policy, metrics, and more! | 13:47 |
*** mgoddard has quit IRC | 13:47 | |
openstackstatus | jroll: Added success to Success page | 13:47 |
lucasagomes | * [new tag] 6.1.0 -> 6.1.0 | 13:48 |
lucasagomes | w00t | 13:48 |
dtantsur | \o/ | 13:48 |
dtantsur | also the ironic-inspector release features LLDP discovery | 13:49 |
jroll | awesome! | 13:49 |
*** hamzy has joined #openstack-ironic | 13:49 | |
sambetts | \o/ | 13:49 |
*** thiagop has joined #openstack-ironic | 13:50 | |
thiagop | Good morning, ironic | 13:50 |
jroll | hey thiagop :) | 13:52 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: IPMITool: Check the boot mode when setting the boot device https://review.openstack.org/352820 | 13:53 |
lucasagomes | sambetts, jroll ^ fixed the commit message as well | 13:53 |
lucasagomes | thanks for the review | 13:53 |
*** abashmak has joined #openstack-ironic | 13:54 | |
sambetts | lucasagomes: awesome :D | 13:54 |
*** hoangcx has joined #openstack-ironic | 13:54 | |
*** Lord_Phoenix has quit IRC | 13:54 | |
*** abashmak has left #openstack-ironic | 13:54 | |
*** vikrant has quit IRC | 13:54 | |
jroll | lucasagomes: cool, thanks | 13:57 |
sambetts | lucasagomes: do you think we should add a comment to the raw_cmd as jroll suggested? | 13:58 |
* jroll +2 | 13:58 | |
lucasagomes | sambetts, I can do that if needed yes | 13:58 |
jroll | I'm not too worried about it | 13:58 |
lucasagomes | yeah, there are other raw byte strings around the code without explanation as well | 13:59 |
sambetts | me neither, if its all going to be removed, just wanted to check | 13:59 |
lucasagomes | maybe we can put a patch explaning 'em all | 13:59 |
lucasagomes | sambetts, yeah the idea is to remove that code once distros update their ipmitool package | 13:59 |
lucasagomes | thing is, it may take some time | 13:59 |
*** anush_ has joined #openstack-ironic | 14:00 | |
*** vikrant has joined #openstack-ironic | 14:01 | |
*** vikrant is now known as vikrant|gone | 14:01 | |
sambetts | yeah, you refer to the ipmi spec further up anyway if people are interested | 14:01 |
*** baoli_ has quit IRC | 14:02 | |
*** dprince_ has joined #openstack-ironic | 14:03 | |
*** dprince has quit IRC | 14:04 | |
*** sdake_ has joined #openstack-ironic | 14:07 | |
*** sdake has quit IRC | 14:09 | |
jroll | oh, this can go now that we did the release https://review.openstack.org/#/c/352991/ | 14:09 |
rloo | jroll: ? i thought that needed to wait til the newton release. | 14:09 |
rloo | jroll: you know that the next one will be the newton release? | 14:10 |
jroll | rloo: I planned to have it in the final newton release | 14:10 |
rloo | jroll: so shouldn't we wait til you are about to do the final release? | 14:10 |
jroll | rloo: yeah, maybe we should | 14:11 |
jroll | or at least until we know there's nothing between | 14:11 |
*** [1]cdearborn has joined #openstack-ironic | 14:11 | |
rloo | jroll: right, cuz we could do 4 releases between now and then :) | 14:11 |
jroll | could and will are different, but yeah maybe we can get a bunch of stuff done and have 6.2 in a few weeks | 14:11 |
rloo | jroll: yup | 14:12 |
jroll | okay, I re-minus-2'd | 14:13 |
rloo | jroll: wrt upgrade documentation. do you know how the 'official' openstack doc handle that kind of documentation? | 14:13 |
jroll | it could still use reviews though :D | 14:13 |
jroll | rloo: there's an upgrades section in the ops guide or something | 14:13 |
rloo | jroll: yeah, i'll review, but figured i'd procrastinate on it. i was actually wondering if we should only do that in an ocata release but didn't want to think about that too much. | 14:14 |
jroll | yeah, no worries | 14:14 |
jroll | I don't want to block us from removing it in ocata | 14:14 |
jroll | but I think it will depend on how well the new thing works in prod :) | 14:14 |
rloo | jroll: any idea how we can get our upgrades in that ops guide? | 14:14 |
jroll | rloo: I suspect they will 'big tent' it soon, like the api-ref and the install guide | 14:15 |
jroll | so I'm kinda just waiting for that | 14:15 |
jroll | I want to do the plugin thing for our install guide in ocata | 14:15 |
rloo | jroll: oh, so not there yet. ok. we *should* get our install guide there though. | 14:15 |
jroll | +1 | 14:15 |
jroll | rloo: I think after the install guide we can start asking about the upgrade guide | 14:16 |
rloo | how much work is it to get the plugin? would be nice to have it in newton. | 14:16 |
jroll | I'm not sure, it shouldn't be terrible | 14:16 |
*** jistr is now known as jistr|call | 14:16 | |
jroll | here's the guide: http://docs.openstack.org/contributor-guide/project-install-guide.html | 14:16 |
rloo | where's JayF? Does he know? | 14:16 |
jroll | some project-config changes | 14:16 |
jroll | some in-repo changes | 14:16 |
rloo | jroll: am going to add a bug about that unless there is one already. i wonder if it is low-hanging-fruit... | 14:17 |
jroll | my worry toward end-of-cycle would be about conflicting with other install guide changes | 14:17 |
vdrok | lucasagomes: around? | 14:17 |
jroll | rloo: I'm not sure if there is a bug or not yet | 14:17 |
vdrok | morning jroll and rloo | 14:17 |
jroll | hey vdrok :) | 14:17 |
rloo | jroll: no bug. is it a bug or a RFE to move the install guide? | 14:18 |
rloo | hi vdrok | 14:18 |
jroll | rloo: probably RFE, but I can approve it now if you file one :) | 14:18 |
lucasagomes | vdrok, hi there | 14:19 |
vdrok | lucasagomes: I was wondering about enabling the iboot driver in the staging drivers gate job, I thought you said you'll do that, or do you want to do that in a separate change? | 14:19 |
lucasagomes | vdrok, oh, I think I forgot to do that... but if we could do in a separated change it would be fine too | 14:20 |
lucasagomes | would save me some time actually | 14:20 |
vdrok | lucasagomes: yup, +2 then :) | 14:20 |
*** wajdi has joined #openstack-ironic | 14:20 | |
lucasagomes | cool, I'm just testing the ipminative with persistent + uefi and I will see that right after | 14:20 |
* lucasagomes adds a todo | 14:20 | |
*** wajdi has joined #openstack-ironic | 14:21 | |
lucasagomes | vdrok, thanks for the reminder | 14:22 |
rloo | jroll: here's the RFE: https://bugs.launchpad.net/ironic/+bug/1612278 | 14:22 |
openstack | Launchpad bug 1612278 in Ironic "[RFE] publish install guide on docs.openstack.org" [Wishlist,New] | 14:22 |
*** cdearborn has quit IRC | 14:22 | |
*** beekneemech is now known as bnemec | 14:23 | |
jroll | thanks rloo, approved | 14:23 |
rloo | jroll: might be more than 'just' following those instructions. we've been talking about reorg'ing the install guide. something to keep in mind i guess. will think about it later :) | 14:23 |
jroll | rloo: yeah, I think we should :) | 14:24 |
jroll | or at least break it up a bit | 14:24 |
*** causten has joined #openstack-ironic | 14:24 | |
vsaienk0 | jroll, sambetts: could you please have look https://review.openstack.org/#/c/353921/1/jenkins/jobs/ironic.yaml | 14:24 |
jroll | maybe JayF can architect how that's laid out :D | 14:24 |
*** joprovost has joined #openstack-ironic | 14:24 | |
*** syed_ has joined #openstack-ironic | 14:25 | |
rloo | jroll: heh, yeah, i was going to ping JayF about that :) | 14:25 |
vsaienk0 | jroll, sambetts: that patch switch multinode job to use multitenancy | 14:25 |
sambetts | vsaienk0: making all jobs test multitenant? | 14:25 |
sambetts | oooh right yes | 14:25 |
jroll | vsaienk0: +1 | 14:25 |
vsaienk0 | sambetss: no only multinode jobs | 14:25 |
jroll | rloo: I think his ops experience will be super helpful figuring out how to group that | 14:26 |
rloo | jroll: i totally agree! ;) | 14:26 |
rloo | jroll: mat128 might also be helpful there | 14:26 |
jroll | +1 | 14:26 |
sambetts | vsaienk0: can we get rid of other multitenancy job then? | 14:26 |
mat128 | reading scrollback :) | 14:27 |
*** joprovost has quit IRC | 14:27 | |
vsaienk0 | jroll: once more, when you have a time https://review.openstack.org/#/c/340295/, we talked about multitest job. | 14:27 |
jroll | mat128: tl;dr breaking up the install guide | 14:27 |
mat128 | oh | 14:27 |
mat128 | sure | 14:27 |
*** joprovost has joined #openstack-ironic | 14:27 | |
jroll | sambetts: oh, we probably could | 14:27 |
jroll | gotta get multinode stable first | 14:27 |
rloo | mat128: in preparation for putting the install guide in the ops documentation | 14:27 |
vsaienk0 | sambetts: you mean when we have green multinode job with multitenancy remove current single node multitenancy job? | 14:27 |
sambetts | yup | 14:28 |
vsaienk0 | sambetts: sure, I think this is our goal | 14:28 |
rloo | mat128: in fact, you interested in doing it? https://bugs.launchpad.net/ironic/+bug/1612278 | 14:28 |
openstack | Launchpad bug 1612278 in Ironic "[RFE] publish install guide on docs.openstack.org" [Wishlist,Confirmed] | 14:28 |
mat128 | yup sure | 14:28 |
rloo | mat128: awesome! and thank you! | 14:29 |
sambetts | vsaienk0: I'm interested to what we are gaining by testing multutenantcy in the mutinode job though | 14:29 |
lucasagomes | jroll, sambetts *sigh* ipminative does not check the boot mode either | 14:29 |
lucasagomes | good news is that pygmi itself works setting persistent + uefi | 14:29 |
* lucasagomes uploads a patch | 14:29 | |
mat128 | rloo: assigned myself to the bug, added to my todo list | 14:30 |
mat128 | thanks for the link | 14:30 |
rloo | thank YOU mat128. I expect great things from you. Ha ha. | 14:30 |
mat128 | =) | 14:30 |
sambetts | lucasagomes: tbh I wonder how many other drivers don't take that into account? and if that sort of thing happens in non-ipmi cases | 14:30 |
vsaienk0 | sambetts: we will be able to add 7 VMs on primary node and 7 VMs on subnode, it will allow to increase tempest concurrency | 14:30 |
lucasagomes | sambetts, probably many others | 14:30 |
jroll | vsaienk0: is there a patch to add exercise.sh yet? | 14:31 |
lucasagomes | sambetts, I think we know that drac doesn't work with efi either | 14:31 |
mat128 | sambetts, lucasagomes: wasnt it only an IPMI bug? | 14:31 |
mat128 | ah, that kind of drivers :) | 14:31 |
*** hjanssen-hpe has joined #openstack-ironic | 14:31 | |
vsaienk0 | jroll: not yet, but I'm going to start working on it once we have a job :) | 14:31 |
*** hpe-hj has joined #openstack-ironic | 14:31 | |
jroll | sambetts: we're using multitenancy because our current "flat" doesn't work with multinode, we either need real flat networking or multitenant, and multitenant is the future :D | 14:31 |
*** amotoki has joined #openstack-ironic | 14:31 | |
jroll | vsaienk0: okay | 14:31 |
lucasagomes | sambetts, it would be good to have a uefi test in gate | 14:31 |
sambetts | lucasagomes: right | 14:31 |
lucasagomes | sambetts, we could configure qemu with uefi I think | 14:32 |
vsaienk0 | sambetts: btw I already prepared patches to work with real FLAT setup | 14:32 |
sambetts | lucasagomes: it would be good to have a tempest test case for an EFI node that can be run | 14:32 |
jroll | vsaienk0: +1 | 14:32 |
lucasagomes | sambetts, ++ | 14:32 |
* lucasagomes fix ipminative for now | 14:32 | |
sambetts | ++ | 14:32 |
sambetts | sigh deployment purmutation testing is a pain in the ass :-P | 14:33 |
*** chlong|mtg has quit IRC | 14:33 | |
openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Negative tests for port-related commands https://review.openstack.org/259354 | 14:33 |
lucasagomes | sambetts, yeah, too many configs :-) | 14:33 |
jroll | yeah, we really need to work on that next cycle | 14:33 |
lucasagomes | jroll, re gate jobs... I want to use the wipe disk metadata cleaning step once merged | 14:34 |
lucasagomes | to test the cleaning steps in gate | 14:34 |
*** michauds has joined #openstack-ironic | 14:34 | |
lucasagomes | we have a bug open for that (I guess) | 14:34 |
jroll | lucasagomes: +1 | 14:34 |
dtantsur | great idea | 14:34 |
jroll | lucasagomes: you plan to validate it was wiped as well? | 14:34 |
lucasagomes | wiping the disk metadata is really fast, so I think it suits for gate tests | 14:34 |
sambetts | I can imagine infra continuing to get annoyed at us for too many jobs ;) | 14:34 |
lucasagomes | jroll, yeah, I would mount the device in a loop and see if the partition table has been wipes | 14:34 |
lucasagomes | wiped | 14:34 |
lucasagomes | after the node is cleaned | 14:35 |
jroll | lucasagomes: ++ | 14:35 |
jroll | awesome | 14:35 |
lucasagomes | the device I mean, disk device | 14:35 |
dtantsur | sambetts, I think nova/neutron still have more, but I may be wrong | 14:35 |
jroll | dtantsur: seems about the same for nova https://review.openstack.org/#/c/354030/ | 14:35 |
jroll | compared to https://review.openstack.org/#/c/352991/ | 14:35 |
mat128 | we can always specify which jobs for which parts of the codebase changed | 14:37 |
mat128 | to reduce total jobs that run | 14:37 |
mat128 | lucasagomes: wow that'd be great | 14:37 |
jroll | please no, that'll get way too complex | 14:37 |
mat128 | hahaha | 14:37 |
lucasagomes | mat128, yeah, shouldn't be difficult to write the tempest bits | 14:37 |
lucasagomes | mat128, once the patch merges I will look at it | 14:37 |
jroll | I just noticed sambetts' refactor accidentally(?) made our job on nova voting | 14:38 |
sambetts | I'd really like us to do job that test parts of the deployment process, e.g. specifcally provisioning, specifically cleaning etc | 14:38 |
sambetts | instead of the whole process | 14:39 |
sambetts | jroll: did i?! | 14:39 |
jroll | heh, yeah | 14:39 |
sambetts | whoops XD | 14:39 |
*** jtomasek_ has quit IRC | 14:40 | |
sambetts | I must have missed an -nv somewhere | 14:40 |
jroll | we might want to fix that before the end-of-release rush starts pounding on the gate | 14:40 |
jroll | switching that to ipmitool will make it worse, probably | 14:40 |
jroll | gotta step away for a bit | 14:42 |
*** PollyZ has joined #openstack-ironic | 14:44 | |
sambetts | jroll: https://review.openstack.org/354124 | 14:44 |
*** chlong|mtg has joined #openstack-ironic | 14:46 | |
jroll | sambetts: thanks | 14:46 |
*** sdake has joined #openstack-ironic | 14:47 | |
*** sdake_ has quit IRC | 14:47 | |
openstackgerrit | Akilan Pughazhendi proposed openstack/ironic: Devstack setup fails on Fedora 23 https://review.openstack.org/354136 | 14:52 |
*** sborkows has quit IRC | 14:52 | |
*** rajinir has joined #openstack-ironic | 14:58 | |
dtantsur | with our product we hit all kinds of strange hardware/drivers, and here is an example: | 14:59 |
dtantsur | dmidecode does not report memory: https://bugzilla.redhat.com/show_bug.cgi?id=1366309 | 14:59 |
openstack | bugzilla.redhat.com bug 1366309 in openstack-ironic-inspector "ironic introspection fails on GCE node" [Unspecified,New] - Assigned to dtantsur | 14:59 |
dtantsur | any ideas what else to use/how to fix it? | 14:59 |
mat128 | dtantsur: No Module Installed | 15:00 |
mat128 | wow | 15:00 |
lucasagomes | dtantsur, maybe fallback to /proc/meminfo for ram if it fails to get it from dmidecoce ? | 15:01 |
mat128 | dtantsur: maybe also a dmesg dump | 15:01 |
lucasagomes | and yeah, that's an odd output there >.< | 15:01 |
mat128 | dtantsur: Ravello backed by something else doesnt do it? | 15:01 |
mat128 | according to reporter | 15:01 |
dtantsur | lucasagomes, /proc/meminfo gives wrong data IIRC | 15:01 |
mat128 | ehhh | 15:02 |
mat128 | how much ram does it have? | 15:02 |
dtantsur | dunno, just got this report | 15:02 |
mat128 | dtantsur: is it something like the xen memory balloning thats the issue? | 15:02 |
openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add tests for testing node-delete command https://review.openstack.org/251458 | 15:02 |
mat128 | i understand it's not xen, but this bit me in the past :) | 15:02 |
lucasagomes | dtantsur, really? It may be because some of the ram is used for hardware interruption | 15:02 |
lucasagomes | but I thought you had all information there | 15:03 |
dtantsur | lucasagomes, yeah, it does not account for reserved memory | 15:03 |
lucasagomes | you may have to consider few other fields apart from TotalMem | 15:03 |
mgould | `man free` suggests MemTotal + SwapTotal | 15:04 |
mat128 | free -m reports usable, not machine capacity :( | 15:04 |
mat128 | mgould: ^ | 15:04 |
mat128 | http://paste.openstack.org/show/554193/ | 15:05 |
mgould | mat128: so "capacity - space taken up by the kernel itself"? | 15:05 |
mat128 | or even this on a physical machine: http://paste.openstack.org/show/554194/ | 15:05 |
mat128 | oh, journal dump has kernel logs | 15:06 |
mat128 | for anyone not wanting to download the file http://paste.openstack.org/show/554195/ | 15:07 |
mat128 | L19 and 20, thats a kvm virtual machine | 15:07 |
mat128 | I wonder how they came up with no memory bank installed | 15:07 |
dtantsur | that's what I wonder too | 15:08 |
mat128 | is Ravello open source? guess not :( | 15:08 |
dtantsur | "It is a Google Compute Engine VM" hmm, I wonder if we support that at all | 15:08 |
mat128 | can we have the qemu/kvm cmdline ? | 15:08 |
mat128 | look up Ravello | 15:09 |
mat128 | Aug 11 09:38:15 localhost.localdomain kernel: Memory: 16048252k/17825792k available (6442k kernel code, 1049096k absent, 728444k reserved, 4264k data, 1620k init) | 15:09 |
openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add tests for testing node-delete command https://review.openstack.org/251458 | 15:09 |
dtantsur | nevermind folks, I suspect it should be CLOSED WONTFIX | 15:10 |
mat128 | well | 15:10 |
mat128 | I guess we shouldn't be solely relying on dmidecode | 15:10 |
mat128 | but at the same time, I have never seen dmidecode report "no memory bank installed" | 15:10 |
dtantsur | yeah, probably a limitation of GCE | 15:10 |
mat128 | well | 15:11 |
dtantsur | I'd not rush changing how we detect memory based on it alone | 15:11 |
*** penick has quit IRC | 15:12 | |
jroll | lol, GCE VM? | 15:14 |
*** PollyZ has quit IRC | 15:15 | |
mat128 | Ravello (i think) runs a vm on top of GCE | 15:15 |
mat128 | that apparently has only 1 memory slot with no memory bank installed | 15:15 |
jroll | so much meh | 15:15 |
*** vikrant|gone has quit IRC | 15:17 | |
*** hshiina has quit IRC | 15:17 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Fix Ubuntu 16.04 pxelinux path handling https://review.openstack.org/326024 | 15:17 |
mat128 | dtantsur: pretty sure they're using something like this https://github.com/qemu/qemu/blob/master/docs/memory-hotplug.txt | 15:18 |
mat128 | dtantsur: probably defining it with no memory and hotplugging some before starting the machine | 15:18 |
dtantsur | >_< | 15:18 |
mat128 | I'd love to try it :) | 15:19 |
mat128 | maybe I can find smt | 15:19 |
dtantsur | I'd love folks to stop using Ironic to manage VMs in production | 15:19 |
sambetts | this ^ | 15:19 |
jroll | +1000 | 15:19 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: IPMINative: Check the boot mode when setting the boot device https://review.openstack.org/354153 | 15:19 |
sambetts | rm -rf VM Drivers | 15:19 |
jroll | gotta wait a few more weeks, sambetts :P | 15:19 |
mat128 | Ravello is a lab thing, they might be doing exactly what we are doing with devstack | 15:19 |
lucasagomes | ^ patch fixing pygmi set_boot_device() to consider the boot mode | 15:19 |
mat128 | sambetts: you deleted all drivers and all VMs ;) | 15:20 |
sambetts | mat128: yup :-P | 15:20 |
jroll | lucasagomes: LGTM | 15:21 |
*** ifarkas is now known as ifarkas_afk | 15:21 | |
lucasagomes | jroll, that was quick... thanks :D | 15:21 |
jroll | yeah np :) | 15:21 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Fix Ubuntu 16.04 pxelinux path handling https://review.openstack.org/326024 | 15:21 |
jroll | it was a pretty straightforward patch :P | 15:21 |
*** jistr|call is now known as jistr | 15:21 | |
lazy_prince | has anyone tried using devuser element with ubuntu..? In my case, the user gets deleted after instance boot | 15:22 |
lucasagomes | yeah, luckily pyghmi already had a parameter for it | 15:22 |
sambetts | lazy_prince: I've had it working with ubuntu | 15:22 |
*** tesseract- has quit IRC | 15:23 | |
lucasagomes | and the behavior described in the pyghmi docstring is exactly what we've experienced https://github.com/openstack/pyghmi/blob/657608ca605034245cc537022b414708f9d10ea0/pyghmi/ipmi/command.py#L332-L336 | 15:23 |
sambetts | lucasagomes: Nice! | 15:24 |
lazy_prince | sambetts: can you share how you built your image.. just dib cmd and parameters and any environment vars that i need to export | 15:24 |
openstackgerrit | Ricardo Carrillo Cruz proposed openstack/bifrost: Remove auth line to fallback on default(omit) behaviour https://review.openstack.org/353990 | 15:24 |
*** baoli has joined #openstack-ironic | 15:25 | |
sambetts | lazy_prince: export DIB_DEV_USER_PASSWORD=password | 15:26 |
sambetts | disk-image-create ubuntu baremetal dhcp-all-interfaces grub2 devuser -o my-image | 15:26 |
sambetts | then I've logged in with devuser and password | 15:27 |
lazy_prince | I did same but its not working for me.. | 15:27 |
*** catintheroof has joined #openstack-ironic | 15:27 | |
lazy_prince | I can see the image has necessary info.. but after cloud-init is over, they disappear.. | 15:28 |
sambetts | lazy_prince: I don't think cloud-init should touch the users built into the image unless you have a custom cloud config that is deleteing it :/ | 15:30 |
lazy_prince | k.. let me try again... | 15:31 |
lucasagomes | lazy_prince, you can try the dynamic-login element as well | 15:31 |
lucasagomes | well tho it's more for the ramdisk... | 15:31 |
* lazy_prince checking | 15:31 | |
openstackgerrit | Merged openstack/ironic: Fix ssh credential validation message https://review.openstack.org/354008 | 15:32 |
mgould | dtantsur: people use Ironic for managing VMs in production? I thought those drivers were only used for testing... | 15:37 |
*** rbrud has quit IRC | 15:37 | |
dtantsur | both are correct, unfortunately | 15:38 |
mat128 | mgould: I think it's their test environment | 15:38 |
mat128 | Ravello is a product specifically for that use case | 15:38 |
mgould | ah, OK | 15:38 |
mat128 | but dtantsur is closer to the customer than I could be | 15:38 |
mat128 | so consider me wrong ;) | 15:38 |
mgould | CLOSED YOUHAVEMADEYOURBEDANDMUSTNOWLIEINIT | 15:38 |
rloo | hi dtantsur (and others). about https://bugs.launchpad.net/ironic/+bug/1569254, we allow updates to the node in lots of other situations (besides the console). eg, if node is active or available, i think we allow ipmi-related stuff to be modified. Maybe we need some better guidelines as to when things are allowed to be updated. | 15:39 |
openstack | Launchpad bug 1569254 in Ironic "It's possible to update node's console related properties while console is running" [Low,In progress] - Assigned to Pavlo Shchelokovskyy (pshchelo) | 15:39 |
mat128 | rloo: interesting bug | 15:39 |
dtantsur | yeah, maybe, I haven't put much thought in it | 15:39 |
mat128 | rloo: preventing IPMI config changes while node is active or available might a problem though | 15:40 |
mat128 | what if you actually need to change the values? | 15:40 |
rajinir | thiagop, sambetts: Is there a specific setting like port_enabled_security for tftp? The slave VM is not seeing the tftp traffic | 15:40 |
mat128 | rloo: console isnt too bad because you can just stop it, change and start it again | 15:40 |
rloo | dtantsur: me neither. but clearly, something we need to keep in mind when reviewing stuff... | 15:40 |
*** jcoufal has quit IRC | 15:40 | |
sambetts | rajinir: once port security is disabled all traffic should be allowed through | 15:40 |
mgould | mat128, rloo: can you prevent the user from calling ipmitool directly? | 15:41 |
mat128 | mgould: how could you? | 15:41 |
rloo | mat128: yeah, just wondering if we can come up with some guidelines to make it consistent/easy to know when things can be updated or not. | 15:41 |
mgould | mat128: that's what I thought | 15:41 |
rloo | mat128: anyway, just a thought. not going to think about it now :) | 15:41 |
mgould | so preventing IPMI changes at the Ironic level doesn't buy us much safety | 15:41 |
mat128 | mgould: the bug was about changing ipmi_address, user and pass | 15:42 |
rloo | mat128: otherwise, we are piecemeal addressing them as someone encounters an issue and they are coded/addressed in different ways. | 15:42 |
mat128 | or, more precisely, equivalents for the serial console | 15:42 |
lazy_prince | sambetts: figured out that the server has two disks with each one having "cloudimg-rootfs" partition... and post boot, it was mounting wrong partition | 15:42 |
mat128 | rloo: we need a guideline thats for sure | 15:42 |
mgould | oh, OK | 15:42 |
sambetts | lazy_prince: ah :( thats annoying | 15:42 |
mat128 | but if for some reason my bmc is at a different address now, i dont want Ironic to prevent me (and possibly keep acting on a machine thats not the right one) | 15:42 |
*** psachin has quit IRC | 15:42 | |
*** links has quit IRC | 15:43 | |
mgould | mat128: ah, gotcha | 15:43 |
sambetts | mgould, mat128: tbh I think we should prevent all updates like that for nodes in active unless moved to maintenance | 15:44 |
mat128 | sambetts: makes sense | 15:44 |
mat128 | they could be validated when you get out of maintenance | 15:45 |
sambetts | problem is that, that then leads to API behavior changes :'( | 15:45 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add "erase_devices_metadata_priority" config option https://review.openstack.org/344179 | 15:45 |
mat128 | rloo: i think something like what sambetts is suggesting could make a lot of sense and would be a general approach | 15:45 |
mat128 | sambetts: what do you mean? | 15:45 |
rloo | mat128, sambetts: yup. | 15:46 |
sambetts | mat128: something that was allowed before now is disallowed, so it technically breaks the API | 15:46 |
lucasagomes | vdrok, updated the sample file (plenty of new inclusions there) ^ | 15:46 |
mat128 | sambetts: argh... not again :) | 15:46 |
mat128 | microversion | 15:46 |
vdrok | lucasagomes: thx :) | 15:46 |
jroll | thiagop: are you still working on dropping the tests in tempest tree? | 15:46 |
rloo | sambetts: interesting. does it break the API, or is it fixing a bug so it is OK. | 15:47 |
rloo | sambetts: eg, look at the actual bug that was mentioned. if we fix it, does it need a version bump/ | 15:47 |
jroll | lucasagomes: would you mind landing this so that sambetts can have a build going? https://review.openstack.org/#/c/353992/ | 15:48 |
lucasagomes | jroll, boom done :-) | 15:48 |
jroll | thanks | 15:48 |
*** sdake_ has joined #openstack-ironic | 15:49 | |
sambetts | rloo: right, this is somewhat like what me and dtantsur did with the port mac address in a recent patch to prevent updates on an active node unless its in maintenance because it will result in network disruption | 15:49 |
*** mgoddard_ has quit IRC | 15:49 | |
*** mgoddard has joined #openstack-ironic | 15:50 | |
*** sdake has quit IRC | 15:51 | |
*** milan has quit IRC | 15:51 | |
jlvillal | dtantsur: Do you customize your colors for gertty? I'm not sure I'm thrilled with the defaults but not sure if I can come up with better ones :) | 15:52 |
dtantsur | jlvillal, no, I use the defaults | 15:52 |
jlvillal | dtantsur: Thanks | 15:52 |
sambetts | lucasagomes: thanks, hopefully once that merges the post job will pass | 15:53 |
lucasagomes | sambetts, ++ | 15:53 |
rajinir | sambetts: Able to see dhcp traffic, and the node gets ip assigned, but times out on TFTP. Any ideas on how to debug this? | 15:54 |
mat128 | rajinir: tftp server not reachable? | 15:54 |
mat128 | rajinir: tftp service not running? | 15:54 |
rajinir | mat128: TFTP open timeout | 15:55 |
sambetts | rajinir: TFTP address in the Ironic configuration the correct one? based on the subnet the node is in | 15:55 |
mat128 | can you tftp get from another machine? | 15:55 |
lazy_prince | sambetts: probably, I should have disk cleaning enabled to avoid such issues.. | 15:55 |
openstackgerrit | Merged openstack/ironic-python-agent: Make code blocks real code blocks in metrics docs https://review.openstack.org/353992 | 15:56 |
rajinir | mat128, sambetts: where is this configuration? Is it in ironic.conf? | 15:57 |
mat128 | rajinir: in your DHCP traffic dump, is the "next-server" IP the right one? | 15:57 |
sambetts | rajinir: you may need to set HOST_IP= the IP address of the slave VM in the network that the baremetal node is | 15:57 |
sambetts | rajinir: yes ironic.conf | 15:57 |
mat128 | rajinir: https://github.com/openstack/ironic/blob/master/etc/ironic/ironic.conf.sample#L2448 | 15:57 |
openstackgerrit | Akilan Pughazhendi proposed openstack/ironic: Fixes devstack setup failure on Fedora 23 https://review.openstack.org/354136 | 15:58 |
*** praneshp has joined #openstack-ironic | 16:01 | |
*** bnemec has quit IRC | 16:04 | |
rajinir | mat128>, sambetts: tftp_server set to the HOST_IP, looks right | 16:04 |
rajinir | mat128, sambetts: I can see the host machine is forwarding the packets to the correct IP. | 16:05 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: IPMITool: Check the boot mode when setting the boot device https://review.openstack.org/354186 | 16:06 |
mat128 | rajinir: so the node is making tftp requests and the server is replying but they reply isn't getting through? | 16:06 |
rajinir | sambetts: tftp is not running on the salve vm. | 16:06 |
lucasagomes | backport ^ | 16:06 |
mat128 | rajinir: can you describe/draw your logical setup? | 16:06 |
mat128 | lucasagomes: are you submitting ipminative for backport too? | 16:06 |
lucasagomes | mat128, yup doing it now | 16:06 |
sambetts | rajinir: oh? have your not using a normal devstack deployment? | 16:06 |
*** sabeen has joined #openstack-ironic | 16:07 | |
rajinir | mat128: This is thirdparty CI. so the request flows from Hardwarenode->Devstack Host Node -> Slave Vm running ironic tests | 16:07 |
*** e0ne has quit IRC | 16:07 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: IPMINative: Check the boot mode when setting the boot device https://review.openstack.org/354187 | 16:07 |
lucasagomes | mat128, there you go :D ^ | 16:07 |
mat128 | rajinir: no problem, but where is the node? | 16:07 |
mat128 | within "Slave Vm running ironic tests" ? | 16:07 |
jlvillal | lucasagomes: Was there a bug in that patch? | 16:08 |
rajinir | sambetts: I do use the devstack gate . Is there any configuration I'm missing? | 16:08 |
mat128 | rajinir: you mentioned the TFTP running somewhere else | 16:08 |
jlvillal | lucasagomes: Ah, back port! Nevermind | 16:08 |
mat128 | can you describe that? | 16:08 |
lucasagomes | jlvillal, those are the backport :D | 16:08 |
sambetts | rajinir: devstack should set up tftp for you on the jenkin slave | 16:08 |
rajinir | mat128: tftp should be running on the slave vm's | 16:09 |
lucasagomes | jlvillal, thanks for approving btw | 16:09 |
jlvillal | lucasagomes: Thanks for fixing it! :) | 16:09 |
lucasagomes | :-) | 16:09 |
rajinir | sambetts: no idea why tftp is not running, it is supposed to bet setup, agree | 16:11 |
*** bnemec has joined #openstack-ironic | 16:12 | |
NobodyCam | Good Morning Ironicers | 16:12 |
mat128 | hi NobodyCam | 16:12 |
NobodyCam | morning mat128 :) | 16:13 |
*** rcernin has quit IRC | 16:15 | |
*** baoli has quit IRC | 16:17 | |
*** baoli has joined #openstack-ironic | 16:18 | |
dtantsur | morning NobodyCam and good night to everyone else :) | 16:18 |
*** dtantsur is now known as dtantsur|afk | 16:18 | |
sambetts | Night dtantsur|afk | 16:20 |
sambetts | rajinir: how are you checking that its running? | 16:20 |
rajinir | sambetts: I don't see the service running. "service tftp status". There is no service installed. | 16:21 |
sambetts | in the ironic devstack script we run TFTP using xinetd | 16:22 |
*** daemontool has quit IRC | 16:23 | |
sambetts | jroll: do you know how to check if TFTP is running the way we run in it our devstack script ? | 16:23 |
NobodyCam | have a good night dtantsur|afk | 16:24 |
rajinir | sambetts: yeah, figured that out. But the file is setup right | 16:24 |
rajinir | https://www.irccloud.com/pastebin/EVrUKBuL/tftp | 16:24 |
jroll | sambetts: I've no clue, ps aux? | 16:25 |
sambetts | maybe netstat and look for the tftp port might be the best option then :-P | 16:25 |
jroll | netstat would be helpful, too, make sure it's running on the right interface | 16:25 |
jroll | yeah | 16:25 |
sambetts | on your VM slave host machine, can you see the packets arriving from the baremetal node/ | 16:26 |
sambetts | ? | 16:26 |
sambetts | if you do a tcpdump on the network interface that goes into that network | 16:26 |
lucasagomes | NobodyCam, hi there | 16:27 |
*** PollyZ has joined #openstack-ironic | 16:27 | |
rajinir | sambetts: I can see the packets on the host machine and getting passed to the Slave VM ip/HOST_IP | 16:28 |
*** ijw has joined #openstack-ironic | 16:29 | |
lucasagomes | sambetts, perhaps sending a "tftp get" (or another command) and check if it's running and listen ? | 16:29 |
*** sabeen1 has joined #openstack-ironic | 16:29 | |
lucasagomes | or check the tftp port via netstat idk | 16:30 |
NobodyCam | hey hey lucasagomes Good morning to you :) | 16:30 |
NobodyCam | also Good morning jroll sambetts rloo thiagop jlvillal JayF TheJulia and everyone else not listed here | 16:31 |
jaypipes | mornin | 16:31 |
rloo | hi, morning NobodyCam! which means lunchtime for me :) | 16:32 |
openstackgerrit | Michael Turek proposed openstack/ironic: Add classes for Portgroups API enablement https://review.openstack.org/347549 | 16:32 |
*** sabeen has quit IRC | 16:32 | |
NobodyCam | :) | 16:32 |
jlvillal | Hi NobodyCam :) | 16:33 |
NobodyCam | :) hey jlvillal :) | 16:34 |
jlvillal | Hey jaypipes :) I vote for a Salmon as the Nova mascot ;) | 16:34 |
jroll | morning NobodyCam :) | 16:34 |
* jlvillal learned that a type of salmon is called Nova | 16:34 | |
openstackgerrit | Michael Turek proposed openstack/ironic: Add classes for Portgroups API enablement https://review.openstack.org/347549 | 16:34 |
jlvillal | Or maybe that is, cured and smoked salmon | 16:35 |
jlvillal | mmmmm. smoked salmon :) | 16:35 |
NobodyCam | hey jroll :) | 16:35 |
lucasagomes | sambetts, http://paste.openstack.org/show/554225/ | 16:35 |
lucasagomes | rloo, hello too :-) didn't see you around | 16:35 |
rloo | lucasagomes: hi! no worries. i'm trying to be invisible :) | 16:36 |
lucasagomes | :D | 16:36 |
*** jpich has quit IRC | 16:37 | |
*** fxpester|2 has quit IRC | 16:40 | |
*** sabeen1 has quit IRC | 16:40 | |
TheJulia | good morning NobodyCam | 16:43 |
mgould | morning NobodyCam | 16:44 |
*** aNupoisc has joined #openstack-ironic | 16:44 | |
mgould | and hi to TheJulia jlvillal rloo jroll rajinir and anyone else who snuck in while I wasn't paying attention :-) | 16:45 |
jroll | hey mgould :) | 16:45 |
* mgould is currently rather distracted by last-minute prep for https://twitter.com/AndyMilesExped | 16:45 | |
sambetts | mgould: oooh, nice! | 16:47 |
mgould | :-) | 16:48 |
NobodyCam | morning mgould , TheJulia | 16:49 |
mgould | hurrah, you fixed the autocorrect problem! | 16:49 |
*** Romanenko_K has quit IRC | 16:49 | |
lucasagomes | I'm calling it a day | 16:50 |
lucasagomes | have a great evening all | 16:50 |
TheJulia | goodnight lucasagomes | 16:50 |
mgould | lucasagomes: good night! | 16:50 |
lucasagomes | TheJulia, oh morning | 16:50 |
*** lucasagomes is now known as lucas-dinner | 16:50 | |
TheJulia | :) | 16:50 |
NobodyCam | night lucas-dinner | 16:51 |
*** fragatina has joined #openstack-ironic | 16:51 | |
*** rbudden has quit IRC | 16:52 | |
*** fragatin_ has quit IRC | 16:54 | |
*** rama_y has joined #openstack-ironic | 16:56 | |
*** sdake_ has quit IRC | 16:57 | |
*** anush_ has quit IRC | 17:00 | |
*** vmud213 has joined #openstack-ironic | 17:02 | |
* sambetts -> home | 17:02 | |
*** rcernin has joined #openstack-ironic | 17:02 | |
sambetts | Have a great night everyone! | 17:02 |
*** sambetts is now known as sambetts|afk | 17:03 | |
*** wajdi has quit IRC | 17:03 | |
*** rcernin has quit IRC | 17:05 | |
*** vmud213 has quit IRC | 17:05 | |
*** rcernin has joined #openstack-ironic | 17:05 | |
NobodyCam | night sambetts|afk | 17:05 |
*** fragatina has quit IRC | 17:06 | |
*** fragatina has joined #openstack-ironic | 17:07 | |
*** vmud213 has joined #openstack-ironic | 17:07 | |
jlvillal | mgould: :) | 17:07 |
vdrok | night lucas-dinner and sambetts|afk | 17:09 |
*** fragatina has quit IRC | 17:12 | |
*** ksumit has joined #openstack-ironic | 17:12 | |
*** rbrud has joined #openstack-ironic | 17:12 | |
*** baoli has quit IRC | 17:13 | |
*** e0ne has joined #openstack-ironic | 17:13 | |
*** baoli has joined #openstack-ironic | 17:13 | |
*** rbrud has quit IRC | 17:14 | |
*** rbrud has joined #openstack-ironic | 17:14 | |
*** rbrud has quit IRC | 17:15 | |
*** rbrud has joined #openstack-ironic | 17:15 | |
* mgould -> home; good night! | 17:17 | |
*** mgould is now known as mgould|afk | 17:17 | |
*** PollyZ has quit IRC | 17:18 | |
*** abehl has joined #openstack-ironic | 17:20 | |
vdrok | good night mgould|afk | 17:21 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Source openrc on subnode in multinode topology https://review.openstack.org/353302 | 17:25 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Skip db configuration on subnodes https://review.openstack.org/353303 | 17:25 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 17:25 |
*** fragatina has joined #openstack-ironic | 17:27 | |
*** electrofelix has quit IRC | 17:28 | |
*** Goneri has quit IRC | 17:28 | |
jlvillal | vsaienk0: I just learned about permalinks yesterday :) | 17:29 |
*** sdake has joined #openstack-ironic | 17:29 | |
*** catintheroof has quit IRC | 17:30 | |
*** ijw has quit IRC | 17:36 | |
*** ijw has joined #openstack-ironic | 17:37 | |
*** abehl has quit IRC | 17:41 | |
*** ijw has quit IRC | 17:41 | |
openstackgerrit | Merged openstack/ironic: Update proliantutils version required for Newton release https://review.openstack.org/353301 | 17:42 |
openstackgerrit | Akilan Pughazhendi proposed openstack/ironic: Doc: Recommend users to update their systems https://review.openstack.org/354136 | 17:42 |
openstackgerrit | Merged openstack/ironic: IPMITool: Check the boot mode when setting the boot device https://review.openstack.org/352820 | 17:50 |
*** ijw has joined #openstack-ironic | 17:51 | |
*** rbrud has quit IRC | 17:52 | |
*** ijw has quit IRC | 17:52 | |
*** ijw has joined #openstack-ironic | 17:52 | |
*** rbudden has joined #openstack-ironic | 17:54 | |
*** ijw has quit IRC | 17:54 | |
*** ijw has joined #openstack-ironic | 17:55 | |
*** fragatina has quit IRC | 17:57 | |
*** ohamada has quit IRC | 17:57 | |
*** PollyZ has joined #openstack-ironic | 17:57 | |
*** ametts has joined #openstack-ironic | 17:57 | |
*** alineb has quit IRC | 18:00 | |
openstackgerrit | Merged openstack/ironic: test_console_utils: using mock_open for builtin open() https://review.openstack.org/352756 | 18:02 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Remove unused [pxe]disk_devices option https://review.openstack.org/354308 | 18:13 |
vdrok | rloo: btw, just noticed that ^^ | 18:14 |
openstackgerrit | Stephanie Miller proposed openstack/ironic: [WIP] Add storage_interface DB field and object https://review.openstack.org/348005 | 18:14 |
jroll | vdrok: needs a sample config generation :) | 18:15 |
*** vmud213 has quit IRC | 18:15 | |
*** amotoki has quit IRC | 18:15 | |
vdrok | jroll: ooo, right :D | 18:15 |
rloo | vdrok: thx for noticing :) | 18:15 |
*** ijw has quit IRC | 18:16 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Remove unused [pxe]disk_devices option https://review.openstack.org/354308 | 18:17 |
vdrok | done! | 18:17 |
vdrok | rloo: this was used at some point, but I don't know what to write in the release note | 18:18 |
vdrok | rloo: as we've had a couple of releases since it became unused | 18:18 |
rloo | vdrok: would be a good idea to spend a few minutes to find out what happened. in a meeting right now, can't look. did we mention in some release, that it was deprecated? | 18:19 |
vdrok | or you mean to change the release note with the ramdisk deprecation? | 18:19 |
jroll | rloo: it was part of dropping the bash ramdisk, see the commit message :) | 18:19 |
* jroll thinks it's fine without but welcomes disagreement | 18:19 | |
vdrok | lemme find the old ramdisk deprecation note | 18:20 |
*** e0ne has quit IRC | 18:20 | |
rloo | jroll, vdrok: so probably ok w/o relnote | 18:20 |
*** amotoki has joined #openstack-ironic | 18:22 | |
*** bvandewa has joined #openstack-ironic | 18:26 | |
vdrok | rloo: I looked at the commit, we just forgot about that option. but as the old ramdisk was deprecated for a long time and this option was never used in ipa ramdisk, it might be ok without a separate note about for this option | 18:26 |
rloo | vdrok: should be fine w/o relno then. thx for looking. | 18:27 |
vdrok | np :) | 18:27 |
*** ChubYann has joined #openstack-ironic | 18:33 | |
*** catintheroof has joined #openstack-ironic | 18:34 | |
*** fragatina has joined #openstack-ironic | 18:35 | |
*** chlong|mtg has quit IRC | 18:37 | |
vdrok | good night everyone! | 18:38 |
*** chlong|mtg has joined #openstack-ironic | 18:38 | |
*** amoralej is now known as amoralej|off | 18:47 | |
devananda | two more patches for keystone policy support are ready to be landed | 18:49 |
devananda | https://review.openstack.org/326768 and https://review.openstack.org/345556 | 18:49 |
*** aarefiev_ has joined #openstack-ironic | 18:50 | |
*** ijw has joined #openstack-ironic | 18:55 | |
*** fragatin_ has joined #openstack-ironic | 19:00 | |
*** fragatina has quit IRC | 19:03 | |
*** athomas has quit IRC | 19:07 | |
*** dprince_ has quit IRC | 19:08 | |
*** sabeen has joined #openstack-ironic | 19:09 | |
*** sabeen has quit IRC | 19:14 | |
*** sabeen has joined #openstack-ironic | 19:16 | |
*** aarefiev_ has quit IRC | 19:20 | |
openstackgerrit | Michael Turek proposed openstack/ironic: Add classes for Portgroups API enablement https://review.openstack.org/347549 | 19:21 |
*** Sukhdev has joined #openstack-ironic | 19:22 | |
*** bvandewa has quit IRC | 19:22 | |
rloo | devananda: just reviewed 326768. I could be wrong... | 19:27 |
*** catintheroof has quit IRC | 19:28 | |
*** ijw has quit IRC | 19:41 | |
openstackgerrit | Merged openstack/ironic: IPMINative: Check the boot mode when setting the boot device https://review.openstack.org/354153 | 19:43 |
*** PollyZ has quit IRC | 19:43 | |
*** PollyZ has joined #openstack-ironic | 19:43 | |
openstackgerrit | Merged openstack/ironic: Rename some variables in test_ipminative.py https://review.openstack.org/353914 | 19:46 |
rajinir | sambetts,mat128: tftp is running , tested it | 19:48 |
rajinir | sambetts,mat128: I can get the a file from /opt/stack/data/ironic/tftpboot folder from another machine. | 19:49 |
rajinir | sambetts,mat128: so concluding tftp is running and setup on the slave vm. May be images are missing | 19:50 |
*** e0ne has joined #openstack-ironic | 19:55 | |
rajinir | sambetts: the pxelinux.o is missing from the /opt/stack/data/ironic/tftpboot | 19:55 |
rajinir | sambetts: could it be the issue? | 19:55 |
*** catintheroof has joined #openstack-ironic | 20:00 | |
mat128 | sambetts is gone for the day and I'm about to leave | 20:03 |
mat128 | hmm | 20:03 |
mat128 | rajinir: ^ | 20:03 |
mat128 | so are ytou able to dump network traffic on the VM tap? | 20:03 |
mat128 | is tftp responding with "file not found" or simply not responding anything | 20:04 |
rajinir | mat128: I haven't done that, probably now | 20:04 |
*** e0ne has quit IRC | 20:08 | |
*** ijw has joined #openstack-ironic | 20:09 | |
*** ijw has quit IRC | 20:09 | |
*** ijw has joined #openstack-ironic | 20:10 | |
*** fragatin_ has quit IRC | 20:15 | |
*** fragatina has joined #openstack-ironic | 20:16 | |
*** amotoki has quit IRC | 20:20 | |
rajinir | mat128: Might have found the problem. The slave vm has two IPs, the tftp request is going to one of the IP and the tap is setup on the other. Not sure how to fix this | 20:20 |
*** ijw has quit IRC | 20:21 | |
*** Goneri has joined #openstack-ironic | 20:22 | |
*** ijw has joined #openstack-ironic | 20:22 | |
*** e0ne has joined #openstack-ironic | 20:23 | |
*** ijw has quit IRC | 20:23 | |
*** ijw has joined #openstack-ironic | 20:23 | |
mat128 | hmm | 20:25 |
mat128 | rajinir: I remember a devstack option for "which interface should we setup the tap on?" | 20:25 |
mat128 | vaguely | 20:25 |
mat128 | Look into the devstack plugin in the ironic tree | 20:26 |
rajinir | mat128:ok | 20:27 |
rajinir | mat128: thanks | 20:27 |
mat128 | I'll have to go, but let us know if you get something going | 20:27 |
mat128 | we'll all be here tomorrow :) | 20:27 |
rajinir | mat128: I think I have specified the right interface for the tap, its the forwarding from the host machine. Sure, good night | 20:29 |
rajinir | mat128: will update you tommorow, thanks | 20:29 |
*** hamzy has quit IRC | 20:30 | |
*** aNupoisc has quit IRC | 20:30 | |
*** e0ne has quit IRC | 20:32 | |
*** [1]cdearborn has quit IRC | 20:36 | |
*** ametts has quit IRC | 20:45 | |
*** sdake has quit IRC | 20:48 | |
*** PollyZ has quit IRC | 20:50 | |
*** ametts has joined #openstack-ironic | 20:51 | |
*** sabeen has quit IRC | 20:52 | |
*** ametts has quit IRC | 20:56 | |
*** bvandewa has joined #openstack-ironic | 20:59 | |
jroll | out for the night after a long day of meetings | 20:59 |
jroll | I'm also out tomorrow | 20:59 |
jroll | feel free to ping me on hangouts if I'm needed though, I won't be far | 21:00 |
jroll | have a good weekend everyone :D | 21:00 |
TheJulia | you too jroll :) | 21:01 |
*** e0ne has joined #openstack-ironic | 21:02 | |
*** e0ne has quit IRC | 21:05 | |
*** causten has quit IRC | 21:08 | |
*** causten has joined #openstack-ironic | 21:10 | |
*** aNupoisc has joined #openstack-ironic | 21:11 | |
mrda | Morning Ironic | 21:13 |
rloo | morning mrda | 21:15 |
*** sdake has joined #openstack-ironic | 21:16 | |
mrda | o/ | 21:17 |
*** PollyZ has joined #openstack-ironic | 21:19 | |
*** baoli has quit IRC | 21:25 | |
*** michauds has quit IRC | 21:25 | |
*** thiagop has quit IRC | 21:28 | |
*** Goneri has quit IRC | 21:28 | |
*** causten has quit IRC | 21:32 | |
*** sdake has quit IRC | 21:33 | |
*** ijw has quit IRC | 21:34 | |
*** causten has joined #openstack-ironic | 21:38 | |
devananda | rloo: thanks for the review! I'm reading .. and updating now | 21:40 |
*** ijw has joined #openstack-ironic | 21:40 | |
rloo | devananda: i'm around for another 20 min or so, ping me if you want me to re-review. | 21:40 |
*** ijw has joined #openstack-ironic | 21:40 | |
*** Goneri has joined #openstack-ironic | 21:43 | |
*** ksumit has quit IRC | 21:45 | |
*** hjanssen-hpe has quit IRC | 21:45 | |
*** hpe-hj has quit IRC | 21:45 | |
openstackgerrit | Ruby Loo proposed openstack/ironic: Update renos for fix to ipmi's set-boot-device https://review.openstack.org/354418 | 21:48 |
*** PollyZ has quit IRC | 21:49 | |
*** mlupton has joined #openstack-ironic | 22:00 | |
mlupton | Hey everyone, I'm having some networking trouble with an instance I've spawned on my overcloud. | 22:01 |
mlupton | I created a security group that allows ssh and icmp, but I still can't ping the instance, even if I'm pinging it from the compute node that it sits on. | 22:01 |
*** causten has quit IRC | 22:02 | |
*** PollyZ has joined #openstack-ironic | 22:03 | |
*** rama_y has quit IRC | 22:10 | |
*** rama_y has joined #openstack-ironic | 22:13 | |
*** mlupton has quit IRC | 22:15 | |
*** bvandewa has quit IRC | 22:20 | |
*** Goneri has quit IRC | 22:21 | |
*** bvandewa has joined #openstack-ironic | 22:26 | |
*** bvandewa has quit IRC | 22:27 | |
*** bvandewa has joined #openstack-ironic | 22:27 | |
openstackgerrit | Polly Zhou proposed openstack/ironic: Use the new microversion-parse library to parse the version headers https://review.openstack.org/354432 | 22:33 |
*** bvandewa_ has joined #openstack-ironic | 22:34 | |
*** bvandewa has quit IRC | 22:34 | |
*** catintheroof has quit IRC | 22:35 | |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Update documentation for keystone policy support https://review.openstack.org/345556 | 22:40 |
devananda | rloo: took me an hour to update it ... E_DISTRACTIONS | 22:41 |
devananda | all good points. thanks again! | 22:41 |
*** rbudden has quit IRC | 22:50 | |
*** PollyZ has quit IRC | 22:51 | |
*** hoangcx2 has joined #openstack-ironic | 23:07 | |
*** hoangcx has quit IRC | 23:08 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [pxe] https://review.openstack.org/309612 | 23:09 |
*** sabeen has joined #openstack-ironic | 23:09 | |
openstackgerrit | jxiaobin proposed openstack/ironic-python-agent: Deactivate LVM before write image to disk https://review.openstack.org/354440 | 23:12 |
*** ijw has quit IRC | 23:13 | |
*** Sukhdev has quit IRC | 23:13 | |
*** thrash is now known as thrash|g0ne | 23:15 | |
openstackgerrit | jxiaobin proposed openstack/ironic-python-agent: Deactivate LVM before write image to disk https://review.openstack.org/354440 | 23:24 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [iscsi] https://review.openstack.org/309615 | 23:25 |
openstackgerrit | jxiaobin proposed openstack/ironic-python-agent: Deactivate LVM before write image to disk https://review.openstack.org/354440 | 23:26 |
openstackgerrit | jxiaobin proposed openstack/ironic-python-agent: Deactivate LVM before write image to disk https://review.openstack.org/354440 | 23:28 |
*** testing1234 has joined #openstack-ironic | 23:29 | |
*** ijw has joined #openstack-ironic | 23:31 | |
*** ijw has quit IRC | 23:35 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize Config Options - patch merge, cleanup https://review.openstack.org/330160 | 23:40 |
*** ijw has joined #openstack-ironic | 23:43 | |
*** ijw has quit IRC | 23:45 | |
*** ijw has joined #openstack-ironic | 23:45 | |
*** rama_y has quit IRC | 23:47 | |
*** jvgrant_ has joined #openstack-ironic | 23:57 | |
*** rloo has quit IRC | 23:58 | |
*** rloo has joined #openstack-ironic | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!