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

If a service has non-string metadata, convoy-gluster wont work. #3012

Closed
cjellick opened this issue Dec 11, 2015 · 4 comments
Closed

If a service has non-string metadata, convoy-gluster wont work. #3012

cjellick opened this issue Dec 11, 2015 · 4 comments
Assignees
Labels
area/storage kind/bug Issues that are defects reported by users or that we know have reached a real release

Comments

@cjellick
Copy link

When deploying convoy-gluster, if any service in the environment has a metadata entry with a map as the value like:

{
  "elasticsearch": {
    "yml": {
      "cluster.name": "es",
      "node.data": "false",
      "node.master": "true",
      "node.name": "${HOSTNAME}"
    }
  },
  "io.rancher.service.hash": "6cf9dab7d556f3abbb3646b209a98d0794723867"
}

you'll get an errror like this in the convoy-gluster storagepool container:

12/11/2015 12:16:23 PMtime="2015-12-11T19:16:23Z" level=error msg="Error while reading host info [json: cannot unmarshal object into Go value of type string]"

So, steps to repro would be to deplo[y the elasticsearch catalog entry and then deploy convoy-gluster. Then look at the storagepool container logs.

The convoy-glsuter stack comes up fine, but the storage pool is never registered in rancher.

@cjellick cjellick added kind/bug Issues that are defects reported by users or that we know have reached a real release area/storage labels Dec 11, 2015
@cjellick cjellick self-assigned this Dec 11, 2015
@joostliketoast
Copy link

is there any workaround for now? I still have this error after removing the elasticsearch catalog entry

@cjellick
Copy link
Author

cjellick commented Jan 7, 2016

@joostliketoast no work around. i'll try to squeeze in a fix this week though

@cjellick
Copy link
Author

Fixed with the new release of convoy-gluster convoy-nfs

@sangeethah
Copy link
Contributor

Tested with convoy-gluster version - v0.2.

Deployed convoy-gluster
Deployed Elastic search template.
After this deployed convoy-gluster template.
Once service is activated fully ,convoy-gluster storage pool gets created successfully.
No errors seen in the convoy-gluster storagepool container.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/storage kind/bug Issues that are defects reported by users or that we know have reached a real release
Projects
None yet
Development

No branches or pull requests

4 participants