Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

[WIP] Test PR DO NOT MERGE #648

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[WIP] Test PR DO NOT MERGE #648

wants to merge 1 commit into from

Conversation

BenTheElder
Copy link
Member

/hold

@k8s-ci-robot k8s-ci-robot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Jun 24, 2019
@k8s-ci-robot k8s-ci-robot requested review from krzyzacy and munnerz June 24, 2019 22:30
@BenTheElder
Copy link
Member Author

/test pull-kind-conformance-parallel-ipv6

@k8s-ci-robot k8s-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jun 24, 2019
@BenTheElder BenTheElder removed the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 24, 2019
@BenTheElder
Copy link
Member Author

/test pull-kind-conformance-parallel-ipv6

1 similar comment
@aojea
Copy link
Contributor

aojea commented Jun 26, 2019

/test pull-kind-conformance-parallel-ipv6

@aojea
Copy link
Contributor

aojea commented Oct 1, 2019

/close
I think that I have a lot of these though 😅
just because this is not used for several months

@k8s-ci-robot
Copy link
Contributor

@aojea: Closed this PR.

In response to this:

/close
I think that I have a lot of these though 😅
just because this is not used for several months

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@BenTheElder
Copy link
Member Author

this PR is a no-op PR that I've already muted the notifications for for testing the CI w/o any changes.

there's no point in filing a new one, and it's obvious that we won't merge this.

@BenTheElder BenTheElder reopened this Oct 1, 2019
@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 1, 2019
@BenTheElder
Copy link
Member Author

/test all

@BenTheElder
Copy link
Member Author

/test pull-kind-build

3 similar comments
@BenTheElder
Copy link
Member Author

/test pull-kind-build

@BenTheElder
Copy link
Member Author

/test pull-kind-build

@BenTheElder
Copy link
Member Author

/test pull-kind-build

@BenTheElder
Copy link
Member Author

/test pull-kind-conformance-parallel-1-15

@BenTheElder
Copy link
Member Author

/test pull-kind-build

1 similar comment
@BenTheElder
Copy link
Member Author

/test pull-kind-build

@BenTheElder BenTheElder mentioned this pull request Oct 20, 2019
Closed
@BenTheElder
Copy link
Member Author

/test all

@BenTheElder BenTheElder removed the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 23, 2019
@BenTheElder BenTheElder changed the title Test PR DO NOT MERGE [WIP] Test PR DO NOT MERGE Oct 23, 2019
@BenTheElder BenTheElder mentioned this pull request Dec 15, 2021
Closed
@BenTheElder BenTheElder closed this May 9, 2022
@BenTheElder BenTheElder reopened this May 9, 2022
@ameukam
Copy link
Member

ameukam commented Jan 12, 2024

/test all

1 similar comment
@BenTheElder
Copy link
Member Author

/test all

@BenTheElder BenTheElder mentioned this pull request May 13, 2024
Merged
@BenTheElder BenTheElder reopened this Jun 11, 2024
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign bentheelder for approval. For more information see the Kubernetes Code Review Process.

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

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@BenTheElder BenTheElder mentioned this pull request Jun 11, 2024
Merged
@stmcginnis
Copy link
Contributor

Issue with the Fedora image itself?

There was an error while downloading the metadata for this box.
The error message is shown below:

The requested URL returned error: 404
==> default: VM not created. Moving on...

@aojea
Copy link
Contributor

aojea commented Jun 11, 2024 •
edited
Loading

yeah, Not Found

kind/hack/ci/Vagrantfile

Line 10 in dda3e5b

config.vm.box_url = "https://iad.mirror.rackspace.com/fedora/releases/37/Cloud/x86_64/images/Fedora-Cloud-Base-Vagrant-37-1.7.x86_64.vagrant-virtualbox.box"

@BenTheElder
Copy link
Member Author

yeah. #3653

@BenTheElder
Copy link
Member Author

That'll do it ...

@BenTheElder BenTheElder reopened this Jun 11, 2024
@BenTheElder
Copy link
Member Author

/test pull-kind-test

@BenTheElder BenTheElder mentioned this pull request Jul 1, 2024
Merged
@BenTheElder
Copy link
Member Author

/test all

@BenTheElder
Copy link
Member Author

/test pull-kind-test

1 similar comment
@BenTheElder
Copy link
Member Author

/test pull-kind-test

@BenTheElder
Copy link
Member Author

ok so, the docker network tests are now flaky because of some change in docker or the host. https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_kind/648/pull-kind-test/1807905535295492096

@BenTheElder BenTheElder mentioned this pull request Jul 1, 2024
Open
@BenTheElder
Copy link
Member Author

BenTheElder commented Jul 2, 2024 •
edited
Loading

/test all
( after new CI images ref #3677 )

@BenTheElder
Copy link
Member Author

/retest

1 similar comment
@BenTheElder
Copy link
Member Author

/retest

@BenTheElder
Copy link
Member Author

/test pull-kind-test
[ checking for flakiness ]

2 similar comments
@BenTheElder
Copy link
Member Author

/test pull-kind-test
[ checking for flakiness ]

@BenTheElder
Copy link
Member Author

/test pull-kind-test
[ checking for flakiness ]

@BenTheElder
Copy link
Member Author

/test all

# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@BenTheElder @aojea @k8s-ci-robot @liggitt @k8s-triage-robot @ameukam @stmcginnis