You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by DaanHoogland <gi...@git.apache.org> on 2015/12/04 10:07:30 UTC

[GitHub] cloudstack pull request: 4.6.0 upgrade path is to pass 4.6.1 to cr...

GitHub user DaanHoogland opened a pull request:

    https://github.com/apache/cloudstack/pull/1173

    4.6.0 upgrade path is to pass 4.6.1 to create the extra view in there

    test build done only

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/DaanHoogland/cloudstack 4.7.0-upgrade-path

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cloudstack/pull/1173.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1173
    
----
commit 3de117c8beafdfb9b602606715dc0694303b8f36
Author: Daan Hoogland <da...@onecht.net>
Date:   2015-12-04T09:06:19Z

    4.6.0 upgrade path is to pass 4.6.1 to create the extra view in there

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: 4.6.0 upgrade path is to pass 4.6.1 to cr...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/cloudstack/pull/1173


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: 4.6.0 upgrade path is to pass 4.6.1 to cr...

Posted by ustcweizhou <gi...@git.apache.org>.
Github user ustcweizhou commented on a diff in the pull request:

    https://github.com/apache/cloudstack/pull/1173#discussion_r46686423
  
    --- Diff: setup/db/db/schema-460to470.sql ---
    @@ -1,32 +0,0 @@
    --- Licensed to the Apache Software Foundation (ASF) under one
    --- or more contributor license agreements.  See the NOTICE file
    --- distributed with this work for additional information
    --- regarding copyright ownership.  The ASF licenses this file
    --- to you under the Apache License, Version 2.0 (the
    --- "License"); you may not use this file except in compliance
    --- with the License.  You may obtain a copy of the License at
    ---
    ---   http://www.apache.org/licenses/LICENSE-2.0
    ---
    --- Unless required by applicable law or agreed to in writing,
    --- software distributed under the License is distributed on an
    --- "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
    --- KIND, either express or implied.  See the License for the
    --- specific language governing permissions and limitations
    --- under the License.
    -
    ---;
    --- Schema upgrade from 4.6.0 to 4.7.0;
    ---;
    -
    -CREATE TABLE IF NOT EXISTS `cloud`.`domain_vlan_map` (
    --- End diff --
    
    yes


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: 4.6.0 upgrade path is to pass 4.6.1 to cr...

Posted by bhaisaab <gi...@git.apache.org>.
Github user bhaisaab commented on the pull request:

    https://github.com/apache/cloudstack/pull/1173#issuecomment-161929046
  
    LGTM, thanks @DaanHoogland for fixing this so fast


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: 4.6.0 upgrade path is to pass 4.6.1 to cr...

Posted by bhaisaab <gi...@git.apache.org>.
Github user bhaisaab commented on the pull request:

    https://github.com/apache/cloudstack/pull/1173#issuecomment-161929234
  
    once merged on master, I'll rebase quota-master branch (the quota PR) with latest master and then fix it's db upgrade paths.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: 4.6.0 upgrade path is to pass 4.6.1 to cr...

Posted by bhaisaab <gi...@git.apache.org>.
Github user bhaisaab commented on the pull request:

    https://github.com/apache/cloudstack/pull/1173#issuecomment-162030722
  
    Merging now, that we've enough LGTMs.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: 4.6.0 upgrade path is to pass 4.6.1 to cr...

Posted by DaanHoogland <gi...@git.apache.org>.
Github user DaanHoogland commented on a diff in the pull request:

    https://github.com/apache/cloudstack/pull/1173#discussion_r46698656
  
    --- Diff: setup/db/db/schema-460to470.sql ---
    @@ -1,32 +0,0 @@
    --- Licensed to the Apache Software Foundation (ASF) under one
    --- or more contributor license agreements.  See the NOTICE file
    --- distributed with this work for additional information
    --- regarding copyright ownership.  The ASF licenses this file
    --- to you under the Apache License, Version 2.0 (the
    --- "License"); you may not use this file except in compliance
    --- with the License.  You may obtain a copy of the License at
    ---
    ---   http://www.apache.org/licenses/LICENSE-2.0
    ---
    --- Unless required by applicable law or agreed to in writing,
    --- software distributed under the License is distributed on an
    --- "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
    --- KIND, either express or implied.  See the License for the
    --- specific language governing permissions and limitations
    --- under the License.
    -
    ---;
    --- Schema upgrade from 4.6.0 to 4.7.0;
    ---;
    -
    -CREATE TABLE IF NOT EXISTS `cloud`.`domain_vlan_map` (
    --- End diff --
    
    @ustcweizhou 460to470 is to be removed. all upgrades wil go 4.6.0 -> 4.6.1 -> 4.7.0


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: 4.6.0 upgrade path is to pass 4.6.1 to cr...

Posted by ustcweizhou <gi...@git.apache.org>.
Github user ustcweizhou commented on a diff in the pull request:

    https://github.com/apache/cloudstack/pull/1173#discussion_r46694480
  
    --- Diff: setup/db/db/schema-460to470.sql ---
    @@ -1,32 +0,0 @@
    --- Licensed to the Apache Software Foundation (ASF) under one
    --- or more contributor license agreements.  See the NOTICE file
    --- distributed with this work for additional information
    --- regarding copyright ownership.  The ASF licenses this file
    --- to you under the Apache License, Version 2.0 (the
    --- "License"); you may not use this file except in compliance
    --- with the License.  You may obtain a copy of the License at
    ---
    ---   http://www.apache.org/licenses/LICENSE-2.0
    ---
    --- Unless required by applicable law or agreed to in writing,
    --- software distributed under the License is distributed on an
    --- "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
    --- KIND, either express or implied.  See the License for the
    --- specific language governing permissions and limitations
    --- under the License.
    -
    ---;
    --- Schema upgrade from 4.6.0 to 4.7.0;
    ---;
    -
    -CREATE TABLE IF NOT EXISTS `cloud`.`domain_vlan_map` (
    --- End diff --
    
    @DaanHoogland I added it to both 460to470 and 461to470


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: 4.6.0 upgrade path is to pass 4.6.1 to cr...

Posted by DaanHoogland <gi...@git.apache.org>.
Github user DaanHoogland commented on a diff in the pull request:

    https://github.com/apache/cloudstack/pull/1173#discussion_r46687151
  
    --- Diff: setup/db/db/schema-460to470.sql ---
    @@ -1,32 +0,0 @@
    --- Licensed to the Apache Software Foundation (ASF) under one
    --- or more contributor license agreements.  See the NOTICE file
    --- distributed with this work for additional information
    --- regarding copyright ownership.  The ASF licenses this file
    --- to you under the Apache License, Version 2.0 (the
    --- "License"); you may not use this file except in compliance
    --- with the License.  You may obtain a copy of the License at
    ---
    ---   http://www.apache.org/licenses/LICENSE-2.0
    ---
    --- Unless required by applicable law or agreed to in writing,
    --- software distributed under the License is distributed on an
    --- "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
    --- KIND, either express or implied.  See the License for the
    --- specific language governing permissions and limitations
    --- under the License.
    -
    ---;
    --- Schema upgrade from 4.6.0 to 4.7.0;
    ---;
    -
    -CREATE TABLE IF NOT EXISTS `cloud`.`domain_vlan_map` (
    --- End diff --
    
    actually it is in schema-461to470


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: 4.6.0 upgrade path is to pass 4.6.1 to cr...

Posted by pdube <gi...@git.apache.org>.
Github user pdube commented on a diff in the pull request:

    https://github.com/apache/cloudstack/pull/1173#discussion_r46685020
  
    --- Diff: setup/db/db/schema-460to470.sql ---
    @@ -1,32 +0,0 @@
    --- Licensed to the Apache Software Foundation (ASF) under one
    --- or more contributor license agreements.  See the NOTICE file
    --- distributed with this work for additional information
    --- regarding copyright ownership.  The ASF licenses this file
    --- to you under the Apache License, Version 2.0 (the
    --- "License"); you may not use this file except in compliance
    --- with the License.  You may obtain a copy of the License at
    ---
    ---   http://www.apache.org/licenses/LICENSE-2.0
    ---
    --- Unless required by applicable law or agreed to in writing,
    --- software distributed under the License is distributed on an
    --- "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
    --- KIND, either express or implied.  See the License for the
    --- specific language governing permissions and limitations
    --- under the License.
    -
    ---;
    --- Schema upgrade from 4.6.0 to 4.7.0;
    ---;
    -
    -CREATE TABLE IF NOT EXISTS `cloud`.`domain_vlan_map` (
    --- End diff --
    
    Is this in the 4.6.0 to 4.6.1?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: 4.6.0 upgrade path is to pass 4.6.1 to cr...

Posted by pdube <gi...@git.apache.org>.
Github user pdube commented on the pull request:

    https://github.com/apache/cloudstack/pull/1173#issuecomment-161987586
  
    LGTM by code review. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---