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

Export all TMT_ROLE_* envvars to all phases #2044

Closed
Tracked by #2070
happz opened this issue May 2, 2023 · 1 comment
Closed
Tracked by #2070

Export all TMT_ROLE_* envvars to all phases #2044

happz opened this issue May 2, 2023 · 1 comment
Labels
Milestone

Comments

@happz
Copy link
Collaborator

happz commented May 2, 2023

A follow-up for #1790: patch constructs and exposes TMT_ROLE_* when running a phase. But:

  • queue task may see only a subset of all guests, therefore envvars might not list all existing guests for a given role, or miss some roles completely.
  • there is a similar functionality in multihost plugin, but that one depends on multihost metadata, therefore does not consume the role and name of provisioned guests.

Neither of these two blocks involved in the TMT_ROLE_* generation seems to be good enough, we need a proper code to expose all known roles and all their corresponding guests.

There are also envvars for all guests (e.g. TMT_GUESTS) and all guests in the current phase.

@happz
Copy link
Collaborator Author

happz commented Jun 7, 2023

Implemented in #2072

@happz happz added this to the 1.24 milestone Jun 7, 2023
@happz happz closed this as completed Jun 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant