Add the landscape-client charm

Bug #1101010 reported by David Britton
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juju Charms Collection
Fix Released
Undecided
Unassigned

Bug Description

The landscape-client charm is a base level of functionality. It's ready to be included in the charm store.

Related branches

Revision history for this message
David Britton (dpb) wrote :

bzr+ssh://bazaar.launchpad.net/~davidpbritton/charms/precise/landscape-client/trunk/

Revision history for this message
Juan L. Negron (negronjl) wrote :

Reviewing this now.

-Juan

Revision history for this message
Juan L. Negron (negronjl) wrote :

Hi David:

Thanks for the charm.

The charm works as expected but, I feel that the README file is a bit too light on details. Would you elaborate a bit on the necessary values in config.yaml?

Deploying this could be a little confusing with all of those options and no documentation to go with it.

I would be happy to review this again once the issues above have been addressed.

Thanks,

Juan

Changed in charms:
status: New → Incomplete
Revision history for this message
David Britton (dpb) wrote : Re: [Bug 1101010] Re: Add the landscape-client charm

On 01/17/2013 06:35 PM, Juan L. Negron wrote:
> Hi David:
>
> Thanks for the charm.
>
> The charm works as expected but, I feel that the README file is a bit
> too light on details. Would you elaborate a bit on the necessary values
> in config.yaml?
>
> Deploying this could be a little confusing with all of those options and
> no documentation to go with it.
>
> I would be happy to review this again once the issues above have been
> addressed.

Yes, thanks Juan

I'll do this and re-apply. I supposed that I just need to set the bug
back to "New"?

--
David Britton <email address hidden>

Revision history for this message
David Britton (dpb) wrote :

Updated the README with an appropriate amount of detail. The other options in there are more rare, so I don't want to clutter up the README with things that not many people would use.

Let me know what you think.

Changed in charms:
status: Incomplete → New
Revision history for this message
Juan L. Negron (negronjl) wrote :

David:

Thanks for the quick reply ...

Approved.

I'll promulgate in just a minute.

-Juan

Changed in charms:
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.