Add MQTT container log limitation

[atrovato]

Pull Request check-list

To ensure your Pull Request can be accepted as fast as possible, make sure to review and check all of these items:

• If your changes affects code, did your write the tests?
• Are tests passing? (npm test on both front/server)
• Is the linter passing? (npm run eslint on both front/server)
• Did you run prettier? (npm run prettier on both front/server)
• [ ] If you are adding a new features/services, did you run integration comparator? (npm run compare-translations on front)
• Did you test this pull request in real life? With real devices? If this development is a big feature or a new service, we recommend that you provide a Docker image to the community for testing before merging.
• [ ] If your changes modify the API (REST or Node.js), did you modify the API documentation? (Documentation is based on comments in code)
• [ ] If you are adding a new features/services which needs explanation, did you modify the user documentation? See the GitHub repo and the website.
• [ ] Did you add fake requests data for the demo mode (front/src/config/demo.js) so that the demo website is working without a backend? (if needed) See https://demo.gladysassistant.com.

NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open.

Description of change

Add log limit on MQTT container.
I add a new service variable to track the container update.
Image : atrovato/gladys:mqtt-log-limit

[atrovato]

Sorry, I didn't have the time to test it in real life yet.

[codecov]

Codecov Report

Base: 97.27% // Head: 97.27% // Increases project coverage by +0.00% 🎉

Coverage data is based on head (adfe9c0) compared to base (3d2ffc8).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1686   +/-   ##
=======================================
  Coverage   97.27%   97.27%           
=======================================
  Files         643      643           
  Lines        9686     9687    +1     
=======================================
+ Hits         9422     9423    +1     
  Misses        264      264           

Impacted Files	Coverage Δ
server/services/mqtt/lib/constants.js	100.00% <ø> (ø)
server/services/mqtt/lib/updateContainer.js	100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[atrovato]

I changed my choice, I use the existing service variable to know if docker file is at the latest revision.

[atrovato]

Ok, I tested it, I can see this when I docker inspect gladys:

            "LogConfig": {
                "Type": "json-file",
                "Config": {
                    "max-size": "10m"
                }
            },

Works fine according to me :)

[Pierre-Gilles]

Tested on my side, works fine! Thanks for this PR 🙏

[relativeci]

Job #998: Bundle Size — 7.01MiB (0%).

1e02ce6(current) vs 6d7d3e8 master#997(baseline)

Metrics (no changes)

	Current Job #998	Baseline Job #997
Initial JS	3.03MiB	3.03MiB
Initial CSS	294.78KiB	294.78KiB
Cache Invalidation	0%	0%
Chunks	52	52
Assets	148	148
Modules	1258	1258
Duplicate Modules	2	2
Duplicate Code	0.03%	0.03%
Packages	111	111
Duplicate Packages	8	8

Total size by type (no changes)

	Current Job #998	Baseline Job #997
CSS	312.43KiB	312.43KiB
Fonts	93.55KiB	93.55KiB
HTML	13.58KiB	13.58KiB
IMG	1.64MiB	1.64MiB
JS	4.95MiB	4.95MiB
Media	0B	0B
Other	4.95KiB	4.95KiB

---

View job #998 report View master branch activity