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

Added export/import of SmartState Analysis Profiles #17427

Merged
merged 1 commit into from
May 18, 2018

Conversation

juliancheal
Copy link
Member

@juliancheal juliancheal commented May 16, 2018

@juliancheal
Copy link
Member Author

@miq-bot assign @gtanzillo

@juliancheal juliancheal changed the title Added export/import of SmartState Analysis Profiles [WIP] Added export/import of SmartState Analysis Profiles May 17, 2018
@miq-bot miq-bot added the wip label May 17, 2018
@juliancheal
Copy link
Member Author

@jerryk55 Hey Jerry, does this like correct to you? Is there anything you would expect in a Scan Profile, export / import.

@jerryk55
Copy link
Member

@juliancheal looks good except for the error handling on the exception during the import - commented out puts right now. The actual import/export from/to yaml seems fine.

@juliancheal
Copy link
Member Author

@jerryk55 Thanks :)

@juliancheal juliancheal changed the title [WIP] Added export/import of SmartState Analysis Profiles Added export/import of SmartState Analysis Profiles May 18, 2018
@miq-bot miq-bot removed the wip label May 18, 2018
@juliancheal juliancheal force-pushed the export_analysis_profiles branch 3 times, most recently from ab4105c to 667c212 Compare May 18, 2018 08:32
@miq-bot
Copy link
Member

miq-bot commented May 18, 2018

Checked commit juliancheal@207fcaa with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
3 files checked, 1 offense detected

lib/task_helpers/imports/scan_profiles.rb

@juliancheal
Copy link
Member Author

@gtanzillo merge? We're all 🍏🛶🥝🌱

Copy link
Member

@gtanzillo gtanzillo left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@gtanzillo gtanzillo added this to the Sprint 86 Ending May 21, 2018 milestone May 18, 2018
@gtanzillo gtanzillo merged commit 6480221 into ManageIQ:master May 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants