profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/aquarhead/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.
LOU Xun aquarhead @controlant-org Reykjavík, Ísland https://aqd.is a Rusty programmer

ElaWorkshop/TagListView 2333

Simple and highly customizable iOS tag list view, in Swift.

aquarhead/protect-the-rabbit 10

"L7" Policies for RabbitMQ, powered by BPF

aquarhead/blog 2

Yet another blog.

aquarhead/rabbids 2

Chaos Engineering toolset on RabbitMQ

aquarhead/ellit 1

EVE LogLite in terminal

mashiro-no-rabo/AsReader-Sinatra 1

My life as a reader, the Sinatra version.

aquarhead/advent-of-rust 0

Advent of Code with Rust!

aquarhead/avail 0

dead simple solution to "when are you available"

aquarhead/aya 0

Aya is an eBPF library for the Rust programming language, built with a focus on developer experience and operability.

aquarhead/bpf-linker 0

Simple BPF static linker

startedwillcrichton/flowistry

started time in 3 days

startedmitsuhiko/minijinja

started time in 4 days

startedtikv/tikv

started time in 6 days

pull request commentvectordotdev/vector

feat(kubernetes_logs source): add pod annotations

@jszwedko Thanks! I totally missed that, I thought docs are for Vector's documentation... It might help future contributors if it's in the root directory, or at least have a link in the root README.

I added docs now and signed off, also run a make check-docs locally which was all good, I don't know if there's a way for me to see the rendered website? Though I mostly followed the format of pod_labels so it should be fine.

aquarhead

comment created time in 6 days

push eventcontrolant-org/vector

LOU Xun

commit sha ae498196301ef3d8f79c38e9205ee391ae06232f

Fetch pod annotations in kubernetes_logs source Close #8338 Signed-off-by: LOU Xun <xun.lou@controlant.com>

view details

LOU Xun

commit sha d4776cc8a72ccb06c7f4b87395d2a3da9885be83

Add doc for pod_annotations also test for disabling annotations scraping Signed-off-by: LOU Xun <xun.lou@controlant.com>

view details

push time in 6 days

pull request commentvectordotdev/vector

feat(kubernetes_logs source): add pod annotations

Will do, did I miss a contributing guide somewhere?

aquarhead

comment created time in 7 days

startedTaKO8Ki/gobang

started time in 8 days

pull request commentvectordotdev/vector

feat(kubernetes_logs source): add pod annotations

For instance, think about kubectl apply - it adds an annotation with the whole resource as in the source of apply. It can easily be KBs of data per pod - per event.

That should be on the "controller" resource, such as Deployment or StatefulSet? Each pod would not have that annotation attached, I believe? (not with access to a kube cluster atm, will check later) I guess it would apply for Pod resources directly, but I don't think that necessarily translates to "increase the average amount of metadata on the average cluster tenfold without providing useful info".

aquarhead

comment created time in 9 days

pull request commentvectordotdev/vector

feat(kubernetes_logs source): add pod annotations

Thanks! I do wonder if we need to consider exposing some controls so that users can decide what/how many labels and annotations to add - we could definitely bloat some events depending on all this metadata

Didn't notice this comment since I was on phone, I'll give it a try to add some filters, is there existing benchmark suite I can use to compare the effect?

aquarhead

comment created time in 10 days

pull request commentvectordotdev/vector

feat(kubernetes_logs source): add pod annotations

Absolutely, sorry I forgot about that.

aquarhead

comment created time in 10 days

startedjohanhelsing/matchbox

started time in 10 days

startedgschup/ggrs

started time in 10 days

push eventaquarhead/starship

LOU Xun

commit sha 5cbb33646a592f58728c40bf26f74429beeb2058

Add kube namespace

view details

push time in 10 days

startedawwbees/BespokeSynth

started time in 10 days

pull request commentvectordotdev/vector

docs(unit tests): Improve unit testing documentation

Awesome work! I've been thinking about contributing to unit test related docs too since it's kinda outdated, though I probably need to play with it a bit more myself before feeling comfortable to contribute, but let me know if you want me to take over?

lucperkins

comment created time in 10 days

issue commenttokio-rs/prost

oneof field name can conflict with embedded message names

I do recognize this is a breaking change, but I'm not sure this is necessarily "sacrificing" ergonomic.

I don't think this (conflict) will happen "often" in code bases - it's a pretty bad naming practice - but my intuition is that when it happens it's pretty hard to convince people to change it - especially if Rust is a new addition in a heterogeneous system. Through that lens it probably does more harm to adoption of Rust in gRPC ecosystem - when something works perfectly fine in other languages (when I create this issue there're already code generation for Python 2/3, Go, Elixir, C# and C++ that consumes a central protobuf code base) but doesn't work w/ Rust.

(To be completely honest I don't have a critical need for it, this just comes up when working on some toy projects at my previous employer, but even then I don't have the power to change the gRPC definition given my use case)

Maybe leave the issue open for now and see if more people encounter this problem?

aquarhead

comment created time in 11 days

PR opened vectordotdev/vector

feat(kubernetes_logs): add pod annotations

Close #8338

<!-- Your PR title must conform to the conventional commit spec!

<type>(<scope>)!: <description>

  • type = chore, enhancement, feat, fix, docs
  • ! = OPTIONAL: signals a breaking change
  • scope = Optional when type is "chore" or "docs", available scopes https://github.com/timberio/vector/blob/master/.github/semantic.yml#L20
  • description = short description of the change

Examples:

  • enhancement(file source): Add sort option to sort discovered files
  • feat(new source): Initial statsd source
  • fix(file source): Fix a bug discovering new files
  • chore(external docs): Clarify batch_size option -->

That should be enough context but I'll add more details if necessary (need to go rn).

+21 -0

0 comment

1 changed file

pr created time in 11 days

create barnchcontrolant-org/vector

branch : kube-annotations

created branch time in 11 days

issue commentvectordotdev/vector

Expose pod annotations in kubernetes_logs

Hello, is there any progress on this? I'd like to contribute if there's no existing effort, cheers.

djablonski-moia

comment created time in 11 days

push eventcontrolant-org/sonarqube-prometheus-exporter

LOU Xun

commit sha 9d2a0370c151d9401273cf720ee1c50b682af021

Fix init

view details

push time in 12 days

push eventcontrolant-org/sonarqube-prometheus-exporter

LOU Xun

commit sha 9900a5c28826ebeb89b4e8752630a9eaa198bc80

Fix type

view details

push time in 12 days

push eventcontrolant-org/sonarqube-prometheus-exporter

LOU Xun

commit sha 96a77461db5dc9e187b282ac56fdb9fcf5e6cb3b

Force add 2 missing ratings

view details

push time in 12 days

push eventcontrolant-org/sonarqube-prometheus-exporter

LOU Xun

commit sha dd7ae619246b4598a12cc1012697189fea8c8470

Fix and add action

view details

push time in 13 days

create barnchcontrolant-org/sonarqube-prometheus-exporter

branch : ctrl

created branch time in 13 days

delete branch aquarhead/ghorg

delete branch : patch-1

delete time in 14 days

PR opened gabrie30/ghorg

Fix sample config for gitlab URL

I got this error when following the current doc on self-hosted GitLab 13.x:

<$> ghorg clone org

 +-+-+-+-+ +-+-+ +-+-+-+-+-+
 |T|I|M|E| |T|O| |G|H|O|R|G|
 +-+-+-+-+ +-+-+ +-+-+-+-+-+

*************************************
* SCM           : gitlab
* Type          : org
* Protocol      : ssh
* Location      : ...
* Concurrency   : 25
* Base URL      : https://somegitlab.instance.com/api/v3/
* Config Used   : .../conf.yaml
*************************************

Encountered an error, aborting
GET https://somegitlab.instance.com/api/v3/api/v4/groups/org/projects: 410 {error: API V3 is no longer supported. Use API V4 instead.}

While removing the api/v3/ fixed the issue.

+1 -1

0 comment

1 changed file

pr created time in 15 days

push eventaquarhead/ghorg

LOU Xun

commit sha e2dc07d4b8e8fd227cc45987d5fa8d297488e528

Fix sample config for gitlab URL

view details

push time in 15 days

fork aquarhead/ghorg

Quickly clone an entire org/users repositories into one directory - Supports GitHub, GitLab, Bitbucket, and more

fork in 15 days

startedDataDog/go-profiler-notes

started time in 16 days

push eventcontrolant-org/terraform-provider-aws

Angie Pinilla

commit sha dbf9e6260c881d37e7d4db03e33cca413ba505ec

update to use finder method's table propagation

view details

Angie Pinilla

commit sha a0ee1d7d16f613cdabf9caa6be4468b85a107062

update return handling

view details

Angie Pinilla

commit sha 164f0a30b3fe651a1b8e742657bc64218de3ad6a

carry over same missing param behavior

view details

Mohamed El Mouctar HAIDARA

commit sha 6563d7a505754d9cfe2d5ab4c138a27c2173089b

docs: Add link to the supported statistics for cloudwatch alarm for MetricStat CloudWatch metric alarm supports more statistics than `SampleCount`, `Average`, `Sum`, `Minimum`, `Maximum` when using MetricStat. Keeping these values in documentation may confuse users thinking only these values are supported.

view details

LOU Xun

commit sha 133181dbc6ef2bd1fc2b2891a3f07df2a21eed99

Move protocol to required section in SG doc

view details

Kit Ewbank

commit sha a1a5a40ca4518df0baf901b3524852b46ea56376

Merge pull request #20837 from controlant-org/d-security_group Move protocol to required section in SG doc

view details

Kit Ewbank

commit sha 58b587b7b85f04c352c905c37446db0acb8a139c

Merge pull request #20835 from haidaraM/docs/cloudWatchAlarmMetricStat docs: Add link to the supported statistics for cloudwatch alarm

view details

angie pinilla

commit sha 2794bcad8047682d88cc1dd5a7fa92a48fa2db6e

Merge pull request #20830 from hashicorp/f-pagination-ec2-gateways r/ec2_transit_gateway: replace `GetTransitGatewayRouteTablePropagations` with pagination method in ec2/finder

view details

changelogbot

commit sha 8e8a842217bd72e75ecc9a64a499690be1b40635

Update CHANGELOG.md for #20830

view details

nnovaes

commit sha 0a3e6f5d79695fabb8419f3cc206f3c833e6c9f7

Update cognito_identity_pool_roles_attachment.markdown (#15991) It's not clear that terraform requires the identity provider to be created for identity_provider in cognito_identity_pool_roles to work

view details

push time in 18 days