docs: keptn Scheduler architecture documentation

[YashPimple]

Relate: #770
Fixes: #1682

signed-off-by: YashPimple yashpimple22@gmail.com

[netlify]

✅ Deploy Preview for keptn-lifecycle-toolkit ready!

Name	Link
🔨 Latest commit	f27644b
🔍 Latest deploy log	https://app.netlify.com/sites/keptn-lifecycle-toolkit/deploys/64e4f90173bd9000086b6184
😎 Deploy Preview	https://deploy-preview-1777--keptn-lifecycle-toolkit.netlify.app/docs/concepts/architecture/components/scheduler
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

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

[RealAnna]

Hi @YashPimple thankyou for your contribution! I added a few suggestions, the main changes needed regards the scheduling logic. The keptn scheduler implements a plugin of the default k8s scheduler, this means that it just adds our own custom logic to one step of the default scheduler logic where we wait for a specific evaluation CRD to be successful before binding pods to nodes. I would not specify in this document how all other parts work, since this would overwhelm the reader.

[RealAnna]

Commented above

[StackScribe]

Nice start, Yash, but I think this includes a lot of information that is already present in the rest of the documentation and we don't want to be needlessly duplicating information -- it becomes a maintenance headache.

This should probably be a shorter piece that talks about how the scheduler works "under the hood", as it were. It can reference other parts of the docs that have details about the resources and some how-to instructions but should basically talk about how the scheduler processes what it finds.

[YashPimple]

Nice start, Yash, but I think this includes a lot of information that is already present in the rest of the documentation and we don't want to be needlessly duplicating information -- it becomes a maintenance headache.

This should probably be a shorter piece that talks about how the scheduler works "under the hood", as it were. It can reference other parts of the docs that have details about the resources and some how-to instructions but should basically talk about how the scheduler processes what it finds.

Thanks for your feedback! Will keep it concise and focus on the scheduler's inner workings also avoid duplication . Looking forward to crafting a more streamlined and informative piece.
Thanks again for the guidance! 🙌

[YashPimple]

Hi @YashPimple thankyou for your contribution! I added a few suggestions, the main changes needed regards the scheduling logic. The keptn scheduler implements a plugin of the default k8s scheduler, this means that it just adds our own custom logic to one step of the default scheduler logic where we wait for a specific evaluation CRD to be successful before binding pods to nodes. I would not specify in this document how all other parts work, since this would overwhelm the reader.

Thank you @RealAnna for your valuable feedback! I will prioritize refining the scheduling logic for the keptn scheduler plugin, while also ensuring to maintain a clear and concise explanation👍🏻

[RealAnna]

Hey @YashPimple this is great 🚀 I left a few nitpicks on some things, but I think we can merge soon 😃

[StackScribe]

Good job, Yash!

[YashPimple]

@RealAnna @StackScribe I have resolved the changes as per suggested in the Scheduler docs. Do have a look 👍🏻

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

adapted according to suggestions