Skip to content
This repository has been archived by the owner on Nov 15, 2019. It is now read-only.

Middleware: Add Datasource - several issues #2

Closed
miq-bot opened this issue Apr 7, 2017 · 5 comments
Closed

Middleware: Add Datasource - several issues #2

miq-bot opened this issue Apr 7, 2017 · 5 comments

Comments

@miq-bot
Copy link
Member

miq-bot commented Apr 7, 2017

There are several issues while adding new Datasource on EAP7 server via MiQ.

  1. In "Specify Driver" step of adding "Postgres" Datasource, there is a typo in Name field's default value: "postresql". This causes not recognizing driver.
  2. While creating Postgres Datasource and choosing already created JDBC Driver, it fails to create, as it uses default name "PostgresDS" no matter what name you type in input. Says "Service org.wildfly.data-source.PostgresDS is already registered" in server logs if PostgresDS already exists.

@pilhuhn


This issue was moved to this repository from ManageIQ/manageiq#13740, originally opened by @hhovsepy

@mtho11
Copy link
Contributor

mtho11 commented Apr 7, 2017

Issue 1 should already be fixed 2 weeks ago by: ManageIQ/manageiq-ui-classic#807

Issue 2 is currently under development via: https://bugzilla.redhat.com/show_bug.cgi?id=1427141

@abonas
Copy link
Member

abonas commented Apr 9, 2017

@mtho11 is the second issue fixed by ManageIQ/manageiq-ui-classic#941?
if yes, then this GH issue should be moved to the classic ui repo as there's no affected code in this repo

@mtho11
Copy link
Contributor

mtho11 commented Apr 12, 2017

@abonas yes, you are correct, the second issue is fixed by: ManageIQ/manageiq-ui-classic#941 and already covered by the previously mentioned BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1427141. I don't think it is necessary to duplicate BZs with github issues.

@abonas
Copy link
Member

abonas commented Apr 13, 2017

I don't think it is necessary to duplicate BZs with github issues.

the duplication was already there initially, this is not a newly opened issue, I am just moving the issue to be on the correct repo on github

@mtho11
Copy link
Contributor

mtho11 commented Sep 1, 2017

This was fixed quite a while ago.

@mtho11 mtho11 closed this as completed Sep 1, 2017
lucasponce pushed a commit to lucasponce/manageiq-providers-hawkular that referenced this issue Dec 1, 2017
Fix rubocop warnings and stop using inventory collection secondary refs
lucasponce pushed a commit to lucasponce/manageiq-providers-hawkular that referenced this issue Dec 1, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants