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

Cross-post Dask observability post #168

Closed
wants to merge 1 commit into from

Conversation

scharlottej13
Copy link
Contributor

@hendrikmakait wrote this for the Coiled blog, but the content is Dask-focused so I think it would be a good fit for cross-posting on blog.dask.org (see dask/community#291 (comment) for some discussion around this, noting we did something similar in #151).

I made no content changes from the original post.

cc @jacobtomlinson

@netlify
Copy link

netlify bot commented May 17, 2023

Deploy Preview for dask-blog ready!

Name Link
🔨 Latest commit 2afffc2
🔍 Latest deploy log https://app.netlify.com/sites/dask-blog/deploys/64652285b85adb0008d32411
😎 Deploy Preview https://deploy-preview-168--dask-blog.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@scharlottej13 scharlottej13 marked this pull request as ready for review May 17, 2023 18:54
Copy link
Member

@jacobtomlinson jacobtomlinson left a comment

Choose a reason for hiding this comment

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

Thanks @scharlottej13. I love the general intent of this post. However, the post interweaves Dask features with Coiled features throughout so I'm not sure it's the best fit for the Dask community blog without some heavy modification. It feels like too much of a Coiled sales pitch in its current format.

I see three options here:

  1. Rework the post to remove the Coiled-specific content and instead point readers to the Coiled blog version at the end for further reading. I think it's fine to direct folks to Coiled to learn about how Coiled solves this problem after discussing the OSS options and their pros/cons.
  2. Add a "Sponsored Post" banner at the top to make it clear that this is a Coiled advert. I'm not sure if we want to go down this road in general, but I know it's been discussed before so maybe the Dask Owners need to chat about this.
  3. We don't cross-post this one

If we want to go with option 1 let me know and I'll go through and review it through that lens. Also happy to chat about the other options, maybe a call would be easier to get into the nuance here?


When dealing with unexpected issues in a distributed system, you need to understand what and _why_ it happened, how interactions between individual pieces contributed to the problems, and how to avoid them in the future.
In other words, you need **observability**.
This article explains what observability is, how Dask implements it, what pain points remain, and how Coiled helps you overcome these.
Copy link
Member

Choose a reason for hiding this comment

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

This line especially sets the reader up for a sales pitch.

@hendrikmakait
Copy link
Member

I agree with @jacobtomlinson, in its current form the post does not seem to be appropriate for the Dark blog. I'd personally be up for modifying in into a format that feels less sponsored, but I could use an independent sparring partner to make this a quick piece of work. @jacobtomlinson, would you want to collaborate on this?

@jacobtomlinson
Copy link
Member

Very happy to help. I don't have capacity this week but maybe next week we could chat about this?

@scharlottej13
Copy link
Contributor Author

Thanks @jacobtomlinson and @hendrikmakait! I'll close this then (and open a new PR for the css changes I suggested, which is what I should have done in the first place 😄)

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.

3 participants