foreman_remote_execution 10.0.3 → 10.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c6cbb748f9c9d67aba38a2fa62ad4e82306d665bd9ad0021c2125926207fd95f
-  data.tar.gz: 435f4a1aee85997d3ef430abf68a24da459bd55a09832a130e51c405638dbfd0
+  metadata.gz: afb55ca9ac902c2d13fa825bb4bf6b2a31cd3729f9f073b98f1add34001fdfa2
+  data.tar.gz: a3155cf9f3c74f9d7588984015dc47ba6b7a81d4bea04bcb3b870c4ad6306405
 SHA512:
-  metadata.gz: 2b3b8b5d63dfc60911069228a9f0ff136ccfe536331bb53bf348b902ee4e0cbb1fb2e26ca3d209a409a5d78a03530d5e490460a6d024baf7a584ae755bbe88d3
-  data.tar.gz: e54b93592be98be9c6577f662d6a6e3298c36c9e7576b9325f4b1f981f94ff560865e37aaec4a87a8fe06ed933e433e66420326b9c7b5f6e72e13503b8ff4671
+  metadata.gz: d9e34ed092397296e54f100d3a5697ef18ee86e79f5a293e01c0fcf19b33c195d7cfdd1a8046079ab70fce503e2760b93ca616346ab3cd7a21c4df9ef2c10983
+  data.tar.gz: 54cb3b829257039f3f3568544ec3f944e247a966ffd8e06f6f9a4ef1d71c7a3aaf4d0b2b3f5cea78cec32fa0eaa7e67d818e264db5bcf8b63eb0ab0d7b4f1bff

data/.eslintrc

@@ -1,5 +1,8 @@
 {
-  "plugins": ["@theforeman/foreman"],
+  "plugins": [
+    "@theforeman/foreman",
+    "@theforeman/rules"
+  ],
   "extends": [
     "plugin:@theforeman/foreman/core",
     "plugin:@theforeman/foreman/plugins"

data/.github/workflows/js_ci.yml

@@ -24,6 +24,9 @@ jobs:
       - name: Run plugin linter
         run: |
           npm run lint
+      - name: Run custom plugin linter
+        run: |
+          npm run lint:custom
       - name: Run plugin tests
         run: |
           npm run test

data/app/controllers/ui_job_wizard_controller.rb

@@ -1,7 +1,7 @@
 class UiJobWizardController < ApplicationController
   include FiltersHelper
   def categories
-    job_categories = resource_scope
+    job_categories = resource_scope(permission: action_permission)
                      .search_for("job_category ~ \"#{params[:search]}\"")
                      .where(:snippet => false)
                      .select(:job_category).distinct

data/app/views/job_invocations/_card_schedule.html.erb

@@ -24,6 +24,10 @@
       <% end %>
       <% if job_invocation.execution_timeout_interval.present? %>
         <li><b><%= _("Timeout to kill after") %></b>: <%= job_invocation.execution_timeout_interval %> <%= _('seconds') %><br></li>
+        <% end %>
+      <% if job_invocation.time_to_pickup.present? %>
+        <li><b><%= _("Time to pickup") %></b>: <%= job_invocation.time_to_pickup %> <%= _('seconds') %><br>
+        </li>
       <% end %>
     </ul>
     </p>

data/app/views/job_templates/_custom_tabs.html.erb

@@ -1,7 +1,7 @@
 <div class="tab-pane" id="template_job">
 
   <%= autocomplete_f(f, :job_category,
-        :search_query => '',
+        :search_query => @template.job_category,
         :placeholder => _("Job category") + ' ...',
         :disabled => @template.locked?) %>
   <%= text_f f, :description_format,

data/lib/foreman_remote_execution/engine.rb

@@ -158,7 +158,7 @@ module ForemanRemoteExecution
               collection: proc { ForemanRemoteExecution.job_invocation_report_templates_select }
             setting 'remote_execution_time_to_pickup',
               type: :integer,
-              description: N_('Time in seconds within which the host has to pick up a job. If the job is not picked up within this limit, the job will be cancelled. Defaults to 1 day.'),
+              description: N_('Time in seconds within which the host has to pick up a job. If the job is not picked up within this limit, the job will be cancelled. Defaults to 1 day. Applies only to pull-mqtt based jobs.'),
               default: 24 * 60 * 60,
               full_name: N_('Time to pickup')
           end

data/lib/foreman_remote_execution/version.rb

@@ -1,3 +1,3 @@
 module ForemanRemoteExecution
-  VERSION = '10.0.3'.freeze
+  VERSION = '10.0.4'.freeze
 end

data/package.json

@@ -4,6 +4,7 @@
   "license": "GPL-3.0",
   "scripts": {
     "lint": "tfm-lint --plugin -d /webpack",
+    "lint:custom": "eslint ./webpack",
     "test": "tfm-test --plugin",
     "test:watch": "tfm-test --plugin --watchAll",
     "test:current": "tfm-test --plugin --watch",
@@ -23,6 +24,7 @@
     "@babel/core": "^7.7.0",
     "@theforeman/builder": "^12.0.1",
     "@theforeman/eslint-plugin-foreman": "^12.0.1",
+    "@theforeman/eslint-plugin-rules": "^12.0.2",
     "@theforeman/test": "^12.0.1",
     "@theforeman/vendor-dev": "^12.0.1",
     "babel-eslint": "^10.0.0",

data/webpack/JobWizard/steps/AdvancedFields/Fields.js

@@ -59,7 +59,7 @@ export const TimeToPickupField = ({ value, setValue }) => (
       label: __('Time to pickup'),
       labelIcon: helpLabel(
         __(
-          'Interval in seconds, if the job is not picked up by a client within this interval it will be cancelled.'
+          'Interval in seconds, if the job is not picked up by a client within this interval it will be cancelled. Applies only to pull-mqtt based jobs'
         ),
         'time-to-pickup'
       ),

data/webpack/JobWizard/steps/ReviewDetails/helpers.js

@@ -23,9 +23,8 @@ export const parseRepeat = (repeatType, repeatData) => {
       const daysKeys = Object.keys(repeatData.daysOfWeek).filter(
         k => repeatData.daysOfWeek[k]
       );
-      const days = getWeekDays()
-        .filter((d, index) => index in daysKeys)
-        .join(', ');
+      const dayNames = getWeekDays();
+      const days = daysKeys.map(day => dayNames[day]).join(', ');
       return sprintf(__('Every week on %s at %s'), days, repeatData.at);
     }
     case repeatTypes.monthly:

data/webpack/JobWizard/steps/ReviewDetails/index.js

@@ -30,6 +30,9 @@ import { WizardTitle } from '../form/WizardTitle';
 import { parseEnd, parseRepeat } from './helpers';
 import { HostPreviewModal } from '../HostsAndInputs/HostPreviewModal';
 
+const maskValue = (isHidden, value) =>
+  isHidden ? '*'.repeat(value.length) : value;
+
 const ReviewDetails = ({
   jobCategory,
   jobTemplateID,
@@ -122,11 +125,11 @@ const ReviewDetails = ({
       ),
       value: stringHosts(),
     },
-    ...templateInputs.map(({ name }) => ({
+    ...templateInputs.map(({ name, hidden_value: isHidden }) => ({
       label: (
         <StepButton stepName={WIZARD_TITLES.hostsAndInputs}>{name}</StepButton>
       ),
-      value: templateValues[name],
+      value: maskValue(isHidden, templateValues[name]),
     })),
     {
       label: (
@@ -272,9 +275,9 @@ const ReviewDetails = ({
         ? __('Randomized')
         : __('Alphabetical'),
     },
-    ...advancedTemplateInputs.map(({ name }) => ({
+    ...advancedTemplateInputs.map(({ name, hidden_value: isHidden }) => ({
       label: name,
-      value: advancedValues.templateValues[name],
+      value: maskValue(isHidden, advancedValues.templateValues[name]),
     })),
   ];
 

data/webpack/JobWizard/steps/Schedule/RepeatCron.js

@@ -1,6 +1,14 @@
 import React, { useEffect } from 'react';
 import PropTypes from 'prop-types';
-import { TextInput, FormGroup, ValidatedOptions } from '@patternfly/react-core';
+import {
+  List,
+  ListItem,
+  ListComponent,
+  OrderType,
+  TextInput,
+  FormGroup,
+  ValidatedOptions,
+} from '@patternfly/react-core';
 import { translate as __ } from 'foremanReact/common/I18n';
 import { helpLabel } from '../form/FormHelpers';
 
@@ -19,15 +27,15 @@ export const RepeatCron = ({ repeatData, setRepeatData, setValid }) => {
       label={__('Cron line')}
       labelIcon={helpLabel(
         <div>
-          {__("Cron line format 'a b c d e', where:")}
+          {__("Cron line format '1 2 3 4 5', where:")}
           <br />
-          <ol>
-            <li>{__('is minute (range: 0-59)')}</li>
-            <li>{__('is hour (range: 0-23)')}</li>
-            <li>{__('is day of month (range: 1-31)')}</li>
-            <li>{__('is month (range: 1-12)')}</li>
-            <li>{__('is day of week (range: 0-6)')}</li>
-          </ol>
+          <List component={ListComponent.ol} type={OrderType.number}>
+            <ListItem>{__('is minute (range: 0-59)')}</ListItem>
+            <ListItem>{__('is hour (range: 0-23)')}</ListItem>
+            <ListItem>{__('is day of month (range: 1-31)')}</ListItem>
+            <ListItem>{__('is month (range: 1-12)')}</ListItem>
+            <ListItem>{__('is day of week (range: 0-6)')}</ListItem>
+          </List>
         </div>
       )}
       isRequired

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_remote_execution
 version: !ruby/object:Gem::Version
-  version: 10.0.3
+  version: 10.0.4
 platform: ruby
 authors:
 - Foreman Remote Execution team
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-06-20 00:00:00.000000000 Z
+date: 2023-07-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: deface
@@ -576,7 +576,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.13
+rubygems_version: 3.4.16
 signing_key:
 specification_version: 4
 summary: A plugin bringing remote execution to the Foreman, completing the config