Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Individual loggers for each ChargePoint instance #655

Closed
jainmohit2001 opened this issue Aug 28, 2024 · 13 comments
Closed

Individual loggers for each ChargePoint instance #655

jainmohit2001 opened this issue Aug 28, 2024 · 13 comments

Comments

@jainmohit2001
Copy link
Collaborator

jainmohit2001 commented Aug 28, 2024

Hi there.
I have been using the library for quite a while now.

I have recently come up with a small requirement: I want to log the OCPP stream to AWS Cloudwatch. After integrating with watchtower, things are working fine, and all the OCPP logs are logged in a single log stream.

I wanted to understand if there is any way I can create an individual stream for OCPP logs based on the ID of the charger.

Any insights or direction would be of great help.

@jainmohit2001
Copy link
Collaborator Author

I think this requirement that I am facing could be a part of a bigger feature where the ChargePoint class can accept a logger instance from the user. Allowing users to handle the logging of OCPP messages on a per-charger basis.

@jainmohit2001
Copy link
Collaborator Author

@OrangeTux, is the repo still maintained?

@jainmohit2001
Copy link
Collaborator Author

@OrangeTux @Jared-Newell-Mobility @proelke, let us know if the library is still being maintained.

@OrangeTux
Copy link
Collaborator

OrangeTux commented Sep 3, 2024

You're right. This library isn't really maintained anymore.

@OrangeTux
Copy link
Collaborator

@mdwcrft You have expressed interest to take over ownership of this project. Is that offer still up?

@jainmohit2001
Copy link
Collaborator Author

@OrangeTux , I am also willing to contribute and keep this library up to date.

@jainmohit2001
Copy link
Collaborator Author

Hey @OrangeTux, any chance I can get the opportunity to maintain this package?

@jainmohit2001
Copy link
Collaborator Author

@OrangeTux @Jared-Newell-Mobility @proelke, let me know if I can help you manage this package.

@OrangeTux
Copy link
Collaborator

Hello @jainmohit2001, thanks for your offer. Ideally, I'm looking for multiple people to take over ownership, so I'll announce publicly that this project looking for new owners.

@mdwcrft
Copy link
Collaborator

mdwcrft commented Sep 19, 2024

@mdwcrft You have expressed interest to take over ownership of this project. Is that offer still up?

Yep I'm happy to be added as a maintainer if there are others to make up for my rusty Python!

@OrangeTux
Copy link
Collaborator

Thanks @mdwcrft ! I'll move the discussion about maintainers to #662

@hhuseyinpay
Copy link

I wanted to let you know that I implemented this feature and submitted a pull request about four months ago: #641 The pull request is still awaiting review.

@jainmohit2001
Copy link
Collaborator Author

Duplicate of #641

@jainmohit2001 jainmohit2001 marked this as a duplicate of #641 Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

4 participants