Skip to content

Conversation

@nqyy
Copy link
Contributor

@nqyy nqyy commented Sep 12, 2018

No description provided.

@randall
Copy link
Contributor

randall commented Sep 12, 2018

[approve ci autest]

Advanced
--------

When ``runroot`` generated from ``traffic_layout`` is used, :file:`storage.config` should be updated
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"updated accordingly pointing"?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I need to polish the grammar a bit.

@nqyy nqyy force-pushed the doc_upda branch 2 times, most recently from 2a456f4 to 2e99ad1 Compare September 13, 2018 15:25
Advanced
--------

When ``runroot`` generated from ``traffic_layout`` is used, :file:`storage.config` should be updated.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Because relative paths in :filestorage.config are relative to the base prefix, when using runroot it may be necessary to adjust such paths in :file:storage.config. Despite the name, the cachedir value is not used for this file."


.. Note::

Please update the directory for cache in ``storage.config`` according to the customized runroot.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

":file:storage.config does not use the cachedir value, but makes its relatives paths relative to the base prefix".

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Describe mechanism, not policy. Explain what TS does, try to avoid telling admins what they should do.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure.

@dragon512 dragon512 added this to the 9.0.0 milestone Sep 13, 2018
@dragon512 dragon512 merged commit d81630b into apache:master Sep 13, 2018
@nqyy nqyy deleted the doc_upda branch September 21, 2018 18:14
@bryancall bryancall modified the milestones: 9.0.0, 8.0.1 Oct 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants