Check hardware for AP and if so record 'can_be_ap = True' to /etc/iiab/iiab.ini#3222
Check hardware for AP and if so record 'can_be_ap = True' to /etc/iiab/iiab.ini#3222holta merged 7 commits intoiiab:masterfrom
Conversation
|
@jvonau is this PR hopefully good enough, in your opinion? |
|
#3216 has the whole test series, why was that one not updated, are you not considering the other 2 pending PRs? |
|
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? |
|
'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. |
Could I be so bold as to ask which 5 and are their iiab-diagnostics available for review?
Were altered in the roll-up PR. |
I definitely want to work through these 2 PR's also: |
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?)
|
|
Guess providing the roll-up PR was a waste of my time. |
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 |
Then toss this entire PR and place the detection in iiab-hotspot-on |
I clearly referenced what changes for 3070 where done within the roll up. |
Age-old Intel NUC's mostly, each of which had their own weird output when running things like Some more long-winded than others. I can re-run several of those examples and paste the relevant output into this ticket. |
|
This new feature is summarized at: |
|
Thanks to @jvonau whose PR #3179 further cleaned up here: iiab/roles/network/templates/hostapd/iiab-hotspot-on Lines 2 to 6 in 3575085 |
Tested on 5 different hardware scenarios.
Refines the following PR, which also needed a couple Ansible fixes: