Tomas Barton

Results 165 comments of Tomas Barton

@tskirvin It does work :smiley: Just the `fmt` function seems to be limited to Puppet >=8 - somehow deferred unwrap doesn't work well with Puppet 7.

Yes, that would be great. There are [quite a few changes already](https://github.com/puppetlabs/puppetlabs-kubernetes/compare/v8.0.0...main) and few more PR for next major release. cc @Ramesh7 @GSPatton @david22swan

@oxzi PostgreSQL limits the maximum query size `track_activity_query_size`, so I was unable to get the full SQL. I'll try to increase that. ``` table_name | table_size | indexes_size | total_size...

> Your history has become quite large. I don't want to put the blame on you, I just want to state the fact and that there are potentially performance issues...

@lippserd it would produce following query plan: ``` Nested Loop Left Join (cost=200919.74..3242886.88 rows=10 width=5550) (actual time=4790.756..12811.623 rows=26 loops=1) -> Nested Loop Left Join (cost=200919.18..3242879.66 rows=10 width=4399) (actual time=4790.747..12806.814 rows=26...

@bastelfreak I think bumping from `4` -> `8` is the right move right now. It would be nice to prepare at least a minor version with all those changes. Moving...

The lower stdlib version change should have been done in https://github.com/puppetlabs/puppetlabs-kubernetes/commit/9f9f0ebdd4933a47056632b78763347909edea59, that is part of `v8.0.0` release. It makes sense to fix that issue in `v8.x` branch. However, if you...

@hfrog Could you please rebase the PR?

Sure, the server was overloaded I have upgrade it. However `icingadb` shouldn't be crashing... ``` Thu 2024-09-12 17:03:24 UTC icingadb[1121651]: runtime-updates: Upserted 10 ServiceState items Thu 2024-09-12 17:03:39 UTC icingadb[1121651]:...

I'm probably having the same issue with 2 `icingadb` instances and PostgreSQL backend. @oxzi Do you need more verbose logs? 1st instance ``` Nov 19 07:41:25 icingadb[2362535]: retention: Removed 5450...