Re: [PATCH 1/2] aws: add dynamic cloud configuration support using AWS CLI

On 9/7/25 6:10 PM, Luis Chamberlain wrote:
> On Sun, Sep 07, 2025 at 01:24:43PM -0400, Chuck Lever wrote:
>> On 9/7/25 12:23 AM, Luis Chamberlain wrote:
>>> +3. **Commit the static files**:
>>> +   ```bash
>>> +   git add terraform/*/kconfigs/*.static
>>> +   git add terraform/*/kconfigs/instance-types/*.static
>>> +   git commit -m "cloud: update static configurations for AWS/Azure/GCE
>>> +
>>> +   Update instance types, regions, and AMI options to current offerings.
>>> +
>>> +   Generated with AWS CLI version X.Y.Z on YYYY-MM-DD."
>>> +   git push
>>> +   ```
>>
>> Thanks, this is very helpful.
> 
> Great, glad it was. BTW just one request, any chance you can trim
> context over your replies, like I did here? Otherwise it becomes
> a bit hard to scroll down on longer patches for your reviews.

That's generally a community preference - some communities I'm
involved with prefer that the context be preserved. That certainly
does become cumbersome with large patches.

I will trim more aggressively on kdevops@.


> The other reason this is useful is it can be useful to
> reduce context window for AIs to process. Although I'm not an AI,
> I sometimes test AIs to try to evaluate how well they can process
> replies to adjust patches. In the future I hope this to be a standard
> test.

I haven't tried to feed an email thread back into an LLM, so I
might miss the mark. But, this makes sense.

-- 
Chuck Lever