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

DTT2 - Testing module. Convert the track.yaml file into an object when the module starts #5296

Open
mhamra opened this issue Apr 25, 2024 · 0 comments

Comments

@mhamra
Copy link
Contributor

mhamra commented Apr 25, 2024

Target version Related issue Related PR/dev branch
4.9.0 #5229 PR #5264 4495-dtt1-release

Description

Many classes of the testing module, such as the HostInformation and Executor classes, rely on the inventory.yaml file to get information. This implementation makes the code complex. Accessing an in-memory structure will simplify the code and improve performance and reliability.

I suggest:

  • Implement an Inventory class that holds the information in the inventory.yaml.
  • Validate the existence of the inventory.yaml and create the Inventory instance.
  • All the functions that currently access the inventory file should be changed to access the Inventory instance.

Related PR comment: comment

@rauldpm rauldpm changed the title DTT1-Testing module. Convert the track.yaml file into an object when the module starts DTT2 - Testing module. Convert the track.yaml file into an object when the module starts Apr 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant