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

Content for Blog Post - Optimize OpenSearch Refresh Interval #2397

Merged
merged 21 commits into from
Nov 13, 2023
Merged

Conversation

ev2900
Copy link
Contributor

@ev2900 ev2900 commented Oct 28, 2023

Pull request with the content for blog post Optimize OpenSearch Refresh Interval.

If needed reference:

Signed-off-by: Christopher Sharkey <[email protected]>

Signed-off-by: Christopher Sharkey <[email protected]>
Signed-off-by: Christopher Sharkey <[email protected]>

Signed-off-by: Christopher Sharkey <[email protected]>
Signed-off-by: Christopher Sharkey <[email protected]>

Signed-off-by: Christopher Sharkey <[email protected]>
Signed-off-by: Christopher Sharkey <[email protected]>
Signed-off-by: Christopher Sharkey <[email protected]>

Signed-off-by: Christopher Sharkey <[email protected]>
Copy link
Contributor

@vagimeli vagimeli left a comment

Choose a reason for hiding this comment

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

Great writing! Some general copyedits for clarity and writing conventions.

_posts/2023-10-28-optimize-refresh-interval.markdown Outdated Show resolved Hide resolved
_posts/2023-10-28-optimize-refresh-interval.markdown Outdated Show resolved Hide resolved
_posts/2023-10-28-optimize-refresh-interval.markdown Outdated Show resolved Hide resolved
_posts/2023-10-28-optimize-refresh-interval.markdown Outdated Show resolved Hide resolved
_posts/2023-10-28-optimize-refresh-interval.markdown Outdated Show resolved Hide resolved
_posts/2023-10-28-optimize-refresh-interval.markdown Outdated Show resolved Hide resolved
_posts/2023-10-28-optimize-refresh-interval.markdown Outdated Show resolved Hide resolved
_posts/2023-10-28-optimize-refresh-interval.markdown Outdated Show resolved Hide resolved
_posts/2023-10-28-optimize-refresh-interval.markdown Outdated Show resolved Hide resolved
_posts/2023-10-28-optimize-refresh-interval.markdown Outdated Show resolved Hide resolved
ev2900 and others added 10 commits November 1, 2023 11:15
Co-authored-by: Melissa Vagi <[email protected]>
Signed-off-by: Christopher Sharkey <[email protected]>
Co-authored-by: Melissa Vagi <[email protected]>
Signed-off-by: Christopher Sharkey <[email protected]>
Co-authored-by: Melissa Vagi <[email protected]>
Signed-off-by: Christopher Sharkey <[email protected]>
Co-authored-by: Melissa Vagi <[email protected]>
Signed-off-by: Christopher Sharkey <[email protected]>
Co-authored-by: Melissa Vagi <[email protected]>
Signed-off-by: Christopher Sharkey <[email protected]>
Co-authored-by: Melissa Vagi <[email protected]>
Signed-off-by: Christopher Sharkey <[email protected]>
Co-authored-by: Melissa Vagi <[email protected]>
Signed-off-by: Christopher Sharkey <[email protected]>
Co-authored-by: Melissa Vagi <[email protected]>
Signed-off-by: Christopher Sharkey <[email protected]>
Co-authored-by: Melissa Vagi <[email protected]>
Signed-off-by: Christopher Sharkey <[email protected]>
Co-authored-by: Melissa Vagi <[email protected]>
Signed-off-by: Christopher Sharkey <[email protected]>
@ev2900
Copy link
Contributor Author

ev2900 commented Nov 1, 2023

Thanks @vagimeli appreciate you reviewing the blog. I approved all the changes.

@pajuric
Copy link

pajuric commented Nov 3, 2023

@ev2900 @vagimeli - Are we good to go on this blog?

@vagimeli
Copy link
Contributor

vagimeli commented Nov 3, 2023

@ev2900 @vagimeli - Are we good to go on this blog?

@pajuric Yes, ready to go. Thanks!

@ev2900
Copy link
Contributor Author

ev2900 commented Nov 3, 2023

@pajuric ready to go

@pajuric
Copy link

pajuric commented Nov 9, 2023

@nateynateynate @dtaivpp @krisfreedain @kolchfa-aws - This blog is ready to post. Not sure which maintainers are online, but if we could update the blog with today's date and push it live, I would appreciate it.

Copy link
Member

@krisfreedain krisfreedain 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 what's happening here, but when I pull it down to run locally, for some weird reason the front page looks like this
Screenshot 2023-11-09 at 12 09 35 PM

Please investigate @ev2900 @vagimeli

@nateynateynate
Copy link
Member

Can confirm the same behavior over here. Seeing the blog content populate the homepage.

@ev2900
Copy link
Contributor Author

ev2900 commented Nov 9, 2023

@vagimeli, @nateynateynate or @krisfreedain do you know why this is happening?

I don't see any differences in the formatting and structure of this post compared to other posts.

also can we approve the pull request to add the co-authors information - #2399 to our list of author's bios ?

ev2900 and others added 4 commits November 9, 2023 15:23
Co-authored-by: Melissa Vagi <[email protected]>
Signed-off-by: Christopher Sharkey <[email protected]>
Co-authored-by: Melissa Vagi <[email protected]>
Signed-off-by: Christopher Sharkey <[email protected]>
Co-authored-by: Melissa Vagi <[email protected]>
Signed-off-by: Christopher Sharkey <[email protected]>
Co-authored-by: Melissa Vagi <[email protected]>
Signed-off-by: Christopher Sharkey <[email protected]>
@pajuric
Copy link

pajuric commented Nov 10, 2023

@nateynateynate @dtaivpp @krisfreedain - Just getting caught up here. Looks like an issue with the blog, so we're holding here?

@krisfreedain
Copy link
Member

@ev2900 @pajuric - I'm not completely positive what's going on to cause this. @kolchfa-aws - do you think it would have anything to do with making img-fluid the default for any images within #main_content (@nateynateynate wondered that, but neither of us see why it would) @astephanus - could you take a look at this one too? I'm stumped.

@kolchfa-aws
Copy link
Collaborator

@krisfreedain I'm not able to reproduce what you're seeing locally. Maybe try clearing the cache and reloading?

@pajuric
Copy link

pajuric commented Nov 13, 2023

@krisfreedain @nateynateynate @dtaivpp - If you can take another look at what went wrong while launching this blog and push it again, I would appreciate it.

@nateynateynate
Copy link
Member

I'll spend some time on this today for sure.

…ntire contents of teh blog to be used as the excerpt, spilling it all over the front page from where we feature it.

Just adding a quick excerpt to get this thing out.

Signed-off-by: Nate Boot <[email protected]>
Copy link
Member

@nateynateynate nateynateynate left a comment

Choose a reason for hiding this comment

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

After adding the excerpt it looked great. Let's ship it!

Copy link
Member

@nateynateynate nateynateynate left a comment

Choose a reason for hiding this comment

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

Ok let's ship it.

Copy link
Member

@nateynateynate nateynateynate left a comment

Choose a reason for hiding this comment

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

Hokedokey. Let's ship it.

@krisfreedain krisfreedain merged commit 2e5f41a into opensearch-project:main Nov 13, 2023
4 checks passed
@krisfreedain krisfreedain mentioned this pull request Nov 13, 2023
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants