| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Remove IloPXEDeploy.validate() https://review.openstack.org/148136 | 00:02 |
|---|---|---|
| *** naohirot has joined #openstack-ironic | 00:38 | |
| naohirot | GM ironic! | 00:38 |
| *** Masahiro has joined #openstack-ironic | 00:54 | |
| kylestev | hey mrda :) | 01:09 |
| mrda | hey kylestev | 01:13 |
| *** achanda has quit IRC | 01:17 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic-specs: Replace requests lib to python-scciclient lib https://review.openstack.org/148145 | 01:20 |
| *** achanda has joined #openstack-ironic | 01:23 | |
| *** achanda has quit IRC | 01:34 | |
| *** achanda has joined #openstack-ironic | 01:37 | |
| *** Marga_ has quit IRC | 01:45 | |
| *** Marga_ has joined #openstack-ironic | 01:46 | |
| *** yuanying_ has joined #openstack-ironic | 01:49 | |
| *** yuanying has quit IRC | 01:49 | |
| *** Marga__ has joined #openstack-ironic | 01:50 | |
| *** Marga_ has quit IRC | 01:50 | |
| *** chenglch has joined #openstack-ironic | 01:54 | |
| *** kylestev is now known as kylestev|afk | 01:55 | |
| *** Marga__ has quit IRC | 01:57 | |
| *** jerryz has joined #openstack-ironic | 02:14 | |
| *** achanda has quit IRC | 02:23 | |
| *** achanda has joined #openstack-ironic | 02:23 | |
| *** achanda has quit IRC | 02:24 | |
| *** ryanpetrello has joined #openstack-ironic | 02:25 | |
| *** jerryz has quit IRC | 02:41 | |
| *** HenryG has joined #openstack-ironic | 02:44 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic-specs: iRMC Management Driver for Ironic https://review.openstack.org/136020 | 02:49 |
| *** achanda has joined #openstack-ironic | 02:56 | |
| *** jerryz has joined #openstack-ironic | 02:57 | |
| *** pensu has joined #openstack-ironic | 03:00 | |
| *** ryanpetrello has quit IRC | 03:13 | |
| *** naohirot has quit IRC | 03:27 | |
| *** Masahiro has quit IRC | 03:28 | |
| *** achanda has quit IRC | 03:46 | |
| *** krtaylor has quit IRC | 03:50 | |
| *** krtaylor has joined #openstack-ironic | 03:52 | |
| *** coolsvap|afk is now known as coolsvap | 03:56 | |
| *** Masahiro has joined #openstack-ironic | 03:57 | |
| *** naohirot has joined #openstack-ironic | 04:01 | |
| *** achanda has joined #openstack-ironic | 04:02 | |
| *** pensu has quit IRC | 04:26 | |
| *** eghobo has joined #openstack-ironic | 04:39 | |
| *** rameshg87 has joined #openstack-ironic | 04:42 | |
| *** jerryz has quit IRC | 04:46 | |
| *** kbyrne has quit IRC | 04:55 | |
| *** ryanpetrello has joined #openstack-ironic | 04:58 | |
| *** ryanpetrello has quit IRC | 05:03 | |
| *** Marga_ has joined #openstack-ironic | 05:05 | |
| *** pradipta_away is now known as pradipta | 05:08 | |
| *** pensu has joined #openstack-ironic | 05:12 | |
| *** achanda has quit IRC | 05:31 | |
| *** pcaruana has quit IRC | 05:32 | |
| *** achanda has joined #openstack-ironic | 05:37 | |
| *** achanda has quit IRC | 05:41 | |
| *** achanda has joined #openstack-ironic | 06:08 | |
| *** rakesh_hs has joined #openstack-ironic | 06:10 | |
| *** achanda has quit IRC | 06:18 | |
| *** alex_xu has quit IRC | 06:21 | |
| *** eghobo has quit IRC | 06:37 | |
| *** yuanying has joined #openstack-ironic | 07:07 | |
| *** yuanying_ has quit IRC | 07:07 | |
| *** chlong has quit IRC | 07:10 | |
| *** achanda has joined #openstack-ironic | 07:12 | |
| *** pcaruana has joined #openstack-ironic | 07:42 | |
| *** yuriyz has joined #openstack-ironic | 07:45 | |
| *** Krast has joined #openstack-ironic | 07:47 | |
| *** Krast has quit IRC | 07:47 | |
| *** ifarkas has joined #openstack-ironic | 08:11 | |
| *** pcrews has quit IRC | 08:14 | |
| *** ndipanov has joined #openstack-ironic | 08:14 | |
| *** achanda has quit IRC | 08:15 | |
| *** mkerrin has quit IRC | 08:31 | |
| *** enterprisedc has joined #openstack-ironic | 08:32 | |
| *** dtantsur|afk is now known as dtantsur | 08:40 | |
| dtantsur | Morning Ironic | 08:40 |
| *** gilllliard is now known as gilliard | 08:43 | |
| gilliard | Morning | 08:43 |
| *** andreykurilin_ has joined #openstack-ironic | 08:49 | |
| *** jcoufal has joined #openstack-ironic | 08:51 | |
| *** bauwser is now known as bauzas | 09:01 | |
| *** athomas has joined #openstack-ironic | 09:13 | |
| *** lucasagomes has joined #openstack-ironic | 09:15 | |
| *** andreykurilin_ has quit IRC | 09:21 | |
| *** eghobo has joined #openstack-ironic | 09:22 | |
| *** jistr has joined #openstack-ironic | 09:24 | |
| *** eghobo has quit IRC | 09:28 | |
| *** romcheg has joined #openstack-ironic | 09:38 | |
| *** derekh has joined #openstack-ironic | 09:40 | |
| *** stelfer has joined #openstack-ironic | 09:46 | |
| *** naohirot has quit IRC | 09:48 | |
| *** MattMan has joined #openstack-ironic | 09:54 | |
| *** MattMan has left #openstack-ironic | 09:54 | |
| *** MattMan has joined #openstack-ironic | 09:55 | |
| *** kylestev|afk is now known as kylestev | 09:58 | |
| *** alex_xu has joined #openstack-ironic | 09:58 | |
| *** mkerrin has joined #openstack-ironic | 10:04 | |
| openstackgerrit | Frederic Lepied proposed stackforge/ironic-discoverd: Add eDeploy plugin https://review.openstack.org/146599 | 10:17 |
| *** kbyrne has joined #openstack-ironic | 10:28 | |
| ifarkas | lucasagomes, hey, could you please review my 2 drac patches whenever you will have time for them? https://review.openstack.org/#/c/146879/ and https://review.openstack.org/#/c/147203/ | 10:32 |
| lucasagomes | ifarkas, hey, sure will do! | 10:32 |
| ifarkas | lucasagomes, thanks! | 10:32 |
| openstackgerrit | Frederic Lepied proposed stackforge/ironic-discoverd: Add eDeploy plugin https://review.openstack.org/146599 | 10:44 |
| openstackgerrit | Frederic Lepied proposed stackforge/ironic-discoverd: Add eDeploy plugin https://review.openstack.org/146599 | 10:55 |
| *** kylestev is now known as kylestev|zzz | 10:57 | |
| *** rameshg87 has quit IRC | 11:05 | |
| *** pelix1 has joined #openstack-ironic | 11:10 | |
| openstackgerrit | Merged openstack/ironic: DracClient to handle ClientOptions creation https://review.openstack.org/146879 | 11:17 |
| gilliard | Hello Ironic. I backported the patches which fix the rebuild-with-different-images bugs. Never done any backporting before so I hope I did it correctly; https://review.openstack.org/#/c/148218/ and https://review.openstack.org/#/c/148219/ | 11:21 |
| *** pensu has quit IRC | 11:21 | |
| *** igordcard has joined #openstack-ironic | 11:31 | |
| *** enterprisedc has quit IRC | 11:36 | |
| *** Haomeng has joined #openstack-ironic | 11:42 | |
| *** Haomeng|2 has quit IRC | 11:44 | |
| *** igordcard has quit IRC | 11:52 | |
| *** pensu has joined #openstack-ironic | 12:04 | |
| *** naohirot has joined #openstack-ironic | 12:05 | |
| *** pradipta is now known as pradipta_away | 12:05 | |
| *** jerryz has joined #openstack-ironic | 12:12 | |
| *** ndipanov has quit IRC | 12:12 | |
| *** dlpartain has joined #openstack-ironic | 12:14 | |
| *** dlpartain has quit IRC | 12:14 | |
| *** ndipanov has joined #openstack-ironic | 12:23 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Remove IloPXEDeploy.validate() https://review.openstack.org/148136 | 12:25 |
| *** pensu has quit IRC | 12:35 | |
| *** dprince has joined #openstack-ironic | 12:39 | |
| *** pelix1 is now known as pelix | 12:40 | |
| *** afaranha has quit IRC | 12:42 | |
| naohirot | dtantsur: Hi, thank you for the review, https://review.openstack.org/#/c/148145/1 | 12:50 |
| dtantsur | hi, you're welcome | 12:50 |
| naohirot | dtantsur: can I ask a question about stackforge? | 12:50 |
| dtantsur | sure | 12:51 |
| *** lucasagomes is now known as lucas-hungry | 12:51 | |
| naohirot | dtantsur: I was think to check in scciclient code into plain GitHub, and I'm in the process of company's procedure. | 12:51 |
| naohirot | dtantsur: Is it better to use StackForge instead of plain GitHub? | 12:52 |
| dtantsur | naohirot, that will also work. stackforge is just a nice place to take advantage of the openstack infrastructure. | 12:52 |
| openstackgerrit | Frederic Lepied proposed stackforge/ironic-discoverd: Add eDeploy plugin https://review.openstack.org/146599 | 12:53 |
| naohirot | dtantsur: If I choose stackforge, how do I check in the scci code? I mean how do I review? | 12:54 |
| dtantsur | naohirot, you'll be using Gerrit. please refer to http://ci.openstack.org/stackforge.html for details | 12:54 |
| naohirot | dtantsur: I know you are working on https://github.com/stackforge/ironic-discoverd | 12:55 |
| dtantsur | naohirot, correct | 12:55 |
| naohirot | dtantsur: Is it reviewed by readhat engineers? | 12:55 |
| naohirot | dtantsur: If I chose stackforge, who reviews sccicode? | 12:56 |
| dtantsur | naohirot, currently the core team consists of Red Hat engineers (which I hope to change). But you can treat it as any other upstream project. | 12:56 |
| dtantsur | naohirot, you :) and people you add to the core team | 12:56 |
| dtantsur | naohirot, only initial review (when checking in code the first time) is done by infra, and they don't review code, they just check that you're project is appropriate | 12:57 |
| naohirot | dtantsur: currently I'm only engineer who is working in scci client in my company. | 12:58 |
| dtantsur | naohirot, btw I don't _insist_ on using stackforge, that was just an idea | 12:58 |
| dtantsur | you can do well with github only (though you won't get gerrit, jenkins jobs etc) | 12:58 |
| naohirot | dtantsur: Yes, of course, I know that. I just would like to know the actual process. | 12:59 |
| naohirot | dtantsur: If there were no reviewer of scci, could I check in the code change into github? | 13:01 |
| openstackgerrit | Julien Danjou proposed openstack/ironic: Remove lockfile from dependencies https://review.openstack.org/148250 | 13:01 |
| openstackgerrit | Julien Danjou proposed openstack/ironic: Remove jsonutils from openstack.common https://review.openstack.org/148251 | 13:01 |
| dtantsur | naohirot, not sure what you mean. if you don't use stackforge, you just check in code to github as usual | 13:02 |
| naohirot | dtantsur: Maybe I just don't know some very basic thing, that's would be the reason of strange question :) | 13:03 |
| dtantsur | naohirot, unfortunately I can no longer find good docs on StackForge, so you might be better off just publishing it on github and calling it a day :) | 13:04 |
| *** rev has quit IRC | 13:05 | |
| naohirot | dtantsur: In case of official openstack like Ironic, core reviewer has to endorse the code before checking in. | 13:05 |
| naohirot | dtantsur: In case of stackforge, who endorse the code? | 13:05 |
| dtantsur | naohirot, also cores, e.g. we have a group called ironic-discoverd-core which consists of me and a couple more guys. if you go with stackforge, you'll have your own team (maybe only consisting of you) to approve stuff | 13:06 |
| naohirot | dtantsur: Okay, in case of ironic-discoverd | 13:08 |
| naohirot | , the team consists of redhat engineers right now. | 13:08 |
| dtantsur | yep | 13:08 |
| dtantsur | (just because nobody else wants to do discoverd reviews) | 13:08 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: Add a new driver for Fuel Agent https://review.openstack.org/138115 | 13:09 |
| naohirot | dtantsur: In case of scci, I'm only engineer who works on scci. Is it possible to be a team which member is only me from stackforge's point of view? | 13:10 |
| dtantsur | naohirot, yes. initially it's always only one person, then you may add more people, if you feel like | 13:10 |
| openstackgerrit | Tan Lin proposed openstack/ironic-specs: Bare Metal Trust Using Intel TXT https://review.openstack.org/133902 | 13:11 |
| naohirot | dtantsur: I see, now I got it. thanks. and there is another question regarding __init__ check. | 13:11 |
| naohirot | dtantsur: do you mean __init__ check like this? https://github.com/openstack/ironic/blob/master/ironic/drivers/ilo.py#L39-L43 | 13:12 |
| dtantsur | naohirot, exactly | 13:12 |
| naohirot | Okay, I got it. I'll update the spec. thanks for sparing your time. :) | 13:13 |
| *** athomas_ has joined #openstack-ironic | 13:19 | |
| dtantsur | np) | 13:19 |
| *** athomas_ has joined #openstack-ironic | 13:19 | |
| *** athomas_ has quit IRC | 13:19 | |
| *** athomas_ has joined #openstack-ironic | 13:20 | |
| *** athomas has quit IRC | 13:20 | |
| *** athomas has joined #openstack-ironic | 13:20 | |
| *** EmilienM|afk is now known as EmilienM | 13:22 | |
| *** jjohnson2 has joined #openstack-ironic | 13:37 | |
| *** chenglch has quit IRC | 13:43 | |
| *** vdrok_afk is now known as vdrok | 13:43 | |
| *** lucas-hungry is now known as lucasagomes | 13:54 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic-specs: Replace requests lib to python-scciclient lib https://review.openstack.org/148145 | 13:55 |
| openstackgerrit | John Trowbridge proposed stackforge/ironic-discoverd: Allows passing of config file via --config-file option. https://review.openstack.org/147987 | 13:55 |
| *** ryanpetrello has joined #openstack-ironic | 14:03 | |
| *** rloo has joined #openstack-ironic | 14:03 | |
| *** ryanpetrello has quit IRC | 14:07 | |
| naohirot | good night ironic! | 14:08 |
| *** naohirot has quit IRC | 14:08 | |
| *** enikanorov has joined #openstack-ironic | 14:13 | |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Update README for 1.0.0 release https://review.openstack.org/148267 | 14:20 |
| *** mjturek has joined #openstack-ironic | 14:22 | |
| openstackgerrit | Merged stackforge/ironic-discoverd: Allows passing of config file via --config-file option. https://review.openstack.org/147987 | 14:28 |
| clif_h | morning ironic | 14:33 |
| *** rakesh_hs has quit IRC | 14:34 | |
| dtantsur | clif_h, morning | 14:36 |
| *** ryanpetrello has joined #openstack-ironic | 14:37 | |
| *** gridinv has joined #openstack-ironic | 14:45 | |
| *** ryanpetrello has quit IRC | 14:46 | |
| clif_h | dtantsur: morning! | 14:48 |
| *** jerryz has quit IRC | 14:52 | |
| *** erwan_taf has quit IRC | 14:52 | |
| *** EmilienM is now known as EmilienM|afk | 14:57 | |
| *** Marga_ has quit IRC | 14:59 | |
| *** stelfer1 has joined #openstack-ironic | 15:01 | |
| *** stelfer has quit IRC | 15:02 | |
| *** erwan_taf has joined #openstack-ironic | 15:04 | |
| *** igordcard has joined #openstack-ironic | 15:20 | |
| jroll | gilliard: you'll want to ping devananda and adam_g about backport things | 15:23 |
| jroll | morning everybody :) | 15:23 |
| gilliard | Hi jroll :) | 15:23 |
| dtantsur | jroll, gilliard, morning | 15:24 |
| jroll | \o | 15:24 |
| jroll | did y'all see dlaube's post? https://www.packet.net/blog/how-we-failed-at-openstack | 15:26 |
| lucasagomes | jroll, morning | 15:28 |
| * lucasagomes reads it | 15:28 | |
| jroll | heya lucasagomes :) | 15:28 |
| *** coolsvap is now known as coolsvap|afk | 15:30 | |
| *** jcoufal_ has joined #openstack-ironic | 15:31 | |
| *** jcoufal has quit IRC | 15:33 | |
| openstackgerrit | steverweber proposed stackforge/pyghmi: x# This is a combination of 2 commits. https://review.openstack.org/148291 | 15:35 |
| lucasagomes | if someone has a time to review this patch in the client by jiangfei|3 https://review.openstack.org/#/c/142021/ | 15:39 |
| lucasagomes | it would be good to get it merged soon | 15:39 |
| rloo | hi everyone | 15:42 |
| rloo | lucasagomes: I'll take a look at that patch in a few minutes | 15:42 |
| openstackgerrit | steverweber proposed stackforge/pyghmi: add more commands https://review.openstack.org/148036 | 15:43 |
| lucasagomes | rloo, good morning! Thanks no hurry | 15:43 |
| * rloo sad about dlaube's experience with OpenStack | 15:45 | |
| *** Marga_ has joined #openstack-ironic | 15:45 | |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Update README for 1.0.0 release https://review.openstack.org/148267 | 15:49 |
| *** ifarkas has quit IRC | 15:49 | |
| *** pcrews has joined #openstack-ironic | 15:51 | |
| dtantsur | lucasagomes, would be awesome if both you and Imre reviewed ^^^ as it's going to be discoverd documentation :) | 15:51 |
| lucasagomes | dtantsur, cool, will do | 15:51 |
| *** igordcard has quit IRC | 15:52 | |
| lucasagomes | devananda, hi there, ping re release ironicclient | 15:54 |
| *** EmilienM|afk is now known as EmilienM | 16:08 | |
| *** Marga_ has quit IRC | 16:15 | |
| *** Marga_ has joined #openstack-ironic | 16:17 | |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Update README for 1.0.0 release https://review.openstack.org/148267 | 16:20 |
| *** agordeev has joined #openstack-ironic | 16:20 | |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Update README for 1.0.0 release https://review.openstack.org/148267 | 16:22 |
| openstackgerrit | Merged openstack/python-ironicclient: ironicclient handle faultstring when using SessionClient https://review.openstack.org/142021 | 16:28 |
| openstackgerrit | Merged openstack/ironic: iboot power driver: unbound variable error https://review.openstack.org/147575 | 16:30 |
| *** gridinv has quit IRC | 16:32 | |
| *** yginger5 is now known as yjiang5 | 16:44 | |
| *** erwan_taf has quit IRC | 16:52 | |
| *** jcoufal_ has quit IRC | 16:57 | |
| *** jcoufal has joined #openstack-ironic | 16:57 | |
| *** pensu has joined #openstack-ironic | 16:57 | |
| *** rwsu has joined #openstack-ironic | 16:59 | |
| *** Nisha has joined #openstack-ironic | 17:01 | |
| *** thiagop has joined #openstack-ironic | 17:01 | |
| *** enterprisedc has joined #openstack-ironic | 17:03 | |
| *** enterprisedc has quit IRC | 17:04 | |
| *** coolsvap|afk is now known as coolsvap | 17:10 | |
| devananda | lucasagomes: pong | 17:11 |
| lucasagomes | devananda, hi there... Mind doing another release of the client? | 17:13 |
| lucasagomes | we got all the queue merged | 17:13 |
| devananda | sure | 17:13 |
| lucasagomes | there's a couple of bug fixed since the last release | 17:13 |
| lucasagomes | (4 or 5, will check) | 17:13 |
| lucasagomes | important ones, like the last one fixing the client to show the error messages | 17:13 |
| lucasagomes | and adding support for updating the node with non-string fields (since it now the api supports json types) | 17:14 |
| lucasagomes | devananda, ta much | 17:14 |
| *** stelfer1 has left #openstack-ironic | 17:15 | |
| lucasagomes | yeah 4 bugs fixed (bug/1402840, bug/1403046, bug/1403491 and bug/1402893) | 17:16 |
| devananda | yup, got it from the git log | 17:16 |
| *** romcheg has quit IRC | 17:17 | |
| *** romcheg has joined #openstack-ironic | 17:17 | |
| devananda | lucasagomes: looks like another minor point release, ie 0.3.3 | 17:17 |
| devananda | ya? | 17:17 |
| lucasagomes | oh yeah, +1 minor | 17:17 |
| dtantsur | morning devananda | 17:17 |
| devananda | dtantsur: g'morning | 17:17 |
| *** romcheg has quit IRC | 17:17 | |
| dtantsur | and g'night to everyone :) | 17:17 |
| trown | good night dtantsur | 17:18 |
| *** dtantsur is now known as dtantsur|afk | 17:18 | |
| lucasagomes | dtantsur|afk, night | 17:19 |
| *** eghobo has joined #openstack-ironic | 17:20 | |
| devananda | lucasagomes: tag pushed | 17:20 |
| lucasagomes | devananda, w00t, thanks | 17:21 |
| *** dlaube has joined #openstack-ironic | 17:22 | |
| devananda | adam_g: any thoughts on how we might test the migration from NOSTATE -> AVAILABLE ? | 17:23 |
| jroll | devananda: ensure resources available, upgrade, ensure resources still available? | 17:24 |
| jroll | probably nova boot/delete on both sides of that | 17:24 |
| jroll | make sure the instance deletes when it sees it's available | 17:24 |
| devananda | jroll: so, grenade will do the deploy/delete pre- and post-migrate | 17:26 |
| devananda | jroll: i'm not sure what you mean by "delete when it sees it's available" | 17:27 |
| jroll | nova removes the instance when it sees the node go to NOSTATE yes? | 17:27 |
| devananda | ah. during deletion. yes | 17:27 |
| *** EmilienM is now known as EmilienM|afk | 17:27 | |
| devananda | do we test with a changed nova.virt.ironic driver? or do we test compatibility with the current driver? | 17:27 |
| jroll | mmm | 17:28 |
| devananda | I'd love to test both | 17:28 |
| jroll | yeah | 17:28 |
| jroll | I think we decided nova needs to upgrade first | 17:28 |
| devananda | I think our existing tests will continue to get the "tip + tip" testing | 17:28 |
| devananda | jroll: did we? | 17:29 |
| jroll | otherwise the api has to return NOSTATE until when? | 17:29 |
| jroll | like, when do we change ironic-api to return AVAILABLE | 17:29 |
| devananda | right | 17:29 |
| jroll | the nova driver has to be upgraded by then | 17:29 |
| jroll | and I don't think it's valuable to deprecate that over an entire cycle | 17:29 |
| devananda | most users are still going from distro releases | 17:30 |
| devananda | even if the larger deployments are closer to trunk | 17:30 |
| devananda | we shouldn't completely break juno-nova + kilo-ironic | 17:31 |
| JayF | We support integration with different versioned components? | 17:31 |
| devananda | because someone's going to be in that state during an upgrade | 17:32 |
| jroll | can't they upgrade to kilo-nova before ironic? | 17:32 |
| JayF | So by that logic, it'll be L before we can start using the new provision states, right? | 17:32 |
| devananda | we aspire to | 17:32 |
| devananda | jroll: if kilo-nova can understand old states and new states -- then yes | 17:33 |
| jroll | yes | 17:33 |
| devananda | we just say "upgrade nova first" | 17:33 |
| jroll | it should | 17:33 |
| jroll | yeah. | 17:33 |
| jroll | that's what we talked about doing a long time ago | 17:33 |
| devananda | great. then holidays and other things happened :) | 17:33 |
| devananda | thanks for reminding me | 17:33 |
| jroll | heh | 17:33 |
| jroll | np | 17:33 |
| *** romcheg has joined #openstack-ironic | 17:33 | |
| devananda | so we should plan to start testing tip-nova + juno-ironic | 17:34 |
| devananda | I don't even know if that works today (though I dont see why it wouldn't) | 17:34 |
| devananda | adam_g: ^ | 17:34 |
| *** romcheg has quit IRC | 17:50 | |
| *** romcheg has joined #openstack-ironic | 17:51 | |
| *** afaranha has joined #openstack-ironic | 17:52 | |
| lucasagomes | ok folks I will call it a day | 17:55 |
| lucasagomes | have a good night everyone! | 17:55 |
| * lucasagomes still jetlagged from the trip back home | 17:56 | |
| *** lucasagomes is now known as lucas-dinner | 17:56 | |
| *** derekh has quit IRC | 18:03 | |
| *** pcaruana has quit IRC | 18:08 | |
| *** jcoufal_ has joined #openstack-ironic | 18:14 | |
| *** eghobo has quit IRC | 18:15 | |
| *** jcoufal has quit IRC | 18:17 | |
| *** eghobo has joined #openstack-ironic | 18:18 | |
| *** EmilienM|afk is now known as EmilienM | 18:24 | |
| *** jcoufal_ has quit IRC | 18:30 | |
| Nisha | devananda, related to states....do we plan to introduce states MANAGEABLE and AVAILABLE in kilo? | 18:33 |
| *** openstackgerrit has quit IRC | 18:37 | |
| *** openstackgerrit has joined #openstack-ironic | 18:38 | |
| *** ndipanov has quit IRC | 18:39 | |
| adam_g | devananda, yeah, thats in the works | 18:40 |
| adam_g | devananda, oh wait, misread. | 18:41 |
| adam_g | devananda, i think we'd be able to put together a compat job that tests tip of everything /w ironic N-1 | 18:44 |
| *** achanda has joined #openstack-ironic | 18:46 | |
| *** coolsvap is now known as coolsvap|afk | 18:49 | |
| adam_g | devananda, as for testing the state transitions, im not sure. the current tempest test tests the expected state changes according to the current state diagram, but we may want to consider removing that otherwise we'll have a hard time modifying the state machine, b/c right now its tested as part of the API | 18:49 |
| adam_g | oh actually, currently that compat job would be made impossible by requirement conflicts | 18:49 |
| devananda | adam_g: impossible? | 18:51 |
| adam_g | well, as it is now, devstack installs everything system wide | 18:51 |
| adam_g | so any version constraints applied to stable/juno's requirements.txt that conflict with master's end up in things hitting version conflicts in one place or another | 18:52 |
| adam_g | made up example: juno eventlet <=0.15, master's eventlet >=0.16 | 18:52 |
| adam_g | this is becoming a growing problem tho and may change soon | 18:53 |
| devananda | Nisha: I'm working on AVAILABLE right now. I believe we need to get MANAGEABLE into kilo as well, at a minimum, because that enables both ZAP and INSPECT, which is where a lot of work has been put by several teams | 18:56 |
| devananda | adam_g: I see. so no projects have testing for partial upgrades at this point | 18:57 |
| Nisha | devananda, i proposed following code review for introspection....states related only https://review.openstack.org/147857. | 18:58 |
| Nisha | devananda, for AVAILABLE i am aware it requires changes in the nova ironic virt driver too | 18:59 |
| devananda | Nisha: AVAILABLE will also require changes in many parts of Ironic. I don't have a patch up for it yet, though | 19:00 |
| Nisha | devananda, yes...i am aware...for introspection to work i took the minimum change required which i proposed | 19:01 |
| Nisha | :) | 19:01 |
| devananda | :) | 19:01 |
| adam_g | devananda, actually come to think of it, there *is* a n-cpu partial upgrade test that does an upgrade of everythign except n-cpu. lemme find out how its done there | 19:02 |
| Nisha | devananda, required attention on following spec reviews https://review.openstack.org/134022 and https://review.openstack.org/137024 from core reviewers | 19:02 |
| devananda | Nisha: please continue working on that -- I think we can do it in parallel to the work on AVAILABLE that I'm doing | 19:02 |
| Nisha | devananda, ok. | 19:02 |
| Nisha | devananda, ^^^^ | 19:03 |
| devananda | Nisha: it may end up needing a rebase or two, but I think discovery is fairly separate, aside from the s/NOSTATE/AVAILABLE/ change | 19:03 |
| Nisha | yes | 19:03 |
| Nisha | devananda, correct | 19:03 |
| *** dprince has quit IRC | 19:04 | |
| *** pelix has quit IRC | 19:05 | |
| devananda | Nisha: are those two specs related to INSPECT state? | 19:05 |
| *** achanda has quit IRC | 19:05 | |
| Nisha | devananda, No. Sorry but this is off-topic here.... | 19:06 |
| devananda | k | 19:06 |
| Nisha | these are uefi related changes | 19:06 |
| Nisha | but specs are still pending... | 19:06 |
| devananda | gotcha | 19:07 |
| devananda | Nisha: for https://review.openstack.org/#/c/134022/4/specs/kilo/automate-uefi-bios-iso-creation.rst - is there a reason that this image can not be shared across many nodes? | 19:08 |
| Nisha | it can be shared but as of now one has to create it manually and upload and update the glance image to be deployed with the property "boot_iso" which contains boot_iso uuid | 19:09 |
| Nisha | with this change this manual step is avoided | 19:10 |
| *** achanda has joined #openstack-ironic | 19:10 | |
| Nisha | since boot_iso need to be of the same kernel and ramdisk which belongs to that OS (which is being deployed), it is better to automate it | 19:11 |
| Nisha | In juno we faced some issues , hence backed out the changes at the last minute for automation part | 19:11 |
| Nisha | devananda, needed reviews on https://review.openstack.org/103007 (ilo spec for introspection) too....got +2 from Dmitry...awaiting others reviews | 19:16 |
| *** dprince has joined #openstack-ironic | 19:21 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-specs: Discover node properties for iLO drivers https://review.openstack.org/103007 | 19:25 |
| devananda | Nisha: is the problem with https://review.openstack.org/#/c/137024/3/specs/kilo/uefi-support-for-agent-ilo-driver.rst merely that the driver doesn't switch the boot mode, or is it more than that? | 19:27 |
| Nisha | devananda, it is that only | 19:28 |
| devananda | Nisha: could you clarify "The iso creation script in the IPA needs to be enhanced for UEFI mode." ? | 19:28 |
| Nisha | the deploy iso and the image needs to be uefi capable | 19:28 |
| devananda | Nisha: sure. but how is that related to the driver? | 19:29 |
| devananda | afaik, today the driver isn't creating either of those images -- just pulling from glance | 19:29 |
| devananda | Nisha: oh. are you saying that IPA's image building tools don't create UEFI-capable images? | 19:30 |
| Nisha | devananda, there is a script hosted in ironic-python-agent/imagebuild/coreos/iso-image-create which is only capable to generate iso for bios | 19:30 |
| devananda | Nisha: I see. what about the diskimage-builder work for IPA that NobodyCam was doing? | 19:31 |
| devananda | Nisha: would that help? | 19:31 |
| Nisha | devananda, yes and when i tried to enhance that script it didnt work for me for ubuntu/cirros images.... but DIB is capable to buiild IPA deploy iso for fedora alone ...the deploy iso works....image i still need to test because my image creation for fedora fails | 19:32 |
| Nisha | devananda, deploy iso worked only for fedora :( | 19:33 |
| Nisha | using DIB | 19:33 |
| Nisha | when i enhance the existing script it doesnt work for agent drivers...while the same script works fine for iscsi_ilo :( | 19:34 |
| Nisha | devananda, so i was thinking i will go with DIB for now...but it definitely requires enhancement in IPA/DIB to support uefi images for agent drivers | 19:35 |
| rloo | hi devananda, when you get a minute, this BP's spec has been approved: https://blueprints.launchpad.net/ironic/+spec/ironic-virtualbox-webservice-support | 19:37 |
| devananda | rloo: ty | 19:38 |
| Nisha | devananda, if DIB creates UEFI deploy ISO and images for IPA, we are fine to use that | 19:39 |
| Nisha | because both need to be uefi capable | 19:39 |
| Nisha | for local boot | 19:39 |
| devananda | Nisha: for ilo_agent to set the node's boot mode, it looks like you only need a very small change in modules/ilo/deplpy.py | 19:41 |
| Nisha | yes | 19:41 |
| Nisha | thats done and it works | 19:41 |
| Nisha | ust need to raise code review after spec is in | 19:42 |
| devananda | Nisha: great. fwiw, I don't think that needs a spec, but you have one up now, so ... :) | 19:42 |
| Nisha | :) i raised it as initially we required script enhancement too :) | 19:44 |
| devananda | ahh | 19:44 |
| Nisha | devananda, was going thru your comments on the spec automation of boot_iso | 19:45 |
| devananda | Nisha: would you be horribly upset if I said, this is too small for a spec, and these are just two separate bugs? | 19:45 |
| Nisha | devananda, just to understand how rebuild differs from deploy | 19:45 |
| Nisha | No i can put them as bug fixes | 19:46 |
| Nisha | :) | 19:46 |
| devananda | Nisha: I'll happily review the change for IloVirtualMediaAgentDeploy.prepare right now -- I think it'll only be about 5 lines of code, and a small unit test | 19:46 |
| devananda | which would take less time to review than the spec did :) | 19:46 |
| Nisha | devananda, so nice of now....will post the patch soon..not now as our devstack systems are down :( | 19:47 |
| Nisha | devananda, i will push the patch soon | 19:48 |
| Nisha | devananda, just to understand how node rebuild differs from deploy | 19:48 |
| devananda | Nisha: "nova rebuild" preserves the instance UUID in nova, as well as network allocation | 19:48 |
| devananda | Nisha: it triggers a delete & recreate in Ironic | 19:49 |
| devananda | Nisha: and a user may optionally change the instance image during a rebuild | 19:49 |
| Nisha | devananda, then it should be fine because itpicks up the kernel and ramdisk uuid from the image in the code ... | 19:50 |
| devananda | k k | 19:50 |
| Nisha | so for it should be same as a fresh deploy | 19:50 |
| *** Marga_ has quit IRC | 20:01 | |
| *** Marga_ has joined #openstack-ironic | 20:02 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-specs: Automate UEFI-BIOS Iso Creation https://review.openstack.org/134022 | 20:06 |
| Nisha | devananda, ^^^^ | 20:06 |
| Nisha | devananda, for ilo-ipa uefi support do i need to raise two bugs to push the code changes? | 20:08 |
| devananda | Nisha: for tracking, a very simple "ilo_agent deploy driver does not set boot mode properly" bug would be good. | 20:09 |
| Nisha | ok | 20:09 |
| devananda | so that it is captured in automated release notes | 20:09 |
| Nisha | k | 20:09 |
| *** andreykurilin_ has joined #openstack-ironic | 20:10 | |
| Nisha | devananda, raised the bugs | 20:19 |
| *** pensu has quit IRC | 20:21 | |
| *** jistr has quit IRC | 20:23 | |
| Nisha | good night folks | 20:28 |
| Nisha | bye | 20:28 |
| *** achanda has quit IRC | 20:31 | |
| *** Nisha has quit IRC | 20:33 | |
| *** Marga_ has quit IRC | 20:41 | |
| *** Marga_ has joined #openstack-ironic | 20:42 | |
| *** dprince has quit IRC | 20:54 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Consistent way to set details for Error instances https://review.openstack.org/146924 | 20:58 |
| openstackgerrit | Merged openstack/ironic-python-agent: Make all IPA error classes inherit from RESTError https://review.openstack.org/148014 | 21:00 |
| *** achanda has joined #openstack-ironic | 21:02 | |
| *** achanda_ has joined #openstack-ironic | 21:03 | |
| *** achanda has quit IRC | 21:07 | |
| *** romcheg1 has joined #openstack-ironic | 21:17 | |
| *** lucas-dinner has quit IRC | 21:18 | |
| mrda | Morning Ironic | 21:18 |
| *** romcheg has quit IRC | 21:19 | |
| jroll | heya mrda :) | 21:30 |
| openstackgerrit | Clif Houck proposed openstack/ironic: Fix RPCService and Ironic Conductor so they shut down gracefully https://review.openstack.org/147325 | 21:31 |
| openstackgerrit | Merged openstack/ironic-specs: Automate UEFI-BIOS Iso Creation https://review.openstack.org/134022 | 21:32 |
| JayF | devananda: https://review.openstack.org/#/c/117376 - conversation happening here about if Ironic should be in Tuskar/Horizon/both | 21:42 |
| JayF | devananda: seemed relevant to your interests, so wanted to point you at it | 21:42 |
| openstackgerrit | Clif Houck proposed openstack/ironic: Fix RPCService and Ironic Conductor so they shut down gracefully https://review.openstack.org/147325 | 21:43 |
| *** dtantsur|afk has quit IRC | 21:54 | |
| *** lsmola_ has quit IRC | 21:55 | |
| *** dtantsur has joined #openstack-ironic | 21:59 | |
| devananda | JayF: ty | 22:07 |
| *** lsmola_ has joined #openstack-ironic | 22:08 | |
| devananda | anyone planning to attend the meeting tonight? | 22:08 |
| JayF | I was going to, although I have nothing in particular for it. | 22:08 |
| JoshNang | i was going to see if anyone wanted to take on horizon work :) | 22:09 |
| *** alexm__ has joined #openstack-ironic | 22:09 | |
| jroll | devananda: I'll be around | 22:10 |
| devananda | k k | 22:10 |
| jroll | also, that horizon/tuskar question is interesting | 22:10 |
| jroll | as horizon is intended for users AIUI | 22:11 |
| jroll | ironic is not intended for users | 22:11 |
| devananda | does horizon have any admin-only panels? | 22:11 |
| JoshNang | right. and users can still provision ironic nodes via nova flavors. | 22:11 |
| jroll | no idea. | 22:11 |
| JoshNang | not sure. | 22:11 |
| devananda | re: tuskar, from their readme: Tuskar UI is a user interface for Tuskar, a management API for OpenStack deployments. It is a plugin for OpenStack Horizon. | 22:11 |
| jroll | right | 22:12 |
| devananda | so it makes sense that needs some panel for ironic's resources | 22:12 |
| *** jjohnson2 has quit IRC | 22:12 | |
| jroll | I'm not saying tuskar is the right place for this, btw | 22:12 |
| jroll | because ironic isn't just for deploying openstack | 22:12 |
| devananda | but someone who is not using TripleO should *also* have access to such panels, IMHO | 22:12 |
| devananda | right | 22:12 |
| jroll | right | 22:12 |
| devananda | http://docs.openstack.org/user-guide-admin/content/log_in_dashboard.html | 22:15 |
| devananda | admin tab. right there. | 22:15 |
| devananda | seems like it should fit in fine | 22:15 |
| devananda | it == an ironic tab | 22:16 |
| jroll | yeah, agree. | 22:16 |
| JoshNang | wfm | 22:16 |
| devananda | comments added | 22:17 |
| *** ryanpetrello has joined #openstack-ironic | 22:27 | |
| *** rwsu has quit IRC | 22:35 | |
| *** krtaylor has quit IRC | 22:36 | |
| *** rwsu has joined #openstack-ironic | 22:36 | |
| *** rwsu has quit IRC | 22:36 | |
| *** achanda_ has quit IRC | 22:42 | |
| *** ryanpetrello has quit IRC | 22:44 | |
| *** romcheg has joined #openstack-ironic | 22:46 | |
| *** romcheg1 has quit IRC | 22:48 | |
| *** achanda has joined #openstack-ironic | 22:54 | |
| *** krtaylor has joined #openstack-ironic | 22:55 | |
| *** kbyrne has quit IRC | 22:57 | |
| *** achanda has quit IRC | 22:59 | |
| *** Marga_ has quit IRC | 23:01 | |
| openstackgerrit | Adam Gandelman proposed openstack/ironic: Simplify policy.json https://review.openstack.org/148381 | 23:06 |
| *** achanda has joined #openstack-ironic | 23:09 | |
| *** ryanpetrello has joined #openstack-ironic | 23:09 | |
| *** kbyrne has joined #openstack-ironic | 23:11 | |
| *** chlong has joined #openstack-ironic | 23:11 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Clean up ilo's parse_driver_info() https://review.openstack.org/148382 | 23:14 |
| *** ryanpetrello has quit IRC | 23:22 | |
| *** ijw has joined #openstack-ironic | 23:31 | |
| *** achanda has quit IRC | 23:35 | |
| *** achanda has joined #openstack-ironic | 23:40 | |
| *** EmilienM is now known as EmilienM|afk | 23:52 | |
| openstackgerrit | Hadi Bannazadeh proposed openstack/ironic: region missing in endpoint selection https://review.openstack.org/148128 | 23:54 |
| openstackgerrit | Hadi Bannazadeh proposed openstack/ironic: region missing in endpoint selection https://review.openstack.org/148128 | 23:57 |
| *** ryanpetrello has joined #openstack-ironic | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!