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

Qemu 12 nodes in 4 different clouds #813

Merged
merged 24 commits into from
Apr 29, 2021

Conversation

altergui
Copy link
Member

started working in a virtualized mesh cloud

@codecov-io
Copy link

codecov-io commented Nov 24, 2020

Codecov Report

Merging #813 (41a7b9e) into master (19cc376) will decrease coverage by 0.07%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #813      +/-   ##
==========================================
- Coverage   72.27%   72.19%   -0.08%     
==========================================
  Files          37       38       +1     
  Lines        3293     3334      +41     
==========================================
+ Hits         2380     2407      +27     
- Misses        913      927      +14     
Impacted Files Coverage Δ
...ckages/pirania/files/usr/lib/lua/voucher/utils.lua 47.93% <0.00%> (-7.26%) ⬇️
...oto-batadv/files/usr/lib/lua/lime/proto/batadv.lua 96.66% <0.00%> (ø)
...-lime-location/files/usr/lib/lua/lime/location.lua 96.42% <0.00%> (ø)
...oto-babeld/files/usr/lib/lua/lime/proto/babeld.lua 95.89% <0.00%> (+0.23%) ⬆️
...ges/lime-system/files/usr/lib/lua/lime/network.lua 78.18% <0.00%> (+0.36%) ⬆️
...ubus-lime-utils/files/usr/lib/lua/lime/upgrade.lua 60.09% <0.00%> (+0.58%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 19cc376...41a7b9e. Read the comment docs.

@spiccinini
Copy link
Contributor

Looks good! I will test it :)

tools/qemu_cloud_start Outdated Show resolved Hide resolved
@spiccinini
Copy link
Contributor

What is the idea behind the ETH2 interface?

@altergui
Copy link
Member Author

altergui commented Dec 17, 2020

after major work, made an ansible playbook that reads a YAML describing a complex set of nodes distributed accross different clouds with links connecting them, using eth2 interface

@altergui altergui changed the title Qemu 11node cloud Qemu 12 nodes in 4 different clouds Dec 17, 2020
@altergui altergui force-pushed the qemu-11node-cloud branch 2 times, most recently from 3c5fc9c to 252dbb6 Compare December 17, 2020 04:17
@altergui
Copy link
Member Author

with this rebase i dropped the --no-serial argument (it's not really important) and used argbash to handle arguments, as indicated
https://github.com/libremesh/lime-packages/compare/1143e39753d7a43953366e17ca79254223db1bc9..5407071393ed5c3868d377e0973e2b4d3b502804

…sts only

Signed-off-by: Gui Iribarren <gui@altermundi.net>
@spiccinini
Copy link
Contributor

I tested the non cloud operation and it is working :)

At first it the --enable-wan options wasn't working and It took me some time to discover that as the MAC configuration changed the dnsmasq lease time was preventing dnsmasq to offer an ip as the range is only one IP .... Anyway, everything good, I've just deleted the lease file (/var/lib/misc/dnsmasq.leases in my distro).

@spiccinini
Copy link
Contributor

spiccinini commented Feb 4, 2021

One thing before merging. Which copyright and license the python files have?

tools/qemu_cloud_start Outdated Show resolved Hide resolved
@spiccinini
Copy link
Contributor

@altergui please can you answer the remaining questions? I want to merge this :)

Signed-off-by: Gui Iribarren <gui@altermundi.net>
@altergui
Copy link
Member Author

One thing before merging. Which copyright and license the python files have?

good catch! no license whatsoever. wrote an email to the author asking for clarification.
i think (based on other code by same author) it's safe to assume the author intended to publish under MIT license and just ommitted it.
we can wait a bit until a proper author statement, and if that doesn't happen... no idea

Signed-off-by: Gui Iribarren <gui@altermundi.net>
@altergui
Copy link
Member Author

success. MIT license it is.
https://gist.github.com/upa/94f82e00dfffbb25d896/revisions

@codecov-commenter
Copy link

Codecov Report

Merging #813 (d13d38d) into master (19cc376) will increase coverage by 0.18%.
The diff coverage is 76.94%.

❗ Current head d13d38d differs from pull request most recent head 10b7dc2. Consider uploading reports for the commit 10b7dc2 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master     #813      +/-   ##
==========================================
+ Coverage   72.27%   72.45%   +0.18%     
==========================================
  Files          37       41       +4     
  Lines        3293     3616     +323     
==========================================
+ Hits         2380     2620     +240     
- Misses        913      996      +83     
Impacted Files Coverage Δ
...-boot-wizard/files/usr/lib/lua/firstbootwizard.lua 42.99% <15.00%> (-1.01%) ⬇️
...kages/lime-system/files/usr/lib/lua/lime/utils.lua 81.61% <50.00%> (-0.70%) ⬇️
...es/shared-state/files/usr/lib/lua/shared-state.lua 72.00% <71.34%> (ø)
packages/ubus-tmate/files/usr/lib/lua/tmate.lua 84.09% <86.11%> (ø)
...ed-state-bat_hosts/files/usr/lib/lua/bat-hosts.lua 92.85% <96.96%> (ø)
...oto-babeld/files/usr/lib/lua/lime/proto/babeld.lua 95.89% <100.00%> (+0.23%) ⬆️
...oto-batadv/files/usr/lib/lua/lime/proto/batadv.lua 96.66% <100.00%> (ø)
...-lime-location/files/usr/lib/lua/lime/location.lua 96.42% <100.00%> (ø)
...ubus-lime-utils/files/usr/lib/lua/lime/upgrade.lua 60.66% <100.00%> (+1.15%) ⬆️
tests/utils.lua 98.86% <100.00%> (+0.01%) ⬆️
... and 8 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 19cc376...10b7dc2. Read the comment docs.

@spiccinini
Copy link
Contributor

Thank gui!

@spiccinini spiccinini merged commit 0a26286 into libremesh:master Apr 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants