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 3 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 9 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 sa...
      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
    • Omid Azizi's avatar
      DynamicBPFTrace cleanup · d9fd8761
      Omid Azizi authored 
      Summary: Address a TODO, make BPFtrace a member, add FieldAnalyser to be consistent with upstream usage model.

Test Plan: Existing tests

Reviewers: #stirling, yzhao

Reviewed By: #stirling, yzhao

Subscribers: yzhao

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

GitOrigin-RevId: e2131a04bc948613d4380066123cd692bda80417
      d9fd8761
    • Michelle Nguyen's avatar
      Remove storage class check from CLI · e39b7c9f
      Michelle Nguyen authored 
      Summary: this is now in the operator, so we dont need the duplicated logic

Test Plan: n/a

Reviewers: nserrino, vihang, philkuz

Reviewed By: vihang

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

GitOrigin-RevId: 88f4b90f7e7bb7c700927ba1cde8bb9930cdf12b
      e39b7c9f

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