Skip to content

Commit 9a756c3

Browse files
authored
Show viz routes/tap/top in dashboard instead of routes/tap/top (#7230)
because they are deprecated Signed-off-by: Takumi Sue <[email protected]>
1 parent d9e5eec commit 9a756c3

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

web/app/js/components/QueryToCliCmd.jsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ class QueryToCliCmd extends React.Component {
4545
<br />
4646

4747
<code>
48-
linkerd {cmdName} {resource}
48+
linkerd viz {cmdName} {resource}
4949
{ displayOrder(cmdName, query).map(item => {
5050
return !toCliParam[item] ? null : this.renderCliItem(toCliParam[item], query[item]);
5151
})}

web/app/js/components/QueryToCliCmd.test.jsx

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ describe('QueryToCliCmd', () => {
2121
);
2222

2323
expect(component.html()).to.include("Current Routes query");
24-
expect(component.html()).to.include("linkerd routes deploy/controller --namespace linkerd");
24+
expect(component.html()).to.include("linkerd viz routes deploy/controller --namespace linkerd");
2525
});
2626

2727
it('shows the linkerd namespace if the controller is not in the default namespace', () => {
@@ -39,7 +39,7 @@ describe('QueryToCliCmd', () => {
3939
);
4040

4141
expect(component.html()).to.include("Current Routes query");
42-
expect(component.html()).to.include("linkerd routes deploy/controller --namespace linkerd --linkerd-namespace my-linkerd-ns");
42+
expect(component.html()).to.include("linkerd viz routes deploy/controller --namespace linkerd --linkerd-namespace my-linkerd-ns");
4343
});
4444

4545
it('does not render flags for items that are not populated in the query', () => {
@@ -60,7 +60,7 @@ describe('QueryToCliCmd', () => {
6060
);
6161

6262
expect(component.html()).to.include("Current Tap query");
63-
expect(component.html()).to.include("linkerd tap deploy/controller --namespace linkerd --authority foo.bar:8080");
63+
expect(component.html()).to.include("linkerd viz tap deploy/controller --namespace linkerd --authority foo.bar:8080");
6464
});
6565

6666
it('displays the flags in the specified order per cli command', () => {
@@ -82,7 +82,7 @@ describe('QueryToCliCmd', () => {
8282
);
8383

8484
expect(component.html()).to.include("Current Tap query");
85-
expect(component.html()).to.include("linkerd tap deploy/controller --namespace linkerd --to deploy/prometheus --scheme HTTPS --authority foo.bar:8080");
85+
expect(component.html()).to.include("linkerd viz tap deploy/controller --namespace linkerd --to deploy/prometheus --scheme HTTPS --authority foo.bar:8080");
8686
});
8787

8888
it("doesn't render a namespace flag when the resource is a namespace", () => {
@@ -100,7 +100,7 @@ describe('QueryToCliCmd', () => {
100100
);
101101

102102
expect(component.html()).to.include("Current Top query");
103-
expect(component.html()).to.include("linkerd top namespace/linkerd");
103+
expect(component.html()).to.include("linkerd viz top namespace/linkerd");
104104
});
105105

106106
it("doesn't render commands for which a flag is not defined", () => {
@@ -120,6 +120,6 @@ describe('QueryToCliCmd', () => {
120120
);
121121

122122
expect(component.html()).to.include("Current Tap query");
123-
expect(component.html()).to.include("linkerd tap deploy/controller --namespace linkerd --scheme HTTPS");
123+
expect(component.html()).to.include("linkerd viz tap deploy/controller --namespace linkerd --scheme HTTPS");
124124
});
125125
});

0 commit comments

Comments
 (0)