Skip to content

change(debug): move 'enable_debug' form config.yaml to debug.yaml#5046

Merged
spacewander merged 5 commits into
apache:masterfrom
tzssangglass:mv_ed
Sep 14, 2021
Merged

change(debug): move 'enable_debug' form config.yaml to debug.yaml#5046
spacewander merged 5 commits into
apache:masterfrom
tzssangglass:mv_ed

Conversation

@tzssangglass

Copy link
Copy Markdown
Member

What this PR does / why we need it:

move 'enable_debug' form config.yaml to debug.yaml, make the configuration of debug mode clear and central

Pre-submission checklist:

  • Did you explain what problem does this PR solve? Or what new features have been added?
  • Have you added corresponding test cases?
  • Have you modified the corresponding document?
  • Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

### 基本调试模式

设置 `conf/config.yaml` 中的 `apisix.enable_debug` 为 `true`,即可开启基本调试模式。
设置 `conf/debug.yaml` 中的 `enable_debug` 为 `true`,即可开启基本调试模式。

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hi, for those who using apisix versions before 2.9.0, this document may be confused

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We can add a note for the version before 2.10.0?

Comment thread conf/debug.yaml Outdated
# See the License for the specific language governing permissions and
# limitations under the License.
#
enable_debug: false

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
enable_debug: false
basic:
enable: false

would be better?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Looks better, but this enable will conflict with

$debug_config =~ s/enable: false/enable: true/;

I don't know how to modify this with perl.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Let me do it for you 😃

spacewander and others added 2 commits September 14, 2021 11:31
Signed-off-by: spacewander <[email protected]>
@spacewander spacewander changed the title chore(debug-mode): move 'enable_debug' form config.yaml to debug.yaml change(debug): move 'enable_debug' form config.yaml to debug.yaml Sep 14, 2021
@spacewander
spacewander merged commit b5aca8c into apache:master Sep 14, 2021
@tzssangglass
tzssangglass deleted the mv_ed branch October 26, 2021 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants