that I can
> handle
> > reverting.
> >
> > If it's not one of mine, I can run the default logic that's provided from
> > the class my strategy extends.
> >
> > Sounds about right?
> >
> >
> > On Thu, Aug 28, 2014 at 1:42 PM, Chris Su
http://solidfire.com/solution/overview/?video=play>*(tm)*
> >>
> >
> >
> >
> > --
> > *Mike Tutkowski*
> > *Senior CloudStack Developer, SolidFire Inc.*
> > e: mike.tutkow...@solidfire.com
> > o: 303.746.7302
> > Advancing the way the world uses the cloud
> > <http://solidfire.com/solution/overview/?video=play>*(tm)*
> >
>
>
>
> --
> *Mike Tutkowski*
> *Senior CloudStack Developer, SolidFire Inc.*
> e: mike.tutkow...@solidfire.com
> o: 303.746.7302
> Advancing the way the world uses the cloud
> <http://solidfire.com/solution/overview/?video=play>*(tm)*
>
--
Chris Suich
919-334-8193
csui...@gmail.com
s. Does anyone have any suggestions?
--
Chris Suich
919-334-8193
csui...@gmail.com
10:04 AM, sebgoa wrote:
>
> On May 9, 2014, at 6:39 PM, "La Motta, David"
> wrote:
>
>> I wanted to give a shout-out to Chris Suich, who was a key player in the
>> team that worked on the NetApp plugin--aka Virtual Storage Console for
>> Apache CloudStack
I seemed to notice that the email list went silent for 24-48 hours
around last Thursday/Friday.
On Tue, May 13, 2014 at 7:57 PM, Alex Huang wrote:
> I've find some of my emails did not make it to the mailing list. Does anyone
> else have this problem?
>
> --Alex
--
Chris S
ifying
> applicationContext.xml.inand commands.properties.in. 4.3 appears to have
> introduces discoverable plugins. What needs to be done differently in order
> to take advantage of this?
> Thank you for your time,
> Matt
--
Chris Suich
919-334-8193
csui...@gmail.com
hether this was just a typo of
'!=' instead of '=='?
Diffs
-
plugins/hypervisors/simulator/src/com/cloud/agent/manager/MockVmManagerImpl.java
f487eb4
Diff: https://reviews.apache.org/r/20250/diff/
Testing
---
Thanks,
Chris Suich
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19460/#review37891
---
Ship it!
Ship It!
- Chris Suich
On March 20, 2014, 2:42 p.m
into VolumeServiceImpl.takeSnapshot()'s catch instead of
just ignoring the exception it would be handled for anyone who tries to take a
snapshot.
- Chris Suich
On March 20, 2014, noon, Alex Hitchins wrote:
>
> ---
> This is an automa
/diff/
Testing
---
Thanks,
Chris Suich
/
Testing
---
Thanks,
Chris Suich
: cloudstack-git
Description
---
Added load vs refresh context/indicator for dataProviders for CLOUDSTACK-5384
Diffs
-
ui/scripts/ui/widgets/detailView.js 3a3cd79
Diff: https://reviews.apache.org/r/16041/diff/
Testing
---
Thanks,
Chris Suich
remove the overlay when an
error occurs during the operation.
This is for JIRA-5368. If accepted, please apply to 4.3 branch as well.
Diffs
-
ui/scripts/ui/widgets/listView.js 53b37f1
Diff: https://reviews.apache.org/r/16001/diff/
Testing
---
Thanks,
Chris Suich
/src/com/cloud/storage/StorageManagerImpl.java e81e30e
Diff: https://reviews.apache.org/r/15521/diff/
Testing
---
Thanks,
Chris Suich
VirtualNetworkApplianceManagerImpl.java. Was that intentional?
- Chris Suich
On Nov. 25, 2013, 6:55 p.m., Nitin Mehta wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache
e81e30e
Diff: https://reviews.apache.org/r/15521/diff/
Testing
---
Thanks,
Chris Suich
mitted, the password is still visible in plain text.
This patch fixes the issue by respecting the 'isPassword' attribute on detail
view fields.
Diffs
-
ui/scripts/ui/widgets/detailView.js 65f71b3
Diff: https://reviews.apache.org/r/15362/diff/
Testing
---
Thanks,
Chris Suich
/
Testing (updated)
---
Thanks,
Chris Suich
advantage of.
Thanks,
Chris Suich
/
Testing (updated)
---
This has been tested locally with a custom plugin. However, the changes are not
used by the existing CloudStack codebase and should not impact any existing
workflows or use cases. They are simply for plugins to take advantage of.
Thanks,
Chris Suich
Testing
---
This has been tested locally with a custom plugin. However, the changes are not
used by the existing CloudStack codebase and should not impact any existing
workflows or use cases. They are simply for plugins to take advantage of.
Thanks,
Chris Suich
://reviews.apache.org/r/15313/diff/
Testing
---
Thanks,
Chris Suich
supposed to be changed to
store.getUuid() but was actually changed to volumeTo.getUuid().
Diffs
-
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareStorageManagerImpl.java
0e2423e
Diff: https://reviews.apache.org/r/15309/diff/
Testing
---
Thanks,
Chris
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14914/#review27529
---
Ship it!
Works great!
- Chris Suich
On Oct. 24, 2013, 10:06 p.m
d-commit-of-the-following.patch
Thanks,
Chris Suich
test this type of code. If anyone has any recommendations, please let me know.
File Attachments (updated)
https://reviews.apache.org/media/uploaded/files/2013/10/22/0001-Squashed-commits-from-review-revisions.patch
Thanks,
Chris Suich
how to properly unit
test this type of code. If anyone has any recommendations, please let me know.
File Attachments (updated)
https://reviews.apache.org/media/uploaded/files/2013/10/22/commits.patch
Thanks,
Chris Suich
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14522/#review27303
---
On Oct. 22, 2013, 2:48 p.m., Chris Suich wrote:
>
> --
trategy-sorting-selection.patch
https://reviews.apache.org/media/uploaded/files/2013/10/22/0003-Added-context-to-strategy-sorting-error-responses.patch
https://reviews.apache.org/media/uploaded/files/2013/10/22/0004-Updated-DataMotionServiceImpl-and-ApiResponseHelper-.patch
Thanks,
Chris Suich
Testing
---
I have tested all of this locally with a custom storage provider.
Unfortunately, I'm still in the middle of figuring out how to properly unit
test this type of code. If anyone has any recommendations, please let me know.
Thanks,
Chris Suich
em. Is there some way
around this?
- Chris
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14522/#review27298
-------
On Oct. 17, 2013, 6:46 p.m., Chris Suich wrote:
>
> -
> On Oct. 22, 2013, 1:19 p.m., Chris Suich wrote:
> > Darren, the design of all of this has already changed here:
> > https://reviews.apache.org/r/14522/. Take a look at the newer revisions
> > which haven't made it in yet (still waiting on a final review from Joh
imply chose
one by iterating over the list of strategies once and returning the best
strategy.
- Chris Suich
On Oct. 22, 2013, 3:34 a.m., Darren Shepherd wrote:
>
> ---
> This is an automatically generated e-mail. To reply,
: cloudstack-git
Description
---
Fixed a typo causing vm snapshot requests to always instantly return successful
in the CloudStack UI.
Diffs
-
ui/scripts/instances.js 41d1772
Diff: https://reviews.apache.org/r/14782/diff/
Testing
---
Thanks,
Chris Suich
/
Testing
---
Thanks,
Chris Suich
context information to error message to help operators and users
> > determine what couldn't be snapshotted.
>
> Chris Suich wrote:
> Fair enough - I just copied these existing error messages. What kind of
> context would you like to see in this error message? The src
Testing
---
I have tested all of this locally with a custom storage provider.
Unfortunately, I'm still in the middle of figuring out how to properly unit
test this type of code. If anyone has any recommendations, please let me know.
Thanks,
Chris Suich
e volume.
- Chris
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14522/#review27126
---
On Oct. 16, 2013, 2:50 p.m., Chris Suich wrote:
>
>
is locally with a custom storage provider.
Unfortunately, I'm still in the middle of figuring out how to properly unit
test this type of code. If anyone has any recommendations, please let me know.
Thanks,
Chris Suich
derlying device can take a snapshot without
> > accounting for whether or not the volume is in a state to take a consistent
> > snapshot. I am also concerned that we don't have a semantic for handling
> > errors that might result from the actual snapshot operation.
>
&g
d to throw an exception when a
> > snapshot can't be reverted. In addition to be a simpler approach, it
> > covers both the scenario when a device does not support reverting snapshots
> > and, as well as, the device or the snapshot not being a revertable state.
>
>
d to throw an exception when a
> > snapshot can't be reverted. In addition to be a simpler approach, it
> > covers both the scenario when a device does not support reverting snapshots
> > and, as well as, the device or the snapshot not being a revertable state.
>
>
d to throw an exception when a
> > snapshot can't be reverted. In addition to be a simpler approach, it
> > covers both the scenario when a device does not support reverting snapshots
> > and, as well as, the device or the snapshot not being a revertable state.
>
>
8>
> >
> > Very poor performance as it circumvents the primitive pool. Change to
> > Integer.valueOf(i1).compareTo(Integer.valueOf(i2)).
Will do.
> On Oct. 15, 2013, 2:41 p.m., John Burwell wrote:
> > engine/storage/snapshot/src/org/apache/c
recommendations, please let me know.
Thanks,
Chris Suich
---
Thanks,
Chris Suich
there shouldn't be any
issues.
However, I did do a clean build and played around with the UI while watching
vmops.log to make sure there were no errors being thrown over this.
Thanks,
Chris Suich
).
Thanks,
Chris Suich
re were no errors being thrown over this.
Thanks,
Chris Suich
:
https://reviews.apache.org/r/14522/#review26786
-------
On Oct. 7, 2013, 8:26 p.m., Chris Suich wrote:
>
> ---
> This is an automatically generated e-m
know.
Thanks,
Chris Suich
---
server/src/com/cloud/api/query/vo/StoragePoolJoinVO.java 69f2204
setup/db/db/schema-420to430.sql 653ff77
Diff: https://reviews.apache.org/r/14516/diff/
Testing
---
Thanks,
Chris Suich
Resource.java
3ef950b
Diff: https://reviews.apache.org/r/14477/diff/
Testing
---
Thanks,
Chris Suich
b
Diff: https://reviews.apache.org/r/14285/diff/
Testing
---
Testing has been done in conjunction with a UI plugin I am developing.
Additional testing will be performed as this is integrated into existing parts
of the UI.
Thanks,
Chris Suich
itional testing will be performed as this is integrated into existing parts
of the UI.
Thanks,
Chris Suich
---
UserVmResponse was missing some getters, so I generated them with Eclipse.
Diffs
-
api/src/org/apache/cloudstack/api/response/UserVmResponse.java e49e16b
Diff: https://reviews.apache.org/r/13833/diff/
Testing
---
Thanks,
Chris Suich
3131/diff/
Testing
---
File Attachments
https://reviews.apache.org/media/uploaded/files/2013/07/31/field-identify-value.patch
Thanks,
Chris Suich
ermined at run time.
Diffs
-
ui/scripts/sharedFunctions.js 8ddaeaf
ui/scripts/ui/dialog.js 1ff5401
Diff: https://reviews.apache.org/r/13132/diff/
Testing
---
Thanks,
Chris Suich
ome way to fix this.
Diffs
-
Diff: https://reviews.apache.org/r/13134/diff/
Testing
---
File Attachments
https://reviews.apache.org/media/uploaded/files/2013/07/31/has_units-field.patch
Thanks,
Chris Suich
Diffs
-
ui/scripts/ui-custom/zoneChart.js 1a5fb59
Diff: https://reviews.apache.org/r/13128/diff/
Testing
---
Thanks,
Chris Suich
appropriate plugin
dictionary is loaded into the global js dictionary (with the global dictionary
taking precedence in the event of a conflict).
Diffs
-
ui/scripts/plugins.js 96ed229
Diff: https://reviews.apache.org/r/13127/diff/
Testing
---
Thanks,
Chris Suich
61 matches
Mail list logo
