[openwrt] Convert into an OpenWRT package #6

[devkapilbansal]

Closes #6

• Create package NetJSON Monitoring and OpenWISP Monitoring.
• Add a CI Workflow
• Add directory structure with common files and docs as other OpenWISP modules
• Add runbuild script to automatically compile packages

[devkapilbansal]

Opened this WIP PR so that it becomes easy to stack all relevant conversation.

[devkapilbansal]

@nemesisdesign finally I am able to create an image but when I try to install it I got this error:-

* opkg_install_cmd: Cannot install package https://github.com/devkapilbansal/test-repo/blob/main/openwisp-openwrt-monitoring_0.0.1-1_all.ipk?raw=True.

P.S.:- I uploaded the image generated here
Update: Solved

[nemesifier]

@devkapilbansal I see that you're putting effort, but the makefile looks kinda wrong which is normal if you start copying things without really understanding what you're doing, so my suggestion is to start creating a very simple package for OpenWRT with something trivial (eg: a script in /usr/sbin which prints hello world, get it to work (eg: you can call it from openwrt and it prints hello world) and then gradually add things.

In this specific case, giving us that output will not help, always try to look if the program you're running has a verbose output and include the verbose output in your request for help. Check with opkg --help) and let me know what you can find.

[nemesifier]

This is on the right track, please read again the deliverable:

Convert lua-monitoring into two OpenWRT packages:

One package for the netjson-monitoring utility, which aims to simply return NetJSON DeviceMonitoring output

One package which provides the OpenWISP Monitoring deamon, which depends on netjson-monitoring and openwisp-config (it takes the server information from the openwisp config file).

We need two packages, one of them is a daemon (crontab was a quick way to implement it).

Also remove netjson-monitoring-wan.lua please.

[nemesifier]

please start preparing a basic README

[nemesifier]

Can you please show the content of your feeds and commands to compile?

Ensure the feeds are correct and that this branch is being installed when you run ./scripts/feeds install -a && ./scripts/feeds update -a, you should see this repo and this branch being cloned and installed on the openwrt source directory (you will find it in ./feeds, if it's not there, something is wrong with the feeds).

Once you're sure the feeds are pulled correctly, ensure your .config contains:

CONFIG_PACKAGE_openwisp-monitoring=y
CONFIG_PACKAGE_netjson-monitoring=y

Although you should be able to see these packages from make menuconfig.

Then compile everything with make -j <number-of-cores> V=s where j <number-of-cores> stands for the number of cores your machine has available.

See also my comments below.

[devkapilbansal]

Can you please show the content of your feeds and commands to compile?

Ensure the feeds are correct and that this branch is being installed when you run ./scripts/feeds install -a && ./scripts/feeds update -a, you should see this repo and this branch being cloned and installed on the openwrt source directory (you will find it in ./feeds, if it's not there, something is wrong with the feeds).

Once you're sure the feeds are pulled correctly, ensure your .config contains:

CONFIG_PACKAGE_openwisp-monitoring=y
CONFIG_PACKAGE_netjson-monitoring=y

Although you should be able to see these packages from make menuconfig.

Then compile everything with make -j <number-of-cores> V=s where j <number-of-cores> stands for the number of cores your machine has available.

See also my comments below.

I am not able to reproduce the error that I was encountoring yesterday and package too compiled successfully.

[devkapilbansal]

Also remove netjson-monitoring-wan.lua please.

@nemesisdesign does #9 makes sense now? I think they are merged already

[mips171]

Can you please show the content of your feeds and commands to compile?

Ensure the feeds are correct and that this branch is being installed when you run ./scripts/feeds install -a && ./scripts/feeds update -a, you should see this repo and this branch being cloned and installed on the openwrt source directory (you will find it in ./feeds, if it's not there, something is wrong with the feeds).

Once you're sure the feeds are pulled correctly, ensure your .config contains:

CONFIG_PACKAGE_openwisp-monitoring=y
CONFIG_PACKAGE_netjson-monitoring=y

Although you should be able to see these packages from make menuconfig.

Then compile everything with make -j <number-of-cores> V=s where j <number-of-cores> stands for the number of cores your machine has available.

See also my comments below.

You can also do make -j$(nproc) or if you have a CPU with hyperthreading make -j$(($(nproc) * 2)). nproc shows the number of CPU cores on your system. More tips

[devkapilbansal]

You can also do make -j$(nproc) or if you have a CPU with hyperthreading make -j$(($(nproc) * 2)). nproc shows the number of CPU cores on your system. More tips

Hi, the problem was solved already but thanks for this. I will remember this for future

[nemesifier]

Great progress @devkapilbansal, see my comments below.

[nemesifier]

Please make sure of the following:

• the packages compile
• packages can be installed on a virgin OpenWRT installation and when this is done it pulls openwisp-config as a dependency (and its dependencies)
• ensure it can successfully post data to OpenWISP monitoring

Then we have to double check this, once those are checked we can merge this and go ahead with the other work.

[devkapilbansal]

• packages can be installed on a virgin OpenWRT installation and when this is done it pulls openwisp-config as a dependency (and its dependencies)

Hi, I tried it on fresh OpenWRT but as netjson-monitoring is a dependency of openwisp-monitoring and it is not available in openwrt feeds, I have to install it first.
Then openwisp-monitoring package was able to pull openwisp-config and its dependencies.

I tested and all 5 packages compile ( 1 netjson-monitoring and 4 variants of openwisp-monitoring). The data was sent successfully to the server and error was logged when there was no connection to server.

P.S.:- I use openssl variant for testing.

Update:- There was a minor error that I fixed in f7d3ad2

[nemesifier]

Great progress, please specify in the README that the openwisp-monitoring openwrt package depends on openwisp-config (please also link "openwisp-config" to the URL of the repo) and that installing it will automatically install also openwisp-config and its dependencies.

[nemesifier]

Let's move on so you can test this with github actions CI.