Skip to content

(feat) Migrate all markdown generation to use JINJA snippets #1000

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.

Sign up for GitHub

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

Merged
jsuereth merged 52 commits into from
May 15, 2024
Merged

(feat) Migrate all markdown generation to use JINJA snippets #1000

Show file tree
Hide file tree
Changes from 15 commits
Commits
Show all changes
52 commits
Select commit Hold shift + click to select a range
4c5619a
Update makefile so weaver generates ALL snippets using templates.
jsuereth May 2, 2024
92761a4
Update snippet templates to be more accurate.
jsuereth May 2, 2024
ca094cc
Run table generation.
jsuereth May 2, 2024
1990a9f
Update to use new sorting for attributes.
jsuereth May 3, 2024
8b9263b
Cleanup some whitespace issues.
jsuereth May 3, 2024
e4d209c
Appropriately render instrument types.
jsuereth May 3, 2024
350fb4a
Fix attribute registry links.
jsuereth May 3, 2024
cc54cb7
Add tag filtering to template.
jsuereth May 3, 2024
7df545f
Also filter enums by tag.
jsuereth May 3, 2024
f0931af
Fix enum table values.
jsuereth May 3, 2024
e1eb1c4
Share enum table macro between registry and snippets.
jsuereth May 3, 2024
72548c0
Remove odd passing of dependency.
jsuereth May 3, 2024
3119f1d
Fix requirement level.
jsuereth May 3, 2024
092be4e
Update requirement to optionally do notes.
jsuereth May 3, 2024
68fd3f5
Update snippet start/end.
jsuereth May 3, 2024
8412a09
Fix example ordering logic.
jsuereth May 4, 2024
d80446c
remove some unneeded whitespace.
jsuereth May 4, 2024
bdecec7
Remove another whitespace issue.
jsuereth May 4, 2024
2fd885c
Remove newline on enum table template.
jsuereth May 4, 2024
c6a4d27
Fix enum table to use id instead of value when no description.
jsuereth May 4, 2024
b5d99ec
more attempted whitespace fixes and cleanups.
jsuereth May 4, 2024
b86530d
Fix more whitespace issues around enum tables.
jsuereth May 4, 2024
6f244c2
Disbale markdown linter on autogen code.
jsuereth May 4, 2024
e934db7
update table check to use jinja templates.
jsuereth May 4, 2024
c2bb2f6
Actually disable markdown lint on snippets.
jsuereth May 4, 2024
e0d5dc3
Fix markdownlint enable/disable.
jsuereth May 4, 2024
b0cbc77
Add changelog.
jsuereth May 4, 2024
613292e
Merge remote-tracking branch 'origin/main' into wip-weaver-snippet-ge…
jsuereth May 4, 2024
8b5c7ad
Fix merge issues.
jsuereth May 4, 2024
009106d
Fix enum filtering and not rendering empty tables.
jsuereth May 6, 2024
0631394
Add sampling relevant attributes.
jsuereth May 6, 2024
5c1b4e2
Move sampling relevant sections to the right location.
jsuereth May 6, 2024
a831fd5
Clean up enum table rendering.
jsuereth May 6, 2024
e8e789f
Merge remote-tracking branch 'origin/main' into wip-weaver-snippet-ge…
jsuereth May 6, 2024
f2e7a39
Fixes from merge of main.
jsuereth May 6, 2024
0aed8ba
Merge remote-tracking branch 'origin/main' into wip-weaver-snippet-ge…
jsuereth May 7, 2024
086a4ad
Fix markdown from latest merge.
jsuereth May 7, 2024
d9ea0a2
Clean up templates to use more macros in consistent locatoins.
jsuereth May 7, 2024
391c736
Sort sampling relevant attributes for consistent rendering.
jsuereth May 7, 2024
c98339e
Merge main.
jsuereth May 7, 2024
03d5b6f
Merge main.
jsuereth May 7, 2024
b2801c9
Fix sorting for sampling attributes to be stable.
jsuereth May 7, 2024
4d6994d
Merge remote-tracking branch 'origin/main' into wip-weaver-snippet-ge…
jsuereth May 7, 2024
bead7e1
merge main.
jsuereth May 9, 2024
b201acd
Pull in fix for markdown snippet id parsing.
jsuereth May 9, 2024
cb699b9
Merge main.
jsuereth May 9, 2024
f2daaee
Merge main.
jsuereth May 13, 2024
ba22e4b
Add end of lines to macros files.
jsuereth May 13, 2024
a41e88f
Render event names again.
jsuereth May 13, 2024
c2076be
Fix event name logic.
jsuereth May 14, 2024
3ded9df
Update docs to link to weaver for markdown gen
jsuereth May 15, 2024
1e94921
Fix link
jsuereth May 15, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -95,10 +95,12 @@ yamllint:
# Generate markdown tables from YAML definitions
.PHONY: table-generation
table-generation:
docker run --rm -v $(PWD)/model:/source -v $(PWD)/docs:/spec \
docker run --rm -v $(PWD)/model:/source -v $(PWD)/docs:/spec -v $(PWD)/templates:/weaver/templates \
otel/weaver:${WEAVER_VERSION} registry update-markdown \
--registry=/source \
--attribute-registry-base-url="/docs/attributes-registry" \
--templates=/weaver/templates \
--target=markdown \
/spec

# Generate attribute registry markdown.
Expand Down
4 changes: 2 additions & 2 deletions docs/attributes-registry/aws.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -69,8 +69,8 @@ This document defines attributes for AWS Elastic Container Service (ECS).

| Value | Description | Stability |
| --------- | ----------- | ---------------------------------------------------------------- |
| `ec2` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `fargate` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `ec2` | ec2 | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `fargate` | fargate | ![Experimental](https://img.shields.io/badge/-experimental-blue) |

## Aws Eks Attributes

Expand Down
68 changes: 34 additions & 34 deletions docs/attributes-registry/db.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,8 +40,8 @@ If a parameter has no name and instead is referenced only by index, then `<key>`

| Value | Description | Stability |
| ------ | ----------- | ---------------------------------------------------------------- |
| `idle` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `used` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `idle` | idle | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `used` | used | ![Experimental](https://img.shields.io/badge/-experimental-blue) |

`db.system` has the following list of well-known values. If one of them applies, then the respective value MUST be used; otherwise, a custom value MAY be used.

Expand Down Expand Up @@ -115,19 +115,19 @@ This group defines attributes for Cassandra.

`db.cassandra.consistency_level` has the following list of well-known values. If one of them applies, then the respective value MUST be used; otherwise, a custom value MAY be used.

| Value | Description | Stability |
| -------------- | ----------- | ---------------------------------------------------------------- |
| `all` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `each_quorum` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `quorum` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `local_quorum` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `one` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `two` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `three` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `local_one` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `any` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `serial` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `local_serial` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| Value | Description | Stability |
| -------------- | ------------ | ---------------------------------------------------------------- |
| `all` | all | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `each_quorum` | each_quorum | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `quorum` | quorum | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `local_quorum` | local_quorum | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `one` | one | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `two` | two | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `three` | three | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `local_one` | local_one | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `any` | any | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `serial` | serial | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `local_serial` | local_serial | ![Experimental](https://img.shields.io/badge/-experimental-blue) |

## Db Cosmosdb Attributes

Expand All @@ -152,23 +152,23 @@ This group defines attributes for Azure Cosmos DB.

`db.cosmosdb.operation_type` has the following list of well-known values. If one of them applies, then the respective value MUST be used; otherwise, a custom value MAY be used.

| Value | Description | Stability |
| ------------------- | ----------- | ---------------------------------------------------------------- |
| `Invalid` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `Create` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `Patch` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `Read` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `ReadFeed` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `Delete` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `Replace` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `Execute` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `Query` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `Head` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `HeadFeed` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `Upsert` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `Batch` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `QueryPlan` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `ExecuteJavaScript` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| Value | Description | Stability |
| ------------------- | ----------------- | ---------------------------------------------------------------- |
| `Invalid` | Invalid | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `Create` | Create | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `Patch` | Patch | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `Read` | Read | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `ReadFeed` | ReadFeed | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `Delete` | Delete | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `Replace` | Replace | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `Execute` | Execute | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `Query` | Query | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `Head` | Head | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `HeadFeed` | HeadFeed | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `Upsert` | Upsert | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `Batch` | Batch | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `QueryPlan` | QueryPlan | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `ExecuteJavaScript` | ExecuteJavaScript | ![Experimental](https://img.shields.io/badge/-experimental-blue) |

## Db Deprecated Attributes

Expand Down Expand Up @@ -215,5 +215,5 @@ This group defines attributes for Elasticsearch.

| Value | Description | Stability |
| ------ | ----------- | ---------------------------------------------------------------- |
| `idle` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `used` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `idle` | idle | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `used` | used | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
4 changes: 2 additions & 2 deletions docs/attributes-registry/disk.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,5 +18,5 @@ These attributes may be used for any disk related operation.

| Value | Description | Stability |
| ------- | ----------- | ---------------------------------------------------------------- |
| `read` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `write` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `read` | read | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `write` | write | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
22 changes: 12 additions & 10 deletions docs/attributes-registry/network.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -72,18 +72,18 @@ different processes could be listening on TCP port 12345 and UDP port 12345.

| Value | Description | Stability |
| ------------- | ----------- | ---------------------------------------------------------------- |
| `wifi` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `wired` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `cell` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `unavailable` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `unknown` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `wifi` | wifi | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `wired` | wired | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `cell` | cell | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `unavailable` | unavailable | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `unknown` | unknown | ![Experimental](https://img.shields.io/badge/-experimental-blue) |

`network.io.direction` has the following list of well-known values. If one of them applies, then the respective value MUST be used; otherwise, a custom value MAY be used.

| Value | Description | Stability |
| ---------- | ----------- | ---------------------------------------------------------------- |
| `transmit` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `receive` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `transmit` | transmit | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `receive` | receive | ![Experimental](https://img.shields.io/badge/-experimental-blue) |

`network.transport` has the following list of well-known values. If one of them applies, then the respective value MUST be used; otherwise, a custom value MAY be used.

Expand Down Expand Up @@ -133,8 +133,10 @@ These attributes may be used for any network related operation.

| Value | Description | Stability |
| -------- | ------------------------------ | ---------------------------------------------------------------- |
| `ip_tcp` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `ip_udp` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `ip_tcp` | ip_tcp | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `ip_udp` | ip_udp | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `pipe` | Named or anonymous pipe. | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `inproc` | In-process communication. | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `inproc` | In-process communication. [5] | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `other` | Something else (non IP-based). | ![Experimental](https://img.shields.io/badge/-experimental-blue) |

**[5]:** Signals that there is only in-process communication not using a "real" network protocol in cases where network attributes would normally be expected. Usually all other network attributes can be left out in that case.
44 changes: 22 additions & 22 deletions docs/attributes-registry/rpc.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,24 +48,24 @@ This document defines attributes for remote procedure calls.

`rpc.connect_rpc.error_code` has the following list of well-known values. If one of them applies, then the respective value MUST be used; otherwise, a custom value MAY be used.

| Value | Description | Stability |
| --------------------- | ----------- | ---------------------------------------------------------------- |
| `cancelled` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `unknown` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `invalid_argument` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `deadline_exceeded` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `not_found` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `already_exists` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `permission_denied` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `resource_exhausted` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `failed_precondition` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `aborted` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `out_of_range` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `unimplemented` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `internal` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `unavailable` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `data_loss` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `unauthenticated` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| Value | Description | Stability |
| --------------------- | ------------------- | ---------------------------------------------------------------- |
| `cancelled` | cancelled | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `unknown` | unknown | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `invalid_argument` | invalid_argument | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `deadline_exceeded` | deadline_exceeded | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `not_found` | not_found | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `already_exists` | already_exists | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `permission_denied` | permission_denied | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `resource_exhausted` | resource_exhausted | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `failed_precondition` | failed_precondition | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `aborted` | aborted | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `out_of_range` | out_of_range | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `unimplemented` | unimplemented | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `internal` | internal | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `unavailable` | unavailable | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `data_loss` | data_loss | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `unauthenticated` | unauthenticated | ![Experimental](https://img.shields.io/badge/-experimental-blue) |

`rpc.grpc.status_code` has the following list of well-known values. If one of them applies, then the respective value MUST be used; otherwise, a custom value MAY be used.

Expand Down Expand Up @@ -93,8 +93,8 @@ This document defines attributes for remote procedure calls.

| Value | Description | Stability |
| ---------- | ----------- | ---------------------------------------------------------------- |
| `SENT` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `RECEIVED` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `SENT` | SENT | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `RECEIVED` | RECEIVED | ![Experimental](https://img.shields.io/badge/-experimental-blue) |

`rpc.system` has the following list of well-known values. If one of them applies, then the respective value MUST be used; otherwise, a custom value MAY be used.

Expand All @@ -121,5 +121,5 @@ Deprecated rpc message attributes.

| Value | Description | Stability |
| ---------- | ----------- | ---------------------------------------------------------------- |
| `SENT` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `RECEIVED` | none | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `SENT` | SENT | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
| `RECEIVED` | RECEIVED | ![Experimental](https://img.shields.io/badge/-experimental-blue) |
Loading