-
Notifications
You must be signed in to change notification settings - Fork 61.1k
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
Explain critical blocking feature that causes tremendous misunderstan… #1092
Conversation
…dings I've seen it a 1000 times: User A gets angrier and angrier because User B is not responding, even to @mentions. That's because User A never read that blocking actually prevents User B from getting the message. This has exacerbated friction on teams I have worked with and needs to be documented.
Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines. |
That takes care of documentation potential blockers should read.
|
Thanks for opening a PR 👍 Please fill out the pull request template in the top comment in this pull request, then we'll get this up for review! |
Why:Many users block other users due to poor interaction. Then the interaction only becomes worse, due to the block. Because even when @mentioned, the blockee no longer responds to the blocker. Ah, if only the blockee could see what the blocker asked them. But alas, he won't get the email notification, due to the block. But the blocker doesn't realize this, and gets even angrier. What's being changed:Here I update the documentation, so blockers understand this situation, so won't get angry anymore. Check off the following:
|
Thanks so much @jidanni! I'll get this triaged for review 🌟 |
In fact not only email notifications, but web notifications are blocked too. |
This PR is stale because it has been open 7 days with no activity and will be automatically closed in 3 days. To keep this PR open, update the PR by adding a comment or pushing a commit. |
This is not stale! Thanks for your patience @jidanni 💛 |
Actually I tried very hard to get people to be aware of the problems involved with this issue. So here's what I posted elsewhere: Description: Here we see how GitHub's simplistic implementation of blocking Steps To Reproduce: Mallory visits https://github.com/settings/blocked_users Mallory then proceeds to add insulting comments to all 1000 of Alice's Hey sleepy Alice, you are so dumb that you don't even notice I defaced Even using Hey The defacement needn't even be on an issue Alice opened. But merely below a comment she left What legitimate GitHub user would See also: Also... Mallory double checks Also... the following scenarios could happen, sooner or later. Scenario A: 10:30 AM: real Bob, the company president, posts Black tie required. 10:35 AM: Oops, I meant 1234 S. Main St. Not North. (And that's where the terrorist ambush is waiting. The Primes are 10:40: Mary' (Mary prime) anesthetizes the (same photo) real Mary, and This is Mary from the financial committee. (But of course real Bob won't be notified that she asked him.) 10:42: Bob' (Bob prime) adds comment: Oops, you are right 11:15 Real Bob (unaware of the Primes' comments) adds comment via email: Oh, and remember to bring your silly party hats, everybody! 11:30 Some other employees post simileys, etc. 12:00 Lunch time: everybody waves at real Bob and real Mary, who are 15:00 Real Bob and real Mary are approving pull requests etc. as normal 18:00 Real Bob is wondering where everybody is. And real Mary is The next day families of the victims start suing, but lose the case as Why wouldn't this also be possible on Facebook? Well on Facebook, if you block somebody, there is no chance you could And real Bob is still going to be notified of any fake Mary positing on Anesthetizing a user is going beyond simply not allowing you to "follow Scenario B: Real Linus T. comments Those components are located in a/b/c/d. Fake Linus T. blocks real Linus T., and comments: Oops, I mean w/x/y/z. Developers download the fake software and the kernel gets infected, etc. |
lol |
Where is everybody? |
Sorry I had to paste it all in from a cell phone and couldn't clean it up. |
holla at your boy |
@brentjo says elsewhere:
Yes, but I am talking about notifications to that user. In my discussions here I am only talking about "blockee's rights are You see, people get used to https://github.com/notifications being like Anybody "tampering" with their issues will show up on the security Ah ha, but one day someone comes along and slides a cheery picture of OK, all the security guard needs to do is rig up his own security camera
That is good! (But a side effect is they forget they blocked the user
Yes. But (after Bill blocks them,) instead of getting a notification
Yes but I am only talking about notifications to blocked users. @benbalter's article only touches on that at
But such a coarse implementation fails to distinguish the good case of
That's right. In jidanni/mymaps#1, clearly a OK, that is the case of X being able to "anesthetize" (block) Y and then Boy, this is getting hard to remember. Let's just use two roles, 1. So rewriting that sentence: OK, that is the case of an evil user being able to "anesthetize" (block) Then there is the case Linus writing on Evil User's stuff... but that is Then there is a interesting third party case: Issue A started by user X So how does Linus ever find out? Well say unrelated user C adds a
(I just want to thank you all at GitHub for still being available to (Also I bet my "rapid tone" here is part of the reason I have become a Anyway: about the blocking party: Fine: the less stuff they see the My concern is entirely from the point of blockees (people who have been |
Thank you @jidanni for your thorough explanation of the various permutations of the problem you’re describing. We understand the point you’re making and will review your PR as soon as we can. However, continuing to comment will not make the review happen any faster and, in actuality, may slow down the review process because of the time spent ensuring we’ve given due consideration to any new information. Thanks again for your passion and interest in improving our documentation. |
Thanks @lee-dohm (Actually I am hoping they change the implementation. Then no documentation change would be necessary.) |
@jidanni I agree this could be improved. Unfortunately in this repository, we only have control over how our content on docs.github.com is written 😄 I'm taking a look at your PR ⚡ thank you for your contribution and explanations! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thank you for your contribution and providing clarity around notifications. I suggested a better place for this content, and a little tweak for wording to mirror the related content. Let me know if you have any questions.
@@ -49,6 +49,9 @@ In repositories you own, blocked users also cannot: | |||
- Comment on issues, pull requests, or commits | |||
- Add or edit wiki pages | |||
|
|||
(Keep in mind users you block will not receive email notifications of | |||
comments you make, even if [@mentioning](/articles/basic-writing-and-formatting-syntax/#mentioning-people-and-teams) them directly.) | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is helpful clarification ✨ but it seems like it would be more beneficial in one of the lists that already exist, just to keep content all in one place. I suggest making it a bullet point in the "After you've blocked a user, they cannot:" list, close to the related information 😄
After you've blocked a user, they cannot:
- Send you any notifications, including by @mentioning your username
- Receive any notifications sent by you, including comments @mentioning their username
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@megbird those are all great changes. I would just wait to see what direction @brentjo and @benbalter want to take this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👋 @jidanni We have a process to ensure that pull requests are reviewed as quickly as we can, directly at-mentioning individuals will not get your pull request reviewed any more quickly and can actually slow the response due to people being distracted by notification noise.
@brentjo
and @benbalter
are not maintainers in this repository. You can add the changes @megbird suggested, as a maintainer of this repo, if you'd like to see an update to the docs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@janiceilene , I would love to add them. But whatever I press seems to mess things up. Perhaps you folks can add them for me. Thanks.
This PR is stale because it has been open 7 days with no activity and will be automatically closed in 3 days. To keep this PR open, update the PR by adding a comment or pushing a commit. |
I wish there was a way to get through to the block designers to hear their opinions on all this. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This PR is stale because it has been open 7 days with no activity and will be automatically closed in 3 days. To keep this PR open, update the PR by adding a comment or pushing a commit. |
Too bad.
|
@jidanni Thanks for your patience! Our small team is working our way through all of the contributions, so it could be a while before someone from the team has the bandwidth to make this change for you. |
This PR is stale because it has been open 7 days with no activity and will be automatically closed in 3 days. To keep this PR open, update the PR by adding a comment or pushing a commit. |
* [Update] ABL - removed beta information (github#937) * removed beta information * removed the additional config * Update OpenTelemetry send data to Promscale (github#976) * [Update] ABL - removed beta information (github#937) * removed beta information * removed the additional config * Update send data for OpenTelemetry * add metrics pipeline and address review comments * fix review comments * opentelemetry updates * vale fixes * vale fixes * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * final commnets * language edits * vale fixes * rephrased a few sentences * removed redundant words * redundant sentences * fix nits * added steps to configure promscale * Update opentelemetry.md * Update opentelemetry.md * redundant info removed * updated the default port for the parameter * rephrased a sentence * Update promscale/send-data/opentelemetry.md Co-authored-by: Lana Brindley <[email protected]> Co-authored-by: Rajakavitha Kodhandapani <[email protected]> Co-authored-by: Ramon Guiu <[email protected]> Co-authored-by: Lana Brindley <[email protected]> * Update Grafana, Add Jaeger to visualize data in Promscale (github#963) * [Update] ABL - removed beta information (github#937) * removed beta information * removed the additional config * Add Jaeger to visualize data in Promscale * iterate over review comments * review iteration 2 * validated the steps * updated steps based on the grafana UI * added more description, and UI related content update * revert the bold UI * updated jaegar content * fixed a typo * some more updates * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * incorporated the changes suggested by Lana * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Ramon Guiu <[email protected]> * incorporated ramon's suggestions * typo fix * update nits in index.md * Update promscale/visualize-data/jaeger.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Ramon Guiu <[email protected]> * Update jaeger.md * Update grafana.md Co-authored-by: Rajakavitha Kodhandapani <[email protected]> Co-authored-by: Lana Brindley <[email protected]> Co-authored-by: Ramon Guiu <[email protected]> * Add send data from Jaeger and Zipkin (#1014) * [Update] ABL - removed beta information (github#937) * removed beta information * removed the additional config * Add Jaeger to visualize data in Promscale * iterate over review comments * Add send data from Jaeger and Zipkin * language and copy edits * Update promscale/visualize-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * fix indent nit * Update promscale/send-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/send-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/send-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/send-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/send-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/send-data/zipkin.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/send-data/zipkin.md Co-authored-by: Lana Brindley <[email protected]> * Some small fixes * Some small fixes * Some small fixes Co-authored-by: Rajakavitha Kodhandapani <[email protected]> Co-authored-by: Lana Brindley <[email protected]> Co-authored-by: Ramon Guiu <[email protected]> * Extension is now mandatory * USe to the timecsaledb ha image moving forward * small fix * [Update] ABL - removed beta information (github#937) * removed beta information * removed the additional config * Update OpenTelemetry send data to Promscale (github#976) * [Update] ABL - removed beta information (github#937) * removed beta information * removed the additional config * Update send data for OpenTelemetry * add metrics pipeline and address review comments * fix review comments * opentelemetry updates * vale fixes * vale fixes * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/send-data/opentelemetry.md Co-authored-by: Ramon Guiu <[email protected]> * final commnets * language edits * vale fixes * rephrased a few sentences * removed redundant words * redundant sentences * fix nits * added steps to configure promscale * Update opentelemetry.md * Update opentelemetry.md * redundant info removed * updated the default port for the parameter * rephrased a sentence * Update promscale/send-data/opentelemetry.md Co-authored-by: Lana Brindley <[email protected]> Co-authored-by: Rajakavitha Kodhandapani <[email protected]> Co-authored-by: Ramon Guiu <[email protected]> Co-authored-by: Lana Brindley <[email protected]> * Update Grafana, Add Jaeger to visualize data in Promscale (github#963) * [Update] ABL - removed beta information (github#937) * removed beta information * removed the additional config * Add Jaeger to visualize data in Promscale * iterate over review comments * review iteration 2 * validated the steps * updated steps based on the grafana UI * added more description, and UI related content update * revert the bold UI * updated jaegar content * fixed a typo * some more updates * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * incorporated the changes suggested by Lana * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Ramon Guiu <[email protected]> * incorporated ramon's suggestions * typo fix * update nits in index.md * Update promscale/visualize-data/jaeger.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Ramon Guiu <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Ramon Guiu <[email protected]> * Update jaeger.md * Update grafana.md Co-authored-by: Rajakavitha Kodhandapani <[email protected]> Co-authored-by: Lana Brindley <[email protected]> Co-authored-by: Ramon Guiu <[email protected]> * Add send data from Jaeger and Zipkin (#1014) * [Update] ABL - removed beta information (github#937) * removed beta information * removed the additional config * Add Jaeger to visualize data in Promscale * iterate over review comments * Add send data from Jaeger and Zipkin * language and copy edits * Update promscale/visualize-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * fix indent nit * Update promscale/send-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/send-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/send-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/send-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/send-data/jaeger.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/send-data/zipkin.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/grafana.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/send-data/zipkin.md Co-authored-by: Lana Brindley <[email protected]> * Some small fixes * Some small fixes * Some small fixes Co-authored-by: Rajakavitha Kodhandapani <[email protected]> Co-authored-by: Lana Brindley <[email protected]> Co-authored-by: Ramon Guiu <[email protected]> * Fix manage data for traces (github#1050) * make it clear that retention can be applied to metrics and traces and use a consistent structure for both * Add delete traces * clarify downsampling only apply to metrics * Update promscale/manage-data/delete-data.md Co-authored-by: Charis <[email protected]> * Update promscale/manage-data/delete-data.md Co-authored-by: Charis <[email protected]> * Update promscale/manage-data/retention.md Co-authored-by: Charis <[email protected]> * Update promscale/manage-data/retention.md Co-authored-by: Charis <[email protected]> * Update promscale/manage-data/retention.md Co-authored-by: Charis <[email protected]> * Update promscale/manage-data/retention.md Co-authored-by: Charis <[email protected]> * Update promscale/manage-data/retention.md Co-authored-by: Charis <[email protected]> * Update promscale/manage-data/retention.md Co-authored-by: Charis <[email protected]> * Update promscale/manage-data/retention.md Co-authored-by: Charis <[email protected]> * Update promscale/manage-data/retention.md Co-authored-by: Charis <[email protected]> Co-authored-by: Charis <[email protected]> * Extension is now mandatory * USe to the timecsaledb ha image moving forward * small fix * Update resource recommendation guide (github#1093) * Update resource recommendation guide * copy edits * straight quotes * Update promscale/installation/recomm-guide.md code review Co-authored-by: Lana Brindley <[email protected]> * Update promscale/installation/recomm-guide.md code review Co-authored-by: Lana Brindley <[email protected]> * Update promscale/installation/recomm-guide.md code review Co-authored-by: Lana Brindley <[email protected]> * Update promscale/installation/recomm-guide.md code review Co-authored-by: Lana Brindley <[email protected]> * Update promscale/installation/recomm-guide.md code review Co-authored-by: Lana Brindley <[email protected]> * fix opentelemetry collector config file * clarify one postgres parameter Co-authored-by: Rajakavitha1 <[email protected]> Co-authored-by: Lana Brindley <[email protected]> Co-authored-by: Ramon Guiu <[email protected]> * Monitoring Promscale (github#1090) * add details on alerts, dashboards and runbooks * add review comments * fix nit * Update promscale/monitoring-promscale.md copy edits Co-authored-by: Lana Brindley <[email protected]> * Update promscale/monitoring-promscale.md copy edits Co-authored-by: Lana Brindley <[email protected]> * Update promscale/monitoring-promscale.md copy edits Co-authored-by: Lana Brindley <[email protected]> * Update promscale/monitoring-promscale.md copy edits Co-authored-by: Lana Brindley <[email protected]> * Update promscale/monitoring-promscale.md copy edits Co-authored-by: Lana Brindley <[email protected]> * fix-nit * Update promscale/monitoring-promscale.md copy edits Co-authored-by: Lana Brindley <[email protected]> * Update promscale/monitoring-promscale.md copy edits Co-authored-by: Lana Brindley <[email protected]> * Update promscale/monitoring-promscale.md copy edits Co-authored-by: Lana Brindley <[email protected]> * fix nit * fix nit * Lana's changes and procedure rewrite * improve wording Co-authored-by: Rajakavitha Kodhandapani <[email protected]> Co-authored-by: Lana Brindley <[email protected]> Co-authored-by: Ramon Guiu <[email protected]> * Alerting, Recording rules in Promscale (github#1092) * add alerting, recoridng rules support in Promscale * added the procedure * formatting fixes * fixed the numbering * formatting issues * formatting numbers * formatting again * Update alerting-recording.md * added feedback from Ramon * Update promscale/downsampling/index.md Co-authored-by: Ramon Guiu <[email protected]> * fix nits * Update promscale/alerting.md code review Co-authored-by: Lana Brindley <[email protected]> * Update promscale/downsampling/recording.md code review Co-authored-by: Lana Brindley <[email protected]> * Update promscale/downsampling/recording.md code review Co-authored-by: Lana Brindley <[email protected]> * Update promscale/downsampling/recording.md copy edits Co-authored-by: Lana Brindley <[email protected]> * Update promscale/downsampling/recording.md copy edit Co-authored-by: Lana Brindley <[email protected]> * Update promscale/downsampling/recording.md copy edits Co-authored-by: Lana Brindley <[email protected]> * Update promscale/alerting.md copy edits Co-authored-by: Lana Brindley <[email protected]> * Update promscale/alerting.md copy edits Co-authored-by: Lana Brindley <[email protected]> * Update promscale/alerting.md copy edits Co-authored-by: Lana Brindley <[email protected]> * added a tag * fix nit * fix nit * fix nits * Incorporated Lana's changes and a few nits * fixed the github action error * improvements Co-authored-by: Rajakavitha1 <[email protected]> Co-authored-by: Ramon Guiu <[email protected]> Co-authored-by: Lana Brindley <[email protected]> * add upgrade instructions from Alpine image * remove distribute tracing page, no longer needed * renaming and clean up * Remove invalid reference to downsampling * Reorganize topics * Add APM experience Reference (#1071) * Add APM experience Reference * iterated over review comments * Add how to install toolkit * fix query * copy edits and validated on grafana * fixed numbering * fixed the highlight issue * added a procedure for installing extension * fixed the code block * Update promscale/visualize-data/apm-experience.md copy edits Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/apm-experience.md copy edits Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/apm-experience.md copy edits Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/apm-experience.md style guide edits Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/apm-experience.md Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/apm-experience.md copy edits Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/apm-experience.md copy edits Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/apm-experience.md copy edits Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/apm-experience.md copy edits Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/apm-experience.md copy edits Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/apm-experience.md copy edits Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/apm-experience.md copy edits Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/apm-experience.md copy edits Co-authored-by: Lana Brindley <[email protected]> * Update promscale/visualize-data/apm-experience.md copy edits Co-authored-by: Lana Brindley <[email protected]> * introduced procedure headings * copy edits * Update apm-experience.md * Update apm-experience.md * removed mounting method * reword apm entry Co-authored-by: Ramon Guiu <[email protected]> Co-authored-by: Rajakavitha1 <[email protected]> Co-authored-by: Lana Brindley <[email protected]> * fix links * add APM experience * spelling mistake Co-authored-by: Rajakavitha Kodhandapani <[email protected]> Co-authored-by: Lana Brindley <[email protected]> Co-authored-by: Ramon Guiu <[email protected]> Co-authored-by: Charis <[email protected]>
I've seen it a 1000 times: User A gets angrier and angrier because User B is not responding, even to @mentions.
That's because User A never read that blocking actually prevents User B from getting the message.
This has exacerbated friction on teams I have worked with and needs to be documented.