Re: [DISCUSS][PROPOSAL] git workflow

2014-07-29 Thread Sam Schmit
>>>>>>>>>>> >>>>>>>>>>>> The issue is with the transition. >>>>>>>>>>>>> >>>>>>>>>>>>> Our latest releasable product is the 4.4 branch (4.4.0 tag), so >>>>>>>>

Re: [DISCUSS][PROPOSAL] git workflow

2014-07-23 Thread Sam Schmit
Hey everyone, I've been a developer for a handful of years and have had my share of experience with different version control systems. I've used (for better or worse) Git, Perforce, Rational ClearCast, and SVN. Each of these solutions offers their own unique set of features, strengths and weakne

Re: Review Request 21498: CLOUDSTACK-6009: listHosts API fixed to return all "memory" stats in Bytes instead of a mix of Bytes and Kilobytes

2014-05-19 Thread Sam Schmit
the next official revision, and > work around the problem on the client side. > > > - Stephen Turner > > On May 15th, 2014, 7:54 p.m. UTC, Sam Schmit wrote: > Review request for cloudstack. > By Sam Schmit. > > *Updated May 15, 2014, 7:54 p.m.* > *Bugs: * > CL

Re: List Usage Records using Cloud Monkey

2014-05-16 Thread Sam Schmit
akash Rao Banuka < prakas...@simplifydc.com> wrote: > Hi Sam > > We are using 4.3 and 4.2 > > Thank you > Prakash > > > > -Original Message- > From: Sam Schmit [mailto:sam.sch...@appcore.com] > Sent: Tuesday, May 13, 2014 6:43 PM > To: dev@cloudstack.

Review Request 21498: CLOUDSTACK-6009: listHosts API fixed to return all "memory" stats in Bytes instead of a mix of Bytes and Kilobytes

2014-05-16 Thread Sam Schmit
. 2) Validated that memoryAvailable, memoryTotal, and memoryUsed are all in Bytes, and memoryAvailable + memoryUsed = memoryTotal Thanks, Sam Schmit

Re: uploadCustomCertificate

2014-05-16 Thread Sam Schmit
Michael, Check the CS-Users list, there's a discussing going on right now about intermediate certs and how to deal with this. Sam On Thu, May 8, 2014 at 5:18 PM, Michael Phillips wrote: > When using this command to upload the intermediate cert how do you specify > the newline character that i

Re: List Usage Records using Cloud Monkey

2014-05-13 Thread Sam Schmit
Prakash, What version of CS are you using? We have seen issues with NPEs being thrown in 4.3 onward (prior to a fix I have submitted) when there were removed VMs. Sam On Mon, May 12, 2014 at 12:47 PM, Prakash Rao Banuka < prakas...@simplifydc.com> wrote: > Hi: > > I installed and configured

Re: Review Request 20557: CLOUDSTACK-6472 listUsageRecords generates NPEs for expunging instances

2014-05-05 Thread Sam Schmit
Olivier, I've submitted the following review for 4.3: https://reviews.apache.org/r/21088/ It does, in fact, work to add the findByIdIncludingRemoved to the EntityManager. Thus, this should merge into 4.3 with no problems. Thanks, Sam On Mon, May 5, 2014 at 10:23 AM, Sam Schmit

Review Request 21088: CLOUDSTACK-6472 (4.3-specific) listUsageRecords: Pull information from removed items as well, fixing NPEs/Null UUIDs with usage API calls.

2014-05-05 Thread Sam Schmit
rned 4) Expunged VM 5) Ran usage 6) ran "listUsageRecords" API call, successful data returned (this would previously return an error) Thanks, Sam Schmit

Re: Review Request 20557: CLOUDSTACK-6472 listUsageRecords generates NPEs for expunging instances

2014-05-05 Thread Sam Schmit
I see that the implementing class com.cloud.dao.EntityManagerImpl still > has findByIdIncludingRemoved(). Do you know why it was removed from the > interface in f5e5b39c9bc8d? > > -- > Olivier > > > On Mon, May 5, 2014 at 3:45 PM, Sam Schmit wrote: > > > Sebastien, > > > > I took a l

Re: Review Request 20557: CLOUDSTACK-6472 listUsageRecords generates NPEs for expunging instances

2014-05-05 Thread Sam Schmit
stien Goasguen wrote: > > On May 2, 2014, at 2:17 PM, Sam Schmit wrote: > > > All, > > > > I have submitted a bug fix patch for this issue: > > > > https://reviews.apache.org/r/21015/diff/ > > > > If approved, this fix should also find its way i

Re: Review Request 20557: CLOUDSTACK-6472 listUsageRecords generates NPEs for expunging instances

2014-05-02 Thread Sam Schmit
All, I have submitted a bug fix patch for this issue: https://reviews.apache.org/r/21015/diff/ If approved, this fix should also find its way into the 4.3 and 4.4 builds. It is fairly critical for getting usage data out of cloudstack. Sam Schmit On Thu, May 1, 2014 at 5:35 PM, Nate Gordon

Review Request 21015: CLOUDSTACK-6472 listUsageRecords: Pull information from removed items as well, fixing NPEs/Null UUIDs with usage API calls.

2014-05-02 Thread Sam Schmit
istUsageRecords" API call, successful data returned (this would previously return an error) Thanks, Sam Schmit

Question on CLOUDSTACK-5837

2014-04-29 Thread Sam Schmit
All, I'm looking into addressing CLOUDSTACK-5837, but I wanted to make sure this wasn't intended functionality before proceeding. If no VM ID is not passed into the listHostsCmd, the response object is built via the QueryServiceImpl (and more

Re: HypervisorType - possible feature idea

2014-04-25 Thread Sam Schmit
> > > > There are 1097 references to HypervisorType according to eclipse. > > > > > > On Thu, Apr 24, 2014 at 10:57 PM, Sam Schmit > wrote: > > > >> Hello everyone, > >> > >> I've recently started working with Cloudstack, and

HypervisorType - possible feature idea

2014-04-24 Thread Sam Schmit
n work on this feature already? Anyone have thoughts on where the easy/difficult areas for such an undertaking would reside? Thanks for any thoughts on this, and I look forward to working with everyone in the near future. Sam Schmit