Skip to content

Commit

Permalink
fix: Unbreak termination of operation with apps in other namespaces (a…
Browse files Browse the repository at this point in the history
…rgoproj#11239) (argoproj#11724)

* fix: Unbreak operation termination

Signed-off-by: jannfis <jann@mistrust.net>

* Revert change to Dockerfile

Signed-off-by: jannfis <jann@mistrust.net>

Signed-off-by: jannfis <jann@mistrust.net>
Signed-off-by: schakrad <chakradari.sindhu@gmail.com>
  • Loading branch information
jannfis authored and schakrad committed Mar 14, 2023
1 parent 90bdf46 commit 7ecdbc6
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,7 @@ export const ApplicationOperationState: React.StatelessComponent<Props> = ({appl
const confirmed = await ctx.apis.popup.confirm('Terminate operation', 'Are you sure you want to terminate operation?');
if (confirmed) {
try {
await services.applications.terminateOperation(application.metadata.name);
await services.applications.terminateOperation(application.metadata.name, application.metadata.namespace);
} catch (e) {
ctx.apis.notifications.show({
content: <ErrorNotification title='Unable to terminate operation' e={e} />,
Expand Down
1 change: 1 addition & 0 deletions ui/src/app/shared/services/applications-service.ts
Original file line number Diff line number Diff line change
Expand Up @@ -387,6 +387,7 @@ export class ApplicationsService {
public terminateOperation(applicationName: string, appNamespace: string): Promise<boolean> {
return requests
.delete(`/applications/${applicationName}/operation`)
.query({appNamespace})
.send()
.then(() => true);
}
Expand Down

0 comments on commit 7ecdbc6

Please sign in to comment.