-
Notifications
You must be signed in to change notification settings - Fork 294
Getting started guide fails? #1118
Comments
Hello @Lorquas, could you start from the beginning and do the following steps once again as below:
In another terminal window in snap directory:
When I execute these commands above I don't receive any error:
However, your issue might point that there is some bug which occures under some circumstances. |
Here is the full console logs of running this: Note that I am using both snap and plugins from the latest release here: https://github.com/intelsdi-x/snap/releases/tag/v0.15.0-beta and not from the git repo directly. |
@Lorquas This could be because we renamed the included file publishing plugin in this PR. The current getting started guide is using As for the documentation, @mjbrender is there already any ongoing tasks to update the documentation or should I create a new issue? |
@thomastaylor312 yup, I've tried both the |
Duplicating selected lines from snapd logs for convenience:
|
@Lorquas I'm trying to reproduce the error as you are seeing it.
|
@jcooklin I'm running on Linux. Doing more testing it seems to not work on Fedora 23 or 24, but does seem to work on an older RHEL 7.2 release. I'm downloading and running the binaries/plugins from: |
Sets the current time as the advertised time in the returned metric when using grpc.
Sets the current time as the advertised time in the returned metric when using grpc.
@Lorquas A PR has been submitted to fix this issue. You can download the binaries built with the fix here. Will you test the fix in your environment? (I was able to reproduce the error you reported on Ubuntu 16.04 but not on 14.04 or 12.04.) The issue was due to the fact that the last advertised time field was not getting set on the metrics returned to the framework by the mock1 plugin and on newer OS's when the default (empty) value came through it had a fractional time zone offset which, as it turns out, gob serialization does not like ( |
@jcooklin this is strange, I'd avoid resolving this by setting metric time to Now() especially during conversion. If time.Time is empty it's .Location() should be &UTC https://golang.org/src/time/time.go?s=23385:24735#L810, and marshalling should recognize that. Perhaps it was set previously using time.Unix() and when marshalled again but not using grpc but gob it crashes. |
@jcooklin I've tested your build on all of my systems and things are working. I think this issue can be resolved with your PR and the documentation tweaks as @thomastaylor312 suggested. Thanks everyone for the quick replies! |
This issue was first reported in intelsdi-x#1118 but didn't address the TimeStamp. This commit does.
I'm running through the getting started guide, found in this repo's README.md and off your vimeo page. I'm going along and everything is peachy until:
I open up the yaml and check out that its looking for a publisher named "mock-file" and not "file", run
snapctl plugin unload publisher:file:1
followed bysnapctl plugin load plugins/snap-plugin-publisher-mock-file
. I then run the task create again:seems better, but now:
and in the console output of snapd:
Until the fail limit of 10 consecutive fails is reached and it aborts completely.
I've retried the whole process using the plain "file" plugin (and rewriting the yaml), I've also changed the target file to a file/folder with 777 permissions, selinux and firewalls off. And I get the same thing every time. Being as new to this framework as I am (following the getting started guide), any ideas as to what could be wrong?
The text was updated successfully, but these errors were encountered: