weizhouapache commented on issue #3584:
URL: https://github.com/apache/cloudstack/issues/3584#issuecomment-738643526
> I don't agree (obviously) with @weizhouapache (not sure if @ustcweizhou is
still prefered in use?) but lets give researching it a place in next cycle.
> In my personal o
This is an automated email from the ASF dual-hosted git repository.
rohit pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack-primate.git.
from 99d8f36 zone: Update physical network traffic labels (#884)
add dadeac1 storage: Close snapshot for
This is an automated email from the ASF dual-hosted git repository.
rohit pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack-primate.git
The following commit(s) were added to refs/heads/master by this push:
new 841c085 Disable edit and delete of U
sureshanaparti commented on pull request #4516:
URL: https://github.com/apache/cloudstack/pull/4516#issuecomment-738699205
@alexandru-bagu why _host.getHypervisorType()_ is null in your case, here:
https://github.com/apache/cloudstack/blob/fb1e903532d943bc98a1a8b95d4499013234a917/server/sr
DaanHoogland commented on pull request #4131:
URL: https://github.com/apache/cloudstack/pull/4131#issuecomment-738728135
not sure if this was in sync with the last critical merge yesterday so
re-running
@blueorangutan package
blueorangutan commented on pull request #4131:
URL: https://github.com/apache/cloudstack/pull/4131#issuecomment-738728642
@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
DaanHoogland commented on pull request #3595:
URL: https://github.com/apache/cloudstack/pull/3595#issuecomment-738734580
@nvazquez Is this worth moving forward for 4.16 or should we close and
restart?
This is an automated me
DaanHoogland commented on issue #3598:
URL: https://github.com/apache/cloudstack/issues/3598#issuecomment-738736864
I am not sure what the severity is for this @mike-tutkowski @skattoju4
@andrijapanicsb . Is this something we are going to support or should we just
add a line in docs somewh
DaanHoogland commented on issue #3603:
URL: https://github.com/apache/cloudstack/issues/3603#issuecomment-738738879
@div8cn I would expect reinstall to always be based on the original
template. I am not sure if it should be supported for ISO based VMs. This
actually sounds like a feature r
DaanHoogland commented on issue #3609:
URL: https://github.com/apache/cloudstack/issues/3609#issuecomment-738742575
a middle ground solution could be to start IDs for new guest osses and -
mappings at 1 from now on autoadding them as we go on and leave old mapping
at lower IDs. Users s
blueorangutan commented on pull request #4131:
URL: https://github.com/apache/cloudstack/pull/4131#issuecomment-738743140
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2459
This is an automated message from the Apache Git
DaanHoogland commented on issue #3513:
URL: https://github.com/apache/cloudstack/issues/3513#issuecomment-738745417
@DmB991 did you test on latest master?
This is an automated message from the Apache Git Service.
To respond t
DaanHoogland commented on issue #3652:
URL: https://github.com/apache/cloudstack/issues/3652#issuecomment-738746275
@nvazquez @harikrishna-patnala you did a lot of refactoring on vmware
lately, Did you encounter this as well and do you have any opinion on how to
improve?
DaanHoogland commented on issue #3664:
URL: https://github.com/apache/cloudstack/issues/3664#issuecomment-738748058
@AndreaGHG there are some improvements with respect to status processing in
the newer versions. Can you check if your issue is still happening in newer
versions? (btw The fix
DaanHoogland commented on issue #3677:
URL: https://github.com/apache/cloudstack/issues/3677#issuecomment-738748933
marking this as a feature request.
This is an automated message from the Apache Git Service.
To respond to th
rhtyd opened a new pull request #4518:
URL: https://github.com/apache/cloudstack/pull/4518
As notified in the 4.14.0.0 release notes, this deprecates the legacy UI
and moves it under the ui/legacy directory which will be served at
/client/legacy URL path. This will also ensure that use
rhtyd commented on pull request #4518:
URL: https://github.com/apache/cloudstack/pull/4518#issuecomment-738752662
This is strictly UI only change, screenshots:
 were added to refs/heads/master by this push:
new 982197f createnetworkform: Displayi
DaanHoogland commented on issue #3708:
URL: https://github.com/apache/cloudstack/issues/3708#issuecomment-738753503
if you expect the MS local time-zone to be on every VR this is a
straightforward feature to implement. I don't think this is very useful for
most clouds as they span time-zon
DaanHoogland commented on a change in pull request #4518:
URL: https://github.com/apache/cloudstack/pull/4518#discussion_r536062526
##
File path: ui/index.html
##
@@ -16,1978 +16,14 @@
specific language governing permissions and limitations
under the License.
-->
-
onitake commented on issue #3557:
URL: https://github.com/apache/cloudstack/issues/3557#issuecomment-738760809
It's not a CloudStack issue any more, but there is still a PR open to make
the documentation clearer:
https://github.com/apache/cloudstack-documentation/pull/132
I think it
DaanHoogland commented on a change in pull request #4518:
URL: https://github.com/apache/cloudstack/pull/4518#discussion_r536070687
##
File path: ui/legacy/index.html
##
@@ -0,0 +1,1993 @@
+
+http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd";>
+http://www.w3.org/1999/xhtml";>
DaanHoogland commented on issue #3708:
URL: https://github.com/apache/cloudstack/issues/3708#issuecomment-738764452
@onitake are you planning any implementation (for next release)?
This is an automated message from the Apache
DK101010 commented on pull request #4399:
URL: https://github.com/apache/cloudstack/pull/4399#issuecomment-738766622
Hi @DaanHoogland,
I have added a marvin test and tried to implement a generic solution, but a
couple of things must be adapt like host, network name and hypervisor as well
weizhouapache commented on issue #3513:
URL: https://github.com/apache/cloudstack/issues/3513#issuecomment-738767392
this is related to #3807
you can set allow.duplicate.networkname to false (it is true by default) in
global setting.
blueorangutan commented on pull request #4518:
URL: https://github.com/apache/cloudstack/pull/4518#issuecomment-738768953
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2460
This is an automated message from the Apache Git
DaanHoogland commented on pull request #4131:
URL: https://github.com/apache/cloudstack/pull/4131#issuecomment-738774268
@blueorangutan test matrix
This is an automated message from the Apache Git Service.
To respond to the m
blueorangutan commented on pull request #4131:
URL: https://github.com/apache/cloudstack/pull/4131#issuecomment-738774739
@DaanHoogland a Trillian-Jenkins matrix job (centos7 mgmt + xs71, centos7
mgmt + vmware65, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests
--
DaanHoogland commented on issue #3557:
URL: https://github.com/apache/cloudstack/issues/3557#issuecomment-738775422
I think it needs a bit of your tlc @onitake ;)
This is an automated message from the Apache Git Service.
To r
DaanHoogland commented on pull request #4399:
URL: https://github.com/apache/cloudstack/pull/4399#issuecomment-738776832
@blueorangutan package
This is an automated message from the Apache Git Service.
To respond to the messa
blueorangutan commented on pull request #4399:
URL: https://github.com/apache/cloudstack/pull/4399#issuecomment-738777223
@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
blueorangutan commented on pull request #4399:
URL: https://github.com/apache/cloudstack/pull/4399#issuecomment-738795145
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2461
This is an automated message from the Apache Git
ngrosc opened a new issue #4519:
URL: https://github.com/apache/cloudstack/issues/4519
# ISSUE TYPE
* Bug Report
# COMPONENT NAME
~~~
UI: SAML login
~~~
# CLOUDSTACK VERSION
~~~
4.14.0.0
~~~
# CONFIGURATIO
DaanHoogland commented on issue #3717:
URL: https://github.com/apache/cloudstack/issues/3717#issuecomment-738834331
seems related to a multi-tag issue with had. Can you test this with a recent
(pre)release version, @div8cn ?
DaanHoogland commented on pull request #4399:
URL: https://github.com/apache/cloudstack/pull/4399#issuecomment-738840591
@blueorangutan test matrix
This is an automated message from the Apache Git Service.
To respond to the m
blueorangutan commented on pull request #4399:
URL: https://github.com/apache/cloudstack/pull/4399#issuecomment-738841444
@DaanHoogland a Trillian-Jenkins matrix job (centos7 mgmt + xs71, centos7
mgmt + vmware65, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests
--
DaanHoogland commented on issue #3718:
URL: https://github.com/apache/cloudstack/issues/3718#issuecomment-738846614
@RoMi1981 I am guessing you have diskcontroller type IDE on that VM do you?
@GabrielBrascher Do you know if this linit is hard coded or implicit and
taken over from the bac
kiwiflyer commented on issue #4519:
URL: https://github.com/apache/cloudstack/issues/4519#issuecomment-738850954
@DaanHoogland @rhtyd Guys, this looks like a pretty nasty security bug. SAML
isn't used by default, but regardless this may be a blocker for 4.15 and 4.14.x.
--
rhtyd commented on a change in pull request #4518:
URL: https://github.com/apache/cloudstack/pull/4518#discussion_r536216136
##
File path: ui/index.html
##
@@ -16,1978 +16,14 @@
specific language governing permissions and limitations
under the License.
-->
-http://
rhtyd commented on a change in pull request #4518:
URL: https://github.com/apache/cloudstack/pull/4518#discussion_r536216643
##
File path: ui/legacy/index.html
##
@@ -0,0 +1,1993 @@
+
+http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd";>
+http://www.w3.org/1999/xhtml";>
+
+
rhtyd commented on issue #4519:
URL: https://github.com/apache/cloudstack/issues/4519#issuecomment-738883712
@kiwiflyer yes let's discuss and understand how the env was setup, I suspect
it may related to env configuration.
@ngrosc can you share details of your IDP setup? For example
rhtyd commented on issue #4519:
URL: https://github.com/apache/cloudstack/issues/4519#issuecomment-738883827
cc @PaulAngus @DaanHoogland
This is an automated message from the Apache Git Service.
To respond to the message, pl
rhtyd edited a comment on issue #4519:
URL: https://github.com/apache/cloudstack/issues/4519#issuecomment-738883712
@kiwiflyer yes let's discuss and understand how the env was setup, I suspect
it may related to env configuration.
@ngrosc can you share details of your IDP setup? For e
rhtyd commented on issue #4519:
URL: https://github.com/apache/cloudstack/issues/4519#issuecomment-73486
I did a quick code investigation, looks like when there is no encryption in
place we get the unencrypted username (`uid` usually) parameter from the SAML
token: (if I recall correct
rhtyd commented on issue #4519:
URL: https://github.com/apache/cloudstack/issues/4519#issuecomment-738893634
I guess we need exact steps of reproduction if encryption and signing are
both enabled. In which case, @ngrosc can you email me privately on rohit AT
apache.org? Thanks.
I ra
PaulAngus commented on issue #4519:
URL: https://github.com/apache/cloudstack/issues/4519#issuecomment-738907120
Hand-waving the details, it _sounds_ to me like we're saying that it's the
IDP that controls whether the token is encrypted.
IF so, I agree with Rohit; This is not a CloudStac
rhtyd commented on pull request #4518:
URL: https://github.com/apache/cloudstack/pull/4518#issuecomment-738922396
Thanks @wido
@DaanHoogland I've added the banner, can you review:
 some logs and we were going to
troubleshoot/debug in his e
andrijapanicsb commented on issue #4519:
URL: https://github.com/apache/cloudstack/issues/4519#issuecomment-738940766
+1 on what Paul and Rohit said.
That sounds the same as "if user uses insecure HTTP instead of HTTPS it's
ACS security issue"
Cloud operator is in charge of the config
kiwiflyer commented on issue #4519:
URL: https://github.com/apache/cloudstack/issues/4519#issuecomment-738949122
So, agreed that everything should be encrypted. I'm not going to claim in
anyway that I'm a SAML expert, but if the username changes, shouldn't we be
sending a new AuthnRequest
GabrielBrascher commented on issue #3718:
URL: https://github.com/apache/cloudstack/issues/3718#issuecomment-738972682
I totally agree with @DaanHoogland.
Maybe it is something that we could improve on documentation then(?) ... to
be honest I never checked this at the documentation.
GabrielBrascher edited a comment on issue #3718:
URL: https://github.com/apache/cloudstack/issues/3718#issuecomment-738972682
I totally agree with @DaanHoogland.
Maybe it is something that we could improve on documentation then(?) ... to
be honest I never checked this at the document
andrijapanicsb commented on issue #4517:
URL: https://github.com/apache/cloudstack/issues/4517#issuecomment-738976871
Identical error during restarting the VPC with the cleanup - new VR to be
created failed on the host and we never tried to start it on another host in
the cluster - no retr
ngrosc commented on issue #4519:
URL: https://github.com/apache/cloudstack/issues/4519#issuecomment-738984435
hi guys
sorry for the late answer.
we're using https, not http. so that should not be the the main issue.
as IDP, we're using keycloack with the following confi
alexandru-bagu commented on pull request #4516:
URL: https://github.com/apache/cloudstack/pull/4516#issuecomment-738990736
> @alexandru-bagu why _host.getHypervisorType()_ is null in your case, here:
>
>
https://github.com/apache/cloudstack/blob/fb1e903532d943bc98a1a8b95d4499013234a9
blueorangutan commented on pull request #4518:
URL: https://github.com/apache/cloudstack/pull/4518#issuecomment-738995695
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2462
This is an automated message from the Apache Git
alexandru-bagu opened a new issue #4520:
URL: https://github.com/apache/cloudstack/issues/4520
Setting up a test environment from master I had issues adding an xcp-ng host
failing without any information. Turns out
https://github.com/apache/cloudstack/blob/fb1e903532d943bc98a1a8b95d4499013
blueorangutan commented on pull request #4131:
URL: https://github.com/apache/cloudstack/pull/4131#issuecomment-739052727
Trillian test result (tid-3303)
Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7
Total time taken: 30978 seconds
Marvin logs:
https://gith
inmanturbo commented on issue #4509:
URL: https://github.com/apache/cloudstack/issues/4509#issuecomment-739079954
@weizhouapache Yes thank you I realized that when I went through the script.
I was actually missing /usr/bin/file (!) as well.
Anyway @DaanHoogland as well thank you bot
blueorangutan commented on pull request #4131:
URL: https://github.com/apache/cloudstack/pull/4131#issuecomment-739081212
Trillian test result (tid-3304)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 36435 seconds
Marvin logs:
https://githu
GabrielBrascher edited a comment on issue #3718:
URL: https://github.com/apache/cloudstack/issues/3718#issuecomment-738972682
I totally agree with @DaanHoogland regarding the issue of not being a bug.
And as far as I know, there is no limitation on Ceph that would be necessary to
leave a m
blueorangutan commented on pull request #4131:
URL: https://github.com/apache/cloudstack/pull/4131#issuecomment-739108809
Trillian test result (tid-3305)
Environment: vmware-65u2 (x2), Advanced Networking with Mgmt server 7
Total time taken: 45895 seconds
Marvin logs:
https://githu
blueorangutan commented on pull request #4399:
URL: https://github.com/apache/cloudstack/pull/4399#issuecomment-739111822
Trillian test result (tid-3307)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 39356 seconds
Marvin logs:
https://githu
blueorangutan commented on pull request #4399:
URL: https://github.com/apache/cloudstack/pull/4399#issuecomment-739125988
Trillian test result (tid-3308)
Environment: vmware-65u2 (x2), Advanced Networking with Mgmt server 7
Total time taken: 46883 seconds
Marvin logs:
https://githu
weizhouapache commented on a change in pull request #4516:
URL: https://github.com/apache/cloudstack/pull/4516#discussion_r536536086
##
File path:
plugins/storage/volume/default/src/main/java/org/apache/cloudstack/storage/datastore/lifecycle/CloudStackPrimaryDataStoreLifeCycleImpl.jav
69 matches
Mail list logo