add owner tags for jobs and pods

[shubhindia]

Problem

Currently, komiser doesn't support listing cronJobs

Solution

This PR adds support for listing cronJobs and their pricing as well

Changes Made

• Added separate function to fetch cronJobs
• Fixed incorrect log line in Jobs

How to Test

komiser start --config=config.toml

Screenshots

Checklist

• Code follows the contributing guidelines
• Changes have been thoroughly tested
• Documentation has been updated, if necessary
• Any dependencies have been added to the project, if necessary

[shubhindia]

QQ here, should we have tags based on ownerReference? A cronJob can create multiple jobs and then it would be easy to filter based on owner of that job. Something like this. Same can be done for others as well like pods (most of the objects can/will have ownerReferences and it will be easier to group them by their ownerReference). Let me know if I should add it or not?

[Azanul]

opencost currently classifies jobs and cronjobs under jobs, ref: https://github.com/opencost/opencost/blob/26b5ba12ac73e612b5328969debf2056478ef0d7/pkg/costmodel/costmodel.go#L128C1-L138C1

There is no separate aggregation by cronjob.

[Azanul]

@mlabouardy What kind of coverage are we looking for regarding k8s cron jobs?

[shubhindia]

opencost currently classifies jobs and cronjobs under jobs, ref: https://github.com/opencost/opencost/blob/26b5ba12ac73e612b5328969debf2056478ef0d7/pkg/costmodel/costmodel.go#L128C1-L138C1

There is no separate aggregation by cronjob.

My bad. I didn't check the opencost price aggregation. So, I think here it makes more sense to just add owner tags than having a separate service for cronJob? Or we can continue having cronJobs without cost?

[mlabouardy]

@mlabouardy What kind of coverage are we looking for regarding k8s cron jobs?

as they belong to jobs, I would say collecting the owner tag would be enough

[Azanul]

@mlabouardy What kind of coverage are we looking for regarding k8s cron jobs?

as they belong to jobs, I would say collecting the owner tag would be enough

@shubhindia They you go

[shubhindia]

@Azanul pushed the changes. I think same can be done for pods as well. Tags can help group them together

[Azanul]

@Azanul pushed the changes. I think same can be done for pods as well. Tags can help group them together

Feel free to do the same in this PR only as I don't think it's that big of a change. Also, do update the PR title and description accordingly