spack application added

[Rupam-It]

spark application add

📚 Description of Changes

Add in the interface about spark application !

Summary by CodeRabbit

• New Features
  • Added support for managing Spark Applications and Scheduled Spark Applications as new resource types.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

Added two new ResourceType enum values (SparkApplication and ScheduledSparkApplication) to the resource type collection system, with corresponding mappings in String() and ProtoType() methods to support protobuf conversion and string representation.

Changes

Cohort / File(s)	Summary
ResourceType Enum Extension internal/collector/interface.go	Added SparkApplication and ScheduledSparkApplication enum values; extended String() and ProtoType() methods with mappings for new resource types to "spark_application", "scheduled_spark_application" and corresponding protobuf enum constants

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• container update changes were being reported as type unspecified #212: Modifies the same ResourceType enum in internal/collector/interface.go and updates String()/ProtoType() method mappings

Suggested reviewers

• dray92

Poem

🐰 Spark applications now shine so bright,
Two new types added to our collector's sight,
SparkApplication and ScheduledSparkApplication too,
Protobuf mappings making dreams come true! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The title 'spack application added' appears to be a typo for 'spark application added' based on the PR description and actual changes which add SparkApplication and ScheduledSparkApplication enum values.	Correct the title to 'spark application added' or 'Add Spark application resource types' to accurately reflect the changes being introduced.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}