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. 01 Oct, 2021 5 commits
  3. 30 Sep, 2021 15 commits
  5. 29 Sep, 2021 13 commits
    • Omid Azizi's avatar
      Naming: test_exe · 82dd1428
      Omid Azizi authored 
      Summary: More inclusive naming.

Test Plan: Existing tests

Reviewers: #stirling, zasgar, yzhao

Reviewed By: #stirling, yzhao

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

GitOrigin-RevId: 07356679198629299722b206cdd978e438b0e440
      82dd1428
    • Ryan Cheng's avatar
      [Kafka] Fix stitcher bug for older fetch and produce api versions · 1aa98d1f
      Ryan Cheng authored 
      Summary: Api_version checks were not set correctly when parsing fetch and produce reqs/resps.

Test Plan: Newly added tests

Reviewers: #stirling, oazizi

Reviewed By: #stirling, oazizi

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

GitOrigin-RevId: 46fb24dd1734263d3c5161ba5dcf74391d3111ac
      1aa98d1f
    • Omid Azizi's avatar
      PIDCPUUseBPFTraceConnector update · 3ef53509
      Omid Azizi authored 
      Summary:
This source had experienced some bit-rot.
It's left in for reference on how to hook in a BPFTrace connector.
Fix some of the bit-rot.

It's not used by default.

Test Plan: Existing tests

Reviewers: #stirling, yzhao

Reviewed By: #stirling, yzhao

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

GitOrigin-RevId: c69c69a64063ec9f1b23cbaa78e2232a4c1fdcc4
      3ef53509
    • Omid Azizi's avatar
      [Cleanup] Split out go and cc targets into separate directories · 69f56b75
      Omid Azizi authored 
      Summary:
For organizational purposes.

Also solves an issue where the go_library was picking up a cc header file.

Test Plan: Existing tests

Reviewers: #stirling, yzhao

Reviewed By: #stirling, yzhao

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

GitOrigin-RevId: 0f4cf6d56d486d5fe80bb4391af5dfbb9d2ab0e1
      69f56b75
    • Omid Azizi's avatar
      Naming: sample_host · 14f1cfbd
      Omid Azizi authored 
      Summary: More inclusive naming.

Test Plan: Existing tests

Reviewers: #stirling, zasgar, yzhao

Reviewed By: #stirling, yzhao

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

GitOrigin-RevId: b5d4383fe9d66846ff726650faf3bfa7c9221d10
      14f1cfbd
    • Omid Azizi's avatar
      BPFTraceWrapper: Fix issue where re-compilations of BPFTrace scripts fail. · 941457d3
      Omid Azizi authored 
      Summary:
Issue had to do with some global state in BPFTrace that needed to be reset.

In the future, should update BPFTrace such that it doesn't use global state.

resolves #317

Test Plan: Existing tests

Reviewers: #third_party_approvers, #stirling, yzhao

Reviewed By: #third_party_approvers, #stirling, yzhao

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

GitOrigin-RevId: afc706f937dd648d77d57171bec5849746cf04e4
      941457d3
    • Michelle Nguyen's avatar
      Make UI credits public · 385f11d9
      Michelle Nguyen authored 
      Summary:
users should be able to view this regardless of whether they are authorized are not.
as a result, we need to move it out of vizier.tsx, and remove the side/nav bars since those only show up when the user is auth'd.
we can add a link to it in the footer, but it is also fine to keep credits pages hidden.

Test Plan:
yarn dev and go to '/credits'
{F169265}

Reviewers: zasgar, vihang, nlanam, philkuz

Reviewed By: vihang

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

GitOrigin-RevId: 1c75730e81ea940865570bc9c415e7c20cb49a8f
      385f11d9
    • Vihang Mehta's avatar
      Even longer BES timeouts · cf76cce8
      Vihang Mehta authored 
      Summary: TSIA

Test Plan: Jenkins

Reviewers: michelle, zasgar

Reviewed By: michelle

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

GitOrigin-RevId: 83d3d035cdc3c566614bc97f80ef8e5f03e8079d
      cf76cce8
    • Nick Lanam's avatar
      [PC-1196] Show `px/*` scripts before others · 2d633fd5
      Nick Lanam authored 
      Summary: TSIA.

Test Plan: Run the UI locally. Open the script selector. Alphabetical sorting should be intact, except that `px/*` scripts appear before others.

Reviewers: michelle, vihang

Reviewed By: michelle

JIRA Issues: PC-1196

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

GitOrigin-RevId: 6da9391c0b3c24549a5498c7fff7d2ff0bd358ad
      2d633fd5
    • Michelle Nguyen's avatar
      Add `patches` flag to `px deploy` · e4b14d8a
      Michelle Nguyen authored 
      Summary: Before, we only supported patches by passing them through the `values.yaml` in the helm chart. We should also support patches for users using the CLI.

Test Plan: `bazel run :px -- deploy --patches="vizier-pem:{'spec':{'template':{'spec':{'nodeSelector':{'pixie': 'allowed'}}}}}"`

Reviewers: vihang, zasgar

Reviewed By: vihang

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

GitOrigin-RevId: 58d445315158bcdaf0f0c565b6f385ee52b385df
      e4b14d8a
    • Omid Azizi's avatar
      Naming: Use test_go_binary · cae71895
      Omid Azizi authored 
      Summary: Fixes naming to be more inclusive.

Test Plan: Existing tests

Reviewers: #stirling, zasgar, vihang

Reviewed By: vihang

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

GitOrigin-RevId: 9c3e78759bce59958e0c10faf83b63560f8c8be0
      cae71895
    • Vihang Mehta's avatar
      Fix licenses · 0d5e19e5
      Vihang Mehta authored 
      Summary: Looks like this repo accidentally dropped the LICENSE file.

Test Plan: Run build with stamping

Reviewers: michelle

Reviewed By: michelle

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

GitOrigin-RevId: eb6613746232d3756574806d9df8dcd01b03db51
      0d5e19e5
    • Vihang Mehta's avatar
      Increase BES timeout · 1f088987
      Vihang Mehta authored 
      Summary: We seem to be running into this timeout from `gen_compilation_database`

Test Plan: N/A

Reviewers: zasgar, michelle

Reviewed By: michelle

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

GitOrigin-RevId: f481f4c4bbf4d7aa4624833ef46a824f3e18af4b
      1f088987
  7. 28 Sep, 2021 7 commits
    • Yaxiong Zhao's avatar
      [BUILD FIX] Skip test for coverage build · c98d5c9f
      Yaxiong Zhao authored 
      Summary: Similar to the other test above this one. Coverage build meddles test binary. So skip.

Test Plan: Jenkins

Reviewers: #stirling, oazizi, vihang

Reviewed By: vihang

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

GitOrigin-RevId: 1a90ffe0d4079c6bfa7c6865e6bd37fd9e935614
      c98d5c9f
    • Vihang Mehta's avatar
      Make the bpftrace checkout shallow · 1a560d44
      Vihang Mehta authored 
      Summary: TSIA. This was printed by bazel debug logs.

Test Plan: Exsiting

Reviewers: zasgar, oazizi, #third_party_approvers

Reviewed By: zasgar, #third_party_approvers

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

GitOrigin-RevId: d4880419730285227bc44576179a6977021c7bc5
      1a560d44
    • Yaxiong Zhao's avatar
      [CLEANUP] Rename elf_tools* to elf_reader* · b2a71fc1
      Yaxiong Zhao authored 
      Summary:
As part of the renaming, ExtractGolangInterfaces() is moved to go_syms.{h,cc}
This cleanup is to make it easier to discover where ElfReader locates.

Test Plan: Jenkins

Reviewers: #stirling, oazizi

Reviewed By: #stirling, oazizi

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

GitOrigin-RevId: 83d2f95d2e1a0ba4f000ec7d81b216a925346fa9
      b2a71fc1
    • Nick Lanam's avatar
      [Chore] Update Webpack; ES targets; Node. Fix axios ref. · b54a5e77
      Nick Lanam authored 
      Summary:
In preparation for web workers, which rely on some bug fixes in Webpack and some features in ES2020.
- Webpack: version 5.51.2 fixes [a bug](https://github.com/webpack/webpack/releases/tag/v5.51.2) that prevents web worker code from building properly.
- Axios: We updated to 0.21.4 [two weeks ago](https://github.com/pixie-io/pixie/commit/c46d717baca758d72470f15f6ce2f6b3a819ccc9) for a security advisory, but forgot to update `package.json` to lock it in.
- `tsconfig.json`: `es2020` includes several features that we'll need for web workers.
- `jest.config.js`, `webpack.config.js`: Match `es2020` configuration.
- NodeJS: We already expect Node 16 everywhere else and install it in our Chef configuration. It's the currently supported release, and becomes LTS next month. So, just lining up here.

Test Plan: `yarn test`. `yarn dev`; smoke test scripts, admin pages, and so on for anything that behaves differently from normal. It should behave the same as before. If you aren't already running Node 16, install it through your preferred method (tested with a bare installation and with `nvm use 16` on my system).

Reviewers: michelle, vihang, #third_party_approvers

Reviewed By: vihang, #third_party_approvers

JIRA Issues: PP-3039

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

GitOrigin-RevId: 9ffb7fe6f181974524fe5213da3284aca848de65
      b54a5e77
    • Yaxiong Zhao's avatar
      Add API that invoke on an executable to get version · 45636973
      Yaxiong Zhao authored 
      Test Plan: Jenkins

Reviewers: #stirling, jps

Reviewed By: #stirling, jps

Subscribers: jps

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

GitOrigin-RevId: 2f278a50ddc24ec416aa16cafdd844a8bee8268c
      45636973
    • Natalie Serrino's avatar
      PC-1190: Refactor Service Graph to fix some bugs, show IP when pod/service not present · 82ac7bc3
      Natalie Serrino authored 
      Summary:
Request Graph evolved over time, and in the meantime some cruft has built up. This diff addresses the following issues:

1. In clustered mode, previously we would actually select one of the pod->pod percentiles randomly that applied to that service->service latency. This is fine when there is one pod per service, but incorrect when there are multiple pods communicating per service.
2. We would show <unknown service> instead of the information that we do have about a particular edge/node. Instead, let's show IP address (PxL updates to come next -- but this works without the PxL updates so it's not a breaking change) for inbound/outbound traffic that we can't attribute to a particular pod/service.
3. Move some of the pure logic outside of the React component in order to make the React component a bit easier to understand.

Next up will be updating the PxL script to produce IP addresses.

Test Plan: ran the UI with various options being toggled.

Reviewers: michelle, nlanam, zasgar

Reviewed By: nlanam

Subscribers: oazizi

JIRA Issues: PC-1190

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

GitOrigin-RevId: b6a3cc176a75b91b54fb84715e15ab0243b98dad
      82ac7bc3
    • Vihang Mehta's avatar
      Upgrade rules_docker · 41443f11
      Vihang Mehta authored 
      Summary:
https://github.com/bazelbuild/rules_docker/pull/1831 made it into
a release so we can stop patching.

Test Plan: build and test all

Reviewers: zasgar, #third_party_approvers

Reviewed By: zasgar, #third_party_approvers

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

GitOrigin-RevId: 50c3b0705aa8039ed57b1b9f39bfea909bb48372
      41443f11

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