changes to fix the migration UI pipeline #6908

vinay033 · 2022-04-13T14:17:43Z

Signed-off-by: Vinay Sharma vsharma@chef.io

🔩 Description: What code changed, and why?

The Infra chef-server migration process is not working properly, need to fix the below points

Org list is not loaded after refreshing or navigating the server details page
After confirming migration org list not showing
After uploading the backup zip file migration steps are not showing properly
On the server details page migration status is not showing properly
After canceling the migration, the status is showing failed instead of canceled.
Fixed for username conflict check
restricted confirm migration according to different conditions.

⛓️ Related Resources

https://chefio.atlassian.net/browse/STALWART-95
https://chefio.atlassian.net/browse/STALWART-102
https://chefio.atlassian.net/browse/STALWART-118
https://chefio.atlassian.net/browse/STALWART-112

👍 Definition of Done

I have added some changes to fix the above issues.

👟 How to Build and Test the Change

✅ Checklist

All PRs must tick these:

I have read the CONTRIBUTING document.
All commits signed-off for the Developer Certification of Origin.

With occasional exceptions, all PRs from Progress employees must tick these:

Is the code clear? (complicated code or lots of comments--subdivide and use well-named methods, meaningful variable names, etc.)
Consistency checked? (user notifications, user prompts, visual patterns, code patterns, variable names)
Repeated code blocks eliminated? (adapt and reuse existing components, blocks, functions, etc.)
Spelling, grammar, typos checked? (at a minimum use make spell in any component directory)
Code well-formatted? (indents, line breaks, etc. improve rather than hinder readability)

All PRs from Progress employees should tick these if appropriate:

Tests added/updated? (all new code needs new tests)
Docs added/updated? (all customer-facing changes)

Please add a note next to any checkbox above if you are NOT ticking it.

📷 Screenshots, if applicable

netlify · 2022-04-13T14:17:48Z

👷 Deploy Preview for chef-automate processing.

Name	Link
🔨 Latest commit	`aae0027`
🔍 Latest deploy log	https://app.netlify.com/sites/chef-automate/deploys/6272264e0abcea0008978632

chaitali-mane

migration code fixed

Signed-off-by: Abdul-Az <aazeez@progress.com>

* Sync status Signed-off-by: Pappu Kumar <pappu.kumar@progress.com> * Sync migration time Signed-off-by: Pappu Kumar <pappu.kumar@progress.com>

Signed-off-by: Vinay Sharma <vsharma@chef.io>

sonarcloud · 2022-05-04T07:22:05Z

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
13 Code Smells

No Coverage information
0.0% Duplication

