Closed eifrach closed 3 years ago
Hi @eifrach. Thanks for your PR.
I'm waiting for a kubevirt member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test
on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test
label.
I understand the commands that are listed here.
waiting for a merge on different repo /hold
/assign @mmazur
@eifrach This is an XL pull req that at the heart is fatally broken. Jekyll runtime has changed and the app is no longer accessible.
I really think we're trying to take on too much here. Near as I can tell we have 3 items going on ... Add linting, consolidate build, update dictionary.
I highly suggest these items be broken out into smaller pull reqs.
/retest all
@mazzystr: The /retest
command does not accept any targets.
The following commands are available to trigger jobs:
/test kubevirt-io-presubmit-link-checker
Use /test all
to run all jobs.
/test all
@eifrach: The following test failed, say /retest
to rerun all failed tests:
Test name | Commit | Details | Rerun command |
---|---|---|---|
kubevirt-io-presubmit-link-checker | e62c485dfb668992f7bfbe7aa6c84ded0968e497 | link | /test kubevirt-io-presubmit-link-checker |
@.** can you add that Dockerfile to the pull req to project-infra? I didn't have a copy so wasn't able to do it on Fri
On Sun, Apr 18, 2021, 2:35 AM Eran Ifrach @.***> wrote:
@.**** commented on this pull request.
In Makefile https://github.com/kubevirt/kubevirt.github.io/pull/725#discussion_r615371218 :
@echo
- @echo "${GREEN}Makefile: Check links and selectors to userguide on website content${RESET}"
+## Stop site +stop: | envvar
- @echo "${GREEN}Makefile: Stop site${RESET}"
- ${CONTAINER_ENGINE} rm -f website 2> /dev/null; echo
- @echo -n
+## Build image localhost/kubevirt.io +build_image: | envvar
- ${DEBUG}$(eval export TAG='localhost/kubevirt.io')
- @echo "${GREEN}Makefile: Cloneing Repo ${RESET}"
- ${DEBUG}git clone https://github.com/kubevirt/project-infra.git /tmp/project-infra; \
@mazzystr https://github.com/mazzystr This Dockerfile needs all the files in the directory. what you are suggesting we do take a look the the repo folder
https://github.com/eifrach/project-infra/tree/kubevirt.io/images/kubevirt-kubevirt.github.io
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/kubevirt/kubevirt.github.io/pull/725#discussion_r615371218, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAS5KZ4NS4JRKAOOZQAWA4DTJKRW3ANCNFSM42ZNILQQ .
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: To complete the pull request process, please ask for approval from mazzystr after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.
@eifrach: Cannot trigger testing until a trusted user reviews the PR and leaves an /ok-to-test
message.
Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:
Closing PR reopening in a new branch
What this PR does / why we need it: