profile
viewpoint
Cyril Tovena cyriltovena @grafana Avignon, France Hacking in golang

cyriltovena/demo 5

some work demo

cyriltovena/logger 5

golang application to generate random logs

cyriltovena/DisableCharmBar 4

DisableCharmBar shows how to disable charmbar on a full screen WPF application

cyriltovena/agent 0

A lightweight subset of Prometheus for Grafana Cloud.

cyriltovena/aglio 0

An API Blueprint renderer with theme support that outputs static HTML

cyriltovena/agones 0

Dedicated Game Server Hosting and Scaling for Multiplayer Games on Kubernetes

cyriltovena/amazon-ecs-agent 0

Amazon Elastic Container Service Agent

cyriltovena/AutoFixture 0

AutoFixture is an open source library for .NET designed to minimize the 'Arrange' phase of your unit tests in order to maximise maintainability.Its primary goal is to allow developers to focus on what is being tested rather than how to setup the test scenario, by making it easier to create object graphs containing test data.

cyriltovena/autoroute 0

adds gcp route for kubernetes service

cyriltovena/backoff 0

Simple backoff algorithm in Go (golang)

pull request commentgoogleforgames/agones

#54 Preliminary Windows Image Support

At the end, it worked fine on both workstations running Docker Desktop 2.5.0.1 with WSL2 integration.

jeremyje

comment created time in 27 minutes

issue openedgrafana/iot-sitewise-datasource

Add link to issues folder in the plugions Overview page

https://grafana.com/grafana/plugins/grafana-iot-sitewise-datasource

created time in 41 minutes

issue commentgrafana/iot-sitewise-datasource

Grafana throws ExpiredTokenException after a certain period

the unzip path there is for darwin. Are you running the grafana server on an OSX machine? or ec2 instance?

sounavo

comment created time in an hour

push eventgoogleforgames/agones

Nomadic Bits

commit sha 8cf74420d49c9a4918b3e4eb3b3a6b7c5683f555

Issue #1854: Fix getplayercount() description on index (#1912) * Fix Getplayercount discription to match the rest api * Update Guides/ClientSDKS/index GetPlayerCount() add alert back in Co-authored-by: Mark Mandel <markmandel@google.com>

view details

push time in 2 hours

PR merged googleforgames/agones

Reviewers
Issue #1854: Fix getplayercount() description on index approved cla: yes size/XS

What type of PR is this?

/kind documentation

What this PR does / Why we need it:

Fixes the documentation for getplayercount() on the index guide page to match the description for this function in the rest api. Current description is not for the getplayercount function

Which issue(s) this PR fixes: Closes #1854

Special notes for your reviewer: First PR for agones, let me know if it needs updating!

+3 -3

8 comments

1 changed file

bnwhorton

pr closed time in 2 hours

issue closedgoogleforgames/agones

Agones Game Server Client SDKs

You mistakenly write the explanation of Alpha().IsPlayerConnected(playerID) to Alpha().GetPlayerCount() explanation :

Alpha().GetPlayerCount() This function returns if the playerID is currently connected to the GameServer. This is always accurate from what has been set through this SDK, even if the value has yet to be updated on the GameServer status resource.

Alpha().IsPlayerConnected(playerID) This function returns if the playerID is currently connected to the GameServer. This is always accurate from what has been set through this SDK, even if the value has yet to be updated on the GameServer status resource.

closed time in 2 hours

Natanil

pull request commentgoogleforgames/agones

Issue #1854: Fix getplayercount() description on index

Build Succeeded :clap:

Build Id: 47861e51-d2c8-4c40-8b23-5d469c3f8480

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

  • https://f1747b5-dot-preview-dot-agones-images.appspot.com/

To install this version:

  • git fetch https://github.com/googleforgames/agones.git pull/1912/head:pr_1912 && git checkout pr_1912
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.11.0-f1747b5
bnwhorton

comment created time in 2 hours

pull request commentgoogleforgames/agones

Issue #1854: Fix getplayercount() description on index

New changes are detected. LGTM label has been removed.

bnwhorton

comment created time in 2 hours

pull request commentgoogleforgames/agones

Issue #1854: Fix getplayercount() description on index

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: <a href="https://github.com/googleforgames/agones/pull/1912#" title="Author self-approved">bnwhorton</a>, <a href="https://github.com/googleforgames/agones/pull/1912#pullrequestreview-543057419" title="Approved">markmandel</a>

The full list of commands accepted by this bot can be found here.

The pull request process is described here

<details > Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment </details> <!-- META={"approvers":[]} -->

bnwhorton

comment created time in 2 hours

pull request commentgoogleforgames/agones

#54 Preliminary Windows Image Support

I've seen this a few times you can see others on the internet as well reasons why this happens off the top of my head.

  1. The buildx context is pointing to default and is not backed by the qemu instance that's created.
  2. I find that buildx works better on a Linux host. If you're on windows run it straight with docker build though the Makefile is not configured for that since the expectation is that it's always running in a Linux host. (It probably will run in WSL2 but this might be an awkward artifact).
  3. Running concurrent builds. If that's the case each artifact can have a separate context though you'll want to delete them after a build completes.

The way I develop is running a Hyper-V VM that runs Linux and build there. On the Windows host I run in Windows Container mode which allows me to build and run those containers. There's a Xonotic example that is not linked up that should build and run exclusively in a Windows Container environment. The qemu backed buildx context will not build Xonotic because it has powershell in it.

jeremyje

comment created time in 3 hours

issue commentgrafana/loki

Add Azure blob storage config examples to the docs

Is there support for an azure service for index too (blob, cosmosdb)? or just chunks.

wardbekker

comment created time in 4 hours

create barnchgrafana/loki

branch : k39

created branch time in 4 hours

issue openedgrafana/loki

Promtail configuration support for environment variable interpolation not working as documented

Describe the bug Documentation for promtail on grafana indicates that variable interpolation happens before the config file is read

scrape_configs:
  - job_name: varlogs
ec2_sd_configs:
  - role_arn: ${INSTANCE_PROFILE_ARN}

The above section of the config is for a scrape with EC2 service discovery. SInce we're using service profiles on EC2, it makes sense to just use the instance profile which we can set from the env.

Attempting to start promtail works, but authentication for the SD fails:

is not authorized to perform: sts:AssumeRole on resource: ${INSTANCE_PROFILE_ARN}

Whenever the EC2 authorization happens, it should have already interpolated and read those values from

To Reproduce Steps to reproduce the behavior:

  1. Started Promtail (Version 2.0.0) specifying config file in dry-run mode referencing config file
  2. Immediately get EC2 auth errors
  3. Logging for other scrape targets continues unimpeded

Expected behavior The config would have already interpolated the INSTANCE_PROFILE_ARN from the environment (or used a null string) per the documentation and not actually just try to use that raw text as the ARN.

Environment:

  • Infrastructure: EC2 Instance
  • Deployment tool: promtail binary from 2.0.0 release

Screenshots, Promtail config, or terminal output

Terminal output

Clients configured:
----------------------
url: https://<REDACTED>/loki/api/v1/push
batchwait: 10s
batchsize: 102400
tls_config:
  ca_file: /etc/loki/ca.crt
  cert_file: /etc/loki/client.crt
  key_file: /etc/loki/client.key
  server_name: <REDACTED>
  insecure_skip_verify: false
backoff_config:
  min_period: 500ms
  max_period: 5m0s
  max_retries: 10
external_labels: |
  ami_name: ${AMI_NAME}
  node_type: ${NODE_TYPE}
timeout: 10s
tenant_id: ""

level=info ts=2020-12-02T15:33:06.360175051Z caller=server.go:225 http=[::]:8080 grpc=[::]:9095 msg="server listening on addresses"
level=info ts=2020-12-02T15:33:06.361160507Z caller=main.go:108 msg="Starting Promtail" version="(version=2.0.0, branch=HEAD, revision=6978ee5d)"
2020-12-02T14:37:32	{job="systemd-journal"}pam_unix(sudo:session): session opened for user root by ubuntu(uid=0)
2020-12-02T14:37:37	{job="systemd-journal"}pam_unix(sudo:session): session closed for user root
2020-12-02T15:17:01	{job="systemd-journal"}pam_unix(cron:session): session opened for user root by (uid=0)
2020-12-02T15:17:01	{job="systemd-journal"}(root) CMD (   cd / && run-parts --report /etc/cron.hourly)
2020-12-02T15:17:01	{job="systemd-journal"}pam_unix(cron:session): session closed for user root
2020-12-02T15:31:20	{job="systemd-journal"}  ubuntu : TTY=pts/1 ; PWD=/home/ubuntu ; USER=root ; COMMAND=./promtail-linux-amd64 -config.file=/etc/loki/promtail-config.yaml --dry-run
2020-12-02T15:31:20	{job="systemd-journal"}pam_unix(sudo:session): session opened for user root by ubuntu(uid=0)
2020-12-02T15:31:22	{job="systemd-journal"}pam_unix(sudo:session): session closed for user root
2020-12-02T15:32:55	{job="systemd-journal"}  ubuntu : TTY=pts/1 ; PWD=/home/ubuntu ; USER=root ; COMMAND=/usr/bin/vim /etc/loki/promtail-config.yaml
2020-12-02T15:32:55	{job="systemd-journal"}pam_unix(sudo:session): session opened for user root by ubuntu(uid=0)
2020-12-02T15:33:03	{job="systemd-journal"}pam_unix(sudo:session): session closed for user root
2020-12-02T15:33:06	{job="systemd-journal"}  ubuntu : TTY=pts/1 ; PWD=/home/ubuntu ; USER=root ; COMMAND=./promtail-linux-amd64 -config.file=/etc/loki/promtail-config.yaml --dry-run
2020-12-02T15:33:06	{job="systemd-journal"}pam_unix(sudo:session): session opened for user root by ubuntu(uid=0)
level=error ts=2020-12-02T15:33:06.435743867Z caller=refresh.go:79 component=discovery discovery=ec2 msg="Unable to refresh target groups" err="could not describe instances: AccessDenied: User: arn:aws:sts::906087756158:assumed-role/<REDACTED>/<REDACTED> is not authorized to perform: sts:AssumeRole on resource: ${INSTANCE_PROFILE_ARN}\n\tstatus code: 403, request id: 2f96df6e-b1ca-47b2-a405-f52b16e8ad79"

