Update e2e tests and add Combiner to python wrappers #343
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
agrski
added a commit
that referenced
this pull request
Dec 2, 2022
* Move operational metrics docs under general metrics page * Capitalise proper nouns & acronyms/initialisms * Use one sentence per line for operational metrics * Add empty docs page for usage metrics * Add list of metrics to docs from Hodometer README * Add intro & privacy sections to Hodometer docs * Add link to table of metrics in Hodometer docs * Add section on metrics levels in Hodometer docs * Change list of metrics level to table * Use metrics levels name corresponding to allowed values in table * Add table of Hodometer env vars to usage metrics docs * Add sed script adding Helm templating for Hodometer deployment * Split sed script over multiple lines for legibility * Simplify sed script for maintainability Inserting before and appending after a multi-line string is simpler than replacing parts of it. The output is now shared across branches, regardless of whether or not the deployment is for Hodometer. A further (arguable) benefit is that YAML document markers are per-document, rather than potentially creating an empty document; this would be peculiar for anyone viewing Helm template output. * Refactor Helm template paths to Make vars for concision & consistency * Add enabled field for Hodometer in Helm values template * Add Helm templating for Hodometer installation toggle in Makefile * Regenerate Helm charts * Update sed script to apply to all resources with hodometer in the name * Regenerate Helm charts * Add Helm/k8s configuration section for Hodometer * Add tab for Helm install of Hodometer for k8s * Add tab for raw YAML install of Hodometer for k8s * Shorten section heading for clarity in usage metrics docs * Add sentence on configuring k8s raw YAML manifests in usage metrics docs * Add docs section on Compose configuration for Hodometer * Add section on performance & resource requirements of Hodometer * Add section on using extra publish URLs with Hodometer * Use monospace for Hodometer metric names in README table * Add brief descriptions of op & usage metrics on metrics overview page * Hide TOC on metrics page as links are given above * Add architecture diagram for Hodometer * Add architecture section to usage metrics docs * Fix typo
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #296