axiomhq/axiom-go

Official Go bindings for the Axiom API

axiom-go's Issues

Not compatible with latest go.opentelemetry.io/otel version(s)

I recently upgraded all my go packages one of those updates were alot of go.opentelemetry.io/otel/ modules

my go mod file
go_mod.txt

currently using github.com/axiomhq/axiom-go v0.18.0

And I am getting conflicting Schema URL: https://opentelemetry.io/schemas/1.25.0 and https://opentelemetry.io/schemas/1.24.0 which is most likely caused by

axiom-go/axiom/otel/trace.go

Line 17 in 06adf55

semconv "go.opentelemetry.io/otel/semconv/v1.24.0"

After downgrading some of the otel modules to 1.24.0 instead of 1.27.0 it works again

How do I pass time to the events in the client

README is missing an example of passing events with custom timestamps!

where to obtain `org_id`

me stupid, already figured out but...

when README.md mention about AXIOM_ORG_ID it should mention it's the simple string under your organisation's name in setting's page, otherwise I have been hunting for a long uuid with title Organisation Id on settings' page and found nothing.

when you provide me a click to copy button it doesn't indicate anything that is an org id, I only found out after I

1. ran axiom --help and in the FLAGS it told me what's my org id
2. google axiom org id and found this blog, this should be under any sdk uses or mention org_id

little thing, could save people minutes

HTTP2 errors with ingestion

http2: Transport: cannot retry err [http2: Transport received Server's graceful shutdown GOAWAY] after Request.Body was written; define Request.GetBody to avoid this error

Solved by setting request.GetBody(), axiom-go would need to provide an additional ingest api, something like

client.IngestStream(ctx, "my-dataset", func() (io.ReadCloser, error) {
	return os.Open("/some/file")
})

so that the go http client can retry correctly

add queries docs to axiom-go

https://pkg.go.dev/github.com/axiomhq/axiom-go/axiom

Accept APL time ranges on queries and shared access filters

Axiom 1.19 supports APL time ranges on query endpoints and the filters for shared access signatures. This should be addressed in this package as well.

This might be a good option to give Go Generics (coming in Go 1.18 Mid February 2022) a shoot. Instead of having interface{}/any, we could define a custom constraint for time.Time | APL where APL is basically a string based type with some sugar on it.

Change Op in EntryGroupAgg to Alias

What the server returns as "op" isn't named correctly, should be an alias.

axiom-go/axiom/query/result.go

Line 216 in fd368e7

Op AggregationOp `json:"op"`

Change to: Alias string json:"op"

Support for Axiom Integrations

Support the /integrations endpoint of the Axiom API.

No Client option to disable retries

Axiom go client Options doesn't include an Option to disable retries which is enabled by default.

It would be nice if you could opt out of the automatic retry feature

Ability to ingest OTEL data using proto format

Is it possible to add ability to ingest to OTEL endpoint using proto format?

This is what I am currently using for one of my services to send logs:

req := plogotlp.NewExportRequestFromLogs(logs)
		data, err := req.MarshalProto()
		if err != nil {
			log.Println(err)
			return err
		}

		httpReq, err := http.NewRequest("POST", axiomUrl+"/v1/logs", bytes.NewBuffer(data))
		if err != nil {
			log.Println(err)
			return err
		}

		httpReq.Header.Set("X-Axiom-Dataset", axiomDatasetName)
		httpReq.Header.Set("X-Axiom-OrgId", axiomOrgId)
		httpReq.Header.Set("Authorization", "Bearer "+ axiomToken)
		httpReq.Header.Set("Content-Type", "application/x-protobuf")

		_, err = http.DefaultClient.Do(httpReq)
		if err != nil {
			log.Println(err)
			return err
		}

Example should include IngestEvents and not Ingest stream since its not developer friendly

Developers by default want to create Event objects and ingest them and not create Streams. Please make this a bit more friendly.