I've dug into the source code for V2 and I don't even see the interpolation happening for any variables. Was this removed in 2.0 and the docs not updated?

Screen Shot 2020-12-02 at 10 21 56 AM

At this point Promtail is created and getting ready to be run, but the config still has the interpolation strings, not null values or values from the environment

created time in 4 hours

startedcharmbracelet/bubbletea

started time in 5 hours

push eventgrafana/loki

owen-d

commit sha 90a8453de052a9181c7df91082e739c7037d25b1

[skip ci] Publishing helm charts: 05f3d6e0daafabcca8442e60dd70617ca76011a1

view details

push time in 5 hours

push eventgrafana/loki

Owen Diehl

commit sha 05f3d6e0daafabcca8442e60dd70617ca76011a1

Updates chunkenc doc for V3 (#2956)

view details

push time in 5 hours

delete branch grafana/loki

delete branch : owen-d-patch-1

delete time in 5 hours

PR merged grafana/loki

Updates chunkenc doc for V3 size/S
+8 -8

0 comment

1 changed file

owen-d

pr closed time in 5 hours

push eventgrafana/loki

owen-d

commit sha 350fd2ce96b80cc508349da3111e24301145f23c

[skip ci] Publishing helm charts: 794c324e64c4eb2f984b0f9b1a2ac3471d4d6faa

view details

push time in 5 hours

push eventgrafana/loki

owen-d

commit sha 1fde3fe0450ad411c6646fcbf0f0c5232054547f

[skip ci] Publishing helm charts: 315972409b58664bf55d8f24cc3bdde60548e50f

view details

push time in 6 hours

push eventgrafana/loki

owen-d

commit sha b1dd555373daa380dbf85006e63ba9225cecf366

[skip ci] Publishing helm charts: 5c9a913caa69546b3127d6320f052b6501f43459

view details

push time in 6 hours

delete branch grafana/loki

delete branch : jdb/cleanup-troubleshooting-formatting

delete time in 6 hours

push eventgrafana/loki

Jack Baldry

commit sha 794c324e64c4eb2f984b0f9b1a2ac3471d4d6faa

doc: Cleanup formatting (#3017) * doc: Add missing backticks Signed-off-by: Jack Baldry <jack.baldry@grafana.com> * doc: Use absolute path to positions.yaml file Signed-off-by: Jack Baldry <jack.baldry@grafana.com> * doc: Remove .. from URL paths I'm not sure what the `..` means exactly. Signed-off-by: Jack Baldry <jack.baldry@grafana.com>

view details

push time in 6 hours

PR merged grafana/loki

doc: Cleanup formatting size/XS

Signed-off-by: Jack Baldry jack.baldry@grafana.com

+3 -3

2 comments

1 changed file

jdbaldry

pr closed time in 6 hours

pull request commentgrafana/loki

doc: Cleanup formatting

Thanks.

re: highlighting - sounds good to me.

jdbaldry

comment created time in 6 hours

push eventgrafana/loki

Cyril Tovena

commit sha 315972409b58664bf55d8f24cc3bdde60548e50f

Add more information about why a tailer would stop. (#3015) Signed-off-by: Cyril Tovena <cyril.tovena@gmail.com>

view details

push time in 6 hours

PR merged grafana/loki

Add more information about why a tailer would stop. size/XS

Signed-off-by: Cyril Tovena cyril.tovena@gmail.com

Should help #2979 discover the reason why it stopped.

+1 -1

1 comment

1 changed file

cyriltovena

pr closed time in 6 hours

push eventgrafana/loki

Owen Diehl

commit sha 5c9a913caa69546b3127d6320f052b6501f43459

loki wal libsonnet (#3014)

view details

push time in 6 hours

PR merged grafana/loki

loki wal libsonnet size/M

Adds WAL support to jsonnet.

+59 -7

0 comment

3 changed files

owen-d

pr closed time in 6 hours

issue closedgrafana/loki

Can't install Loki with Helm

Hey there, I can't install Loki with Helm, facing this error:

helm install -f values.yaml loki --namespace=loki loki/loki
Error: rendered manifests contain a resource that already exists. Unable to continue with install: PodSecurityPolicy "loki" in namespace "" exists and cannot be imported into the current release: invalid ownership metadata; label validation error: missing key "app.kubernetes.io/managed-by": must be set to "Helm"; annotation validation error: missing key "meta.helm.sh/release-name": must be set to "loki"; annotation validation error: missing key "meta.helm.sh/release-namespace": must be set to "loki"

Have anyone know how to fix it? Thanks,

closed time in 6 hours

mrgoonie
more