Skip to content

Conversation

@AndreMarcel99
Copy link
Collaborator

@AndreMarcel99 AndreMarcel99 commented Oct 10, 2025

SUMMARY

Fixes new adoption of gdgs utilities from ZOAU 1.4.0

Fixes #2208

ISSUE TYPE
  • Enabler Pull Request
ADDITIONAL INFORMATION
Screenshot 2025-10-20 at 3 47 57 p m Screenshot 2025-10-20 at 3 48 13 p m

@AndreMarcel99 AndreMarcel99 changed the title Enabler/2208/adapat generation data group view generations [Enabler][2208]Adopt_generation_data_group_generations Oct 10, 2025
richp405
richp405 previously approved these changes Oct 21, 2025
Copy link
Collaborator

@fernandofloresg fernandofloresg left a comment

Choose a reason for hiding this comment

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

Requested a small change and asked a question

@@ -0,0 +1,13 @@
minor_changes:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Unfortunately, this time this will have to be trivial this time since we will announce 1.4.0 in one announcement line, so no need to show this to the general public.

generations = gdg.generations()
gds = generations[rel_generation - 1]
generations = gdg.generations
# On 1.4 zoau version if you give 0 or -1 go to proper generation
Copy link
Collaborator

Choose a reason for hiding this comment

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

This comment is confusing, could you rephrase it or remove it?

return str(datasets.create(dataset, type).name), "shr"
else:
return str(datasets.create(dataset, "seq").name), "shr"
return dataset, "new"
Copy link
Collaborator

Choose a reason for hiding this comment

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

How this change will affect current behavior? if user specifies data set type as "PDS" will this default it to "seq" or other type?

Copy link
Collaborator

Choose a reason for hiding this comment

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

or is the type being considered when executing mvs_cmd? I couldn't find the line where type is considered

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

For the new approach of gdg will take the base of the gdg set on the creation.

richp405
richp405 previously approved these changes Oct 22, 2025
@AndreMarcel99
Copy link
Collaborator Author

Screenshot 2025-10-23 at 2 45 16 p m Screenshot 2025-10-23 at 2 45 28 p m

Copy link
Collaborator

@fernandofloresg fernandofloresg left a comment

Choose a reason for hiding this comment

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

LGTM Thanks for addressing my comments and answering my questions

Copy link
Collaborator

@mayankmani-sde mayankmani-sde left a comment

Choose a reason for hiding this comment

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

LGTM!

@fernandofloresg fernandofloresg added the Do not Merge When a pull request should not be merged for issue noted reasons label Oct 28, 2025
@fernandofloresg
Copy link
Collaborator

Added no merge label, waiting for SPS pipeline screenshots

@AndreMarcel99
Copy link
Collaborator Author

Screenshot 2025-10-28 at 2 57 06 p m

@fernandofloresg fernandofloresg merged commit c1d42bf into dev Oct 28, 2025
19 checks passed
@fernandofloresg fernandofloresg deleted the enabler/2208/adapat_GenerationDataGroupView_generations branch October 28, 2025 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Do not Merge When a pull request should not be merged for issue noted reasons

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants