Unverified Commit 2a3a758c authored by Anurag Paliwal's avatar Anurag Paliwal Committed by GitHub
Browse files

Fix few error checks (#1795)

Signed-off-by: default avatarAnurag Paliwal <apaliwal@adobe.com>
Co-authored-by: default avatarAnurag Paliwal <apaliwal@adobe.com>
Co-authored-by: default avatarCWen <cwenyin0@gmail.com>
parent c69e6cae
master WangXiangUSTC-patch-1 ci-update-check-markdown-link dependabot/go_modules/github.com/containerd/containerd-1.4.8 dependabot/go_modules/github.com/containerd/containerd-1.5.10 dependabot/go_modules/github.com/containerd/containerd-1.5.13 dependabot/go_modules/github.com/containerd/containerd-1.5.4 dependabot/go_modules/github.com/containerd/containerd-1.5.8 dependabot/go_modules/github.com/ethereum/go-ethereum-1.10.17 dependabot/go_modules/github.com/ethereum/go-ethereum-1.10.22 dependabot/go_modules/helm.sh/helm/v3-3.9.4 dependabot/npm_and_yarn/ui/async-2.6.4 dependabot/npm_and_yarn/ui/axios-0.21.2 dependabot/npm_and_yarn/ui/hosted-git-info-2.8.9 dependabot/npm_and_yarn/ui/loader-utils-1.4.1 dependabot/npm_and_yarn/ui/minimist-1.2.6 dependabot/npm_and_yarn/ui/nanoid-3.2.0 dependabot/npm_and_yarn/ui/node-fetch-2.6.7 dependabot/npm_and_yarn/ui/path-parse-1.0.7 dependabot/npm_and_yarn/ui/tar-6.1.11 dependabot/npm_and_yarn/ui/url-parse-1.5.3 dependabot/npm_and_yarn/ui/ws-6.2.2 fix-build-env migration-fix platform_dev promote-zhiqiang-maintainer release-2.0 release-2.1 release-2.2 release-2.3 release-2.4 release-2.5 revert-1777-shell-checks update-company update-maintainer-info-zhiqiang v2.5.0 v2.5.0-beta.1 v2.5.0-beta.0 v2.5.0-alpha.1 v2.5.0-alpha.0 v2.4.3 v2.4.2 v2.4.1 v2.4.0 v2.4.0-beta.0 v2.4.0-alpha.0 v2.3.3 v2.3.2 v2.3.1 v2.3.0 v2.3.0-beta.0 v2.3.0-alpha.5 v2.3.0-alpha.4 v2.3.0-alpha.3 v2.3.0-alpha.2 v2.3.0-alpha.1 v2.3.0-alpha.0 v2.2.3 v2.2.2 v2.2.1 v2.2.0 v2.1.8 v2.1.7 v2.1.6 v2.1.5 v2.1.4 v2.1.3 v2.1.2 v2.1.1 v2.1.0 v2.0.7 v2.0.6 v2.0.5 v2.0.4 v2.0.3 v2.0.2 v2.0.1 v2.0.0 v2.0.0-beta.3 v2.0.0-beta.2 v2.0.0-beta.1 fake-release-0.2.3 chart-2.5.0 chart-2.5.0-beta.1 chart-2.5.0-beta.0 chart-2.4.3 chart-2.4.2 chart-2.4.1 chart-2.4.0 chart-2.4.0-beta.0 chart-2.4.0-alpha.0 chart-2.3.3 chart-2.3.2 chart-2.3.1 chart-2.3.0 chart-2.3.0-beta.0 chart-2.3.0-alpha.5 chart-2.3.0-alpha.4 chart-2.3.0-alpha.3 chart-2.3.0-alpha.2 chart-2.3.0-alpha.1 chart-2.3.0-alpha.0 chart-2.2.3 chart-2.2.2 chart-2.2.1 chart-2.2.0 chart-2.1.8 chart-2.1.7 chart-2.1.6 chart-2.1.5 chart-2.1.4 chart-2.1.3 chart-2.1.2 chart-2.1.1 chart-2.1.0 chart-2.0.7 chart-2.0.6 chart-2.0.5 chart-2.0.4 chart-2.0.3 chart-2.0.2 chart-2.0.1 chart-2.0.0 chart-2.0.0-beta.3 chart-2.0.0-beta.2 chart-2.0.0-beta.1
No related merge requests found
Showing with 8 additions and 2 deletions
+8 -2
......@@ -290,11 +290,14 @@ func (r *endpoint) applyTc(
Name: pod.Name,
Namespace: pod.Namespace,
})
t.Append(v1alpha1.RawTrafficControl{
err := t.Append(v1alpha1.RawTrafficControl{
Type: tcType,
TcParameter: networkchaos.Spec.TcParameter,
Source: m.Source,
})
if err != nil {
return err
}
}
return nil
}
......@@ -311,12 +314,15 @@ func (r *endpoint) applyTc(
Namespace: pod.Namespace,
})
t.Append(dstIpset)
t.Append(v1alpha1.RawTrafficControl{
err := t.Append(v1alpha1.RawTrafficControl{
Type: tcType,
TcParameter: networkchaos.Spec.TcParameter,
Source: m.Source,
IPSet: dstIpset.Name,
})
if err != nil {
return err
}
}
return nil
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment