foreman_remote_execution 17.2.3 → 18.0.0

data/locale/ja/foreman_remote_execution.po

@@ -195,7 +195,7 @@ msgid "Cancel job invocation or matching tasks only"
 msgstr "ジョブ呼び出しまたは一致するタスクのみをキャンセルします"
 
 msgid "Cancel recurring"
-msgstr "繰り返しをキャンセルする"
+msgstr "繰り返しのキャンセル"
 
 msgid "Cancel selected"
 msgstr "選択項目をキャンセル"
@@ -378,13 +378,13 @@ msgid "Default SSH password"
 msgstr "デフォルトの SSH パスワード"
 
 msgid "Default key passphrase to use for SSH. You may override per host by setting a parameter called remote_execution_ssh_key_passphrase"
-msgstr "SSH に使用するデフォルトの鍵パスフレーズ。remote_execution_ssh_key_passphrase という名前のパラメーターを設定することにより、ホストごとにオーバーライドできます"
+msgstr "SSH に使用するデフォルトの鍵パスフレーズ。remote_execution_ssh_key_passphrase という名前のパラメーターを設定することにより、ホストごとに上書きできます"
 
 msgid "Default password to use for SSH. You may override per host by setting a parameter called remote_execution_ssh_password"
-msgstr "SSH に使用するデフォルトパスワード。remote_execution_ssh_password という名前のパラメーターを設定することにより、ホストごとにオーバーライドできます"
+msgstr "SSH に使用するデフォルトパスワード。remote_execution_ssh_password という名前のパラメーターを設定することにより、ホストごとに上書きできます"
 
 msgid "Default user to use for SSH.  You may override per host by setting a parameter called remote_execution_ssh_user."
-msgstr "SSH に使用するデフォルトユーザー。remote_execution_ssh_user という名前のパラメーターを設定することにより、ホストごとにオーバーライドできます。"
+msgstr "SSH に使用するデフォルトユーザー。remote_execution_ssh_user という名前のパラメーターを設定することにより、ホストごとに上書きできます。"
 
 msgid "Default user to use for executing the script. If the user differs from the SSH user, su or sudo is used to switch the user."
 msgstr "スクリプトを実行するために使用するデフォルトユーザー。ユーザーが SSH ユーザーと異なる場合は、su または sudo を使用してユーザーを切り替えます。"
@@ -408,7 +408,7 @@ msgid "Designation of a special purpose"
 msgstr "特別な目的の指定"
 
 msgid "Disable recurring"
-msgstr "繰り返しを無効化する"
+msgstr "繰り返しの無効化"
 
 msgid "Display advanced fields"
 msgstr "詳細フィールドを表示"
@@ -468,7 +468,7 @@ msgid "Enable Global Proxy"
 msgstr "グローバルプロキシーを有効にする"
 
 msgid "Enable recurring"
-msgstr "繰り返しを有効化する"
+msgstr "繰り返しの有効化"
 
 msgid "End time needs to be after start time"
 msgstr "終了時刻は開始時刻の後でなければなりません"
@@ -828,7 +828,7 @@ msgid "Manual selection"
 msgstr "手動選択"
 
 msgid "Minute can only be a number between 0-59"
-msgstr "分は 0 から 59 までの数値である必要があります"
+msgstr "分は 0-59 の間の数字のみ指定できます。"
 
 msgid "Missing the required permissions: ${missingPermissions.join( ', ' )}"
 msgstr "必須パーミッションがありません: ${missingPermissions.join( ', ' )}"
@@ -861,10 +861,10 @@ msgid "Next occurrence"
 msgstr "次の実行"
 
 msgid "No (override)"
-msgstr "No (オーバーライド)"
+msgstr "No (上書き)"
 
 msgid "No Results"
-msgstr "該当する結果がありません"
+msgstr "結果なし"
 
 msgid "No Target Hosts"
 msgstr "ターゲットホストがありません"
@@ -882,7 +882,7 @@ msgid "No output for the selected filters"
 msgstr "選択したフィルターの出力はありません"
 
 msgid "No results found"
-msgstr "該当する結果が見つかりませんでした"
+msgstr "結果は見つかりませんでした"
 
 msgid "No template mapped to feature %{feature_name}"
 msgstr "機能 %{feature_name} にマッピングされたテンプレートがありません"
@@ -930,19 +930,19 @@ msgid "Open selected in new tab"
 msgstr "選択項目を新規タブで開きます"
 
 msgid "Opening job invocation form"
-msgstr "ジョブ呼び出しフォームを開いています"
+msgstr "ジョブ呼び出しフォームを開く"
 
 msgid "Organization"
 msgstr "組織"
 
 msgid "Override the description format from the template for this invocation only"
-msgstr "この呼び出しに対してのみテンプレートから説明形式をオーバーライド"
+msgstr "この呼び出しに対してのみテンプレートから説明形式を上書き"
 
 msgid "Override the global time to pickup interval for this invocation only"
-msgstr "グローバル時間をこの呼び出しだけの取得間隔にオーバーライドする"
+msgstr "グローバル時間をこの呼び出しだけの取得間隔に上書きする"
 
 msgid "Override the timeout interval from the template for this invocation only"
-msgstr "この呼び出しに対してのみテンプレートからタイムアウト間隔をオーバーライド"
+msgstr "この呼び出しに対してのみテンプレートからタイムアウト間隔を上書き"
 
 msgid "Overview"
 msgstr "概要"
@@ -975,7 +975,7 @@ msgid "Please enter a search query"
 msgstr "検索クエリーを入力してください"
 
 msgid "Please go back to \\\"Schedule\\\" - \\\"Future execution\\\" or \\\"Recurring execution\\\" step to fix the error"
-msgstr "エラーを修正するには、スケジュール - 後で実行または繰り返し実行の手順に戻ってください"
+msgstr "エラーを修正するには、スケジュール - 後で実行または定期的な実行の手順に戻ってください。"
 
 msgid "Please go back to \\\"Schedule\\\" - \\\"Future execution\\\" step to fix the error"
 msgstr "「スケジュール」- 「後で実行」の手順に戻り、エラーを修正してください"
@@ -993,7 +993,7 @@ msgid "Please select at least one host"
 msgstr "ホストを 1 つ以上選択してください"
 
 msgid "Please select at least one host collection"
-msgstr "ホストコレクションを少なくとも 1 つ選択してください"
+msgstr "ホストコレクションを少なくとも 1 つ選択してください。"
 
 msgid "Please select at least one host group"
 msgstr "少なくとも 1 つのホストグループを選択してください"
@@ -1002,7 +1002,7 @@ msgid "Popups are blocked by your browser. Please allow popups for this site to
 msgstr "ブラウザーでポップアップがブロックされています。このサイトのすべての呼び出しを新しいタブで開くには、ポップアップを許可してください。"
 
 msgid "Port to use for SSH communication. Default port 22. You may override per host by setting a parameter called remote_execution_ssh_port."
-msgstr "SSH 通信に使用するポート。デフォルトのポートは 22 です。remote_execution_ssh_port という名前のパラメーターを設定することにより、ホストごとにオーバーライドできます。"
+msgstr "SSH 通信に使用するポート。デフォルトのポートは 22 です。remote_execution_ssh_port という名前のパラメーターを設定することにより、ホストごとに上書きできます。"
 
 msgid "Prefer IPv6 over IPv4"
 msgstr "IPv4 よりも IPv6 を優先する"
@@ -1071,16 +1071,16 @@ msgid "Recurring execution"
 msgstr "繰り返し実行"
 
 msgid "Recurring logic"
-msgstr "繰り返し実行ロジック"
+msgstr "再帰論理"
 
 msgid "Recurring logic %s cancelled successfully."
-msgstr "繰り返し実行ロジック %s が正常にキャンセルされました。"
+msgstr "再帰論理 %s が正常にキャンセルされました。"
 
 msgid "Recurring logic %s disabled successfully."
-msgstr "繰り返し実行ロジック %s が正常に無効化されました。"
+msgstr "再帰論理 %s が正常に無効になりました。"
 
 msgid "Recurring logic %s enabled successfully."
-msgstr "繰り返し実行ロジック %s が正常に有効化されました。"
+msgstr "再帰論理 %s が正常に有効になりました。"
 
 msgid "Recursive rendering of templates detected"
 msgstr "検出されたテンプレートの再帰的なレンダリング"
@@ -1311,7 +1311,7 @@ msgid "Setup remote execution pull mode. If set to `Yes`, pull provider client w
 msgstr "リモート実行のプルモードを設定します。`はい` に設定すると、登録したホストにプルプロパイダークライアントがデプロイされます。継承される値は、`host_registration_remote_execution_pull` パラメーターに基づきます。これは、ホストグループ、オペレーティングシステム、組織などから継承できます。オーバーライドされると、選択した値がホストパラメーターレベルで保存されます。"
 
 msgid "Should the ip addresses on host interfaces be preferred over the fqdn? It is useful when DNS not resolving the fqdns properly. You may override this per host by setting a parameter called remote_execution_connect_by_ip. For dual-stacked hosts you should consider the remote_execution_connect_by_ip_prefer_ipv6 setting"
-msgstr "ホストインターフェイスの ip アドレスは fqdn よりも優先されますか? DNS が fqdn を適切に解決しない場合、これは役に立ちます。remote_execution_connect_by_ip というパラメーターを設定し、ホストごとにこれをオーバーライドすることができます。デュアルスタックのホストの場合は、remote_execution_connect_by_ip_prefer_ipv6 設定を検討する必要があります"
+msgstr "ホストインターフェイスの ip アドレスは fqdn よりも優先されますか? DNS が fqdn を適切に解決しない場合、これは役に立ちます。remote_execution_connect_by_ip というパラメーターを設定し、ホストごとにこれを上書きすることができます。デュアルスタックのホストの場合は、remote_execution_connect_by_ip_prefer_ipv6 設定を検討する必要があります"
 
 msgid "Should this interface be used for remote execution?"
 msgstr "このインターフェイスをリモート実行に使用する必要がありますか?"
@@ -1380,7 +1380,7 @@ msgid "Static query"
 msgstr "静的クエリー"
 
 msgid "Status"
-msgstr "状態"
+msgstr "ステータス"
 
 msgid "Submit"
 msgstr "送信"
@@ -1720,16 +1720,16 @@ msgid "What user should be used to run the script (using sudo-like mechanisms).
 msgstr "スクリプトを実行するために使用するユーザー (sudo と同様のメカニズムを使用)。デフォルト値は、テンプレートパラメーターまたはグローバル設定です。"
 
 msgid "When connecting using ip address, should the IPv6 addresses be preferred? If no IPv6 address is set, it falls back to IPv4 automatically. You may override this per host by setting a parameter called remote_execution_connect_by_ip_prefer_ipv6. By default and for compatibility, IPv4 will be preferred over IPv6 by default"
-msgstr "ip address を使用して接続する場合は、IPv6 アドレスを優先する必要がありますか? IPv6 アドレスが設定されていない場合は、IPv4 に自動的にフォールバックします。remote_execution_connect_by_ip_prefer_ipv6 というパラメーターを設定し、ホストごとにこれをオーバーライドすることができます。互換性のために、デフォルトでは IPv4 が IPv6 よりも優先されます"
+msgstr "ip address を使用して接続する場合は、IPv6 アドレスを優先する必要がありますか? IPv6 アドレスが設定されていない場合は、IPv4 に自動的にフォールバックします。remote_execution_connect_by_ip_prefer_ipv6 というパラメーターを設定し、ホストごとにこれを上書きすることができます。互換性のために、デフォルトでは IPv4 が IPv6 よりも優先されます"
 
 msgid "When enabled, working directories will be removed after task completion. You may override this per host by setting a parameter called remote_execution_cleanup_working_dirs."
-msgstr "有効にすると、作業ディレクトリーはタスクの完了後に削除されます。remote_execution_cleanup_working_dirs と呼ばれるパラメーターを設定して、ホストごとにこれをオーバーライドすることができます。"
+msgstr "有効にすると、作業ディレクトリーはタスクの完了後に削除されます。remote_execution_cleanup_working_dirs と呼ばれるパラメーターを設定して、ホストごとにこれを上書きすることができます。"
 
 msgid "Where to find the Cockpit instance for the Web Console button.  By default, no button is shown."
 msgstr "Web コンソールボタンの Cockpit インスタンスを検索する場所。デフォルトでは、ボタンは表示されません。"
 
 msgid "Whether it should be allowed to override the effective user from the invocation form."
-msgstr "呼び出し形式から実効ユーザーをオーバーライドするのを許可するかどうか。"
+msgstr "呼び出し形式から実効ユーザーを上書きするのを許可するかどうか。"
 
 msgid "Whether or not the template is locked for editing"
 msgstr "テンプレートの編集機能をロックするかどうか"
@@ -1744,7 +1744,7 @@ msgid "Whether we should sync templates from disk when running db:seed."
 msgstr "db:seed を実行するときにディスクからテンプレートを同期するかどうか。"
 
 msgid "Yes (override)"
-msgstr "Yes (オーバーライド)"
+msgstr "Yes (上書き)"
 
 msgid "You are not allowed to see the currently assigned template. Saving the form now would unassign the template."
 msgstr "現在割り当てられているテンプレートを参照できません。フォームを保存すると、テンプレートの割り当てが解除されます。"

data/locale/ko/foreman_remote_execution.po

@@ -360,7 +360,7 @@ msgid "Daily"
 msgstr "매일"
 
 msgid "Date must be in the future"
-msgstr "날짜는 미래 시점이어야 합니다."
+msgstr "날짜는 미래여야 합니다."
 
 msgid "Days"
 msgstr "일"
@@ -423,7 +423,7 @@ msgid "Dynamic query"
 msgstr "동적 쿼리"
 
 msgid "Edit %s"
-msgstr "%s 편집"
+msgstr "%s 편집 "
 
 msgid "Edit Job Template"
 msgstr "작업 템플릿 편집"
@@ -837,7 +837,7 @@ msgid "Must select a bookmark or enter a search query"
 msgstr "북마크를 선택하거나 검색 쿼리를 입력해야 합니다."
 
 msgid "N/A"
-msgstr "해당 없음"
+msgstr "해당 없음 "
 
 msgid "Name"
 msgstr "이름 "
@@ -1341,7 +1341,7 @@ msgid "Smart proxy"
 msgstr "스마트 프록시"
 
 msgid "Snippet"
-msgstr "스니펫"
+msgstr "조각 모음 "
 
 msgid "Start"
 msgstr "시작"
@@ -1377,7 +1377,7 @@ msgid "Static query"
 msgstr "정적 쿼리"
 
 msgid "Status"
-msgstr "상태"
+msgstr "상태 "
 
 msgid "Submit"
 msgstr "보내기"
@@ -1525,7 +1525,7 @@ msgid "This can happen if the host is removed or moved to another organization o
 msgstr "작업이 시작된 후 호스트가 제거되거나 다른 조직이나 위치로 이동된 경우 이러한 문제가 발생할 수 있습니다."
 
 msgid "This template is locked for editing."
-msgstr "이 템플릿은 편집이 잠금되어 있습니다. "
+msgstr "이 템플릿은 편집 용으로 잠금되어 있습니다. "
 
 msgid "This template is locked. Please clone it to a new template to customize."
 msgstr "이 템플릿은 잠겨 있습니다. 사용자 정의하려면 이를 새 템플릿에 복제하십시오. "
@@ -1636,7 +1636,7 @@ msgid "Unable to save template. Correct highlighted errors"
 msgstr "템플릿을 저장할 수 없습니다. 강조된 오류를 수정합니다."
 
 msgid "Unknown"
-msgstr "알 수 없음"
+msgstr "알 수 없음 "
 
 msgid "Unknown execution status"
 msgstr "알 수 없는 실행 상태"
@@ -1729,7 +1729,7 @@ msgid "Whether it should be allowed to override the effective user from the invo
 msgstr "호출할 유효 사용자를 덮어쓰도록 허용할지 여부입니다."
 
 msgid "Whether or not the template is locked for editing"
-msgstr "템플릿 편집 잠금 여부"
+msgstr "편집을 위한 템플릿의 잠금 여부 "
 
 msgid "Whether the current user login should be used as the effective user"
 msgstr "현재 사용자 로그인을 유효 사용자로 사용할지 여부입니다."

data/locale/zh_CN/foreman_remote_execution.po

@@ -1074,7 +1074,7 @@ msgid "Recurring logic"
 msgstr "重复逻辑"
 
 msgid "Recurring logic %s cancelled successfully."
-msgstr "成功取消循环逻辑 %s。"
+msgstr "成功取消重复逻辑 %s。"
 
 msgid "Recurring logic %s disabled successfully."
 msgstr "成功禁用重复逻辑 %s。"

data/test/unit/remote_execution_provider_test.rb

@@ -125,7 +125,7 @@ class RemoteExecutionProviderTest < ActiveSupport::TestCase
     describe 'effective user' do
       it 'takes the effective user from value from the template invocation' do
         template_invocation.effective_user = 'my user'
-        assert_equal 'my\\ user', proxy_options[:effective_user]
+        assert_equal 'my user', proxy_options[:effective_user]
       end
     end
 

data/webpack/JobInvocationDetail/JobInvocationConstants.js

@@ -55,6 +55,16 @@ export const STATUS_TITLES = {
   NOT_STARTED: { id: 'N/A', title: __('Scheduled') },
 };
 
+export const TERMINAL_HOST_STATUSES = new Set([
+  'success',
+  'error',
+  'cancelled',
+]);
+
+export const areAllHostsTerminal = results =>
+  results?.length > 0 &&
+  results.every(host => TERMINAL_HOST_STATUSES.has(host.job_status));
+
 export const DATE_OPTIONS = {
   day: 'numeric',
   month: 'short',

data/webpack/JobInvocationDetail/JobInvocationHostTable.js

@@ -42,6 +42,7 @@ import Columns, {
   STATUS_UPPERCASE,
   AWAITING_STATUS_FILTER,
   AUTO_REFRESH_INTERVAL_MS,
+  areAllHostsTerminal,
 } from './JobInvocationConstants';
 import { TemplateInvocation } from './TemplateInvocation';
 import { RowActions } from './TemplateInvocationComponents/TemplateActionButtons';
@@ -196,7 +197,13 @@ const JobInvocationHostTable = ({
             if (thisRequest !== requestIdRef.current) return;
             if (!mountedRef.current) return;
             updateHostsState(data);
-            if (!jobFinishedRef.current) {
+            const hasActiveFilter = !!(
+              currentPollParams.current.search ||
+              currentPollParams.current.awaiting
+            );
+            const pageAllTerminal =
+              !hasActiveFilter && areAllHostsTerminal(data.data?.results);
+            if (!jobFinishedRef.current && !pageAllTerminal) {
               pollTimeoutId.current = setTimeout(
                 () => makeApiCall(currentPollParams.current),
                 AUTO_REFRESH_INTERVAL_MS

data/webpack/JobInvocationDetail/__tests__/JobInvocationHostTablePolling.test.js

@@ -1,3 +1,4 @@
+/* eslint-disable max-lines */
 import React from 'react';
 import { render, act } from '@testing-library/react';
 import '@testing-library/jest-dom';
@@ -44,13 +45,28 @@ const hostsResponse = {
       operatingsystem_name: 'RHEL 9',
       hostgroup_id: 1,
       hostgroup_name: 'default',
-      job_status: 'success',
+      job_status: 'running',
       smart_proxy_id: 1,
       smart_proxy_name: 'proxy1',
     },
   ],
 };
 
+const terminalHostsResponse = {
+  ...hostsResponse,
+  results: [{ ...hostsResponse.results[0], job_status: 'success' }],
+};
+
+const mixedHostsResponse = {
+  ...hostsResponse,
+  total: 2,
+  subtotal: 2,
+  results: [
+    { ...hostsResponse.results[0], id: 1, job_status: 'success' },
+    { ...hostsResponse.results[0], id: 2, job_status: 'running' },
+  ],
+};
+
 let apiGetSpy;
 let hostsCalls;
 let pendingCallbacks;
@@ -66,9 +82,9 @@ const flushPendingCallbacks = () => {
   batch.forEach(cb => cb());
 };
 
-const renderTable = (props = {}) => {
+const renderTable = (props = {}, historyEntries = ['/']) => {
   const store = createStore();
-  const history = createMemoryHistory();
+  const history = createMemoryHistory({ initialEntries: historyEntries });
   const Wrapper = createForemanContextWrapper();
 
   const defaultProps = {
@@ -93,16 +109,14 @@ const renderTable = (props = {}) => {
   return { ...result, store, history };
 };
 
-const setupSuccessMock = () => {
+const setupSuccessMock = (response = hostsResponse) => {
   apiGetSpy = jest
     .spyOn(APIActions, 'get')
     .mockImplementation(opts => dispatch => {
       if (opts.key === JOB_INVOCATION_HOSTS) {
         hostsCalls.push(opts);
         if (opts.handleSuccess) {
-          pendingCallbacks.push(() =>
-            opts.handleSuccess({ data: hostsResponse })
-          );
+          pendingCallbacks.push(() => opts.handleSuccess({ data: response }));
         }
       }
     });
@@ -311,6 +325,98 @@ describe('JobInvocationHostTable polling', () => {
     expect(hostsCalls.length).toBeGreaterThan(callsAfterFilterChange);
   });
 
+  it('stops polling when all hosts on the page are terminal', () => {
+    apiGetSpy.mockRestore();
+    setupSuccessMock(terminalHostsResponse);
+
+    renderTable();
+
+    expect(hostsCalls).toHaveLength(1);
+
+    act(() => {
+      flushPendingCallbacks();
+    });
+
+    act(() => {
+      jest.advanceTimersByTime(10000);
+    });
+
+    act(() => {
+      flushPendingCallbacks();
+    });
+
+    expect(hostsCalls).toHaveLength(1);
+  });
+
+  it('continues polling when at least one host on the page is non-terminal', () => {
+    apiGetSpy.mockRestore();
+    setupSuccessMock(mixedHostsResponse);
+
+    renderTable();
+
+    expect(hostsCalls).toHaveLength(1);
+
+    act(() => {
+      flushPendingCallbacks();
+    });
+
+    act(() => {
+      jest.advanceTimersByTime(5000);
+    });
+
+    act(() => {
+      flushPendingCallbacks();
+    });
+
+    expect(hostsCalls).toHaveLength(2);
+  });
+
+  it('continues polling when all hosts on the page are terminal but a status filter is active', () => {
+    apiGetSpy.mockRestore();
+    setupSuccessMock(terminalHostsResponse);
+
+    renderTable({ initialFilter: 'success' });
+
+    expect(hostsCalls).toHaveLength(1);
+
+    act(() => {
+      flushPendingCallbacks();
+    });
+
+    act(() => {
+      jest.advanceTimersByTime(5000);
+    });
+
+    act(() => {
+      flushPendingCallbacks();
+    });
+
+    expect(hostsCalls).toHaveLength(2);
+  });
+
+  it('continues polling when all hosts on the page are terminal but a search query is active', () => {
+    apiGetSpy.mockRestore();
+    setupSuccessMock(terminalHostsResponse);
+
+    renderTable({}, ['/?search=host1']);
+
+    expect(hostsCalls).toHaveLength(1);
+
+    act(() => {
+      flushPendingCallbacks();
+    });
+
+    act(() => {
+      jest.advanceTimersByTime(5000);
+    });
+
+    act(() => {
+      flushPendingCallbacks();
+    });
+
+    expect(hostsCalls).toHaveLength(2);
+  });
+
   it('sends include_permissions only on the first request', () => {
     renderTable();
 

data/webpack/JobInvocationDetail/__tests__/areAllHostsTerminal.test.js

@@ -0,0 +1,40 @@
+import { areAllHostsTerminal } from '../JobInvocationConstants';
+
+describe('areAllHostsTerminal', () => {
+  it('returns false for an empty results array', () => {
+    expect(areAllHostsTerminal([])).toBe(false);
+  });
+
+  it('returns false for undefined results', () => {
+    expect(areAllHostsTerminal(undefined)).toBe(false);
+  });
+
+  it('returns true when every host is in a terminal state', () => {
+    expect(
+      areAllHostsTerminal([
+        { id: 1, job_status: 'success' },
+        { id: 2, job_status: 'error' },
+        { id: 3, job_status: 'cancelled' },
+      ])
+    ).toBe(true);
+  });
+
+  it('returns false when at least one host is non-terminal', () => {
+    expect(
+      areAllHostsTerminal([
+        { id: 1, job_status: 'success' },
+        { id: 2, job_status: 'running' },
+      ])
+    ).toBe(false);
+  });
+
+  it('returns false when every host is non-terminal', () => {
+    expect(
+      areAllHostsTerminal([
+        { id: 1, job_status: 'running' },
+        { id: 2, job_status: 'planned' },
+        { id: 3, job_status: 'N/A' },
+      ])
+    ).toBe(false);
+  });
+});