Github user sateesh-chodapuneedi commented on the issue:
https://github.com/apache/cloudstack/pull/815
@karuturi This seems to be ready for merge.
It has 2 LGTMs and tests passed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1950
@blueorangutan package
---
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 user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1950
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user SudharmaJain commented on the issue:
https://github.com/apache/cloudstack/pull/1725
LGTM on the code changes.
---
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
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1949
@shwetaag I see some new failures in Travis around component tests, can you
close and reopen this PR to rekick Travis tests? If Travis fails again, please
see the failed tests and fix them.
---
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1961
@serg38 okay, we can add this to test_data /cc @PaulAngus
---
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
GitHub user rhtyd reopened a pull request:
https://github.com/apache/cloudstack/pull/1960
[4.11/Future] CLOUDSTACK-9782: Host HA and KVM HA provider
Host-HA offers investigation, fencing and recovery mechanisms for host that
for
any reason are malfunctioning. It uses Activity an
Github user rhtyd closed the pull request at:
https://github.com/apache/cloudstack/pull/1960
---
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 user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1948
LGTM, @karuturi can this be merged?
---
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
enabl
Github user SudharmaJain commented on the issue:
https://github.com/apache/cloudstack/pull/1810
LGTM for code changes.
---
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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1950
Packaging result: âcentos6 âcentos7 âdebian. JID-538
---
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 d
Github user priyankparihar commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1813#discussion_r103154553
--- Diff: server/src/com/cloud/vm/UserVmManagerImpl.java ---
@@ -3520,27 +3520,17 @@ public UserVmVO doInTransaction(TransactionStatus
status) th
Github user priyankparihar commented on the issue:
https://github.com/apache/cloudstack/pull/1813
@borisstoyanov as far as i can see test failures are not happening because
of this change.
---
If your project is set up for it, you can reply to this email and have your
reply appear o
Github user priyankparihar commented on the issue:
https://github.com/apache/cloudstack/pull/1779
@rhtyd and @syed. If everything looks good. Please provide your precious
LGTM.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as w
Github user priyankparihar commented on the issue:
https://github.com/apache/cloudstack/pull/1775
@borisstoyanov please trigger smoke test.
---
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 user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1813
@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep
you posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
re
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1813
@blueorangutan package
---
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 an
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1775
@blueorangutan package
---
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 an
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1775
@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep
you posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
re
Github user SudharmaJain commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1972#discussion_r103159426
--- Diff: engine/schema/src/com/cloud/network/dao/LoadBalancerDao.java ---
@@ -29,4 +30,6 @@
List listInTransitionStateByNetworkIdAnd
Github user jayantpatil1234 commented on the issue:
https://github.com/apache/cloudstack/pull/1940
@borisstoyanov I have done required fix and now all test cases are passing,
please have a look.
---
If your project is set up for it, you can reply to this email and have your
reply ap
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1813
Packaging result: âcentos6 âcentos7 âdebian. JID-539
---
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 d
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1813
test centos7 vmware-60u2
---
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
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1940
Thanks @jayantpatil1234, will kick the tests again
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1940
@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep
you posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
re
Raja,
apt-get.eu is Wido's.
--
Sent from the Delta quadrant using Borg technology!
Nux!
www.nux.ro
- Original Message -
> From: "Raja Pullela"
> To: "dev"
> Sent: Monday, 27 February, 2017 06:17:36
> Subject: Re: Modern template hosting
> Hi will,
>
> I believe, we didn’t get to clo
Chiradeep,
I wouldn't know, needs investigating. Could be that.
--
Sent from the Delta quadrant using Borg technology!
Nux!
www.nux.ro
- Original Message -
> From: "Chiradeep Vittal"
> To: "dev"
> Sent: Sunday, 26 February, 2017 00:07:53
> Subject: Re: Modern template hosting
> Is th
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1775
Packaging result: âcentos6 âcentos7 âdebian. JID-540
---
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 d
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1813
@blueorangutan test centos7 vmware-60u2
---
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
Github user shwetaag closed the pull request at:
https://github.com/apache/cloudstack/pull/1949
---
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 user shwetaag reopened a pull request:
https://github.com/apache/cloudstack/pull/1949
Automated Cloudstack bugs 9277 9276 9275 9274 9273 9179 9178 9177
Addressed all review comments. Changed all URL to pubic available url where
ever possible . For few tests needing rhel7 te
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1940
Packaging result: âcentos6 âcentos7 âdebian. JID-541
---
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 d
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1775
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has
been kicked to run smoke tests
---
If your project is set up for it, you can reply to this email and have your
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1775
@blueorangutan test
---
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 w
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1940
@blueorangutan test
---
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 w
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1940
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has
been kicked to run smoke tests
---
If your project is set up for it, you can reply to this email and have your
Github user bvbharatk commented on the issue:
https://github.com/apache/cloudstack/pull/1971
@borisstoyanov
We do not have any unit tests for this, but we have rvr marvin test, to
test the basic functionality.
test/integration/component/maint/test_redundant_router.py .
---
Github user priyankparihar commented on the issue:
https://github.com/apache/cloudstack/pull/1774
Hi @ramkatru,
These failures/errors are not related to this code change( like in
'test_03_delete_template' template deletion is failing but we are not
disturbing delete template AP
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1971
Thanks @bvbharatk, have you run any tests? If so, could you please share
results?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user bvbharatk commented on the issue:
https://github.com/apache/cloudstack/pull/1971
@borisstoyanov
I have only run BVTs, same as what you have already posted. Will also run
the above mentioned test and post the results.
thanks,
Bharat.
---
If your project is se
GitHub user krissterckx opened a pull request:
https://github.com/apache/cloudstack-docs/pull/19
Updating The Nuage VSP Plugin Documentation for ACS 4.10 release
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/krissterckx/cloudst
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1950
LGTM. Tested manually on local Ubuntu installation. Fixed the permission
issue by fixing sudoers manually and JAVA_HOME issue by fixing the variable in
/etc/default/cloudstack.
@blueorang
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1950
@rhtyd a Trillian-Jenkins test job (ubuntu mgmt + kvm-centos7) has been
kicked to run smoke tests
---
If your project is set up for it, you can reply to this email and have your
reply app
Any follow up Koushik? I want to refactor our proof of concept and integrate it
in master.
On 21/02/17 10:42, "Kishan Kavala" wrote:
Sure Daan. I'll publish the design on cwiki and share the link.
-Original Message-
From: Daan Hoogland [mailto:daan.hoogl...@shapeblue.co
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1768
@DaanHoogland Can you post test results?
---
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 feat
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1927
merging this.
---
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,
Github user DaanHoogland commented on the issue:
https://github.com/apache/cloudstack/pull/1768
@blueorangutan test
---
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 wi
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1768
@DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has
been kicked to run smoke tests
---
If your project is set up for it, you can reply to this email and have your
r
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1927
---
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 user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1954
---
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 user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1768
@marcaurele can we have a unit test to confirm that this does not break
upgrade for older systems, also by changing the order for all older versions,
the way someone would upgrade from say 4.3 to
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1944
@borisstoyanov can you see the failure in Travis with
test_list_infrastructure_metrics test case?
Thanks @rashmidixit you're right I added this as a plugin to keep it
separate from the core AP
> Op 27 februari 2017 om 7:17 schreef Raja Pullela
> :
>
>
> Hi will,
>
> I believe, we didn’t get to close ‘getting a mirror on Apache’ because we
> needed someone on the Apache Infra side to close this. BTW,
> cloudstack-apt.get.eu (I think Nux manages this?) has all/most of the
> conten
Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1944
@rhtyd On the contrary, I think you adding metrics as pluggable API is a
good idea. However, to seriously make the feature pluggable the UI component
should also load based on a plugin based
sorry Wido… forgot… thank you for correcting!
On 2/27/17, 4:50 PM, "Wido den Hollander" wrote:
> Op 27 februari 2017 om 7:17 schreef Raja Pullela
> :
>
>
> Hi will,
>
> I believe, we didn’t get to close ‘getting a mirror on Apache’ because we
> needed someone on the Apache Infra side to clo
Github user niteshsarda commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1972#discussion_r103190716
--- Diff: engine/schema/src/com/cloud/network/dao/LoadBalancerDao.java ---
@@ -29,4 +30,6 @@
List listInTransitionStateByNetworkIdAndS
Github user SudharmaJain commented on the issue:
https://github.com/apache/cloudstack/pull/1972
@niteshsarda I don't see any change in file createNetworkOffering.xml, but
still there are many lines being removed and added. As there is not change in
this file, please revert.
A
Github user cloudmonger commented on the issue:
https://github.com/apache/cloudstack/pull/1886
### ACS CI BVT Run
**Sumarry:**
Build Number 398
Hypervisor xenserver
NetworkType Advanced
Passed=104
Failed=1
Skipped=7
_Link to logs Folder (searc
Github user prashanthvarma commented on the issue:
https://github.com/apache/cloudstack-docs/pull/19
These changes in the documentation are in accordance with our merged
feature support (including bug fix) PRs. Here are some of our merged feature
support PRs:
https://github.c
Github user prashanthvarma commented on the issue:
https://github.com/apache/cloudstack-docs/pull/19
@rhtyd @karuturi Please do review this cloudstack-docs PR, which includes
changes to the Nuage VSP Plugin documentation for ACS 4.10 release.
---
If your project is set up for it, you
Hi guys,
The next meetup is Wednesday, April 12, hosted by Interoute Prague. As always,
there will be some great speakers, the chance to meet and collaborate with
other CloudStack users, and an informal gathering afterwards over a drink!
Agenda and speakers: TBC; Venue: Interoute Czech s.r.o. S
Github user dmabry commented on the issue:
https://github.com/apache/cloudstack/pull/1954
@rhtyd Sure, no problem. How would you like to handle the back porting?
Do you want me to open PRs against 4.8 and 4.9?
---
If your project is set up for it, you can reply to this email and ha
Github user marcaurele commented on the issue:
https://github.com/apache/cloudstack/pull/1768
@rhtyd I don't see how I could write such a test case. My point is exactly
what you're saying, the upgrade would be different today for anyone who comes
from an older version, which I don't t
Github user DaanHoogland commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1960#discussion_r103212914
--- Diff:
api/src/org/apache/cloudstack/api/command/admin/ha/EnableHAForClusterCmd.java
---
@@ -0,0 +1,114 @@
+// Licensed to the Apache Softw
Github user DaanHoogland commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1960#discussion_r103213259
--- Diff:
api/src/org/apache/cloudstack/api/command/admin/ha/EnableHAForZoneCmd.java ---
@@ -0,0 +1,115 @@
+// Licensed to the Apache Software
Github user DaanHoogland commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1960#discussion_r103210828
--- Diff:
api/src/org/apache/cloudstack/api/command/admin/ha/DisableHAForClusterCmd.java
---
@@ -0,0 +1,114 @@
+// Licensed to the Apache Soft
Github user DaanHoogland commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1960#discussion_r103207344
--- Diff: .travis.yml ---
@@ -42,13 +42,18 @@ env:
smoke/test_dynamicroles
smoke/test_global_settings
Github user DaanHoogland commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1960#discussion_r103212736
--- Diff:
api/src/org/apache/cloudstack/api/command/admin/ha/DisableHAForZoneCmd.java ---
@@ -0,0 +1,115 @@
+// Licensed to the Apache Software
Github user syed commented on the issue:
https://github.com/apache/cloudstack/pull/1951
Thanks @rhtyd for confirming. LGTM :+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
e
Github user syed commented on the issue:
https://github.com/apache/cloudstack/pull/1829
@rhtyd @anshul1886 I suggest merging this first as the changes here are
very limited and tested. We can refactor the change out from #1941
---
If your project is set up for it, you can reply to
Github user syed commented on the issue:
https://github.com/apache/cloudstack/pull/1962
@mike-tutkowski If you already have incorporated these changes, I'll close
this one. If possible, please add me as reviewer when you open the other PR
---
If your project is set up for it, you ca
Github user syed closed the pull request at:
https://github.com/apache/cloudstack/pull/1962
---
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 e
Paul Angus wrote:
> Hi Nathan,
>
> Ideally, if you put the template location in (or use a template defined
> in) test_data.py then the actual location can be overridden by anyone
> testing.
>
> For Trillian, we've copied all of the templates that people have define
> to a local repo and th
What steps are needed to set up a mirror? What does Infra need to do? Has
anybody filed a ticket with Infra?
On Sun, Feb 26, 2017 at 10:17 PM, Raja Pullela
wrote:
> Hi will,
>
> I believe, we didn’t get to close ‘getting a mirror on Apache’ because we
> needed someone on the Apache Infra side to
Github user kiwiflyer commented on the issue:
https://github.com/apache/cloudstack/pull/1941
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 so,
We haven't opened a ticket yet because we don't have a strategy yet.
What do you guys think of this:
- We setup a new github repo in the 'apache' org which consists of a single
file with a list of active/supported mirrors.
- I write a small web server, distributed as a binary, which can be hosted
Sounds workable. The downloader code in the SSVM won't follow redirects I
think.
https://github.com/apache/cloudstack/blob/5511065fc20787619d9cd0444a65a3155fc9c921/core/src/com/cloud/storage/template/HttpTemplateDownloader.java#L93
https://goo.gl/dSi0r5
Might need to add
client.setRedirectStrategy
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1940
Trillian test result (tid-901)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 30078 seconds
Marvin logs:
https://github.com/blueoranguta
OK. Thanks for the heads up.
On Feb 27, 2017 1:08 PM, "Chiradeep Vittal" wrote:
> Sounds workable. The downloader code in the SSVM won't follow redirects I
> think.
> https://github.com/apache/cloudstack/blob/5511065fc20787619d9cd0444a65a3
> 155fc9c921/core/src/com/cloud/storage/template/
> Http
The alternative foolproof way is to simply not provide the automatic
download.
Just document the mirror list URL where the templates can be found. I don't
know why people reject this option.
https://shankerbalan.net/blog/seed-cloudstack-templates-offline/
On Mon, Feb 27, 2017 at 10:15 AM, Will S
My bad. A few lines down, this has been added recently:
this.request.setFollowRedirects(true);
On Mon, Feb 27, 2017 at 10:15 AM, Will Stevens
wrote:
> OK. Thanks for the heads up.
>
> On Feb 27, 2017 1:08 PM, "Chiradeep Vittal" wrote:
>
> > Sounds workable. The downloader code in the SSVM won'
I’m looking forward to attending. Thanks for host this, Interoute!
On 2/27/17, 5:54 AM, "Steve Roles" wrote:
Hi guys,
The next meetup is Wednesday, April 12, hosted by Interoute Prague. As
always, there will be some great speakers, the chance to meet and collaborate
with other Clo
GitHub user dmabry opened a pull request:
https://github.com/apache/cloudstack/pull/1973
CLOUDSTACK-9795: moved logrotate from cron.daily to cron.hourly for
vpcrouter
[BACKPORT PR1954]
moved logrotate from cron.daily to cron.hourly for vpcrouter in
cloud-early-config. This
so this is what I am looking to do. Please let me know if you have
suggestions for me or think I should be solving the problem a different way.
- We request a new Github repository from the ASF at:
'apache/cloudstack-mirror-list'
- In this repository we track a text file in the 'gh-pages' branch
Github user mike-tutkowski commented on the issue:
https://github.com/apache/cloudstack/pull/1962
@syed Yes, I can include you as a reviewer for the relevant 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 pro
GitHub user dmabry opened a pull request:
https://github.com/apache/cloudstack/pull/1974
CLOUDSTACK-9795: moved logrotate from cron.daily to cron.hourly for
vpcrouter
[BACKPORT PR1954]
moved logrotate from cron.daily to cron.hourly for vpcrouter in
cloud-early-config. This
Github user dmabry commented on the issue:
https://github.com/apache/cloudstack/pull/1954
@rhtyd I have created PRs for 4.8 and 4.9. They are PR #1973 and PR #1974
respectively.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1974
@dmabry what about using `git cherry-pick` to introduce the changes that
were already merged into master in 4.9? Then, we can maintain the same commit
hash. This would facilitate trac
I've added another project to the list.
CLOUDSTACK-9804 : Add Cinder as a storage driver to Cloudstack
The idea is to add cinder support for managed storage. Cinder is *The*
platform for doing Software defined storage. Every vendor has plugged in
their device to Cinder. We can run Cinder standal
Will, I think we could support different path structures. This can
facilitate different deployment of mirrors based on the structure the host
has.
Could I add something else to the discussion? Have we discussed the
security impacts of setting up this mirrors approach?
I mean, if any of the mirrors
Infra supports Ubuntu VMs apparently so you could do what you want there
https://www.apache.org/dev/services#virtual-servers
Another idea is to distribute the http server binary with ACS - the
CloudStack admin installs the http server and gives it a reachable fqdn /
ip and updates the templates ta
Github user dmabry commented on the issue:
https://github.com/apache/cloudstack/pull/1974
@rafaelweingartner No problem.
---
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 a
Hashes are checked (md5 IIRC) today.
But given the issues, I think the project should steer away from hosting
templates except the systemvm template.
On Mon, Feb 27, 2017 at 1:31 PM, Rafael Weingärtner <
rafaelweingart...@gmail.com> wrote:
> Will, I think we could support different path structure
Github user dmabry commented on the issue:
https://github.com/apache/cloudstack/pull/1974
Done. I'll also do a cherry pick on the other PR. Thanks for the guidance
@rafaelweingartner.
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
Github user dmabry commented on the issue:
https://github.com/apache/cloudstack/pull/1973
Did a git cherry-pick as suggested by @rafaelweingartner.
---
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
My worry is exactly with system VMs templates.
Currently, we indicate administrators to download them from
http://cloudstack.apt-get.eu/systemvm/4.6/ [1]. However, the installation
docs do not mention the expected hashes for the file that is going to be
downloaded.
Also, I do not know the code tha
@rafael: Yes, I think it makes sense to support sub-paths. It will make
things a lot easier to work with. The security question is an interesting
one. I am not sure I would be the best person to answer that. I would say
that we would want to have the MD5s hosted in the same repo as the mirror
l
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1973
@dmabry as I posted on the other PR, I only suggested `git cherry-pick`
because I thought it would preserve the commit hash. It was a misunderstanding
from me. The git hash is generate
Another thing, in my opinion, we should avoid these hashes (MD5, SHA1, and
others) where hash collisions have already been found; if we will be
working to create a new working flow for managing these system VMs
templates, I think we should instead use SHA256 or SHA512.
On Mon, Feb 27, 2017 at 4:55
Github user cloudmonger commented on the issue:
https://github.com/apache/cloudstack/pull/1965
### ACS CI BVT Run
**Sumarry:**
Build Number 400
Hypervisor xenserver
NetworkType Advanced
Passed=105
Failed=0
Skipped=7
_Link to logs Folder (searc
1 - 100 of 160 matches
Mail list logo
