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

cnf_manager: Refactor sample_setup #2050

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kosstennbl
Copy link
Collaborator

Description

Change instalation time measurement method to reduce complexity Extract waiting for all resources and liveness_time calculation Extract elapsed time config map creation
Add comments as markers for different stages of sample setup

Markers are to be removed once method is simplified enough.

Issues:

Refs: #2036

How has this been tested:

Successfully built

Types of changes:

Refactor

Checklist:

Documentation
No documentation updates required.

@kosstennbl
Copy link
Collaborator Author

In my opinion - changes are simple enough to be coupled together like this.
Further changes will come as this will be merged.

@martin-mat martin-mat self-requested a review June 10, 2024 19:18
Copy link
Collaborator

@martin-mat martin-mat left a comment

Choose a reason for hiding this comment

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

I don't think that this one should be merged. A real redactoring should be done. Until this is ready, marking this as Draft.

@martin-mat martin-mat marked this pull request as draft June 10, 2024 19:23
Change instalation time measurement method to reduce complexity
Extract waiting for all resources and liveness_time calculation
Extract elapsed time config map creation
Add comments as markers for different stages of sample setup

Refs: cnti-testcatalog#2036
Signed-off-by: Konstantin Yarovoy <konstantin.yarovoy@tietoevry.com>
@kosstennbl
Copy link
Collaborator Author

Added comment to the referenced issue.
I consider that changes in this PR are useful and can be considered as "low-hanging fruit" in topic of refactoring the setup process.
For full refactor - multiple project-wide problems (check #2036) should be solved first, only then it seems sensible to me to fix this function.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants