-
Notifications
You must be signed in to change notification settings - Fork 22
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
Create external csms demo #109
base: main
Are you sure you want to change the base?
Conversation
merge upstream
demo follows demo-iso15118-2-ocpp-201.sh except: checks for url in environment, if not configured, exits tested by: setting url as patch would, working normally, removed, failed with error message Signed-off-by: Abby Wheelis <[email protected]>
include instructions to set csms url Signed-off-by: Abby Wheelis <[email protected]>
Signed-off-by: Abby Wheelis <[email protected]>
clarify direction and give and example Signed-off-by: Abby Wheelis <[email protected]>
make it more clear what is code, and what is prose Signed-off-by: Abby Wheelis <[email protected]>
Signed-off-by: Abby Wheelis <[email protected]>
Testing done: had regular 1-line demo up, ran before setting url then after
saw log error before setting, after setting seemed to work. Able to run manager and charge in UI just as expected! |
Wanted to test with something else ... maeve requires me to work out the certificate situation ... I don't think I can chase that down tonight, but I am reasonably confident that this script does what we think it does because I was able to set the url to the working maeve container and see an error when it was not set - so the environment variable does what we want it to do. If I manage to find some time in the morning I'll work through as much of the certificates as I can. |
maeve failed:
so did citrene:
|
Quick test w/ Cintrene by calling our docker compose, then exporting the same url as the patch: Getting issues with the nodered container in EVerest which won't start, so wiping and trying again |
Error:
It looks like citrene and everest are trying to use the same port? In any case, this is looking like a fail for now, running the manager and OCPP is not connecting to the client:
|
Trying MobilityHouse with the EVerest implementation (https://github.com/EVerest/ocpp-csms) and it is also erroring out, complaining about some dependencies:
|
This is because ocpp (directly from MobilityHouse) is a requirement of the EVerest scripts, but not by a tagged version ... so they're out of sync |
Signed-off-by: Abby Wheelis <[email protected]>
In alignment with #101 (comment), creating a quick script to highlight to CSMS's that they can test with EVerest!