search

kubevirt / kubevirt.github.io

KubeVirt website repo, documentation at https://kubevirt.io/user-guide/
https://kubevirt.io
MIT License
30 stars 112 forks source link

Update index page #762

Closed mazzystr closed 3 years ago

mazzystr commented 3 years ago

Add adopters (end-users, vendors, integrations) list to index page Add adopters data file Fix footer toolbox pop ups

TESTING.... Scroll down main page

This PR reqs https://github.com/kubevirt/project-infra/pull/1406

Fixes #741

@matthewcarleton Could you please review and comment a /lgtm if it looks good to you? Thanks!

mazzystr commented 3 years ago

@jberkus Does the new front page make sense? https://deploy-preview-762--kubevirtio.netlify.app/

mazzystr commented 3 years ago

/hold

Waiting on approval from

salanki commented 3 years ago

Approved from CoreWeave /lgtm

kubevirt-bot commented 3 years ago

@salanki: changing LGTM is restricted to collaborators

In response to [this](https://github.com/kubevirt/kubevirt.github.io/pull/762#issuecomment-875145317): >Approved from CoreWeave >/lgtm Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
mfranczy commented 3 years ago

Approved from Kubermatic /lgtm

AlexsJones commented 3 years ago

Approved from Civo /lgtm

kubevirt-bot commented 3 years ago

@AlexsJones: changing LGTM is restricted to collaborators

In response to [this](https://github.com/kubevirt/kubevirt.github.io/pull/762#issuecomment-876207712): >Approved from Civo >/lgtm Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
poojaghumre commented 3 years ago

Approved from Platform9 /lgtm

kubevirt-bot commented 3 years ago

@poojaghumre: changing LGTM is restricted to collaborators

In response to [this](https://github.com/kubevirt/kubevirt.github.io/pull/762#issuecomment-876918025): >Approved from Platform9 >/lgtm Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
dirkmueller commented 3 years ago

Approved from SUSE side /lgtm

kubevirt-bot commented 3 years ago

@dirkmueller: changing LGTM is restricted to collaborators

In response to [this](https://github.com/kubevirt/kubevirt.github.io/pull/762#issuecomment-877093841): >Approved from SUSE side >/lgtm Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
vielmetti commented 3 years ago

@mazzystr - received an email from you regarding approving a change to this page, but I'm not seeing any sign of an Equinix Metal logo or reference in the diffs provided.

For reference, our brand guidelines are here https://brand.equinix.com/

mazzystr commented 3 years ago

@mazzystr - received an email from you regarding approving a change to this page, but I'm not seeing any sign of an Equinix Metal logo or reference in the diffs provided.

For reference, our brand guidelines are here https://brand.equinix.com/

@vielmetti Equinix is added. Validation url is here... https://deploy-preview-762--kubevirtio.netlify.app/

vielmetti commented 3 years ago

@mazzystr I've asked our marketing people to take a look. The only thing I note is that the logo looks fuzzy - if there's a specific dimension that you need, I think we can create it for you from the original vectors so that it's sharp.

vielmetti commented 3 years ago

@mazzystr - based on feedback from our marketing, two requests.

One, please use the vertically aligned "Equinix Metal" logo at https://github.com/packethost/metal-logo/blob/main/Vertical/Full%20Color/Equinix%20Metal%20Vertical%20RGB.png . That directory has a lot of formats, I'm guessing that the PNG is the best.

Second, please use the link "https://metal.equinix.com" instead of "https://equinix.com" as the click through.

mazzystr commented 3 years ago

@vielmetti Logo and link updated. and thank you for that nicely sized logo. I was starting to sweat trying to figure out how to resize with css. ;-) Preview link is here ... https://deploy-preview-762--kubevirtio.netlify.app/

vielmetti commented 3 years ago

Approved from Equinix Metal, thanks! /lgtm

kubevirt-bot commented 3 years ago

@vielmetti: changing LGTM is restricted to collaborators

In response to [this](https://github.com/kubevirt/kubevirt.github.io/pull/762#issuecomment-877302990): >Approved from Equinix Metal, thanks! >/lgtm Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
cre8minus1 commented 3 years ago

Approved from Platform9. Thanks team.

fabiand commented 3 years ago

@rthallisey who can help to approve for nvidia?

mazzystr commented 3 years ago

@rthallisey please review and make a comment of approval ... https://deploy-preview-762--kubevirtio.netlify.app

rthallisey commented 3 years ago

/lgtm from NVIDIA

mazzystr commented 3 years ago

@jberkus I think we have a great list so far and we should ship this pull req asap. I'm can split out Rancher, H3C, Cloudflare, and StackPath into their own issues. I will hunt down their representatives and obtain approval individually.

kubevirt-bot commented 3 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: stu-gott

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/kubevirt/kubevirt.github.io/blob/master/OWNERS)~~ [stu-gott] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
stu-gott commented 3 years ago

/hold cancel No need to hold at this point