feat: allow project scoped generic kubernetes secrets

[Marvin9]

This PR now allows create generic k8s secrets

TODO:

• API - Update secret
• Tests

[netlify]

✅ Deploy Preview for docs-kargo-io ready!

Name	Link
🔨 Latest commit	793f29e
🔍 Latest deploy log	https://app.netlify.com/sites/docs-kargo-io/deploys/67821c65eb43fd00080a7ec7
😎 Deploy Preview	https://deploy-preview-2975.docs.kargo.io
📱 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.

[codecov]

Codecov Report

Attention: Patch coverage is 50.98814% with 124 lines in your changes missing coverage. Please review.

Project coverage is 52.05%. Comparing base (6dfa01e) to head (793f29e).
Report is 11 commits behind head on main.

Files with missing lines	Patch %	Lines
internal/api/update_project_secret_v1alpha1.go	53.94%	27 Missing and 8 partials ⚠️
internal/api/delete_project_secret_v1alpha1.go	30.23%	23 Missing and 7 partials ⚠️
internal/api/create_project_secret_v1alpha1.go	67.79%	13 Missing and 6 partials ⚠️
internal/api/delete_credentials_v1alpha1.go	0.00%	19 Missing ⚠️
internal/api/list_project_secrets_v1alpha1.go	63.82%	12 Missing and 5 partials ⚠️
internal/api/update_credentials_v1alpha1.go	0.00%	4 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2975      +/-   ##
==========================================
+ Coverage   51.40%   52.05%   +0.64%     
==========================================
  Files         288      295       +7     
  Lines       26108    26695     +587     
==========================================
+ Hits        13421    13895     +474     
- Misses      11961    12041      +80     
- Partials      726      759      +33     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Marvin9]

@krancour generic secret creation and listing those is ready. Till I figure out what to do for updates could you start looking at the backend code?

Also do you think its better to re-use the create resource API instead of modifying the create credentials?

[Marvin9]

@krancour could you start looking.. I will bring code coverage to threshold

[krancour]

Sorry this took me so long.

Overall, this looks fantastic. Test drive went really smoothly.

I have some small bits of feedback, that I think are easy to address.

In the screenshot above, I was initially thrown off by not seeing any kind of "Add Secret" button. It took me a while to realize that clicking "Add Credentials" would pop up a dialog that could be used for adding generic Secrets as well.

I think this could potentially be less confusing if each of the two sections of this page each got their own "Add Credentials" and "Add Secret" buttons.

In the same screen shot, upper right, the alt text for the tab says "Credentials". With Secrets being more broad/generic than credentials, I would suggest making the alt text there "Secrets." Credentials are a kind of Secret. (If it's not a problem, I think the URL of the page could also change to end in "/secrets".

Two things tripped me up in the screenshot above.

The title of the dialog is "Edit Credentials" even when I'm working on a generic Secret. Could it be at all practical to have two separate dialogs? One for creds and one for generic?

When editing an existing Secret, not seeing any ****** in the value fields left me wondering whether I was going to accidentally blank out those fields when I hit Update. I confirmed that this is not a problem, but seeing ****** in those fields would have stopped me from wondering.

Last -- a weird one. If I try to add a new key "foobar" above, the new line disappears the moment I hit the second "o" in "foobar". I assume there's some real time validation that keys aren't being duplicated and after hitting that second "o", it looks like I am duplicating "foo" although I wasn't done typing. If I type any other value not staring with the characters of an existing key, everything works fine.

I think/hope these are all relatively minor tweaks. As I said, overall, this worked great and did exactly what it's supposed to.

[Marvin9]

Great suggestions! @krancour you might want to take a look now, except your last point (in progress) all other review points are addressed. Thanks

[krancour]

@Marvin9 I promise I'll 👀 the latest changes tomorrow. Thank you for the hard work on this.