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

First attempt at a data driven adopters consumed from kubevirt/kubevirt #797

Closed mazzystr closed 2 years ago

mazzystr commented 2 years ago

Signed-off-by: Chris Callegari mazzystr@gmail.com

/kind/enhancement

What this PR does / why we need it: ADOPTERS list is getting out of hand We have kubevirt/kubevirt:ADOPTERS.md, kubevirt/kubevirt:~/_data/adopters.yml, Karena's doc with a list of adopters. We also have adopters scattered throughout the project via mailing list subscription, slack membership, pull req creation, YouTube video publishing, Blog publishing.

Let's start to get a handle on this and consolidate all adopters to kubevirt/kubevirt:ADOPTERS.md... See kubevirt/kubevirt/pull/6385. Then we can use adopters.py to populate a yaml file that the jekyll engine can use to build the index.html file for the website.

Note: https://github.com/kubevirt/kubevirt/pull/6385 should be merged before this merges

jobbler commented 2 years ago

@mazzystr thanks for making the changes. /lgtm

cwilkers commented 2 years ago

/retest

kubevirt-bot commented 2 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cwilkers

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)~~ [cwilkers] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
mazzystr commented 2 years ago

The script works 100% as is. We can manually update adopters.yml manually since we have a lot of eyes on kubevirt/kubevirt ADOPTERS right now.

We will address automation in a future pull req.

I'm removing the hold tag so that we can rebase/merge when the link gets fixed via the istio pull.

cwilkers commented 2 years ago

/lgtm