✨ Adding Availability enum to catalog spec

[LalatenduMohanty]

No description provided.

[codecov]

Codecov Report

Attention: Patch coverage is 92.59259% with 2 lines in your changes missing coverage. Please review.

Project coverage is 38.36%. Comparing base (b1b145a) to head (a3281bf).
Report is 7 commits behind head on main.

Files with missing lines	Patch %	Lines
...rnal/controllers/core/clustercatalog_controller.go	92.59%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #421      +/-   ##
==========================================
+ Coverage   37.31%   38.36%   +1.05%     
==========================================
  Files          15       15              
  Lines         922      941      +19     
==========================================
+ Hits          344      361      +17     
- Misses        529      530       +1     
- Partials       49       50       +1     

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

[joelanford]

Looking pretty good. Can you also add some tests? I'm thinking primarily focused on unit tests, but also an e2e that sets up a catalog as "Enabled", then disables it, and checks that:

1. you get a 404 from the webserver for the content
2. all of the status fields are updated appropriately
3. the finalizer is removed

[joelanford]

Implementation looks good to me! Are you planning to add tests to this PR?

[LalatenduMohanty]

Implementation looks good to me! Are you planning to add tests to this PR?

yeah, working on that. Will remove the WIP attached to the PR once that is done.