Skip to content

Commit

Permalink
Merge pull request #2340 from tomasaschan/fix-exec-provider
Browse files Browse the repository at this point in the history
Fix exec provider
  • Loading branch information
k8s-ci-robot authored Feb 14, 2025
2 parents 234a493 + a588f51 commit de280fb
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion kubernetes/base/config/exec_provider.py
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ def run(self, previous_response=None):
if previous_response:
kubernetes_exec_info['spec']['response'] = previous_response
if self.cluster:
kubernetes_exec_info['spec']['cluster'] = self.cluster
kubernetes_exec_info['spec']['cluster'] = self.cluster.value

self.env['KUBERNETES_EXEC_INFO'] = json.dumps(kubernetes_exec_info)
process = subprocess.Popen(
Expand Down
2 changes: 1 addition & 1 deletion kubernetes/base/config/exec_provider_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -175,7 +175,7 @@ def test_with_cluster_info(self, mock):
instance = mock.return_value
instance.wait.return_value = 0
instance.communicate.return_value = (self.output_ok, '')
ep = ExecProvider(self.input_with_cluster, None, {'server': 'name.company.com'})
ep = ExecProvider(self.input_with_cluster, None, ConfigNode("cluster", {'server': 'name.company.com'}))
result = ep.run()
self.assertTrue(isinstance(result, dict))
self.assertTrue('token' in result)
Expand Down

0 comments on commit de280fb

Please sign in to comment.