You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@jclouds.apache.org by Adrian Cole <no...@github.com> on 2014/11/17 07:04:41 UTC

[jclouds-site] JCLOUDS-780 Remove vcloud. (#142)

You can merge this Pull Request by running:

  git pull https://github.com/adriancole/jclouds-site adrian.rm-vcloud

Or you can view, comment on it, or merge it online at:

  https://github.com/jclouds/jclouds-site/pull/142

-- Commit Summary --

  * JCLOUDS-780 Remove vcloud.

-- File Changes --

    M guides/index.md (2)
    D guides/vcloud-director.md (90)
    D guides/vcloud.md (303)
    M start/compute.md (9)
    M start/what-is-jclouds.md (2)

-- Patch Links --

https://github.com/jclouds/jclouds-site/pull/142.patch
https://github.com/jclouds/jclouds-site/pull/142.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/142

Re: [jclouds-site] JCLOUDS-780 Remove vcloud. (#142)

Posted by Ignasi Barrera <no...@github.com>.
Change lgtm.
Would it be worth to translate the different threads in the mailing list into a blog post announcing the retirement of such apis?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/142#issuecomment-63309526

Re: [jclouds-site] JCLOUDS-780 Remove vcloud. (#142)

Posted by jclouds-commentator <no...@github.com>.
  Go to http://05f42ce8be44ed544067-4725249979ef59bb6678444d530b2239.r45.cf5.rackcdn.com/ to review your changes.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/142#issuecomment-63332456

Re: [jclouds-site] JCLOUDS-780 Remove vcloud. (#142)

Posted by jclouds-commentator <no...@github.com>.
  Go to http://f76bfc986772b9fd9507-1a7d0f12851cc89c08136853c7e19ebb.r79.cf5.rackcdn.com/ to review your changes.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/142#issuecomment-63263509

Re: [jclouds-site] JCLOUDS-780 Remove vcloud. (#142)

Posted by Andrew Phillips <no...@github.com>.
> @@ -69,14 +69,14 @@ to specify your requirements with statements such as `minCores` and `imageId`.
>  An Image is different from an Operating System. It is really a bunch of state that includes an operating system,
>  software and configuration. It is often versioned independently of the operating system.
>  Before, we lumped together operating system with image, as this is what amazon and some others did.
> -However, this assumes nodes are created from images. This is not the case in OVF-based systems like vCloud,
> +However, this assumes nodes are created from images.images This is not the case in OVF-based systems like vCloud,

Typo?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/142/files#r20445754

Re: [jclouds-site] JCLOUDS-780 Remove vcloud. (#142)

Posted by Adrian Cole <no...@github.com>.
That was the intention of my summary on the jira/mailing list. However,
we've retired a lot of providers, lately, so I'm not sure if we should
single-out vcloud in a blog.

We still have more to go. For example, I doubt openstack-nova-ec2 works,
and there are probably others we should drop.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/142#issuecomment-63327623

Re: [jclouds-site] JCLOUDS-780 Remove vcloud. (#142)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-site-pull-requests #433](https://jclouds.ci.cloudbees.com/job/jclouds-site-pull-requests/433/) SUCCESS
This pull request looks good

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/142#issuecomment-63332255

Re: [jclouds-site] JCLOUDS-780 Remove vcloud. (#142)

Posted by Adrian Cole <no...@github.com>.
good catch. fixed

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/142#issuecomment-63331909

Re: [jclouds-site] JCLOUDS-780 Remove vcloud. (#142)

Posted by Adrian Cole <no...@github.com>.
> @@ -69,14 +69,14 @@ to specify your requirements with statements such as `minCores` and `imageId`.
>  An Image is different from an Operating System. It is really a bunch of state that includes an operating system,
>  software and configuration. It is often versioned independently of the operating system.
>  Before, we lumped together operating system with image, as this is what amazon and some others did.
> -However, this assumes nodes are created from images. This is not the case in OVF-based systems like vCloud,
> +However, this assumes nodes are created from images.images This is not the case in OVF-based systems like vCloud,

thx will fix

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/142/files#r20445880

Re: [jclouds-site] JCLOUDS-780 Remove vcloud. (#142)

Posted by Andrew Phillips <no...@github.com>.
Looks like there still a vCloud reference on the providers page:

![image](https://cloud.githubusercontent.com/assets/223702/5073011/da6177a6-6e4c-11e4-8964-ce6dfea9afa1.png)

Otherwise, also looks good to go for me!

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/142#issuecomment-63331279

Re: [jclouds-site] JCLOUDS-780 Remove vcloud. (#142)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-site-pull-requests #432](https://jclouds.ci.cloudbees.com/job/jclouds-site-pull-requests/432/) SUCCESS
This pull request looks good

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/142#issuecomment-63263438