[GitHub] cloudstack pull request #1282: CLOUDSTACK-9200: Fixed failed to delete snaps...

2017-04-13 Thread Slair1
Github user Slair1 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1282#discussion_r111500346 --- Diff: engine/storage/snapshot/src/org/apache/cloudstack/storage/snapshot/XenserverSnapshotStrategy.java --- @@ -245,6 +245,12 @@ public boolean

[GitHub] cloudstack issue #2041: CLOUDSTACK-9873: VR related periodic jobs are schedu...

2017-04-13 Thread Slair1
Github user Slair1 commented on the issue: https://github.com/apache/cloudstack/pull/2041 Any reason that wasn't back-ported to 4.9? --- 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

[GitHub] cloudstack pull request #2041: CLOUDSTACK-9873: VR related periodic jobs are...

2017-04-12 Thread Slair1
GitHub user Slair1 opened a pull request: https://github.com/apache/cloudstack/pull/2041 CLOUDSTACK-9873: VR related periodic jobs are scheduled twice and run twice on management servers The "start" method of VirtualNetworkApplianceManagerImpl schedules several per

[GitHub] cloudstack pull request #2040: CLOUDSTACK-9872: Gather all S2S vpn statuses ...

2017-04-12 Thread Slair1
GitHub user Slair1 opened a pull request: https://github.com/apache/cloudstack/pull/2040 CLOUDSTACK-9872: Gather all S2S vpn statuses before outputting The checkbatchs2svpn.sh VR script returns ("via echo") that status of each requested S2S VPN check one

[GitHub] cloudstack issue #1923: CLOUDSTACK-9765: centos: packaging: fix missing path...

2017-03-01 Thread Slair1
Github user Slair1 commented on the issue: https://github.com/apache/cloudstack/pull/1923 This LGTM too, we hit this issue during our upgrade --- 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

[GitHub] cloudstack pull request #1966: CLOUDSTACK-9801: IPSec VPN does not work afte...

2017-02-23 Thread Slair1
GitHub user Slair1 opened a pull request: https://github.com/apache/cloudstack/pull/1966 CLOUDSTACK-9801: IPSec VPN does not work after vRouter reboot or recreate After a vRouter is recreated (e.g. reboot via CloudStack UI) and Remote Access VPN enabled, VPN won't work anymore

[GitHub] cloudstack issue #1282: CLOUDSTACK-9200: Fixed failed to delete snapshot if ...

2017-02-09 Thread Slair1
Github user Slair1 commented on the issue: https://github.com/apache/cloudstack/pull/1282 @anshul1886 any idea why this hasn't been merged yet? are there concerns/issues? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] cloudstack pull request: Addresses CLOUDSTACK-9300 where the MySQL...

2016-05-05 Thread Slair1
Github user Slair1 commented on the pull request: https://github.com/apache/cloudstack/pull/1428#issuecomment-217338822 @swill it is confirmation the code worked for me --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] cloudstack pull request: Addresses CLOUDSTACK-9300 where the MySQL...

2016-05-02 Thread Slair1
Github user Slair1 commented on the pull request: https://github.com/apache/cloudstack/pull/1428#issuecomment-216396939 @kiwiflyer and @rhtyd I've been troubleshooting the proxy issue and it was an issue in my environment! i have things worked out and everything works fine

[GitHub] cloudstack pull request: CLOUDSTACK-9305: Cloudstack Usage Breaks ...

2016-05-02 Thread Slair1
Github user Slair1 commented on the pull request: https://github.com/apache/cloudstack/pull/1433#issuecomment-216244116 Hi @rhtyd since it uses "ls" to get a list of packages, if the directory or .jar file does not exist it is no big deal. It will output a No such file or

[GitHub] cloudstack pull request: Addresses CLOUDSTACK-9300 where the MySQL...

2016-04-29 Thread Slair1
Github user Slair1 commented on the pull request: https://github.com/apache/cloudstack/pull/1428#issuecomment-215859262 @kiwiflyer No, we were not failing over our ACS mgmt servers. We still had both up and functional and were able to log directly into both (and of course through

[GitHub] cloudstack pull request: Addresses CLOUDSTACK-9300 where the MySQL...

2016-04-29 Thread Slair1
Github user Slair1 commented on the pull request: https://github.com/apache/cloudstack/pull/1428#issuecomment-215854412 this LGTM. we have been running this in our environment for over a month (ACS 4.8). This PR fixes the issue we had with cloudstack-mangaement not starting when

[GitHub] cloudstack pull request: CLOUDSTACK-9305: Cloudstack Usage Breaks ...

2016-04-28 Thread Slair1
Github user Slair1 commented on the pull request: https://github.com/apache/cloudstack/pull/1433#issuecomment-215615141 That's ok, 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

[GitHub] cloudstack pull request: CLOUDSTACK-9200: Fixed failed to delete s...

2016-04-26 Thread Slair1
Github user Slair1 commented on the pull request: https://github.com/apache/cloudstack/pull/1282#issuecomment-214804222 Thanks! I had the same issue, retention settings for snapshots were failing because it couldn't delete two snapshots that were stuck in "allocated'

[GitHub] cloudstack pull request: CLOUDSTACK-9342: Site to Site VPN PFS not...

2016-04-08 Thread Slair1
Github user Slair1 commented on the pull request: https://github.com/apache/cloudstack/pull/1478#issuecomment-207543327 Create new PR #1480 in 4.7 branch --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-9342: Site to Site VPN PFS not...

2016-04-08 Thread Slair1
GitHub user Slair1 opened a pull request: https://github.com/apache/cloudstack/pull/1480 CLOUDSTACK-9342: Site to Site VPN PFS not being set correctly Bug in code set PFS to the same value (yes/no) as DPD. file.addeq(" pfs=%s" % CsHelper.bool_to_yn(obj['dpd'])

[GitHub] cloudstack pull request: CLOUDSTACK-9342: Site to Site VPN PFS not...

2016-04-08 Thread Slair1
Github user Slair1 closed the pull request at: https://github.com/apache/cloudstack/pull/1478 --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-9342: Site to Site VPN PFS not...

2016-04-08 Thread Slair1
Github user Slair1 commented on the pull request: https://github.com/apache/cloudstack/pull/1478#issuecomment-207531359 @kiwiflyer , wow i feel stupid now :-) --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-9342: Site to Site VPN PFS not...

2016-04-08 Thread Slair1
Github user Slair1 commented on the pull request: https://github.com/apache/cloudstack/pull/1478#issuecomment-207530559 @kiwiflyer thanks, how do i delete this PR? --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-9342: Site to Site VPN PFS not...

2016-04-08 Thread Slair1
Github user Slair1 commented on the pull request: https://github.com/apache/cloudstack/pull/1478#issuecomment-207526795 ah, thanks @kiwiflyer . How do i change this to 4.7.x? Also i check PR #872 and the PFS code looks good there, it is totally re-written. --- If your

[GitHub] cloudstack pull request: CLOUDSTACK-9342: Site to Site VPN PFS not...

2016-04-07 Thread Slair1
GitHub user Slair1 opened a pull request: https://github.com/apache/cloudstack/pull/1478 CLOUDSTACK-9342: Site to Site VPN PFS not being set correctly Bug in code set PFS to the same value (yes/no) as DPD. file.addeq(" pfs=%s" % CsHelper.bool_to_yn(obj['dpd'])

[GitHub] cloudstack pull request: CLOUDSTACK-9305: Cloudstack Usage Breaks ...

2016-04-07 Thread Slair1
Github user Slair1 commented on the pull request: https://github.com/apache/cloudstack/pull/1433#issuecomment-207150675 thanks @swill , it looks much better now, i needed the -f on the git push --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request: CLOUDSTACK-9305: Cloudstack Usage Breaks ...

2016-04-07 Thread Slair1
Github user Slair1 commented on the pull request: https://github.com/apache/cloudstack/pull/1433#issuecomment-207141177 @swill thanks, that makes sense. I did it to my branch the first time, but for some reason when i tried to save to here it just created two more commits. i'll try

[GitHub] cloudstack pull request: CLOUDSTACK-9305: Cloudstack Usage Breaks ...

2016-04-07 Thread Slair1
Github user Slair1 commented on the pull request: https://github.com/apache/cloudstack/pull/1433#issuecomment-207077724 @swill i seem to get an access denied when pushing my change: C:\slair\Source\Github\cloudstack [pr/1433]> git push origin pr/1433 --force remote: Permiss

[GitHub] cloudstack pull request: CLOUDSTACK-9305: Cloudstack Usage Breaks ...

2016-04-07 Thread Slair1
Github user Slair1 commented on the pull request: https://github.com/apache/cloudstack/pull/1433#issuecomment-207038085 @swill , i apologize, but this is my first PR. I squashed it in Github Desktop on my local copy and did a push, but it sees like it it just added additional

[GitHub] cloudstack pull request: CLOUDSTACK-9305: Cloudstack Usage Breaks ...

2016-03-11 Thread Slair1
Github user Slair1 commented on the pull request: https://github.com/apache/cloudstack/pull/1433#issuecomment-195575292 If anyone still has issues, i also had to make the change from this PR: #1409 --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: CLOUDSTACK-9305: Cloudstack Usage Breaks ...

2016-03-11 Thread Slair1
Github user Slair1 commented on the pull request: https://github.com/apache/cloudstack/pull/1433#issuecomment-195572826 @kiwiflyer Very good point, let me make a different change here so it is more generic! --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: Addresses CLOUDSTACK-9300 where the MySQL...

2016-03-10 Thread Slair1
Github user Slair1 commented on the pull request: https://github.com/apache/cloudstack/pull/1428#issuecomment-195054079 Looks like the cloudstack-usage server has a similar problem. it continuously restarts if db.ha.enable = true. i got it to work in my environment

[GitHub] cloudstack pull request: Cloudstack Usage Breaks if DB HA enabled

2016-03-09 Thread Slair1
GitHub user Slair1 opened a pull request: https://github.com/apache/cloudstack/pull/1433 Cloudstack Usage Breaks if DB HA enabled With DB HA enabled in db.properties, the cloudstack-usage service restarts every 10 seconds. Making the suggested change has fixed it for me

[GitHub] cloudstack pull request: CLOUDSTACK-9283: add pid to java argument...

2016-03-09 Thread Slair1
Github user Slair1 commented on the pull request: https://github.com/apache/cloudstack/pull/1409#issuecomment-194409665 I applied to my 4.8 source and repackaged and installed, it worked. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack pull request: CLOUDSTACK-9283: add pid to java argument...

2016-03-09 Thread Slair1
Github user Slair1 commented on the pull request: https://github.com/apache/cloudstack/pull/1409#issuecomment-194394917 FYI, this fix also worked for me, Cloudstack 4.8 on Centos7 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack pull request: Addresses CLOUDSTACK-9300 where the MySQL...

2016-03-08 Thread Slair1
Github user Slair1 commented on the pull request: https://github.com/apache/cloudstack/pull/1428#issuecomment-194096939 Thanks guys, FYI i applied these change to my 4.8 source, running on Centos7 and it fixed the problem for me also. Thanks again!! --- If your project is set up