Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

add development_kind #65

Merged
merged 1 commit into from
May 11, 2022
Merged

add development_kind #65

merged 1 commit into from
May 11, 2022

Conversation

eranra
Copy link
Contributor

@eranra eranra commented Mar 3, 2022

This PR adds an end-to-end kind deployment for the operator to allow quick and efficient development of the operator locally. It deployed Kind with OVN-Kubernetes and all relevant components

@eranra
Copy link
Contributor Author

eranra commented Mar 3, 2022

@jotak the kind.sh is not working for me --- I am using windows WSL as my "linux machine" and I get::


eranra@ERANRAICHS-1MM6-wsl [3059] ~
$ kubectl logs ovs-node-5fvl4 -n ovn-kubernetes
================== ovnkube.sh --- version: 3 ================
 ==================== command: ovs-server
 =================== hostname: ovn-worker
 =================== daemonset version 3
 =================== Image built from ovn-kubernetes ref: refs/heads/master  commit: fc51865676fd515ebaaa7b43ebeeb139f30a4ce6
Starting ovsdb-server.
Configuring Open vSwitch system IDs.
Enabling remote OVSDB managers.
modprobe: FATAL: Module openvswitch not found in directory /lib/modules/5.10.60.1-microsoft-standard-WSL2
Inserting openvswitch module ... failed!
rmmod: ERROR: could not open '/sys/module/bridge/holders': No such file or directory
rmmod: ERROR: Module unloading is not supported
removing bridge module ... failed!

@openshift-ci
Copy link

openshift-ci bot commented Mar 3, 2022

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please ask for approval from eranra after the PR has been reviewed.

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

.mk/development.mk Outdated Show resolved Hide resolved
@jotak
Copy link
Member

jotak commented Apr 20, 2022

Sorry Eran, this PR has been pending for a long time...
My fear is that it could create more maintenance burden, to deal with all the various environment that the team is using , when starting something like kind.
I have some failures when I try to run it, I'd need to investigate what's going on

 ✗ Ensuring node image (kindest/node:v1.23.3) 🖼 
ERROR: failed to create cluster: failed to pull image "kindest/node:v1.23.3": command "podman pull kindest/node:v1.23.3" failed with error: exit status 125
Command Output: Error: short-name resolution enforced but cannot prompt without a TTY
make: *** [.mk/development.mk:5 : create-kind-cluster] Erreur 1

I also remember I had to do some special stuff wrt. selinux

.mk/development.mk Outdated Show resolved Hide resolved
@jotak
Copy link
Member

jotak commented Apr 21, 2022

After fixing the issues, I'm still getting a timeout error during control plane init :/
When I used kind previously, I was passing -wk 1 to kind.sh which used to work better with my small 16GB laptop, but now even that isn't enough (or there's another problem that doesn't show up in my logs).
Hopefully I should soon receive a better laptop :)

@eranra
Copy link
Contributor Author

eranra commented Apr 21, 2022

/retest

@eranra eranra force-pushed the development_kind branch 2 times, most recently from 3e397c1 to 30cc920 Compare April 21, 2022 15:31
@eranra
Copy link
Contributor Author

eranra commented Apr 21, 2022

Hi @jotak after fighting with OVN-k8s for half a day I moved to the old get logs from simulator on kind approach. This is not 100% testing all the functionality of the operator but it does provide a lot of value ... let me know what do you think on the updated PR code.

@eranra eranra force-pushed the development_kind branch 3 times, most recently from a093cfd to afb219c Compare April 24, 2022 07:57
@eranra eranra requested a review from jotak April 24, 2022 08:01
@eranra eranra force-pushed the development_kind branch 2 times, most recently from 60e390a to 83e5b05 Compare April 24, 2022 09:58
.mk/local.mk Outdated Show resolved Hide resolved
.mk/local.mk Outdated Show resolved Hide resolved
@jotak
Copy link
Member

jotak commented May 11, 2022

Not sure exactly why the CI fails, but I can reproduce locally, if I delete my bin directory then run make build, it fails due to missing bin/controller-gen. It works on the main branch.

I guess it is related to .bingo ? By the way, could you explain to the team a little bit about bingo, why using it and what it changes for us? (I don't know if others are familiar with it, personally I'm not)

@eranra
Copy link
Contributor Author

eranra commented May 11, 2022

Not sure exactly why the CI fails, but I can reproduce locally, if I delete my bin directory then run make build, it fails due to missing bin/controller-gen. It works on the main branch.

I guess it is related to .bingo ? By the way, could you explain to the team a little bit about bingo, why using it and what it changes for us? (I don't know if others are familiar with it, personally, I'm not)

Hi @jotak it was because the list of MakeFile targets changed ... it was not because of bingo but because of the additional .mk files under the .mk folder. I changed the PROJECT_DIR variable to use firstword and not lastword so now it uses the first Makefile which is the main Makefile and now things are working as expected :-)

Copy link
Member

@jotak jotak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

thanks @eranra !

@openshift-ci openshift-ci bot added the lgtm label May 11, 2022
@eranra eranra merged commit 02d9152 into netobserv:main May 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants