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

[patch][MSO-1505] Update fvt-optimizer pipeline to remove optimizer-manage-user step #995

Merged
merged 4 commits into from
May 29, 2024

Conversation

lgustavoibm
Copy link
Contributor

@lgustavoibm lgustavoibm commented May 27, 2024

Update Pipeline test from FVT Env to remove the creation of user in Selenium (fvt-optimizer-manage-user), the team of FVT has created a new script to to this step using pytest, so it's only necessary to run using that credentials.

andrefgj
andrefgj previously approved these changes May 27, 2024
Update fvt-optimizer.yml.j2
@andrefgj andrefgj self-requested a review May 27, 2024 21:27
andrefgj
andrefgj previously approved these changes May 27, 2024
Copy link
Contributor

@andrefgj andrefgj left a comment

Choose a reason for hiding this comment

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

It looks good to me

[patch] MSO-1505 Update fvt-optimizer.yml.j2 - Update Pipeline test from FVT Env to remove the creation of user in Selenium (fvt-optimizer-manage-user), the team of FVT has created a new script to to this step using pytest, so it's only necessary to run using that credentials.
@lgustavoibm lgustavoibm changed the title Update yml file from Optimizer removing user creation [patch] MSO-1505 Update fvt-optimizer.yml.j2 - Update yml file from Optimizer removing user creation May 27, 2024
@andrercm andrercm changed the title [patch] MSO-1505 Update fvt-optimizer.yml.j2 - Update yml file from Optimizer removing user creation [patch][MSO-1505] Update fvt-optimizer pipeline to remove optimizer-manage-user step May 27, 2024
Copy link
Contributor

@andrercm andrercm left a comment

Choose a reason for hiding this comment

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

looks good

@lgustavoibm lgustavoibm merged commit e300699 into master May 29, 2024
3 checks passed
@lgustavoibm lgustavoibm deleted the lglp1320 branch May 29, 2024 11:42
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.

4 participants