Skip to content

Account for Concurrent AiMesh Node Operations - #604

Merged
ExtremeFiretop merged 4 commits into
ExtremeFiretop:devfrom
Martinski4GitHub:dev
Sep 22, 2026
Merged

ExtremeFiretop merged 4 commits into
ExtremeFiretop:devfrom
Martinski4GitHub:dev

Conversation

@Martinski4GitHub

Copy link
Copy Markdown
Collaborator

Adjusted code to account for concurrent AiMesh node operations.

@Martinski4GitHub

Copy link
Copy Markdown
Collaborator Author

@ExtremeFiretop,
Here's a solution for the concurrent Aimesh node operations. Great catch!!!

Fixed a typo which would have cause a bug!!
Comment thread MerlinAU.sh Outdated
local responseFPath="${curlTmpRespFile}.NODE.LOGIN"
local responseFPath="${curlTmpRespFile}.${4}.NODE.LOGIN"
local curlErrLogFile="${curlErrLogFPath}.${4}.NODE.LOGIN"
local curlTmpLogFile="${curlTmpLogFPath}.${4}.NONE.LOGIN"

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is a typo? Did you mean NODE instead of NONE?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is a typo? Did you mean NODE instead of NONE?

Yes, you're absolutely correct; it's a typo, LOL!!!
I had slippery fingers last night!!! 😜😉

P.S.
I just finished my lunch hour, so going offline and back to work...
Talk to you later, bud!!!

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You kill me sometimes hahaha 😂😂
All good! Otherwise the PR looks great to me and ready to merge!

I think we might want to consider a release afterwards. Maybe target the Friday or weekend if we can!

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo is now fixed!! LOL!!

You kill me sometimes hahaha 😂😂 All good! Otherwise the PR looks great to me and ready to merge!

I think we might want to consider a release afterwards. Maybe target the Friday or weekend if we can!

Fully agreed. This coming Friday, or sometime over the weekend sounds good to me.

BTW, this coming weekend, I'll be helping a close friend install and configure his new router (Ubiquiti Dream Router 7). He got a good deal with a FTTH symmetric 1.0Gbps plan, so he's retiring his RT-AX86S router. In good conscience, I couldn't recommend buying any of the new BE-class ASUS routers, given the high prices plus the quality issues reported in the forums, especially when compared with the UDR7 at ~$300.00. However, we might try setting up the RT-AX86S as an AP if it integrates well with the UDR7; otherwise, he may just give it to me (he knows I can use it for add-on testing/development). We'll see how it goes.

P.S.
I've got 1 hour to 90 more minutes of work to do, so going offline now...

Comment thread MerlinAU.sh
Fixed another typo, LOL!!!!
@ExtremeFiretop
ExtremeFiretop merged commit d179552 into ExtremeFiretop:dev Sep 22, 2026
1 check passed
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.

2 participants