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

Update eks-observe.md #766

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Update eks-observe.md #766

wants to merge 2 commits into from

Conversation

azarboon
Copy link
Contributor

@azarboon azarboon commented Jun 9, 2024

Added a fallacy of distributed computing (observability is optional), so readers know why they should care about observability. It's coined by one of the thought leaders in software architecture Mark Richards.

Here you can briefly familiarize yourself with this fallacy: https://youtu.be/UDtQgXDfkO8

And you can read about it further on these two books: https://a.co/d/4Tsadn3 https://a.co/d/9GFrPtC

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Added a fallacy of distributed computing (observability is optional), so readers know why they should care about observability. It's coined by one of the thought leaders in software architecture Mark Richards.

Here you can briefly familiarize yourself with this fallacy: 
https://youtu.be/UDtQgXDfkO8

And you can read about it further on these two books: https://a.co/d/4Tsadn3 https://a.co/d/9GFrPtC
@bryantbiggs

This comment was marked as off-topic.

@azarboon azarboon closed this Jun 10, 2024
@azarboon
Copy link
Contributor Author

azarboon commented Jun 10, 2024

Based on your GitHub profile, this looks like you're simply trying to add superfluous PRs to increase your profile. I don't see this as value added and would encourage you to focus on real improvements that are not superficial

@bryantbiggs your comment was unprofessional, to say the least. Nobody has paid me nor will pay me for the free labor I'm putting in here. I'm doing it just to help the community so please stop acting entitled. I really hope your behavior is not representative of this repo's maintainers.

Putting that fallacy in this page, can help some non-observability-practitioners to think about it at least for a few minutes. I, formerly, used to be one of those people who thought observability is optional or it can be delayed.

That fallacy is coined by one of the thought leaders in the field and I'm honored to spread the word, where it matters. My similar PRs have been accepted in official repositories by both AWS e.g. here and Azure, and in Azure documentations (yes, you can track them all on my profile)

@azarboon azarboon reopened this Jun 10, 2024
@azarboon
Copy link
Contributor Author

azarboon commented Jun 10, 2024

@pgasca @fincd-aws @geoffcline @jimdial-aws @judyheflin @lizsnyder @matteofigus @mwunderl @Paul-B-AWS @yanjieniu I'm wondering, is such abusive comment common or tolerated? Below is the screenshot, in case he modifies it.

Basically, I'm trying to spread the word about a common fallacy which is coined by one of the thought leaders in the field. Knowing that fallacy can help novice readers to care about observability, to begin with. My similar PR has been accepted in both AWS and Azure official repositories for example here. I'm a new contributor to this repository and that comment was off-putting, to say the least. Personally attacking a new contributor is not the best welcoming message that he can receive. I've had +25 merged PRs on both AWS and Azure repos. This is the first time I encounter such an abusive behavior and I'm really disappointed. I hope you will take a proper action otherwise he can dissuade other new contributors, too.

github-abuse

@bryantbiggs

This comment was marked as abuse.

@azarboon
Copy link
Contributor Author

azarboon commented Jun 10, 2024

I am sorry if my comment upset you that deeply, but I do believe it is an accurate statement. I don't see this behavior of opening multiple 1 line change PRs with subjective statements as something that is solving a pain point/problem or improving on what is provided. Perhaps we can start with an issue first that is describing the situation and work backwards from that before we jump in to proposing changes?

Go and read the PRs, look at each one's context then express your opinion POLITELY not by making a single comment which has nothing but a personal attacking. I don't know about this repo but in all other official repositories that I've contributed it, none of the maintainers judged my PR based on the fact that it was only one line or a few lines. This speaks volume of your professionalism.

Now, I know that I don't want to contribute anymore neither to this repo or nor to any other repo that you are involved.

@azarboon
Copy link
Contributor Author

azarboon commented Jun 10, 2024

Ironically, I made the same PR in other AWS repository which was well accepted.

@mwunderl
Copy link
Member

Let's keep the discussion focused on the content and the utility of the proposed change for customers.

The guidance in this topic is intended to help customers understand the use cases for observability with EKS. I would expect a reader of this document to have some interest in observability and an open mind, and not place any assumptions on them.

The third paragraph of the intro discusses how and why monitoring is important in cloud applications. I think there's room at the end of the intro (after the list of questions) to emphasize why observability is critical to good application architecture, if you can relate it directly to reliability, efficiency, operational excellence, et cetera.

Copy link
Contributor

@geoffcline geoffcline left a comment

Choose a reason for hiding this comment

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

Hi @azarboon

I'm a technical writer who works on maintaining this repo. I noticed you opened several PRs with similar small changes. We welcome contributions. In the future, could you combine small edits into one PR? This helps expedite review and processing.

Thanks!
Geoffrey

@@ -1,6 +1,6 @@
# Observability in Amazon EKS<a name="eks-observe"></a>

You can observe your data in Amazon EKS using many available monitoring or logging tools\. Your Amazon EKS log data can be streamed to AWS services or to partner tools for data analysis\. There are many services available in the AWS Management Console that provide data for troubleshooting your Amazon EKS issues\. You can also use an AWS\-supported open\-source solution for [monitoring Amazon EKS infrastructure](https://docs.aws.amazon.com/grafana/latest/userguide/solution-eks.html)\.
One of the fallacies of distributed computing is that observability is optional. It's a prevalent but wrong assumption. To have a reliable and changeable system, you need to have proper observability in place. You can observe your data in Amazon EKS using many available monitoring or logging tools\. Your Amazon EKS log data can be streamed to AWS services or to partner tools for data analysis\. There are many services available in the AWS Management Console that provide data for troubleshooting your Amazon EKS issues\. You can also use an AWS\-supported open\-source solution for [monitoring Amazon EKS infrastructure](https://docs.aws.amazon.com/grafana/latest/userguide/solution-eks.html)\.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this may confuse users if it's the first sentence on the page. I think the first paragraph should focus on what users will learn from the entire page, instead of high level considerations. Can you move this edit into another paragraph?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@azarboon
Copy link
Contributor Author

azarboon commented Jun 18, 2024

Hi @azarboon

I'm a technical writer who works on maintaining this repo. I noticed you opened several PRs with similar small changes. We welcome contributions. In the future, could you combine small edits into one PR? This helps expedite review and processing.

Thanks! Geoffrey

Thanks for your comment. I can see some positive actions from your side. I'm happy that not everybody in this repo is toxic and abusive.

Regarding your comment: sure, I noted it. FYI, I use corporate laptop so I can use Github on browser. But yes, I'll try to combine my future and relevant PRs into one.

Moved my edit to lower section
@azarboon
Copy link
Contributor Author

@geoffcline thanks for your constructive feedback. I just moved it to a lower section. Also I considered @mwunderl comment. Feel free to edit it further.

@geoffcline geoffcline self-assigned this Jun 24, 2024
@geoffcline
Copy link
Contributor

Thanks, you can use the github web based editor to change multiple files at once https://docs.github.com/en/codespaces/the-githubdev-web-based-editor

@azarboon
Copy link
Contributor Author

Thanks, you can use the github web based editor to change multiple files at once https://docs.github.com/en/codespaces/the-githubdev-web-based-editor

Thanks. Yes, I've already combined all my PRs in EKS best practices repo. Kindly please check

aws/aws-eks-best-practices#524 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants