Re: [openstack-dev] [Fuel] Let's remove "fuelweb" from repo paths

2014-10-13 Thread Igor Kalnitsky
Hi folks,

I want to bring this topic up again. We had a blocker in Fuel-Web
project - Evgeniy L found a bug for old releases, so I had to add data
migration.

Today I built a new ISO and it successfully passed BVT tests. So I
would ask you to merge this patches if there are no objections.

Thanks,
Igor

On Fri, Oct 10, 2014 at 2:55 PM, Igor Kalnitsky  wrote:
> Folks,
>
> BVT tests are passed successfully.
> What about merging?
>
> Thanks,
> Igor
>
> On Fri, Oct 10, 2014 at 12:40 PM, Igor Kalnitsky
>  wrote:
>> As I mentioned early, I already have an ISO with patches and it works
>> fine in my own deployment.
>>
>> However, I ran the BVT tests on centos [1] and ubuntu [2].
>>
>> [1]: 
>> http://jenkins-product.srt.mirantis.net:8080/view/custom_iso/job/custom.centos.bvt_1/198/
>> [2]: 
>> http://jenkins-product.srt.mirantis.net:8080/view/custom_iso/job/custom.ubuntu.bvt_2/170/
>>
>> On Fri, Oct 10, 2014 at 12:25 PM, Mike Scherbakov
>>  wrote:
>>> I have no objections, and essentially I'm for such initiatives at the
>>> beginning of development cycle, when risks are lower.
>>> If we ensure tests coverage, and do it carefully (for instance, building
>>> custom ISO with changes and making sure it passes BVTs), then let's do it.
>>>
>>> On Wed, Oct 8, 2014 at 9:08 PM, Igor Kalnitsky 
>>> wrote:

 Hi fuelers,

 I'm going to propose you remove "fuelweb" word from repos' paths. What
 am I talking about? Let me show you.

 Currently we have the following paths to repos:

 /var/www/nailgun/2014.2-6.0/centos/fuelweb/x86_64/
 /var/www/nailgun/2014.2-6.0/ubuntu/fuelweb/x86_64/

 Obviously, the word "fuelweb" is redundant here and doesn't reflect
 reality, because our repos contain not only fuel packages, but
 openstack.

 Moreover, fuel-upgrade script installs repos without that word
 ("fuelweb", I mean) so we have inconsistent file structure for repos,
 which may lead to problems in future.

 So I propose to do it now, while we can do it without risks and
 safety. I prepared a set of patches

 https://review.openstack.org/#/c/126885/
 https://review.openstack.org/#/c/126886/
 https://review.openstack.org/#/c/126887/

 and built an ISO #508 [1] - both master node and centos cluster was
 deployed successfully.

 Folks, please, take a look over patches above and let's merge it.

 Thanks,
 Igor


 [1]:
 http://jenkins-product.srt.mirantis.net:8080/view/custom_iso/job/custom_master_iso/508/

 ___
 OpenStack-dev mailing list
 OpenStack-dev@lists.openstack.org
 http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>>>
>>>
>>>
>>>
>>> --
>>> Mike Scherbakov
>>> #mihgen
>>>
>>>
>>> ___
>>> OpenStack-dev mailing list
>>> OpenStack-dev@lists.openstack.org
>>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>>>

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


Re: [openstack-dev] [Fuel] Let's remove "fuelweb" from repo paths

2014-10-10 Thread Igor Kalnitsky
Folks,

BVT tests are passed successfully.
What about merging?

Thanks,
Igor

On Fri, Oct 10, 2014 at 12:40 PM, Igor Kalnitsky
 wrote:
> As I mentioned early, I already have an ISO with patches and it works
> fine in my own deployment.
>
> However, I ran the BVT tests on centos [1] and ubuntu [2].
>
> [1]: 
> http://jenkins-product.srt.mirantis.net:8080/view/custom_iso/job/custom.centos.bvt_1/198/
> [2]: 
> http://jenkins-product.srt.mirantis.net:8080/view/custom_iso/job/custom.ubuntu.bvt_2/170/
>
> On Fri, Oct 10, 2014 at 12:25 PM, Mike Scherbakov
>  wrote:
>> I have no objections, and essentially I'm for such initiatives at the
>> beginning of development cycle, when risks are lower.
>> If we ensure tests coverage, and do it carefully (for instance, building
>> custom ISO with changes and making sure it passes BVTs), then let's do it.
>>
>> On Wed, Oct 8, 2014 at 9:08 PM, Igor Kalnitsky 
>> wrote:
>>>
>>> Hi fuelers,
>>>
>>> I'm going to propose you remove "fuelweb" word from repos' paths. What
>>> am I talking about? Let me show you.
>>>
>>> Currently we have the following paths to repos:
>>>
>>> /var/www/nailgun/2014.2-6.0/centos/fuelweb/x86_64/
>>> /var/www/nailgun/2014.2-6.0/ubuntu/fuelweb/x86_64/
>>>
>>> Obviously, the word "fuelweb" is redundant here and doesn't reflect
>>> reality, because our repos contain not only fuel packages, but
>>> openstack.
>>>
>>> Moreover, fuel-upgrade script installs repos without that word
>>> ("fuelweb", I mean) so we have inconsistent file structure for repos,
>>> which may lead to problems in future.
>>>
>>> So I propose to do it now, while we can do it without risks and
>>> safety. I prepared a set of patches
>>>
>>> https://review.openstack.org/#/c/126885/
>>> https://review.openstack.org/#/c/126886/
>>> https://review.openstack.org/#/c/126887/
>>>
>>> and built an ISO #508 [1] - both master node and centos cluster was
>>> deployed successfully.
>>>
>>> Folks, please, take a look over patches above and let's merge it.
>>>
>>> Thanks,
>>> Igor
>>>
>>>
>>> [1]:
>>> http://jenkins-product.srt.mirantis.net:8080/view/custom_iso/job/custom_master_iso/508/
>>>
>>> ___
>>> OpenStack-dev mailing list
>>> OpenStack-dev@lists.openstack.org
>>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>>
>>
>>
>>
>> --
>> Mike Scherbakov
>> #mihgen
>>
>>
>> ___
>> OpenStack-dev mailing list
>> OpenStack-dev@lists.openstack.org
>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>>

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


Re: [openstack-dev] [Fuel] Let's remove "fuelweb" from repo paths

2014-10-10 Thread Igor Kalnitsky
As I mentioned early, I already have an ISO with patches and it works
fine in my own deployment.

However, I ran the BVT tests on centos [1] and ubuntu [2].

[1]: 
http://jenkins-product.srt.mirantis.net:8080/view/custom_iso/job/custom.centos.bvt_1/198/
[2]: 
http://jenkins-product.srt.mirantis.net:8080/view/custom_iso/job/custom.ubuntu.bvt_2/170/

On Fri, Oct 10, 2014 at 12:25 PM, Mike Scherbakov
 wrote:
> I have no objections, and essentially I'm for such initiatives at the
> beginning of development cycle, when risks are lower.
> If we ensure tests coverage, and do it carefully (for instance, building
> custom ISO with changes and making sure it passes BVTs), then let's do it.
>
> On Wed, Oct 8, 2014 at 9:08 PM, Igor Kalnitsky 
> wrote:
>>
>> Hi fuelers,
>>
>> I'm going to propose you remove "fuelweb" word from repos' paths. What
>> am I talking about? Let me show you.
>>
>> Currently we have the following paths to repos:
>>
>> /var/www/nailgun/2014.2-6.0/centos/fuelweb/x86_64/
>> /var/www/nailgun/2014.2-6.0/ubuntu/fuelweb/x86_64/
>>
>> Obviously, the word "fuelweb" is redundant here and doesn't reflect
>> reality, because our repos contain not only fuel packages, but
>> openstack.
>>
>> Moreover, fuel-upgrade script installs repos without that word
>> ("fuelweb", I mean) so we have inconsistent file structure for repos,
>> which may lead to problems in future.
>>
>> So I propose to do it now, while we can do it without risks and
>> safety. I prepared a set of patches
>>
>> https://review.openstack.org/#/c/126885/
>> https://review.openstack.org/#/c/126886/
>> https://review.openstack.org/#/c/126887/
>>
>> and built an ISO #508 [1] - both master node and centos cluster was
>> deployed successfully.
>>
>> Folks, please, take a look over patches above and let's merge it.
>>
>> Thanks,
>> Igor
>>
>>
>> [1]:
>> http://jenkins-product.srt.mirantis.net:8080/view/custom_iso/job/custom_master_iso/508/
>>
>> ___
>> OpenStack-dev mailing list
>> OpenStack-dev@lists.openstack.org
>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>
>
>
> --
> Mike Scherbakov
> #mihgen
>
>
> ___
> OpenStack-dev mailing list
> OpenStack-dev@lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>

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


Re: [openstack-dev] [Fuel] Let's remove "fuelweb" from repo paths

2014-10-10 Thread Mike Scherbakov
I have no objections, and essentially I'm for such initiatives at the
beginning of development cycle, when risks are lower.
If we ensure tests coverage, and do it carefully (for instance, building
custom ISO with changes and making sure it passes BVTs), then let's do it.

On Wed, Oct 8, 2014 at 9:08 PM, Igor Kalnitsky 
wrote:

> Hi fuelers,
>
> I'm going to propose you remove "fuelweb" word from repos' paths. What
> am I talking about? Let me show you.
>
> Currently we have the following paths to repos:
>
> /var/www/nailgun/2014.2-6.0/centos/fuelweb/x86_64/
> /var/www/nailgun/2014.2-6.0/ubuntu/fuelweb/x86_64/
>
> Obviously, the word "fuelweb" is redundant here and doesn't reflect
> reality, because our repos contain not only fuel packages, but
> openstack.
>
> Moreover, fuel-upgrade script installs repos without that word
> ("fuelweb", I mean) so we have inconsistent file structure for repos,
> which may lead to problems in future.
>
> So I propose to do it now, while we can do it without risks and
> safety. I prepared a set of patches
>
> https://review.openstack.org/#/c/126885/
> https://review.openstack.org/#/c/126886/
> https://review.openstack.org/#/c/126887/
>
> and built an ISO #508 [1] - both master node and centos cluster was
> deployed successfully.
>
> Folks, please, take a look over patches above and let's merge it.
>
> Thanks,
> Igor
>
>
> [1]:
> http://jenkins-product.srt.mirantis.net:8080/view/custom_iso/job/custom_master_iso/508/
>
> ___
> OpenStack-dev mailing list
> OpenStack-dev@lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>



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


Re: [openstack-dev] [Fuel] Let's remove "fuelweb" from repo paths

2014-10-10 Thread Matthew Mosesohn
+1. It was a legacy item and it should go away. I'll review the patches.

On Wed, Oct 8, 2014 at 9:08 PM, Igor Kalnitsky  wrote:
> Hi fuelers,
>
> I'm going to propose you remove "fuelweb" word from repos' paths. What
> am I talking about? Let me show you.
>
> Currently we have the following paths to repos:
>
> /var/www/nailgun/2014.2-6.0/centos/fuelweb/x86_64/
> /var/www/nailgun/2014.2-6.0/ubuntu/fuelweb/x86_64/
>
> Obviously, the word "fuelweb" is redundant here and doesn't reflect
> reality, because our repos contain not only fuel packages, but
> openstack.
>
> Moreover, fuel-upgrade script installs repos without that word
> ("fuelweb", I mean) so we have inconsistent file structure for repos,
> which may lead to problems in future.
>
> So I propose to do it now, while we can do it without risks and
> safety. I prepared a set of patches
>
> https://review.openstack.org/#/c/126885/
> https://review.openstack.org/#/c/126886/
> https://review.openstack.org/#/c/126887/
>
> and built an ISO #508 [1] - both master node and centos cluster was
> deployed successfully.
>
> Folks, please, take a look over patches above and let's merge it.
>
> Thanks,
> Igor
>
>
> [1]: 
> http://jenkins-product.srt.mirantis.net:8080/view/custom_iso/job/custom_master_iso/508/
>
> ___
> OpenStack-dev mailing list
> OpenStack-dev@lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

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


Re: [openstack-dev] [Fuel] Let's remove "fuelweb" from repo paths

2014-10-10 Thread Vladimir Kuklin
I do not have any objections about removing legacy "fuelweb" suffix from
our repo URLs. So I am +1 here.

On Wed, Oct 8, 2014 at 9:08 PM, Igor Kalnitsky 
wrote:

> Hi fuelers,
>
> I'm going to propose you remove "fuelweb" word from repos' paths. What
> am I talking about? Let me show you.
>
> Currently we have the following paths to repos:
>
> /var/www/nailgun/2014.2-6.0/centos/fuelweb/x86_64/
> /var/www/nailgun/2014.2-6.0/ubuntu/fuelweb/x86_64/
>
> Obviously, the word "fuelweb" is redundant here and doesn't reflect
> reality, because our repos contain not only fuel packages, but
> openstack.
>
> Moreover, fuel-upgrade script installs repos without that word
> ("fuelweb", I mean) so we have inconsistent file structure for repos,
> which may lead to problems in future.
>
> So I propose to do it now, while we can do it without risks and
> safety. I prepared a set of patches
>
> https://review.openstack.org/#/c/126885/
> https://review.openstack.org/#/c/126886/
> https://review.openstack.org/#/c/126887/
>
> and built an ISO #508 [1] - both master node and centos cluster was
> deployed successfully.
>
> Folks, please, take a look over patches above and let's merge it.
>
> Thanks,
> Igor
>
>
> [1]:
> http://jenkins-product.srt.mirantis.net:8080/view/custom_iso/job/custom_master_iso/508/
>
> ___
> OpenStack-dev mailing list
> OpenStack-dev@lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>



-- 
Yours Faithfully,
Vladimir Kuklin,
Fuel Library Tech Lead,
Mirantis, Inc.
+7 (495) 640-49-04
+7 (926) 702-39-68
Skype kuklinvv
45bk3, Vorontsovskaya Str.
Moscow, Russia,
www.mirantis.com 
www.mirantis.ru
vkuk...@mirantis.com
___
OpenStack-dev mailing list
OpenStack-dev@lists.openstack.org
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


[openstack-dev] [Fuel] Let's remove "fuelweb" from repo paths

2014-10-08 Thread Igor Kalnitsky
Hi fuelers,

I'm going to propose you remove "fuelweb" word from repos' paths. What
am I talking about? Let me show you.

Currently we have the following paths to repos:

/var/www/nailgun/2014.2-6.0/centos/fuelweb/x86_64/
/var/www/nailgun/2014.2-6.0/ubuntu/fuelweb/x86_64/

Obviously, the word "fuelweb" is redundant here and doesn't reflect
reality, because our repos contain not only fuel packages, but
openstack.

Moreover, fuel-upgrade script installs repos without that word
("fuelweb", I mean) so we have inconsistent file structure for repos,
which may lead to problems in future.

So I propose to do it now, while we can do it without risks and
safety. I prepared a set of patches

https://review.openstack.org/#/c/126885/
https://review.openstack.org/#/c/126886/
https://review.openstack.org/#/c/126887/

and built an ISO #508 [1] - both master node and centos cluster was
deployed successfully.

Folks, please, take a look over patches above and let's merge it.

Thanks,
Igor


[1]: 
http://jenkins-product.srt.mirantis.net:8080/view/custom_iso/job/custom_master_iso/508/

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