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 Graz to census #27

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

mkg20001
Copy link
Member

@mkg20001 mkg20001 commented Mar 3, 2023

We've got non-gluon nodes in our hopglass aswell (https://hopglass.ffgraz.net). You could just ignore those, they don't have any version that matches the gluon pattern.

Also do you prefer meshviewer.json or nodes.json? We've got both.

Note that graz is part of funkfeuer - which has similar goals as freifunk - and is in austria. We're currently giving out neighbours in batman-compatible format.

@mweinelt mweinelt closed this May 14, 2023
@mweinelt mweinelt reopened this May 14, 2023
@mweinelt
Copy link
Contributor

Wondering why the actions aren't running here.

@mkg20001
Copy link
Member Author

I rebased, that didn't fix it either. It's in a different repository, maybe that makes a difference?

@mkg20001
Copy link
Member Author

Actions were disabled on the fork, I enabled them. Still I don't see them being ran.

@mkg20001
Copy link
Member Author

This scheduled workflow is disabled because scheduled workflows are disabled by default in forks.

You may want to make another copy of the workflow that is not scheudled because github appears to handle it stupidly

@mkg20001
Copy link
Member Author

@mkg20001
Copy link
Member Author

mkg20001 commented May 14, 2023

Hrm so there's no Graz in the output

Our version numbers look like this: gluon-v2021.1-997-g2cfcbe1

Might that be the issue?

@mkg20001
Copy link
Member Author

mkg20001 commented May 14, 2023

Ok so the issue is that one of the nodeid fields is null (something wrong with our data). I made it so the broken value is ignored and exceptions are printed instead of silently ignored.

New CI: https://github.com/ffgraz/census-exporter/actions/runs/4974688289

gluon_model_total{community="Graz",model="0x467 00"} 3.0

It adds the non-gluon nodes (this is a boardid from https://git.mkg20001.io/mkg20001/respondd.sh from openwrt 8 - congrats to our age old infrastructure)

We also have:

gluon_model_total{community="Graz",model="Raspberry Pi 4 Model B Rev 1.2"} 1.0

(you know the joke about temporary things. this isn't gluon, this is a random server. with some hacks it's running respondd - offtopic, i could publish a flake with respondd nixpkg + patches for server usage)

@maurerle
Copy link
Member

I am in favor of merging this so that the gluon nodes from the ffgraz community are also taken into account

@mkg20001
Copy link
Member Author

Is there anything left for me to do on this PR?

(The nodes may appear offline until end of month for a bit, because we're moving infrastructure arround)

@AiyionPrime
Copy link
Member

AiyionPrime commented Aug 2, 2024

I'm in favor of merging as well, Graz contributed to freifunk gluon in the past and likely in the future as well. Are there any technical reasons not to merge left?

@T-X
Copy link
Contributor

T-X commented Oct 16, 2024

I'm currently getting a timeout from this:

$ wget "https://hopglass.ffgraz.net/data/nodes.json"
--2024-10-16 04:29:13--  https://hopglass.ffgraz.net/data/nodes.json
Resolving hopglass.ffgraz.net (hopglass.ffgraz.net)... 2001:470:508d::66, 193.33.151.50
Connecting to hopglass.ffgraz.net (hopglass.ffgraz.net)|2001:470:508d::66|:443... failed: Connection timed out.
Connecting to hopglass.ffgraz.net (hopglass.ffgraz.net)|193.33.151.50|:443... failed: Connection timed out.
Retrying.

--2024-10-16 04:33:43--  (try: 2)  https://hopglass.ffgraz.net/data/nodes.json
Connecting to hopglass.ffgraz.net (hopglass.ffgraz.net)|2001:470:508d::66|:443...
...

But I would otherwise also love to see this merged. (And I tested these changes, they don't seem to otherwise break anything, as far as I can tell. And look fine to me.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants