Skip to content
GitLab
Switch branch/tag
This project is mirrored from https://gitee.com/cowcomic/pixie.git. Pull mirroring failed .
Repository mirroring has been paused due to too many failed attempts. It can be resumed by a project maintainer.
  1. 29 Jun, 2022 6 commits
  3. 28 Jun, 2022 13 commits
  5. 27 Jun, 2022 9 commits
    • Vikranth Srivatsa's avatar
      Extend AMQP type gen template to support Frame and Record · b2c3ef3c
      Vikranth Srivatsa authored 
      
Summary: Adds extra types in type gen template. This can be used to try fully generating the constants/types file

Test Plan: This will be tested in a later pr by checking if actual packets decoded correctly.

Reviewers: #stirling, oazizi

Reviewed By: #stirling, oazizi

Subscribers: oazizi
Signed-off-by: default avatarVikranth Srivatsa <vsrivatsa@pixielabs.ai>

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

GitOrigin-RevId: 84d0770209ccd0d2ae49b213252740de74c67031
      b2c3ef3c
    • Michelle Nguyen's avatar
      Remove podSecurityPolicy · 938bfce9
      Michelle Nguyen authored 
      
Summary:
PodSecurityPolicies have been deprecated as of k8s 1.21, and will be removed in K8s 1.25.
We currently include the podSecurityPolicy by default. Most of the time it is ignored, as people need to enable the admission controller. When the admission controller is enabled, the current podSecurityPolicy isn't correct anyway, as we would also need podSecurityPolicies for the operator. In most cases, users have needed some customization around the podSecurityPolicy to best fit their configurations anyway.

Fixes #371

Test Plan: built Vizier YAMLs

Reviewers: philkuz, vihang

Reviewed By: vihang
Signed-off-by: default avatarMichelle Nguyen <michellenguyen@pixielabs.ai>

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

GitOrigin-RevId: 17a279c94575bf83e403ee0383db9e6314b8dcff
      938bfce9
    • Omid Azizi's avatar
      Reduce flakiness in pgsql_trace_bpf_test · 1699420b
      Omid Azizi authored 
      
Summary:
Remove flakiness in pgsql_trace_bpf_test by removing a dependence on an external docker image.

Instead use the existing postgres container for the client that we need for the tes.

Test Plan: This is a test update.

Reviewers: #stirling, zasgar

Reviewed By: zasgar
Signed-off-by: default avatarOmid Azizi <oazizi@pixielabs.ai>

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

GitOrigin-RevId: ec560ecfbcb2fe0d944939d119fc7c95b0d40a75
      1699420b
    • Phillip Kuznetsov's avatar
      PXC-221: Update pxapi pypi page · c3027daa
      Phillip Kuznetsov authored 
      
Summary:
Updated the pxapi pypi page with more links, removing old links, and changing
Authors -> Maintainers.

Test Plan: uploaded to test pypi

Reviewers: vihang, htroisi

Reviewed By: htroisi

JIRA Issues: PXC-221
Signed-off-by: default avatarPhillip Kuznetsov <pkuznetsov@pixielabs.ai>

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

GitOrigin-RevId: 697d2758987ff31fedba2ee4e21326c92a0ea393
      c3027daa
    • Nick Lanam's avatar
      Simpler, more robust embed detection · 30c7f4fb
      Nick Lanam authored 
      
Summary:
We were detecting embed state by checking the URL. This is clumsy, and breaks when one embedded view links to another without remembering to check that first.

Sorry for the semantic satiation in the following text...

With this change:
- Embed state is checked by testing if Pixie appears to be in an iframe
- Most links and buttons that accounted for embed state don't bother anymore, because...
- A top-level conditional redirect now automatically fixes the URL if it doesn't line up (puts `/embed` at the front of the path if embedded, and removes it otherwise)
- Routes for embeddable views still need to exist, so those stay.
- Trying to reach a route that doesn't have an embedded variant defined while embedded will now go to the 404 page instead of breaking out of embedded mode
- If Pixie is incorrectly embedded or if an embed link is accessed directly, it will fix itself before continuing

Test Plan:
Try a wide variety of links throughout the app:
- Access live views normally. Click entity links. Visit credits. Visit admin. Visit `/configure-data-export`.
- Try bouncing between plugin page and data export overview and data export details and data export history.
- Do the same when Pixie is embedded.
- Try embedding without `/embed` in the URL, or going to a `/embed` link in a new tab. Both should correct themselves.
- It should no longer be possible to, while embedded, "break out" of this mode
- Trying to embed a path that doesn't have a `/embed` version should now 404 instead of loading like it wasn't embedded. For example, try to embed `/credits`. It should try to change that to `/embed/credits`, which will 404.

Reviewers: michelle, vihang

Reviewed By: michelle
Signed-off-by: default avatarNick Lanam <nlanam@pixielabs.ai>

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

GitOrigin-RevId: c9033f4e4f8b5b6d9188acbb0a05af16db72b28c
      30c7f4fb
    • Nick Lanam's avatar
      [PC-1542]: Prevent unintended horizontal scrollbars · 8bf18010
      Nick Lanam authored 
      
Summary:
Scrollbars are not consistent between browsers, operating systems, and settings for either.

Customizing scrollbars I even more complex, and makes detection of their dimensions somehow even less consistent.

This diff guarantees that `useScrollbarSize` will detect the actual dimensions of the scrollbar (and if it's overlay vs inlay) in Safari, Chrome, and Firefox on macOS, Linux, and Windows, and especially for macOS users who change their system settings for scrollbars (General -> Show scroll bars -> defaults to "When scrolling").

Result is that data tables should no longer fail to reserve space for scrollbars when they should.

This diff also deletes a long-unused file that was still laying around.

Test Plan:
Run any PxL script with tables in Chrome on macOS with system preference of "When scrolling".
Do the same in Firefox and also try changing the system setting (reload between changes).
Try Firefox and Chrome on Ubuntu as well.

In all cases, there should not be any _unexpected_ horizontal scrollbars (tables with too many columns for the space available will still show one).

Reviewers: michelle, vihang

Reviewed By: michelle

JIRA Issues: PC-1542
Signed-off-by: default avatarNick Lanam <nlanam@pixielabs.ai>

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

GitOrigin-RevId: cd96499fcd87f1e99b1f9189aeb51be31ab9c2d4
      8bf18010
    • Nick Lanam's avatar
      [PP-3426]: Show status icon of cron history in details · d1b31e76
      Nick Lanam authored 
      
Summary:
The error column already has this information, but now there's an icon on the left to make it easier to notice.

This involved adding a new feature to the data tables.

{F188519}
{F188520}

Test Plan:
Visit cron history for any given script.

Try those with successful runs and those with unsuccessful runs to see both versions.

Reviewers: michelle, vihang

Reviewed By: michelle

JIRA Issues: PP-3426
Signed-off-by: default avatarNick Lanam <nlanam@pixielabs.ai>

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

GitOrigin-RevId: 42ee20e58cdeea3e41b1f3e1dd5fec49a7d35da6
      d1b31e76
    • Phillip Kuznetsov's avatar
      PXC-205: Add missing styleguide files · 44dccbe4
      Phillip Kuznetsov authored 
      
Summary:
The styleguide was missing some linked files. We had an internal document with some styles
so I figured I would just publish that as the cc and go style guides.

I did a brief run through to make sure everything makes sense.

Test Plan: Made sure links work and the guide renders correctly.

Reviewers: michelle, vihang, zasgar, nserrino, oazizi, yzhao

Reviewed By: zasgar

Subscribers: htroisi

JIRA Issues: PXC-205
Signed-off-by: default avatarPhillip Kuznetsov <pkuznetsov@pixielabs.ai>

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

GitOrigin-RevId: 6aee975c7405a48599559be2b72de0c1cb1e0d36
      44dccbe4
    • Tharindu Wijewardane's avatar
      Add capability to get pod labels · c3d610b4
      Tharindu Wijewardane authored 
      
Summary:
Track pod labels and adds a UDF to get pod labels given IDs.

Fixes #449
Closes #468

Test Plan: Added unit tests

Reviewers: zasgar, michelle

Reviewed By: michelle
Signed-off-by: default avatarTharindu Wijewardane <tharindu.moratuwa@gmail.com>

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

GitOrigin-RevId: fd84b4dfbe9bfce8da853232cee604417bdf07a7
      c3d610b4
  7. 25 Jun, 2022 2 commits
  9. 24 Jun, 2022 3 commits
  11. 23 Jun, 2022 5 commits
  13. 22 Jun, 2022 2 commits
    • Vihang Mehta's avatar
      Cleanup remote_download_output opts · 6963a93e
      Vihang Mehta authored 
      
Summary:
Using `--remote_download_toplevel` to override `--remote_download_minimal`
causes bazel to warn us since those are both expansions. Instead use `--remote_download_outputs=all`
to override when we need it.

Test Plan: All the modified scripts continue to work.

Reviewers: zasgar, michelle, jamesbartlett

Reviewed By: jamesbartlett
Signed-off-by: default avatarVihang Mehta <vihang@pixielabs.ai>

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

GitOrigin-RevId: ba485982afb1c91332495fb6f4d183e4059ebadf
      6963a93e
    • Ryan Cheng's avatar
      Enable mux tracing · b12f6a46
      Ryan Cheng authored 
      
Summary: Enable mux tracing. If the kernel version is too old, turn it off due to instruction limits. Enable mux_trace_bpf_test.

Test Plan: Existing tests

Reviewers: #stirling, jps

Reviewed By: #stirling, jps

Subscribers: jps
Signed-off-by: default avatarRyan Cheng <rcheng@pixielabs.ai>

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

GitOrigin-RevId: 604e9bf68a6fb0ef48944ae814d2e314a326bb70
      b12f6a46

免费DevSecOps平台，让您的项目体验完整的DevSecOps流程，让项目更安全