You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Rohit Yadav <ro...@citrix.com> on 2012/10/04 14:10:37 UTC

Review Request: CLOUDSTACK-248: Fix rpm build for cloud-scripts

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/7436/
-----------------------------------------------------------

Review request for cloudstack and edison su.


Description
-------

CLOUDSTACK-248: Fix rpm build for cloud-scripts

Make rpm correctly handle new installs, upgrades and uninstalls.

Additional steps for the user while upgrading: (applies for mgmt server and agent)
1. Verify that files are missing (uninstalled by obsoleted cloud-agent-scripts package)
rpm -V cloud-scripts
2. Force reinstall/upgrade cloud-scripts package:
rpm -Uvh --force cloud-scripts-4.0.0-0.385.el6.4.0.x86_64.rpm
3. Verify once again, there should n't be any missing files:
rpm -V cloud-scripts

Signed-off-by: Rohit Yadav <ro...@citrix.com>

Download the original patch and git am <patch-file>:
http://patchbin.baagi.org/p?id=wfbyrz

---
Posted with bhaisaab's rbtool fork: https://github.com/bhaisaab/RBTool


This addresses bug CLOUDSTACK-248.


Diffs
-----

  cloud.spec 342b525 

Diff: https://reviews.apache.org/r/7436/diff/


Testing
-------

4.0


Thanks,

Rohit Yadav


Re: Review Request: CLOUDSTACK-248: Fix rpm build for cloud-scripts

Posted by ASF Subversion and Git Services <as...@urd.zones.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/7436/#review17957
-----------------------------------------------------------


Commit edaa72d0fc39f1c03e38b3f7f61405cc54e269bd in branch refs/heads/master from Abhinandan Prateek <ap...@apache.org>
[ https://git-wip-us.apache.org/repos/asf?p=incubator-cloudstack.git;h=edaa72d ]

CLOUDSTACK-24: mipn feature for basiczone kvmhost

Signed-off-by: Abhinandan Prateek <ap...@apache.org>


- ASF Subversion and Git Services


On Oct. 4, 2012, 12:10 p.m., Rohit Yadav wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/7436/
> -----------------------------------------------------------
> 
> (Updated Oct. 4, 2012, 12:10 p.m.)
> 
> 
> Review request for cloudstack and edison su.
> 
> 
> Description
> -------
> 
> CLOUDSTACK-248: Fix rpm build for cloud-scripts
> 
> Make rpm correctly handle new installs, upgrades and uninstalls.
> 
> Additional steps for the user while upgrading: (applies for mgmt server and agent)
> 1. Verify that files are missing (uninstalled by obsoleted cloud-agent-scripts package)
> rpm -V cloud-scripts
> 2. Force reinstall/upgrade cloud-scripts package:
> rpm -Uvh --force cloud-scripts-4.0.0-0.385.el6.4.0.x86_64.rpm
> 3. Verify once again, there should n't be any missing files:
> rpm -V cloud-scripts
> 
> Signed-off-by: Rohit Yadav <ro...@citrix.com>
> 
> Download the original patch and git am <patch-file>:
> http://patchbin.baagi.org/p?id=wfbyrz
> 
> ---
> Posted with bhaisaab's rbtool fork: https://github.com/bhaisaab/RBTool
> 
> 
> This addresses bug CLOUDSTACK-248.
> 
> 
> Diffs
> -----
> 
>   cloud.spec 342b525 
> 
> Diff: https://reviews.apache.org/r/7436/diff/
> 
> 
> Testing
> -------
> 
> 4.0
> 
> 
> Thanks,
> 
> Rohit Yadav
> 
>


Re: Review Request: CLOUDSTACK-248: Fix rpm build for cloud-scripts

Posted by Rohit Yadav <ro...@citrix.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/7436/#review12167
-----------------------------------------------------------


At present this patch is frivolous and does nothing useful.
To fix the bug, after upgrading these are the upgrade steps that should be done manually; if there is any way to automate this, please suggest;

1. Reinstall cloud-scripts package:
 rpm -Uvh --force cloud-scripts-4.0.0-0.385.el6.4.0.x86_64.rpm (build number may be different)
2. Copy host, workers, guid, etc. fields and any updated conf. variable from agent.properties to agent.properties.rpmnew and rename it to agent.properties; in /etc/cloud/agent/

- Rohit Yadav


On Oct. 4, 2012, 12:10 p.m., Rohit Yadav wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/7436/
> -----------------------------------------------------------
> 
> (Updated Oct. 4, 2012, 12:10 p.m.)
> 
> 
> Review request for cloudstack and edison su.
> 
> 
> Description
> -------
> 
> CLOUDSTACK-248: Fix rpm build for cloud-scripts
> 
> Make rpm correctly handle new installs, upgrades and uninstalls.
> 
> Additional steps for the user while upgrading: (applies for mgmt server and agent)
> 1. Verify that files are missing (uninstalled by obsoleted cloud-agent-scripts package)
> rpm -V cloud-scripts
> 2. Force reinstall/upgrade cloud-scripts package:
> rpm -Uvh --force cloud-scripts-4.0.0-0.385.el6.4.0.x86_64.rpm
> 3. Verify once again, there should n't be any missing files:
> rpm -V cloud-scripts
> 
> Signed-off-by: Rohit Yadav <ro...@citrix.com>
> 
> Download the original patch and git am <patch-file>:
> http://patchbin.baagi.org/p?id=wfbyrz
> 
> ---
> Posted with bhaisaab's rbtool fork: https://github.com/bhaisaab/RBTool
> 
> 
> This addresses bug CLOUDSTACK-248.
> 
> 
> Diffs
> -----
> 
>   cloud.spec 342b525 
> 
> Diff: https://reviews.apache.org/r/7436/diff/
> 
> 
> Testing
> -------
> 
> 4.0
> 
> 
> Thanks,
> 
> Rohit Yadav
> 
>


Re: Review Request: CLOUDSTACK-248: Fix rpm build for cloud-scripts

Posted by ASF Subversion and Git Services <as...@urd.zones.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/7436/#review17988
-----------------------------------------------------------


Commit edaa72d0fc39f1c03e38b3f7f61405cc54e269bd in branch refs/heads/Health-Check-UI from Abhinandan Prateek <ap...@apache.org>
[ https://git-wip-us.apache.org/repos/asf?p=incubator-cloudstack.git;h=edaa72d ]

CLOUDSTACK-24: mipn feature for basiczone kvmhost

Signed-off-by: Abhinandan Prateek <ap...@apache.org>


- ASF Subversion and Git Services


On Oct. 4, 2012, 12:10 p.m., Rohit Yadav wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/7436/
> -----------------------------------------------------------
> 
> (Updated Oct. 4, 2012, 12:10 p.m.)
> 
> 
> Review request for cloudstack and edison su.
> 
> 
> Description
> -------
> 
> CLOUDSTACK-248: Fix rpm build for cloud-scripts
> 
> Make rpm correctly handle new installs, upgrades and uninstalls.
> 
> Additional steps for the user while upgrading: (applies for mgmt server and agent)
> 1. Verify that files are missing (uninstalled by obsoleted cloud-agent-scripts package)
> rpm -V cloud-scripts
> 2. Force reinstall/upgrade cloud-scripts package:
> rpm -Uvh --force cloud-scripts-4.0.0-0.385.el6.4.0.x86_64.rpm
> 3. Verify once again, there should n't be any missing files:
> rpm -V cloud-scripts
> 
> Signed-off-by: Rohit Yadav <ro...@citrix.com>
> 
> Download the original patch and git am <patch-file>:
> http://patchbin.baagi.org/p?id=wfbyrz
> 
> ---
> Posted with bhaisaab's rbtool fork: https://github.com/bhaisaab/RBTool
> 
> 
> This addresses bug CLOUDSTACK-248.
> 
> 
> Diffs
> -----
> 
>   cloud.spec 342b525 
> 
> Diff: https://reviews.apache.org/r/7436/diff/
> 
> 
> Testing
> -------
> 
> 4.0
> 
> 
> Thanks,
> 
> Rohit Yadav
> 
>


Re: Review Request: CLOUDSTACK-248: Fix rpm build for cloud-scripts

Posted by ASF Subversion and Git Services <as...@urd.zones.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/7436/#review17782
-----------------------------------------------------------


Commit a49261c3b1f2d52d637f481e0bcff4b0d58d7b56 in branch refs/heads/master from Abhinandan Prateek <ap...@apache.org>
[ https://git-wip-us.apache.org/repos/asf?p=incubator-cloudstack.git;h=a49261c ]

CLOUDSTACK-24: mipn feature for basiczone

Signed-off-by: Abhinandan Prateek <ap...@apache.org>


- ASF Subversion and Git Services


On Oct. 4, 2012, 12:10 p.m., Rohit Yadav wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/7436/
> -----------------------------------------------------------
> 
> (Updated Oct. 4, 2012, 12:10 p.m.)
> 
> 
> Review request for cloudstack and edison su.
> 
> 
> Description
> -------
> 
> CLOUDSTACK-248: Fix rpm build for cloud-scripts
> 
> Make rpm correctly handle new installs, upgrades and uninstalls.
> 
> Additional steps for the user while upgrading: (applies for mgmt server and agent)
> 1. Verify that files are missing (uninstalled by obsoleted cloud-agent-scripts package)
> rpm -V cloud-scripts
> 2. Force reinstall/upgrade cloud-scripts package:
> rpm -Uvh --force cloud-scripts-4.0.0-0.385.el6.4.0.x86_64.rpm
> 3. Verify once again, there should n't be any missing files:
> rpm -V cloud-scripts
> 
> Signed-off-by: Rohit Yadav <ro...@citrix.com>
> 
> Download the original patch and git am <patch-file>:
> http://patchbin.baagi.org/p?id=wfbyrz
> 
> ---
> Posted with bhaisaab's rbtool fork: https://github.com/bhaisaab/RBTool
> 
> 
> This addresses bug CLOUDSTACK-248.
> 
> 
> Diffs
> -----
> 
>   cloud.spec 342b525 
> 
> Diff: https://reviews.apache.org/r/7436/diff/
> 
> 
> Testing
> -------
> 
> 4.0
> 
> 
> Thanks,
> 
> Rohit Yadav
> 
>