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

Ocpp: reduce number of configurable timeouts #15842

Merged
merged 10 commits into from
Sep 5, 2024
Merged

Ocpp: reduce number of configurable timeouts #15842

merged 10 commits into from
Sep 5, 2024

Conversation

andig
Copy link
Member

@andig andig commented Sep 1, 2024

TODO

  • fix tests

@andig andig added the enhancement New feature or request label Sep 1, 2024
@@ -48,8 +47,8 @@ func NewOCPPFromConfig(other map[string]interface{}) (api.Charger, error) {
MeterInterval time.Duration
MeterValues string
ConnectTimeout time.Duration // Initial Timeout
Copy link
Member

Choose a reason for hiding this comment

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

Das hier sollten wir wohl auch besser umbenennen, da dies mit dem festen OCPP-Begriff ConnectionTimeOut Verwechslungsgefahr birgt aber etwas ganz anderes meint:

Interval *from beginning of status: 'Preparing' until incipient Transaction is automatically canceled, due to failure of EV driver to (correctly) insert the charging cable connector(s) into the appropriate socket(s). The Charge Point SHALL go back to the original state, probably: 'Available'.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ich möchte zu zwei existierenden Timeouts jetzt keinen weiteren einführen der noch neue Verwirrung stiftet. Immer dran denken: dass müssen die Anwender alles nachvollziehen...

Copy link
Member

Choose a reason for hiding this comment

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

Das will ich ja auch nicht.
Es ging nur um die Namensgebung, die speziell im OCPP-Umfeld halt etwas verwirrend sein könnte.

charger/ocpp/const.go Outdated Show resolved Hide resolved
util/templates/defaults.yaml Show resolved Hide resolved
@premultiply
Copy link
Member

Ich glaub in ocpp-go hab ich auch schon mal entsprechende Konstanten gesichtet.
Könnte man die auch nutzen?

Co-authored-by: premultiply <4681172+premultiply@users.noreply.github.com>
@andig
Copy link
Member Author

andig commented Sep 1, 2024

Könnte man die auch nutzen?

Findest Du welche das ist? Konnte auf die Schnelle nix entdecken.

@premultiply
Copy link
Member

Hatte das wahrscheinlich verwechselt mit dem hier:

const (
messageExpiry = 30 * time.Second
transactionExpiry = time.Hour
)

@andig
Copy link
Member Author

andig commented Sep 5, 2024

Ich habe message/ meter/ transaction expiry jetzt auch noch weg geworfen. Warum wir auf 1h alte Transaktionen reagieren sollten ist nicht so richtig klar. Wenn die Uhren auseinander laufen haben wir ohnehin schon andere Probleme.

@andig andig merged commit 04c5429 into master Sep 5, 2024
6 checks passed
@andig andig deleted the chore/ocpp-timeout branch September 5, 2024 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants