RedisCommander should be child of Redis

### Is there an existing issue for this?

- [x] I have searched the existing issues

### Is your feature request related to a problem? Please describe the problem.

When using `builder.AddRedis().AddRedisCommander()`, a relationship is defined between them. To me, it makes the most sense for the `rediscommander` resource to be a _child_ of the `redis` resource, since it depends on the `redis` resource.

### Describe the solution you'd like

I think the only thing that needs to be changed is the following line:
https://github.com/dotnet/aspire/blob/5d1f4afdea2b42063cd4d8f736f2c5169383d4df/src/Aspire.Hosting.Redis/RedisBuilderExtensions.cs#L291

### Additional context

_No response_