{"action":"created","issue":{"url":"https://api.github.com/repos/ManageIQ/manageiq/issues/4869","labels_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/4869/labels{/name}","comments_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/4869/comments","events_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/4869/events","html_url":"https://github.com/ManageIQ/manageiq/pull/4869","id":111460494,"number":4869,"title":"[WIP] [RFE] Validating VMWare Credentials on a UI Appliance","user":{"login":"agrare","id":12851112,"avatar_url":"https://avatars.githubusercontent.com/u/12851112?v=3","gravatar_id":"","url":"https://api.github.com/users/agrare","html_url":"https://github.com/agrare","followers_url":"https://api.github.com/users/agrare/followers","following_url":"https://api.github.com/users/agrare/following{/other_user}","gists_url":"https://api.github.com/users/agrare/gists{/gist_id}","starred_url":"https://api.github.com/users/agrare/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/agrare/subscriptions","organizations_url":"https://api.github.com/users/agrare/orgs","repos_url":"https://api.github.com/users/agrare/repos","events_url":"https://api.github.com/users/agrare/events{/privacy}","received_events_url":"https://api.github.com/users/agrare/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ManageIQ/manageiq/labels/bug","name":"bug","color":"fc2929"},{"url":"https://api.github.com/repos/ManageIQ/manageiq/labels/providers/vmware","name":"providers/vmware","color":"5319e7"},{"url":"https://api.github.com/repos/ManageIQ/manageiq/labels/wip","name":"wip","color":"eeeeee"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-10-14T18:27:05Z","updated_at":"2015-10-15T20:58:33Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ManageIQ/manageiq/pulls/4869","html_url":"https://github.com/ManageIQ/manageiq/pull/4869","diff_url":"https://github.com/ManageIQ/manageiq/pull/4869.diff","patch_url":"https://github.com/ManageIQ/manageiq/pull/4869.patch"},"body":"[RFE] Validating VMWare Credentials on a UI Appliance\r\nhttps://bugzilla.redhat.com/show_bug.cgi?id=1109425\r\n\r\nWhen validating credentials and adding a provider to a different zone, the UI appliance attempts to do the credential validation itself which is an issue if that appliance has no network connectivity to the provider that is being added.\r\n\r\nFix this by making credential validation a queued task that gets run on the appliance in the zone that is going to own that provider."},"comment":{"url":"https://api.github.com/repos/ManageIQ/manageiq/issues/comments/148519526","html_url":"https://github.com/ManageIQ/manageiq/pull/4869#issuecomment-148519526","issue_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/4869","id":148519526,"user":{"login":"miq-bot","id":7806257,"avatar_url":"https://avatars.githubusercontent.com/u/7806257?v=3","gravatar_id":"","url":"https://api.github.com/users/miq-bot","html_url":"https://github.com/miq-bot","followers_url":"https://api.github.com/users/miq-bot/followers","following_url":"https://api.github.com/users/miq-bot/following{/other_user}","gists_url":"https://api.github.com/users/miq-bot/gists{/gist_id}","starred_url":"https://api.github.com/users/miq-bot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/miq-bot/subscriptions","organizations_url":"https://api.github.com/users/miq-bot/orgs","repos_url":"https://api.github.com/users/miq-bot/repos","events_url":"https://api.github.com/users/miq-bot/events{/privacy}","received_events_url":"https://api.github.com/users/miq-bot/received_events","type":"User","site_admin":false},"created_at":"2015-10-15T20:58:33Z","updated_at":"2015-10-15T20:58:33Z","body":"<rubocop />Checked commits https://github.com/agrare/manageiq/compare/04a45f246fda328b0c8ea1d896588af500d3d7d4~...b87c92c5d6c73353dbcd4a4ea4e52106b6e3ae93 with ruby 1.9.3, rubocop 0.34.2, and haml-lint 0.13.0\n4 files checked, 10 offenses detected\n\n**app/controllers/ems_common.rb**\n- [ ] :large_orange_diamond: - [Line 874](https://github.com/agrare/manageiq/blob/b87c92c5d6c73353dbcd4a4ea4e52106b6e3ae93/app/controllers/ems_common.rb#L874), Col 7 - [Style/AccessorMethodName](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AccessorMethodName) - Do not prefix writer method names with `set_`.\n- [ ] :small_blue_diamond: - [Line 434](https://github.com/agrare/manageiq/blob/b87c92c5d6c73353dbcd4a4ea4e52106b6e3ae93/app/controllers/ems_common.rb#L434), Col 3 - [Metrics/AbcSize](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Metrics/AbcSize) - Assignment Branch Condition size for queue_credential_validation_task is too high. [37.48/15]\n- [ ] :small_blue_diamond: - [Line 434](https://github.com/agrare/manageiq/blob/b87c92c5d6c73353dbcd4a4ea4e52106b6e3ae93/app/controllers/ems_common.rb#L434), Col 3 - [Metrics/CyclomaticComplexity](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Metrics/CyclomaticComplexity) - Cyclomatic complexity for queue_credential_validation_task is too high. [7/6]\n- [ ] :small_blue_diamond: - [Line 434](https://github.com/agrare/manageiq/blob/b87c92c5d6c73353dbcd4a4ea4e52106b6e3ae93/app/controllers/ems_common.rb#L434), Col 3 - [Metrics/MethodLength](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Metrics/MethodLength) - Method has too many lines. [37/25]\n- [ ] :small_blue_diamond: - [Line 434](https://github.com/agrare/manageiq/blob/b87c92c5d6c73353dbcd4a4ea4e52106b6e3ae93/app/controllers/ems_common.rb#L434), Col 3 - [Metrics/PerceivedComplexity](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Metrics/PerceivedComplexity) - Perceived complexity for queue_credential_validation_task is too high. [9/7]\n- [ ] :small_blue_diamond: - [Line 874](https://github.com/agrare/manageiq/blob/b87c92c5d6c73353dbcd4a4ea4e52106b6e3ae93/app/controllers/ems_common.rb#L874), Col 3 - [Metrics/AbcSize](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Metrics/AbcSize) - Assignment Branch Condition size for set_record_creds is too high. [46.88/15]\n- [ ] :small_blue_diamond: - [Line 874](https://github.com/agrare/manageiq/blob/b87c92c5d6c73353dbcd4a4ea4e52106b6e3ae93/app/controllers/ems_common.rb#L874), Col 3 - [Metrics/CyclomaticComplexity](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Metrics/CyclomaticComplexity) - Cyclomatic complexity for set_record_creds is too high. [10/6]\n- [ ] :small_blue_diamond: - [Line 874](https://github.com/agrare/manageiq/blob/b87c92c5d6c73353dbcd4a4ea4e52106b6e3ae93/app/controllers/ems_common.rb#L874), Col 3 - [Metrics/MethodLength](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Metrics/MethodLength) - Method has too many lines. [30/25]\n- [ ] :small_blue_diamond: - [Line 874](https://github.com/agrare/manageiq/blob/b87c92c5d6c73353dbcd4a4ea4e52106b6e3ae93/app/controllers/ems_common.rb#L874), Col 3 - [Metrics/PerceivedComplexity](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Metrics/PerceivedComplexity) - Perceived complexity for set_record_creds is too high. [10/7]\n\n**app/models/manageiq/providers/infra_manager.rb**\n- [ ] :large_orange_diamond: - [Line 47](https://github.com/agrare/manageiq/blob/b87c92c5d6c73353dbcd4a4ea4e52106b6e3ae93/app/models/manageiq/providers/infra_manager.rb#L47), Col 1 - [Style/EmptyLinesAroundClassBody](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/EmptyLinesAroundClassBody) - Extra empty line detected at class body end.\n"}}