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.git.
from 36ef850 Merge remote-tracking branch 'origin/4.14'
add ba6e2ac plugins: Redfish Client & Redfish OOBM
andrijapanicsb commented on pull request #4068:
URL: https://github.com/apache/cloudstack/pull/4068#issuecomment-665762139
The failed test in this last env, the "test_01_redundant_vpc_site2site_vpn"
from the "test_vpc_vpn.py" is passing fine when repeated manually (via margin)
- i.e.
DaanHoogland commented on pull request #4035:
URL: https://github.com/apache/cloudstack/pull/4035#issuecomment-665520825
textual changes only no furhter testing needed.
This is an automated message from the Apache Git
ravening opened a new pull request #4230:
URL: https://github.com/apache/cloudstack/pull/4230
## Description
Provide reset button to zone,cluster,domain,account,
primary and secondary storage so that config values
can be reset to the default value
## Types of changes
DaanHoogland commented on pull request #4047:
URL: https://github.com/apache/cloudstack/pull/4047#issuecomment-665506324
as per your description @ravening, "If the template from which VR is created
got deleted, the state
is set to inactive and removed to null.", it seems the 'real' bug
DaanHoogland commented on pull request #3997:
URL: https://github.com/apache/cloudstack/pull/3997#issuecomment-665510566
@kioie is this ready for review/ testing again?
And do you intent to move this forward before 4.15?
gaaray2k commented on issue #4153:
URL: https://github.com/apache/cloudstack/issues/4153#issuecomment-665621027
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
DaanHoogland commented on issue #4229:
URL: https://github.com/apache/cloudstack/issues/4229#issuecomment-665546509
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
wido commented on pull request #4111:
URL: https://github.com/apache/cloudstack/pull/4111#issuecomment-665742105
Suggestions:
- Unavailable
- Degraded
- Offline
- Defect
- Inactive
This is an automated message
weizhouapache commented on pull request #4019:
URL: https://github.com/apache/cloudstack/pull/4019#issuecomment-665507567
> code looks good @weizhouapache but two questions:
>
> 1. I see the test was deleted and not replaced. any directive on other
than manual testing (it might be
andrijapanicsb commented on issue #4153:
URL: https://github.com/apache/cloudstack/issues/4153#issuecomment-665632509
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
DaanHoogland commented on pull request #4205:
URL: https://github.com/apache/cloudstack/pull/4205#issuecomment-665598103
Nice to see this moving on. Please let us know as soon as you need review or
other feedback, @radu-todirica
DaanHoogland commented on issue #4217:
URL: https://github.com/apache/cloudstack/issues/4217#issuecomment-665532055
there is [the users list](mailto:us...@cloudstack.apache.org>) for these
questions. hope that helps.
This
DaanHoogland commented on pull request #4215:
URL: https://github.com/apache/cloudstack/pull/4215#issuecomment-665526708
This quite a functional change @ravening .
Did you test settings for domains linked to ldap?
I read in your description that you are (kind of) mixing account level
DaanHoogland commented on a change in pull request #4212:
URL: https://github.com/apache/cloudstack/pull/4212#discussion_r462174163
##
File path: server/src/main/java/com/cloud/vm/UserVmManagerImpl.java
##
@@ -5619,7 +5619,7 @@ public VirtualMachine migrateVirtualMachine(Long
blueorangutan commented on pull request #3952:
URL: https://github.com/apache/cloudstack/pull/3952#issuecomment-665796142
Trillian test result (tid-2223)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 33746 seconds
Marvin logs:
Spaceman1984 commented on a change in pull request #4207:
URL: https://github.com/apache/cloudstack/pull/4207#discussion_r462165542
##
File path: utils/src/test/java/com/cloud/utils/HumanReadableJsonTest.java
##
@@ -0,0 +1,53 @@
+//
+// Licensed to the Apache Software
ravening commented on pull request #4200:
URL: https://github.com/apache/cloudstack/pull/4200#issuecomment-665579509
> @ravening when creating an offering in the domain admins domain, are those
automatically visible in subdomains and in parent domain(-chain)?
it will be visible only
blueorangutan commented on pull request #4150:
URL: https://github.com/apache/cloudstack/pull/4150#issuecomment-665766362
Trillian test result (tid-2221)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 38742 seconds
Marvin logs:
rhtyd closed pull request #4225:
URL: https://github.com/apache/cloudstack/pull/4225
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
DaanHoogland merged pull request #4035:
URL: https://github.com/apache/cloudstack/pull/4035
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
blueorangutan commented on pull request #4220:
URL: https://github.com/apache/cloudstack/pull/4220#issuecomment-665569508
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
andrijapanicsb commented on issue #4227:
URL: https://github.com/apache/cloudstack/issues/4227#issuecomment-665799984
@gaaray2k the correct place for QUESTIONS is the mailing list - you'll get
better help there.
- USER = just username/password (or API/SECRET key)
blueorangutan commented on pull request #4226:
URL: https://github.com/apache/cloudstack/pull/4226#issuecomment-665726754
Trillian test result (tid-2217)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 39959 seconds
Marvin logs:
Spaceman1984 commented on pull request #4225:
URL: https://github.com/apache/cloudstack/pull/4225#issuecomment-665738409
@rhtyd I had a look at this, the physical size is reflecting correctly, but
that utilization field is a bit strange on the front end, seems like the value
should be
wido commented on pull request #4175:
URL: https://github.com/apache/cloudstack/pull/4175#issuecomment-665553410
@DaanHoogland The problem is that for Redfish testing (and IPMI) you need a
physical machine do test this on.
Spaceman1984 commented on pull request #4207:
URL: https://github.com/apache/cloudstack/pull/4207#issuecomment-665508759
@rhtyd, I checked the commas vs dots in the conversion, looks like it
changes based on the locale and JDK version.
DaanHoogland commented on issue #4211:
URL: https://github.com/apache/cloudstack/issues/4211#issuecomment-665518391
This doesn't sound familiar @weizhouapache . To recap,
- a hosts is HA enabled? (genuine question , here)
- a vm with a ha enabled offering is created on it.
- the
andrijapanicsb edited a comment on issue #4227:
URL: https://github.com/apache/cloudstack/issues/4227#issuecomment-665799984
This is an automated message from the Apache Git Service.
To respond to the message, please log on
DaanHoogland commented on pull request #4111:
URL: https://github.com/apache/cloudstack/pull/4111#issuecomment-665535646
Like your thinking, mr operator ;).
one reservation @GabrielBrascher; is the word 'dead' the best to use here?
the life/dead metaphore for hosts might incite
DaanHoogland commented on pull request #4200:
URL: https://github.com/apache/cloudstack/pull/4200#issuecomment-665577184
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
ravening commented on pull request #4215:
URL: https://github.com/apache/cloudstack/pull/4215#issuecomment-665547798
> This quite a functional change @ravening .
> Did you test settings for domains linked to ldap?
> I read in your description that you are (kind of) mixing account
DaanHoogland commented on issue #4227:
URL: https://github.com/apache/cloudstack/issues/4227#issuecomment-665793646
@gaaray2k I think your question makes sense, even when not understanding it
completely; you want the resource count to be on domain level, or shared over
the domain admin
Spaceman1984 edited a comment on pull request #4207:
URL: https://github.com/apache/cloudstack/pull/4207#issuecomment-665508759
@rhtyd, I checked the commas vs dots in the conversion, looks like it
changes based on the locale.
gaaray2k commented on issue #4229:
URL: https://github.com/apache/cloudstack/issues/4229#issuecomment-665561519
Which requirements have to be met?. If I create an offering and choose the
options that I want, shouldn't it just work?
GabrielBrascher commented on pull request #4111:
URL: https://github.com/apache/cloudstack/pull/4111#issuecomment-665738632
@DaanHoogland your observation makes sense, I will look for a better word ;)
Thanks for the input!
ravening commented on pull request #4047:
URL: https://github.com/apache/cloudstack/pull/4047#issuecomment-665551768
> as per your description @ravening, "If the template from which VR is
created got deleted, the state
> is set to inactive and removed to null.", it seems the 'real' bug
rhtyd commented on a change in pull request #4207:
URL: https://github.com/apache/cloudstack/pull/4207#discussion_r462184183
##
File path: utils/src/test/java/com/cloud/utils/HumanReadableJsonTest.java
##
@@ -50,4 +51,14 @@ public void parseMatchJsonTest() {
DaanHoogland commented on pull request #4175:
URL: https://github.com/apache/cloudstack/pull/4175#issuecomment-665547812
> Do you mean
[@TestTemplate](https://junit.org/junit5/docs/5.0.2/api/org/junit/jupiter/api/TestTemplate.html)
from JUnit5?
no, nice link though,
rhtyd commented on pull request #4220:
URL: https://github.com/apache/cloudstack/pull/4220#issuecomment-665569219
@blueorangutan package
This is an automated message from the Apache Git Service.
To respond to the message,
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 41aa289 components: Adding
This is an automated email from the ASF dual-hosted git repository.
dahn pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack.git.
from 7e3b61b Merge branch '4.14'
add c856614 Document how to pass CIDRs lists API calls (#4035)
No new
DaanHoogland commented on pull request #4078:
URL: https://github.com/apache/cloudstack/pull/4078#issuecomment-665057020
@sureshanaparti do you approve the changes?
This is an automated message from the Apache Git Service.
DaanHoogland commented on pull request #4020:
URL: https://github.com/apache/cloudstack/pull/4020#issuecomment-665082680
@blueorangutan package
This is an automated message from the Apache Git Service.
To respond to the
rvalle closed issue #4223:
URL: https://github.com/apache/cloudstack/issues/4223
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
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.git
commit 36ef85012f69af59657c87ad88184769549d8792
Merge: c856614 eec5602
Author: Rohit Yadav
AuthorDate: Wed Jul 29 14:08:02
rafaelweingartner commented on a change in pull request #4194:
URL: https://github.com/apache/cloudstack/pull/4194#discussion_r461945911
##
File path:
server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java
##
@@ -3183,7 +3184,7 @@ public DiskOffering
rhtyd commented on pull request #4131:
URL: https://github.com/apache/cloudstack/pull/4131#issuecomment-665425178
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
blueorangutan commented on pull request #4165:
URL: https://github.com/apache/cloudstack/pull/4165#issuecomment-664785902
Trillian test result (tid-2205)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 66215 seconds
Marvin logs:
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.git.
from c856614 Document how to pass CIDRs lists API calls (#4035)
add e225db4 ca: Removed check on client mode
DaanHoogland commented on a change in pull request #4078:
URL: https://github.com/apache/cloudstack/pull/4078#discussion_r461600605
##
File path:
services/secondary-storage/server/src/main/java/org/apache/cloudstack/storage/template/UploadManagerImpl.java
##
@@ -307,12
rhtyd commented on pull request #4150:
URL: https://github.com/apache/cloudstack/pull/4150#issuecomment-665425111
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
Spaceman1984 opened a new pull request #4226:
URL: https://github.com/apache/cloudstack/pull/4226
## Description
The SSL engine defaults to "Server mode" when doing handshaking. This PR
removes the check on the mode which was causing tests to fail.
This behavior was
ravening opened a new pull request #4228:
URL: https://github.com/apache/cloudstack/pull/4228
## Description
Fixes #4218
If a host is removed from cloudstack, it will be
added back if we restart the agent service on
the host. It should not be added back if manualy
blueorangutan commented on pull request #4144:
URL: https://github.com/apache/cloudstack/pull/4144#issuecomment-665077347
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
DaanHoogland commented on pull request #4144:
URL: https://github.com/apache/cloudstack/pull/4144#issuecomment-665076769
@blueorangutan package
This is an automated message from the Apache Git Service.
To respond to the
andrijapanicsb opened a new issue #4224:
URL: https://github.com/apache/cloudstack/issues/4224
Pasting below the runinfo.txt related parts - these are making it hard to
see if the tests are not robust enough, or if there is some kind of failure on
the ACS side.
Could someone please
rhtyd commented on pull request #4207:
URL: https://github.com/apache/cloudstack/pull/4207#issuecomment-665472945
@Spaceman1984 failures related to vm lifecycle seen here as well, I suspect
it may be related to the fix you added in #4226
gaaray2k opened a new issue #4227:
URL: https://github.com/apache/cloudstack/issues/4227
I am a little confused as to how user accounts work on CS. right now I have
an account called A which is domain admin in domain lab.com . if I add another
user in account A, it gets the same "domain
blueorangutan commented on pull request #4068:
URL: https://github.com/apache/cloudstack/pull/4068#issuecomment-664677157
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
DaanHoogland commented on a change in pull request #4144:
URL: https://github.com/apache/cloudstack/pull/4144#discussion_r461636110
##
File path: usage/src/main/java/com/cloud/usage/UsageManagerImpl.java
##
@@ -275,7 +277,14 @@ public boolean configure(String name, Map
blueorangutan commented on pull request #4213:
URL: https://github.com/apache/cloudstack/pull/4213#issuecomment-664762505
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
rhtyd commented on a change in pull request #4207:
URL: https://github.com/apache/cloudstack/pull/4207#discussion_r462080672
##
File path: utils/src/test/java/com/cloud/utils/HumanReadableJsonTest.java
##
@@ -0,0 +1,53 @@
+//
+// Licensed to the Apache Software Foundation
rhtyd commented on pull request #4225:
URL: https://github.com/apache/cloudstack/pull/4225#issuecomment-665015182
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
blueorangutan removed a comment on pull request #4068:
URL: https://github.com/apache/cloudstack/pull/4068#issuecomment-664193373
This is an automated message from the Apache Git Service.
To respond to the message, please log
blueorangutan commented on pull request #4226:
URL: https://github.com/apache/cloudstack/pull/4226#issuecomment-665052216
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
ravening commented on issue #4209:
URL: https://github.com/apache/cloudstack/issues/4209#issuecomment-665208650
we are using haproxy as a loadbalancer and when transparent mode is enabled,
it uses XFF address instead of loadbalancer ip. This will be fixed in
blueorangutan commented on pull request #4128:
URL: https://github.com/apache/cloudstack/pull/4128#issuecomment-664838452
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
Pearl1594 commented on pull request #4213:
URL: https://github.com/apache/cloudstack/pull/4213#issuecomment-664762152
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
blueorangutan commented on pull request #4214:
URL: https://github.com/apache/cloudstack/pull/4214#issuecomment-664627461
Trillian test result (tid-2196)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 50718 seconds
Marvin logs:
blueorangutan commented on pull request #3952:
URL: https://github.com/apache/cloudstack/pull/3952#issuecomment-664857096
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
blueorangutan commented on pull request #4150:
URL: https://github.com/apache/cloudstack/pull/4150#issuecomment-665425548
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
DaanHoogland commented on a change in pull request #3902:
URL: https://github.com/apache/cloudstack/pull/3902#discussion_r461593221
##
File path:
server/src/main/java/com/cloud/network/guru/ExternalGuestNetworkGuru.java
##
@@ -261,6 +265,17 @@ public NicProfile
rhtyd merged pull request #4226:
URL: https://github.com/apache/cloudstack/pull/4226
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
blueorangutan commented on pull request #4207:
URL: https://github.com/apache/cloudstack/pull/4207#issuecomment-664602670
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
rhtyd commented on pull request #4222:
URL: https://github.com/apache/cloudstack/pull/4222#issuecomment-664748427
@blueorangutan test
This is an automated message from the Apache Git Service.
To respond to the message,
DaanHoogland commented on pull request #4149:
URL: https://github.com/apache/cloudstack/pull/4149#issuecomment-664840747
@GabrielBrascher then you have to change your version to the one you did
install, or not set it at all. I have to switch between 2 and 3 as well. That
is exactly the
Spaceman1984 commented on pull request #4222:
URL: https://github.com/apache/cloudstack/pull/4222#issuecomment-665016249
> I had some issues on master, where the test checks for true
`Assert.assertTrue(e.getUseClientMode()); `, not `assertFalse`. will this be
forwarded to master as well?
Spaceman1984 commented on pull request #4207:
URL: https://github.com/apache/cloudstack/pull/4207#issuecomment-664977855
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
DaanHoogland commented on a change in pull request #4032:
URL: https://github.com/apache/cloudstack/pull/4032#discussion_r461380678
##
File path:
plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtDeleteVMSnapshotCommandWrapper.java
##
@@
rvalle commented on issue #4223:
URL: https://github.com/apache/cloudstack/issues/4223#issuecomment-664593182
ok @GabrielBrascher let me check...
I have configured
- guest.network.device
- public.network.device
- private.network.device
I have not configured
RodrigoDLopez commented on a change in pull request #4194:
URL: https://github.com/apache/cloudstack/pull/4194#discussion_r461859143
##
File path:
server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java
##
@@ -3183,7 +3184,7 @@ public DiskOffering
blueorangutan commented on pull request #4137:
URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-665084642
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
rhtyd opened a new pull request #4225:
URL: https://github.com/apache/cloudstack/pull/4225
- This fixes issues of virtual size to be twice in case the disk is a
linked-clone root disk. The virtual size of root disk (first in chain) must be
used.
- Fixes to report the physical size for
GabrielBrascher commented on issue #4223:
URL: https://github.com/apache/cloudstack/issues/4223#issuecomment-664597022
I thought that this was the case. All looking good then :-)
This is an automated message from the Apache
GabrielBrascher commented on pull request #3730:
URL: https://github.com/apache/cloudstack/pull/3730#issuecomment-664590843
Sorry for the late reply, @rhtyd. I just fixed conflicts and ported to the
current master.
I am now performing tests and if all goes right I shall have this PR
RodrigoDLopez commented on a change in pull request #4213:
URL: https://github.com/apache/cloudstack/pull/4213#discussion_r461837337
##
File path: server/src/main/java/com/cloud/vm/snapshot/VMSnapshotManagerImpl.java
##
@@ -229,11 +238,32 @@ public boolean stop() {
Spaceman1984 commented on pull request #4226:
URL: https://github.com/apache/cloudstack/pull/4226#issuecomment-665051451
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
blueorangutan commented on pull request #4131:
URL: https://github.com/apache/cloudstack/pull/4131#issuecomment-665425488
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
rhtyd commented on pull request #4226:
URL: https://github.com/apache/cloudstack/pull/4226#issuecomment-665412298
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
blueorangutan commented on pull request #4225:
URL: https://github.com/apache/cloudstack/pull/4225#issuecomment-665015464
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
blueorangutan commented on pull request #4222:
URL: https://github.com/apache/cloudstack/pull/4222#issuecomment-664728845
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
DaanHoogland commented on pull request #4137:
URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-665083905
@blueorangutan test
This is an automated message from the Apache Git Service.
To respond to the
GabrielBrascher commented on pull request #4175:
URL: https://github.com/apache/cloudstack/pull/4175#issuecomment-665250531
@DaanHoogland you are right, docs should be updated indeed. I will work on
that as well.
> and can we create a template for nested testing
Do you mean
borisstoyanov commented on pull request #4128:
URL: https://github.com/apache/cloudstack/pull/4128#issuecomment-664910401
@blueorangutan test
This is an automated message from the Apache Git Service.
To respond to the
RodrigoDLopez commented on pull request #4226:
URL: https://github.com/apache/cloudstack/pull/4226#issuecomment-665270280
Code LGTM.
This is an automated message from the Apache Git Service.
To respond to the message, please
weizhouapache commented on pull request #4068:
URL: https://github.com/apache/cloudstack/pull/4068#issuecomment-664626019
> @weizhouapache - related to the Ubuntu 20 changes (your original PR was
closed, and Rohit cherry-picked the changes to this PR) - please feel free to
test Ubuntu 20
andrijapanicsb commented on issue #4224:
URL: https://github.com/apache/cloudstack/issues/4224#issuecomment-664882291
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
gaaray2k opened a new issue #4229:
URL: https://github.com/apache/cloudstack/issues/4229
when I create a new network offering I give it a name and fill out all the
require fields. I then enable the new offering but the new network offering
doesnt show up when creating a new network. am I
rhtyd merged pull request #4214:
URL: https://github.com/apache/cloudstack/pull/4214
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
1 - 100 of 114 matches
Mail list logo