Skip to content

Check hardware for AP and if so record 'can_be_ap = True' to /etc/iiab/iiab.ini#3222

Merged
holta merged 7 commits intoiiab:masterfrom
holta:check4AP
May 19, 2022
Merged

Check hardware for AP and if so record 'can_be_ap = True' to /etc/iiab/iiab.ini#3222
holta merged 7 commits intoiiab:masterfrom
holta:check4AP

Conversation

@holta
Copy link
Copy Markdown
Member

@holta holta commented May 19, 2022

Tested on 5 different hardware scenarios.

Refines the following PR, which also needed a couple Ansible fixes:

@holta holta added this to the 8.0 milestone May 19, 2022
@holta
Copy link
Copy Markdown
Member Author

holta commented May 19, 2022

@jvonau is this PR hopefully good enough, in your opinion?

@jvonau
Copy link
Copy Markdown
Contributor

jvonau commented May 19, 2022

#3216 has the whole test series, why was that one not updated, are you not considering the other 2 pending PRs?

@tim-moody
Copy link
Copy Markdown
Contributor

Is anyone worried that a user will type iiab-hotspot on and get a file not found error rather than being told that their system does not support a hotspot?

@jvonau
Copy link
Copy Markdown
Contributor

jvonau commented May 19, 2022

'Release Notes' should be altered to explain why that command may not be found and where to look in the iiab.ini file to confirm what was or not detected. I'm not concerned as that should prompt a question from the user for further clarification or the user could figure that out from the 'can_be_ap' entry in iiab.ini should there some familiarity with the code base or adequate notes available.

@jvonau
Copy link
Copy Markdown
Contributor

jvonau commented May 19, 2022

Tested on 5 different hardware scenarios.

Could I be so bold as to ask which 5 and are their iiab-diagnostics available for review?

Refines the following PR, which also needed a couple Ansible fixes:

Were altered in the roll-up PR.

@holta
Copy link
Copy Markdown
Member Author

holta commented May 19, 2022

are you not considering the other 2 pending PRs?

I definitely want to work through these 2 PR's also:

@holta
Copy link
Copy Markdown
Member Author

holta commented May 19, 2022

Is anyone worried that a user will type iiab-hotspot on and get a file not found error rather than being told that their system does not support a hotspot?

It's a great question: in the sense that this certainly needs to documented prominently (somewhere!) so that the answer's right in your face (or worst case loudly advertise a URL, explaining better the details?)

  1. Should iiab-hotspot-on explain better, in similar such circumstances?

  2. Or is it rather too late at that point — maybe this coming-to-terms with machine/topology ground-truths should happen earlier on — e.g. is this more the job of things like ./iiab-network i.e. roles/network/tasks/main.yml ?

@jvonau
Copy link
Copy Markdown
Contributor

jvonau commented May 19, 2022

Guess providing the roll-up PR was a waste of my time.

@holta
Copy link
Copy Markdown
Member Author

holta commented May 19, 2022

'Release Notes' should be altered to explain why that command may not be found and where to look in the iiab.ini file to confirm what was or not detected.

Yes we should start there.

But "nobody reads" in this era (unless they're absolutely forced to!)

So we should "point their nose" precisely in the right direction — e.g. in Ansible's output and/or iiab-hotspot-on output — in plain English as much as possible.

@holta
Copy link
Copy Markdown
Member Author

holta commented May 19, 2022

Guess providing the roll-up PR was a waste of my time.

I'll look at it too. I thought it was equivalent to #3070 + #3173 + #3179 combined into one PR. But if I misunderstood, i.e. that it also includes subsequent improvements, I apologize.

@jvonau
Copy link
Copy Markdown
Contributor

jvonau commented May 19, 2022

But "nobody reads" in this era (unless they're absolutely forced to!)

So we should "point their nose" precisely in the right direction — e.g. in Ansible's output and/or iiab-hotspot-on output — in plain English as much as possible.

Then toss this entire PR and place the detection in iiab-hotspot-on

@jvonau
Copy link
Copy Markdown
Contributor

jvonau commented May 19, 2022

Guess providing the roll-up PR was a waste of my time.

I'll look at it too. I thought it was equivalent to #3070 + #3173 + #3179 combined into one PR. But if I misunderstood, i.e. that it also includes subsequent improvements, I apologize.

I clearly referenced what changes for 3070 where done within the roll up.

@holta
Copy link
Copy Markdown
Member Author

holta commented May 19, 2022

Tested on 5 different hardware scenarios.

which 5

Age-old Intel NUC's mostly, each of which had their own weird output when running things like iw list | grep AP

Some more long-winded than others.

I can re-run several of those examples and paste the relevant output into this ticket.

@holta
Copy link
Copy Markdown
Member Author

holta commented May 20, 2022

This new feature is summarized at:
https://twitter.com/internet_in_box/status/1527690188637540352

@holta
Copy link
Copy Markdown
Member Author

holta commented May 21, 2022

Thanks to @jvonau whose PR #3179 further cleaned up here:

{% if not can_be_ap %}
echo -e "\nUH-OH: Your Wi-Fi firmware doesn't support AP mode, according to 'iw list'\n"
echo -e "If you add Wi-Fi hardware, run 'cd /opt/iiab/iiab' then 'sudo ./iiab-network'\n"
echo -e "For details, see: https://github.com/iiab/iiab/pull/3179\n"
exit 1

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.

3 participants