Re: [jclouds/jclouds-examples] Added Google Cloud Storage parameterization to blobstore_basics (#97)

2019-01-29 Thread Andrew Gaul
Sorry we moved to the apache/jclouds-examples repository so that this merge 
actually did not take place.  I manually committed it to the correct place.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/97#issuecomment-458694063

Re: [jclouds/jclouds-examples] Added Google Cloud Storage parameterization to blobstore_basics (#97)

2019-01-28 Thread Andrew Gaul
Apache has an older mailing list subscription model; refer to these 
instructions:

https://www.apache.org/foundation/mailinglists.html

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/97#issuecomment-458356573

Re: [jclouds/jclouds-examples] Added Google Cloud Storage parameterization to blobstore_basics (#97)

2019-01-28 Thread bcli4d
Hi Andrew,

I'm an apache newbie. After logging into
https://lists.apache.org/list.html?u...@jclouds.apache.org with this same
Google account, I tried starting a new thread. It bounced with the message:


mailer-dae...@apache.org
2:30 PM (52 minutes ago)
to me

Hi. This is the qmail-send program at apache.org.
I'm afraid I wasn't able to deliver your message to the following addresses.
This is a permanent error; I've given up. Sorry it didn't work out.

:
Must be sent from an @apache.org address or a subscriber address or an
address in LDAP.

Can you tell me what I am doing wrong?

Thanks,
Bill

On Mon, Jan 28, 2019 at 11:15 AM Andrew Gaul 
wrote:

> Thank you for your contribution @bcli4d !
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> ,
> or mute the thread
> 
> .
>


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/97#issuecomment-458345414

Re: [jclouds/jclouds-examples] Added Google Cloud Storage parameterization to blobstore_basics (#97)

2019-01-28 Thread Andrew Gaul
Thank you for your contribution @bcli4d!

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/97#issuecomment-458263237

Re: [jclouds/jclouds-examples] Added Google Cloud Storage parameterization to blobstore_basics (#97)

2019-01-28 Thread Andrew Gaul
Merged #97 into master.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/97#event-2101700031

Re: [jclouds/jclouds-examples] Added Google Cloud Storage parameterization to blobstore_basics (#97)

2019-01-28 Thread Andrew Gaul
gaul approved this pull request.





-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/97#pullrequestreview-197210978

[jclouds/jclouds-examples] Added Google Cloud Storage parameterization to blobstore_basics (#97)

2019-01-28 Thread bcli4d
Modified blobstore-basics to expect a json key file as the credential when 
provider is google-cloud-storage, and, in that case, extract the private key 
from the json file. This is consistent with the compute-basics example
Added documentation on GCS parameterization to the README.md
You can view, comment on, or merge this pull request online at:

  https://github.com/jclouds/jclouds-examples/pull/97

-- Commit Summary --

  * Added Google Cloud Storage parameterization
  * When provider is Google Cloud Storage, credential is a json key file, which 
must be read into a string
  * Improve consistency with other providers
  * Get the name right

-- File Changes --

M blobstore-basics/README.md (9)
M 
blobstore-basics/src/main/java/org/jclouds/examples/blobstore/basics/MainApp.java
 (40)

-- Patch Links --

https://github.com/jclouds/jclouds-examples/pull/97.patch
https://github.com/jclouds/jclouds-examples/pull/97.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/97