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

V1.4.0 Failed to parse /etc/grafana/grafana.ini, open /etc/grafana/grafana.ini: no such file or directory #1709

Closed
Robbilie opened this Issue Jun 30, 2017 · 29 comments

Comments

Projects
None yet
@Robbilie

Robbilie commented Jun 30, 2017

Failed to parse /etc/grafana/grafana.ini, open /etc/grafana/grafana.ini: no such file or directory

there is no such file in there since the config dir is in /usr/share/grafana
theres at least a defaults file…

@andyxning

This comment has been minimized.

Show comment
Hide comment
@andyxning

andyxning Jul 1, 2017

Member

@Robbilie there is a PR will add a default and empty grafana.ini if not exist already. See this diff for more info.

BTW, you can mount a configuration file into /etc/grafana/grafana.ini at runtime.

Member

andyxning commented Jul 1, 2017

@Robbilie there is a PR will add a default and empty grafana.ini if not exist already. See this diff for more info.

BTW, you can mount a configuration file into /etc/grafana/grafana.ini at runtime.

@Robbilie

This comment has been minimized.

Show comment
Hide comment
@Robbilie

Robbilie Jul 1, 2017

i know that i can mount a config file into it but i would rather not.
i have no idea why the config file is missing and this was really annoying when i tried to upgrade to 1.4.

any eta when a release with the fix is expected?

Robbilie commented Jul 1, 2017

i know that i can mount a config file into it but i would rather not.
i have no idea why the config file is missing and this was really annoying when i tried to upgrade to 1.4.

any eta when a release with the fix is expected?

@andyxning

This comment has been minimized.

Show comment
Hide comment
@andyxning

andyxning Jul 2, 2017

Member

@Robbilie The fix can only add an empty grafana.ini which may not be what you want. The desired way is to mount a customized grafana.ini. :)

Member

andyxning commented Jul 2, 2017

@Robbilie The fix can only add an empty grafana.ini which may not be what you want. The desired way is to mount a customized grafana.ini. :)

@Robbilie

This comment has been minimized.

Show comment
Hide comment
@Robbilie

Robbilie Jul 2, 2017

thats not the desired way. the official grafana and previous versions of heapsters grafana had a grafana.ini and i dont see why the 1.4.0 release shouldnt have one

Robbilie commented Jul 2, 2017

thats not the desired way. the official grafana and previous versions of heapsters grafana had a grafana.ini and i dont see why the 1.4.0 release shouldnt have one

@andyxning

This comment has been minimized.

Show comment
Hide comment
@andyxning

andyxning Jul 2, 2017

Member

@Robbilie Having not know that we have a grafana.ini in previous release. Will check this later.

Member

andyxning commented Jul 2, 2017

@Robbilie Having not know that we have a grafana.ini in previous release. Will check this later.

@Robbilie

This comment has been minimized.

Show comment
Hide comment
@Robbilie

Robbilie Jul 2, 2017

lol theres literally no reason to not have one, 1.3 aswell as official grafana releases had one

Robbilie commented Jul 2, 2017

lol theres literally no reason to not have one, 1.3 aswell as official grafana releases had one

@andyxning

This comment has been minimized.

Show comment
Hide comment
@andyxning

andyxning Jul 4, 2017

Member

@Robbilie You mean gcr.io/google_containers//heapster-grafana-amd64:v4.0.2

Member

andyxning commented Jul 4, 2017

@Robbilie You mean gcr.io/google_containers//heapster-grafana-amd64:v4.0.2

@Robbilie

This comment has been minimized.

Show comment
Hide comment
@Robbilie

Robbilie Jul 4, 2017

@andyxning with one less slash yeah, thats the previous one which had the ini, https://hub.docker.com/r/grafana/grafana/ has the ini aswell

Robbilie commented Jul 4, 2017

@andyxning with one less slash yeah, thats the previous one which had the ini, https://hub.docker.com/r/grafana/grafana/ has the ini aswell

@trunet

This comment has been minimized.

Show comment
Hide comment
@trunet

trunet Jul 5, 2017

any workaround? trying to deploy a brand new heapster and having problems

trunet commented Jul 5, 2017

any workaround? trying to deploy a brand new heapster and having problems

@Robbilie

This comment has been minimized.

Show comment
Hide comment
@Robbilie

Robbilie Jul 5, 2017

mount the file into the pod…
although this should imo be fixed ASAP in a 4.2.1 release bc this is just annoying and breaking…

Robbilie commented Jul 5, 2017

mount the file into the pod…
although this should imo be fixed ASAP in a 4.2.1 release bc this is just annoying and breaking…

@DirectXMan12

This comment has been minimized.

Show comment
Hide comment
@DirectXMan12

DirectXMan12 Jul 5, 2017

Contributor

I believe @piosz is working on it, and it's marked as high priority. We should have a fix soon.

Contributor

DirectXMan12 commented Jul 5, 2017

I believe @piosz is working on it, and it's marked as high priority. We should have a fix soon.

@loburm

This comment has been minimized.

Show comment
Hide comment
@loburm

loburm Jul 6, 2017

Contributor

/assign @loburm

Contributor

loburm commented Jul 6, 2017

/assign @loburm

@doubaokun

This comment has been minimized.

Show comment
Hide comment
@doubaokun

doubaokun Jul 6, 2017

Change the image to be gcr.io/google_containers/heapster-grafana-amd64:v4.0.2 will fix this issue.

doubaokun commented Jul 6, 2017

Change the image to be gcr.io/google_containers/heapster-grafana-amd64:v4.0.2 will fix this issue.

@Robbilie

This comment has been minimized.

Show comment
Hide comment
@Robbilie

Robbilie Jul 6, 2017

thats not a fix, i used to use a newer grafana image while waiting for heapster 1.4 because i need certain grafana notification channels

Robbilie commented Jul 6, 2017

thats not a fix, i used to use a newer grafana image while waiting for heapster 1.4 because i need certain grafana notification channels

@piosz

This comment has been minimized.

Show comment
Hide comment
@piosz

piosz Jul 10, 2017

Member

Apologies for the problems. @loburm is investigating this.

Member

piosz commented Jul 10, 2017

Apologies for the problems. @loburm is investigating this.

@trunet

This comment has been minimized.

Show comment
Hide comment
@trunet

trunet Jul 12, 2017

any news?

trunet commented Jul 12, 2017

any news?

@loburm

This comment has been minimized.

Show comment
Hide comment
@loburm

loburm Jul 12, 2017

Contributor

Sorry for the delay, I'm looking at this problem now. I will post update as soon as will find a root cause of issue.

Contributor

loburm commented Jul 12, 2017

Sorry for the delay, I'm looking at this problem now. I will post update as soon as will find a root cause of issue.

@loburm

This comment has been minimized.

Show comment
Hide comment
@loburm

loburm Jul 14, 2017

Contributor

This commit on the side of grafana is the root cause of the issue:
grafana/grafana@b191638

Shortly: previously they have been always creating grafana.ini file but in January this part was moved to the postinst script which is executed only when we actually install package on the system (in our case we just extracting files, so script is not executed).

I'll try to find workaround. Maybe we can use official grafana image for this.

Contributor

loburm commented Jul 14, 2017

This commit on the side of grafana is the root cause of the issue:
grafana/grafana@b191638

Shortly: previously they have been always creating grafana.ini file but in January this part was moved to the postinst script which is executed only when we actually install package on the system (in our case we just extracting files, so script is not executed).

I'll try to find workaround. Maybe we can use official grafana image for this.

@loburm

This comment has been minimized.

Show comment
Hide comment
@loburm

loburm Jul 17, 2017

Contributor

Unfortunately, solution with docker image officially provided by the grafana is not going to work. This image is built only for amd64 (we support different arch) and image size almost 2 times bigger.

So I have prepared PR that just manually copies configuration files to /etc/grafana/: #1728

Contributor

loburm commented Jul 17, 2017

Unfortunately, solution with docker image officially provided by the grafana is not going to work. This image is built only for amd64 (we support different arch) and image size almost 2 times bigger.

So I have prepared PR that just manually copies configuration files to /etc/grafana/: #1728

@ffahri

This comment has been minimized.

Show comment
Hide comment
@ffahri

ffahri Jul 17, 2017

check this #1722
v4.0.2
change to --> image: gcr.io/google_containers/heapster-grafana-amd64:v4.0.2

ffahri commented Jul 17, 2017

check this #1722
v4.0.2
change to --> image: gcr.io/google_containers/heapster-grafana-amd64:v4.0.2

@loburm

This comment has been minimized.

Show comment
Hide comment
@loburm

loburm Jul 17, 2017

Contributor

@ffahri this is not a fix of problem. Some users need features that have been released in 4.2.0 or later we just can't stuck with version 4.0.2 forever.

Contributor

loburm commented Jul 17, 2017

@ffahri this is not a fix of problem. Some users need features that have been released in 4.2.0 or later we just can't stuck with version 4.0.2 forever.

@loburm

This comment has been minimized.

Show comment
Hide comment
@loburm

loburm Jul 17, 2017

Contributor

PR was merged. Tomorrow I'll push a new image to the gcr.

Contributor

loburm commented Jul 17, 2017

PR was merged. Tomorrow I'll push a new image to the gcr.

@loburm

This comment has been minimized.

Show comment
Hide comment
@loburm

loburm Jul 18, 2017

Contributor

Image is pushed. In #1731 updates deploy file.

Contributor

loburm commented Jul 18, 2017

Image is pushed. In #1731 updates deploy file.

@discostur

This comment has been minimized.

Show comment
Hide comment
@discostur

discostur Jul 18, 2017

@loburm just tested the new image and it works without any problems ;)

discostur commented Jul 18, 2017

@loburm just tested the new image and it works without any problems ;)

@loburm loburm marked this as a duplicate of #1702 Jul 18, 2017

@loburm

This comment has been minimized.

Show comment
Hide comment
@loburm

loburm Jul 18, 2017

Contributor

@discostur thanks for checking it.

Considering that last PR was merged, I'm closing this issue.

Contributor

loburm commented Jul 18, 2017

@discostur thanks for checking it.

Considering that last PR was merged, I'm closing this issue.

@loburm loburm closed this Jul 18, 2017

@emansom

This comment has been minimized.

Show comment
Hide comment
@emansom

emansom Jul 18, 2017

It works again. Thanks guys. 👍
screen shot 2017-07-18 at 11 26 58

emansom commented Jul 18, 2017

It works again. Thanks guys. 👍
screen shot 2017-07-18 at 11 26 58

@jpds

This comment has been minimized.

Show comment
Hide comment
@jpds

jpds Aug 23, 2017

As a heads-up, this is still broken in the https://github.com/kubernetes/heapster/tree/release-1.4 branch

jpds commented Aug 23, 2017

As a heads-up, this is still broken in the https://github.com/kubernetes/heapster/tree/release-1.4 branch

@andyxning

This comment has been minimized.

Show comment
Hide comment
@andyxning

andyxning Aug 23, 2017

Member

@DirectXMan12 @piosz @loburm we should cherrypick #1728 to release-1.4.

@jpds As a workaround, give v1.4.1 a try.

Member

andyxning commented Aug 23, 2017

@DirectXMan12 @piosz @loburm we should cherrypick #1728 to release-1.4.

@jpds As a workaround, give v1.4.1 a try.

@BruceShenC

This comment has been minimized.

Show comment
Hide comment
@BruceShenC

BruceShenC Aug 30, 2017

@andyxning I have tried but it still fail

BruceShenC commented Aug 30, 2017

@andyxning I have tried but it still fail

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment