Re: [openstack-dev] [ceilometer] gate-ceilometer-python33 failed because of wrong setup.py in happybase

2014-08-14 Thread Dina Belova
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

2014-08-13 Thread Osanai, Hisashi

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

2014-08-13 Thread Dina Belova
Here it is: https://review.openstack.org/#/c/113842/

Thanks,
Dina


On Wed, Aug 13, 2014 at 2:40 PM, Dina Belova  wrote:

> Julien, will do right now.
>
> Thanks
> Dina
>
>
> On Wed, Aug 13, 2014 at 2:35 PM, Julien Danjou  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

2014-08-13 Thread Dina Belova
Julien, will do right now.

Thanks
Dina


On Wed, Aug 13, 2014 at 2:35 PM, Julien Danjou  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

2014-08-13 Thread Julien Danjou
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

2014-08-13 Thread Julien Danjou
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

2014-08-13 Thread Dina Belova
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

2014-08-13 Thread Osanai, Hisashi

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

2014-08-13 Thread Julien Danjou
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

2014-08-12 Thread Osanai, Hisashi

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


Re: [openstack-dev] [ceilometer] gate-ceilometer-python33 failed because of wrong setup.py in happybase

2014-08-12 Thread Julien Danjou
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


[openstack-dev] [ceilometer] gate-ceilometer-python33 failed because of wrong setup.py in happybase

2014-08-12 Thread Osanai, Hisashi

Hi,

# I'm sending same content email again because previous subject is not suitable.

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 happybase is not OpenStack components so I would like to have advices for 
how to deal with this. 

- console.html
2014-08-08 09:17:45.901 | Downloading/unpacking happybase>=0.5,!=0.7 (from -r 
/home/jenkins/workspace/gate-ceilometer-python33/requirements.txt (line 7))
2014-08-08 09:17:45.901 |   http://pypi.openstack.org/simple/happybase/ uses an 
insecure transport scheme (http). Consider using https if pypi.openstack.org 
has it available
2014-08-08 09:17:45.901 |   Storing download in cache at 
./.tox/_download/http%3A%2F%2Fpypi.openstack.org%2Fpackages%2Fsource%2Fh%2Fhappybase%2Fhappybase-0.8.tar.gz
2014-08-08 09:17:45.901 |   Running setup.py 
(path:/home/jenkins/workspace/gate-ceilometer-python33/.tox/py33/build/happybase/setup.py)
 egg_info for package happybase
2014-08-08 09:17:45.902 | Traceback (most recent call last):
2014-08-08 09:17:45.902 |   File "", line 17, in 
2014-08-08 09:17:45.902 |   File 
"/home/jenkins/workspace/gate-ceilometer-python33/.tox/py33/build/happybase/setup.py",
 line 5, in 
2014-08-08 09:17:45.902 | execfile('happybase/_version.py')
2014-08-08 09:17:45.902 | NameError: name 'execfile' is not defined
2014-08-08 09:17:45.902 | Complete output from command python setup.py 
egg_info:
2014-08-08 09:17:45.902 | Traceback (most recent call last):
2014-08-08 09:17:45.902 | 
2014-08-08 09:17:45.902 |   File "", line 17, in 
2014-08-08 09:17:45.902 | 
2014-08-08 09:17:45.902 |   File 
"/home/jenkins/workspace/gate-ceilometer-python33/.tox/py33/build/happybase/setup.py",
 line 5, in 
2014-08-08 09:17:45.903 | 
2014-08-08 09:17:45.903 | execfile('happybase/_version.py')
2014-08-08 09:17:45.903 | 
2014-08-08 09:17:45.903 | NameError: name 'execfile' is not defined

- happybase-0.8/setup.py
1 from os.path import join, dirname
2 from setuptools import find_packages, setup
3
4 __version__ = None
5 execfile('happybase/_version.py')

- python's doc
https://docs.python.org/3.3/library/2to3.html?highlight=execfile#2to3fixer-execfile

Best Regards,
Hisashi Osanai

___
OpenStack-dev mailing list
OpenStack-dev@lists.openstack.org
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev