server-tools [16.0][IMP] module_change_auto_install: also allow using environment variables

Jan 18 '24 15:01 vincent-hatakeyama

Hi @legalsylvain, some modules you are maintaining are being modified, check this out!

Jan 18 '24 15:01 OCA-git-bot

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

Apr 19 '24 13:04 OCA-git-bot

I added information on how to use the environment variables too.

May 13 '24 13:05 vincent-hatakeyama

Thanks !

/ocabot merge minor

May 13 '24 18:05 legalsylvain

What a great day to merge this nice PR. Let's do it! Prepared branch 16.0-ocabot-merge-pr-2811-by-legalsylvain-bump-minor, awaiting test results.

May 13 '24 18:05 OCA-git-bot

@legalsylvain The merge process could not be finalized, because command oca-gen-addon-readme --if-source-changed --org-name OCA --repo-name server-tools --branch 16.0 --addons-dir /tmp/tmpia9n936m --commit failed with output:

/tmp/tmpia9n936m/module_change_auto_install/README.rst:82: (ERROR/3) Error in "code-block" directive:
unknown option: "emphasize-lines".

.. code-block:: cfg
   :emphasize-lines: 1

    server_wide_modules = web,module_change_auto_install

    modules_auto_install_disabled =
        partner_autocomplete,
        iap,
        mail_bot

    modules_auto_install_enabled =
        web_responsive:web,
        base_technical_features,
        disable_odoo_online,
        account_usability

Traceback (most recent call last):
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 2142, in run_directive
    self.parse_directive_block(indented, line_offset,
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 2194, in parse_directive_block
    options, arg_block = self.parse_directive_options(
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 2230, in parse_directive_options
    raise MarkupError(data)
docutils.parsers.rst.states.MarkupError: unknown option: "emphasize-lines"

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/bin/oca-gen-addon-readme", line 11, in <module>
    load_entry_point('oca-maintainers-tools', 'console_scripts', 'oca-gen-addon-readme')()
  File "/ocamt/lib/python3.8/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/ocamt/lib/python3.8/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
  File "/ocamt/lib/python3.8/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/ocamt/lib/python3.8/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/ocamt/src/oca-maintainers-tools/tools/gen_addon_readme.py", line 575, in gen_addon_readme
    check_rst(readme_filename)
  File "/ocamt/src/oca-maintainers-tools/tools/gen_addon_readme.py", line 398, in check_rst
    publish_file(
  File "/ocamt/lib/python3.8/site-packages/docutils/core.py", line 422, in publish_file
    output, publisher = publish_programmatically(
  File "/ocamt/lib/python3.8/site-packages/docutils/core.py", line 722, in publish_programmatically
    output = publisher.publish(enable_exit_status=enable_exit_status)
  File "/ocamt/lib/python3.8/site-packages/docutils/core.py", line 234, in publish
    self.document = self.reader.read(self.source, self.parser,
  File "/ocamt/lib/python3.8/site-packages/docutils/readers/__init__.py", line 70, in read
    self.parse()
  File "/ocamt/lib/python3.8/site-packages/docutils/readers/__init__.py", line 76, in parse
    self.parser.parse(self.input, document)
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/__init__.py", line 184, in parse
    self.statemachine.run(inputlines, document, inliner=self.inliner)
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 169, in run
    results = StateMachineWS.run(self, input_lines, input_offset,
  File "/ocamt/lib/python3.8/site-packages/docutils/statemachine.py", line 233, in run
    context, next_state, result = self.check_line(
  File "/ocamt/lib/python3.8/site-packages/docutils/statemachine.py", line 445, in check_line
    return method(match, context, next_state)
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 3024, in text
    self.section(title.lstrip(), source, style, lineno + 1, messages)
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 325, in section
    self.new_subsection(title, lineno, messages)
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 391, in new_subsection
    newabsoffset = self.nested_parse(
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 279, in nested_parse
    state_machine.run(block, input_offset, memo=self.memo,
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 195, in run
    results = StateMachineWS.run(self, input_lines, input_offset)
  File "/ocamt/lib/python3.8/site-packages/docutils/statemachine.py", line 233, in run
    context, next_state, result = self.check_line(
  File "/ocamt/lib/python3.8/site-packages/docutils/statemachine.py", line 445, in check_line
    return method(match, context, next_state)
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 2785, in underline
    self.section(title, source, style, lineno - 1, messages)
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 325, in section
    self.new_subsection(title, lineno, messages)
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 391, in new_subsection
    newabsoffset = self.nested_parse(
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 279, in nested_parse
    state_machine.run(block, input_offset, memo=self.memo,
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 195, in run
    results = StateMachineWS.run(self, input_lines, input_offset)
  File "/ocamt/lib/python3.8/site-packages/docutils/statemachine.py", line 233, in run
    context, next_state, result = self.check_line(
  File "/ocamt/lib/python3.8/site-packages/docutils/statemachine.py", line 445, in check_line
    return method(match, context, next_state)
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 2355, in explicit_markup
    nodelist, blank_finish = self.explicit_construct(match)
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 2367, in explicit_construct
    return method(self, expmatch)
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 2104, in directive
    return self.run_directive(
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 2145, in run_directive
    error = self.reporter.error(
  File "/ocamt/lib/python3.8/site-packages/docutils/utils/__init__.py", line 231, in error
    return self.system_message(self.ERROR_LEVEL, *args, **kwargs)
  File "/ocamt/lib/python3.8/site-packages/docutils/utils/__init__.py", line 197, in system_message
    raise SystemMessage(msg, level)
docutils.utils.SystemMessage: /tmp/tmpia9n936m/module_change_auto_install/README.rst:82: (ERROR/3) Error in "code-block" directive:
unknown option: "emphasize-lines".

.. code-block:: cfg
   :emphasize-lines: 1

    server_wide_modules = web,module_change_auto_install

    modules_auto_install_disabled =
        partner_autocomplete,
        iap,
        mail_bot

    modules_auto_install_enabled =
        web_responsive:web,
        base_technical_features,
        disable_odoo_online,
        account_usability

May 13 '24 18:05 OCA-git-bot

@vincent-hatakeyama could you take a look ?

May 13 '24 19:05 legalsylvain

@vincent-hatakeyama could you take a look ?

I’ve remove the rst that seems to cause trouble.

May 17 '24 09:05 vincent-hatakeyama

On my way to merge this fine PR! Prepared branch 16.0-ocabot-merge-pr-2811-by-legalsylvain-bump-minor, awaiting test results.

May 17 '24 09:05 OCA-git-bot

@legalsylvain your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-2811-by-legalsylvain-bump-minor.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

May 17 '24 09:05 OCA-git-bot

The tests in error are unrelated, they are from module attachment_synchronize.

May 17 '24 10:05 vincent-hatakeyama

@vincent-hatakeyama : waiting for https://github.com/OCA/storage/pull/372

May 17 '24 12:05 legalsylvain

@vincent-hatakeyama : waiting for OCA/storage#372

@legalsylvain it is merged

May 28 '24 14:05 SirAionTech

Thanks !

/ocabot merge patch

May 28 '24 14:05 legalsylvain

This PR looks fantastic, let's merge it! Prepared branch 16.0-ocabot-merge-pr-2811-by-legalsylvain-bump-patch, awaiting test results.

May 28 '24 14:05 OCA-git-bot

@legalsylvain The merge process could not be finalized, because command oca-gen-addon-readme --if-source-changed --org-name OCA --repo-name server-tools --branch 16.0 --addons-dir /tmp/tmpg_pk2nem --commit failed with output:

/tmp/tmpg_pk2nem/module_change_auto_install/README.rst:82: (ERROR/3) Error in "code-block" directive:
unknown option: "emphasize-lines".

.. code-block:: cfg
   :emphasize-lines: 1

    server_wide_modules = web,module_change_auto_install

    modules_auto_install_disabled =
        partner_autocomplete,
        iap,
        mail_bot

    modules_auto_install_enabled =
        web_responsive:web,
        base_technical_features,
        disable_odoo_online,
        account_usability

Traceback (most recent call last):
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 2142, in run_directive
    self.parse_directive_block(indented, line_offset,
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 2194, in parse_directive_block
    options, arg_block = self.parse_directive_options(
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 2230, in parse_directive_options
    raise MarkupError(data)
docutils.parsers.rst.states.MarkupError: unknown option: "emphasize-lines"

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/bin/oca-gen-addon-readme", line 11, in <module>
    load_entry_point('oca-maintainers-tools', 'console_scripts', 'oca-gen-addon-readme')()
  File "/ocamt/lib/python3.8/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/ocamt/lib/python3.8/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
  File "/ocamt/lib/python3.8/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/ocamt/lib/python3.8/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/ocamt/src/oca-maintainers-tools/tools/gen_addon_readme.py", line 575, in gen_addon_readme
    check_rst(readme_filename)
  File "/ocamt/src/oca-maintainers-tools/tools/gen_addon_readme.py", line 398, in check_rst
    publish_file(
  File "/ocamt/lib/python3.8/site-packages/docutils/core.py", line 422, in publish_file
    output, publisher = publish_programmatically(
  File "/ocamt/lib/python3.8/site-packages/docutils/core.py", line 722, in publish_programmatically
    output = publisher.publish(enable_exit_status=enable_exit_status)
  File "/ocamt/lib/python3.8/site-packages/docutils/core.py", line 234, in publish
    self.document = self.reader.read(self.source, self.parser,
  File "/ocamt/lib/python3.8/site-packages/docutils/readers/__init__.py", line 70, in read
    self.parse()
  File "/ocamt/lib/python3.8/site-packages/docutils/readers/__init__.py", line 76, in parse
    self.parser.parse(self.input, document)
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/__init__.py", line 184, in parse
    self.statemachine.run(inputlines, document, inliner=self.inliner)
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 169, in run
    results = StateMachineWS.run(self, input_lines, input_offset,
  File "/ocamt/lib/python3.8/site-packages/docutils/statemachine.py", line 233, in run
    context, next_state, result = self.check_line(
  File "/ocamt/lib/python3.8/site-packages/docutils/statemachine.py", line 445, in check_line
    return method(match, context, next_state)
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 3024, in text
    self.section(title.lstrip(), source, style, lineno + 1, messages)
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 325, in section
    self.new_subsection(title, lineno, messages)
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 391, in new_subsection
    newabsoffset = self.nested_parse(
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 279, in nested_parse
    state_machine.run(block, input_offset, memo=self.memo,
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 195, in run
    results = StateMachineWS.run(self, input_lines, input_offset)
  File "/ocamt/lib/python3.8/site-packages/docutils/statemachine.py", line 233, in run
    context, next_state, result = self.check_line(
  File "/ocamt/lib/python3.8/site-packages/docutils/statemachine.py", line 445, in check_line
    return method(match, context, next_state)
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 2785, in underline
    self.section(title, source, style, lineno - 1, messages)
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 325, in section
    self.new_subsection(title, lineno, messages)
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 391, in new_subsection
    newabsoffset = self.nested_parse(
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 279, in nested_parse
    state_machine.run(block, input_offset, memo=self.memo,
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 195, in run
    results = StateMachineWS.run(self, input_lines, input_offset)
  File "/ocamt/lib/python3.8/site-packages/docutils/statemachine.py", line 233, in run
    context, next_state, result = self.check_line(
  File "/ocamt/lib/python3.8/site-packages/docutils/statemachine.py", line 445, in check_line
    return method(match, context, next_state)
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 2355, in explicit_markup
    nodelist, blank_finish = self.explicit_construct(match)
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 2367, in explicit_construct
    return method(self, expmatch)
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 2104, in directive
    return self.run_directive(
  File "/ocamt/lib/python3.8/site-packages/docutils/parsers/rst/states.py", line 2145, in run_directive
    error = self.reporter.error(
  File "/ocamt/lib/python3.8/site-packages/docutils/utils/__init__.py", line 231, in error
    return self.system_message(self.ERROR_LEVEL, *args, **kwargs)
  File "/ocamt/lib/python3.8/site-packages/docutils/utils/__init__.py", line 197, in system_message
    raise SystemMessage(msg, level)
docutils.utils.SystemMessage: /tmp/tmpg_pk2nem/module_change_auto_install/README.rst:82: (ERROR/3) Error in "code-block" directive:
unknown option: "emphasize-lines".

.. code-block:: cfg
   :emphasize-lines: 1

    server_wide_modules = web,module_change_auto_install

    modules_auto_install_disabled =
        partner_autocomplete,
        iap,
        mail_bot

    modules_auto_install_enabled =
        web_responsive:web,
        base_technical_features,
        disable_odoo_online,
        account_usability

May 28 '24 14:05 OCA-git-bot

/ocabot rebase

May 28 '24 15:05 legalsylvain

@legalsylvain The rebase process failed, because command git push --force xcgd tmp-pr-2811:16.0-module_change_auto_install-add-env-var failed with output:

remote: Permission to xcgd/server-tools.git denied to OCA-git-bot.
fatal: unable to access 'https://github.com/xcgd/server-tools/': The requested URL returned error: 403

May 28 '24 15:05 OCA-git-bot

@vincent-hatakeyama could you rebase ?

May 28 '24 15:05 legalsylvain

@vincent-hatakeyama could you rebase ?

I’ve rebased after removing the line emphasis.

May 28 '24 15:05 vincent-hatakeyama

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

May 28 '24 15:05 OCA-git-bot

/ocabot merge patch

May 28 '24 18:05 legalsylvain

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-2811-by-legalsylvain-bump-patch, awaiting test results.

May 28 '24 18:05 OCA-git-bot

Congratulations, your PR was merged at 311fe7e353ddde9ac7cbcd1820e642ebb449e6ce. Thanks a lot for contributing to OCA. ❤️

May 28 '24 19:05 OCA-git-bot