| jroll | the latest code seems to be handling it | 00:00 |
|---|---|---|
| jroll | but it looks like it only handles portgroups, maybe? | 00:00 |
| jroll | line 193 https://review.openstack.org/#/c/139687/30/ironic/networks/neutron_plugin.py | 00:00 |
| jroll | so I think the portgroup vs ports thing is the kicker | 00:01 |
| jroll | er, no, it handles ports later | 00:02 |
| *** naohirot has joined #openstack-ironic | 00:02 | |
| jroll | I'd like to take some time to hack on this.... so little time | 00:02 |
| Sukhdev | yup - I was about to say that | 00:02 |
| openstackgerrit | Merged openstack/ironic: Add testresources and testscenarios used by oslo.db fixture https://review.openstack.org/233719 | 00:03 |
| jroll | this appears like it should work | 00:03 |
| jroll | I'd get it to the point where it's trying to boot, and see what those ports look like | 00:03 |
| Sukhdev | yup - | 00:03 |
| jroll | jlvillal: gate is fixed \o/ | 00:04 |
| jlvillal | jroll: Woo hoo :) | 00:04 |
| jroll | and etherpad is hanging /o\ | 00:04 |
| jlvillal | jroll: I posted a question: https://review.openstack.org/#/c/231215/6..8/ironic/tests/unit/drivers/modules/test_agent.py | 00:04 |
| Sukhdev | I think this part is OK - I think the OVS underneath may be screwing up | 00:04 |
| jroll | Sukhdev: maybe, but this part is much easier to verify :) | 00:04 |
| jroll | jlvillal: yeah, probably, can you -1 me for that pls | 00:05 |
| jlvillal | jroll: Will do | 00:05 |
| jroll | actually I'll fix it nowe | 00:05 |
| jroll | now | 00:05 |
| Sukhdev | jroll: Yes, I will verify this - it takes me few seconds to verify this | 00:05 |
| jroll | cool | 00:06 |
| jroll | just seems weird to me that ovs would break that way | 00:07 |
| *** opal has joined #openstack-ironic | 00:07 | |
| *** kozhukalov_ has quit IRC | 00:07 | |
| Sukhdev | jroll: I just verified - the extra gets filled correctly with both ports | 00:09 |
| Sukhdev | i mean neutron ports | 00:09 |
| jroll | Sukhdev: and those are the ports on the provisioning network? | 00:09 |
| jroll | I have to run after I get this patch up, unfortunately | 00:10 |
| *** achanda has joined #openstack-ironic | 00:11 | |
| jroll | jlvillal: incoming | 00:12 |
| jroll | I'm out, have a good night all :) | 00:12 |
| jlvillal | jroll: Ciao | 00:12 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Fix AgentDeploy compatibility with pre-Liberty drivers https://review.openstack.org/231215 | 00:12 |
| *** achanda has quit IRC | 00:13 | |
| *** opal has left #openstack-ironic | 00:18 | |
| *** shadower has quit IRC | 00:23 | |
| *** shadower has joined #openstack-ironic | 00:23 | |
| *** Marga_ has quit IRC | 00:27 | |
| *** garthb has quit IRC | 00:29 | |
| *** mrmoje has quit IRC | 00:30 | |
| jlvillal | mrda: Are you around? | 00:31 |
| jlvillal | devananda: Are you around? | 00:32 |
| * devananda is here, reading backscroll | 00:34 | |
| *** Marga_ has joined #openstack-ironic | 00:35 | |
| devananda | jlvillal: what's up? | 00:36 |
| jlvillal | devananda: Question at: https://review.openstack.org/#/c/231215/9/ironic/tests/unit/drivers/modules/test_agent.py | 00:36 |
| devananda | Sukhdev: trying to follow your discussion with jroll - but not sure where you're stuck. | 00:36 |
| jlvillal | My comment near bottom | 00:36 |
| devananda | Sukhdev: I can try to help for a bit if you still are stuck | 00:36 |
| *** penick has joined #openstack-ironic | 00:38 | |
| jlvillal | devananda: Not sure if you saw my question while you were reading the back-scroll. https://review.openstack.org/#/c/231215/9/ironic/tests/unit/drivers/modules/test_agent.py | 00:41 |
| devananda | jlvillal: yea, looking at that code noe | 00:42 |
| devananda | now | 00:42 |
| jlvillal | devananda: Thanks | 00:42 |
| *** Marga_ has quit IRC | 00:43 | |
| devananda | jlvillal: I believe the test is ensuring that all tests in the TestAgentDeploy class pass, whether driver.boot is None or FakeBoot() | 00:43 |
| devananda | jlvillal: your assertion failed because, in the parent class, driver.boot is None | 00:44 |
| *** sinval_ has joined #openstack-ironic | 00:44 | |
| devananda | this class is overriding the setUp() method to set it to an instance of FakeBoot -- and then run throug all the test cases | 00:44 |
| jlvillal | devananda: It failed because it doesn't exist, not that it was none. I believe. | 00:44 |
| mrda | jlvillal: I'm here now, sorry was in a meeting | 00:44 |
| jlvillal | devananda: AttributeError: 'AgentDeploy' object has no attribute 'boot' | 00:44 |
| jlvillal | devananda: So why I was confused... | 00:45 |
| *** Marga_ has joined #openstack-ironic | 00:45 | |
| sinval_ | hey guys, good morning all | 00:45 |
| sinval_ | are we facing some problems with the gate? | 00:45 |
| jlvillal | mrda: Thanks. devananda is helping to answer my questions. I was confused about: https://review.openstack.org/#/c/231215/9/ironic/tests/unit/drivers/modules/test_agent.py | 00:46 |
| jlvillal | sinval_: Gate is fixed | 00:46 |
| jlvillal | s/I was confused/I am confused/ :) | 00:46 |
| sinval_ | jlvillal, thanks for the quick update | 00:46 |
| jlvillal | sinval_: Feel free to 'recheck' the OneView patch now. | 00:46 |
| sinval_ | jlvillal: great idea :) | 00:47 |
| * jlvillal is going to go home. Back online in 15-20 minutes... | 00:47 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/233819 | 00:52 |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Refactor IRMCVirtualMediaIscsiDeploy by applying new BootInterface https://review.openstack.org/221371 | 00:55 |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Refactor IRMCVirtualMediaAgentDeploy by applying new BootInterface https://review.openstack.org/221577 | 00:57 |
| lintan_ | Morning Ironic | 01:04 |
| jlvillal | lintan_, Hello | 01:04 |
| lintan_ | morning, jlvillal | 01:05 |
| *** Sukhdev has quit IRC | 01:05 | |
| *** meghal has quit IRC | 01:09 | |
| devananda | jlvillal: hah! so. self.driver is NOT a driver | 01:19 |
| devananda | it's an instance of AgentDeploy() interface class | 01:19 |
| *** Marga_ has quit IRC | 01:19 | |
| jlvillal | devananda, Yeah | 01:21 |
| *** Marga_ has joined #openstack-ironic | 01:21 | |
| jlvillal | devananda, The test code is confusing because it says self.driver = ... But not a driver. | 01:21 |
| devananda | yup | 01:21 |
| *** Marga_ has quit IRC | 01:23 | |
| jlvillal | devananda, The real driver seems to be done retrieved in the the 'with task_manager.acquire()' calls. As it is inside task.driver. | 01:23 |
| devananda | indeed | 01:23 |
| jlvillal | s/be done/be/ | 01:23 |
| devananda | jlvillal: in looking at this class, I don't actually see the usefulness of self.driver | 01:27 |
| *** Marga_ has joined #openstack-ironic | 01:28 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add db migration and model for tags table https://review.openstack.org/192576 | 01:29 |
| *** Marga__ has joined #openstack-ironic | 01:29 | |
| jlvillal | devananda, Okay. I haven't looked at it that much to know either way. | 01:29 |
| *** Marga_ has quit IRC | 01:29 | |
| * jlvillal is watching Monday Night Football... | 01:30 | |
| *** achanda has joined #openstack-ironic | 01:31 | |
| *** baoli has joined #openstack-ironic | 01:32 | |
| *** zhenguo has joined #openstack-ironic | 01:33 | |
| *** mtanino has quit IRC | 01:34 | |
| *** thrash is now known as thrash|g0ne | 01:36 | |
| sinval_ | jlvillal: go Steelers!!! | 01:36 |
| jlvillal | sinval_, :) | 01:37 |
| *** Marga__ has quit IRC | 01:54 | |
| *** praneshp has quit IRC | 01:58 | |
| *** Marga_ has joined #openstack-ironic | 01:59 | |
| devananda | jlvillal: welp. looking deeper and I'm pretty sure this wasn't testing what I thought it was testing | 02:00 |
| devananda | jroll: I'm rewriting the tests for the boot compat patch | 02:00 |
| *** Haomeng has joined #openstack-ironic | 02:00 | |
| Haomeng | zhenguo: ping | 02:05 |
| zhenguo | Haomeng: hi | 02:05 |
| Haomeng | zhenguo: https://review.openstack.org/#/c/164586/20/specs/approved/console-log.rst | 02:05 |
| Haomeng | zhenguo: let me check | 02:05 |
| zhenguo | Haomeng: thanks | 02:05 |
| *** baoli has quit IRC | 02:05 | |
| Haomeng | zhenguo: yes, it is created by IBM guy who is Cheng Long | 02:06 |
| Haomeng | zhenguo: do you have any concern, let me know | 02:06 |
| Haomeng | zhenguo: I will invate Cheng Long into this irc if needed | 02:06 |
| zhenguo | Haomeng: Some third party tools like ``conserver``, ``conman``, ``confluent` in the specs, but not specify which one we should use. | 02:07 |
| zhenguo | Haomeng: and does he plan to replace shellinbox with confluent? | 02:08 |
| Haomeng | zhenguo: ok, it is free to add comments, so can you just review and add comnments to address your concern? | 02:08 |
| zhenguo | Haomeng: ok, | 02:08 |
| Haomeng | zhenguo: he is owner, so it is better that we discuss with him directly, let me find him | 02:08 |
| zhenguo | Haomeng: thanks | 02:09 |
| Haomeng | zhenguo: he will logon irc soon:) | 02:10 |
| *** chenglch has joined #openstack-ironic | 02:10 | |
| zhenguo | chenglch: hi | 02:10 |
| chenglch | zhenguo: hi | 02:11 |
| zhenguo | chenglch: want to discuss something about your console log spec | 02:11 |
| zhenguo | chenglch: do you plan to replace shellinbox with confluent? | 02:12 |
| devananda | jroll: i'm not sure how I didn't catch this sooner -- 7 of the unit tests for TestAgentDeploy depend *directly* on PXEBoot | 02:12 |
| chenglch | zhenguo: not replace, but hope to provide additional method for the console access | 02:12 |
| zhenguo | chenglch: so both shellinbox and confluent demon will work? | 02:13 |
| *** ijw has joined #openstack-ironic | 02:14 | |
| zhenguo | chenglch: and I find you said conserver, conman, and confluent in the specs, which one do you want to choose? | 02:15 |
| chenglch | zhenguo: I hope to add confluent as a console driver, so depends on which driver we hope to use. | 02:15 |
| *** penick has quit IRC | 02:15 | |
| zhenguo | chenglch: ok so we can specify whether shellinbox or confluent, but only confluent can support both console access and console log, right? | 02:16 |
| chenglch | zhenguo: confluent has some limitation, as it depends on pyghmi, conserver is flexable, it support ipmi, ssh, etc. | 02:16 |
| chenglch | zhenguo: yes , only confluent support console access and console log | 02:17 |
| zhenguo | chenglch: so conserver is better? I think you should specify which tool we should use in the specs | 02:18 |
| chenglch | zhenguo: but conserver can work with shellinabox to support both console access and console log | 02:18 |
| *** sinval_ has quit IRC | 02:19 | |
| zhenguo | chenglch: that's good, but if conserver can work with shellinabox, that's mean we don't need to add a separate console driver, right? | 02:21 |
| chenglch | zhenguo: yes | 02:21 |
| zhenguo | chenglch: ok thanks, | 02:22 |
| *** Marga_ has quit IRC | 02:23 | |
| *** Marga_ has joined #openstack-ironic | 02:27 | |
| *** links has joined #openstack-ironic | 02:28 | |
| *** dims_ has quit IRC | 02:28 | |
| *** derekh has quit IRC | 02:29 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Fix AgentDeploy compatibility with pre-Liberty drivers https://review.openstack.org/231215 | 02:37 |
| devananda | jroll, jlvillal: ^ | 02:37 |
| devananda | significant rewrite of the unit tests in that revision | 02:38 |
| *** dims__ has joined #openstack-ironic | 02:41 | |
| *** puranamr has joined #openstack-ironic | 02:53 | |
| *** dims__ has quit IRC | 02:59 | |
| openstackgerrit | Merged openstack/ironic: OneView Driver for Ironic https://review.openstack.org/191822 | 03:05 |
| *** puranamr has quit IRC | 03:08 | |
| *** puranamr has joined #openstack-ironic | 03:17 | |
| *** mdbooth has quit IRC | 03:22 | |
| *** puranamr has quit IRC | 03:23 | |
| *** ijw has quit IRC | 03:24 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Fix AgentDeploy compatibility with pre-Liberty drivers https://review.openstack.org/231215 | 03:24 |
| *** mdbooth has joined #openstack-ironic | 03:28 | |
| *** VikasC has joined #openstack-ironic | 03:29 | |
| *** rameshg87 has joined #openstack-ironic | 03:32 | |
| *** Marga_ has quit IRC | 03:44 | |
| *** amotoki has joined #openstack-ironic | 03:44 | |
| *** chenglch|2 has joined #openstack-ironic | 03:48 | |
| *** chenglch has quit IRC | 03:50 | |
| *** amotoki has quit IRC | 03:51 | |
| *** amotoki has joined #openstack-ironic | 03:55 | |
| *** links has quit IRC | 03:58 | |
| *** dims__ has joined #openstack-ironic | 03:59 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Fixes logging of failure in deletion of swift temporary object https://review.openstack.org/232440 | 04:05 |
| *** dims__ has quit IRC | 04:06 | |
| *** MattMan has quit IRC | 04:09 | |
| *** MattMan has joined #openstack-ironic | 04:10 | |
| *** praneshp has joined #openstack-ironic | 04:13 | |
| *** amotoki has quit IRC | 04:18 | |
| *** Sukhdev has joined #openstack-ironic | 04:20 | |
| *** amotoki has joined #openstack-ironic | 04:29 | |
| *** amotoki_ has joined #openstack-ironic | 04:29 | |
| *** early` has quit IRC | 04:30 | |
| *** pradipta has joined #openstack-ironic | 04:33 | |
| *** opal has joined #openstack-ironic | 04:35 | |
| *** early has joined #openstack-ironic | 04:36 | |
| *** amotoki_ has quit IRC | 04:36 | |
| *** mmedvede has quit IRC | 04:41 | |
| *** mmedvede has joined #openstack-ironic | 04:41 | |
| *** opal has left #openstack-ironic | 04:43 | |
| *** yog_ has joined #openstack-ironic | 04:49 | |
| *** Marga_ has joined #openstack-ironic | 04:50 | |
| *** Marga_ has quit IRC | 04:53 | |
| *** Marga_ has joined #openstack-ironic | 04:53 | |
| *** links has joined #openstack-ironic | 05:06 | |
| *** mrmoje has joined #openstack-ironic | 05:08 | |
| *** praneshp_ has joined #openstack-ironic | 05:13 | |
| *** praneshp has quit IRC | 05:16 | |
| *** praneshp_ is now known as praneshp | 05:16 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic-specs: iLO virtual media drivers to deploy without DHCP https://review.openstack.org/137567 | 05:25 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/233819 | 05:26 |
| *** Marga_ has quit IRC | 05:28 | |
| *** yog_ has quit IRC | 05:30 | |
| *** Sukhdev_ has joined #openstack-ironic | 05:35 | |
| *** Sukhdev_ has quit IRC | 05:36 | |
| *** itamarl has joined #openstack-ironic | 05:37 | |
| *** Sukhdev has quit IRC | 05:37 | |
| *** praneshp has quit IRC | 05:46 | |
| *** yog_ has joined #openstack-ironic | 05:47 | |
| openstackgerrit | Tan Lin proposed openstack/ironic: Validate the input of properties of nodes. https://review.openstack.org/215505 | 05:59 |
| *** dims__ has joined #openstack-ironic | 06:03 | |
| *** dims__ has quit IRC | 06:09 | |
| *** yog_ has quit IRC | 06:10 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: ucs_hostname changed to ucs_address https://review.openstack.org/232834 | 06:18 |
| *** links has quit IRC | 06:20 | |
| *** openstack has joined #openstack-ironic | 06:29 | |
| *** amotoki has joined #openstack-ironic | 06:30 | |
| *** achanda has quit IRC | 06:46 | |
| *** achanda has joined #openstack-ironic | 06:48 | |
| openstackgerrit | Haomeng,Wang proposed openstack/ironic: add python-tox into dev-quickstart doc https://review.openstack.org/233989 | 06:49 |
| *** amotoki has quit IRC | 06:54 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add db api layer for CRUD operations on node tags https://review.openstack.org/232598 | 06:54 |
| openstackgerrit | Haomeng,Wang proposed openstack/ironic: add python-tox into dev-quickstart doc https://review.openstack.org/233989 | 06:54 |
| *** pas-ha has joined #openstack-ironic | 06:55 | |
| *** e0ne has joined #openstack-ironic | 06:58 | |
| *** e0ne has quit IRC | 06:59 | |
| *** kozhukalov_ has joined #openstack-ironic | 07:00 | |
| *** yog_ has joined #openstack-ironic | 07:02 | |
| *** dims__ has joined #openstack-ironic | 07:05 | |
| *** dims__ has quit IRC | 07:10 | |
| *** links has joined #openstack-ironic | 07:13 | |
| *** ndipanov has quit IRC | 07:13 | |
| *** e0ne has joined #openstack-ironic | 07:16 | |
| *** achanda has quit IRC | 07:20 | |
| *** mrmoje has quit IRC | 07:22 | |
| *** mrmoje has joined #openstack-ironic | 07:23 | |
| *** xek_ is now known as xek | 07:29 | |
| *** kozhukalov_ has quit IRC | 07:33 | |
| *** amotoki has joined #openstack-ironic | 07:34 | |
| *** aarefiev22 has joined #openstack-ironic | 07:37 | |
| *** amotoki has quit IRC | 07:42 | |
| *** e0ne has quit IRC | 07:44 | |
| *** kozhukalov_ has joined #openstack-ironic | 07:45 | |
| *** amotoki has joined #openstack-ironic | 07:50 | |
| *** persia has quit IRC | 07:50 | |
| *** persia has joined #openstack-ironic | 07:51 | |
| *** subscope has joined #openstack-ironic | 07:52 | |
| *** ifarkas has joined #openstack-ironic | 07:54 | |
| *** jistr has joined #openstack-ironic | 08:03 | |
| *** dims__ has joined #openstack-ironic | 08:06 | |
| *** jcoufal has joined #openstack-ironic | 08:06 | |
| *** romainh has joined #openstack-ironic | 08:06 | |
| *** mrmoje has quit IRC | 08:07 | |
| vdrok | morning ironic | 08:07 |
| *** links has quit IRC | 08:08 | |
| Haomeng | vdrok: good morning:) | 08:09 |
| vdrok | morning Haomeng | 08:09 |
| Haomeng | vdrok: :) | 08:09 |
| *** amotoki has quit IRC | 08:10 | |
| *** dims__ has quit IRC | 08:11 | |
| *** derekh has joined #openstack-ironic | 08:11 | |
| *** lsmola_ has joined #openstack-ironic | 08:11 | |
| *** aarefiev22 has quit IRC | 08:12 | |
| *** pas-ha has quit IRC | 08:13 | |
| *** mrmoje has joined #openstack-ironic | 08:14 | |
| *** athomas has quit IRC | 08:14 | |
| *** athomas has joined #openstack-ironic | 08:15 | |
| *** pas-ha has joined #openstack-ironic | 08:16 | |
| *** amotoki has joined #openstack-ironic | 08:17 | |
| *** links has joined #openstack-ironic | 08:18 | |
| *** ndipanov has joined #openstack-ironic | 08:21 | |
| *** subscope has quit IRC | 08:23 | |
| *** links has quit IRC | 08:24 | |
| *** subscope has joined #openstack-ironic | 08:24 | |
| *** amotoki has quit IRC | 08:25 | |
| *** mgoddard has joined #openstack-ironic | 08:25 | |
| *** mbound has joined #openstack-ironic | 08:26 | |
| *** derekh has quit IRC | 08:26 | |
| *** lucasagomes has joined #openstack-ironic | 08:29 | |
| *** derekh has joined #openstack-ironic | 08:29 | |
| *** amotoki has joined #openstack-ironic | 08:29 | |
| *** pelix has joined #openstack-ironic | 08:31 | |
| *** links has joined #openstack-ironic | 08:31 | |
| *** openstack has joined #openstack-ironic | 08:48 | |
| *** amotoki has joined #openstack-ironic | 08:49 | |
| *** mdbooth has joined #openstack-ironic | 08:51 | |
| *** BharatK_ has quit IRC | 08:52 | |
| *** mkovacik has quit IRC | 08:54 | |
| *** mkovacik has joined #openstack-ironic | 08:54 | |
| *** links has quit IRC | 08:54 | |
| *** links has joined #openstack-ironic | 08:54 | |
| *** marzif has joined #openstack-ironic | 08:57 | |
| *** subscope has quit IRC | 09:03 | |
| *** BharatK_ has joined #openstack-ironic | 09:06 | |
| *** dims__ has joined #openstack-ironic | 09:08 | |
| *** dims__ has quit IRC | 09:13 | |
| * mkovacik is away: lunch | 09:13 | |
| *** links has quit IRC | 09:15 | |
| *** subscope has joined #openstack-ironic | 09:15 | |
| *** amotoki has quit IRC | 09:20 | |
| *** amotoki has joined #openstack-ironic | 09:21 | |
| smoriya | good morning, ironic | 09:24 |
| smoriya | rameshg87: around here? | 09:24 |
| *** amotoki has quit IRC | 09:25 | |
| sambetts | Morning all o/ | 09:29 |
| smoriya | morning, sambetts | 09:29 |
| sambetts | o/ smoriya | 09:29 |
| *** openstackstatus has joined #openstack-ironic | 09:38 | |
| *** ChanServ sets mode: +v openstackstatus | 09:38 | |
| *** amotoki has joined #openstack-ironic | 09:38 | |
| *** stendulker has quit IRC | 09:38 | |
| *** Marga_ has joined #openstack-ironic | 09:41 | |
| -openstackstatus- NOTICE: gerrit is undergoing an emergency restart to investigate load issues | 09:42 | |
| *** ChanServ changes topic to "gerrit is undergoing an emergency restart to investigate load issues" | 09:42 | |
| *** dims__ has joined #openstack-ironic | 09:49 | |
| *** marzif has quit IRC | 09:50 | |
| *** chenglch|2 has quit IRC | 09:50 | |
| *** marzif has joined #openstack-ironic | 09:51 | |
| *** mrmoje has quit IRC | 09:51 | |
| *** BharatK_ has quit IRC | 09:52 | |
| *** smoriya has quit IRC | 09:53 | |
| lucasagomes | smoriya, sambetts morning | 09:54 |
| sambetts | o/ lucasagomes | 09:56 |
| vdrok | morning sambetts lucasagomes | 09:56 |
| lucasagomes | vdrok, hey hey morning | 09:57 |
| *** naohirot has quit IRC | 09:57 | |
| *** e0ne has joined #openstack-ironic | 09:58 | |
| vdrok | oh, gerrit is down | 10:00 |
| *** zhenguo has quit IRC | 10:00 | |
| sambetts | yup :( | 10:00 |
| *** openstackgerrit has quit IRC | 10:01 | |
| *** openstackgerrit has joined #openstack-ironic | 10:02 | |
| *** BharatK_ has joined #openstack-ironic | 10:04 | |
| *** amotoki has quit IRC | 10:06 | |
| *** Marga_ has quit IRC | 10:09 | |
| *** marzif has quit IRC | 10:10 | |
| *** marzif has joined #openstack-ironic | 10:11 | |
| *** dims__ is now known as dims | 10:18 | |
| *** cemason has quit IRC | 10:20 | |
| lucasagomes | damn | 10:21 |
| lucasagomes | it's back | 10:22 |
| *** cemason has joined #openstack-ironic | 10:22 | |
| *** amotoki has joined #openstack-ironic | 10:26 | |
| *** amotoki has quit IRC | 10:28 | |
| *** yog_ has quit IRC | 10:31 | |
| *** getvasanth has joined #openstack-ironic | 10:33 | |
| *** ionutbalutoiu has joined #openstack-ironic | 10:47 | |
| rameshg87 | smori<tab> | 10:49 |
| ionutbalutoiu | Hello guys! I'm Ironic Liberty release and agent_ipmitool to deploy nodes with the coreos (ramdisk+kernel) images. Everything is properly configured. Quick question: Does IPA from CoreOS downloads the full raw image into RAM before writing it to disk ? | 10:49 |
| rameshg87 | ionutbalutoiu: it downloads the compressed image and uncompreses directly to dis | 10:50 |
| rameshg87 | *disk | 10:50 |
| rameshg87 | ionutbalutoiu: i.e we don't need that much RAM required to hold the full raw image | 10:50 |
| *** romainh has left #openstack-ironic | 10:50 | |
| ionutbalutoiu | I have an issue. I'm trying to deploy Windows (the image is 22GB), but it fails. I'm used two nodes: one with 16GB RAM and another with 32GB RAM. But it fails on both, while with an ubuntu image of 2 GB, deploy is succesful. | 10:52 |
| ionutbalutoiu | I SSHed into the coreos during the deploy and observed that on the 16GB the RAM fills up to ~8GB and on the 32GB node up to ~16GB, and at that point on both the deploy fails. | 10:53 |
| sambetts | Any errors appearing in the conductor or IPA logs? | 10:55 |
| ionutbalutoiu | No. Just on the nova-compute, let me just give you a pastebin with them. | 10:55 |
| ionutbalutoiu | I mean the nova-compute configured with ironic compute driver. | 10:56 |
| ionutbalutoiu | http://paste.ubuntu.com/12763609/ | 10:56 |
| *** subscope has quit IRC | 10:56 | |
| ionutbalutoiu | Everything is normal until "Failed to provision instance". | 10:57 |
| lucasagomes | ionutbalutoiu, how big is the windows image? | 10:58 |
| sambetts | ionutbalutoiu: you say there is no related output on the ironic conductor? | 10:58 |
| ionutbalutoiu | lucasgomes, it's 22GB big | 10:58 |
| lucasagomes | ionutbalutoiu, if memory is a problem I would suggest you to use pxe_ipmitool (you can still use the IPA ramdisk) with whole disk image | 10:58 |
| lucasagomes | cause that way the image is copied via the iSCSI to the disk | 10:59 |
| *** jaypipes has joined #openstack-ironic | 10:59 | |
| ionutbalutoiu | I want to use IPA since I need to use config-drive | 10:59 |
| lucasagomes | oh fair | 10:59 |
| lucasagomes | yeah we still don't have whole disk image + config drive for iscsi :-( | 10:59 |
| lucasagomes | do we have a patch for it? | 10:59 |
| * lucasagomes checks | 10:59 | |
| rameshg87 | lucasagomes: yes | 10:59 |
| *** pradipta has quit IRC | 11:00 | |
| ionutbalutoiu | sambetts, I can clean the logs for ironic-api, ironic-conductor and nova-compute and do a run again. But I'm assuming this it's releated to the lack of RAM. | 11:00 |
| rameshg87 | lucasagomes: https://review.openstack.org/#/c/225115/6 | 11:00 |
| lucasagomes | rameshg87, thank you! | 11:01 |
| rameshg87 | ionutbalutoiu: but I would expect it would have gone through with a system with 32gb ram | 11:01 |
| ionutbalutoiu | rameshg87, I would also. | 11:01 |
| *** marzif has quit IRC | 11:01 | |
| *** marzif has joined #openstack-ironic | 11:02 | |
| sambetts | ionutbalutoiu: Like you said its appears to only be using half of the avalible ram before crashing, thats why I wondered if it was something else | 11:03 |
| *** Nisha has joined #openstack-ironic | 11:07 | |
| *** cemason has quit IRC | 11:10 | |
| * mkovacik is back (gone 01:57:35) | 11:11 | |
| *** cemason has joined #openstack-ironic | 11:11 | |
| *** ChanServ changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic" | 11:17 | |
| -openstackstatus- NOTICE: Gerrit has been restarted and is responding to normal load again. | 11:17 | |
| ionutbalutoiu | Hey, guys! There are the logs for a single deploy of Windows image 22G with IPA: ironic-api.log (errors starting from line 1190: http://paste.ubuntu.com/12772498/), ironic-conductor.log (line 618 and it only deployment failed logged: http://paste.ubuntu.com/12772502/) and nova-compute (part of the log since the entire log was too big: http://paste.ubuntu.com/12772511/). | 11:28 |
| *** cemason has quit IRC | 11:29 | |
| *** marzif has quit IRC | 11:29 | |
| rameshg87 | ionutbalutoiu: Unable to write image to /tmp/87d3d575-c67b-4e6a-ae37-5adba72c97b4. Error: [Errno 28] No space left on device' | 11:30 |
| rameshg87 | ionutbalutoiu: that says it all. the ramdisk ran out of space while caching the image in a temporary file in /tmp | 11:30 |
| *** jistr is now known as jistr|biab | 11:32 | |
| lucasagomes | by default tmpfs is created with half of the total RAM (physical) you have | 11:32 |
| lucasagomes | so you may have only 16G in /tmp | 11:33 |
| ionutbalutoiu | lucasgomes, ohh, that's explaining why it gets stuck at half of RAM. | 11:33 |
| lucasagomes | ionutbalutoiu, yeah, you can remount it to have more, I would say the IPA ramdisk creation could have something to configure it | 11:34 |
| lucasagomes | since we use /tmp for cache, that's a valid thing to configure IMHO | 11:34 |
| ionutbalutoiu | I'm going to check that now. | 11:34 |
| lucasagomes | ionutbalutoiu, you have access to the console in IPA? if so you can remount it in real time | 11:34 |
| * lucasagomes finds the appropriated command | 11:34 | |
| TheJulia | Good morning | 11:35 |
| lucasagomes | ionutbalutoiu, not tested: # mount -o remount,size=23G,noatime /tmp | 11:35 |
| lucasagomes | that may work for ur image, but yeah making configurable in IPA seems more reasonable | 11:35 |
| ionutbalutoiu | lucasagomes, yes, I have access, I can surely do that, but a config during the image creation would be way cleaner. :) | 11:36 |
| lucasagomes | ionutbalutoiu, an idea... you can use size=90% I think | 11:37 |
| lucasagomes | that would cover most of the cases I image | 11:37 |
| *** thrash|g0ne is now known as thrash | 11:37 | |
| lucasagomes | better than a fixed value based on units like G etc... | 11:37 |
| lucasagomes | imagine* | 11:37 |
| *** subscope has joined #openstack-ironic | 11:38 | |
| lucasagomes | ionutbalutoiu, question I see "openstack-ironic-2.maas" in the logs | 11:38 |
| lucasagomes | is it some maas or JuJu integration with Ironic ? | 11:39 |
| lucasagomes | (only answer it if you are allowed to) | 11:39 |
| ionutbalutoiu | Yep, it's a Juju charm for Ironic, it's public. And I'm using maas as a provider. | 11:39 |
| ionutbalutoiu | ironic charm - https://jujucharms.com/u/cloudbaseit/ironic/trusty/3 | 11:40 |
| lucasagomes | ionutbalutoiu, ohhh that's awesome! | 11:40 |
| ionutbalutoiu | nova-compute-ironic charm -> https://jujucharms.com/u/cloudbaseit/nova-compute-ironic/trusty/2 | 11:40 |
| * rameshg87 goes home | 11:41 | |
| ionutbalutoiu | And now we (Cloudbase Solutions) have added config drive into cloudbase-init and I'm trying to test it, that's why I ran into these issues. :) | 11:41 |
| *** rameshg87 has quit IRC | 11:41 | |
| lucasagomes | ionutbalutoiu, gotcha! That's excellent | 11:41 |
| lucasagomes | pretty cool indeed | 11:41 |
| lucasagomes | ionutbalutoiu, you can use the Ironic icon as the icon for that charm https://wiki.openstack.org/wiki/Ironic#Pixie_Boots.2C_the_Ironic_drummer_bear :-) | 11:43 |
| * lucasagomes goes grab some food | 11:44 | |
| *** lucasagomes is now known as lucas-hungry | 11:44 | |
| ionutbalutoiu | tnx, lucasagomes, we were looking for an icon. :) | 11:45 |
| sambetts | lucas-hungry: woah its lunch time already?! Man time flys | 11:47 |
| *** dims has quit IRC | 11:47 | |
| *** dims has joined #openstack-ironic | 11:48 | |
| ionutbalutoiu | I have a question. Why does IPA caches the whole image and not chunk the data it receives ? For example, we could use a buffer of 1GB (this could be configurable), fill the buffer, write it and fill another buffer while filling another one. | 11:56 |
| ionutbalutoiu | write the buffer while filling another one ** (sorry). Use one buffer for writing, one for reading. | 11:57 |
| *** Nisha has quit IRC | 12:00 | |
| *** e0ne has quit IRC | 12:00 | |
| *** trown|outttypeww is now known as trown | 12:01 | |
| TheJulia | ionutbalutoiu: Because a full hash checksum would be useless then | 12:02 |
| TheJulia | the downloading client wouldn't be able verify the checksum matches prior to writing to disk | 12:03 |
| openstackgerrit | Haomeng,Wang proposed openstack/python-ironicclient: support multi-nodes operation for set_power_state command https://review.openstack.org/228301 | 12:04 |
| *** dprince has joined #openstack-ironic | 12:08 | |
| *** romainh has joined #openstack-ironic | 12:15 | |
| liliars | morning all (: | 12:16 |
| TheJulia | good morning | 12:17 |
| liliars | TheJulia: morning! quick question. about the oneview driver docs, since the code was merged yesterday (whoop! btw thanks guys), do you still want me to put the new docs patch depending on the driver patch? | 12:18 |
| TheJulia | at this point gerrit doesn't care :) | 12:21 |
| TheJulia | Depends-On is kind of nice for linking things together easilly by commit messages | 12:21 |
| *** mrmoje has joined #openstack-ironic | 12:24 | |
| liliars | yeah I thought so too.. anyways, I'll just put the new patch to depend on the driver patch for the sake of nice linking then (: working on it, thanks! | 12:27 |
| TheJulia | Thank you! | 12:31 |
| *** nicodemos has joined #openstack-ironic | 12:31 | |
| *** achanda has joined #openstack-ironic | 12:32 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add reboot_delay option to snmp driver https://review.openstack.org/234186 | 12:36 |
| *** achanda has quit IRC | 12:37 | |
| *** VikasC has quit IRC | 12:37 | |
| *** ukalifon1 has quit IRC | 12:39 | |
| *** ionutbalutoiu has quit IRC | 12:42 | |
| *** lucas-hungry is now known as lucasagomes | 12:44 | |
| lucasagomes | sambetts, heh indeed it does | 12:44 |
| *** e0ne has joined #openstack-ironic | 12:46 | |
| openstackgerrit | Tan Lin proposed openstack/ironic: Wake up AMT interface before send request https://review.openstack.org/234190 | 12:49 |
| *** lintan_ has quit IRC | 12:57 | |
| *** [1]cdearborn has joined #openstack-ironic | 12:59 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add documentation for Ceph Object Gateway support https://review.openstack.org/232068 | 13:01 |
| *** jistr|biab is now known as jistr | 13:03 | |
| *** rloo has joined #openstack-ironic | 13:05 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Fixes logging of failure in deletion of swift temporary object https://review.openstack.org/232440 | 13:05 |
| *** mbound has quit IRC | 13:06 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add reboot_delay option to snmp driver https://review.openstack.org/234186 | 13:07 |
| *** ionutbalutoiu has joined #openstack-ironic | 13:08 | |
| thiagop | Good morning Ironicers! | 13:10 |
| *** mbound has joined #openstack-ironic | 13:10 | |
| *** mbound has quit IRC | 13:11 | |
| *** achanda has joined #openstack-ironic | 13:11 | |
| *** mbound has joined #openstack-ironic | 13:11 | |
| sambetts | o. thiagop | 13:12 |
| sambetts | o/ | 13:12 |
| *** [1]cdearborn has quit IRC | 13:14 | |
| *** baoli has joined #openstack-ironic | 13:14 | |
| thiagop | jroll lucasagomes glad to merge that thing, thank you both for the attention | 13:15 |
| lucasagomes | thiagop, np at all! I mean, thank you guys for writing it | 13:15 |
| lucasagomes | and morning :-) | 13:16 |
| *** amotoki has joined #openstack-ironic | 13:19 | |
| lucasagomes | jroll, now that IPA has it's own release, we still using the ironic bug tracker for it? Or it has it's own bug tracker now ? | 13:19 |
| *** achanda has quit IRC | 13:20 | |
| jroll | lucasagomes: https://launchpad.net/ironic-python-agent | 13:20 |
| jroll | separate launchpad is a requirement to release independently of ironic | 13:21 |
| lucasagomes | :-( | 13:22 |
| * lucasagomes gets sad using launchpad | 13:23 | |
| *** pelix has quit IRC | 13:24 | |
| *** Marga_ has joined #openstack-ironic | 13:24 | |
| *** pelix has joined #openstack-ironic | 13:25 | |
| sambetts | lucasagomes: what happened to story board? Wasn't that meant to be replacing launchpad at somepoint? | 13:26 |
| lucasagomes | sambetts, no idea, I think -infra was using/testing it? But idk | 13:27 |
| lucasagomes | I don't even know how good is it to compare with launchpad | 13:27 |
| *** cdearborn has joined #openstack-ironic | 13:27 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add documentation for Ceph Object Gateway support https://review.openstack.org/232068 | 13:28 |
| lucasagomes | ionutbalutoiu, https://bugs.launchpad.net/ironic-python-agent/+bug/1505685 | 13:28 |
| openstack | Launchpad bug 1505685 in ironic-python-agent "Stream raw images" [Wishlist,New] - Assigned to Lucas Alvares Gomes (lucasagomes) | 13:28 |
| lucasagomes | ionutbalutoiu, will see if I can cook up a patch for it this afternoon | 13:28 |
| sambetts | lucasagomes: Yeah, I don't know, infra seem to use it for a bunch of their stuff :/ | 13:29 |
| *** amotoki has quit IRC | 13:29 | |
| lucasagomes | sambetts, right, ok so it may come eventually | 13:29 |
| *** mgoddard_ has joined #openstack-ironic | 13:29 | |
| jroll | sambetts: lucasagomes they ditched storyboard dev, and are evaluating some phabricator thing | 13:29 |
| sambetts | jroll, lucasagomes: oh :/ | 13:30 |
| lucasagomes | hmmm | 13:30 |
| jroll | it's a good thing :P | 13:30 |
| lucasagomes | I think mordred was telling me about it | 13:30 |
| sambetts | jroll: this http://phabricator.org/ ? | 13:30 |
| lucasagomes | jroll, yeah it's good indeed | 13:30 |
| * lucasagomes looks at phabricator | 13:30 | |
| jroll | sambetts: just one part of that yeah | 13:30 |
| * jroll forgets the component | 13:30 | |
| jroll | aha, maniphest | 13:31 |
| sambetts | jroll: that suite looks like it could replace gerrit and launchpad | 13:31 |
| jroll | https://wiki.openstack.org/wiki/Phabricator | 13:31 |
| *** penick has joined #openstack-ironic | 13:31 | |
| jroll | yeah, they aren't planning to replace gerrit | 13:31 |
| jroll | just us maniphest to replace LP | 13:32 |
| lucasagomes | cool | 13:32 |
| *** penick has quit IRC | 13:32 | |
| *** mgoddard has quit IRC | 13:32 | |
| sambetts | jroll, lucasagomes: https://secure.phabricator.com/maniphest/ < demo | 13:33 |
| jroll | yeah | 13:33 |
| jroll | I haven't actually ever used phab but I've always wanted to try, it looks fantastic | 13:34 |
| lucasagomes | rask == blueprint? | 13:34 |
| lucasagomes | task* | 13:34 |
| cinerama | there's some puppet science for setting up phab here: https://github.com/emonty/puppet-phabricator | 13:35 |
| jroll | lucasagomes: yeah, not sure on the details | 13:36 |
| jroll | task seems like it could be a bug or feature | 13:36 |
| lucasagomes | oh YES | 13:36 |
| lucasagomes | I like that | 13:36 |
| lucasagomes | and if they have "severity" and "priority" separately as well that would be great | 13:37 |
| jroll | not sure they do | 13:38 |
| *** penick has joined #openstack-ironic | 13:39 | |
| lucasagomes | will check it later with time | 13:39 |
| jroll | could use tags for one of them if needed | 13:39 |
| *** mjturek1 has joined #openstack-ironic | 13:39 | |
| *** amotoki has joined #openstack-ironic | 13:40 | |
| sambetts | it was only the other day me and a colleage I was saying that I don't understand why blueprints need to exist on launchpad, if they made marking something as a wishlist separate to priotry | 13:40 |
| *** cemason has joined #openstack-ironic | 13:40 | |
| sambetts | then you could just use wishlist bugs for that stuff | 13:40 |
| *** puranamr has joined #openstack-ironic | 13:43 | |
| *** cemason has quit IRC | 13:45 | |
| *** puranamr has quit IRC | 13:45 | |
| *** cemason has joined #openstack-ironic | 13:48 | |
| *** thrash is now known as thrash|biab | 13:52 | |
| lucasagomes | jroll, yeah we can work it around | 13:53 |
| *** r-daneel has joined #openstack-ironic | 13:55 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Fix AgentDeploy compatibility with pre-Liberty drivers https://review.openstack.org/231215 | 14:03 |
| *** BharatK_ has quit IRC | 14:04 | |
| jroll | ^ would love reviews on that | 14:06 |
| *** penick has quit IRC | 14:07 | |
| thiagop | jroll: it will have a backport for stable/liberty? | 14:07 |
| jroll | thiagop: yes | 14:08 |
| ionutbalutoiu | lucasgomes, pretty nice. Thank-you. | 14:13 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Config drive support for whole disk images for iscsi based deploy https://review.openstack.org/225115 | 14:15 |
| *** penick has joined #openstack-ironic | 14:15 | |
| *** thrash|biab is now known as thrash | 14:19 | |
| *** cemason has quit IRC | 14:20 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic-lib: Config drive support for whole disk images for iscsi based deploy https://review.openstack.org/230924 | 14:21 |
| *** ionutbalutoiu has quit IRC | 14:21 | |
| *** stendulker has joined #openstack-ironic | 14:24 | |
| NobodyCam | good morning Ironicers | 14:25 |
| thiagop | Good morning NobodyCam | 14:25 |
| *** amotoki has quit IRC | 14:25 | |
| NobodyCam | :) | 14:25 |
| sambetts | o/ NobodyCam | 14:26 |
| NobodyCam | hey hey sambetts | 14:27 |
| sambetts | Hows it going? | 14:27 |
| NobodyCam | morning to jroll rloo jlvillal sambetts thiagop dtantsur|afk lucasagomes and all others not losted here | 14:28 |
| NobodyCam | listed even | 14:28 |
| lucasagomes | NobodyCam, morning | 14:28 |
| jroll | morning NobodyCam :) | 14:28 |
| jlvillal | NobodyCam, Good morning | 14:28 |
| NobodyCam | morning lucasagomes jlvillal | 14:28 |
| jlvillal | And good morning all of Ironic | 14:28 |
| jroll | \o jlvillal | 14:28 |
| NobodyCam | sambetts: going good otp atm lol | 14:28 |
| sambetts | :D | 14:28 |
| openstackgerrit | Alberto Fagner proposed openstack/python-oneviewclient: Author: albertofagner Date: Tue Oct 13 11:15:10 2015 -0300 https://review.openstack.org/234245 | 14:29 |
| rloo | hiya and morning NobodyCam, jroll, sambetts, lucasagomes, jlvillal, thiagop | 14:29 |
| jroll | rloo: good morning, welcome back from the long weekend :D | 14:29 |
| jlvillal | jroll, Commented on AgentDeploy. Basically I think the cleanup section is un-needed. But happy to be proven wrong :) | 14:30 |
| rloo | jroll: thx. still wading through emails :) | 14:30 |
| *** achanda has joined #openstack-ironic | 14:30 | |
| jlvillal | thiagop, Is there a OneView documentation patch to review yet? | 14:32 |
| jroll | jlvillal: meh, unclear to me... I think it may matter with high parallelization etc | 14:32 |
| thiagop | jlvillal: liliars is working on that right now. Should be on gerrit in the next couple of hours | 14:32 |
| jlvillal | thiagop, Thanks | 14:33 |
| thiagop | rloo: morning! | 14:33 |
| jlvillal | jroll, Unclear to me too. My gut feeling is not needed. But maybe some other people know for sure. | 14:33 |
| jroll | jlvillal: yeah, it can't hurt though :) | 14:34 |
| jlvillal | thiagop, You might want to help Alberto on his commit message ^^^^^ | 14:34 |
| lucasagomes | rloo, hi there, morning | 14:34 |
| *** stendulker has quit IRC | 14:34 | |
| thiagop | jlvillal: sure | 14:35 |
| jlvillal | jroll, True. Though I'd prefer to not to have cruft if easily avoided. | 14:35 |
| jlvillal | lucasagomes, What do you think about cleanup function here: https://review.openstack.org/#/c/231215/12/ironic/tests/unit/drivers/modules/test_agent.py | 14:36 |
| lucasagomes | jlvillal, hi there, 1 sec will see | 14:36 |
| jlvillal | Thanks | 14:36 |
| openstackgerrit | Alberto Fagner proposed openstack/python-oneviewclient: Update usage documentation of python-oneviewclient https://review.openstack.org/234245 | 14:37 |
| *** albertoffb has joined #openstack-ironic | 14:38 | |
| *** achanda has quit IRC | 14:40 | |
| *** penick has quit IRC | 14:41 | |
| *** amotoki has joined #openstack-ironic | 14:44 | |
| *** penick has joined #openstack-ironic | 14:45 | |
| vdrok | morning jroll thiagop NobodyCam jlvillal and rloo :) | 14:47 |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Expose versioning information on GET / endpoint https://review.openstack.org/209012 | 14:47 |
| jlvillal | vdrok, Good morning | 14:47 |
| thiagop | morning vdrok | 14:47 |
| jroll | \o vdrok :) | 14:47 |
| rloo | hi vdrok | 14:47 |
| *** mgoddard_ has quit IRC | 14:50 | |
| *** mgoddard has joined #openstack-ironic | 14:50 | |
| *** itamarl has quit IRC | 14:54 | |
| *** dolphm has joined #openstack-ironic | 14:55 | |
| sambetts | o/ vdrok rloo jlvillal | 14:56 |
| vdrok | \o | 14:56 |
| devananda | morning, all | 14:56 |
| vdrok | devananda, morning | 14:56 |
| lucasagomes | jlvillal, indeed it doesn't seem needed | 14:57 |
| lucasagomes | jlvillal, if it's always setting it to None at the beggining of the setUp method | 14:57 |
| thiagop | morning devananda | 14:58 |
| jlvillal | lucasagomes, Thanks that is what I suspected. But jroll and I weren't sure. | 14:58 |
| *** achanda has joined #openstack-ironic | 14:59 | |
| *** ionutbalutoiu has joined #openstack-ironic | 15:03 | |
| *** achanda has quit IRC | 15:04 | |
| *** Nisha has joined #openstack-ironic | 15:06 | |
| sambetts | morning devananda | 15:09 |
| *** penick has quit IRC | 15:10 | |
| *** alex_xu has quit IRC | 15:10 | |
| *** penick has joined #openstack-ironic | 15:17 | |
| NobodyCam | mornign vdrok devananda | 15:19 |
| BadCub | morning ironicville | 15:22 |
| thiagop | morning BadCub | 15:22 |
| BadCub | heya thiagop | 15:23 |
| vdrok | morning BadCub :) | 15:27 |
| BadCub | heya vdrok | 15:27 |
| *** ijw has joined #openstack-ironic | 15:31 | |
| *** ijw has quit IRC | 15:32 | |
| *** ijw has joined #openstack-ironic | 15:33 | |
| *** lazy_prince has joined #openstack-ironic | 15:36 | |
| jlvillal | morning BadCub | 15:39 |
| *** kozhukalov_ has quit IRC | 15:39 | |
| jlvillal | morning devananda ... I'm late. | 15:39 |
| BadCub | heya jlvillal | 15:39 |
| lazy_prince | jroll: regarding the ci for network-provider, you wanted to check the possibility of testing network flip using two networks... Would that not be a problem when the VM is connected to the bridge and two networks will be serving ip address to the vm..? | 15:40 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Config drive support for whole disk images for iscsi based deploy https://review.openstack.org/225115 | 15:40 |
| *** Marga_ has quit IRC | 15:42 | |
| *** mrmoje has quit IRC | 15:42 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic-lib: Config drive support for whole disk images for iscsi based deploy https://review.openstack.org/230924 | 15:43 |
| *** mbound has quit IRC | 15:44 | |
| *** stendulker has joined #openstack-ironic | 15:46 | |
| *** mgoddard has quit IRC | 15:53 | |
| *** mgoddard has joined #openstack-ironic | 15:53 | |
| sambetts | Anyone else noticed .testrepository being huge?? I've never noticed it before but I just ran a du -sh .testrepository in my ironic dev folder and it was >700mb | 15:53 |
| *** kozhukalov_ has joined #openstack-ironic | 15:55 | |
| jroll | lazy_prince: there would only be one network at a time, right? | 15:56 |
| *** e0ne has quit IRC | 15:56 | |
| lazy_prince | but for flip to work, there should be one tenant network and one prov/clean network.. | 15:57 |
| lazy_prince | that makes it two network.. | 15:57 |
| *** romainh has left #openstack-ironic | 15:59 | |
| lazy_prince | jroll: nova will create port on tenant network and ironic will also create port on provisioning network.. that could be a problem.. | 15:59 |
| jroll | lazy_prince: right, but the ...oh I see | 15:59 |
| jroll | er, no. | 16:00 |
| jroll | lazy_prince: if we don't send host_id or device_id or whichever I forget... the tenant network won't be plumbed through right? | 16:00 |
| lazy_prince | there is no plumbing needed as they will be on same physical network.. and in any case, ovs will not plumb them even if you pass those details.. | 16:01 |
| jroll | oh, so it's a devstack oddity? | 16:02 |
| lazy_prince | even if they are in unbound state, the dhcp process will start handing out ip addresses.. | 16:03 |
| lazy_prince | one way is to set the admin_state_up flag.. but i am not sure.. | 16:03 |
| lazy_prince | if we go with proper VLAN network in devstack, it may be possible but then we will need ovs mech driver for (BM) VM.. | 16:05 |
| *** mbound has joined #openstack-ironic | 16:05 | |
| lazy_prince | but if we go with two flat network (non-vlan) networks, then we have some more work to do.. not sure how much.. | 16:05 |
| openstackgerrit | Ruby Loo proposed openstack/ironic-specs: Manual cleaning https://review.openstack.org/231288 | 16:07 |
| openstackgerrit | Merged openstack/ironic-python-agent: Ensure all methods in utils.py have docstrings https://review.openstack.org/220313 | 16:13 |
| stendulker | jroll: Hi | 16:13 |
| stendulker | jroll: This is regarding https://review.openstack.org/#/c/225115/ | 16:14 |
| stendulker | jroll: Review for config drive support for whole disk images for iscsi drivers | 16:14 |
| jroll | lazy_prince: yeah, hmmmmm | 16:14 |
| jroll | stendulker: hi | 16:14 |
| stendulker | jroll: Have addressed your comment, but patch is getting jenkins -1 immediately | 16:15 |
| stendulker | jroll: Is there any additional steps needs to be done from my end ? | 16:15 |
| jroll | stendulker: I'm not sure why that's happening | 16:16 |
| sambetts | stendulker: its getting a -1 from jenkins becuase of the patch it Depends-On in the commit message | 16:16 |
| stendulker | jroll: No logs available... | 16:16 |
| *** Marga_ has joined #openstack-ironic | 16:17 | |
| stendulker | jroll: tht is ironic-lib patch for the same change | 16:17 |
| sambetts | stendulker: if you scroll to the bottom of the page and hit toggle CI you'll see the comment from Jenkins | 16:17 |
| stendulker | jroll: It is there for quite some time | 16:17 |
| sambetts | stendulker: The ironic-lib patch is still waiting for jenkins to return its results for patchset 6, jenkins is being a bit slow | 16:18 |
| sambetts | today | 16:18 |
| jroll | thanks for the help sambetts | 16:18 |
| stendulker | sambetts: Thanks for the help | 16:19 |
| stendulker | sambetts, jroll : It indicates "This change depends on a change that failed to merge." | 16:19 |
| jroll | yes, I read that. | 16:20 |
| jroll | so the ironic-lib patch failed to merge. | 16:20 |
| stendulker | sambetts, jroll : ironic-lib patch is at top of master | 16:21 |
| jroll | if the ironic-lib fails tests I believe that will also happen | 16:21 |
| sambetts | or just hasn't return the results yet too it seems | 16:22 |
| stendulker | jroll: jenkins did pass in my environment. | 16:22 |
| *** mbound has quit IRC | 16:22 | |
| sambetts | stendulker: https://review.openstack.org/#/c/230924/ this the patch your depending on | 16:22 |
| *** praneshp has joined #openstack-ironic | 16:22 | |
| jroll | sambetts: I wonder if updating the ironic-lib patch after submitting the ironic patch made this happen | 16:22 |
| stendulker | jroll: curent patch of ironic-lib is still under progress, may be i will wait for that jenkins test to complete | 16:22 |
| stendulker | sambetts: yes | 16:23 |
| jroll | yeah | 16:23 |
| *** trown is now known as trown|lunch | 16:23 | |
| sambetts | looking at zuul its not even started the tests for the ironic-lib patch yet, so I guess if the ironic ones completed first then you would end up with this result | 16:24 |
| stendulker | sambetts: But for ironic patch it gave me error almost immediately. | 16:25 |
| sambetts | stendulker: thats because it checks the ironic-lib test results first I believe, and as it can't confirm that will pass, then it doesn't bother running the ironic ones | 16:26 |
| *** garthb has joined #openstack-ironic | 16:26 | |
| stendulker | sambetts: ok.. | 16:26 |
| *** mkovacik has quit IRC | 16:26 | |
| *** Nisha has quit IRC | 16:27 | |
| stendulker | sambetts: Then I will wait for ironic-lib patch tests to complete and then recheck on ironic patch | 16:27 |
| sambetts | stendulker: It may automatically correct it for you once they complete, but if it doesn't leave it a recheck :) | 16:28 |
| stendulker | jroll: Regarding your latest comment on patch, We have been fixing in ironic-lib and ironic for some of the defect fixes which involved change at both the palces | 16:29 |
| sambetts | stendulker: you can monitor the progress of the ironic-lib test at http://status.openstack.org/zuul/ just put 230924 in the filter box | 16:29 |
| jroll | stendulker: right, we need to stop doing that, it's only going to slow down progress on the actual conversion | 16:29 |
| stendulker | jroll: Do you want me to wait till ironic-lib integration with ironic gets complete | 16:29 |
| stendulker | sambetts: Thank you for your help | 16:29 |
| jroll | stendulker: I think that would be best, yes, but if everyone else disagrees with me that's fine too | 16:30 |
| *** ionutbalutoiu has quit IRC | 16:30 | |
| stendulker | jroll: I have been addressing comments on both patches irrespctive of where I get to keep them in sync | 16:30 |
| *** ifarkas has quit IRC | 16:31 | |
| *** jistr has quit IRC | 16:31 | |
| *** ionutbalutoiu has joined #openstack-ironic | 16:31 | |
| jroll | stendulker: I know, it just makes things harder for people working on the integration, because they will need to continually rebase | 16:31 |
| stendulker | jroll: ok. Thank you. | 16:32 |
| jroll | you're welcome, thank you for working on this, sorry to delay you :( | 16:33 |
| *** subscope has quit IRC | 16:34 | |
| *** kozhukalov_ has quit IRC | 16:38 | |
| jroll | devananda: have time to look at the comments on the driver fakeboot patch thing? | 16:38 |
| *** mgoddard_ has joined #openstack-ironic | 16:45 | |
| *** praneshp has quit IRC | 16:45 | |
| *** ipukha has quit IRC | 16:45 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Initial support for installation of ironic-inspector https://review.openstack.org/229930 | 16:48 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Initial addition of an inspection module https://review.openstack.org/232745 | 16:48 |
| *** mgoddard has quit IRC | 16:49 | |
| *** mgoddard_ has quit IRC | 16:49 | |
| *** kozhukalov_ has joined #openstack-ironic | 16:50 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Initial support for installation of ironic-inspector https://review.openstack.org/229930 | 16:51 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Initial addition of an inspection module https://review.openstack.org/232745 | 16:52 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: WIP: Additional inspection role and adding role to NV job script https://review.openstack.org/232746 | 16:52 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Refactor the image download and checksum computation bits https://review.openstack.org/234338 | 16:52 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Add support for streaming raw images directly onto the disk https://review.openstack.org/234339 | 16:52 |
| lucasagomes | ionutbalutoiu, schrödinger's code, I haven't tested yet ^ may work or not :-) | 16:53 |
| *** davidlenwell has quit IRC | 16:54 | |
| *** davidlenwell has joined #openstack-ironic | 16:56 | |
| *** mgoddard has joined #openstack-ironic | 17:00 | |
| *** stendulker has quit IRC | 17:01 | |
| *** Nisha has joined #openstack-ironic | 17:04 | |
| *** kozhukalov_ has quit IRC | 17:08 | |
| *** achanda has joined #openstack-ironic | 17:14 | |
| *** ukalifon has joined #openstack-ironic | 17:14 | |
| *** ukalifon has quit IRC | 17:16 | |
| *** ukalifon has joined #openstack-ironic | 17:16 | |
| rloo | jroll, dtantsur|afk: I saw this bug mentioned in the subteam reports, as liberty-rc-potential. https://github.com/openstack/nova/blob/stable/kilo/nova/compute/rpcapi.py#L357-L364 | 17:18 |
| rloo | jroll, dtantsur|afk: no one is working on it. and i thought we couldn't do rolling upgrades anyway. | 17:18 |
| *** thrash is now known as thrash|f00dz | 17:18 | |
| jroll | rloo: that doesn't look like a bug to me? | 17:18 |
| jroll | that's nova kilo code | 17:18 |
| rloo | jroll: it is a bug. | 17:19 |
| rloo | jroll: are you looking at the nova example? | 17:19 |
| jroll | rloo: your link is not a bug, it's just some code | 17:19 |
| rloo | jroll: sigh. sorry, you're right. so much for copy/paste. sec. | 17:19 |
| jroll | hehe | 17:19 |
| rloo | https://bugs.launchpad.net/ironic/+bug/1502903 | 17:19 |
| openstack | Launchpad bug 1502903 in Ironic "Rolling upgrades: backward incompatible changes in RPC methods" [High,Triaged] | 17:20 |
| jroll | aha | 17:20 |
| *** ukalifon has quit IRC | 17:21 | |
| jroll | rloo: yeah, we don't technically support rolling upgrades today but I can see why that's problematic and is totally a bug | 17:21 |
| *** mtanino has joined #openstack-ironic | 17:21 | |
| rloo | jroll: yes, it is a bug. so do we want to fix it? even fixed, rolling upgrades won't work? | 17:22 |
| jroll | rloo: yeah, I'm not sure. if we don't fix it, rolling upgrades definitely won't work. if we do fix it, they might kinda work :) | 17:24 |
| rloo | jroll: then we should probably fix it. | 17:24 |
| jroll | yeah... | 17:24 |
| rloo | jroll: so if someone really needs rolling upgrades, less hacks to try to get it to work. | 17:24 |
| jroll | lucasagomes: ^ you interested in that at all or shall one of us take it on? | 17:24 |
| openstackgerrit | Om Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - RPC https://review.openstack.org/206243 | 17:25 |
| openstackgerrit | Om Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/206244 | 17:25 |
| openstackgerrit | Om Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - net https://review.openstack.org/206245 | 17:25 |
| openstackgerrit | Om Kumar proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 17:25 |
| openstackgerrit | Om Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - DB https://review.openstack.org/206232 | 17:25 |
| openstackgerrit | Om Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - objs https://review.openstack.org/206238 | 17:25 |
| openstackgerrit | Om Kumar proposed openstack/ironic: Add network provider interface and implementations https://review.openstack.org/139687 | 17:25 |
| TheJulia | lazy_prince: wow | 17:25 |
| * lucasagomes reads | 17:25 | |
| lucasagomes | jroll, there wasn't someone looking into it? | 17:26 |
| lazy_prince | its just rebase.. thanks to oneview driver merge.. | 17:26 |
| jroll | lucasagomes: I'm not sure, someone reported it, I'm not sure if they are working on it | 17:26 |
| jroll | this is the first I've heard about it | 17:26 |
| rloo | jroll: I saw it in the subteam reports | 17:27 |
| lucasagomes | jroll, ah right, I can take a peek, but today I probably won't cause it's late already | 17:27 |
| jroll | yeah, I must have overlooked it | 17:27 |
| * lucasagomes not sure what he's doing on IRC still | 17:28 | |
| jroll | lucasagomes: I can try to tackle it too | 17:28 |
| jroll | lol | 17:28 |
| rloo | lucasagomes: go away! :) | 17:28 |
| lucasagomes | jroll, sure, yeah if you want to take a look first I can help wiht the patch tomorrow | 17:28 |
| jroll | oh it was xek | 17:28 |
| lucasagomes | YES | 17:28 |
| *** praneshp has joined #openstack-ironic | 17:28 | |
| lucasagomes | yeah that's the nickname I saw | 17:28 |
| jroll | xek: hey, are you looking into a fix for bug 1502903 ? | 17:28 |
| openstack | bug 1502903 in Ironic "Rolling upgrades: backward incompatible changes in RPC methods" [High,Triaged] https://launchpad.net/bugs/1502903 | 17:28 |
| jroll | we still need to get this landed too https://review.openstack.org/#/c/231215/ | 17:29 |
| lucasagomes | damn people linked the video of someone using OSX | 17:29 |
| * lucasagomes vomits | 17:29 | |
| jroll | ..... | 17:30 |
| lucasagomes | jk, but not really | 17:30 |
| jroll | nothing wrong with os x, I'd take it a million times over windows | 17:30 |
| lucasagomes | idk, I sometimes feel sad of going to a FOSS conf and seem OSX around | 17:31 |
| jroll | there's the whole thing where audio and wifi actually work | 17:31 |
| * jroll ducks | 17:31 | |
| lucasagomes | I don't mind the hardware tho, I would use it with a free os | 17:31 |
| *** praneshp_ has joined #openstack-ironic | 17:31 | |
| jroll | meh, I've spent too much time fighting against linux to be doing it on the road | 17:31 |
| lucasagomes | anyway... sorry it's just a late rant | 17:31 |
| *** achanda has quit IRC | 17:31 | |
| * lucasagomes is not an apple fan | 17:31 | |
| jroll | for example this video conferencing app we use has been segfaulting on my linux system for weeks | 17:31 |
| jroll | so... I use os x for that | 17:32 |
| lucasagomes | jroll, google hangout? | 17:32 |
| jroll | nah, vidyo | 17:32 |
| jroll | though I feel like hangouts doesn't play nice either iirc | 17:32 |
| lucasagomes | yeah, it's dodge sometimes | 17:33 |
| *** praneshp has quit IRC | 17:33 | |
| *** praneshp_ is now known as praneshp | 17:33 | |
| jroll | yeah | 17:34 |
| lucasagomes | but yeah, I don't know I still believe in the linux desktop! (one day, one day...) | 17:34 |
| jroll | me too, but in the meantime, I want something I know works on the road :) | 17:34 |
| lucasagomes | fair enuff | 17:35 |
| lucasagomes | jroll, anyway I diverted the whole conversation with my rant heh sorry | 17:35 |
| lucasagomes | jroll, I will watch the video, but if u want to take a look at the bug today I can pick up with you and help tomorrow | 17:35 |
| jroll | lol | 17:35 |
| jroll | yeah reading code will be quicker :P | 17:35 |
| lucasagomes | +1 (/me is tired) | 17:36 |
| jroll | I think we just need something like if client_has_version(1.29): return new_response else: return old_response | 17:36 |
| jroll | there's a method for it, I'll get something up | 17:36 |
| lucasagomes | oh I saw that in nova yeah | 17:36 |
| lucasagomes | cool, yeah hope it's simple | 17:36 |
| jlvillal | jroll: Something like this? https://review.openstack.org/217853 | 17:37 |
| jroll | no | 17:38 |
| jroll | sec. | 17:38 |
| * jlvillal wandered in during the middle of the conversation and hasn't read the whole backlog. | 17:38 | |
| devananda | jroll: read the comments - is there one in particular? | 17:39 |
| jroll | devananda: dunno, just in general | 17:39 |
| devananda | in general it seems like folks don't understand or agree with what I've proposed | 17:40 |
| devananda | *some folks | 17:40 |
| devananda | we know what breaks if we don't land this | 17:41 |
| lucasagomes | jlvillal, https://github.com/openstack/nova/blob/master/nova/conductor/rpcapi.py#L292 | 17:41 |
| jlvillal | lucasagomes: Thanks | 17:41 |
| lucasagomes | something like that (AFAIUI) | 17:41 |
| jroll | yeah, something like that but more server side | 17:41 |
| *** trown|lunch is now known as trown | 17:43 | |
| *** lazy_prince has quit IRC | 17:45 | |
| *** kkoski has joined #openstack-ironic | 17:48 | |
| *** achanda has joined #openstack-ironic | 17:48 | |
| *** harshs has joined #openstack-ironic | 17:51 | |
| *** dprince has quit IRC | 17:58 | |
| *** harshs has quit IRC | 17:59 | |
| *** Sukhdev has joined #openstack-ironic | 18:00 | |
| *** kkoski has quit IRC | 18:01 | |
| lucasagomes | I'm going to call it a day | 18:02 |
| lucasagomes | have a great night everyone! See you all tomorrow | 18:02 |
| jlvillal | lucasagomes: Good night | 18:02 |
| *** lucasagomes is now known as lucas-dinner | 18:02 | |
| lucas-dinner | jlvillal, thanks you too | 18:02 |
| *** harshs has joined #openstack-ironic | 18:03 | |
| *** Sukhdev has quit IRC | 18:04 | |
| *** kkoski has joined #openstack-ironic | 18:04 | |
| *** athomas has quit IRC | 18:06 | |
| *** thrash|f00dz is now known as thrash | 18:10 | |
| *** Sukhdev has joined #openstack-ironic | 18:11 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Refactor the image download and checksum computation bits https://review.openstack.org/234338 | 18:11 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Add support for streaming raw images directly onto the disk https://review.openstack.org/234339 | 18:11 |
| * lucas-dinner is now gone :-) | 18:11 | |
| *** pas-ha has quit IRC | 18:12 | |
| jroll | lucas-dinner: before you go | 18:12 |
| jroll | md5 supports streaming | 18:12 |
| lucas-dinner | jroll, yup | 18:12 |
| lucas-dinner | jroll, yes I do that | 18:12 |
| jroll | oh cool | 18:12 |
| jroll | I missed that | 18:12 |
| lucas-dinner | as it downlods the image it already computes the md5 | 18:12 |
| jroll | aha, I see it now | 18:12 |
| jroll | yeah | 18:12 |
| lucas-dinner | so the first patch, is a refactor around that | 18:13 |
| lucas-dinner | then the second just use the new class | 18:13 |
| jroll | right, I missed the first patch | 18:13 |
| lucas-dinner | but it's kinda of a schrödinger code now, I didn't test it fully | 18:13 |
| lucas-dinner | will do tomorrow | 18:13 |
| jroll | yeah | 18:13 |
| jroll | have a good night :) | 18:13 |
| *** dprince has joined #openstack-ironic | 18:13 | |
| lucas-dinner | jroll, thanks! You too, enjoy the afternoon! | 18:14 |
| jroll | ty | 18:14 |
| *** ndipanov has quit IRC | 18:14 | |
| *** lazy_prince has joined #openstack-ironic | 18:15 | |
| Sukhdev | devananda: Ping | 18:16 |
| *** e0ne has joined #openstack-ironic | 18:17 | |
| *** nicodemos has quit IRC | 18:20 | |
| *** achanda has quit IRC | 18:25 | |
| *** kozhukalov_ has joined #openstack-ironic | 18:27 | |
| *** achanda has joined #openstack-ironic | 18:27 | |
| *** [2]Rick has joined #openstack-ironic | 18:28 | |
| thiagop | good night lucas-dinner | 18:28 |
| *** e0ne has quit IRC | 18:30 | |
| *** Sukhdev has quit IRC | 18:31 | |
| sambetts | o/ lucas-dinner | 18:31 |
| *** Sukhdev has joined #openstack-ironic | 18:33 | |
| devananda | Sukhdev: pong | 18:37 |
| Sukhdev | devananda: thanks for response. I saw your reply last night - by the time I saw your reply you were gone :-):) | 18:38 |
| devananda | hehe | 18:38 |
| Sukhdev | devananda: Have a quick question for you - in one of your presentations, you had a really nice picture of all the OVS bridges and connectivity for BM | 18:38 |
| devananda | I did?? | 18:39 |
| * krtaylor would like to see that too | 18:39 | |
| Sukhdev | I am having issue reaching the tftp server from provisioning network - I am suspecting that some low level goofup in OVS bridge connectivity or tags issue | 18:39 |
| Sukhdev | I thought will use that picture to follow the packet path (tcpdump, etc.) to nail this down | 18:40 |
| openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/233819 | 18:40 |
| Sukhdev | devananda: I saw this almost a year ago in one of the presentations - I thought it was your presentation | 18:41 |
| Sukhdev | devananda: basically what I am looking for is the packet path for the BM (as a VM) from the tap interface of the brctl bridge to br-int to the other side to the dhcp | 18:42 |
| Sukhdev | so that I can see if this is a tagging issue or what is causing the connectivity issue to the tftp server | 18:43 |
| devananda | Sukhdev: funny thing ... i was just trying to help linggao troubleshoot much the same problem | 18:52 |
| devananda | here's roughly how i'm debugging it | 18:52 |
| devananda | - is ironic-conductor host getting the DHCPBOOTP packet? (tcpdump the interface with some filters) | 18:53 |
| *** albertoffb has quit IRC | 18:54 | |
| devananda | - is dnsmasq recognizing and trying to act on the request? check for MAC filters, local firewall rules, and the like | 18:54 |
| devananda | - is dnsmasq responding? check for IP ranges and configuration | 18:55 |
| *** Marga_ has quit IRC | 18:58 | |
| *** harshs has quit IRC | 18:59 | |
| devananda | Sukhdev: you're talking about a devstack environment where you use VM's to emulate the baremetal nodes? | 19:01 |
| *** mgoddard has quit IRC | 19:01 | |
| *** amotoki has quit IRC | 19:02 | |
| *** harshs has joined #openstack-ironic | 19:03 | |
| Sukhdev | devananda: yes - devstack where I am using VM as a BM | 19:03 |
| *** sambetts has quit IRC | 19:05 | |
| *** sambetts has joined #openstack-ironic | 19:07 | |
| *** harshs has quit IRC | 19:09 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Documentation for Swift-less iLO Virtual Media deploy https://review.openstack.org/228020 | 19:13 |
| *** e0ne has joined #openstack-ironic | 19:19 | |
| *** lazy_prince has quit IRC | 19:19 | |
| openstackgerrit | Lilia Sampaio proposed openstack/ironic: Add OneView driver documentation https://review.openstack.org/234396 | 19:21 |
| liliars | hey TheJulia, jlvillal and all (: just added the docs for the OneView driver. please take a look when you can, thanks! | 19:22 |
| TheJulia | Thank you :) | 19:23 |
| *** mrmoje has joined #openstack-ironic | 19:26 | |
| devananda | Sukhdev: are you running devstack on the host, or in its own VM? | 19:27 |
| Sukhdev | devananda: in a vm | 19:27 |
| *** cemason has joined #openstack-ironic | 19:29 | |
| *** Nisha has quit IRC | 19:29 | |
| jroll | Sukhdev: I just realized there aren't instructions for testing this anywhere | 19:31 |
| devananda | Sukhdev: are the "fake" bm nodes nested inside that VM -- or are they bridged on the *host* outside of it? | 19:31 |
| jroll | those would be massively valuable, moreso than actually testing it | 19:31 |
| devananda | jroll: nope, I dont think there are ... :( | 19:31 |
| jroll | right now you're the only one that can reproduce this, afaik | 19:31 |
| jroll | if I could follow some directions and see the same issues - I could help here | 19:31 |
| devananda | ditto | 19:31 |
| Sukhdev | devananda: nested with the VM | 19:32 |
| devananda | Sukhdev: eeh. well. that's going to make booting very slow, fwiw | 19:32 |
| devananda | last time i tried that (granted it was well over a year ago) the nested vm took about 30 minutes to boot | 19:33 |
| Sukhdev | devananda: no - it just works fine - | 19:33 |
| devananda | o_o | 19:33 |
| Sukhdev | devananda: it has been working fine until last week - just when I uploaded the new patches yesterday, it has stopped working - just the tftp reachability issue - | 19:34 |
| Sukhdev | devananda: trying to debug it just to make sure if this is my setup issue or some issue with the patch - as the patch does mess with the DHCP options :-):-) | 19:34 |
| rloo | this is our main doc page, right? http://docs.openstack.org/developer/ironic/ | 19:35 |
| rloo | do you see the admin guide first, or the developer guide first? | 19:35 |
| jroll | rloo: developer, why? | 19:36 |
| rloo | jroll: that's an old version. | 19:36 |
| rloo | jroll: we changed after last summit, to have admin guides first. | 19:36 |
| jroll | :/ | 19:36 |
| rloo | jroll: i just generated the docs and my local version is correct. | 19:36 |
| Sukhdev | devananda: I have seen a very nice picture (almost a year ago) in one of the ironic presentations - which shows the connectivity of all the OVS bridges and interfaces - I can not find it anymore..... | 19:37 |
| rloo | jroll: not sure what is pushing out that doc. would one of the stable branches be doing it? | 19:37 |
| devananda | Sukhdev: I dont think I've seen that ... | 19:37 |
| jroll | rloo: I also don't see a way from docs.openstack.org to get to ironic docs :( | 19:37 |
| Sukhdev | that would have helped me follow the path to nail it down ---- as I rely on the tcpdump to track the packet path | 19:37 |
| devananda | jroll: there is a link in the developer API guides ... | 19:37 |
| rloo | jroll: yeah, it shows 2015.1.2 documentation at the bottom. so it is old. | 19:37 |
| devananda | jroll: but yah. ironic isn't publishing *user* docs to the user site | 19:38 |
| jroll | -.- | 19:38 |
| jroll | everything I ever knew is wrong | 19:38 |
| devananda | rloo: urgh. well. good catch ... now i wonder if our doc build job is broken :( | 19:38 |
| jroll | devananda: so where are our docs published? | 19:38 |
| devananda | jroll: the site rloo linked | 19:38 |
| devananda | they *should* be published there | 19:38 |
| devananda | if they're not -- then a job broke | 19:38 |
| jroll | hrm | 19:38 |
| rloo | jroll: I don't think our ironic docs were ever linked from docs.openstack.org | 19:39 |
| jroll | I feel like that's publishing kilo docs. not master. | 19:39 |
| rloo | jroll: they don't publish master. | 19:39 |
| rloo | jroll: not that i know of. | 19:39 |
| jroll | *headdesk* | 19:39 |
| *** Marga_ has joined #openstack-ironic | 19:39 | |
| *** baoli has quit IRC | 19:39 | |
| *** marzif has joined #openstack-ironic | 19:39 | |
| devananda | rloo: wha?? | 19:40 |
| devananda | rloo: i'm sure that when we first set up the job for docs.openstack.org/develper/ironic -- it was publishing master | 19:40 |
| rloo | devananda: I mean, I don't think openstack publishes master docs at docs.openstack.org | 19:40 |
| devananda | maybe something changed | 19:40 |
| devananda | i know what it did originally because originally we didn't have a stable branch :) | 19:40 |
| jroll | yeah, I don't see the latest docs change there | 19:40 |
| * jroll rewinds way back | 19:41 | |
| rloo | devananda, jroll: sorry. lets just go back to my original question. http://docs.openstack.org/developer/ironic/ has always shown master until ?? i think I looked 2 weeks ago or maybe even last week and it was right. | 19:42 |
| rloo | devananda, jroll: it looks to me like someone might have it set to only show kilo docs? | 19:43 |
| *** e0ne has quit IRC | 19:43 | |
| jroll | devananda: rloo: yeah, so I agree. the first docs commit after liberty opened isn't there. | 19:43 |
| devananda | gate jobs seem to be building the correct thing -- http://docs-draft.openstack.org/19/233819/3/gate/gate-ironic-docs/a47bef3//doc/build/html/ | 19:47 |
| jroll | stepping out for a bit | 19:53 |
| *** marzif has quit IRC | 19:54 | |
| *** mrmoje has quit IRC | 19:54 | |
| *** coreycb` is now known as coreycb | 19:55 | |
| rloo | devananda, jroll: looks like ironic kilo 2015.1.2 was just released. I wonder if something was invoked that generated the docs for that branch, and whether that caused the kilo docs to show up instead of master. maybe after we merge a patch to master, the docs will get changed to reflect master. | 19:56 |
| devananda | rloo: I was thinking that could be the case too ... | 19:57 |
| devananda | rloo: how recent was that publish completed? | 19:57 |
| *** Sukhdev has quit IRC | 19:58 | |
| rloo | devananda: don't know. i just saw email about kilo being released :) | 19:58 |
| *** mgoddard has joined #openstack-ironic | 19:59 | |
| devananda | ooh - here we go -- https://jenkins06.openstack.org/view/Overview/job/ironic-docs/213/ | 20:01 |
| devananda | Triggered by reference: refs/tags/2015.1.2 | 20:01 |
| devananda | so yes | 20:01 |
| * devananda files a bug | 20:01 | |
| rloo | devananda: ha ha. that's funny. sort of... | 20:02 |
| devananda | I mean - we should totally be publishing docs for kilo | 20:02 |
| devananda | but in a different location :) | 20:02 |
| rloo | devananda: yeah, that's why i think it is funny. IF we had a kilo/, liberty/, master/ :) | 20:02 |
| *** pelix has quit IRC | 20:06 | |
| devananda | https://bugs.launchpad.net/ironic/+bug/1505803 | 20:08 |
| openstack | Launchpad bug 1505803 in Ironic "docs published for previous releases overwrite current developer docs" [High,Confirmed] | 20:08 |
| * devananda looks at gerrit to see what he can land in master to 'fix the docs' | 20:10 | |
| *** mgoddard has quit IRC | 20:10 | |
| *** mgoddard has joined #openstack-ironic | 20:11 | |
| mrda | Morning Ironic | 20:13 |
| sinval | mrda, morning | 20:13 |
| mrda | o/ | 20:14 |
| *** harshs has joined #openstack-ironic | 20:15 | |
| thiagop | morning mrda | 20:17 |
| NobodyCam | morning mrda | 20:17 |
| thiagop | morning mrda | 20:18 |
| thiagop | da hell? irc with free will.. | 20:18 |
| *** e0ne has joined #openstack-ironic | 20:22 | |
| mrda | lots of hellos, o/ | 20:23 |
| *** derekh has quit IRC | 20:24 | |
| *** baoli has joined #openstack-ironic | 20:26 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: Make calling arguments easier to understand https://review.openstack.org/234424 | 20:27 |
| jlvillal | mrda: Hello | 20:27 |
| *** baoli has quit IRC | 20:27 | |
| *** [2]Rick has quit IRC | 20:29 | |
| *** marzif has joined #openstack-ironic | 20:30 | |
| rloo | devananda: I'm hoping this will fix/update the docs: https://review.openstack.org/#/c/233819/ | 20:31 |
| *** [3]Rick has joined #openstack-ironic | 20:32 | |
| *** cdearborn has quit IRC | 20:34 | |
| *** e0ne has quit IRC | 20:35 | |
| mrda | hey jlvillal | 20:36 |
| openstackgerrit | Thiago Paiva Brito proposed openstack/ironic: Update python-oneviewclient version https://review.openstack.org/234428 | 20:37 |
| *** ionutbalutoiu has quit IRC | 20:37 | |
| rloo | hey, morning mrda | 20:38 |
| *** sambetts has quit IRC | 20:41 | |
| *** sambetts has joined #openstack-ironic | 20:41 | |
| mrda | hi rloo | 20:47 |
| mrda | (I like the daylight sayings change in timezone here, I see more of you all :) | 20:47 |
| jroll | morning mrda :) | 20:57 |
| thiagop | Guys, I'm calling it a day | 20:58 |
| thiagop | see you tomorrow | 20:58 |
| *** thiagop has quit IRC | 20:58 | |
| *** harshs has quit IRC | 21:00 | |
| mrda | hi jroll | 21:01 |
| *** harshs has joined #openstack-ironic | 21:05 | |
| *** mgoddard has quit IRC | 21:05 | |
| *** Marga_ has quit IRC | 21:07 | |
| *** Marga_ has joined #openstack-ironic | 21:07 | |
| *** cemason has quit IRC | 21:08 | |
| jlvillal | mrda: Enjoy the daylight savings for a few more weeks. We switch on 1-November | 21:09 |
| *** cemason has joined #openstack-ironic | 21:12 | |
| jroll | devananda: tomorrow is basically the last day to release 4.2.1, we need to get this done https://review.openstack.org/#/c/231215/12 | 21:12 |
| mrda | jlvillal: I think it even gets better then, tbh. It's another hour better. | 21:12 |
| jlvillal | mrda: Oh. We go back one hour. | 21:12 |
| devananda | jroll: yarr | 21:12 |
| jlvillal | mrda: Did you go forward one hour? | 21:12 |
| jroll | yeah, don't you know australia does everything backwards? | 21:13 |
| jlvillal | mrda: I guess it is spring in Australia | 21:13 |
| jroll | :D | 21:13 |
| mrda | yes, we're heading into summer, you're heading into winter. | 21:13 |
| jlvillal | Ah, cool :) | 21:13 |
| rloo | jroll: this should be backported to liberty too: https://review.openstack.org/#/c/234389/ | 21:13 |
| jroll | rloo: urgh | 21:14 |
| rloo | jroll: it is similar to that dd command thing. | 21:14 |
| jroll | yeah. | 21:14 |
| jroll | rloo: it's funny how deploys succeeded with the last fix | 21:15 |
| jroll | and now they don't | 21:15 |
| *** lucas-dinner has quit IRC | 21:15 | |
| rloo | jroll: i was going to ask, but i decided not to. it wasn't the same person | 21:15 |
| *** trown is now known as trown|outttypeww | 21:15 | |
| mrda | jroll: s/different/better/ :) | 21:17 |
| jroll | mrda: I said backwards, not different :D | 21:27 |
| * mrda can't write regexes either | 21:28 | |
| *** thrash is now known as thrash|g0ne | 21:29 | |
| *** harshs has quit IRC | 21:34 | |
| *** harshs has joined #openstack-ironic | 21:37 | |
| *** Sukhdev has joined #openstack-ironic | 21:41 | |
| devananda | jroll: I don't understand why you added the FakeBoot to irmc driver | 21:41 |
| devananda | jroll: it's not mixed with Agent at all | 21:41 |
| jroll | isn't it? | 21:42 |
| devananda | https://review.openstack.org/#/c/231215/12/ironic/drivers/irmc.py,cm | 21:42 |
| jroll | devananda: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/irmc/deploy.py#L917 | 21:42 |
| jroll | it's mixed in with a subclass | 21:43 |
| devananda | oooh | 21:43 |
| devananda | only on the vendor interface | 21:43 |
| mrda | hey jroll | 21:44 |
| devananda | IRMCVirtualMediaAgentDeploy(base.DeployInterface) but then VendorPassthru(agent_base_vendor.BaseAgentVendor) | 21:44 |
| jroll | devananda: yah | 21:44 |
| devananda | that's wonky | 21:44 |
| devananda | ok | 21:44 |
| jroll | eh, it handles heartbeat and such | 21:44 |
| jroll | mrda: sup | 21:44 |
| mrda | jroll: if you have a moment, I wouldn't mind your thoughts on https://bugs.launchpad.net/nova/+bug/1503453 | 21:44 |
| openstack | Launchpad bug 1503453 in OpenStack Compute (nova) "unavailable ironic nodes being scheduled to" [Medium,Triaged] | 21:44 |
| jlvillal | jroll: This sounds like something you ran into: https://bugs.launchpad.net/nova/+bug/1503453 | 21:44 |
| mrda | snap | 21:44 |
| jroll | jlvillal: mrda: Madasi filed that bug, either of us could give you more info | 21:45 |
| jroll | what thoughts do you want? | 21:45 |
| jroll | it's totally a bug | 21:45 |
| *** dprince has quit IRC | 21:45 | |
| Madasi | bad nova, no cookie | 21:45 |
| mrda | so medium is about right? | 21:45 |
| jroll | I also bugged jaypipes a bit about the best way to fix that, but we never came to a resolution | 21:45 |
| jroll | idk, I guess? | 21:45 |
| mrda | ok, so still open for discussion | 21:46 |
| jlvillal | jroll: Madasi: Okay. Sounds like Madasi is actively working on it then. | 21:46 |
| jlvillal | Hello Madasi. I don't think I have ever met you before. | 21:46 |
| Madasi | jlvillal: not currently, got pulled onto other things | 21:46 |
| jroll | more like "should we add nodename to get_per_instance_usage() is super contentious" :) | 21:46 |
| jroll | mrda: ^ | 21:46 |
| Madasi | Hi. I'm mostly a lurker around here | 21:46 |
| mrda | Madasi: would you like to go and fix this one? | 21:46 |
| jlvillal | Madasi: But it should still be assigned to you? | 21:47 |
| jroll | jlvillal: tl;dr Madasi is a badass working on all the things in onmetal | 21:47 |
| Madasi | I think jroll was hoping he could tempt someone else into looking at it before I'd be free to return to it | 21:47 |
| jroll | jlvillal: presumably if mark unassigned himself, it shouldn't be assigned to him :P | 21:48 |
| jlvillal | FYI: mrda and I are doing our weekly Nova bugs related to Ironic bug scrub | 21:48 |
| Madasi | I unassigned myself in case someone else had the bandwidth | 21:48 |
| devananda | jroll: the more I look into how this affects irmc the less happy I am getting | 21:48 |
| jlvillal | jroll: Ah, I hadn't scrolled that far down! | 21:48 |
| *** bpmcl has joined #openstack-ironic | 21:48 | |
| mrda | ok, thanks jroll and Madasi - we'll leave it unassigned for now | 21:48 |
| jroll | devananda: ruh roh? | 21:48 |
| devananda | jroll: ironic/drivers/modules/deploy_utils.py ties directly into task.driver.boot.* methods as well | 21:48 |
| * mrda wanders back to the other conversation | 21:48 | |
| devananda | setting driver.boot = FakeBoot() is going to break that -- but then, if it were to hit those code paths, it's ALREADY broken | 21:49 |
| Madasi | jroll: about that nodename thing. I know nova was wanting to move away from that, what is the alternative and how will it work for us? | 21:49 |
| devananda | we're merely lucky that it doesn't ... maybe ... | 21:49 |
| jroll | devananda: I'm almost inclined to leave this driver api bug as-is, with good comms; the downstream fix is simple and well-understood, this fix is getting very complicated. | 21:49 |
| devananda | yah | 21:49 |
| jroll | Madasi: alternative is unclear at this time | 21:49 |
| devananda | jroll: the boot/deploy interface is just not a clean separation any more :( | 21:50 |
| devananda | I mean, it wasn't, but now I see how tangled it became | 21:50 |
| jroll | devananda: yar, agree | 21:50 |
| devananda | we should have a shim on top of hte drivers that allows task.driver.boot.* to fall back to task.driver.deploy.* | 21:51 |
| jroll | devananda: I actually think a good litmus test of this separation is seeing if the petitboot driver can just be a boot interface | 21:51 |
| devananda | yah | 21:51 |
| jroll | side note: I don't like our rpcapi tests | 21:51 |
| devananda | if we can just have a driver that implements power and boot -- but not deploy -- then we've done good things | 21:52 |
| devananda | jroll: the framework for them, or the coverage? | 21:52 |
| * devananda copied those from nova | 21:52 | |
| jroll | devananda: I think the framework, I need to play with it more a bit | 21:52 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: WIP: fix backward compatibility for RPC version 1.29 https://review.openstack.org/234450 | 21:53 |
| jroll | ^ context. | 21:53 |
| Madasi | jlvillal: mrda: Let me know if I need to add more details to the bug or explain anything. I tried to make it clear what I had found for anyone who tried to pick it up. | 21:53 |
| devananda | jroll: ok - i'm going to be late for an appt if I don't go now(), so, pls to write this into the release notes well ... | 21:53 |
| jroll | devananda: my #1 goal right now is that patch I just pushed, my #2 is comms/relnotes for the driver api thing... if I don't get to #2 could you do it tonight/tomorrow? | 21:54 |
| devananda | yah | 21:54 |
| mrda | thanks Madasi | 21:54 |
| jroll | thanks | 21:54 |
| NobodyCam | have a good night devananda | 21:54 |
| * jroll chugs red bull | 21:54 | |
| mrda | jroll: Red bull, the drink of PTLs :) | 21:55 |
| jroll | heh | 21:55 |
| *** Sukhdev_ has joined #openstack-ironic | 21:55 | |
| *** harshs has quit IRC | 21:55 | |
| jroll | I usually don't bother with it, but feeling busy/tired today :/ | 21:55 |
| *** penick has quit IRC | 21:56 | |
| * jaypipes hands jroll some coffee | 21:58 | |
| jroll | heh, thanks | 21:59 |
| Madasi | coke black man, coffee + soda | 22:00 |
| *** harshs has joined #openstack-ironic | 22:01 | |
| jroll | Madasi: only whiskey belongs in coffee | 22:01 |
| jlvillal | Anyone know of a way to have a bug in launchpad depend on another bug? | 22:02 |
| * jlvillal knows how to do it in Bugzilla | 22:02 | |
| jroll | I don't think there is | 22:03 |
| jlvillal | :( | 22:03 |
| jlvillal | jroll: Thanks | 22:03 |
| *** [3]Rick has quit IRC | 22:03 | |
| NobodyCam | i have not heard of such linking of bugs | 22:03 |
| jlvillal | NobodyCam: Thanks | 22:03 |
| *** kozhukalov_ has quit IRC | 22:06 | |
| Madasi | jlvillal: old and still open - https://bugs.launchpad.net/launchpad/+bug/95419 | 22:07 |
| openstack | Launchpad bug 95419 in Launchpad itself "No way to express that bug A cannot be fixed until bug B is fixed" [High,Triaged] | 22:07 |
| jlvillal | Madasi: And this blog post: http://blog.launchpad.net/general/bug-linking-part-2 From June of 2012 saying they are working on it :( | 22:08 |
| *** david-lyle has quit IRC | 22:10 | |
| jroll | maniphest will save the world | 22:11 |
| *** david-lyle has joined #openstack-ironic | 22:12 | |
| *** cemason has quit IRC | 22:17 | |
| *** cemason has joined #openstack-ironic | 22:18 | |
| *** bpmcl has quit IRC | 22:18 | |
| *** kkoski has quit IRC | 22:19 | |
| jlvillal | Madasi: mrda: We can blame lifeless for https://bugs.launchpad.net/launchpad/+bug/95419 not being fixed ;) | 22:31 |
| openstack | Launchpad bug 95419 in Launchpad itself "No way to express that bug A cannot be fixed until bug B is fixed" [High,Triaged] | 22:31 |
| jlvillal | Kidding! | 22:32 |
| lifeless | hah | 22:32 |
| mrda | :-P | 22:32 |
| mrda | lifeless: we just spent 10 minutes trying to work out how to do that thing | 22:33 |
| lifeless | mrda: did you get an answer? | 22:33 |
| jlvillal | lifeless: We just added a comment to the bug. | 22:33 |
| mrda | we discovered that the feature was planned for implementation in 2012, but didn't get any further | 22:33 |
| lifeless | that would be about when I joined HP :) | 22:34 |
| mrda | oh, so it *is* your fault lifeless :-) | 22:34 |
| jroll | lol | 22:34 |
| jroll | blame HP leadership | 22:35 |
| clif_h | how do you ignore a pep8 error on a specific line? | 22:39 |
| clif_h | ah nm # noqa | 22:40 |
| mrda | clif_h: remember two spaces before the # | 22:40 |
| jroll | yep | 22:40 |
| clif_h | 2 spaces? | 22:40 |
| clif_h | like: obj.func() # noqa | 22:40 |
| clif_h | ? | 22:40 |
| jroll | yep | 22:40 |
| mrda | this_is_a_reall_long_line_that_needs_pep8_ignoring # noqa | 22:41 |
| *** marzif has quit IRC | 22:41 | |
| clif_h | the 2 spaces thing is weird | 22:41 |
| jroll | though I suspect having noqa there would also skip that check :P | 22:41 |
| mrda | jroll: probably | 22:41 |
| *** marzif has joined #openstack-ironic | 22:41 | |
| mrda | just good practice anyways | 22:41 |
| jroll | yep | 22:41 |
| jroll | clif_h: 2 spaces thing is a pep8 rule itself | 22:41 |
| clif_h | I guess I usually dedicate lines to comments | 22:42 |
| clif_h | so i don't run into that | 22:42 |
| jroll | yeah | 22:42 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: WIP: fix backward compatibility for RPC version 1.29 https://review.openstack.org/234450 | 22:43 |
| mrda | just remember clif_h that everytime you use a # noqa you have to go fix an openstack python3 compatibility issue as penance | 22:43 |
| clif_h | haha | 22:43 |
| jroll | lol | 22:43 |
| *** willsama has joined #openstack-ironic | 22:43 | |
| clif_h | erg | 22:43 |
| jroll | I like that rule | 22:43 |
| * mrda is serious | 22:43 | |
| clif_h | its in nova so can I escape? | 22:44 |
| jroll | nop | 22:44 |
| clif_h | though its the ironic virt driver | 22:44 |
| mrda | JayF: Can you get that added to clif_h's goals? :-P | 22:44 |
| lifeless | mrda: jlvillal: just checked --- | 22:45 |
| lifeless | 11:44 < lifeless> wgrant: what happened to bug linking | 22:45 |
| lifeless | 11:44 < lifeless> wgrant: was it behind a feature flag, or never delivered? | 22:45 |
| lifeless | 11:44 < wgrant> lifeless: Never implemented. | 22:45 |
| lifeless | 11:45 < wgrant> The team was vaporised days after you left :) | 22:45 |
| jlvillal | lifeless: Ouch :( | 22:46 |
| mrda | sad panda | 22:46 |
| jroll | hah | 22:46 |
| mrda | clif_h: I am jokin btw | 22:46 |
| lifeless | they are 'vaguely' working on generic cross-object references apparently (#launchpad-dev) | 22:46 |
| clif_h | mrda: they have jokes down there!? | 22:46 |
| jlvillal | lifeless: Good timing on your part when you left. | 22:47 |
| jroll | jlvillal: or was it vaporized because he left? :) | 22:47 |
| jroll | "I can't deal with this without robert" | 22:47 |
| jlvillal | Hmmmm | 22:47 |
| jroll | :P | 22:47 |
| jroll | he does tackle the hard stuff | 22:48 |
| mrda | jroll: s/without// :-) | 22:48 |
| jroll | lol | 22:49 |
| mrda | clif_h: it's the Australian way | 22:49 |
| clif_h | :D | 22:49 |
| lifeless | mrda: .... Australian? | 22:49 |
| * jlvillal learned lifeless is in New Zealand | 22:50 | |
| lifeless | mrda: oh, parallel conversation :) got it | 22:50 |
| mrda | lifeless: sorry for the confusion. It's hard to mock people in parallel you know! | 22:50 |
| lifeless | mrda: from concurrent import future | 22:51 |
| mrda | lifeless: good job! | 22:51 |
| * mrda thinks about goroutines again | 22:51 | |
| jroll | so I don't think this is actually fixable, since we don't have a configurable cap... https://bugs.launchpad.net/ironic/+bug/1502903 | 22:57 |
| openstack | Launchpad bug 1502903 in Ironic "Rolling upgrades: backward incompatible changes in RPC methods" [High,In progress] - Assigned to Jim Rollenhagen (jim-rollenhagen) | 22:57 |
| * jroll thinks harder | 22:57 | |
| jroll | need to change something on the server side | 22:58 |
| jroll | bah | 22:58 |
| * jroll throws this patch away | 22:58 | |
| *** chlong has quit IRC | 22:59 | |
| *** sambetts has quit IRC | 23:00 | |
| *** sambetts has joined #openstack-ironic | 23:02 | |
| *** sambetts has quit IRC | 23:07 | |
| *** willsama has quit IRC | 23:08 | |
| *** sambetts has joined #openstack-ironic | 23:08 | |
| jroll | dansmith: ever changed the return value on an RPC call? (how do you do compat there?) | 23:12 |
| dansmith | jroll: you make a new call :) | 23:13 |
| jroll | hah | 23:13 |
| *** willsama has joined #openstack-ironic | 23:13 | |
| dansmith | or you pass a flag | 23:13 |
| jroll | \o/ for hard-breaking liberty upgrades | 23:13 |
| dansmith | expects_new_thing=False | 23:13 |
| jroll | https://bugs.launchpad.net/ironic/+bug/1502903 | 23:13 |
| openstack | Launchpad bug 1502903 in Ironic "Rolling upgrades: backward incompatible changes in RPC methods" [High,In progress] - Assigned to Jim Rollenhagen (jim-rollenhagen) | 23:13 |
| jroll | we dun goof'd | 23:13 |
| jroll | afaict the server side doesn't get the version that the client sent, so no way to munge it on server side at this point | 23:14 |
| jroll | well, maybe | 23:14 |
| jroll | the flag thing may work | 23:14 |
| dansmith | jroll: it gets it but you don't get to see it | 23:14 |
| jroll | right | 23:14 |
| jroll | o.m gets it, not the manager | 23:14 |
| dansmith | yeah | 23:15 |
| * jroll attempts science with a flag | 23:15 | |
| dansmith | I guess I'm surprised this is news, | 23:15 |
| dansmith | you guys haven't been doing the right thing with rpc stuff so far anyway, right? | 23:15 |
| jroll | right, though I'm not sure if we've broken compat in the past | 23:15 |
| jroll | other than this one | 23:15 |
| dansmith | I mean, last I looked you were just bumping the versions with no compat on either side | 23:15 |
| dansmith | just luck then? | 23:16 |
| jroll | heh right | 23:16 |
| jroll | luck or nobody noticed | 23:16 |
| dansmith | okay | 23:16 |
| jroll | one day from liberty release I'm inclined to WONTFIX this | 23:16 |
| jroll | devananda: rloo: side note, something cool: http://docs.openstack.org/developer/ironic/2015.1.1/ | 23:17 |
| jroll | each tag gets a thing | 23:17 |
| * jroll crosses that one off the todo list | 23:17 | |
| jroll | dansmith: thanks for help | 23:18 |
| dansmith | np! | 23:18 |
| *** harshs has quit IRC | 23:27 | |
| *** harshs has joined #openstack-ironic | 23:39 | |
| *** dims has quit IRC | 23:41 | |
| *** dims has joined #openstack-ironic | 23:42 | |
| *** r-daneel has quit IRC | 23:43 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Fix RPC API compat for 1.29 upgrade https://review.openstack.org/234450 | 23:50 |
| jroll | boom | 23:50 |
| jroll | dansmith: if you're still around, would love a review | 23:50 |
| jroll | and ironic cores :P | 23:50 |
| dansmith | jroll: gotta finish something, but will try to look in a few | 23:51 |
| jroll | thanks | 23:51 |
| * jroll steps away for a couple | 23:51 | |
| *** smoriya has joined #openstack-ironic | 23:52 | |
| *** achanda has quit IRC | 23:53 | |
| *** willsama has quit IRC | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!