Summary: Minor oversight since the function it calls serves double duty

Test Plan: Open the live tour with the sidebar already closed; then try with it already open. Also try opening and closing it from the top left hamburger button. It should close for the tour if open, remember what state it was in when the tour closes, and the hamburger button should switch open/closed as long as said button is available (the tour hides/blocks it).

Reviewers: michelle, vihang

Reviewed By: michelle

JIRA Issues: PC-956

Differential Revision: https://phab.corp.pixielabs.ai/D8732

GitOrigin-RevId: 350fa43d52be5fa451d0d9012b1380e43a395848

Summary: Accidentally switched the icons for hierarchy/clustered modes

Test Plan: yarn dev

Reviewers: vihang, zasgar, philkuz

Reviewed By: vihang

Differential Revision: https://phab.corp.pixielabs.ai/D8728

GitOrigin-RevId: 6b5d3491fe8c13a277139bd1782f1b4a0641b0b8

Summary: after we fixed the tour, we forgot to add it back to the toolbar

Test Plan: yarn dev

Reviewers: nlanam, vihang, zasgar

Reviewed By: vihang

JIRA Issues: PC-955

Differential Revision: https://phab.corp.pixielabs.ai/D8727

GitOrigin-RevId: 9efb64be32c0a07732e137df69548ed2cd08132f

Summary:
we should add a label for now so that its clear what this selector is for.
in the future, we can add an icon

Test Plan:
screenshot
{F157635}

Reviewers: zasgar, vihang, philkuz

Reviewed By: zasgar

Differential Revision: https://phab.corp.pixielabs.ai/D8725

GitOrigin-RevId: ed62a83c41e53d7f801e571a2036be5d0c90aec1

Summary:
Fixes a sizing bug in the menu, restores the context needed for the live tour to open, fixes a logic flaw in `toggleSidebar` for the same purpose, then moves punchouts and tips around in the tour to mostly suit the new UI layout. Not perfect, but good enough for a demo tomorrow.

{F157645}

Test Plan: Open the live tour. You may need to wait a few seconds before the button is responsive (seems to be an unrelated bug).

Reviewers: michelle, vihang

Reviewed By: michelle

JIRA Issues: PC-956

Differential Revision: https://phab.corp.pixielabs.ai/D8726

GitOrigin-RevId: acd5df6528a5bfcd3bfd1032e21aeaa2ba36b10a

Summary:
Change flamegraph colors.
Add better minimap interactions
Add double click to reset view.

Test Plan: Bikeshed

Reviewers: michelle, zasgar, vihang

Reviewed By: vihang

Differential Revision: https://phab.corp.pixielabs.ai/D8723

GitOrigin-RevId: da0d856b272e8268b89a1b6b7908905f071cc2ab

Summary:
Updated the data drawer toggle so that it is not rounded on the bottom.
also updated the background color of the data drawer so that it appears to be on elevation 2.

Test Plan: yarn dev

Reviewers: zasgar, vihang, philkuz

Reviewed By: vihang

Differential Revision: https://phab.corp.pixielabs.ai/D8724

GitOrigin-RevId: d2bb9deb21318a0cfe1cf28491b773add54f6f0f

Warn the user that px delete without clobber will delete the entire namespace. Add y/n verification for delete to match deploy.

Summary: TSIA. If we do px delete, we will delete the whole namespace and the user should give y/n confirmation. Also we should tell them that px delete is going to do that, in case they deployed px to a namespace with other things in it, like a monitoring namespace.

Test Plan: ran it

Reviewers: michelle, zasgar, vihang

Reviewed By: michelle, zasgar

Differential Revision: https://phab.corp.pixielabs.ai/D8718

GitOrigin-RevId: 7c7436ed73996d582edff506d874c942a3beddc5

PP-2737: Vizier pods (besides bootstrap) should wait for cloud connector to assign cluster ID before starting

Summary: We had a problem where users would be confused when deploying via helm, becuase many Vizier pods would show up as CreateConfigError. This was due to the fact that a subset of vizier pods (non-bootstrap pods) are configured to require PL_CLUSTER_ID, which is assigned in the bootstrap phase by cloud-connector. This diff adds a /readyz to cloud-connector which checks whether or not PL_CLUSTER_ID has been assigned. /readyz is then used by the non-bootstrap pods to determine whether or not the pod should start up yet (having received PL_CLUSTER_ID by that point).

Test Plan: ran it in skaffold various times

Reviewers: michelle, zasgar, vihang

Reviewed By: michelle

JIRA Issues: PP-2737

Differential Revision: https://phab.corp.pixielabs.ai/D8712

GitOrigin-RevId: 9cef9278b2dbf919f6bcd02b1aa239e13ca2adfa

Summary: TSIA

Test Plan: {F157586}

Reviewers: michelle, zasgar, philkuz

Reviewed By: michelle

Differential Revision: https://phab.corp.pixielabs.ai/D8721

GitOrigin-RevId: 1834174cc992dede05aee584dfa00b40f6d44a53

Summary: Auth had it already. Credits, admin, and live did not. Made it stick to the bottom of the page if content would not have pushed it further. If content scrolls, footer ends up below it so as not to waste screen real estate. Also renames the component for the footer since it's general-use now.

Test Plan: Check live, credits, and admin. In all cases, scrolling to the bottom should show the footer (ts&cs, privacy policy, copyright). If the page is shorter than the viewport, the footer should push itself to the bottom of the viewport (credits page does this, as does the scratch pad script before putting code into it).

Reviewers: michelle, vihang

Reviewed By: michelle

JIRA Issues: PC-953

Differential Revision: https://phab.corp.pixielabs.ai/D8722

GitOrigin-RevId: 66d73f826ae98547e8b0aff8a6ff6bd1db05cced

Summary: We removed the /script suffix.

Test Plan: N/a

Reviewers: michelle, vihang

Reviewed By: michelle, vihang

Differential Revision: https://phab.corp.pixielabs.ai/D8720

GitOrigin-RevId: 302f99bee54c70f94ebfc8c6d0df6372339bdb8c

Summary:
We used to have long text buttons, but icons are more compact.
Users can hover over the icon to get more information about the state.

Test Plan:
screenshot:
{F157564}

Reviewers: vihang, zasgar, philkuz

Reviewed By: vihang

Differential Revision: https://phab.corp.pixielabs.ai/D8717

GitOrigin-RevId: 2571e20e4bb4938cee84dcab841d2b8d53f35694

Summary: This will allow us to add new fields without breaking older CLIs.

Test Plan: bazel

Reviewers: zasgar, michelle, nserrino

Reviewed By: zasgar

Differential Revision: https://phab.corp.pixielabs.ai/D8715

GitOrigin-RevId: f63cef056d46dcf34cd49eee802f4d08ddef6e6c

Summary:
This diff addresses three things:
1. icon colors were too dark and hard to read. updated it to use a brigher color.
2. namespace icon is too hard to read. updated the svg so that "ns" is larger.
3. updated to better announcements icon

Test Plan:
screenshot
{F157549}

Reviewers: zasgar, philkuz, vihang

Reviewed By: vihang

Differential Revision: https://phab.corp.pixielabs.ai/D8713

GitOrigin-RevId: bc8755036cfef0afd3bdb5d18bdf2170f62eb30a

Summary: the logo wasn't vertically aligned

Test Plan: yarn dev

Reviewers: vihang, zasgar, philkuz

Reviewed By: vihang

Differential Revision: https://phab.corp.pixielabs.ai/D8706

GitOrigin-RevId: f858d76fbfda3440e6ac4a23a50a540a4aa2cea5

Summary: theres a margin in the live view contents, which shows a weird gap when scrolling up/down

Test Plan: yarn dev

Reviewers: zasgar, vihang, philkuz

Reviewed By: vihang

Differential Revision: https://phab.corp.pixielabs.ai/D8708

GitOrigin-RevId: 2c779a7289907b621f536c24cc0f4e1e25692f92

Summary: The tab background is currently transparent, so when you scroll in the tabContents, the contents show up underneath the tab.

Test Plan: yarn dev

Reviewers: zasgar, vihang, philkuz

Reviewed By: vihang, philkuz

Differential Revision: https://phab.corp.pixielabs.ai/D8707

GitOrigin-RevId: 153c49e6f98ccd5fec0b9459b277211e59bfeba5

Summary: the logo is usually used to link to a homepage

Test Plan: yarn dev

Reviewers: vihang, philkuz, zasgar

Reviewed By: vihang

Differential Revision: https://phab.corp.pixielabs.ai/D8705

GitOrigin-RevId: 3f47a07eec278d8cab08d04900438d06eff45a1c

Summary: we may want to provide more info when the user selets the start time.

Test Plan:
screenshot:
{F157450}

Reviewers: zasgar, vihang, philkuz

Reviewed By: vihang

JIRA Issues: PC-934

Differential Revision: https://phab.corp.pixielabs.ai/D8700

GitOrigin-RevId: f004ed95587b84f90c820968367a729d893ce075

Summary:
Create a gutter column to hold status info and stick status columns in
it.

Test Plan: yarn dev with local scripts.

Reviewers: zasgar, philkuz, michelle

Reviewed By: philkuz

JIRA Issues: PC-931

Differential Revision: https://phab.corp.pixielabs.ai/D8703

GitOrigin-RevId: 1c1e54503317ee4c815db90e816a3c86c93b5f74

Summary: we should use our new pixie logo

Test Plan:
screenshot:
{F157456}

{F157457}

Reviewers: zasgar, vihang, philkuz

Reviewed By: philkuz

Differential Revision: https://phab.corp.pixielabs.ai/D8701

GitOrigin-RevId: ab8fdc2ebdfbb16cbf974914976eb70a5f8e5e6d

Summary: we want to make the sidebar configurable

Test Plan:
screenshots:
{F157443}

{F157444}

Reviewers: zasgar, vihang, philkuz

Reviewed By: vihang

JIRA Issues: PC-933

Differential Revision: https://phab.corp.pixielabs.ai/D8699

GitOrigin-RevId: 69c2b9ebaca5387743f6bd2d582bdca970235602

Summary: cluster selector would show a loading message while loading. Not necessary and looks bad. Removing for clarity.

Test Plan: Tested on yarn dev. refresh page and see what pops up.

Reviewers: michelle, vihang

Reviewed By: vihang

Differential Revision: https://phab.corp.pixielabs.ai/D8702

GitOrigin-RevId: 9850a3360dcbd2dc5fd23d901a7f97202aa266a1

Summary:
1. updated breadcrumbs styling so that the breadcrumbs don't stretch across the full width. instead we have one segment on the left for script + args, and another segment on the right for the time arg.
2. script should now be first in the breadcrumbs, with a divider to separate it from regular args
3. moved cluster selector to the top bar out of the breadcrumbs
4. moved "run" button to the top bar, next to the toolbar items

Test Plan:
screenshot:
{F157212}

Reviewers: zasgar, vihang, philkuz, nlanam

Reviewed By: vihang, philkuz

JIRA Issues: PC-940

Differential Revision: https://phab.corp.pixielabs.ai/D8675

GitOrigin-RevId: 7458641176dd4706fb06899680ddf2c15458a937

Summary: TSIA

Test Plan: Still builds

Reviewers: michelle, philkuz

Reviewed By: philkuz

JIRA Issues: PC-917

Differential Revision: https://phab.corp.pixielabs.ai/D8698

GitOrigin-RevId: c789bb7e8f174471adc0c220788597856e498bec

Summary:
This drops the old versions and fixes the new versions to handle all
known interactions

Test Plan: Tested with yarn dev https://www.notion.so/pixielabs/UI-Context-Test-Cases-48f75230172948ee84cc9a78a64a63d0b

Reviewers: philkuz, michelle, zasgar, nlanam

Reviewed By: philkuz, michelle, zasgar

JIRA Issues: PC-917

Differential Revision: https://phab.corp.pixielabs.ai/D8697

GitOrigin-RevId: e2cb0d0430464df736154d3dc82541ec0d7451f9

Summary:
we only need all fields in useListCluster in the admin page... the rest of the pages actually only care about id/name/status.
this diff updates useListCluster so that you can query for the full set of data, or just the necessary subset.
after D8691, more of these can change to a single getClusterInfoByName call.

Test Plan: yarn dev

Reviewers: vihang, philkuz

Reviewed By: vihang

Differential Revision: https://phab.corp.pixielabs.ai/D8693

GitOrigin-RevId: 432d4f1df10a703cf8a6bebed329f7a705765c47

Summary:
In many places in the UI, we reference the cluster by name (for example, URL).
however, the backend only provides a way to fetch a single ClusterInfo by ID. therefore, consumers who need clusterInfo but only have a clusterName currently fetch the entire list of clusters, and filter through the list to get the correct clusterInfo.
Instead, we can clean this logic up by doing this filtering in the backend.

Test Plan: unit test

Reviewers: vihang, philkuz

Reviewed By: vihang

Differential Revision: https://phab.corp.pixielabs.ai/D8691

GitOrigin-RevId: c2c80b16292a476adca197c4ccbcd80bf66d1f54

Summary: D8689 was supposed to fix the topbar in the deployinstructions... It did fix it in the deployInstructions, but didn't work when in an org with clusters. this is because we shouldn't actually make topbar absolute, because other content will go underneath the topbar.

Test Plan: checked org with and without clusters

Reviewers: vihang, philkuz, zasgar

Reviewed By: vihang

JIRA Issues: PC-940

Differential Revision: https://phab.corp.pixielabs.ai/D8692

GitOrigin-RevId: 797caafe2e993526295d9e46913f89355994e68d

Summary: the recent MaterialUI update made tooltips interactive by default. this actually made hovering over items in the sidebar a little annoying, because the tooltip for an item can block the item below it.

Test Plan: yarn dev, hover on tooltips in sidebar

Reviewers: vihang, philkuz, zasgar

Reviewed By: vihang

JIRA Issues: PC-940

Differential Revision: https://phab.corp.pixielabs.ai/D8690

GitOrigin-RevId: bc73387af56f631a96dca0ed11a367a47ef2ce22

Summary: Currently when a user has no clusters and we should the deploy instructions, the topbar is rendered in a weird position rather than the top of the page.

Test Plan: yarn dev

Reviewers: vihang, philkuz, zasgar

Reviewed By: vihang

JIRA Issues: PC-940

Differential Revision: https://phab.corp.pixielabs.ai/D8689

GitOrigin-RevId: bbb7be76af5a53ba6b2ab2c31aea6726ebb626bf

Summary:
we were hardcoding a background color in vega before. this color no longer matches with the gridItem background color.
we can just leave it transparent so that it matches with whatever the gridItem's background is.

Test Plan: yarn dev

Reviewers: vihang, philkuz, nlanam

Reviewed By: vihang

Differential Revision: https://phab.corp.pixielabs.ai/D8677

GitOrigin-RevId: 2004ac793de7d86d0899232b798c8af6625f51b5

Summary: Previously, px/service checked for service string equality (e.g. df.service == 'ns/sv1'). This caused data to now show up for records that are part of multiple services. This change fixes that problem by switching the equality checks to px.has_service_name, which allows for checking if a service is in the list of services.

Test Plan: ran it with a service that comes in a pair.

Reviewers: michelle, philkuz, zasgar, vihang

Reviewed By: michelle

JIRA Issues: PP-2751

Differential Revision: https://phab.corp.pixielabs.ai/D8687

GitOrigin-RevId: 222576192de80021d8e390e3a956074c5edfe292

Summary: TSIA

Test Plan: `yarn dev` naviagte to scripts with required args from breadcrumbs.

Reviewers: michelle, philkuz, zasgar

Reviewed By: michelle

Differential Revision: https://phab.corp.pixielabs.ai/D8686

GitOrigin-RevId: c499d94c1d1d6fe2e0dbde19d714c446e84efa0b

Summary: ClusterContext will not be defined for some sidebar owners. That's because we want to rely on the URL as the source of truth for the cluster, but the URLs in /credits and /admin do not contain this info, nor should they contain this info. Will have to update in a later diff to rethink what goes in the sidebar components for these and other pages.

Test Plan: Tested on yarn dev with a user that has 0 clusters. Tested on the simplified routing approach and both work.

Reviewers: vihang, zasgar, michelle

Reviewed By: zasgar

JIRA Issues: PC-917

Differential Revision: https://phab.corp.pixielabs.ai/D8685

GitOrigin-RevId: 2e5e46a2b7944b898895a4db1dfe9f7a8178f05a

Summary:
Depends on D8680. This diff makes two improvements to the logical planner.
1. If px.has_service_name is used, it considers that a metadata pruning opportunity (not just equals)
2. For a statement like df.service = '["pl/sv1","pl/sv2"]' it now works when it didn't before (it used to get pruned away even though string matching should always work.

Test Plan: edited existing.

Reviewers: zasgar, vihang, philkuz, michelle

Reviewed By: zasgar

JIRA Issues: PP-2751

Differential Revision: https://phab.corp.pixielabs.ai/D8681

GitOrigin-RevId: a33fcee7f9f0eb1144eef1f7343ab94aeb09e3b2

Summary:
We have a problem where doing df.ctx['service'] == 'pl/pl-nats' doesn't work when df.ctx['service'] returns something like ["pl/pl-nats", "pl/pl-nats-mgmt"]. This diff adds a UDF to work in that case. The problem also applies to service ID so added one for that as well.

Even though this logic is somewhat general, I am naming it specifically to refer to services, because we want to deprecate it with full array support.

Test Plan: added units

Reviewers: zasgar, vihang, philkuz, michelle

Reviewed By: zasgar

JIRA Issues: PP-2751

Differential Revision: https://phab.corp.pixielabs.ai/D8680

GitOrigin-RevId: 4d020a56d2167afe3d1c6afc74127f963d07710c

Summary:
Makes vizier-routing a lot simpler and cleans up handling for vanity
URLs. We now defer all validation logic and just rewrite vanities to a standard
url.

Test Plan: `yarn dev`

Reviewers: philkuz, michelle, zasgar

Reviewed By: michelle

JIRA Issues: PC-917

Differential Revision: https://phab.corp.pixielabs.ai/D8684

GitOrigin-RevId: e670014c1120422fdc4322171246d0aea487b6d3

Summary: stringifier.cc: revert code that is unsafe for empty stack trace strings.

Test Plan: Existing.

Reviewers: #stirling, oazizi

Reviewed By: #stirling, oazizi

JIRA Issues: PP-2688

Differential Revision: https://phab.corp.pixielabs.ai/D8682

GitOrigin-RevId: 3d97d5e4747ac747ac9309a0f28ac8c7f570ad59