Re: [PR] system vm destroy behaviour [cloudstack-documentation]

2025-01-16 Thread via GitHub
DaanHoogland merged PR #468: URL: https://github.com/apache/cloudstack-documentation/pull/468 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubs

Re: [PR] system vm destroy behaviour [cloudstack-documentation]

2025-01-09 Thread via GitHub
rajujith closed pull request #416: system vm destroy behaviour URL: https://github.com/apache/cloudstack-documentation/pull/416 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment

Re: [PR] system vm destroy behaviour [cloudstack-documentation]

2025-01-09 Thread via GitHub
rajujith commented on PR #416: URL: https://github.com/apache/cloudstack-documentation/pull/416#issuecomment-2581800870 @DaanHoogland created https://github.com/apache/cloudstack-documentation/pull/468 -- This is an automated message from the Apache Git Service. To respond to the messag

[PR] system vm destroy behaviour [cloudstack-documentation]

2025-01-09 Thread via GitHub
rajujith opened a new pull request, #468: URL: https://github.com/apache/cloudstack-documentation/pull/468 system vm destroy behaviour -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [PR] system vm destroy behaviour [cloudstack-documentation]

2025-01-03 Thread via GitHub
DaanHoogland commented on PR #416: URL: https://github.com/apache/cloudstack-documentation/pull/416#issuecomment-2569172132 It will pull in a lot from main if I just change the target branch @rajujith . Can you rebase it on 4.19? This would require you to do something like ``` git

Re: [PR] system vm destroy behaviour [cloudstack-documentation]

2024-12-29 Thread via GitHub
rajujith commented on PR #416: URL: https://github.com/apache/cloudstack-documentation/pull/416#issuecomment-2565074196 > looks good, is this only for main or can this be added to older versions as well? It can be added to older versions as well. -- This is an automated message fr

Re: [PR] system vm destroy behaviour [cloudstack-documentation]

2024-12-18 Thread via GitHub
DaanHoogland commented on code in PR #416: URL: https://github.com/apache/cloudstack-documentation/pull/416#discussion_r1890402972 ## source/adminguide/systemvm.rst: ## @@ -17,7 +17,7 @@ CloudStack uses several types of system Instances to perform tasks in the cloud. In gener

[PR] system vm destroy behaviour [cloudstack-documentation]

2024-07-23 Thread via GitHub
rajujith opened a new pull request, #416: URL: https://github.com/apache/cloudstack-documentation/pull/416 Adding a note on how system vm expunging is handled. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[DISCUSS] Timeout for Command execution on System VM & Agents

2023-09-27 Thread Vishesh Jindal
Hi all, I came across an interesting bug today which can result in high wait times on System VMs & agents. While processing a Command on system VM or an agent, even if the management server is no longer waiting for an answer the Command still keeps on getting processed. This can lead to

[GitHub] [cloudstack-documentation] rohityadavcloud merged pull request #328: Fix typo in docs about System VM Template

2023-09-14 Thread via GitHub
rohityadavcloud merged PR #328: URL: https://github.com/apache/cloudstack-documentation/pull/328 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-uns

[GitHub] [cloudstack-documentation] rohityadavcloud merged pull request #345: Add notes on System VM live patch

2023-09-14 Thread via GitHub
rohityadavcloud merged PR #345: URL: https://github.com/apache/cloudstack-documentation/pull/345 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-uns

[GitHub] [cloudstack-documentation] rohityadavcloud commented on a diff in pull request #345: Add notes on System VM live patch

2023-09-14 Thread via GitHub
cleanup is initiated to during live patching to ensure all rules are re-applied. - **NOTE:** In case there is an absolute need to upgrade the system VM template due to availability of - security patches or update in a package provided by the template, then the old workflow of recreating

[GitHub] [cloudstack-documentation] weizhouapache commented on pull request #345: Add notes on System VM live patch

2023-09-13 Thread via GitHub
weizhouapache commented on PR #345: URL: https://github.com/apache/cloudstack-documentation/pull/345#issuecomment-1717566414 looks good https://qa.cloudstack.cloud/builds/docs-build/pr/345/upgrading/upgrade/_sysvm_restart.html ![image](https://github.com/apache/cloudstack-documentati

[GitHub] [cloudstack-documentation] Pearl1594 commented on pull request #345: Add notes on System VM live patch

2023-09-13 Thread via GitHub
Pearl1594 commented on PR #345: URL: https://github.com/apache/cloudstack-documentation/pull/345#issuecomment-1717533724 @blueorangutan docbuild -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to t

[GitHub] [cloudstack-documentation] blueorangutan commented on pull request #345: Add notes on System VM live patch

2023-09-13 Thread via GitHub
blueorangutan commented on PR #345: URL: https://github.com/apache/cloudstack-documentation/pull/345#issuecomment-1717538325 QA-Doc build preview: https://qa.cloudstack.cloud/builds/docs-build/pr/345. (QA-JID 119) -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [cloudstack-documentation] blueorangutan commented on pull request #345: Add notes on System VM live patch

2023-09-13 Thread via GitHub
blueorangutan commented on PR #345: URL: https://github.com/apache/cloudstack-documentation/pull/345#issuecomment-1717534459 @Pearl1594 a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress. -- This is an automated message from the Apache Git Servic

[GitHub] [cloudstack-documentation] blueorangutan commented on pull request #345: Add notes on System VM live patch

2023-09-13 Thread via GitHub
blueorangutan commented on PR #345: URL: https://github.com/apache/cloudstack-documentation/pull/345#issuecomment-1717529838 QA-Doc build preview: https://qa.cloudstack.cloud/builds/docs-build/pr/345. (QA-JID 118) -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [cloudstack-documentation] blueorangutan commented on pull request #345: Add notes on System VM live patch

2023-09-13 Thread via GitHub
blueorangutan commented on PR #345: URL: https://github.com/apache/cloudstack-documentation/pull/345#issuecomment-1717528450 @Pearl1594 a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress. -- This is an automated message from the Apache Git Servic

[GitHub] [cloudstack-documentation] Pearl1594 commented on pull request #345: Add notes on System VM live patch

2023-09-13 Thread via GitHub
Pearl1594 commented on PR #345: URL: https://github.com/apache/cloudstack-documentation/pull/345#issuecomment-1717526822 @blueorangutan docbuild -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to t

[GitHub] [cloudstack-documentation] Pearl1594 opened a new pull request, #345: Add notes on System VM live patch

2023-09-13 Thread via GitHub
Pearl1594 opened a new pull request, #345: URL: https://github.com/apache/cloudstack-documentation/pull/345 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[GitHub] [cloudstack-documentation] pedrofurtado opened a new pull request, #328: Fix typo in docs about System VM Template

2023-07-02 Thread via GitHub
pedrofurtado opened a new pull request, #328: URL: https://github.com/apache/cloudstack-documentation/pull/328 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment.

[GitHub] [cloudstack-documentation] rhtyd merged pull request #134: kvm: direct download templates for system VM

2020-06-05 Thread GitBox
rhtyd merged pull request #134: URL: https://github.com/apache/cloudstack-documentation/pull/134 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[GitHub] [cloudstack-documentation] ACSGitBot commented on pull request #134: kvm: direct download templates for system VM

2020-06-02 Thread GitBox
ACSGitBot commented on pull request #134: URL: https://github.com/apache/cloudstack-documentation/pull/134#issuecomment-637516625 Build finished. You can review it at: https://acs-www.shapeblue.com/docs/WIP-PROOFING/pr134 Build Log Output: No log errors found to report

[GitHub] [cloudstack-documentation] ACSGitBot commented on pull request #134: kvm: direct download templates for system VM

2020-06-02 Thread GitBox
ACSGitBot commented on pull request #134: URL: https://github.com/apache/cloudstack-documentation/pull/134#issuecomment-637515504 Your request had been received, i'll go and build the documentation and check the output log for errors. This shouldn't take long.

[GitHub] [cloudstack-documentation] shwstppr commented on pull request #134: kvm: direct download templates for system VM

2020-06-02 Thread GitBox
shwstppr commented on pull request #134: URL: https://github.com/apache/cloudstack-documentation/pull/134#issuecomment-637515453 request docbuild This is an automated message from the Apache Git Service. To respond to the me

[GitHub] [cloudstack-documentation] shwstppr commented on pull request #134: kvm: direct download templates for system VM

2020-06-02 Thread GitBox
shwstppr commented on pull request #134: URL: https://github.com/apache/cloudstack-documentation/pull/134#issuecomment-637515746 @andrijapanicsb should be okay now This is an automated message from the Apache Git Service. To

[GitHub] [cloudstack-documentation] andrijapanicsb commented on pull request #134: kvm: direct download templates for system VM

2020-06-02 Thread GitBox
andrijapanicsb commented on pull request #134: URL: https://github.com/apache/cloudstack-documentation/pull/134#issuecomment-637492571 @shwstppr can you please rebased against 4.14 branch - as we need the fix in 4.14 Docs (and I will forward merge later to master) - thx -

[GitHub] [cloudstack-documentation] andrijapanicsb commented on pull request #134: kvm: direct download templates for system VM

2020-06-02 Thread GitBox
andrijapanicsb commented on pull request #134: URL: https://github.com/apache/cloudstack-documentation/pull/134#issuecomment-637491957 @nvazquez can we also please include the improvements around supporting local/sharedMount points please, and also make correction/update that template is

[GitHub] [cloudstack-documentation] andrijapanicsb commented on a change in pull request #134: kvm: direct download templates for system VM

2020-06-02 Thread GitBox
template: After the template is registered, it is automatically available for VM deployments. +From CloudStack 4.14.0, system VM also support direct download templates. An administrator can register a new system VM template as ROUTING or USER type with the direct download flag, and it can be

[GitHub] [cloudstack-documentation] shwstppr commented on pull request #134: kvm: direct download templates for system VM

2020-06-02 Thread GitBox
shwstppr commented on pull request #134: URL: https://github.com/apache/cloudstack-documentation/pull/134#issuecomment-637462649 @andrijapanicsb please check This is an automated message from the Apache Git Service. To respo

[GitHub] [cloudstack-documentation] ACSGitBot commented on pull request #134: kvm: direct download templates for system VM

2020-06-02 Thread GitBox
ACSGitBot commented on pull request #134: URL: https://github.com/apache/cloudstack-documentation/pull/134#issuecomment-637460039 Build finished. You can review it at: https://acs-www.shapeblue.com/docs/WIP-PROOFING/pr134 Build Log Output: No log errors found to report

[GitHub] [cloudstack-documentation] shwstppr commented on pull request #134: kvm: direct download templates for system VM

2020-06-02 Thread GitBox
shwstppr commented on pull request #134: URL: https://github.com/apache/cloudstack-documentation/pull/134#issuecomment-637458994 request docbuild This is an automated message from the Apache Git Service. To respond to the me

[GitHub] [cloudstack-documentation] ACSGitBot commented on pull request #134: kvm: direct download templates for system VM

2020-06-02 Thread GitBox
ACSGitBot commented on pull request #134: URL: https://github.com/apache/cloudstack-documentation/pull/134#issuecomment-637459037 Your request had been received, i'll go and build the documentation and check the output log for errors. This shouldn't take long.

[GitHub] [cloudstack-documentation] shwstppr opened a new pull request #134: kvm: direct download templates for system VM

2020-06-02 Thread GitBox
shwstppr opened a new pull request #134: URL: https://github.com/apache/cloudstack-documentation/pull/134 System VMs now support direct download templates. Different direct download timeouts have been made configurable with new global settings. Added with https://github.com/apache/

[GitHub] [cloudstack-primate] rhtyd merged pull request #270: view: added get diagnostics action for system vm

2020-04-17 Thread GitBox
rhtyd merged pull request #270: view: added get diagnostics action for system vm URL: https://github.com/apache/cloudstack-primate/pull/270 This is an automated message from the Apache Git Service. To respond to the message

[GitHub] [cloudstack-primate] shwstppr commented on issue #270: view: added get diagnostics action for system vm

2020-04-16 Thread GitBox
shwstppr commented on issue #270: view: added get diagnostics action for system vm URL: https://github.com/apache/cloudstack-primate/pull/270#issuecomment-615039569 @rhtyd this can be reviewed This is an automated message

[GitHub] [cloudstack-primate] shwstppr opened a new pull request #270: view: added get diagnostics action for system vm

2020-03-30 Thread GitBox
shwstppr opened a new pull request #270: view: added get diagnostics action for system vm URL: https://github.com/apache/cloudstack-primate/pull/270 Fixes #201 Fixes #202 Added action for Get Diagnostics Data for system VMs, VR ![Screenshot from 2020-03-30 17-20-03

Re: [ANNOUNCE] System VM Template API

2020-01-29 Thread Andrija Panic
(awesome new feature imho! )... On Wed, 29 Jan 2020, 11:56 Darrin Hüsselmann, < darrin.husselm...@shapeblue.com> wrote: > Hi All, > > There is a new feature we have been working on to minimize the effort of > installing and managing system VM templates in Cloudstack. The wiki p

[ANNOUNCE] System VM Template API

2020-01-29 Thread Darrin Hüsselmann
Hi All, There is a new feature we have been working on to minimize the effort of installing and managing system VM templates in Cloudstack. The wiki page can be found at the link and comments are welcome: https://cwiki.apache.org/confluence/display/CLOUDSTACK/System+VM+Template+API Cheers

[GitHub] [cloudstack-documentation] PaulAngus merged pull request #72: add current system vm version variable ( |sysvm64-version| )

2019-10-02 Thread GitBox
PaulAngus merged pull request #72: add current system vm version variable ( |sysvm64-version| ) URL: https://github.com/apache/cloudstack-documentation/pull/72 This is an automated message from the Apache Git Service. To

[GitHub] [cloudstack-documentation] ACSGitBot commented on issue #72: add current system vm version variable ( |sysvm64-version| )

2019-10-02 Thread GitBox
ACSGitBot commented on issue #72: add current system vm version variable ( |sysvm64-version| ) URL: https://github.com/apache/cloudstack-documentation/pull/72#issuecomment-537561258 Build finished. You can review it at: https://acs-www.shapeblue.com/docs/pr72 Build Log Output

[GitHub] [cloudstack-documentation] PaulAngus commented on issue #72: add current system vm version variable ( |sysvm64-version| )

2019-10-02 Thread GitBox
PaulAngus commented on issue #72: add current system vm version variable ( |sysvm64-version| ) URL: https://github.com/apache/cloudstack-documentation/pull/72#issuecomment-537557408 request docbuild This is an automated

[GitHub] [cloudstack-documentation] ACSGitBot commented on issue #72: add current system vm version variable ( |sysvm64-version| )

2019-10-02 Thread GitBox
ACSGitBot commented on issue #72: add current system vm version variable ( |sysvm64-version| ) URL: https://github.com/apache/cloudstack-documentation/pull/72#issuecomment-537557449 Your request had been received, i'll go and build the documentation and check the output log for e

[GitHub] [cloudstack-documentation] PaulAngus opened a new pull request #72: add current system vm version variable ( |sysvm64-version| )

2019-10-02 Thread GitBox
PaulAngus opened a new pull request #72: add current system vm version variable ( |sysvm64-version| ) URL: https://github.com/apache/cloudstack-documentation/pull/72 update 'update from' 4.11.x and 4.12.x pages with better template requirement

Re: System VM Template

2018-04-10 Thread Tutkowski, Mike
gt; > > > From: Rafael Weingärtner > Sent: Thursday, April 5, 2018 10:43:35 PM > To: dev > Subject: Re: System VM Template > > I am using this template for system VMs: > http://download.cloudstack.org/systemvm/4.11/systemvmtemplate-4.11.0

Re: System VM Template

2018-04-10 Thread Rohit Yadav
27;ve reproduced this and will try to fix this for 4.11.1.0 milestone next week. - Rohit <https://cloudstack.apache.org> From: Rafael Weingärtner Sent: Thursday, April 5, 2018 10:43:35 PM To: dev Subject: Re: System VM Template I am using this template for

Re: System VM Template

2018-04-05 Thread Rafael Weingärtner
te (because it’s stuck > in the Starting state). > > The system VM template I am working with is the following: > http://cloudstack.apt-get.eu/systemvm/4.11/ > > In case anyone sees something, I’ve included the contents of my VR’s > cloud.log file below. > > Thanks! > &g

Re: System VM Template

2018-04-05 Thread Tutkowski, Mike
e Starting state). The system VM template I am working with is the following: http://cloudstack.apt-get.eu/systemvm/4.11/ In case anyone sees something, I’ve included the contents of my VR’s cloud.log file below. Thanks! Thu Apr 5 16:45:01 UTC 2018 Executing c

Re: System VM Template

2018-04-05 Thread Tutkowski, Mike
Upgrade” column says, “Yes”. However, when I try to initiate the upgrade, I get an error message stating that the VR is not in the proper state (because it’s stuck in the Starting state). The system VM template I am working with is the following: http://cloudstack.apt-get.eu/systemvm/4.11/ In case

Re: System VM Template

2018-04-04 Thread Rafael Weingärtner
Hey Mike, This week I have been using ACS 4.12 to do some testing. VRs and system VMs are deploying just fine with the system VM template of 4.11. Of course, by using this template (the 4.11) I am not receiving the changes already made to it in both 4.11 and current master branch. During my

System VM Template

2018-04-02 Thread Tutkowski, Mike
Hi, I may have missed an e-mail about this recently. Can someone provide me with the current URL I can use to download system VM templates for 4.12? I’ve tried 4.11 from here: http://cloudstack.apt-get.eu/systemvm/4.11/ and master from here: https://builds.cloudstack.org/job/build-master

[GitHub] cloudstack pull request #2023: BUG-ID: CLOUDSTACK-9808 Added system Vm upgra...

2017-04-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/2023 --- 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

[GitHub] cloudstack issue #2023: BUG-ID: CLOUDSTACK-9808 Added system Vm upgrade path...

2017-04-06 Thread wido
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/2023 LGTM --- 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 feat

[GitHub] cloudstack issue #2023: BUG-ID: CLOUDSTACK-9808 Added system Vm upgrade path...

2017-04-03 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/2023 LGTM --- 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 fea

[GitHub] cloudstack issue #2023: BUG-ID: CLOUDSTACK-9808 Added system Vm upgrade path...

2017-03-30 Thread DaanHoogland
Github user DaanHoogland commented on the issue: https://github.com/apache/cloudstack/pull/2023 @karuturi I think it is easiest to just reapply and not search for the cause of an 8 month old PR resulting in conflicts. (if the original code has the right URLS in it) --- If your proje

[GitHub] cloudstack issue #2023: BUG-ID: CLOUDSTACK-9808 Added system Vm upgrade path...

2017-03-30 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/2023 code LGTM. Need to check why xenserver and KVM still have wrong urls on PR https://github.com/apache/cloudstack/pull/1582#issuecomment-288310796 --- If your project is set up for it, you can

[GitHub] cloudstack issue #2023: BUG-ID: CLOUDSTACK-9808 Added system Vm upgrade path...

2017-03-30 Thread DaanHoogland
Github user DaanHoogland commented on the issue: https://github.com/apache/cloudstack/pull/2023 LGTM @kishankavala did you check what went wrong with the kvm template. @karuturi reported that and it was probably some auto-conflict-resolution. --- If your project is set up for it,

[GitHub] cloudstack pull request #2023: BUG-ID: CLOUDSTACK-9808 Added system Vm upgra...

2017-03-30 Thread kishankavala
GitHub user kishankavala opened a pull request: https://github.com/apache/cloudstack/pull/2023 BUG-ID: CLOUDSTACK-9808 Added system Vm upgrade path for LXC and OVM `mysql> select id,unique_name,name,type,url,updated,created from vm_template where type=

[GitHub] cloudstack pull request #1915: CLOUDSTACK-9746 system-vm: logrotate config c...

2017-02-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1915 --- 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

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

2017-02-26 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1915 LGTM @karuturi this is ready for merge Ping @abhinandanprateek in case you want to review? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHu

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

2017-02-22 Thread dmabry
Github user dmabry commented on the issue: https://github.com/apache/cloudstack/pull/1915 LGTM. tag:mergeready --- 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

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

2017-02-21 Thread dmabry
Github user dmabry commented on the issue: https://github.com/apache/cloudstack/pull/1915 @serbaut Can you do a force push to kick off jenkins again. I'm guessing Jenkins just had an issue and not the PR. --- If your project is set up for it, you can reply to this email and have you

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

2017-02-21 Thread serbaut
Github user serbaut commented on the issue: https://github.com/apache/cloudstack/pull/1915 Ok. I removed it from rsyslog since it should be safe 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

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

2017-02-21 Thread leprechau
Github user leprechau commented on the issue: https://github.com/apache/cloudstack/pull/1915 We always want `compress` ... but the only time you need or want `delaycompress` is if you can't be sure that the program writing to the log can be successfully told to stop appending to that

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

2017-02-21 Thread serbaut
Github user serbaut commented on the issue: https://github.com/apache/cloudstack/pull/1915 Is it safe to remove delaycompress across the board, I assume it is there for a reason? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

2017-02-21 Thread dmabry
Github user dmabry commented on the issue: https://github.com/apache/cloudstack/pull/1915 @serbaut I agree with @ustcweizhou. Please remove delaycompress and up to 10. I'd like to get this PR in as it is the second part of the problem resolution for my issue. After that LGTM. ---

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

2017-02-20 Thread kiwiflyer
Github user kiwiflyer commented on the issue: https://github.com/apache/cloudstack/pull/1915 We pulled this in and tested it alongside https://github.com/apache/cloudstack/pull/1954 ![image](https://cloud.githubusercontent.com/assets/17278194/23148479/b8c73e42-f7ab-11e6-8ffc-

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

2017-02-20 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1915 @serbaut maxsize is fine for me. Could you please remove delaycompress, and increase rotate in systemvm/patches/debian/config/etc/logrotate.d/cloud (eg from 4 to 10) ? LTGM after that

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

2017-02-20 Thread serbaut
Github user serbaut commented on the issue: https://github.com/apache/cloudstack/pull/1915 @dmabry yes we are using redundant VPC routers. Using `size` instead of `maxsize` would delay the rotation of logs until they reach `size`. 10 logfiles with size 10MB could consume 200M

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

2017-02-20 Thread dmabry
Github user dmabry commented on the issue: https://github.com/apache/cloudstack/pull/1915 @ustcweizhou Thanks for taking the time to test as well. I confirm in our testing over the weekend that we were able to consistently create/destroy/expunge VMs every 2 minutes without issue. Be

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

2017-02-20 Thread dmabry
Github user dmabry commented on the issue: https://github.com/apache/cloudstack/pull/1915 I have submitted a PR against master that fixes cloud-early-config. https://github.com/apache/cloudstack/pull/1954 --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

2017-02-20 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1915 1. As @dmabry said, the logrotate is moved from cron.daily to cron.hourly in isolated network VR in commit. We should make similar change for VPC VR and VR for shared network. ```

Re: Modify the system vm build scripts

2017-02-17 Thread Daan Hoogland
nice explanation Will, c&p to the wiki and docs, i'd say On Fri, Feb 17, 2017 at 6:06 PM, Will Stevens wrote: > Yes. Exactly. The "patches" don't require a new system vm template to be > distributed, but changes to the "appliance" does require a new syst

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

2017-02-17 Thread dmabry
Github user dmabry commented on the issue: https://github.com/apache/cloudstack/pull/1915 @serbaut Just curious, are you using VPC routers? --- 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 thi

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

2017-02-17 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1915 @dmabry I will test it and update you. --- 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 fea

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

2017-02-17 Thread dmabry
Github user dmabry commented on the issue: https://github.com/apache/cloudstack/pull/1915 @ustcweizhou - Thanks for the +1 on cron.hourly move. From looking at the code, it looks like non-redundant VRs are already moving to cron.hourly, but vpcrouter isn't. In our case, we are using

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

2017-02-17 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1915 agree with @dmabry logrotate need to be moved (or copied) to cron.hourly ``` diff --git a/systemvm/patches/debian/config/etc/cron.hourly/logrotate b/systemvm/patches/debian/confi

Re: Modify the system vm build scripts

2017-02-17 Thread Will Stevens
Yes. Exactly. The "patches" don't require a new system vm template to be distributed, but changes to the "appliance" does require a new system vm template. On Feb 17, 2017 12:04 PM, "David Mabry" wrote: > Awesome. Thanks for the quick answer. That t

Re: Modify the system vm build scripts

2017-02-17 Thread David Mabry
, "williamstev...@gmail.com on behalf of Will Stevens" wrote: So the System VM is "built" from two sources. 1) https://github.com/apache/cloudstack/tree/master/tools/appliance/definitions/systemvmtemplate This defines what is actually built and is distrib

Re: Modify the system vm build scripts

2017-02-17 Thread Will Stevens
So the System VM is "built" from two sources. 1) https://github.com/apache/cloudstack/tree/master/tools/appliance/definitions/systemvmtemplate This defines what is actually built and is distributed as the SystemVM Template. You MUST use it if you change the packages included in th

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

2017-02-17 Thread dmabry
Github user dmabry commented on the issue: https://github.com/apache/cloudstack/pull/1915 Agreed with @leprechau . I don't think just changing logrotate to compress and rotate at smaller size intervals will not completely solve our issues here. We really need to adjust logrotate to

Modify the system vm build scripts

2017-02-17 Thread David Mabry
Hello everyone, I’m looking at making some changes to the system vm, but I have found that there looks like there are 2 different places in the code that “build” the systemvm. There is there is https://github.com/apache/cloudstack/tree/13bfdd71e6f52d2f613a802b3d16c9b40af7/systemvm/patches

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

2017-02-17 Thread leprechau
Github user leprechau commented on the issue: https://github.com/apache/cloudstack/pull/1915 We are seeing issues under heavy user interaction cycles with redundant VRs. In testing you can reliably completely fill the `/var/log/` partition with data primarily from `cloud.log` when yo

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

2017-02-16 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1915 @serbaut I agree with reducing the size. However, I suggest to (1) remove the delaycompress, then the files like syslog.1 or rsyslog.1 will be compressed. (2) keep rotate ,

[GitHub] cloudstack pull request #1915: CLOUDSTACK-9746 system-vm: logrotate config c...

2017-01-23 Thread serbaut
GitHub user serbaut opened a pull request: https://github.com/apache/cloudstack/pull/1915 CLOUDSTACK-9746 system-vm: logrotate config causes critical failures * rotate both daily and by size by using maxsize in stead of size * decrease the max size to 10M and reduce the number

[GitHub] cloudstack issue #1559: CLOUDSTACK-9280: System VM volumes can be expunged i...

2016-11-23 Thread ProjectMoon
Github user ProjectMoon commented on the issue: https://github.com/apache/cloudstack/pull/1559 @rhtyd, in my opinion the two commits are functionally separate enough that they should remain separate (and this might have been asked for in one of the various previous incarnations of thi

[GitHub] cloudstack issue #1559: CLOUDSTACK-9280: System VM volumes can be expunged i...

2016-11-23 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1559 Thanks @ProjectMoon can you squash your commits. @murali-reddy @abhinandanprateek can you help review this? --- If your project is set up for it, you can reply to this email and have your repl

[GitHub] cloudstack issue #1559: CLOUDSTACK-9280: System VM volumes can be expunged i...

2016-11-21 Thread ProjectMoon
Github user ProjectMoon commented on the issue: https://github.com/apache/cloudstack/pull/1559 Addressed the comments. For some reason the tests for VolumeDataFactoryImpl were being skipped. Will try to see why. --- If your project is set up for it, you can reply to this email and ha

[GitHub] cloudstack issue #1559: CLOUDSTACK-9280: System VM volumes can be expunged i...

2016-11-21 Thread ProjectMoon
Github user ProjectMoon commented on the issue: https://github.com/apache/cloudstack/pull/1559 Updated to latest 4.8. Will also address the comments too. --- 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 doe

[GitHub] cloudstack issue #1559: CLOUDSTACK-9280: System VM volumes can be expunged i...

2016-11-20 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1559 @ProjectMoon can you fix the conflicts, thanks --- 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 fe

[GitHub] cloudstack pull request #1559: CLOUDSTACK-9280: System VM volumes can be exp...

2016-10-17 Thread GabrielBrascher
) { return getEndPointFromHostId(hostId); } } } + +//Handle case where the volume is a volume of an expunging system VM and there are +//no other system VMs existing in the zone

[GitHub] cloudstack pull request #1559: CLOUDSTACK-9280: System VM volumes can be exp...

2016-10-17 Thread GabrielBrascher
Github user GabrielBrascher commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1559#discussion_r83628876 --- Diff: engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeDataFactoryImpl.java --- @@ -77,22 +80,51 @@ public VolumeInfo ge

[GitHub] cloudstack pull request #1559: CLOUDSTACK-9280: System VM volumes can be exp...

2016-10-17 Thread GabrielBrascher
Github user GabrielBrascher commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1559#discussion_r83647727 --- Diff: engine/storage/test/org/apache/cloudstack/storage/endpoint/DefaultEndPointSelectorTest.java --- @@ -0,0 +1,167 @@ +// Licensed to

[GitHub] cloudstack pull request #1559: CLOUDSTACK-9280: System VM volumes can be exp...

2016-10-17 Thread GabrielBrascher
Github user GabrielBrascher commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1559#discussion_r83600411 --- Diff: engine/storage/test/org/apache/cloudstack/storage/endpoint/DefaultEndPointSelectorTest.java --- @@ -0,0 +1,167 @@ +// Licensed to

[GitHub] cloudstack issue #1559: CLOUDSTACK-9280: System VM volumes can be expunged i...

2016-10-12 Thread ProjectMoon
Github user ProjectMoon commented on the issue: https://github.com/apache/cloudstack/pull/1559 Looks like Travis worked this time. --- 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

[GitHub] cloudstack pull request #1559: CLOUDSTACK-9280: System VM volumes can be exp...

2016-10-12 Thread ProjectMoon
GitHub user ProjectMoon reopened a pull request: https://github.com/apache/cloudstack/pull/1559 CLOUDSTACK-9280: System VM volumes can be expunged if no SSVM exists. This commit adds a special SSVM endpoint which simply returns true for all operations sent to it, without

[GitHub] cloudstack pull request #1559: CLOUDSTACK-9280: System VM volumes can be exp...

2016-10-12 Thread ProjectMoon
Github user ProjectMoon closed the pull request at: https://github.com/apache/cloudstack/pull/1559 --- 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 featu

[GitHub] cloudstack issue #1559: CLOUDSTACK-9280: System VM volumes can be expunged i...

2016-10-12 Thread ProjectMoon
Github user ProjectMoon commented on the issue: https://github.com/apache/cloudstack/pull/1559 The OOBM power off test keeps failing. I've run the test locally with the simulator and it succeeds fine. Going to try to kick Travis again to see if it works (third time's the charm?). --

[GitHub] cloudstack pull request #1559: CLOUDSTACK-9280: System VM volumes can be exp...

2016-10-11 Thread ProjectMoon
GitHub user ProjectMoon reopened a pull request: https://github.com/apache/cloudstack/pull/1559 CLOUDSTACK-9280: System VM volumes can be expunged if no SSVM exists. This commit adds a special SSVM endpoint which simply returns true for all operations sent to it, without

[GitHub] cloudstack pull request #1559: CLOUDSTACK-9280: System VM volumes can be exp...

2016-10-11 Thread ProjectMoon
Github user ProjectMoon closed the pull request at: https://github.com/apache/cloudstack/pull/1559 --- 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 featu

  1   2   3   4   5   6   >