You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@trafficserver.apache.org by "masaori335 (via GitHub)" <gi...@apache.org> on 2024/06/04 04:07:01 UTC
[PR] Add storage.yaml [trafficserver]
masaori335 opened a new pull request, #11010:
URL: https://github.com/apache/trafficserver/pull/11010
Address #10766. Implementation of merging `storage.config` and `volume.config` into `storage.yaml`. Docs will be changed by #11000.
# TODOs
- [x] Test with raw disks
- [ ] Decouple cleanups
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add storage.yaml [trafficserver]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #11010:
URL: https://github.com/apache/trafficserver/pull/11010#issuecomment-2111435273
This pull request has been automatically marked as stale because it has not had recent activity. Marking it stale to flag it for further consideration by the community.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add storage.yaml [trafficserver]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] closed pull request #11010: Add storage.yaml
URL: https://github.com/apache/trafficserver/pull/11010
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add storage.yaml [trafficserver]
Posted by "masaori335 (via GitHub)" <gi...@apache.org>.
masaori335 commented on PR #11010:
URL: https://github.com/apache/trafficserver/pull/11010#issuecomment-1943199126
1. Rebased on the latest 11-Dev branch (4274bc0328b78945cf2405b0745d5d9355b82bbc).
2. Tested with raw devices on Linux. This generated the same result (total cache size and related debug messages) as before this change.
## storage.config and volume.config without this PR
```
# storage.config
/dev/disk/by-path/AAAA volume=1
/dev/disk/by-path/BBBB volume=1
/dev/disk/by-path/CCCC volume=1
/dev/disk/by-path/DDDD volume=1
/dev/ram0 volume=2
# volume.config
volume=1 scheme=http size=100%
volume=2 scheme=http size=10485760000
```
## storege.yaml with this PR
```
# storage.yaml
cache:
spans:
- id: "disk-1"
path: /dev/disk/by-path/AAAA
- id: "disk-2"
path: /dev/disk/by-path/BBBB
- id: "disk-3"
path: /dev/disk/by-path/CCCC
- id: "disk-4"
path: /dev/disk/by-path/DDDD
- id: "ram-1"
path: /dev/ram0
volumes:
- id: 1
spans:
- use: disk-1
- use: disk-2
- use: disk-3
- use: disk-4
- id: 2
spans:
- use: ram-1
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add storage.yaml [trafficserver]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] closed pull request #11010: Add storage.yaml
URL: https://github.com/apache/trafficserver/pull/11010
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add storage.yaml [trafficserver]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] closed pull request #11010: Add storage.yaml
URL: https://github.com/apache/trafficserver/pull/11010
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add storage.yaml [trafficserver]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #11010:
URL: https://github.com/apache/trafficserver/pull/11010#issuecomment-2325464985
This pull request has been automatically marked as stale because it has not had recent activity. Marking it stale to flag it for further consideration by the community.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org