* policyfiles actions added (#6081) Signed-off-by: Abdul-Az <aazeez@progress.com> * added policygroups actions (#6123) Signed-off-by: Abdul-Az <aazeez@progress.com> * Automate API needs to handle sync status (#6860) * Sync status Signed-off-by: Pappu Kumar <pappu.kumar@progress.com> * Sync migration time Signed-off-by: Pappu Kumar <pappu.kumar@progress.com> * changes to fix the migration UI pipeline Signed-off-by: Vinay Sharma <vsharma@chef.io> * added some minor changes Signed-off-by: Vinay Sharma <vsharma@chef.io> * added some changes for conflicts Signed-off-by: Vinay Sharma <vsharma@chef.io> * added some minor changes Signed-off-by: Vinay Sharma <vsharma@chef.io> * added chanegs to fix the migration status Signed-off-by: Vinay Sharma <vsharma@chef.io> * added some changes for code optimization Signed-off-by: Vinay Sharma <vsharma@chef.io> * added some minor changes Signed-off-by: Vinay Sharma <vsharma@chef.io> * added some minor changes Signed-off-by: Vinay Sharma <vsharma@chef.io> * added changes to fix the migration issue + check user conflicts Signed-off-by: Vinay Sharma <vsharma@chef.io> * added proto changes Signed-off-by: Vinay Sharma <vsharma@chef.io> * added temp changes Signed-off-by: Vinay Sharma <vsharma@chef.io> * updated changes for migration process Signed-off-by: Vinay Sharma <vsharma@chef.io> * added changes to fix the user migration Signed-off-by: Vinay Sharma <vsharma@chef.io> Co-authored-by: Abdul Azeez <aazeez@progress.com> Co-authored-by: Dave Augustus <95467821+dave-augustus@users.noreply.github.com>

vinay033 added automate-ui infra-user-mgnt labels Apr 13, 2022

vinay033 self-assigned this Apr 13, 2022

vinay033 requested review from chaitali-mane and kalroy April 13, 2022 14:18

vinay033 force-pushed the Vinay/migration_ui_pipeline_fixes branch from 5de05ad to 39fe925 Compare April 14, 2022 17:17

vinay033 force-pushed the feature/infra_user_management branch from aa344f9 to 9594f88 Compare April 18, 2022 12:18

vinay033 requested a review from a team as a code owner April 18, 2022 12:18

vinay033 force-pushed the Vinay/migration_ui_pipeline_fixes branch 2 times, most recently from a00a235 to 5d30566 Compare April 19, 2022 10:32

chaitali-mane approved these changes Apr 19, 2022

View reviewed changes

vinay033 force-pushed the feature/infra_user_management branch from 7910e16 to dc2dbd0 Compare April 19, 2022 19:27

vinay033 force-pushed the Vinay/migration_ui_pipeline_fixes branch from 5d30566 to 5bc4287 Compare April 19, 2022 19:28

vinay033 force-pushed the feature/infra_user_management branch from 7e39a1d to 2f38f15 Compare April 22, 2022 09:53

vinay033 force-pushed the Vinay/migration_ui_pipeline_fixes branch 2 times, most recently from 473107f to fbc04e9 Compare April 28, 2022 00:10

vinay033 requested a review from chaitali-mane April 28, 2022 00:15

kalroy force-pushed the feature/infra_user_management branch from 073be91 to ef1e46a Compare April 28, 2022 08:21

vinay033 force-pushed the feature/infra_user_management branch from a8cdc10 to 2f89bbe Compare May 2, 2022 10:10

iamazzeez and others added 10 commits May 2, 2022 23:46

policyfiles actions added (#6081)

7d775bf

Signed-off-by: Abdul-Az <aazeez@progress.com>

added policygroups actions (#6123)

7eff559

Signed-off-by: Abdul-Az <aazeez@progress.com>

Automate API needs to handle sync status (#6860)

b745476

* Sync status Signed-off-by: Pappu Kumar <pappu.kumar@progress.com> * Sync migration time Signed-off-by: Pappu Kumar <pappu.kumar@progress.com>

changes to fix the migration UI pipeline

c120d43

Signed-off-by: Vinay Sharma <vsharma@chef.io>

added some minor changes

ae81601

Signed-off-by: Vinay Sharma <vsharma@chef.io>

added some changes for conflicts

f024ac8

Signed-off-by: Vinay Sharma <vsharma@chef.io>

added some minor changes

31d03b7

Signed-off-by: Vinay Sharma <vsharma@chef.io>

added chanegs to fix the migration status

a3f7d7b

Signed-off-by: Vinay Sharma <vsharma@chef.io>

added some changes for code optimization

180d297

Signed-off-by: Vinay Sharma <vsharma@chef.io>

added some minor changes

b2d44c2

Signed-off-by: Vinay Sharma <vsharma@chef.io>

vinay033 added 5 commits May 2, 2022 23:46

added some minor changes

f82a26d

Signed-off-by: Vinay Sharma <vsharma@chef.io>

added changes to fix the migration issue + check user conflicts

a4283c6

Signed-off-by: Vinay Sharma <vsharma@chef.io>

added proto changes

20aebac

Signed-off-by: Vinay Sharma <vsharma@chef.io>

added temp changes

f9c77eb

Signed-off-by: Vinay Sharma <vsharma@chef.io>

updated changes for migration process

0f0d364

Signed-off-by: Vinay Sharma <vsharma@chef.io>

vinay033 force-pushed the Vinay/migration_ui_pipeline_fixes branch from fbc04e9 to 0f0d364 Compare May 3, 2022 19:25

added changes to fix the user migration

aae0027

Signed-off-by: Vinay Sharma <vsharma@chef.io>

GorillaGigabytes approved these changes May 4, 2022

View reviewed changes

vinay033 merged commit 9c61f07 into feature/infra_user_management May 4, 2022

vinay033 deleted the Vinay/migration_ui_pipeline_fixes branch May 4, 2022 11:21

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

changes to fix the migration UI pipeline #6908

changes to fix the migration UI pipeline #6908

vinay033 commented Apr 13, 2022 •

edited

Loading

netlify bot commented Apr 13, 2022 •

edited

Loading

chaitali-mane left a comment

sonarcloud bot commented May 4, 2022

changes to fix the migration UI pipeline #6908

changes to fix the migration UI pipeline #6908

Conversation

vinay033 commented Apr 13, 2022 • edited Loading

🔩 Description: What code changed, and why?

⛓️ Related Resources

👍 Definition of Done

👟 How to Build and Test the Change

✅ Checklist

📷 Screenshots, if applicable

netlify bot commented Apr 13, 2022 • edited Loading

👷 Deploy Preview for chef-automate processing.

chaitali-mane left a comment

Choose a reason for hiding this comment

sonarcloud bot commented May 4, 2022

vinay033 commented Apr 13, 2022 •

edited

Loading

netlify bot commented Apr 13, 2022 •

edited

Loading