[Bug]: Minion with file_client: local crashes on connect: 'AsyncPillar' object has no attribute 'aclose'

オープン 初心者向け
#70,637 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 3 日以内に返信

@shawnrruss がすでに取り組んでいます。

2026年11月3日 から。

  • #70653 @shawnrruss による — オープン

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
72/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
python
領域
devops

調査の方向性

クラッシュは、salt/minion.py の _post_master_init (2086行目付近) 内の aclose() で発生する AttributeError です。同じ呼び出しが、pillar 更新処理の finally: ブロック (4356行目付近) にもあります。まず salt/pillar/init.py の get_async_pillar() を読み、AsyncRemotePillar ではなく AsyncPillar を返すのがどのような場合かを確認してください。その後、両クラスのクリーンアップメソッドを比較してください。file_client: local の minion が接続して master 経由で test.ping に応答し、pillar 更新処理がエラーなく実行されれば完了です。既存の pillar と minion のユニットテストを実行し、回帰がないことを確認してください。

索引モデルが issue の本文から書いたものです。

説明

bug needs-triage
What happened?

A minion that connects to a master and has file_client: local set fails on every connection attempt after upgrading to 3008.5. It never becomes available to the master, so salt <minion> test.ping gets no response. salt-call on the same host still works.

Config (in /etc/salt/minion.d/):

master: 127.0.0.1
file_client: local

Log, on every start of the minion:

[CRITICAL] Unexpected error while connecting to 127.0.0.1
Traceback (most recent call last):
  File ".../salt/minion.py", line 1667, in _connect_minion
    await minion.connect_master(failed=failed)
  File ".../salt/minion.py", line 2048, in connect_master
    await self._post_master_init(master)
  File ".../salt/minion.py", line 2086, in _post_master_init
    await async_pillar.aclose()
AttributeError: 'AsyncPillar' object has no attribute 'aclose'
[ERROR   ] Minion unable to successfully connect to a Salt Master.

Cause: salt.pillar.get_async_pillar() returns AsyncPillar when file_client is local, unless use_master_when_local is set. minion.py now calls await async_pillar.aclose() on the returned object, but only AsyncRemotePillar defines aclose(). Minions with file_client: remote are not affected: four of them on 3008.5 work normally on the same master.

There are two call sites in minion.py: _post_master_init (line 2086 in 3008.5, the crash above) and the pillar refresh path (line 4356, in a finally:). I only observed the first, because the minion never got past it.

Versions: from the PyPI sdists, 3008.0 to 3008.2 call async_pillar.destroy() at both sites, and 3008.3 to 3008.5 call aclose(). So this looks like a regression in 3008.3, probably from #70316, which a comment at that call site refers to. The master and 3008.x branches still had both calls and no AsyncPillar.aclose on 2026-11-02. I ran only 3008.5.

Expected: the minion connects as it did on 3008.2.

Fix that worked here: add aclose() to AsyncPillar in salt/pillar/__init__.py, which covers both call sites:

class AsyncPillar(Pillar):
    async def compile_pillar(self, ext=True):
        ret = super().compile_pillar(ext=ext)
        return ret

    async def aclose(self):
        self.destroy()

With this applied and the minion restarted, test.version, saltutil.refresh_pillar wait=True and state.sls ... test=True all succeed through the master, with no errors in the minion log. I have not run Salt's test suite against it.

Type of salt install

Official deb

Major version

3008.x

What supported OS are you seeing the problem on? Can select multiple. (If bug appears on an unsupported OS, please open a GitHub Discussion instead)

ubuntu-24.04

salt --versions-report output
Salt Version:
          Salt: 3008.5

Python Version:
        Python: 3.14.7 (main, Sep 30 2026, 03:35:11) [GCC 11.2.0]

Dependency Versions:
          cffi: 2.0.0
      cherrypy: 18.10.0
  cryptography: 50.0.1
      dateutil: 2.9.0.post0
     docker-py: Not Installed
         gitdb: 4.0.12
     gitpython: 3.2.0
        Jinja2: 3.1.6
       libgit2: Not Installed
  looseversion: 1.3.0
      M2Crypto: Not Installed
          Mako: Not Installed
       msgpack: 1.2.1
  msgpack-pure: Not Installed
  mysql-python: Not Installed
     packaging: 24.0
     pycparser: 3.00
      pycrypto: Not Installed
  pycryptodome: 3.23.0
        pygit2: Not Installed
  python-gnupg: 0.5.6
        PyYAML: 6.0.3
         PyZMQ: 27.1.0
        relenv: 0.22.27
         smmap: 5.0.2
       timelib: 0.3.0
       Tornado: 6.5.10
           ZMQ: 4.3.5

Salt Package Information:
  Package Type: onedir

System Versions:
          dist: ubuntu 24.04.5 noble
        locale: utf-8
       machine: x86_64
       release: 6.8.0-147-generic
        system: Linux
       version: Ubuntu 24.04.5 noble
主要言語
Python
スター
15.8k
フォーク
5.6k
平均マージ
5日 9時間
マージ済み PR(30日)
99

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

saltstack/salt のほかの issue

saltstack/salt の issue をすべて見る

似ている issue

Python の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。