images/base: disable udev.service

[AkihiroSuda]

Discussed in #1935 (comment)

[k8s-ci-robot]

Hi @AkihiroSuda. Thanks for your PR.

I'm waiting for a kubernetes-sigs 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.

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.

[AkihiroSuda]

Failure of Podman CI seems unrelated

• Waiting ≤ 1m0s for control-plane = Ready ⏳  ...
 ✓ Waiting ≤ 1m0s for control-plane = Ready ⏳
 • Ready after 28s 💚
ERROR: failed to create cluster: network details should only be one line, got 0 lines

[AkihiroSuda]

@BenTheElder PTAL

[BenTheElder]

podman CI is indeed broken atm (breaking podman changes...) aojea is on it I took tonight to work on the out of sync contributor guide mostly, I'll get back to PR review in the morning 😅

…

On Thu, Dec 3, 2020 at 1:20 AM Akihiro Suda ***@***.***> wrote: @BenTheElder <https://github.com/BenTheElder> PTAL — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1948 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAHADK4D2PAI4EVUBFHCTETSS5J5PANCNFSM4UIU466A> .

[aojea]

podman CI is indeed broken atm (breaking podman changes...) aojea is on it

fixed by #1790

[BenTheElder]

fix is merged now, but I'm off to exercise the pup. will review tonight.

also, github actions requires that we rebase this PR to pick up the fix, unlike prow 😞

[AkihiroSuda]

thanks, rebased

[BenTheElder]

/ok-to-test

[BenTheElder]

/lgtm
/approve
/hold
(will do follow-up in the morning and get this in)

[AkihiroSuda]

ping maintainers

[aojea]

@BenTheElder we need to bump the image on #1979 , so I think we should have this in before that lands

[BenTheElder]

I did the bump push. I did not add it to the PR because everything is on hold pending the fact that we seem to have broken kind in github actions.
When we get sufficient time to resolve that we can move with more changes.

[BenTheElder]

#1969 should be resolved, finally 🤞

[AkihiroSuda]

rebased

[BenTheElder]

/lgtm
/approve

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: AkihiroSuda, BenTheElder

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 [BenTheElder]

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