Re: [openstack-dev] [ceilometer] gate-ceilometer-python33 failed because of wrong setup.py in happybase
Hisashi Osanai, np :) You're welcome :) On Thu, Aug 14, 2014 at 5:13 AM, Osanai, Hisashi osanai.hisa...@jp.fujitsu.com wrote: On Wed, Aug 13, 2014 at 2:35 PM, Julien Danjou wrote: Means the py33 needs to execute on stable/icehouse. Here I misunderstand something... Not it does not, that line in tox.ini is not use by the gate. this is a problem in the infrastructure config. Means execfile function calls on python33 in happybase is a problem. If my understanding is correct, I agree with you and I think this is the direct cause of this problem. Your idea to solve this is creating a patch for the direct cause, right? My idea to solve this is to create a patch on http://git.openstack.org/cgit/openstack-infra/config/ to exclude py33 on the stable/icehouse branch of Ceilometer in the gate. Sorry to use your time for explanation above again and thanks for it. I'm happy to have clear understanding your thought. On Wednesday, August 13, 2014 7:54 PM, Dina Belova wrote: Here it is: https://review.openstack.org/#/c/113842/ Thank you for providing the fix. I surprised the speed for it. it's really fast... Thanks again! Hisashi Osanai -- Best regards, Dina Belova Software Engineer Mirantis Inc. ___ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [ceilometer] gate-ceilometer-python33 failed because of wrong setup.py in happybase
On Wed, Aug 13 2014, Osanai, Hisashi wrote: On Tuesday, August 12, 2014 10:14 PM, Julien Danjou wrote: The py33 gate shouldn't be activated for the stable/icehouse. I'm no infra-config expert, but we should be able to patch it for that (hint?). Thank you for the response. Now we have two choices: (1) deter to activate the py33 gate (2) a patch to happybase I prefer to choose (1) first because (2) is only problem if we activate the py33 gate in stable/icehouse together with python33 and as you mentioned the py33 gate shouldn't be activated in stable/icehouse but there is the entry for the py33 gate in tox.ini so I would like to remove it from stable/icehouse. If it's ok, I make a bug report for tox.ini in stable/icehouse and commit a fix for it. (then proceed https://review.openstack.org/#/c/112806/ ahead) This is not a problem in tox.ini, this is a problem in the infrastructure config. Removing py33 from the envlist in tox.ini isn't going to fix anything unforunately. -- Julien Danjou // Free Software hacker // http://julien.danjou.info signature.asc Description: PGP signature ___ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [ceilometer] gate-ceilometer-python33 failed because of wrong setup.py in happybase
On Wednesday, August 13, 2014 5:03 PM, Julien Danjou wrote: This is not a problem in tox.ini, this is a problem in the infrastructure config. Removing py33 from the envlist in tox.ini isn't going to fix anything unforunately. Thank you for your quick response. I may misunderstand this topic. Let me clarify ... My understanding is: - the py33 failed because there is a problem that the happybase-0.8 cannot work with python33 env. (execfile function calls on python33 doesn't work) - the happybase is NOT an OpenStack component. - the py33 doesn't need to execute on stable/icehouse One idea to solve this problem is: If the py33 doesn't need to execute on stable/icehouse, just eliminate the py33. This is not a problem in tox.ini, Means the py33 needs to execute on stable/icehouse. Here I misunderstand something... this is a problem in the infrastructure config. Means execfile function calls on python33 in happybase is a problem. If my understanding is correct, I agree with you and I think this is the direct cause of this problem. Your idea to solve this is creating a patch for the direct cause, right? Thanks in advance, Hisashi Osanai ___ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [ceilometer] gate-ceilometer-python33 failed because of wrong setup.py in happybase
Hisashi Osanai, I have really strange feeling about this issue. It happens only with py33 job for icehouse branch? Because actually happy base is the same for the master code Jenkins jobs, so it looks like that exec file issue should appear in master runs as well... Do I understand everything right? As I understand Julien, he proposes to run this job only for master (as it works for now magically for master checks) and skip in for everything earlier - mostly because it won't work for stable branches anyway - as there were no fixed ceilometer code itself there. Thanks, Dina On Wed, Aug 13, 2014 at 2:11 PM, Osanai, Hisashi osanai.hisa...@jp.fujitsu.com wrote: On Wednesday, August 13, 2014 5:03 PM, Julien Danjou wrote: This is not a problem in tox.ini, this is a problem in the infrastructure config. Removing py33 from the envlist in tox.ini isn't going to fix anything unforunately. Thank you for your quick response. I may misunderstand this topic. Let me clarify ... My understanding is: - the py33 failed because there is a problem that the happybase-0.8 cannot work with python33 env. (execfile function calls on python33 doesn't work) - the happybase is NOT an OpenStack component. - the py33 doesn't need to execute on stable/icehouse One idea to solve this problem is: If the py33 doesn't need to execute on stable/icehouse, just eliminate the py33. This is not a problem in tox.ini, Means the py33 needs to execute on stable/icehouse. Here I misunderstand something... this is a problem in the infrastructure config. Means execfile function calls on python33 in happybase is a problem. If my understanding is correct, I agree with you and I think this is the direct cause of this problem. Your idea to solve this is creating a patch for the direct cause, right? Thanks in advance, Hisashi Osanai ___ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev -- Best regards, Dina Belova Software Engineer Mirantis Inc. ___ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [ceilometer] gate-ceilometer-python33 failed because of wrong setup.py in happybase
On Wed, Aug 13 2014, Osanai, Hisashi wrote: One idea to solve this problem is: If the py33 doesn't need to execute on stable/icehouse, just eliminate the py33. Yes, that IS the solution. But modifying tox.ini is not going be a working implementation of that solution. This is not a problem in tox.ini, Means the py33 needs to execute on stable/icehouse. Here I misunderstand something... Not it does not, that line in tox.ini is not use by the gate. this is a problem in the infrastructure config. Means execfile function calls on python33 in happybase is a problem. If my understanding is correct, I agree with you and I think this is the direct cause of this problem. Your idea to solve this is creating a patch for the direct cause, right? My idea to solve this is to create a patch on http://git.openstack.org/cgit/openstack-infra/config/ to exclude py33 on the stable/icehouse branch of Ceilometer in the gate. -- Julien Danjou # Free Software hacker # http://julien.danjou.info signature.asc Description: PGP signature ___ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [ceilometer] gate-ceilometer-python33 failed because of wrong setup.py in happybase
On Wed, Aug 13 2014, Dina Belova wrote: Hisashi Osanai, I have really strange feeling about this issue. It happens only with py33 job for icehouse branch? Because actually happy base is the same for the master code Jenkins jobs, so it looks like that exec file issue should appear in master runs as well... Do I understand everything right? happybase is not installed when running py33 in master because it has a requirements-py3.txt without happybase in it. Which stable/icehouse has not. As I understand Julien, he proposes to run this job only for master (as it works for now magically for master checks) and skip in for everything earlier - mostly because it won't work for stable branches anyway - as there were no fixed ceilometer code itself there. That's what I propose, and that should be done by hacking openstack-infra/config AFAIK. -- Julien Danjou -- Free Software hacker -- http://julien.danjou.info signature.asc Description: PGP signature ___ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [ceilometer] gate-ceilometer-python33 failed because of wrong setup.py in happybase
Julien, will do right now. Thanks Dina On Wed, Aug 13, 2014 at 2:35 PM, Julien Danjou jul...@danjou.info wrote: On Wed, Aug 13 2014, Osanai, Hisashi wrote: One idea to solve this problem is: If the py33 doesn't need to execute on stable/icehouse, just eliminate the py33. Yes, that IS the solution. But modifying tox.ini is not going be a working implementation of that solution. This is not a problem in tox.ini, Means the py33 needs to execute on stable/icehouse. Here I misunderstand something... Not it does not, that line in tox.ini is not use by the gate. this is a problem in the infrastructure config. Means execfile function calls on python33 in happybase is a problem. If my understanding is correct, I agree with you and I think this is the direct cause of this problem. Your idea to solve this is creating a patch for the direct cause, right? My idea to solve this is to create a patch on http://git.openstack.org/cgit/openstack-infra/config/ to exclude py33 on the stable/icehouse branch of Ceilometer in the gate. -- Julien Danjou # Free Software hacker # http://julien.danjou.info ___ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev -- Best regards, Dina Belova Software Engineer Mirantis Inc. ___ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [ceilometer] gate-ceilometer-python33 failed because of wrong setup.py in happybase
Here it is: https://review.openstack.org/#/c/113842/ Thanks, Dina On Wed, Aug 13, 2014 at 2:40 PM, Dina Belova dbel...@mirantis.com wrote: Julien, will do right now. Thanks Dina On Wed, Aug 13, 2014 at 2:35 PM, Julien Danjou jul...@danjou.info wrote: On Wed, Aug 13 2014, Osanai, Hisashi wrote: One idea to solve this problem is: If the py33 doesn't need to execute on stable/icehouse, just eliminate the py33. Yes, that IS the solution. But modifying tox.ini is not going be a working implementation of that solution. This is not a problem in tox.ini, Means the py33 needs to execute on stable/icehouse. Here I misunderstand something... Not it does not, that line in tox.ini is not use by the gate. this is a problem in the infrastructure config. Means execfile function calls on python33 in happybase is a problem. If my understanding is correct, I agree with you and I think this is the direct cause of this problem. Your idea to solve this is creating a patch for the direct cause, right? My idea to solve this is to create a patch on http://git.openstack.org/cgit/openstack-infra/config/ to exclude py33 on the stable/icehouse branch of Ceilometer in the gate. -- Julien Danjou # Free Software hacker # http://julien.danjou.info ___ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev -- Best regards, Dina Belova Software Engineer Mirantis Inc. -- Best regards, Dina Belova Software Engineer Mirantis Inc. ___ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [ceilometer] gate-ceilometer-python33 failed because of wrong setup.py in happybase
On Wed, Aug 13, 2014 at 2:35 PM, Julien Danjou wrote: Means the py33 needs to execute on stable/icehouse. Here I misunderstand something... Not it does not, that line in tox.ini is not use by the gate. this is a problem in the infrastructure config. Means execfile function calls on python33 in happybase is a problem. If my understanding is correct, I agree with you and I think this is the direct cause of this problem. Your idea to solve this is creating a patch for the direct cause, right? My idea to solve this is to create a patch on http://git.openstack.org/cgit/openstack-infra/config/ to exclude py33 on the stable/icehouse branch of Ceilometer in the gate. Sorry to use your time for explanation above again and thanks for it. I'm happy to have clear understanding your thought. On Wednesday, August 13, 2014 7:54 PM, Dina Belova wrote: Here it is: https://review.openstack.org/#/c/113842/ Thank you for providing the fix. I surprised the speed for it. it's really fast... Thanks again! Hisashi Osanai ___ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [ceilometer] gate-ceilometer-python33 failed because of wrong setup.py in happybase
On Tue, Aug 12 2014, Osanai, Hisashi wrote: I did cherry-pick for https://bugs.launchpad.net/ceilometer/+bug/1326250; and executed git review (https://review.openstack.org/#/c/112806/). In review phase I got the error message from Jenkins. The reason of the error is happybase-0.8 (latest one) uses execfile function and the usage of the function is removed from python. The py33 gate shouldn't be activated for the stable/icehouse. I'm no infra-config expert, but we should be able to patch it for that (hint?). -- Julien Danjou /* Free Software hacker http://julien.danjou.info */ signature.asc Description: PGP signature ___ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Re: [openstack-dev] [ceilometer] gate-ceilometer-python33 failed because of wrong setup.py in happybase
On Tuesday, August 12, 2014 10:14 PM, Julien Danjou wrote: The py33 gate shouldn't be activated for the stable/icehouse. I'm no infra-config expert, but we should be able to patch it for that (hint?). Thank you for the response. Now we have two choices: (1) deter to activate the py33 gate (2) a patch to happybase I prefer to choose (1) first because (2) is only problem if we activate the py33 gate in stable/icehouse together with python33 and as you mentioned the py33 gate shouldn't be activated in stable/icehouse but there is the entry for the py33 gate in tox.ini so I would like to remove it from stable/icehouse. If it's ok, I make a bug report for tox.ini in stable/icehouse and commit a fix for it. (then proceed https://review.openstack.org/#/c/112806/ ahead) What do you think? - tox.ini (stable/icehouse) 1 [tox] 2 minversion = 1.6 3 skipsdist = True 4 envlist = py26,py27,py33,pep8 Best Regards, Hisashi Osanai ___ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
