Error handling per D4152
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
All Stories
Sun, Aug 9
- Request and review by Qwen3.8-Max
- Code change by GPT-5.6 Sol (High)
Issue reported by Qwen3.8-Max.
Patch by GPT 5.6-Sol.
Review by Qwen3.8-Max.
Local workstation configuration to test this change with a real site to deploy: https://github.com/dereckson/wynter-salt-workstation/commit/cb84af6986ebbfba633043d362543436f568884d
Sat, Aug 8
Fri, Aug 7
Thu, Aug 6
Qwen3.8-Max insights: https://chat.qwen.ai/s/87951950-46a3-4e64-a2af-60f51458a089?fev=0.2.82
Wed, Aug 5
Models used:
- GPT 5.5 (Codex, from JetBrains PyCharm) for the schema generation and the tests
- Qwen 3.8 for code review
I wonder if T1681 couldn't host them too. If so:
Patch written to test Codex. Model: GPT-5.5 (High).
Remove unused uids
Tue, Aug 4
Improve comment
Actually rewrite the file too
Avoid too many blank lines in Markdown rendering
Improve description
Mon, Aug 3
Wed, Jul 29
Revert to the cnf.sls context-variable version per dereckson: easier to see which values are configurable/pillar-dependant in roles/*.sls than in configuration templates.
Tue, Jul 28
No please revert back to the version with innodb_buffer_pool_size = {{ innodb_buffer_pool_size }}.
Address dereckson feedback: drop the innodb_buffer_pool_size context var in cnf.sls and access pillar["dbserver_mysql"]["server"]["innodb_buffer_pool_size"] directly in server.cnf instead, so it fails loudly if the server pillar block is missing rather than silently falling back via pillar.get.
Sun, Jul 26
That will be useful to implement a bunch of values for T2337
I confirm we probably don't need it on WindRiver right now.
Tested, works fine.