Make config python script detect hash collisions

Open
#545 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
3/5
Estimated time
1-2 days
Newbie friendliness
72/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Active
Tech stack
python
Domain
tooling

Research direction

Start in scripts/config_mapping/lifecycle_config.py and review PR 527 for the portable run-target hashing function. Implement the equivalent hashing and collision validation in the config-validation script. Done means the script detects duplicate hashes, communicates the collision, and exits with status 1.

Written by the indexing model from the issue text.

Description

bug
What

Since PR https://github.com/eclipse-score/lifecycle/pull/527 we have a portable hashing function to get the hash of the run-targets. Currently we assume that no hash collisions happen which is a bit unsafe. But with the changes we could validate that no hash collisions happen in the python script that is ran to validate configs lifecycle_config.py by implementing the same hashing function in python.
I think for now it shall just exit and communicate the error, I don't think we should append or modify the name since we might get into weird behaviour.

Acceptance Criteria (DoD)
  • Script validates names and exits 1 if a hash collision is detected.
How

No response

Dominant language
C++
Stars
6
Forks
34
Avg merge
1d 23h
Merged PRs (30d)
83

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from eclipse-score/lifecycle

All issues in eclipse-score/lifecycle

Similar issues

More C++ issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.