Skip to content
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.

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

doc(ksonnet): include ksonnet-lib #1042

Merged
merged 11 commits into from
Sep 24, 2019
Merged

Conversation

daixiang0
Copy link
Contributor

Since replace ksonnet with Tanka, doc need to update to latest.

Signed-off-by: Xiang Dai [email protected]

@daixiang0
Copy link
Contributor Author

@sh0rez please review it.

Signed-off-by: Xiang Dai <[email protected]>
Signed-off-by: Xiang Dai <[email protected]>
Signed-off-by: Xiang Dai <[email protected]>
Copy link
Member

@sh0rez sh0rez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure about this, we really don't need to repeat all the info for Loki as well, we might rather want to restructure this doc to make sure people are aware of the fact that Tanka basically takes jsonnet libraries and composes them to application stacks.

There is also no need for using the default environment, when deploying loki it should be in a loki env

production/ksonnet/README.md Outdated Show resolved Hide resolved
production/ksonnet/README.md Outdated Show resolved Hide resolved
production/ksonnet/README.md Outdated Show resolved Hide resolved
production/ksonnet/README.md Outdated Show resolved Hide resolved
production/ksonnet/README.md Outdated Show resolved Hide resolved
production/ksonnet/README.md Outdated Show resolved Hide resolved
production/ksonnet/README.md Outdated Show resolved Hide resolved
Signed-off-by: Xiang Dai <[email protected]>
production/ksonnet/README.md Outdated Show resolved Hide resolved
Signed-off-by: Xiang Dai <[email protected]>
Signed-off-by: Xiang Dai <[email protected]>
Signed-off-by: Xiang Dai <[email protected]>
Copy link
Member

@sh0rez sh0rez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While this is correct in theory now, I think it would be nicer to use environments/loki/ instead of ./loki/

Sorry for being so picky about this 😛

production/ksonnet/README.md Outdated Show resolved Hide resolved
production/ksonnet/README.md Outdated Show resolved Hide resolved
production/ksonnet/README.md Outdated Show resolved Hide resolved
production/ksonnet/README.md Outdated Show resolved Hide resolved
@sh0rez sh0rez self-assigned this Sep 24, 2019
@sh0rez sh0rez added component/docs component/packaging type/enhancement Something existing could be improved labels Sep 24, 2019
Signed-off-by: Xiang Dai <[email protected]>
Signed-off-by: Xiang Dai <[email protected]>
Signed-off-by: Xiang Dai <[email protected]>
Copy link
Member

@sh0rez sh0rez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here we go! LGTM 🎉

@sh0rez sh0rez changed the title update ksonnect doc for Tanka doc(ksonnet): include ksonnet-lib Sep 24, 2019
@sh0rez sh0rez merged commit 20048f5 into grafana:master Sep 24, 2019
@daixiang0 daixiang0 deleted the fix-ksonnect branch September 25, 2019 02:15
cyriltovena pushed a commit to cyriltovena/loki that referenced this pull request Jun 11, 2021
Now that QueryPages() runs across multiple goroutines, the callback
can be running in parallel so we need to lock round the single data
structure it is accessing.

Signed-off-by: Bryan Boreham <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/packaging type/enhancement Something existing could be improved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants