foreman_remote_execution 4.2.3 → 4.3.0
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/.github/workflows/js_ci.yml +29 -0
- data/.github/workflows/{ci.yml → ruby_ci.yml} +22 -50
- data/.prettierrc +4 -0
- data/.rubocop.yml +13 -49
- data/.rubocop_todo.yml +326 -102
- data/Gemfile +1 -4
- data/app/controllers/api/v2/job_invocations_controller.rb +1 -1
- data/app/controllers/ui_job_wizard_controller.rb +18 -0
- data/app/lib/actions/remote_execution/run_host_job.rb +38 -1
- data/app/models/host_status/execution_status.rb +7 -0
- data/app/models/job_invocation.rb +1 -1
- data/config/routes.rb +3 -0
- data/foreman_remote_execution.gemspec +1 -2
- data/lib/foreman_remote_execution/engine.rb +12 -1
- data/lib/foreman_remote_execution/version.rb +1 -1
- data/test/functional/api/v2/job_invocations_controller_test.rb +15 -5
- data/test/functional/api/v2/registration_controller_test.rb +4 -13
- data/test/functional/ui_job_wizard_controller_test.rb +16 -0
- data/webpack/JobWizard/JobWizard.js +32 -0
- data/webpack/JobWizard/index.js +32 -0
- data/webpack/Routes/routes.js +12 -0
- data/webpack/__mocks__/foremanReact/history.js +1 -0
- data/webpack/global_index.js +4 -0
- data/webpack/react_app/components/TargetingHosts/TargetingHostsPage.js +1 -1
- data/webpack/react_app/components/TargetingHosts/TargetingHostsPage.scss +0 -3
- data/webpack/react_app/components/TargetingHosts/__tests__/__snapshots__/TargetingHostsPage.test.js.snap +1 -1
- metadata +15 -19
checksums.yaml
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
---
|
2
2
|
SHA256:
|
3
|
-
metadata.gz:
|
4
|
-
data.tar.gz:
|
3
|
+
metadata.gz: f170eb66eb23ab9e671ff1137c9ca830041743eb83836947cf11da32432f119d
|
4
|
+
data.tar.gz: 0f4d52cce40bc8c9f89eb4e65242b1e3a25e4d0ce5ba9cf24b5fc9ff23346267
|
5
5
|
SHA512:
|
6
|
-
metadata.gz:
|
7
|
-
data.tar.gz:
|
6
|
+
metadata.gz: 5964ee1b1bc9b3983f6f2677f9e38dbaf41416d16ace8ee1dcfc43f63d7250154b62722b191cc62b2b29c073d8035e0f813d5e576db167b05b1a2ea56cef6ce4
|
7
|
+
data.tar.gz: 3cf702ec56abf119a67f6bbbd1a0d2b6b2dbb067ee99ea31735d2b5bcedc87b194d992e48ffcfe8848799cc16eb872623523d73d16b5f8e6ddd8d47400d51e80
|
@@ -0,0 +1,29 @@
|
|
1
|
+
name: JS
|
2
|
+
on:
|
3
|
+
pull_request:
|
4
|
+
paths:
|
5
|
+
- 'webpack/**'
|
6
|
+
- 'package.json'
|
7
|
+
- '.github/workflows/js_ci.yml'
|
8
|
+
jobs:
|
9
|
+
test_js:
|
10
|
+
runs-on: ubuntu-latest
|
11
|
+
strategy:
|
12
|
+
fail-fast: false
|
13
|
+
matrix:
|
14
|
+
node-version: [12]
|
15
|
+
steps:
|
16
|
+
- uses: actions/checkout@v2
|
17
|
+
- name: Setup Node
|
18
|
+
uses: actions/setup-node@v1
|
19
|
+
with:
|
20
|
+
node-version: ${{ matrix.node-version }}
|
21
|
+
- name: Npm install
|
22
|
+
run: |
|
23
|
+
npm install
|
24
|
+
- name: Run plugin linter
|
25
|
+
run: |
|
26
|
+
npm run lint
|
27
|
+
- name: Run plugin tests
|
28
|
+
run: |
|
29
|
+
npm run test
|
@@ -1,5 +1,9 @@
|
|
1
|
-
name:
|
2
|
-
on:
|
1
|
+
name: Ruby
|
2
|
+
on:
|
3
|
+
pull_request:
|
4
|
+
push:
|
5
|
+
branches:
|
6
|
+
- master
|
3
7
|
env:
|
4
8
|
RAILS_ENV: test
|
5
9
|
DATABASE_URL: postgresql://postgres:@localhost/test
|
@@ -12,16 +16,17 @@ jobs:
|
|
12
16
|
- name: Setup Ruby
|
13
17
|
uses: ruby/setup-ruby@v1
|
14
18
|
with:
|
15
|
-
ruby-version: 2.
|
16
|
-
|
17
|
-
run: |
|
18
|
-
gem install bundler
|
19
|
-
bundle install --jobs=3 --retry=3
|
19
|
+
ruby-version: 2.5
|
20
|
+
bundler-cache: true
|
20
21
|
- name: Run rubocop
|
21
|
-
|
22
|
+
if: github.event_name != 'push'
|
23
|
+
run: bundle exec rubocop --format github
|
22
24
|
test_ruby:
|
23
25
|
runs-on: ubuntu-latest
|
24
26
|
needs: rubocop
|
27
|
+
timeout-minutes: 30
|
28
|
+
env:
|
29
|
+
BUNDLE_WITHOUT: journald:development:console:libvirt
|
25
30
|
services:
|
26
31
|
postgres:
|
27
32
|
image: postgres:12.1
|
@@ -31,7 +36,7 @@ jobs:
|
|
31
36
|
fail-fast: false
|
32
37
|
matrix:
|
33
38
|
foreman-core-branch: [develop]
|
34
|
-
ruby-version: [2.5, 2.
|
39
|
+
ruby-version: [2.5, 2.7]
|
35
40
|
node-version: [12]
|
36
41
|
steps:
|
37
42
|
- run: sudo apt-get update
|
@@ -43,59 +48,26 @@ jobs:
|
|
43
48
|
- uses: actions/checkout@v2
|
44
49
|
with:
|
45
50
|
path: foreman_remote_execution
|
51
|
+
- name: Setup Bundler
|
52
|
+
run: |
|
53
|
+
echo "gem 'foreman_remote_execution', path: './foreman_remote_execution'" > bundler.d/foreman_remote_execution.local.rb
|
46
54
|
- name: Setup Ruby
|
47
55
|
uses: ruby/setup-ruby@v1
|
48
56
|
with:
|
49
57
|
ruby-version: ${{ matrix.ruby-version }}
|
58
|
+
bundler-cache: true
|
50
59
|
- name: Setup Node
|
60
|
+
if: github.event_name != 'push'
|
51
61
|
uses: actions/setup-node@v1
|
52
62
|
with:
|
53
63
|
node-version: ${{ matrix.node-version }}
|
54
|
-
-
|
55
|
-
|
56
|
-
path: vendor/bundle
|
57
|
-
key: ${{ runner.os }}-fgems-${{ matrix.ruby-version }}-${{ hashFiles('Gemfile.lock') }}
|
58
|
-
restore-keys: |
|
59
|
-
${{ runner.os }}-fgems-${{ matrix.ruby-version }}-
|
60
|
-
- name: Setup Bundler
|
61
|
-
run: |
|
62
|
-
echo "gem 'foreman_remote_execution', path: './foreman_remote_execution'" > bundler.d/foreman_remote_execution.local.rb
|
63
|
-
gem install bundler
|
64
|
-
bundle config set without journald development console libvirt
|
65
|
-
bundle config set path vendor/bundle
|
66
|
-
- name: Prepare test env
|
64
|
+
- name: Prepare test DB
|
65
|
+
if: github.event_name != 'push'
|
67
66
|
run: |
|
68
|
-
bundle install --jobs=3 --retry=3
|
69
67
|
bundle exec rake db:create
|
70
68
|
bundle exec rake db:migrate
|
71
69
|
- name: Run plugin tests
|
70
|
+
if: github.event_name != 'push'
|
72
71
|
run: |
|
73
72
|
bundle exec rake test:foreman_remote_execution
|
74
73
|
bundle exec rake test TEST="test/unit/foreman/access_permissions_test.rb"
|
75
|
-
test_js:
|
76
|
-
runs-on: ubuntu-latest
|
77
|
-
needs: rubocop
|
78
|
-
strategy:
|
79
|
-
fail-fast: false
|
80
|
-
matrix:
|
81
|
-
ruby-version: [2.6]
|
82
|
-
node-version: [10, 12]
|
83
|
-
steps:
|
84
|
-
- uses: actions/checkout@v2
|
85
|
-
- name: Setup Ruby
|
86
|
-
uses: ruby/setup-ruby@v1
|
87
|
-
with:
|
88
|
-
ruby-version: ${{ matrix.ruby-version }}
|
89
|
-
- name: Setup Node
|
90
|
-
uses: actions/setup-node@v1
|
91
|
-
with:
|
92
|
-
node-version: ${{ matrix.node-version }}
|
93
|
-
- name: Nmp install
|
94
|
-
run: |
|
95
|
-
npm install
|
96
|
-
- name: Run plugin linter
|
97
|
-
run: |
|
98
|
-
npm run lint
|
99
|
-
- name: Run plugin tests
|
100
|
-
run: |
|
101
|
-
npm run test
|
data/.prettierrc
ADDED
data/.rubocop.yml
CHANGED
@@ -1,29 +1,27 @@
|
|
1
|
-
|
2
|
-
-
|
3
|
-
|
4
|
-
|
1
|
+
inherit_gem:
|
2
|
+
theforeman-rubocop:
|
3
|
+
- lenient.yml
|
4
|
+
- minitest.yml
|
5
5
|
|
6
6
|
inherit_from: .rubocop_todo.yml
|
7
7
|
|
8
|
+
inherit_mode:
|
9
|
+
merge:
|
10
|
+
- Exclude
|
11
|
+
|
8
12
|
AllCops:
|
9
13
|
TargetRubyVersion: 2.5
|
10
14
|
TargetRailsVersion: 5.2
|
11
15
|
Exclude:
|
12
16
|
- 'node_modules/**/*'
|
17
|
+
- 'vendor/bundle/**/*'
|
13
18
|
|
14
19
|
Bundler/OrderedGems:
|
15
20
|
Enabled: false
|
16
21
|
|
17
|
-
Layout/ArgumentAlignment:
|
18
|
-
EnforcedStyle: with_fixed_indentation
|
19
|
-
IndentationWidth: 2
|
20
|
-
|
21
22
|
Layout/DotPosition:
|
22
23
|
Enabled: false
|
23
24
|
|
24
|
-
Layout/FirstArgumentIndentation:
|
25
|
-
EnforcedStyle: consistent
|
26
|
-
|
27
25
|
Layout/HashAlignment:
|
28
26
|
Enabled: false
|
29
27
|
|
@@ -33,6 +31,9 @@ Layout/LineLength:
|
|
33
31
|
Layout/ParameterAlignment:
|
34
32
|
Enabled: false
|
35
33
|
|
34
|
+
Layout/SpaceInsideHashLiteralBraces:
|
35
|
+
Enabled: false
|
36
|
+
|
36
37
|
Lint/BooleanSymbol:
|
37
38
|
Enabled: false
|
38
39
|
|
@@ -42,12 +43,6 @@ Metrics:
|
|
42
43
|
Performance/Casecmp:
|
43
44
|
Enabled: false
|
44
45
|
|
45
|
-
Performance/RedundantMatch:
|
46
|
-
Enabled: true
|
47
|
-
|
48
|
-
Performance/RedundantMerge:
|
49
|
-
Enabled: true
|
50
|
-
|
51
46
|
Performance/RegexpMatch:
|
52
47
|
Enabled: false
|
53
48
|
|
@@ -66,12 +61,6 @@ Rails/BulkChangeTable:
|
|
66
61
|
Rails/RefuteMethods:
|
67
62
|
Enabled: false
|
68
63
|
|
69
|
-
Style/Alias:
|
70
|
-
EnforcedStyle: prefer_alias_method
|
71
|
-
|
72
|
-
Style/AndOr:
|
73
|
-
EnforcedStyle: conditionals
|
74
|
-
|
75
64
|
# Don't prefer is_a? over kind_of?
|
76
65
|
Style/ClassCheck:
|
77
66
|
Enabled: false
|
@@ -83,13 +72,6 @@ Style/CollectionMethods:
|
|
83
72
|
Style/ConditionalAssignment:
|
84
73
|
Enabled: false
|
85
74
|
|
86
|
-
# Don't enforce documentation
|
87
|
-
Style/Documentation:
|
88
|
-
Enabled: false
|
89
|
-
|
90
|
-
Style/EmptyMethod:
|
91
|
-
EnforcedStyle: expanded
|
92
|
-
|
93
75
|
# Don't enforce frozen string literals
|
94
76
|
Style/FrozenStringLiteralComment:
|
95
77
|
Enabled: false
|
@@ -104,10 +86,6 @@ Style/IfUnlessModifier:
|
|
104
86
|
Style/InverseMethods:
|
105
87
|
Enabled: false
|
106
88
|
|
107
|
-
# disabled until we can configure "+" as concat sign
|
108
|
-
Style/LineEndConcatenation:
|
109
|
-
Enabled: false
|
110
|
-
|
111
89
|
Style/MultipleComparison:
|
112
90
|
Enabled: false
|
113
91
|
|
@@ -123,19 +101,5 @@ Style/ParenthesesAroundCondition:
|
|
123
101
|
Style/PreferredHashMethods:
|
124
102
|
Enabled: false
|
125
103
|
|
126
|
-
|
127
|
-
Style/StringLiterals:
|
128
|
-
Enabled: false
|
129
|
-
|
130
|
-
Style/TernaryParentheses:
|
131
|
-
EnforcedStyle: require_parentheses_when_complex
|
132
|
-
|
133
|
-
Style/TrailingCommaInArrayLiteral:
|
134
|
-
EnforcedStyleForMultiline: comma
|
135
|
-
|
136
|
-
Style/TrailingCommaInHashLiteral:
|
137
|
-
EnforcedStyleForMultiline: comma
|
138
|
-
|
139
|
-
#Allow both ['a', 'b'], %w[a b] and %w(a b) style arrays
|
140
|
-
Style/WordArray:
|
104
|
+
Style/RaiseArgs:
|
141
105
|
Enabled: false
|
data/.rubocop_todo.yml
CHANGED
@@ -1,41 +1,67 @@
|
|
1
1
|
# This configuration was generated by
|
2
|
-
# `rubocop --auto-gen-config
|
3
|
-
# on 2020-
|
2
|
+
# `rubocop --auto-gen-config`
|
3
|
+
# on 2020-12-02 23:05:20 UTC using RuboCop version 1.5.1.
|
4
4
|
# The point is for the user to remove these configuration records
|
5
5
|
# one by one as the offenses are removed from the code base.
|
6
6
|
# Note that changes in the inspected code, or installation of new
|
7
7
|
# versions of RuboCop, may require this file to be generated again.
|
8
8
|
|
9
|
-
Minitest/GlobalExpectations:
|
10
|
-
Enabled: false
|
11
|
-
|
12
9
|
# Offense count: 2
|
13
|
-
#
|
14
|
-
# Configuration parameters: TreatCommentsAsGroupSeparators, Include.
|
10
|
+
# Configuration parameters: Include.
|
15
11
|
# Include: **/*.gemspec
|
16
|
-
Gemspec/
|
17
|
-
|
12
|
+
Gemspec/RequiredRubyVersion:
|
13
|
+
Exclude:
|
14
|
+
- 'foreman_remote_execution.gemspec'
|
15
|
+
- 'foreman_remote_execution_core.gemspec'
|
18
16
|
|
19
|
-
# Offense count:
|
17
|
+
# Offense count: 1
|
18
|
+
# Cop supports --auto-correct.
|
19
|
+
# Configuration parameters: EnforcedStyleAlignWith, Severity.
|
20
|
+
# SupportedStylesAlignWith: start_of_line, begin
|
21
|
+
Layout/BeginEndAlignment:
|
22
|
+
Exclude:
|
23
|
+
- 'lib/foreman_remote_execution_core/script_runner.rb'
|
24
|
+
|
25
|
+
# Offense count: 13
|
20
26
|
# Cop supports --auto-correct.
|
21
27
|
# Configuration parameters: EnforcedStyle, IndentOneStep, IndentationWidth.
|
22
28
|
# SupportedStyles: case, end
|
23
29
|
Layout/CaseIndentation:
|
24
|
-
|
30
|
+
Exclude:
|
31
|
+
- 'app/controllers/job_invocations_controller.rb'
|
32
|
+
- 'app/controllers/job_templates_controller.rb'
|
33
|
+
- 'app/models/host_status/execution_status.rb'
|
25
34
|
|
26
35
|
# Offense count: 4
|
27
36
|
# Cop supports --auto-correct.
|
28
37
|
Layout/EmptyLines:
|
29
|
-
|
38
|
+
Exclude:
|
39
|
+
- 'app/models/template_invocation.rb'
|
40
|
+
- 'test/unit/input_template_renderer_test.rb'
|
41
|
+
- 'test/unit/job_invocation_composer_test.rb'
|
42
|
+
- 'test/unit/remote_execution_feature_test.rb'
|
43
|
+
|
44
|
+
# Offense count: 3
|
45
|
+
# Cop supports --auto-correct.
|
46
|
+
# Configuration parameters: AllowAliasSyntax, AllowedMethods.
|
47
|
+
# AllowedMethods: alias_method, public, protected, private
|
48
|
+
Layout/EmptyLinesAroundAttributeAccessor:
|
49
|
+
Exclude:
|
50
|
+
- 'app/lib/foreman_remote_execution/renderer/scope/input.rb'
|
51
|
+
- 'app/models/job_invocation_composer.rb'
|
30
52
|
|
31
53
|
# Offense count: 6
|
32
54
|
# Cop supports --auto-correct.
|
33
55
|
# Configuration parameters: EnforcedStyle.
|
34
56
|
# SupportedStyles: empty_lines, no_empty_lines
|
35
57
|
Layout/EmptyLinesAroundBlockBody:
|
36
|
-
|
58
|
+
Exclude:
|
59
|
+
- 'extra/cockpit/foreman-cockpit-session'
|
60
|
+
- 'test/unit/input_template_renderer_test.rb'
|
61
|
+
- 'test/unit/job_invocation_composer_test.rb'
|
62
|
+
- 'test/unit/targeting_test.rb'
|
37
63
|
|
38
|
-
# Offense count:
|
64
|
+
# Offense count: 21
|
39
65
|
# Cop supports --auto-correct.
|
40
66
|
# Configuration parameters: EnforcedStyle.
|
41
67
|
# SupportedStyles: empty_lines, empty_lines_except_namespace, empty_lines_special, no_empty_lines, beginning_only, ending_only
|
@@ -47,208 +73,377 @@ Layout/EmptyLinesAroundClassBody:
|
|
47
73
|
# Configuration parameters: EnforcedStyle.
|
48
74
|
# SupportedStyles: empty_lines, empty_lines_except_namespace, empty_lines_special, no_empty_lines
|
49
75
|
Layout/EmptyLinesAroundModuleBody:
|
50
|
-
|
76
|
+
Exclude:
|
77
|
+
- 'app/helpers/concerns/foreman_remote_execution/hosts_helper_extensions.rb'
|
78
|
+
- 'app/lib/actions/middleware/bind_job_invocation.rb'
|
79
|
+
- 'test/functional/api/v2/job_templates_controller_test.rb'
|
51
80
|
|
52
81
|
# Offense count: 6
|
53
82
|
# Cop supports --auto-correct.
|
54
83
|
# Configuration parameters: Width, IgnoredPatterns.
|
55
84
|
Layout/IndentationWidth:
|
56
|
-
|
85
|
+
Exclude:
|
86
|
+
- 'app/models/template_invocation.rb'
|
87
|
+
- 'extra/cockpit/foreman-cockpit-session'
|
57
88
|
|
58
|
-
# Offense count:
|
89
|
+
# Offense count: 7
|
59
90
|
# Cop supports --auto-correct.
|
60
91
|
# Configuration parameters: EnforcedStyle, IndentationWidth.
|
61
92
|
# SupportedStyles: aligned, indented
|
62
93
|
Layout/MultilineOperationIndentation:
|
63
|
-
|
94
|
+
Exclude:
|
95
|
+
- 'app/helpers/job_invocations_helper.rb'
|
96
|
+
- 'app/helpers/remote_execution_helper.rb'
|
97
|
+
- 'app/models/job_invocation_composer.rb'
|
98
|
+
- 'app/models/remote_execution_provider.rb'
|
99
|
+
- 'lib/foreman_remote_execution_core/dispatcher.rb'
|
64
100
|
|
65
|
-
# Offense count:
|
101
|
+
# Offense count: 7
|
66
102
|
# Cop supports --auto-correct.
|
67
103
|
Layout/SpaceAfterComma:
|
68
|
-
|
104
|
+
Exclude:
|
105
|
+
- 'app/helpers/job_invocation_output_helper.rb'
|
106
|
+
- 'db/migrate/20200623073022_rename_sudo_password_to_effective_user_password.rb'
|
107
|
+
- 'test/functional/ui_job_wizard_controller_test.rb'
|
108
|
+
- 'test/unit/job_invocation_composer_test.rb'
|
69
109
|
|
70
|
-
# Offense count:
|
110
|
+
# Offense count: 29
|
71
111
|
# Cop supports --auto-correct.
|
72
112
|
# Configuration parameters: AllowForAlignment, EnforcedStyleForExponentOperator.
|
73
113
|
# SupportedStylesForExponentOperator: space, no_space
|
74
114
|
Layout/SpaceAroundOperators:
|
75
|
-
|
76
|
-
|
77
|
-
|
115
|
+
Exclude:
|
116
|
+
- 'app/controllers/ui_job_wizard_controller.rb'
|
117
|
+
- 'app/helpers/job_invocations_helper.rb'
|
118
|
+
- 'db/migrate/20150708133242_add_invocation.rb'
|
119
|
+
- 'extra/cockpit/foreman-cockpit-session'
|
120
|
+
- 'test/helpers/remote_execution_helper_test.rb'
|
121
|
+
- 'test/unit/input_template_renderer_test.rb'
|
122
|
+
- 'test/unit/job_invocation_composer_test.rb'
|
123
|
+
|
124
|
+
# Offense count: 58
|
78
125
|
# Cop supports --auto-correct.
|
79
126
|
# Configuration parameters: EnforcedStyle, EnforcedStyleForEmptyBrackets.
|
80
127
|
# SupportedStyles: space, no_space, compact
|
81
128
|
# SupportedStylesForEmptyBrackets: space, no_space
|
82
129
|
Layout/SpaceInsideArrayLiteralBrackets:
|
83
|
-
|
130
|
+
Exclude:
|
131
|
+
- 'app/helpers/concerns/foreman_remote_execution/hosts_helper_extensions.rb'
|
132
|
+
- 'app/helpers/remote_execution_helper.rb'
|
133
|
+
- 'app/lib/actions/remote_execution/run_host_job.rb'
|
134
|
+
- 'app/models/host_status/execution_status.rb'
|
135
|
+
- 'app/models/job_invocation.rb'
|
136
|
+
- 'app/models/job_invocation_composer.rb'
|
137
|
+
- 'app/models/job_template.rb'
|
138
|
+
- 'app/models/remote_execution_feature.rb'
|
139
|
+
- 'extra/cockpit/foreman-cockpit-session'
|
140
|
+
- 'lib/foreman_remote_execution/engine.rb'
|
141
|
+
- 'test/unit/execution_task_status_mapper_test.rb'
|
142
|
+
- 'test/unit/job_invocation_composer_test.rb'
|
143
|
+
- 'test/unit/targeting_test.rb'
|
84
144
|
|
85
|
-
# Offense count:
|
86
|
-
#
|
87
|
-
#
|
88
|
-
|
89
|
-
|
90
|
-
|
91
|
-
|
145
|
+
# Offense count: 2
|
146
|
+
# Configuration parameters: AllowedMethods.
|
147
|
+
# AllowedMethods: enums
|
148
|
+
Lint/ConstantDefinitionInBlock:
|
149
|
+
Exclude:
|
150
|
+
- 'lib/foreman_remote_execution/engine.rb'
|
151
|
+
|
152
|
+
# Offense count: 2
|
153
|
+
Lint/MissingSuper:
|
154
|
+
Exclude:
|
155
|
+
- 'app/services/ui_notifications/remote_execution_jobs/base_job_finish.rb'
|
156
|
+
- 'lib/foreman_remote_execution_core/log_filter.rb'
|
92
157
|
|
93
158
|
# Offense count: 10
|
94
159
|
# Cop supports --auto-correct.
|
95
160
|
# Configuration parameters: IgnoreEmptyBlocks, AllowUnusedKeywordArguments.
|
96
161
|
Lint/UnusedBlockArgument:
|
97
|
-
|
162
|
+
Exclude:
|
163
|
+
- 'app/models/job_invocation_composer.rb'
|
164
|
+
- 'extra/cockpit/foreman-cockpit-session'
|
165
|
+
- 'lib/foreman_remote_execution/engine.rb'
|
166
|
+
- 'lib/foreman_remote_execution_core/script_runner.rb'
|
167
|
+
- 'test/factories/foreman_remote_execution_factories.rb'
|
98
168
|
|
99
169
|
# Offense count: 15
|
100
170
|
# Cop supports --auto-correct.
|
101
|
-
# Configuration parameters: AllowUnusedKeywordArguments, IgnoreEmptyMethods.
|
171
|
+
# Configuration parameters: AllowUnusedKeywordArguments, IgnoreEmptyMethods, IgnoreNotImplementedMethods.
|
102
172
|
Lint/UnusedMethodArgument:
|
103
|
-
|
173
|
+
Exclude:
|
174
|
+
- 'app/helpers/concerns/foreman_remote_execution/hosts_helper_extensions.rb'
|
175
|
+
- 'app/helpers/remote_execution_helper.rb'
|
176
|
+
- 'app/lib/actions/remote_execution/run_host_job.rb'
|
177
|
+
- 'app/models/host_status/execution_status.rb'
|
178
|
+
- 'app/models/job_invocation.rb'
|
179
|
+
- 'app/models/job_invocation_composer.rb'
|
180
|
+
- 'app/models/job_template.rb'
|
181
|
+
- 'app/models/remote_execution_provider.rb'
|
182
|
+
- 'lib/foreman_remote_execution_core/script_runner.rb'
|
183
|
+
- 'test/benchmark/run_hosts_job_benchmark.rb'
|
184
|
+
- 'test/support/remote_execution_helper.rb'
|
104
185
|
|
105
186
|
# Offense count: 4
|
106
187
|
Lint/UselessAssignment:
|
107
|
-
|
188
|
+
Exclude:
|
189
|
+
- 'app/lib/foreman_remote_execution/renderer/scope/input.rb'
|
190
|
+
- 'app/models/job_template.rb'
|
191
|
+
- 'app/models/remote_execution_feature.rb'
|
192
|
+
- 'extra/cockpit/foreman-cockpit-session'
|
108
193
|
|
109
|
-
# Offense count:
|
194
|
+
# Offense count: 31
|
195
|
+
# Cop supports --auto-correct.
|
196
|
+
Minitest/GlobalExpectations:
|
197
|
+
Exclude:
|
198
|
+
- 'test/helpers/remote_execution_helper_test.rb'
|
199
|
+
- 'test/unit/actions/run_hosts_job_test.rb'
|
200
|
+
- 'test/unit/concerns/foreman_tasks_cleaner_extensions_test.rb'
|
201
|
+
- 'test/unit/concerns/host_extensions_test.rb'
|
202
|
+
- 'test/unit/job_invocation_test.rb'
|
203
|
+
- 'test/unit/remote_execution_provider_test.rb'
|
204
|
+
- 'test/unit/targeting_test.rb'
|
205
|
+
|
206
|
+
# Offense count: 2
|
110
207
|
Naming/AccessorMethodName:
|
111
|
-
|
208
|
+
Exclude:
|
209
|
+
- 'app/lib/actions/remote_execution/run_hosts_job.rb'
|
210
|
+
- 'app/models/concerns/foreman_remote_execution/host_extensions.rb'
|
112
211
|
|
113
|
-
# Offense count:
|
114
|
-
# Configuration parameters: ExpectMatchingDefinition, Regex, IgnoreExecutableScripts, AllowedAcronyms.
|
212
|
+
# Offense count: 6
|
213
|
+
# Configuration parameters: ExpectMatchingDefinition, CheckDefinitionPathHierarchy, Regex, IgnoreExecutableScripts, AllowedAcronyms.
|
115
214
|
# AllowedAcronyms: CLI, DSL, ACL, API, ASCII, CPU, CSS, DNS, EOF, GUID, HTML, HTTP, HTTPS, ID, IP, JSON, LHS, QPS, RAM, RHS, RPC, SLA, SMTP, SQL, SSH, TCP, TLS, TTL, UDP, UI, UID, UUID, URI, URL, UTF8, VM, XML, XMPP, XSRF, XSS
|
116
215
|
Naming/FileName:
|
117
|
-
|
216
|
+
Exclude:
|
217
|
+
- 'db/seeds.d/100-assign_features_with_templates.rb'
|
218
|
+
- 'db/seeds.d/20-permissions.rb'
|
219
|
+
- 'db/seeds.d/50-notification_blueprints.rb'
|
220
|
+
- 'db/seeds.d/60-ssh_proxy_feature.rb'
|
221
|
+
- 'db/seeds.d/70-job_templates.rb'
|
222
|
+
- 'db/seeds.d/90-bookmarks.rb'
|
118
223
|
|
119
224
|
# Offense count: 1
|
120
225
|
# Configuration parameters: ForbiddenDelimiters.
|
121
226
|
# ForbiddenDelimiters: (?-mix:(^|\s)(EO[A-Z]{1}|END)(\s|$))
|
122
227
|
Naming/HeredocDelimiterNaming:
|
123
|
-
|
228
|
+
Exclude:
|
229
|
+
- 'lib/foreman_remote_execution_core/fake_script_runner.rb'
|
124
230
|
|
125
231
|
# Offense count: 2
|
126
232
|
# Configuration parameters: MinNameLength, AllowNamesEndingInNumbers, AllowedNames, ForbiddenNames.
|
127
|
-
# AllowedNames:
|
233
|
+
# AllowedNames: at, by, db, id, in, io, ip, of, on, os, pp, to
|
128
234
|
Naming/MethodParameterName:
|
129
|
-
|
235
|
+
Exclude:
|
236
|
+
- 'app/helpers/remote_execution_helper.rb'
|
130
237
|
|
131
238
|
# Offense count: 1
|
132
239
|
# Cop supports --auto-correct.
|
133
240
|
# Configuration parameters: PreferredName.
|
134
241
|
Naming/RescuedExceptionsVariableName:
|
135
|
-
|
242
|
+
Exclude:
|
243
|
+
- 'Rakefile'
|
136
244
|
|
137
245
|
# Offense count: 1
|
138
|
-
# Configuration parameters:
|
139
|
-
|
140
|
-
|
141
|
-
|
246
|
+
# Configuration parameters: MinSize.
|
247
|
+
Performance/CollectionLiteralInLoop:
|
248
|
+
Exclude:
|
249
|
+
- 'db/seeds.d/90-bookmarks.rb'
|
250
|
+
|
251
|
+
# Offense count: 11
|
252
|
+
# Cop supports --auto-correct.
|
253
|
+
Rails/ContentTag:
|
254
|
+
Exclude:
|
255
|
+
- 'app/helpers/job_invocations_helper.rb'
|
256
|
+
- 'app/helpers/remote_execution_helper.rb'
|
142
257
|
|
143
258
|
# Offense count: 7
|
144
259
|
# Configuration parameters: Include.
|
145
260
|
# Include: db/migrate/*.rb
|
146
261
|
Rails/CreateTableWithTimestamps:
|
147
|
-
|
262
|
+
Exclude:
|
263
|
+
- 'db/migrate/20150708133241_add_targeting.rb'
|
264
|
+
- 'db/migrate/20150708133242_add_invocation.rb'
|
265
|
+
- 'db/migrate/20150708133305_add_template_invocation.rb'
|
266
|
+
- 'db/migrate/20151124162300_create_job_template_effective_users.rb'
|
267
|
+
- 'db/migrate/20160108134600_create_template_input_sets.rb'
|
268
|
+
- 'db/migrate/20160118124600_create_remote_execution_features.rb'
|
148
269
|
|
149
270
|
# Offense count: 1
|
150
271
|
# Configuration parameters: EnforcedStyle.
|
151
272
|
# SupportedStyles: strict, flexible
|
152
273
|
Rails/Date:
|
153
|
-
|
274
|
+
Exclude:
|
275
|
+
- 'foreman_remote_execution.gemspec'
|
154
276
|
|
155
|
-
# Offense count:
|
277
|
+
# Offense count: 11
|
156
278
|
# Configuration parameters: Include.
|
157
279
|
# Include: app/helpers/**/*.rb
|
158
280
|
Rails/HelperInstanceVariable:
|
159
|
-
|
281
|
+
Exclude:
|
282
|
+
- 'app/helpers/job_invocation_output_helper.rb'
|
283
|
+
- 'app/helpers/job_invocations_chart_helper.rb'
|
284
|
+
- 'app/helpers/remote_execution_helper.rb'
|
160
285
|
|
161
286
|
# Offense count: 13
|
162
287
|
# Configuration parameters: Include.
|
163
288
|
# Include: app/models/**/*.rb
|
164
289
|
Rails/InverseOf:
|
165
|
-
|
290
|
+
Exclude:
|
291
|
+
- 'app/models/concerns/foreman_remote_execution/foreman_tasks_task_extensions.rb'
|
292
|
+
- 'app/models/concerns/foreman_remote_execution/foreman_tasks_triggering_extensions.rb'
|
293
|
+
- 'app/models/concerns/foreman_remote_execution/host_extensions.rb'
|
294
|
+
- 'app/models/concerns/foreman_remote_execution/template_extensions.rb'
|
295
|
+
- 'app/models/job_invocation_task_group.rb'
|
296
|
+
- 'app/models/job_template.rb'
|
297
|
+
- 'app/models/template_invocation.rb'
|
166
298
|
|
167
299
|
# Offense count: 1
|
168
300
|
# Configuration parameters: Include.
|
169
301
|
# Include: app/controllers/**/*.rb
|
170
302
|
Rails/LexicallyScopedActionFilter:
|
171
|
-
|
303
|
+
Exclude:
|
304
|
+
- 'app/controllers/api/v2/job_invocations_controller.rb'
|
172
305
|
|
173
306
|
# Offense count: 1
|
307
|
+
# Cop supports --auto-correct.
|
174
308
|
# Configuration parameters: Include.
|
175
309
|
# Include: **/Rakefile, **/*.rake
|
176
310
|
Rails/RakeEnvironment:
|
177
|
-
|
311
|
+
Exclude:
|
312
|
+
- 'lib/tasks/foreman_remote_execution_tasks.rake'
|
313
|
+
|
314
|
+
# Offense count: 3
|
315
|
+
# Cop supports --auto-correct.
|
316
|
+
Rails/RedundantForeignKey:
|
317
|
+
Exclude:
|
318
|
+
- 'app/models/job_template.rb'
|
319
|
+
- 'app/models/template_invocation.rb'
|
178
320
|
|
179
321
|
# Offense count: 1
|
180
322
|
Rails/ReflectionClassName:
|
181
|
-
|
323
|
+
Exclude:
|
324
|
+
- 'app/models/job_template.rb'
|
182
325
|
|
183
|
-
# Offense count:
|
184
|
-
# Configuration parameters:
|
185
|
-
#
|
326
|
+
# Offense count: 4
|
327
|
+
# Configuration parameters: ForbiddenMethods, AllowedMethods.
|
328
|
+
# ForbiddenMethods: decrement!, decrement_counter, increment!, increment_counter, insert, insert!, insert_all, insert_all!, toggle!, touch, touch_all, update_all, update_attribute, update_column, update_columns, update_counters, upsert, upsert_all
|
186
329
|
Rails/SkipsModelValidations:
|
187
|
-
|
330
|
+
Exclude:
|
331
|
+
- 'app/lib/actions/middleware/bind_job_invocation.rb'
|
332
|
+
- 'app/models/concerns/foreman_remote_execution/smart_proxy_extensions.rb'
|
333
|
+
- 'test/unit/input_template_renderer_test.rb'
|
334
|
+
- 'test/unit/job_invocation_composer_test.rb'
|
188
335
|
|
189
336
|
# Offense count: 6
|
190
337
|
# Cop supports --auto-correct.
|
191
338
|
# Configuration parameters: EnforcedStyle.
|
192
339
|
# SupportedStyles: strict, flexible
|
193
340
|
Rails/TimeZone:
|
194
|
-
|
341
|
+
Exclude:
|
342
|
+
- 'app/models/job_invocation_composer.rb'
|
343
|
+
- 'test/functional/api/v2/job_invocations_controller_test.rb'
|
344
|
+
- 'test/unit/job_invocation_composer_test.rb'
|
195
345
|
|
196
|
-
# Offense count:
|
346
|
+
# Offense count: 2
|
197
347
|
# Cop supports --auto-correct.
|
198
|
-
|
348
|
+
Style/CaseLikeIf:
|
349
|
+
Exclude:
|
350
|
+
- 'app/lib/actions/remote_execution/run_host_job.rb'
|
351
|
+
- 'app/models/job_invocation_composer.rb'
|
352
|
+
|
353
|
+
# Offense count: 5
|
354
|
+
# Cop supports --auto-correct.
|
355
|
+
# Configuration parameters: EnforcedStyle.
|
199
356
|
# SupportedStyles: nested, compact
|
200
357
|
Style/ClassAndModuleChildren:
|
201
|
-
|
358
|
+
Exclude:
|
359
|
+
- 'app/controllers/foreman_remote_execution/concerns/api/v2/registration_controller_extensions.rb'
|
360
|
+
- 'app/controllers/foreman_remote_execution/concerns/api/v2/subnets_controller_extensions.rb'
|
361
|
+
- 'app/models/concerns/foreman_remote_execution/orchestration/ssh.rb'
|
362
|
+
- 'app/models/host_status/execution_status.rb'
|
363
|
+
- 'app/models/setting/remote_execution.rb'
|
364
|
+
|
365
|
+
# Offense count: 3
|
366
|
+
# Cop supports --auto-correct.
|
367
|
+
# Configuration parameters: IgnoredMethods.
|
368
|
+
# IgnoredMethods: ==, equal?, eql?
|
369
|
+
Style/ClassEqualityComparison:
|
370
|
+
Exclude:
|
371
|
+
- 'app/helpers/job_invocations_chart_helper.rb'
|
372
|
+
- 'app/helpers/remote_execution_helper.rb'
|
373
|
+
- 'app/models/ssh_execution_provider.rb'
|
202
374
|
|
203
375
|
# Offense count: 2
|
204
376
|
# Cop supports --auto-correct.
|
205
377
|
# Configuration parameters: Keywords.
|
206
|
-
# Keywords: TODO, FIXME, OPTIMIZE, HACK, REVIEW
|
378
|
+
# Keywords: TODO, FIXME, OPTIMIZE, HACK, REVIEW, NOTE
|
207
379
|
Style/CommentAnnotation:
|
208
|
-
|
209
|
-
|
210
|
-
|
211
|
-
Style/DoubleNegation:
|
212
|
-
Enabled: false
|
380
|
+
Exclude:
|
381
|
+
- 'app/models/job_invocation_composer.rb'
|
382
|
+
- 'db/seeds.d/90-bookmarks.rb'
|
213
383
|
|
214
384
|
# Offense count: 1
|
215
385
|
# Cop supports --auto-correct.
|
216
386
|
Style/Encoding:
|
217
|
-
|
387
|
+
Exclude:
|
388
|
+
- 'foreman_remote_execution_core.gemspec'
|
218
389
|
|
219
390
|
# Offense count: 8
|
220
391
|
# Cop supports --auto-correct.
|
221
392
|
Style/ExpandPathArguments:
|
222
|
-
|
393
|
+
Exclude:
|
394
|
+
- 'Rakefile'
|
395
|
+
- 'foreman_remote_execution.gemspec'
|
396
|
+
- 'foreman_remote_execution_core.gemspec'
|
397
|
+
- 'lib/foreman_remote_execution/engine.rb'
|
398
|
+
- 'lib/foreman_remote_execution_core/polling_script_runner.rb'
|
399
|
+
- 'test/benchmark/run_hosts_job_benchmark.rb'
|
400
|
+
- 'test/benchmark/targeting_benchmark.rb'
|
223
401
|
|
224
402
|
# Offense count: 12
|
225
403
|
# Cop supports --auto-correct.
|
226
404
|
# Configuration parameters: EnforcedStyle.
|
227
405
|
# SupportedStyles: format, sprintf, percent
|
228
406
|
Style/FormatString:
|
229
|
-
|
230
|
-
|
231
|
-
|
232
|
-
|
407
|
+
Exclude:
|
408
|
+
- 'app/controllers/api/v2/job_invocations_controller.rb'
|
409
|
+
- 'app/controllers/job_templates_controller.rb'
|
410
|
+
- 'app/helpers/job_invocations_chart_helper.rb'
|
411
|
+
- 'app/lib/actions/remote_execution/run_host_job.rb'
|
412
|
+
- 'app/lib/actions/remote_execution/run_hosts_job.rb'
|
413
|
+
- 'app/models/job_invocation_composer.rb'
|
414
|
+
- 'app/models/remote_execution_provider.rb'
|
415
|
+
- 'app/models/ssh_execution_provider.rb'
|
416
|
+
- 'extra/cockpit/foreman-cockpit-session'
|
417
|
+
|
418
|
+
# Offense count: 61
|
419
|
+
# Configuration parameters: MaxUnannotatedPlaceholdersAllowed.
|
233
420
|
# SupportedStyles: annotated, template, unannotated
|
234
421
|
Style/FormatStringToken:
|
235
|
-
|
422
|
+
EnforcedStyle: template
|
236
423
|
|
237
|
-
# Offense count:
|
424
|
+
# Offense count: 23
|
238
425
|
# Configuration parameters: MinBodyLength.
|
239
426
|
Style/GuardClause:
|
240
427
|
Enabled: false
|
241
428
|
|
429
|
+
# Offense count: 1
|
430
|
+
# Cop supports --auto-correct.
|
242
431
|
Style/HashEachMethods:
|
243
|
-
|
244
|
-
|
245
|
-
Style/HashTransformKeys:
|
246
|
-
Enabled: false
|
432
|
+
Exclude:
|
433
|
+
- 'app/models/concerns/foreman_remote_execution/errors_flattener.rb'
|
247
434
|
|
248
|
-
|
249
|
-
|
250
|
-
|
251
|
-
|
435
|
+
# Offense count: 6
|
436
|
+
# Configuration parameters: AllowedMethods.
|
437
|
+
# AllowedMethods: respond_to_missing?
|
438
|
+
Style/OptionalBooleanParameter:
|
439
|
+
Exclude:
|
440
|
+
- 'app/helpers/job_invocations_chart_helper.rb'
|
441
|
+
- 'app/models/concerns/foreman_remote_execution/host_extensions.rb'
|
442
|
+
- 'app/models/input_template_renderer.rb'
|
443
|
+
- 'app/models/job_invocation.rb'
|
444
|
+
- 'app/models/job_invocation_composer.rb'
|
445
|
+
|
446
|
+
# Offense count: 29
|
252
447
|
# Cop supports --auto-correct.
|
253
448
|
# Configuration parameters: PreferredDelimiters.
|
254
449
|
Style/PercentLiteralDelimiters:
|
@@ -256,18 +451,33 @@ Style/PercentLiteralDelimiters:
|
|
256
451
|
|
257
452
|
# Offense count: 1
|
258
453
|
# Cop supports --auto-correct.
|
259
|
-
# Configuration parameters:
|
454
|
+
# Configuration parameters: AllowedCompactTypes.
|
260
455
|
# SupportedStyles: compact, exploded
|
261
456
|
Style/RaiseArgs:
|
262
|
-
|
457
|
+
EnforcedStyle: exploded
|
263
458
|
|
264
|
-
# Offense count:
|
459
|
+
# Offense count: 1
|
460
|
+
# Cop supports --auto-correct.
|
461
|
+
Style/RedundantBegin:
|
462
|
+
Exclude:
|
463
|
+
- 'extra/cockpit/foreman-cockpit-session'
|
464
|
+
|
465
|
+
# Offense count: 16
|
265
466
|
# Cop supports --auto-correct.
|
266
467
|
# Configuration parameters: AllowMultipleReturnValues.
|
267
468
|
Style/RedundantReturn:
|
268
|
-
|
269
|
-
|
270
|
-
|
469
|
+
Exclude:
|
470
|
+
- 'app/helpers/remote_execution_helper.rb'
|
471
|
+
- 'app/models/concerns/foreman_remote_execution/errors_flattener.rb'
|
472
|
+
- 'app/models/foreign_input_set.rb'
|
473
|
+
- 'app/models/host_status/execution_status.rb'
|
474
|
+
- 'app/models/job_invocation.rb'
|
475
|
+
- 'extra/cockpit/foreman-cockpit-session'
|
476
|
+
- 'lib/foreman_remote_execution_core/polling_script_runner.rb'
|
477
|
+
- 'lib/foreman_remote_execution_core/script_runner.rb'
|
478
|
+
- 'test/support/remote_execution_helper.rb'
|
479
|
+
|
480
|
+
# Offense count: 79
|
271
481
|
# Cop supports --auto-correct.
|
272
482
|
Style/RedundantSelf:
|
273
483
|
Enabled: false
|
@@ -275,25 +485,39 @@ Style/RedundantSelf:
|
|
275
485
|
# Offense count: 3
|
276
486
|
# Cop supports --auto-correct.
|
277
487
|
Style/RescueModifier:
|
278
|
-
|
488
|
+
Exclude:
|
489
|
+
- 'extra/cockpit/foreman-cockpit-session'
|
490
|
+
- 'lib/foreman_remote_execution/engine.rb'
|
279
491
|
|
280
|
-
# Offense count:
|
492
|
+
# Offense count: 13
|
281
493
|
# Cop supports --auto-correct.
|
282
494
|
# Configuration parameters: EnforcedStyle.
|
283
495
|
# SupportedStyles: implicit, explicit
|
284
496
|
Style/RescueStandardError:
|
285
|
-
|
497
|
+
Exclude:
|
498
|
+
- 'Rakefile'
|
499
|
+
- 'app/lib/actions/remote_execution/run_host_job.rb'
|
500
|
+
- 'app/lib/proxy_api/remote_execution_ssh.rb'
|
501
|
+
- 'app/models/concerns/foreman_remote_execution/orchestration/ssh.rb'
|
502
|
+
- 'app/models/input_template_renderer.rb'
|
503
|
+
- 'app/models/job_invocation_composer.rb'
|
504
|
+
- 'lib/foreman_remote_execution_core/polling_script_runner.rb'
|
505
|
+
- 'lib/foreman_remote_execution_core/script_runner.rb'
|
506
|
+
- 'lib/tasks/foreman_remote_execution_tasks.rake'
|
286
507
|
|
287
|
-
# Offense count:
|
508
|
+
# Offense count: 1
|
288
509
|
# Cop supports --auto-correct.
|
289
|
-
# Configuration parameters:
|
290
|
-
|
291
|
-
|
292
|
-
|
510
|
+
# Configuration parameters: AllowModifier.
|
511
|
+
Style/SoleNestedConditional:
|
512
|
+
Exclude:
|
513
|
+
- 'app/helpers/concerns/foreman_remote_execution/job_templates_extensions.rb'
|
293
514
|
|
294
515
|
# Offense count: 3
|
295
516
|
# Cop supports --auto-correct.
|
296
517
|
# Configuration parameters: IgnoredMethods.
|
297
518
|
# IgnoredMethods: respond_to, define_method
|
298
519
|
Style/SymbolProc:
|
299
|
-
|
520
|
+
Exclude:
|
521
|
+
- 'app/models/concerns/foreman_remote_execution/foreman_tasks_cleaner_extensions.rb'
|
522
|
+
- 'app/models/concerns/foreman_remote_execution/host_extensions.rb'
|
523
|
+
- 'app/models/job_invocation_composer.rb'
|