tektoncd / website

Tekton Website
https://tekton.dev
Apache License 2.0
62 stars 149 forks source link

blog(add article): tekton + tracetest #557

Closed adnanrahic closed 9 months ago

adnanrahic commented 10 months ago

Changes

This PR adds a new blog post for using distributed tracing for testing with Tracetest and automating tests with Tekton. Based on this docs page in Tracetest docs.

Submitter Checklist

These are the criteria that every PR should meet, please check them off as you review them:

See the contribution guide for more details.

linux-foundation-easycla[bot] commented 10 months ago

CLA Signed

The committers listed above are authorized under a signed CLA.

afrittoli commented 10 months ago

@tektoncd/governing-board @AlanGreene @geriom can anyone look into this blog post? I will be on PTO until September and didn't want to delay this until then!

adnanrahic commented 10 months ago

Thanks for responding @afrittoli !

I've set the date of the blog post to August 15th. LMK if we need to change that to something else. πŸ˜„

adnanrahic commented 10 months ago

Hey @AlanGreene @geriom ! Wanted to touch base and ask if tomorrow is a good date to publish the blog post. If not, let me know and we can work out another date. πŸ˜„

adnanrahic commented 10 months ago

Thanks for the review @geriom ! I just added a commit with edits per your suggestions. πŸ™Œ

tekton-robot commented 10 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: geriom

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/tektoncd/website/blob/main/OWNERS)~~ [geriom] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
adnanrahic commented 10 months ago

Hey @geriom πŸ‘‹ Thanks for approving the PR! Wanted to ask when this could be merged? I am not authorized to merge it myself.

geriom commented 10 months ago

Hey @geriom πŸ‘‹ Thanks for approving the PR! Wanted to ask when this could be merged? I am not authorized to merge it myself.

I just pinged Alan on slack in case he has some time to take a look. PRs usually need two approvals to be merged.

adnanrahic commented 10 months ago

Thanks @geriom! Got it. Looking forward to @AlanGreene's review as well. πŸ™Œ

adnanrahic commented 10 months ago

Hey @geriom & @AlanGreene ! πŸ‘‹

I wanted to circle back to this and ping you to ask if we should update the blog post date and get it merged this week.

Alternatively, I'm happy to wait for @afrittoli to get back from PTO.

Thanks! πŸ™Œ