Cambium ePMP Support

[martydingo]

Description

Pull Request for Cambium ePMP devices to add support for these devices. The model name to use is cambiumepmp

I've had this running on our Oxidized instance and it's working well and perfectly stable for the last three months.

This will close off #2000 with people reporting that it works great in that issue.

This will close off this issue also. #2081

[codecov-commenter]

Codecov Report

Merging #2195 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2195   +/-   ##
=======================================
  Coverage   63.32%   63.32%           
=======================================
  Files          30       30           
  Lines        1497     1497           
=======================================
  Hits          948      948           
  Misses        549      549           

---

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 be01808...c2c7136. Read the comment docs.

[martydingo]

Had some trailing whitespaces, probably added by my code documentation tool..

All sorted now.

[wk]

Hi there!

This looks great! It could use just a bit more documentation so that users can find it as part of the PR:

• A mention in CHANGELOG.md taking credit for the new model
• An addition to docs/Supported-OS-Types.md mentioning the relevant model and OS type

[martydingo]

@wk Thanks very much, I've added some more documentation in, let me know if there's anything else required.

[martydingo]

I'm sure the codecov doesn't apply for this situation?

[djh00t]

Hi Guys - any movement on this PR?

[martydingo]

@djh00t No, still waiting for someone to come along and accept the PR.

[ghost]

Please fix the conflict

[martydingo]

Please fix the conflict

You want me to fix the conflict created by this PR being outstanding for over 12 months and therefore the changelog is conflicted?

I fixed this once and this has reoccured, is this going to be merged following this fix?

[ghost]

will merge then

[martydingo]

@mortzu The CHANGELOG has been updated

[martydingo]

lib/oxidized/model/model.rb:196:25: W: Lint/UnusedBlockArgument: Unused block argument - line. You can omit the argument if you don't care about it.

Is the error, which this PR does not touch. Please merge or reply with any outstanding actions required before the conflict reoccurs.

Thanks