TheJulia | devananda: https://review.openstack.org/#/c/319703/ | 00:00 |
---|---|---|
* TheJulia had it in another window from cinerama pinging me :) | 00:00 | |
* devananda goes out for drinks | 00:02 | |
cinerama | heh that's a great idea | 00:03 |
*** mtanino has quit IRC | 00:03 | |
cinerama | devananda, thanks for proposing that (and enjoy drinks!) | 00:03 |
*** ijw has joined #openstack-ironic | 00:04 | |
openstackgerrit | Chris Krelle proposed openstack/bifrost: WIP: initial work to add ISC-DHCP https://review.openstack.org/343947 | 00:04 |
*** shobhar__ has joined #openstack-ironic | 00:07 | |
*** marulkan has quit IRC | 00:15 | |
*** mjturek1 has joined #openstack-ironic | 00:16 | |
*** mjturek1 has quit IRC | 00:20 | |
openstackgerrit | Stephanie Miller proposed openstack/ironic: Add volume_connector table to save connector information https://review.openstack.org/200983 | 00:27 |
*** jaybeale has joined #openstack-ironic | 00:33 | |
openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce VolumeConnector object https://review.openstack.org/214584 | 00:34 |
*** jerrygb has joined #openstack-ironic | 00:36 | |
*** mbound has joined #openstack-ironic | 00:38 | |
*** jerrygb has quit IRC | 00:41 | |
*** Nisha_away has quit IRC | 00:42 | |
*** hoangcx has joined #openstack-ironic | 00:42 | |
*** mbound has quit IRC | 00:43 | |
*** harshs has quit IRC | 00:47 | |
*** ijw has quit IRC | 00:57 | |
*** ijw has joined #openstack-ironic | 00:58 | |
*** rama_y has quit IRC | 01:01 | |
*** ijw has quit IRC | 01:03 | |
*** Guest49585 has quit IRC | 01:03 | |
*** wajdi has joined #openstack-ironic | 01:03 | |
openstackgerrit | Tuan proposed openstack/ironic: Fix invalid import template https://review.openstack.org/344527 | 01:05 |
*** yibo has joined #openstack-ironic | 01:08 | |
openstackgerrit | Tuan proposed openstack/ironic: Fix invalid import template https://review.openstack.org/344527 | 01:08 |
*** rook_ has joined #openstack-ironic | 01:10 | |
*** mtanino has joined #openstack-ironic | 01:17 | |
*** jaybeale has quit IRC | 01:19 | |
*** shobhar__ has quit IRC | 01:23 | |
*** Sukhdev has quit IRC | 01:30 | |
*** Nisha_away has joined #openstack-ironic | 01:30 | |
*** harlowja has joined #openstack-ironic | 01:35 | |
*** harlowja has quit IRC | 01:35 | |
*** harlowja has joined #openstack-ironic | 01:36 | |
*** jerrygb has joined #openstack-ironic | 01:37 | |
*** jerrygb has quit IRC | 01:42 | |
*** ChrisAusten has joined #openstack-ironic | 01:44 | |
*** jjohnson2 has joined #openstack-ironic | 01:53 | |
*** Nisha_away has quit IRC | 01:56 | |
*** marulkan has joined #openstack-ironic | 02:04 | |
*** praneshp has joined #openstack-ironic | 02:10 | |
*** wrouesnel has joined #openstack-ironic | 02:13 | |
*** priteau has joined #openstack-ironic | 02:13 | |
*** rajinir has quit IRC | 02:15 | |
*** priteau has quit IRC | 02:18 | |
*** jerrygb has joined #openstack-ironic | 02:21 | |
*** praneshp has quit IRC | 02:21 | |
*** aNupoisc has quit IRC | 02:27 | |
*** hoonetorg has quit IRC | 02:40 | |
*** praneshp has joined #openstack-ironic | 02:42 | |
*** Nisha_away has joined #openstack-ironic | 02:45 | |
*** harlowja has quit IRC | 02:46 | |
*** yuanying has quit IRC | 02:46 | |
*** hoonetorg has joined #openstack-ironic | 02:53 | |
*** praneshp has quit IRC | 02:55 | |
*** praneshp has joined #openstack-ironic | 02:56 | |
*** praneshp has quit IRC | 03:00 | |
*** livelace has joined #openstack-ironic | 03:07 | |
*** amotoki has quit IRC | 03:08 | |
*** wrouesnel has quit IRC | 03:13 | |
*** livelace has quit IRC | 03:16 | |
*** amotoki has joined #openstack-ironic | 03:22 | |
*** gabriel-bezerra has quit IRC | 03:27 | |
*** xavierr has quit IRC | 03:27 | |
*** liliars has quit IRC | 03:27 | |
*** caiobo-afk has quit IRC | 03:27 | |
*** harshs has joined #openstack-ironic | 03:27 | |
*** liliars has joined #openstack-ironic | 03:28 | |
*** nicodemos has quit IRC | 03:28 | |
*** clenimar has quit IRC | 03:28 | |
*** xavierr has joined #openstack-ironic | 03:29 | |
*** gabriel-bezerra has joined #openstack-ironic | 03:30 | |
*** caiobo-afk has joined #openstack-ironic | 03:30 | |
*** nicodemos has joined #openstack-ironic | 03:30 | |
*** amotoki has quit IRC | 03:30 | |
openstackgerrit | Debayan Ray proposed openstack/proliantutils: Fix to accommodate very quick firmware update https://review.openstack.org/337020 | 03:30 |
*** clenimar has joined #openstack-ironic | 03:32 | |
*** jjohnson2 has quit IRC | 03:35 | |
*** amotoki has joined #openstack-ironic | 03:35 | |
*** ijw has joined #openstack-ironic | 03:37 | |
*** ijw has quit IRC | 03:41 | |
*** ijw has joined #openstack-ironic | 03:46 | |
*** yuanying has joined #openstack-ironic | 03:48 | |
*** ijw_ has joined #openstack-ironic | 03:48 | |
*** ijw has quit IRC | 03:52 | |
*** harshs has quit IRC | 03:57 | |
*** yuanying_ has joined #openstack-ironic | 04:03 | |
*** nicodemos has quit IRC | 04:04 | |
*** liliars has quit IRC | 04:04 | |
*** gabriel-bezerra has quit IRC | 04:04 | |
*** caiobo-afk has quit IRC | 04:04 | |
*** clenimar has quit IRC | 04:04 | |
*** xavierr has quit IRC | 04:05 | |
*** yuanying has quit IRC | 04:06 | |
*** xavierr has joined #openstack-ironic | 04:09 | |
*** gabriel-bezerra has joined #openstack-ironic | 04:09 | |
*** nicodemos has joined #openstack-ironic | 04:09 | |
*** liliars has joined #openstack-ironic | 04:09 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements https://review.openstack.org/345151 | 04:09 |
*** clenimar has joined #openstack-ironic | 04:10 | |
*** caiobo-afk has joined #openstack-ironic | 04:10 | |
*** amotoki has quit IRC | 04:18 | |
openstackgerrit | Yuiko Takada Mori proposed openstack/ironic: IPMITool: add IPMISocatConsole and IPMIConsole class https://review.openstack.org/293873 | 04:23 |
*** poll0172 has joined #openstack-ironic | 04:26 | |
*** poll0172 has quit IRC | 04:33 | |
*** poll0172_ has joined #openstack-ironic | 04:36 | |
*** shobhar_ has joined #openstack-ironic | 04:38 | |
*** vishwanathj is now known as vishwanathj_zzz | 04:39 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add boot management support in inspector https://review.openstack.org/345161 | 04:45 |
*** Sukhdev has joined #openstack-ironic | 04:47 | |
*** d0ugal has quit IRC | 04:56 | |
*** d0ugal has joined #openstack-ironic | 04:57 | |
*** yuanying has joined #openstack-ironic | 05:03 | |
*** michauds has joined #openstack-ironic | 05:05 | |
*** yuanying_ has quit IRC | 05:06 | |
*** jerrygb has quit IRC | 05:09 | |
*** michauds has quit IRC | 05:10 | |
*** ChrisAusten has quit IRC | 05:24 | |
*** ChrisAusten has joined #openstack-ironic | 05:25 | |
*** shobhar_ has quit IRC | 05:27 | |
*** harlowja has joined #openstack-ironic | 05:39 | |
*** bks has joined #openstack-ironic | 05:45 | |
*** poll0172_ has quit IRC | 05:45 | |
*** ChubYann has quit IRC | 05:47 | |
*** Sukhdev has quit IRC | 05:52 | |
*** moshele has joined #openstack-ironic | 05:55 | |
*** ijw_ has quit IRC | 06:03 | |
*** harlowja has quit IRC | 06:04 | |
*** dtantsur|afk is now known as dtantsur | 06:15 | |
*** ccamacho has joined #openstack-ironic | 06:15 | |
dtantsur | Morning Ironic! | 06:17 |
*** edand has joined #openstack-ironic | 06:23 | |
*** _vsaienko has joined #openstack-ironic | 06:24 | |
*** _vs_ has joined #openstack-ironic | 06:24 | |
*** mjura has joined #openstack-ironic | 06:30 | |
*** vsaienk0 has quit IRC | 06:37 | |
*** mjura has quit IRC | 06:38 | |
*** mjura has joined #openstack-ironic | 06:38 | |
*** tesseract- has joined #openstack-ironic | 06:39 | |
*** ijw has joined #openstack-ironic | 06:44 | |
*** ccamacho is now known as ccamacho|afk | 06:44 | |
*** ijw_ has joined #openstack-ironic | 06:45 | |
*** edand has quit IRC | 06:46 | |
*** edand has joined #openstack-ironic | 06:47 | |
*** ijw has quit IRC | 06:49 | |
*** vsaienk0 has joined #openstack-ironic | 06:52 | |
*** cfarquhar has quit IRC | 06:55 | |
*** ohamada has joined #openstack-ironic | 06:57 | |
*** hoangcx_ has joined #openstack-ironic | 06:58 | |
*** cfarquhar has joined #openstack-ironic | 06:58 | |
*** cfarquhar has quit IRC | 06:58 | |
*** cfarquhar has joined #openstack-ironic | 06:58 | |
*** hoangcx has quit IRC | 06:59 | |
*** loki_ has joined #openstack-ironic | 07:03 | |
*** cfarquhar has quit IRC | 07:03 | |
*** pcaruana has joined #openstack-ironic | 07:03 | |
*** athomas has joined #openstack-ironic | 07:05 | |
openstackgerrit | Swapnil Kulkarni (coolsvap) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 07:07 |
*** rcernin has joined #openstack-ironic | 07:07 | |
*** rcernin has quit IRC | 07:10 | |
*** rcernin has joined #openstack-ironic | 07:10 | |
*** ChrisAusten has quit IRC | 07:13 | |
*** ifarkas_afk is now known as ifarkas | 07:17 | |
*** cfarquhar has joined #openstack-ironic | 07:20 | |
*** cfarquhar has quit IRC | 07:20 | |
*** cfarquhar has joined #openstack-ironic | 07:20 | |
openstackgerrit | Swapnil Kulkarni (coolsvap) proposed openstack/bifrost: Remove discover from test-requirements https://review.openstack.org/345203 | 07:28 |
*** d0ugal has quit IRC | 07:37 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Remove duplicate copyright. https://review.openstack.org/345213 | 07:40 |
*** jpich has joined #openstack-ironic | 07:47 | |
*** rbartal has joined #openstack-ironic | 07:53 | |
*** moshele has quit IRC | 07:55 | |
*** moshele has joined #openstack-ironic | 07:55 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-ironic | 08:00 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Fix iLO drivers inconsistent boot mode default value https://review.openstack.org/345231 | 08:04 |
*** dtantsur is now known as dtantsur|bbl | 08:05 | |
openstackgerrit | Tuan proposed openstack/ironic: Fix invalid import template https://review.openstack.org/344527 | 08:06 |
*** stendulker has joined #openstack-ironic | 08:06 | |
*** ccamacho|afk is now known as ccamacho | 08:08 | |
*** ohamada has quit IRC | 08:08 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 08:10 |
*** ijw_ has quit IRC | 08:14 | |
*** ijw has joined #openstack-ironic | 08:15 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Fix iLO drivers inconsistent boot mode default value https://review.openstack.org/345231 | 08:16 |
*** derekh has joined #openstack-ironic | 08:18 | |
*** ijw has quit IRC | 08:20 | |
*** ohamada has joined #openstack-ironic | 08:22 | |
lucasagomes | morning all | 08:22 |
*** kromanenko_ has joined #openstack-ironic | 08:22 | |
*** priteau has joined #openstack-ironic | 08:23 | |
*** priteau has quit IRC | 08:28 | |
Nisha_away | morning all | 08:29 |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Catching error working with database https://review.openstack.org/345248 | 08:33 |
*** devananda has quit IRC | 08:34 | |
openstackgerrit | Tuan proposed openstack/ironic: Fix invalid import template https://review.openstack.org/344527 | 08:35 |
*** hoangcx_ has quit IRC | 08:39 | |
ifarkas | morning all | 08:42 |
*** hoangcx has joined #openstack-ironic | 08:43 | |
*** amotoki has joined #openstack-ironic | 08:47 | |
*** bvandewa has quit IRC | 08:48 | |
aarefiev | morning! | 08:52 |
aarefiev | ifarkas: morning, please take a look tempest fix https://review.openstack.org/#/c/344363/ | 08:53 |
ifarkas | aarefiev, morning, will look! | 08:54 |
*** bvandewa has joined #openstack-ironic | 08:54 | |
aarefiev | thanks | 08:54 |
*** priteau has joined #openstack-ironic | 08:57 | |
openstackgerrit | Saravanan KR proposed openstack/ironic-python-agent: Set the kernel args to the nodes at bootloader install https://review.openstack.org/335052 | 08:57 |
*** priteau has quit IRC | 08:58 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Remove unused import https://review.openstack.org/344527 | 09:03 |
* lucasagomes just updated the commit message ^ | 09:05 | |
*** _vs_ has quit IRC | 09:06 | |
*** _vsaienko has quit IRC | 09:06 | |
ifarkas | dtantsur|bbl, aarefiev, any reason this patch is not approved while having 2x+2s? https://review.openstack.org/#/c/338116/12 | 09:06 |
aarefiev | ifarkas: it's depended on non merged on | 09:07 |
aarefiev | one | 09:07 |
ifarkas | oh right | 09:07 |
*** e0ne has joined #openstack-ironic | 09:08 | |
*** d0ugal has joined #openstack-ironic | 09:10 | |
*** uggla_ has joined #openstack-ironic | 09:12 | |
*** jerrygb has joined #openstack-ironic | 09:13 | |
openstackgerrit | Merged openstack/proliantutils: Fix to accommodate very quick firmware update https://review.openstack.org/337020 | 09:13 |
*** strigazi has joined #openstack-ironic | 09:14 | |
*** electrofelix has joined #openstack-ironic | 09:15 | |
strigazi | Hi all, I'm trying to create an instance on fresh devstack deployment as described in http://docs.openstack.org/developer/ironic/dev/dev-quickstart.html#deploying-ironic-with-devstack | 09:16 |
strigazi | but I keep getting this error on ir-api | 09:17 |
strigazi | http://paste.openstack.org/show/538949/ | 09:17 |
*** jerrygb has quit IRC | 09:18 | |
*** ijw has joined #openstack-ironic | 09:19 | |
strigazi | The nova or ironic try to deploy the instance on another ironic node causing, after some time, all nodes to be in clean failed state | 09:19 |
strigazi | cam anyone help? | 09:20 |
*** d0ugal has quit IRC | 09:22 | |
*** e0ne has quit IRC | 09:23 | |
*** mgould|afk is now known as mgould | 09:23 | |
*** e0ne has joined #openstack-ironic | 09:24 | |
mgould | strigazi: that sounds familiar, one moment... | 09:24 |
mgould | strigazi: what's the output of ironic node-show [node uuid goes here]? | 09:26 |
vdrok | good morning all! | 09:27 |
mgould | also, morning aarefiev ifarkas lucasagomes strigazi Nisha_away vdrok and everyone else! | 09:27 |
lucasagomes | hi there | 09:28 |
aarefiev | morning | 09:28 |
ifarkas | morning mgould | 09:28 |
*** d0ugal has joined #openstack-ironic | 09:28 | |
stendulker | good morning all | 09:29 |
mgould | morning stendulker | 09:29 |
openstackgerrit | Merged openstack/ironic-inspector: Tempest: wrap instance actions into inspector methods https://review.openstack.org/344363 | 09:30 |
strigazi | mgould" http://paste.openstack.org/show/538951/ | 09:30 |
openstackgerrit | Merged openstack/ironic-inspector: Remove redundant white space https://review.openstack.org/344899 | 09:32 |
mgould | strigazi: OK, I haven't seen that error before | 09:32 |
mgould | can you paste your local.conf file? | 09:32 |
strigazi | it's what is on dev-quicksatrt http://paste.openstack.org/show/538952/ | 09:34 |
openstackgerrit | Merged openstack/ironic: Remove unused import https://review.openstack.org/344527 | 09:34 |
mgould | strigazi: OK, thanks, I'll try here and see what happens | 09:34 |
strigazi | mgould: thanks | 09:35 |
lucasagomes | strigazi, it seems that ironic-conductor can't talk to the node booting with IPA | 09:35 |
mgould | you're running devstack in a VM, right? | 09:35 |
openstackgerrit | Merged openstack/ironic-inspector: Add config to skip power off after introspection https://review.openstack.org/343538 | 09:35 |
lucasagomes | which is strange, firewall maybe? | 09:35 |
strigazi | no, physical host with nothing else on it | 09:35 |
openstackgerrit | Merged openstack/ironic-inspector: [doc]Add 'ipa-debug=1' to installation document https://review.openstack.org/343438 | 09:35 |
*** daemontool has joined #openstack-ironic | 09:39 | |
strigazi | These are my iptables rules http://paste.openstack.org/show/538954/ | 09:43 |
*** daemontool_ has joined #openstack-ironic | 09:44 | |
strigazi | Do you see enything fishy? | 09:44 |
*** daemontool has quit IRC | 09:47 | |
mgould | nothing leaps out at me | 09:48 |
* mgould runs stack.sh | 09:52 | |
*** edand has quit IRC | 09:52 | |
*** d0ugal has quit IRC | 09:56 | |
*** d0ugal has joined #openstack-ironic | 10:02 | |
*** d0ugal has quit IRC | 10:03 | |
strigazi | I'm building an environment on a VM too. I'll be back later, going for lunch | 10:03 |
*** d0ugal has joined #openstack-ironic | 10:03 | |
*** yibo has quit IRC | 10:04 | |
*** amotoki has quit IRC | 10:07 | |
*** e0ne has quit IRC | 10:07 | |
moshele | lucasagomes: morning can you please review the infiniband code https://review.openstack.org/#/c/264263/ | 10:10 |
lucasagomes | moshele, will do | 10:11 |
moshele | lucasagomes: thanks :) | 10:14 |
*** jerrygb has joined #openstack-ironic | 10:14 | |
*** amotoki has joined #openstack-ironic | 10:15 | |
*** amotoki has quit IRC | 10:16 | |
*** jerrygb has quit IRC | 10:20 | |
vsaienk0 | Morning lucasagomes, dtantsur, moshele, mgould, aarefiev, vdrok and all Ironic'ers | 10:20 |
lucasagomes | hi | 10:20 |
moshele | vsaienk0: good morning | 10:20 |
mgould | morning vsaienk0 | 10:21 |
*** hoangcx has quit IRC | 10:22 | |
*** amotoki has joined #openstack-ironic | 10:24 | |
*** mbound has joined #openstack-ironic | 10:26 | |
*** d0ugal has quit IRC | 10:31 | |
*** dtantsur|bbl is now known as dtantsur | 10:32 | |
dtantsur | Morning to everyone I've missed while studying English grammar :) | 10:32 |
dtantsur | including ifarkas, lucasagomes, aarefiev, mgould :) | 10:32 |
lucasagomes | hi | 10:33 |
aarefiev | morning | 10:33 |
stendulker | hi | 10:33 |
mgould | morning dtantsur | 10:35 |
*** e0ne has joined #openstack-ironic | 10:35 | |
mgould | what did you learn this morning? | 10:35 |
mgould | strigazi: devstack is failing for me at the "yum install" stage :-( | 10:36 |
*** jpich has quit IRC | 10:36 | |
dtantsur | mgould, we were practicing prepositions | 10:37 |
dtantsur | I'm pretty terrible in (or at?) them :( | 10:37 |
mgould | at | 10:37 |
dtantsur | you see, right? :) | 10:37 |
mgould | yep :-) | 10:37 |
mgould | prepositions are pretty random AIUI :-( | 10:38 |
dtantsur | yeah.. we're having articles when I come back, it's gonna be even more of type 2 fun :) | 10:39 |
* mgould wishes dtantsur luck | 10:39 | |
dtantsur | thanks! | 10:40 |
*** bvandewa has quit IRC | 10:44 | |
mgould | OK, does anyone know why I can't install liberasurecode-devel and openvswitch on my CentOS 7 devstack VM? | 10:45 |
*** jpich has joined #openstack-ironic | 10:45 | |
dtantsur | hmm, what's the problem? | 10:46 |
lucasagomes | mgould, error? | 10:46 |
mgould | No package liberasurecode-devel available. | 10:47 |
*** bvandewa has joined #openstack-ironic | 10:47 | |
mgould | also, I can't ping centos.serverspace.co.uk, which yum says it's using as a base | 10:47 |
dtantsur | mgould, also nobody is using centos with devstack, I suspect | 10:47 |
dtantsur | mgould, we generally use Fedora or Ubuntu | 10:47 |
dtantsur | I think Fedora 23 is ok now | 10:48 |
mgould | OK, I'll set up a Fedora 23 VM | 10:48 |
lucasagomes | mgould, :-( I see... yeah fedora is easier | 10:48 |
lucasagomes | but googling I found this too: https://n40lab.wordpress.com/2015/06/28/centos-7-installing-openvswitch-2-3-2-lts/ | 10:49 |
*** amotoki has quit IRC | 10:49 | |
* mgould can hit centos.serverspace.co.uk in the browser, so I guess they're dropping pings | 10:49 | |
mgould | Fedora Cloud, or Fedora Server? | 10:50 |
lucasagomes | mgould, I use workstation | 10:51 |
mgould | lucasagomes: thanks | 10:51 |
mgould | hah, slightly smaller than the Server download | 10:52 |
lucasagomes | yeah, both should work... it just about the pre-installed packages that each one will contain | 10:53 |
*** bvandewa has quit IRC | 10:55 | |
dtantsur | yeah, I tend to pick server | 10:56 |
*** bcornec1 has quit IRC | 11:05 | |
*** milan has joined #openstack-ironic | 11:06 | |
*** bcornec1 has joined #openstack-ironic | 11:09 | |
*** loki_ has quit IRC | 11:10 | |
ifarkas | me too, workstation includes desktop | 11:11 |
mgould | there seems little point in having X on my devstack machine | 11:13 |
mgould | milan: you were asking about celery last night, yes? | 11:13 |
openstackgerrit | Merged openstack/python-ironicclient: Fix Quick-start example syntax. https://review.openstack.org/340944 | 11:13 |
* mgould used it a bit at $lastjob | 11:13 | |
mgould | it's less ambitious than TaskFlow, AIUI | 11:14 |
mgould | you run multiple copies of your app on different machines | 11:14 |
mgould | and send "run this function with these arguments" messages over RabbitMQ or Redis or whatever | 11:14 |
mgould | celery basically hides the messaging, so it looks like you're calling an async Python function, which runs on another machine | 11:16 |
mgould | apetrich from Red Hat has contributed to it, he's a good person to ask | 11:16 |
*** jerrygb has joined #openstack-ironic | 11:16 | |
dtantsur | he's even in the openstack team | 11:16 |
dtantsur | (iirc) | 11:16 |
mgould | yep | 11:16 |
*** tangchen has quit IRC | 11:17 | |
*** stendulker has quit IRC | 11:17 | |
mgould | working on downstream CI, IIRC | 11:17 |
* mgould worked with him at $lastjob, where he was the local celery expert | 11:17 | |
milan | mgould, yeah, that would be precisely what I'd use for inspector | 11:17 |
mgould | milan: yep, sounds like a sensible choice | 11:18 |
* milan just wonders how many -1s that would bring | 11:18 | |
dtantsur | well, our problem is probably not technical | 11:18 |
milan | as celery isn't yet nowhere in global-dependencies | 11:18 |
mgould | ah | 11:18 |
mgould | yeah, that might cause friction | 11:19 |
mgould | how do other projects use RabbitMQ? | 11:19 |
dtantsur | mgould, oslo.messaging | 11:19 |
mgould | oh, ofc | 11:19 |
dtantsur | a more primitive thing | 11:19 |
milan | and everyone reinvents the RPC | 11:19 |
milan | :-/ | 11:19 |
mgould | no NIH here, nosiree :-) | 11:19 |
milan | *over oslo.messaging | 11:20 |
*** jerrygb has quit IRC | 11:21 | |
milan | mgould, yeah, unfortunately so, I'd say | 11:21 |
aarefiev | There was decision to make at least once queues in tooz | 11:27 |
aarefiev | messaging guys don't want touching existing rpc for this | 11:28 |
aarefiev | milan^ | 11:28 |
*** tangchen has joined #openstack-ironic | 11:30 | |
aarefiev | I've looked on this, I'll try to push wip patch with zookeeper backend tomorrow, before my PTO | 11:30 |
dtantsur | fantastic, thanks | 11:31 |
milan | aarefiev, thx, btw what is the back-end? | 11:33 |
aarefiev | zookeeper for now | 11:33 |
openstackgerrit | Aparna proposed openstack/ironic: [WIP]Support hardware disk erase as a cleaning step https://review.openstack.org/345356 | 11:33 |
aarefiev | I'm find it interesting etcd and consul in future | 11:34 |
openstackgerrit | Aparna proposed openstack/ironic: [WIP]Support hardware_disk_erase as a cleaning step https://review.openstack.org/345356 | 11:35 |
*** ohamada has quit IRC | 11:35 | |
aarefiev | finding | 11:35 |
*** ohamada has joined #openstack-ironic | 11:35 | |
milan | aarefiev, I know oslo.messaging guys didn't want to fix the semantics from up-to-once to at-least-once delivery option neither accepting any patches in that direction | 11:35 |
*** daemontool_ has quit IRC | 11:35 | |
*** ohamada has quit IRC | 11:35 | |
aarefiev | milan: yes, so tooz would good choice | 11:35 |
mgould | milan: I dimly remember this | 11:37 |
mgould | it seems strange to *reject* at-least-once semantics | 11:37 |
mgould | there are situations where you need that | 11:37 |
*** e0ne has quit IRC | 11:37 | |
milan | mgould, yup, not even to have a separate queue implementation that'd support it | 11:38 |
*** daemontool_ has joined #openstack-ironic | 11:38 | |
mgould | yeah | 11:40 |
mgould | "you can't have both at once" is just an unfortunate reality | 11:40 |
mgould | but nothing's stopping you having one or the other | 11:41 |
milan | Ideally one could chose between and select based on use case ;) | 11:41 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Do the VM setup only when requested https://review.openstack.org/345363 | 11:42 |
*** lucasagomes is now known as lucas-hungry | 11:42 | |
openstackgerrit | Aparna proposed openstack/proliantutils: Modify error message of InvalidInputExecption in hpssa https://review.openstack.org/338894 | 11:43 |
*** dprince has joined #openstack-ironic | 11:45 | |
strigazi | mgould: In a new environment I have the same issue | 11:45 |
strigazi | with ubuntu server trusty | 11:45 |
openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient: Do not merge https://review.openstack.org/345366 | 11:47 |
mgould | strigazi: :-( | 11:47 |
* mgould has been having trouble even getting to that stage :-( | 11:47 | |
milan | mgould, https://review.openstack.org/#/c/256342/ | 11:48 |
* mgould looks | 11:49 | |
strigazi | I'm building an environment on a VM too. I'll be back later, going for lunch | 11:49 |
mgould | strigazi: good luck, see you later | 11:49 |
milan | mgould, in rev7 it would still use oslo.messaging instead of tooz | 11:50 |
TheJulia | Good morning | 11:51 |
strigazi | mgould: ok, I saw once the same exception in ir-api but finally the instance made it | 11:52 |
strigazi | mgould: see you later | 11:52 |
mgould | TheJulia: morning! | 11:52 |
mgould | strigazi: hurrah! | 11:52 |
*** Nisha_away has quit IRC | 11:56 | |
*** daemontool__ has joined #openstack-ironic | 12:06 | |
*** _vsaienko has joined #openstack-ironic | 12:07 | |
*** _vs_ has joined #openstack-ironic | 12:07 | |
*** daemontool_ has quit IRC | 12:09 | |
*** Nisha_away has joined #openstack-ironic | 12:09 | |
*** _vsaienko has quit IRC | 12:09 | |
*** _vs_ has quit IRC | 12:09 | |
*** d0ugal has joined #openstack-ironic | 12:11 | |
*** jerrygb has joined #openstack-ironic | 12:17 | |
*** trown|outtypewww is now known as trown | 12:19 | |
*** jerrygb has quit IRC | 12:21 | |
*** jjohnson2 has joined #openstack-ironic | 12:26 | |
*** athomas has quit IRC | 12:27 | |
*** e0ne has joined #openstack-ironic | 12:33 | |
*** athomas has joined #openstack-ironic | 12:34 | |
*** _vs_ has joined #openstack-ironic | 12:35 | |
*** _vs_ has quit IRC | 12:39 | |
*** _vsaienko has joined #openstack-ironic | 12:39 | |
*** _vs_ has joined #openstack-ironic | 12:39 | |
*** jcoufal has joined #openstack-ironic | 12:43 | |
*** e0ne has quit IRC | 12:43 | |
*** _vs_ has quit IRC | 12:46 | |
*** _vsaienko has quit IRC | 12:46 | |
*** lucas-hungry is now known as lucasagomes | 12:49 | |
openstackgerrit | Julia Kreger proposed openstack/ironic: Bug fixes and doc updates for adoption https://review.openstack.org/344973 | 12:52 |
dtantsur | does anybody want to review https://review.openstack.org/#/c/327807/ please? | 12:56 |
dtantsur | I'm not approving because I've been involved in discussing it for some time before the patch got posted | 12:57 |
*** cdearborn has joined #openstack-ironic | 13:02 | |
* TheJulia looks | 13:09 | |
*** jpich has quit IRC | 13:12 | |
*** kirankv has quit IRC | 13:13 | |
TheJulia | my preference is the same as Jay's, but I also think this is something that needs to be in IPA by default | 13:13 |
TheJulia | although, it makes no sense to block progress | 13:14 |
*** kirankv has joined #openstack-ironic | 13:15 | |
*** rloo has joined #openstack-ironic | 13:15 | |
dtantsur | yeah, I think it may be of value to be able to optionally force a hardware manager | 13:16 |
dtantsur | and/or provide a list of enabled hw managers | 13:16 |
*** jerrygb has joined #openstack-ironic | 13:18 | |
TheJulia | I'm much more for the make a small change, iterate, move fast kind of person. I think the release note needs to be a little more specifc. Thoughts? | 13:20 |
TheJulia | I posted a +2, no issue with +A'ing and tossing up a follow-up for the releasenote. | 13:21 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Substrate fixes and changes for adoption testing https://review.openstack.org/344974 | 13:21 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Add tempest tests for node Adoption https://review.openstack.org/344975 | 13:21 |
*** jpich has joined #openstack-ironic | 13:22 | |
*** jerrygb has quit IRC | 13:22 | |
*** cdearborn has quit IRC | 13:23 | |
*** milan has quit IRC | 13:27 | |
*** milan has joined #openstack-ironic | 13:27 | |
*** _vs_ has joined #openstack-ironic | 13:28 | |
*** _vsaienko has joined #openstack-ironic | 13:29 | |
dtantsur | TheJulia, I like your release note idea FWIW. but maybe better post it as a follow-up for quicker turn-around | 13:29 |
*** thiagop has joined #openstack-ironic | 13:31 | |
*** e0ne has joined #openstack-ironic | 13:32 | |
thiagop | Good morning, Ironic | 13:33 |
*** milan has quit IRC | 13:34 | |
TheJulia | dtantsur: absolutely agree, thats why I said I would gladly follow-up on it | 13:34 |
*** jerrygb has joined #openstack-ironic | 13:34 | |
*** milan has joined #openstack-ironic | 13:35 | |
dtantsur | yes please. I have half an hour more here to +2 it :) | 13:35 |
lucasagomes | thiagop, TheJulia morning | 13:35 |
*** Nisha_away has quit IRC | 13:36 | |
*** Nisha_away has joined #openstack-ironic | 13:37 | |
*** harshs has joined #openstack-ironic | 13:37 | |
*** trown is now known as trown|mtg | 13:37 | |
thiagop | lucasagomes: o/ | 13:41 |
*** milan has quit IRC | 13:41 | |
*** milan has joined #openstack-ironic | 13:43 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Enhance ilo drivers to do inband-inspection https://review.openstack.org/343577 | 13:43 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add boot management support in inspector https://review.openstack.org/345161 | 13:43 |
*** _vs_ has quit IRC | 13:43 | |
*** _vsaienko has quit IRC | 13:44 | |
*** _vsaienko has joined #openstack-ironic | 13:47 | |
*** _vs_ has joined #openstack-ironic | 13:47 | |
TheJulia | dtantsur: 10-4 | 13:47 |
*** ametts has joined #openstack-ironic | 13:47 | |
*** Nisha_away has quit IRC | 13:47 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Catching error working with database https://review.openstack.org/345248 | 13:47 |
dtantsur | TheJulia, mmm? | 13:48 |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Catching error working with database https://review.openstack.org/345248 | 13:50 |
*** krotscheck_dcm is now known as krotscheck | 13:50 | |
openstackgerrit | Merged openstack/python-ironic-inspector-client: Updated from global requirements https://review.openstack.org/345151 | 13:50 |
TheJulia | dtantsur: oh, sorry, https://en.wikipedia.org/wiki/Ten-code | 13:52 |
dtantsur | oh, interesting! | 13:53 |
openstackgerrit | Ruby Loo proposed openstack/python-ironicclient: Remove discover from test-requirements https://review.openstack.org/345443 | 13:53 |
dtantsur | I haven't seen it before, thanks | 13:53 |
dtantsur | folks, I'm leaving for PTO (a bit earlier today, have to catch a bus). see you in August :) | 13:55 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent: Follow-up text changes for 327807 https://review.openstack.org/345445 | 13:55 |
TheJulia | dtantsur: ^^ | 13:56 |
TheJulia | Enjoy your PTO dtantsur | 13:56 |
* dtantsur looks real quick | 13:56 | |
milan | dtantsur, enjoy, bye! | 13:56 |
TheJulia | sorry! | 13:56 |
rloo | bye dtantsur, have a great PTO! | 13:56 |
rloo | hi and good morning everyone else :) | 13:56 |
TheJulia | rloo: good morning! | 13:56 |
*** livelace has joined #openstack-ironic | 13:56 | |
*** mtanino has quit IRC | 13:57 | |
*** dtantsur is now known as dtantsur|pto | 13:57 | |
milan | morning rloo! :) | 13:57 |
rloo | hi milan, TheJulia | 13:58 |
jroll | morning everyone | 13:58 |
mgould | morning rloo | 13:58 |
mgould | dtantsur|pto: enjoy PTO | 13:59 |
rloo | hi mgould. morning jroll | 13:59 |
mgould | drat it, missed him | 13:59 |
jroll | vsaienk0: are you around, can we talk about the nova patch for multitenancy and why it wouldn't be needed in the real world? | 13:59 |
milan | morning jroll! :) | 13:59 |
mgould | morning jroll | 13:59 |
*** bks has quit IRC | 13:59 | |
jroll | thiagop: can you tl;dr for me why some of these URIs are in capabilities? I can't remember what exactly you're matching against in nova, etc | 13:59 |
jroll | morning rloo, milan, mgould | 13:59 |
vsaienk0 | hey jroll | 14:00 |
jroll | rloo: let me put this on your radar: https://review.openstack.org/#/c/345040/ https://review.openstack.org/#/q/status:open+branch:master+topic:bug/1604916 | 14:01 |
thiagop | jroll: Morning. The td;dr is that we can put enclosure_groups and server_hardware_types on flavors for cases that the user wants to deploy it's baremetal in an specific location/platform | 14:01 |
jroll | rloo: the code needs a bit of work, but we have a plan (will send an email detailing it) and need to get this done asap so that we can get the nova side done in newton | 14:01 |
rloo | jroll: still going through my emails, but i noticed node.resource_class. | 14:01 |
rloo | jroll: does it have to be called 'resource_class'? | 14:02 |
*** mgoddard_ has joined #openstack-ironic | 14:02 | |
rloo | jroll: I mean, is it generic enough outside of nova land? | 14:02 |
*** joprovost has joined #openstack-ironic | 14:02 | |
jroll | thiagop: okay, so it's a scheduling hack for picking a chassis or a platform? does platform mean like, certain sort of hardware | 14:02 |
*** anush_ has joined #openstack-ironic | 14:02 | |
jroll | rloo: I don't have a better name for it, it's how it will be represented in the placement api (which is currently in nova but may be a separate project later) | 14:02 |
thiagop | jroll: yep. A certain kind of blade or DL, actually | 14:03 |
jroll | thiagop: got it, thank you | 14:03 |
rloo | jroll: it almost seems like tagging. but i haven't looked into it yet. | 14:03 |
thiagop | jroll: yw! Good to know you're taking a look on it already :) | 14:04 |
thiagop | dtantsur|pto: enjoy! | 14:04 |
vsaienk0 | jroll do you have a time to discuss https://review.openstack.org/#/c/297895 ? | 14:04 |
jroll | rloo: sorta. with nova, it will be related to the flavor. outside of nova, it will be a class of resource being provided (like ipv4 block, storage, etc). no rush, just wanted to make you aware of it | 14:04 |
*** mgoddard has quit IRC | 14:05 | |
rloo | jroll: ok if no rush. will look at it next week. is there anything needed for today? I see console stuff didn't get merged in ironic. | 14:05 |
*** cdearborn has joined #openstack-ironic | 14:05 | |
*** kirankv has quit IRC | 14:05 | |
*** ishant|away has quit IRC | 14:05 | |
*** lazy_prince has quit IRC | 14:05 | |
jroll | rloo: don't think so, next week is good. I'm pushing on the network thing today | 14:05 |
jroll | vsaienk0: yeah, I have a few minutes | 14:05 |
jroll | vsaienk0: in the real world, if the ML2 tries to bind that (which it will, if there's a host ID), it might fail, or worse it may actually work (somehow) and then the agent will be connected to the tenant's network | 14:06 |
jroll | vsaienk0: imagine an ML2 mechanism that caches the switchport info for a node (yes, it exists) | 14:07 |
*** joprovost has quit IRC | 14:07 | |
jroll | vsaienk0: I'm told by neutron folks that binding:host_id is the key to tell ML2 things to connect the port or not, so we need to send a None through there | 14:07 |
*** _vs_ has quit IRC | 14:07 | |
*** poll0172 has joined #openstack-ironic | 14:08 | |
*** harshs has quit IRC | 14:08 | |
*** _vsaienko has quit IRC | 14:08 | |
*** kirankv has joined #openstack-ironic | 14:08 | |
jroll | right now we always send the host_id, so it will always try to bind through (except, apparently, the ML2 plugin we use in CI) | 14:09 |
*** lazy_prince has joined #openstack-ironic | 14:09 | |
*** ishant|away has joined #openstack-ironic | 14:09 | |
TheJulia | I thought the ml2 switch fabric stuff was only supposed to execute on matching switch hostname definitions in the config... then again, it would match in a single host devstack | 14:11 |
*** _vsaienko has joined #openstack-ironic | 14:11 | |
*** _vs_ has joined #openstack-ironic | 14:11 | |
jroll | TheJulia: surely that's up to the ML2 driver | 14:12 |
jroll | e.g. ours doesn't have switch hostnames in the config file | 14:12 |
vsaienk0 | jroll, yeah, sending None seems to makes sense, if agents can use caching, I didn't hear about it earlier | 14:12 |
jroll | I don't want to allow a thing to connect tenant networks to the agent, ever, no matter how poorly an ML2 mechanism is written | 14:12 |
*** poll0172 has quit IRC | 14:12 | |
TheJulia | that makes sense :( | 14:13 |
jroll | vsaienk0: if network vendors are writing ML2 mechanisms, anything can happen :) | 14:13 |
*** poll0172 has joined #openstack-ironic | 14:13 | |
jroll | TheJulia: the other thing, in a virt/baremetal combined environment, this will make sure the virt ML2 things don't try to connect the port | 14:13 |
*** ishant|away has quit IRC | 14:14 | |
jroll | since we apparently don't send baremetal vnic type yet :) | 14:14 |
TheJulia | so even if there is caching, there is nothing we can do if the cache is wrong | 14:14 |
*** baoli has joined #openstack-ironic | 14:14 | |
vsaienk0 | jroll, I think that we need to add a comment to the patch about it | 14:14 |
*** amotoki has joined #openstack-ironic | 14:15 | |
jroll | vsaienk0: a code comment? is the docstring not enough? | 14:15 |
*** harshs has joined #openstack-ironic | 14:15 | |
jroll | vsaienk0: I'm happy to put more words in the code, could you leave a comment on the patch? I need to run to breakfast and tell mriedem it's almost ready ;) | 14:15 |
vsaienk0 | it may confuse, since if we don't know that agent may cache binding_profile info, the change looks useless | 14:16 |
*** milan has quit IRC | 14:16 | |
*** daemontool__ is now known as daemontool | 14:16 | |
*** milan has joined #openstack-ironic | 14:16 | |
jroll | vsaienk0: sure, please comment on the patch and I will add something in an hour or two | 14:16 |
*** ishant|away has joined #openstack-ironic | 14:17 | |
*** baoli has quit IRC | 14:17 | |
*** rbartal has quit IRC | 14:17 | |
abashmak_ | Good morning everyone. Can somebody help figure why I can't run Python3 unit tests on Ironic? | 14:18 |
abashmak_ | I'm getting import errors (pecan and ironic_lib modules) when running "tox -epy34" | 14:18 |
* jroll bbl | 14:18 | |
*** poll0172 has quit IRC | 14:18 | |
*** poll0172 has joined #openstack-ironic | 14:18 | |
jroll | abashmak_: try tox -r -epy34, it will cache installed dependencies and -r will rebuild the virtualenv with new deps | 14:18 |
* jroll really bbl | 14:18 | |
*** baoli has joined #openstack-ironic | 14:19 | |
*** joprovost has joined #openstack-ironic | 14:19 | |
*** baoli has quit IRC | 14:19 | |
*** vishwanathj_zzz is now known as vishwanathj | 14:19 | |
*** poll0172 has quit IRC | 14:19 | |
*** poll0172 has joined #openstack-ironic | 14:23 | |
vdrok | morning TheJulia rloo and jroll ! | 14:31 |
vdrok | dtantsur|pto: enjoy! :) | 14:31 |
rloo | hi vdrok! | 14:31 |
vsaienk0 | jroll, I have left a comment in the patch! thank you for clarification :) | 14:31 |
*** moshele has quit IRC | 14:32 | |
vsaienk0 | jroll, once you have a time could you please review https://review.openstack.org/#/c/334382/ and https://review.openstack.org/#/c/340695/ I think we should switch our gate setup to use FLAT network, since vxlan are not supported by Ironic | 14:32 |
NobodyCam | Good Morning Ironicers | 14:34 |
mgould | vsaienk0, idiot question: we *can't* support vxlan, right? | 14:34 |
mgould | morning NobodyCam | 14:34 |
NobodyCam | morning mgould | 14:34 |
NobodyCam | wb roll :) | 14:34 |
vdrok | good morning NobodyCam :) | 14:34 |
NobodyCam | morning vdrok :) | 14:35 |
vsaienk0 | mgould all is possible event vxlans, but until multitenancy only FLAT network was supported | 14:35 |
vsaienk0 | while all tests were performed at VXLAN | 14:35 |
*** ksumit has joined #openstack-ironic | 14:37 | |
*** mjura has quit IRC | 14:37 | |
vsaienk0 | mgould we have a haks, to setup connectivity from VMs to control plane like https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L804 | 14:37 |
mgould | OK, thanks | 14:38 |
mgould | I was thinking "how do you demux the Ethernet frames before sending them to a BM node?" | 14:38 |
mgould | demux? unwrap? | 14:38 |
openstackgerrit | Merged openstack/ironic-python-agent: Handle diskless hardware connected to remote iscsi https://review.openstack.org/327807 | 14:39 |
*** ljjjustin has quit IRC | 14:40 | |
vsaienk0 | jroll switching to flat network is needed also for https://bugs.launchpad.net/neutron/+bug/1599836 and https://bugs.launchpad.net/ironic/+bug/1304673. Once it is done we can remove workaround from tempest https://github.com/openstack/tempest/blob/f11e225b0c1f31e0c74d618af33ab80df6cb8ffb/tempest/scenario/manager.py#L836 | 14:40 |
openstack | Launchpad bug 1599836 in OpenStack Compute (nova) "Booting Ironic instance, neutron port remains in DOWN state" [High,In progress] - Assigned to Vasyl Saienko (vsaienko) | 14:40 |
openstack | Launchpad bug 1304673 in Ironic "Ironic needs to synchronize external events with Neutron" [Medium,In progress] - Assigned to Vasyl Saienko (vsaienko) | 14:40 |
*** _vs_ has quit IRC | 14:40 | |
*** _vsaienko has quit IRC | 14:40 | |
*** _vsaienko has joined #openstack-ironic | 14:42 | |
*** _vs_ has joined #openstack-ironic | 14:42 | |
*** ishant|away has quit IRC | 14:43 | |
*** lazy_prince has quit IRC | 14:43 | |
*** kirankv has quit IRC | 14:43 | |
*** ljjjustin has joined #openstack-ironic | 14:43 | |
*** michauds has joined #openstack-ironic | 14:44 | |
abashmak_ | I still can't run py3 unit tests, somebody help. Here's the tox output: http://paste.openstack.org/show/539046/ | 14:45 |
*** ChrisAusten has joined #openstack-ironic | 14:46 | |
*** kirankv has joined #openstack-ironic | 14:46 | |
*** ishant|away has joined #openstack-ironic | 14:46 | |
*** lazy_prince has joined #openstack-ironic | 14:47 | |
*** MattMan has quit IRC | 14:47 | |
*** MattMan has joined #openstack-ironic | 14:47 | |
lucasagomes | NobodyCam, morning | 14:49 |
mgould | abashmak_: yeah, seen that before | 14:49 |
mgould | https://bugs.launchpad.net/testrepository/+bug/1229445 | 14:49 |
openstack | Launchpad bug 1212909 in Testrepository "duplicate for #1229445 times.dbm incompatible based on Python version it was created with" [Critical,Triaged] | 14:49 |
lucasagomes | abashmak_, rm .testrepository/times.dbm | 14:49 |
mgould | abashmak_: ^^^ | 14:49 |
NobodyCam | morning lucasagomes :) | 14:49 |
lucasagomes | and run tox again | 14:49 |
mgould | or, brute-force option: delete your .tox dir, then run `tox -e py34` *before* you run `tox -e py27` | 14:50 |
abashmak_ | lucasagomes, mgould: trying... | 14:51 |
lucasagomes | mgould, doesn't work... you have to remove the times.dbm at .testrepository | 14:51 |
mgould | hmmm, that shouldn't work, come to think of it | 14:51 |
mgould | lucasagomes: yeah, you're right | 14:52 |
mgould | huh | 14:52 |
mgould | yeah, I was confused | 14:52 |
mgould | but run tox -e py34 first to prevent the issue coming back | 14:52 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add CLI tests to check JSON response body https://review.openstack.org/291239 | 14:52 |
*** amotoki has quit IRC | 14:52 | |
abashmak_ | Yes, removing times.dbm worked! Thank you lucasagomes and mgould. I'll note the BKM for future. | 14:53 |
*** ishant|away has quit IRC | 14:53 | |
*** amotoki has joined #openstack-ironic | 14:53 | |
*** shobhar has joined #openstack-ironic | 14:53 | |
*** mtanino has joined #openstack-ironic | 14:54 | |
lucasagomes | abashmak_, np | 14:54 |
*** ishant|away has joined #openstack-ironic | 14:56 | |
*** michauds has quit IRC | 14:57 | |
TheJulia | jroll: what if we hold off on powering the machine on until we've verified no other port exists?, granted something like WOL could be an issue | 14:57 |
*** amotoki has quit IRC | 14:58 | |
*** jaybeale has joined #openstack-ironic | 15:00 | |
*** jaybeale has quit IRC | 15:00 | |
*** pece has joined #openstack-ironic | 15:01 | |
*** jaybeale has joined #openstack-ironic | 15:01 | |
*** ksumit has quit IRC | 15:01 | |
TheJulia | jroll: additionally, do you happen to know what ml2 driver(s) exhibit this caching functionality presently | 15:01 |
*** e0ne has quit IRC | 15:01 | |
*** d0ugal has quit IRC | 15:02 | |
*** mgoddard_ has quit IRC | 15:02 | |
*** pcaruana has quit IRC | 15:02 | |
*** ksumit has joined #openstack-ironic | 15:02 | |
*** d0ugal has joined #openstack-ironic | 15:05 | |
*** ksumit has quit IRC | 15:05 | |
*** amotoki has joined #openstack-ironic | 15:07 | |
*** mgoddard has joined #openstack-ironic | 15:08 | |
*** hemna has joined #openstack-ironic | 15:10 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Updated tests for db migration scripts https://review.openstack.org/344246 | 15:11 |
*** sdake has joined #openstack-ironic | 15:13 | |
*** harshs has quit IRC | 15:15 | |
*** trown|mtg is now known as trown | 15:15 | |
*** wajdi has quit IRC | 15:16 | |
*** ksumit has joined #openstack-ironic | 15:17 | |
*** sdake_ has joined #openstack-ironic | 15:17 | |
*** jaybeale has quit IRC | 15:18 | |
*** sdake has quit IRC | 15:19 | |
*** skramaja has quit IRC | 15:20 | |
*** ksumit has quit IRC | 15:22 | |
*** baoli has joined #openstack-ironic | 15:25 | |
*** _vsaienko has quit IRC | 15:27 | |
*** _vs_ has quit IRC | 15:27 | |
*** _vsaienko has joined #openstack-ironic | 15:27 | |
*** _vs_ has joined #openstack-ironic | 15:27 | |
openstackgerrit | Ruby Loo proposed openstack/ironic-python-agent: Remove discover from test-requirements https://review.openstack.org/345513 | 15:28 |
*** baoli has quit IRC | 15:31 | |
*** _vsaienko has quit IRC | 15:31 | |
*** _vsaienko has joined #openstack-ironic | 15:32 | |
*** michauds has joined #openstack-ironic | 15:35 | |
*** bks has joined #openstack-ironic | 15:35 | |
vdrok | folks, could you take a look at one-line patch? https://review.openstack.org/345363 | 15:37 |
vdrok | is unblock functional job in ironicclient | 15:37 |
vdrok | s/is unblock/it unblocks | 15:37 |
*** _vs_ has quit IRC | 15:37 | |
vdrok | also I think we can switch that job to voting mode already | 15:38 |
*** michauds has quit IRC | 15:40 | |
*** _vs_ has joined #openstack-ironic | 15:40 | |
*** wajdi has joined #openstack-ironic | 15:42 | |
*** slagle has quit IRC | 15:45 | |
*** rloo has quit IRC | 15:45 | |
*** marios has quit IRC | 15:45 | |
*** rloo has joined #openstack-ironic | 15:46 | |
lucasagomes | vdrok, done | 15:46 |
vdrok | thx lucasagomes and TheJulia :) | 15:46 |
jroll | TheJulia: we have a downstream thing that does it; granted it doesn't use the new stuff yet, but it shows it can happen | 15:47 |
*** slagle has joined #openstack-ironic | 15:47 | |
*** marios has joined #openstack-ironic | 15:48 | |
*** kirankv has quit IRC | 15:48 | |
jroll | vsaienk0: I won't get to it today, probably next week | 15:49 |
*** penick has joined #openstack-ironic | 15:50 | |
vsaienk0 | jroll, sure no hurry, thanks! | 15:50 |
*** e0ne has joined #openstack-ironic | 15:50 | |
jroll | mgould: there's switches that can translate vxlan from above into vlan down to the server | 15:50 |
jroll | mgould: it gets around the 4096 vlan limit | 15:50 |
*** kirankv has joined #openstack-ironic | 15:51 | |
vdrok | and three liner to make it voting :) https://review.openstack.org/345521 | 15:51 |
*** uggla_ has quit IRC | 15:52 | |
TheJulia | Okay. I skimmed through the drivers and see a little caching here and there, it seems like the only real possible action then is to try and delete it, but that actually working would entirely depend on how cache invalidation works for the driver | 15:53 |
*** poll0172 has quit IRC | 15:54 | |
*** praneshp has joined #openstack-ironic | 15:55 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Collect deployment logs from IPA https://review.openstack.org/336102 | 15:55 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for openstackclient plugin https://review.openstack.org/301687 | 15:55 |
*** _vs_ has quit IRC | 15:56 | |
*** _vsaienko has quit IRC | 15:56 | |
*** devananda has joined #openstack-ironic | 15:58 | |
*** poll0172_ has joined #openstack-ironic | 16:01 | |
*** d0ugal has quit IRC | 16:04 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add node.resource_class field https://review.openstack.org/345055 | 16:04 |
*** joprovost has quit IRC | 16:05 | |
*** rama_y has joined #openstack-ironic | 16:08 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add CLI tests to check JSON response body https://review.openstack.org/291239 | 16:11 |
*** _vs_ has joined #openstack-ironic | 16:13 | |
*** _vsaienko has joined #openstack-ironic | 16:13 | |
*** moshele has joined #openstack-ironic | 16:13 | |
*** tesseract- has quit IRC | 16:15 | |
jroll | devananda: responded to your comments on the resource_class spec | 16:16 |
jroll | devananda: we don't *need* filtering, it could be nice, not sure if we should add on work here when we need to get it done quickly | 16:16 |
*** poll01721 has joined #openstack-ironic | 16:17 | |
*** rcernin has quit IRC | 16:19 | |
*** poll0172_ has quit IRC | 16:20 | |
lucasagomes | jroll, is the nova work for resource class already merged? | 16:21 |
* lucasagomes check jay's open patches | 16:21 | |
JayF | lucasagomes: we've basically been designing it at mid-cycle | 16:21 |
lucasagomes | JayF, cool, so the idea is well accepted between nova? | 16:21 |
JayF | lucasagomes: basically the tl;dr is that by end of mitaka, the goal is to start reporting the information into the resource provider system, but not act on it yet | 16:21 |
JayF | s/mitaka/newton/ | 16:22 |
JayF | so that for ocata we can have a smooth upgrade transition (since the data will have been populated) | 16:22 |
jroll | lucasagomes: so they have resource providers, I think there's a bit more to do, they plan to finish putting scheduling data in the resource providers table by newton, and start using it for scheduling in ocata | 16:22 |
*** _vsaienko has quit IRC | 16:22 | |
*** _vsaienko has joined #openstack-ironic | 16:22 | |
lucasagomes | nice | 16:22 |
*** rajinir has joined #openstack-ironic | 16:22 | |
lucasagomes | thanks for the updates | 16:22 |
jroll | and then what we're trying to do here is get this resource_class to the resource tracker so they can put the data in the right place | 16:22 |
jroll | if we get it done in newton, you won't have to wait 5 minutes after ocata upgrade for scheduling to work :) | 16:23 |
jroll | I'll be sending an email summarizing all this sometime today or tomorrow btw | 16:23 |
lucasagomes | yeah that would be handy | 16:23 |
*** shobhar has quit IRC | 16:24 | |
*** ayoung has joined #openstack-ironic | 16:24 | |
*** ifarkas is now known as ifarkas_afk | 16:24 | |
*** ayoung has quit IRC | 16:25 | |
*** ayoung has joined #openstack-ironic | 16:25 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic-inspector: Allow conditions using values from introspection https://review.openstack.org/345535 | 16:26 |
*** derekh has quit IRC | 16:27 | |
JayF | lucasagomes: <3 erase metadata clean step | 16:27 |
JayF | lucasagomes: just put a -1 on it but mainly because I want it to be default. We've run something similar to that in our downstream hardware managers for >1y | 16:27 |
JayF | jroll: tonyb: can we please land https://review.openstack.org/#/c/314147/ ? | 16:28 |
tonyb | JayF: No! | 16:28 |
JayF | it's stable and has buckets full of +1s | 16:28 |
tonyb | JayF: Oh wait I shoudl probably look first ;P | 16:28 |
jroll | JayF: needs more +1s first | 16:28 |
tonyb | Needs documentation and unit tests ;P | 16:29 |
* jroll trying to figure out if that's in mitaka | 16:29 | |
lucasagomes | JayF, hah cool, I left it disabled by default because erase_devices will remove the metadata already | 16:30 |
lucasagomes | JayF, but yeah, I see your comment about having it to run first | 16:30 |
JayF | lucasagomes: I mean, I don't think it's too bad to do both. The metadata erase should take a trivial amount of time | 16:30 |
lucasagomes | it's also very quick to run, so we should do it | 16:30 |
lucasagomes | exactly | 16:30 |
JayF | lucasagomes: and now for folks who don't want to wait for full erase devices to run can disable it via ironic config | 16:30 |
jroll | oh yeah, it's in mitaka | 16:30 |
lucasagomes | JayF, yup, I will update the patch | 16:31 |
JayF | lucasagomes: but we still get a 'cleaner' setup to build on for next time | 16:31 |
jroll | tonyb: I'm going to +W that unless you're vehemently against it | 16:31 |
lucasagomes | JayF, we may even want to run that as part of tempest to guarantee that cleaning is working | 16:31 |
JayF | lucasagomes: ++++ that's exactly what I was thinking; it's the cleaning step we've needed for devstack running too | 16:31 |
lucasagomes | we can erase the metadata and then mount the disk locally and check if all the partition tables are gone etc... | 16:31 |
JayF | lucasagomes: yep took the words outta my irc client :) | 16:31 |
lucasagomes | JayF, hah cool | 16:31 |
lucasagomes | I will update the patch, tomorrow cause I'm about to leave now | 16:32 |
JayF | lucasagomes: we've run a very dumb version of that (for each in list_block_devices(); dd if=zero of=device bs=1M count=1; done) downstream for a long, long time | 16:32 |
*** jpich has quit IRC | 16:32 | |
lucasagomes | JayF, yeah, problem with ^ is that it won't clean things like gpt | 16:32 |
lucasagomes | since there's a backup partition table at the end of the device as well | 16:33 |
lucasagomes | and other things like filesystem metadata | 16:33 |
lucasagomes | but that's cool, wipefs does it for us | 16:33 |
*** _vs_ has quit IRC | 16:33 | |
*** _vsaienko has quit IRC | 16:33 | |
JayF | lucasagomes: yeah; that's why I'm glad you're making it better :P | 16:33 |
* lucasagomes add "update metadata patch" to his todo early in the morning tomorrow | 16:34 | |
lucasagomes | that said, I will call it a day folks | 16:34 |
JayF | o/ | 16:34 |
lucasagomes | gotta get the train back home yet | 16:34 |
lucasagomes | talk soon | 16:34 |
*** lucasagomes is now known as lucas-afk | 16:35 | |
tonyb | jroll: I +2+W'd it | 16:36 |
JayF | ty, I got very tired of seeing that in my review queue | 16:36 |
JayF | lol | 16:36 |
tonyb | jroll: I *think* it's probably against policy but meh | 16:36 |
tonyb | JayF: np | 16:36 |
*** bvandewa has joined #openstack-ironic | 16:37 | |
TheJulia | thiagop, xavierr: I see https://review.openstack.org/#/c/286192/ has been updated, I'll try and look later today. | 16:38 |
*** poll01721 has quit IRC | 16:41 | |
*** moshele has quit IRC | 16:42 | |
xavierr | hi TheJulia, ok thanks ;) | 16:42 |
*** amotoki has quit IRC | 16:42 | |
*** shobhar has joined #openstack-ironic | 16:44 | |
*** d0ugal has joined #openstack-ironic | 16:44 | |
jroll | tonyb: agree :P | 16:44 |
*** sdake_ has quit IRC | 16:46 | |
*** vishwanathj has quit IRC | 16:47 | |
*** jaosorior has joined #openstack-ironic | 16:48 | |
jaosorior | Hey guys, how do I remove an instance uuid from a node? | 16:48 |
*** sdake has joined #openstack-ironic | 16:49 | |
*** trown is now known as trown|lunch | 16:49 | |
*** jmiu has joined #openstack-ironic | 16:49 | |
*** _vsaienko has joined #openstack-ironic | 16:49 | |
*** _vs_ has joined #openstack-ironic | 16:49 | |
TheJulia | jaosorior: ironic node-update <uuid> remove instance_id should work | 16:54 |
JayF | jaosorior: Typically that's something nova would do on delete. The CLI command is something like ironic node-update UUID remove instance_uuid | 16:54 |
jaosorior | TheJulia: We've been trying that and it's not working :( | 16:54 |
jaosorior | and the logs show nothing either | 16:54 |
JayF | jaosorior: I think you can only do that in certian states; I think it's forboten on active state nodes | 16:55 |
jaosorior | I see | 16:55 |
jaosorior | that seems to be the reason I haven't been able to do it then | 16:56 |
*** moshele has joined #openstack-ironic | 16:56 | |
jaosorior | $ ironic node-update 94066ef2-8604-4041-962b-d68ccc5fdf4a remove 3b9bccd6-1d6b-44b4-a8ba-8c9c9f7008e3 | 16:56 |
jaosorior | Couldn't apply patch '[{'path': '/3b9bccd6-1d6b-44b4-a8ba-8c9c9f7008e3', 'op': 'remove'}]'. Reason: u'3b9bccd6-1d6b-44b4-a8ba-8c9c9f7008e3' (HTTP 400) | 16:56 |
TheJulia | jaosorior: You don't specify the value on the remove command line | 16:56 |
JayF | ironic node-update 94066ef2-8604-4041-962b-d68ccc5fdf4a remove instance_uuid | 16:57 |
JayF | you're specifying the field instance_uuid, not whatever the value is | 16:57 |
TheJulia | jaosorior: and, if it was an update, you'd need it to be key=value | 16:57 |
JayF | basically saying "remove the instance_uuid from node 94066ef2-8604-4041-962b-d68ccc5fdf4a" | 16:57 |
*** aNupoisc has joined #openstack-ironic | 16:57 | |
*** shobhar__ has joined #openstack-ironic | 16:57 | |
jaosorior | aaah | 16:58 |
jaosorior | that makes sense | 16:58 |
jaosorior | thanks folks! | 16:58 |
jaosorior | sorry for the stupid question | 16:58 |
jmiu | the magic state machine diagram | 16:58 |
jmiu | http://chunk.io/f/4fcc1a6d0cfb43ea8c30fb0fb9b4c46e | 16:58 |
JayF | it's fine; we all have to learn | 16:58 |
jaosorior | now, the only funky thing is that we ahve a couple of nodes | 16:59 |
jaosorior | and they have power-state None | 16:59 |
*** jaosorior has quit IRC | 16:59 | |
*** jaosorior has joined #openstack-ironic | 16:59 | |
*** shobhar has quit IRC | 17:00 | |
*** moshele has quit IRC | 17:04 | |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Update documentation for keystone policy support https://review.openstack.org/345556 | 17:05 |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Add multi-tenancy section to security doc https://review.openstack.org/345557 | 17:07 |
*** bks has quit IRC | 17:07 | |
*** _vsaienko has quit IRC | 17:07 | |
*** _vsaienko has joined #openstack-ironic | 17:08 | |
openstackgerrit | Merged openstack/ironic: Do the VM setup only when requested https://review.openstack.org/345363 | 17:10 |
*** baoli has joined #openstack-ironic | 17:16 | |
jaosorior | hey guys, so the nodes are in state active, and I'm trying to set them to deleted but I keep getting this | 17:16 |
jaosorior | Failed to validate power driver interface. Can not delete instance. Error: SSH connection cannot be established: Failed to establish SSH connection to host 192.168.23.1. (HTTP 500) | 17:16 |
jaosorior | any idea what's up? | 17:18 |
*** athomas has quit IRC | 17:19 | |
vdrok | jaosorior: it says what's wrong :) do ironic node-validate your-node | 17:19 |
vdrok | it will show whicj parameters are missing | 17:20 |
jaosorior | vdrok: will do | 17:20 |
*** baoli_ has joined #openstack-ironic | 17:22 | |
*** _vs_ has quit IRC | 17:22 | |
*** _vsaienko has quit IRC | 17:23 | |
*** baoli has quit IRC | 17:25 | |
jaosorior | vdrok: So this is what I get http://paste.openstack.org/show/539098/ not sure why the ssh_terminal_port is missing though :/ | 17:25 |
vdrok | jaosorior: console interface is optional | 17:26 |
JayF | jaosorior: I'm presuming this is devstack? if so, unstacking and restacking will likely fix | 17:26 |
JayF | jaosorior: if it's not devstack, you probably should not run ssh driver, lol | 17:26 |
vdrok | it seems that the ssh credentials that are provided are incorrect | 17:26 |
jaosorior | JayF: it's not devstack, but it is a virtual dev environment | 17:26 |
*** trown|lunch is now known as trown | 17:27 | |
vdrok | jaosorior: you can ssh to the ssh_address you specified with the credentials in driver_info? | 17:27 |
jaosorior | it's actually a tripleo-quickstart provisioned stack. And I ended up changing some networking configurations. But it ended up like this. I'm not very familiar with how ironic works though. So that's why my questions are pretty basic :/ | 17:27 |
jaosorior | trown ^^ | 17:28 |
JayF | You may want to ask in tripleo channel then | 17:28 |
jaosorior | yeah... so... that's why I'm here | 17:29 |
jaosorior | been redirected to this channel instead | 17:29 |
trown | I happen to be in both at least :) | 17:29 |
trown | but not really sure how to rcover from that | 17:30 |
*** athomas has joined #openstack-ironic | 17:30 | |
JayF | I mean, that driver requires that information in driver_info to work | 17:31 |
JayF | without it, it fails | 17:31 |
JayF | what those values are set to is 100% environmental | 17:31 |
jaosorior | JayF: How do I set those values? | 17:31 |
JayF | ironic node-update UUID add driver_info/whatever/nested/key=value | 17:32 |
JayF | the / syntax basically traverses the json tree | 17:32 |
jaosorior | I see | 17:33 |
*** jaybeale has joined #openstack-ironic | 17:34 | |
*** electrofelix has quit IRC | 17:35 | |
*** jcoufal has quit IRC | 17:36 | |
openstackgerrit | Merged openstack/ironic-python-agent: Follow-up text changes for 327807 https://review.openstack.org/345445 | 17:38 |
*** joprovost has joined #openstack-ironic | 17:40 | |
*** Goneri has quit IRC | 17:41 | |
*** harlowja has joined #openstack-ironic | 17:44 | |
*** thiagop has quit IRC | 17:44 | |
*** caiobo-afk has quit IRC | 17:45 | |
*** clenimar has quit IRC | 17:45 | |
*** xavierr has quit IRC | 17:45 | |
*** nicodemos has quit IRC | 17:45 | |
*** liliars has quit IRC | 17:46 | |
*** poll0172_ has joined #openstack-ironic | 17:49 | |
jaosorior | JayF, vdrok: So, I can actually ssh with those credentials and still I cannot set the power state of the node | 17:50 |
*** hemna has quit IRC | 17:52 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Add node.resource_class https://review.openstack.org/345040 | 17:54 |
jroll | devananda: ^ did you see my replies to your comments on the last patchset there? | 17:55 |
*** thiagop has joined #openstack-ironic | 17:55 | |
Madasi | is power state trying to hit the node directly or an out of band management card of some sort? | 17:55 |
* Madasi is coming in to the conversation late, sorry if you've already covered this | 17:56 | |
JayF | Madasi: it's using the ssh driver, for development | 17:56 |
JayF | jroll: I'm going to fix a couple small nits in that (mainly caused by my %s/Nova/nova/ suggestion in PM) | 17:57 |
JayF | jroll: I actually suspect it's going to break the tests because it uncapped a header :x | 17:57 |
jroll | lolol | 17:57 |
jroll | I thought I could just global sed JayF :) | 17:57 |
JayF | that's why I'm jfdi with the fix, lol | 17:57 |
thiagop | Lab's NFS went away to coffee. If someone has spoken to me, here I am | 17:57 |
JayF | you can, and I'm going to fix it, lol | 17:57 |
openstackgerrit | Jay Faulkner proposed openstack/ironic-specs: Add node.resource_class https://review.openstack.org/345040 | 17:58 |
*** mgould is now known as mgould|afk | 17:59 | |
*** daemontool has quit IRC | 18:01 | |
JayF | https://review.openstack.org/#/c/345039/ is a relatively easy spec review, updating the policy spec to reflect some realities discovered during implementation, already has my +2 | 18:02 |
jaosorior | sorry for the mess. It was a typo on my side for the key | 18:03 |
jaosorior | it seems to be working now | 18:03 |
jaosorior | thanks a lot for the help! I really appreciate it | 18:03 |
Madasi | jaosorior: it's usually something really minor like that. Glad you figured it out | 18:03 |
*** baoli_ has quit IRC | 18:03 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Add node.resource_class https://review.openstack.org/345040 | 18:09 |
vgadiraj | hi all, I need a +1 workflow on this patch to get it merged. https://review.openstack.org/#/c/344888/ | 18:09 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add node.resource_class field https://review.openstack.org/345055 | 18:09 |
*** nicodemos has joined #openstack-ironic | 18:11 | |
*** thiagop_ has joined #openstack-ironic | 18:11 | |
JayF | jroll: ^ did you expect that specs change to have any changes? | 18:11 |
*** xavierr has joined #openstack-ironic | 18:11 | |
JayF | jroll: it didn't. | 18:11 |
jroll | JayF: commit message, added related-bug | 18:11 |
JayF | weird; gerrit wasn't showing that change before and when I re-opened now it is | 18:12 |
JayF | *shrug* | 18:12 |
*** clenimar has joined #openstack-ironic | 18:12 | |
jroll | probably had /3 or whatever on the end when you refreshed | 18:12 |
*** caiobo-afk has joined #openstack-ironic | 18:13 | |
*** liliars has joined #openstack-ironic | 18:13 | |
JayF | yeah; that sometimes messes with me | 18:14 |
*** thiagop has quit IRC | 18:15 | |
*** thiagop_ is now known as thiagop | 18:15 | |
*** harshs has joined #openstack-ironic | 18:20 | |
*** zzzeek is now known as zzzeek_afk_but_r | 18:22 | |
*** Sukhdev has joined #openstack-ironic | 18:22 | |
*** zzzeek_afk_but_r is now known as zzzeek_afk | 18:23 | |
openstackgerrit | Jay Faulkner proposed openstack/ironic: Added note to local.conf addressing firewall/proxy blocking Git protocol https://review.openstack.org/344888 | 18:24 |
JayF | jroll: you had +2 on https://review.openstack.org/#/c/344888, I fixed a nit, you wanna land it? easy docs fix | 18:25 |
*** thiagop has quit IRC | 18:25 | |
*** catintheroof has quit IRC | 18:25 | |
jroll | JayF: done | 18:26 |
*** jaosorior has quit IRC | 18:27 | |
*** jcoufal has joined #openstack-ironic | 18:30 | |
devananda | jroll: looking now | 18:32 |
*** harlowja has quit IRC | 18:35 | |
*** harlowja has joined #openstack-ironic | 18:38 | |
jroll | thanks | 18:39 |
*** garthb has joined #openstack-ironic | 18:40 | |
devananda | jroll: comments posted on rev1 | 18:41 |
jroll | devananda: yep, saw it | 18:42 |
jroll | so, 1) I don't want to hinge this on a bunch of work in our client | 18:42 |
jroll | 2) I'm fine with a separate table if we think that endpoint would be useful (and something we'd actually do) | 18:42 |
jroll | I'm not sure it is | 18:42 |
jroll | and keep in mind we can move it later pretty easily | 18:43 |
*** thiagop has joined #openstack-ironic | 18:43 | |
devananda | 1) client stuff was a sidebar. I completely agree - no interest in holding this up | 18:43 |
*** shobhar has joined #openstack-ironic | 18:43 | |
devananda | 2) can we take 5 minutes to discuss the table thing? | 18:43 |
jroll | sure | 18:44 |
*** _vsaienko has joined #openstack-ironic | 18:44 | |
*** _vs_ has joined #openstack-ironic | 18:44 | |
devananda | cloud admin will need to do some management of this field in conjunction with their flavor definitions, yes? | 18:44 |
devananda | the goal being a 1:1 mapping | 18:44 |
devananda | (at least for baremetal flavors) | 18:44 |
*** jcoufal_ has joined #openstack-ironic | 18:44 | |
jroll | right | 18:45 |
devananda | that's going to be awkward if it's a VARCHAR field on the node resource | 18:45 |
jroll | what's going to be awkward, specifically? | 18:45 |
devananda | I would need to fetch every node, then in the clientside, group and sort the results of this column | 18:45 |
jroll | ensuring things have the right resource_class? | 18:45 |
devananda | in order to get the list of resoruce_classes in ironic | 18:46 |
devananda | to compare to nova's flavor list | 18:46 |
*** jcoufal has quit IRC | 18:46 | |
*** shobhar__ has quit IRC | 18:46 | |
jroll | right, my wonder is if that is useful | 18:46 |
jroll | the list, without what they're associated to | 18:47 |
devananda | is "nova flavor-list" useful? | 18:47 |
jroll | yes, because nova is intended for someone to choose a flavor, and it finds something matching for you | 18:47 |
devananda | right | 18:48 |
devananda | this field provides similar functionality, AIUI | 18:48 |
*** _vs_ has quit IRC | 18:48 | |
*** _vsaienko has quit IRC | 18:48 | |
devananda | help a user (nova) find hardware that correlates to a nova flavor (resource type) | 18:48 |
JayF | I could very much see useful stuff with having that be a separate api | 18:48 |
jroll | just a list of classes, when you don't have an endpoint that requires one to pass in a class, idk | 18:48 |
JayF | like a dashboard discovering available resource classes to group nodes by, for instance | 18:48 |
jroll | devananda: that's not the plan for nova, though | 18:48 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [api] https://review.openstack.org/309186 | 18:49 |
jroll | I do see some amount of usefulness | 18:49 |
JayF | Although whether or not that usefulness is worth the added complexity is unknown. | 18:49 |
jroll | is a separate table worth the effort compared to select unique(resource_class) from nodes; | 18:49 |
jroll | is my question | 18:49 |
jroll | (assuming we index that field, ofc) | 18:49 |
devananda | as an operator, when I look at the list of nova flavors / resource_classes, how do I determine whether a) all of these exist in Ironic, b) how much availability each flavor has remaining in ironic's fleet? | 18:50 |
JayF | devananda: (b) would be via the nova placement api | 18:50 |
JayF | devananda: not via ironic | 18:50 |
devananda | jroll: that's fair. separate table vs. separate API resource | 18:50 |
devananda | they don't need to be the same | 18:50 |
jroll | devananda: right, I think you've convinced me that a list of resource classes is useful | 18:50 |
jroll | clearly an extra table saves bytes, too | 18:50 |
devananda | yea | 18:51 |
*** michauds has joined #openstack-ironic | 18:51 | |
devananda | at a few thousand nodes, select unique(column) isn't really a concern | 18:51 |
devananda | at hundreds of thousands, it is | 18:52 |
jroll | is it? | 18:52 |
jroll | (I'm not an expert) | 18:52 |
JayF | I suspect we'd break in more ways than just that if you had a big enough nodes table that it couldnt fit into memory | 18:53 |
devananda | jroll: don't assume everyone runs their ironic db on a huge physical server | 18:53 |
jroll | devananda: I don't, we run ours on a smallish VM | 18:53 |
devananda | cool | 18:53 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [deploy] https://review.openstack.org/309206 | 18:54 |
jroll | I just have zero experience with mysql performance | 18:54 |
JayF | My suspicion is this would not end up being a major perf issue on our db compared to other things at hyperscale | 18:54 |
JayF | I have nebulous experience with mysql performance, but I tend to solve mysql perf problems with giant beefy dbs, lol | 18:55 |
devananda | JayF: would you like me to describe the situation in which "GET /resources/" will be slow if it's a VARCHAR(80) on the nodes table, and fast if it's in a separate table? | 18:56 |
JayF | devananda: well presumedly it's the difference between having to do a scan across count(nodes) rows vs just a scan across count(resource_class) rows | 18:56 |
jroll | devananda: keep in mind this is a varchar(100), which means >255 bytes, which means some things have problems >.> | 18:57 |
devananda | jroll: see my comment on rev4? I thought jaypipies said their field is VARCHAR(80) | 18:58 |
jroll | from what I've heard, anyway | 18:58 |
jroll | devananda: dunno, their spec says 100 | 18:58 |
devananda | and yes, you're correct, VARCHAR(100) will be an issue | 18:58 |
devananda | oh | 18:58 |
jroll | or it did yesterday | 18:58 |
devananda | link? I'll -1 that | 18:58 |
jroll | devananda: line 69 https://review.openstack.org/#/c/312696/5/specs/newton/approved/resource-providers-dynamic-resource-classes.rst | 18:58 |
*** harlowja has quit IRC | 18:59 | |
openstackgerrit | Merged openstack/ironic: Doc update for in-band cleaning support on more drivers https://review.openstack.org/338744 | 18:59 |
*** r-daneel has joined #openstack-ironic | 18:59 | |
*** ChubYann has joined #openstack-ironic | 19:00 | |
*** persia has quit IRC | 19:01 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [agent] https://review.openstack.org/309603 | 19:02 |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Update documentation for keystone policy support https://review.openstack.org/345556 | 19:04 |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Mask configdrive contents in API responses https://review.openstack.org/326768 | 19:04 |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Hide driver[_internal]_info from observer role https://review.openstack.org/345076 | 19:04 |
*** harshs has quit IRC | 19:05 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [pxe] https://review.openstack.org/309612 | 19:06 |
*** jjohnson2 has quit IRC | 19:07 | |
*** jjohnson2_ has joined #openstack-ironic | 19:07 | |
*** pshige______ has quit IRC | 19:07 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [iscsi] https://review.openstack.org/309615 | 19:11 |
*** harshs has joined #openstack-ironic | 19:14 | |
*** harshs has quit IRC | 19:15 | |
*** baoli has joined #openstack-ironic | 19:20 | |
*** pshige______ has joined #openstack-ironic | 19:22 | |
*** baoli has quit IRC | 19:22 | |
*** baoli has joined #openstack-ironic | 19:22 | |
*** poll0172_ has quit IRC | 19:23 | |
*** _vsaienko has joined #openstack-ironic | 19:24 | |
*** _vs_ has joined #openstack-ironic | 19:24 | |
openstackgerrit | Ruby Loo proposed openstack/python-ironicclient: Remove unused LONG_FIELDS https://review.openstack.org/345635 | 19:26 |
*** harshs has joined #openstack-ironic | 19:29 | |
*** jerrygb has quit IRC | 19:32 | |
*** _vsaienko has quit IRC | 19:35 | |
*** _vs_ has quit IRC | 19:35 | |
*** joprovost has quit IRC | 19:36 | |
*** _vs_ has joined #openstack-ironic | 19:36 | |
*** _vsaienko has joined #openstack-ironic | 19:36 | |
*** shobhar has quit IRC | 19:37 | |
*** vishwanathj has joined #openstack-ironic | 19:37 | |
*** sdake_ has joined #openstack-ironic | 19:39 | |
*** sdake has quit IRC | 19:40 | |
*** shobhar has joined #openstack-ironic | 19:40 | |
*** jerrygb has joined #openstack-ironic | 19:40 | |
*** baoli has quit IRC | 19:50 | |
*** _vsaienko has quit IRC | 19:57 | |
*** _vs_ has quit IRC | 19:57 | |
openstackgerrit | Merged openstack/python-ironicclient: Add Python 3.5 classifier and venv https://review.openstack.org/342037 | 19:58 |
*** ijw has quit IRC | 20:00 | |
*** _vsaienko has joined #openstack-ironic | 20:02 | |
*** _vs_ has joined #openstack-ironic | 20:02 | |
*** ijw_ has joined #openstack-ironic | 20:03 | |
*** hemna has joined #openstack-ironic | 20:05 | |
*** aNupoisc has quit IRC | 20:07 | |
*** ijw_ has quit IRC | 20:07 | |
*** _vs_ has quit IRC | 20:14 | |
*** _vsaienko has quit IRC | 20:14 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Add node.resource_class https://review.openstack.org/345040 | 20:14 |
jroll | rloo: so when you have time, JayF, devananda, jaypipes, and myself all just agreed to ^ | 20:15 |
rloo | jroll: ok | 20:15 |
rloo | jroll: looking... | 20:16 |
JayF | jroll: ^ also that spec still says VARCHAR(100) | 20:16 |
jroll | devananda: oh, should I s/100/80/ in there? | 20:16 |
jroll | yeah that question | 20:16 |
JayF | +1 | 20:16 |
JayF | I think even if nova allows 100, us allowing 80 for performance is reasonable | 20:16 |
jroll | even if placement engine still allows 100, it won't hurt for us to do that | 20:16 |
* jroll == JayF | 20:16 | |
JayF | yep exactly | 20:16 |
rloo | if nova allows 100, and ironic allows 80, then there won't be a match for >80 chars? | 20:17 |
JayF | it goes the other way; nova is matching on the ironic data | 20:17 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Add node.resource_class https://review.openstack.org/345040 | 20:18 |
TheJulia | thiagop: xavierr: I have one concern on dynamic node allocation implementation, not worth blocking, and truthfully I think it is your guys preference as to the behavior. I explained my thoughts in the review. Just let me know :) | 20:18 |
JayF | so it just means there'd be 20 character of field we wouldn't use | 20:18 |
rloo | jroll: did you open an RFE bug for this? | 20:18 |
jroll | rloo: yep, linked in commit message | 20:18 |
jroll | or it was, I may have lost that | 20:18 |
rloo | jroll: it was lost | 20:18 |
thiagop | TheJulia: ok, will look in a sec | 20:18 |
JayF | and I know devananda has a -1 on the nova side telling them to shrink it to 80varchar | 20:18 |
JayF | b/c 80*3 < 256 | 20:18 |
TheJulia | no rush thiagop | 20:18 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Add node.resource_class https://review.openstack.org/345040 | 20:18 |
rloo | make it 80 chars. can always change it to 100 IF WE HAVE TO :) | 20:18 |
jroll | put it back | 20:18 |
jroll | heh | 20:18 |
jroll | rloo: fwiw, I'll be emailing the game plan in detail today or tomorrow | 20:19 |
jroll | for some context | 20:19 |
rloo | jroll: ok, i'll review it now and wait for the big picture later :) | 20:19 |
jroll | rloo: thanks :) | 20:20 |
*** _vsaienko has joined #openstack-ironic | 20:21 | |
*** _vs_ has joined #openstack-ironic | 20:21 | |
*** poll0172 has joined #openstack-ironic | 20:21 | |
openstackgerrit | Merged openstack/ironic: Added note to local.conf addressing firewall/proxy blocking Git protocol https://review.openstack.org/344888 | 20:25 |
*** moshele has joined #openstack-ironic | 20:33 | |
*** persia has joined #openstack-ironic | 20:35 | |
*** ijw has joined #openstack-ironic | 20:37 | |
*** harlowja has joined #openstack-ironic | 20:39 | |
*** ijw has quit IRC | 20:41 | |
*** aNupoisc has joined #openstack-ironic | 20:42 | |
*** jaybeale has quit IRC | 20:42 | |
rloo | jroll: commented. nothing serious :) | 20:43 |
*** dprince has quit IRC | 20:44 | |
jroll | rloo: thanks, will look | 20:44 |
jroll | rloo: I suspect I don't need to get back to you until monday morning? :) | 20:45 |
rloo | jroll: yup, no hurry. | 20:45 |
jroll | cool, ty | 20:45 |
*** jjohnson2_ has quit IRC | 20:46 | |
* JayF heading out of the nova mid-cycle o/ | 20:50 | |
NobodyCam | :) | 20:50 |
jroll | JayF: later \o | 20:50 |
*** moshele has quit IRC | 20:51 | |
thiagop | TheJulia: wrt L51-57: get_oneview_info is just an acessory method to get all related oneview fields on driver_info and properties. It shouldn't touch OneView or the nodes themselves, but I think not calling it while the node isn't on the right state makes sense (just to avoid a call or raise unrelated errors, as you said). | 20:53 |
thiagop | TheJulia: makes sense? | 20:54 |
*** ametts has quit IRC | 20:59 | |
*** poll0172 has quit IRC | 21:02 | |
*** ccamacho has quit IRC | 21:02 | |
gabriel-bezerra | jroll: did you get what you asked me for already? | 21:10 |
*** _vsaienko has quit IRC | 21:11 | |
gabriel-bezerra | jroll: they are there for us to be able to match flavors against them, as they have some implicit hardware capabilities. | 21:12 |
jroll | gabriel-bezerra: I did, thank you | 21:12 |
devananda | JayF: wdyt of renaming policy.json -> policy.json.sample, and then providing an empty file in tree, to signal this to deployers? | 21:12 |
gabriel-bezerra | jroll: sorry for not having replied earlier, my irc client is in a VM and doesn't notify me (loud enough) of mentions. | 21:13 |
jroll | gabriel-bezerra: no worries | 21:13 |
*** _vs_ has quit IRC | 21:14 | |
*** trown is now known as trown|outtypewww | 21:19 | |
*** poll0172_ has joined #openstack-ironic | 21:19 | |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Update documentation for keystone policy support https://review.openstack.org/345556 | 21:23 |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Mask configdrive contents in API responses https://review.openstack.org/326768 | 21:23 |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Add keystone policy support to Ironic https://review.openstack.org/325599 | 21:23 |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: [WIP] Hide driver[_internal]_info from observer role https://review.openstack.org/345076 | 21:23 |
*** shobhar has quit IRC | 21:23 | |
* devananda drives home from the nova midcycle | 21:23 | |
devananda | see ya'll tomorrow morning! | 21:23 |
NobodyCam | have a safe drive devananda | 21:25 |
*** penick has quit IRC | 21:26 | |
*** _vsaienko has joined #openstack-ironic | 21:27 | |
*** _vs_ has joined #openstack-ironic | 21:27 | |
*** poll0172_ has quit IRC | 21:28 | |
*** ijw has joined #openstack-ironic | 21:40 | |
*** harlowja has quit IRC | 21:41 | |
*** poll0172 has joined #openstack-ironic | 21:42 | |
thiagop | safe travels devananda | 21:44 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Add tempest tests for node Adoption https://review.openstack.org/344975 | 21:45 |
*** pece has quit IRC | 21:45 | |
*** xavierr has quit IRC | 21:46 | |
*** clenimar has quit IRC | 21:47 | |
TheJulia | thiagop: pulling it back up | 21:47 |
*** caiobo-afk has quit IRC | 21:47 | |
*** harlowja has joined #openstack-ironic | 21:48 | |
*** nicodemos has quit IRC | 21:48 | |
TheJulia | thiagop: doh! | 21:48 |
*** harlowja has quit IRC | 21:48 | |
*** liliars has quit IRC | 21:49 | |
*** thiagop has quit IRC | 21:49 | |
*** wajdi_ has joined #openstack-ironic | 21:51 | |
*** amotoki has joined #openstack-ironic | 21:55 | |
*** wajdi has quit IRC | 21:56 | |
*** wajdi_ has quit IRC | 21:56 | |
*** wajdi has joined #openstack-ironic | 21:57 | |
*** wajdi has quit IRC | 22:01 | |
*** thiagop has joined #openstack-ironic | 22:01 | |
*** xavierr has joined #openstack-ironic | 22:02 | |
*** nicodemos has joined #openstack-ironic | 22:02 | |
*** liliars has joined #openstack-ironic | 22:02 | |
*** baoli has joined #openstack-ironic | 22:02 | |
*** clenimar has joined #openstack-ironic | 22:02 | |
*** caiobo-afk has joined #openstack-ironic | 22:03 | |
*** baoli has quit IRC | 22:07 | |
*** cdearborn has quit IRC | 22:11 | |
*** e0ne has quit IRC | 22:12 | |
*** shobhar has joined #openstack-ironic | 22:13 | |
*** ijw has quit IRC | 22:22 | |
*** jaybeale has joined #openstack-ironic | 22:22 | |
*** sdake_ has quit IRC | 22:24 | |
*** liliars has quit IRC | 22:24 | |
*** hemna has quit IRC | 22:26 | |
*** thiagop has quit IRC | 22:26 | |
openstackgerrit | Stephanie Miller proposed openstack/ironic: Add volume_connector table to save connector information https://review.openstack.org/200983 | 22:27 |
openstackgerrit | Stephanie Miller proposed openstack/ironic: Add volume_connector table to save connector information https://review.openstack.org/200983 | 22:32 |
*** ElCoyote_ has left #openstack-ironic | 22:33 | |
*** sdake has joined #openstack-ironic | 22:49 | |
*** ChrisAusten has quit IRC | 22:52 | |
*** aNupoisc has quit IRC | 22:53 | |
*** poll0172 has quit IRC | 22:55 | |
*** jerrygb has quit IRC | 22:55 | |
*** shobhar has quit IRC | 22:59 | |
*** zzzeek_afk is now known as zzzeek | 23:00 | |
*** rama_y has quit IRC | 23:02 | |
*** ijw has joined #openstack-ironic | 23:04 | |
*** rama_y has joined #openstack-ironic | 23:09 | |
*** jaybeale_ has joined #openstack-ironic | 23:13 | |
*** shobhar has joined #openstack-ironic | 23:15 | |
*** aNupoisc has joined #openstack-ironic | 23:16 | |
*** jaybeale has quit IRC | 23:17 | |
*** r-daneel has quit IRC | 23:23 | |
*** _vsaienko has quit IRC | 23:30 | |
*** _vs_ has quit IRC | 23:30 | |
*** harshs has quit IRC | 23:37 | |
*** harshs has joined #openstack-ironic | 23:41 | |
*** anush_ has quit IRC | 23:57 | |
*** harshs has quit IRC | 23:57 | |
*** harshs has joined #openstack-ironic | 23:57 | |
*** harshs has quit IRC | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!