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.
Repository mirroring has been paused due to too many failed attempts. It can be resumed by a project maintainer.
- 15 Dec, 2021 8 commits
-
-
Zain Asgar authored
Summary: Bump memory to 1.5G for bazel to prevent OOM issues. Test Plan: N/a Reviewers: vihang, oazizi Reviewed By: vihang Signed-off-by:
Zain Asgar <zasgar@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10382 GitOrigin-RevId: 82fb6a0e9d79e6955e2278391c957c68403f4257
-
Zain Asgar authored
Summary: This will allow us to add copts to the buld commands. Added parallel builds where possible. Test Plan: Jenkins Reviewers: vihang, oazizi, michelle, #third_party_approvers Reviewed By: vihang, #third_party_approvers Signed-off-by:
Zain Asgar <zasgar@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10380 GitOrigin-RevId: 4030f9282ccf0f085ffe44793ac27c6d49bc1d39
-
Phillip Kuznetsov authored
Summary: Reset invite links now resets the invites. Test Plan: yarn dev and clicked reset. Links used afterwards stopped working - successfully sent the message after the link button was pressed. Reviewers: nlanam, vihang, michelle Reviewed By: vihang JIRA Issues: PC-1328 Signed-off-by:
Phillip Kuznetsov <pkuznetsov@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10377 GitOrigin-RevId: 40af6c745abdbd1ad9fbaf977a8ec3a143add5df
-
Vihang Mehta authored
Summary: Some legacy scenarios require us to support the creation of a self org for users. We check for the flag in the app_metadata and if it's set then we create the self org. Test Plan: Added tests Reviewers: michelle, philkuz Reviewed By: philkuz Signed-off-by:
Vihang Mehta <vihang@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10376 GitOrigin-RevId: 0caf4a0c2735af0d206dc3201a44d4e9faf16a83
-
Nick Lanam authored
Summary: This was causing a few places where the UI would briefly flicker a `Loading...` screen before effectively reloading everything. The problem was that, whenever anything reloads org data, that loading state was causing the top level router to cut off and wait for setup to be marked complete. Instead, it should have only done this until org data has loaded at least once. Test Plan: Open admin tab. For both admin and deploy keys: try adding one, then removing it. The page should not look like it fully reloaded; instead it should only update the key table. I'm not aware of other places that this was happening. Note: switching clusters in the live view still has the same bug, but for what seems to be an unrelated reason. Reviewers: michelle, vihang, philkuz Reviewed By: vihang, philkuz JIRA Issues: PC-1312 Signed-off-by:
Nick Lanam <nlanam@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10372 GitOrigin-RevId: a7b5e379972e8697db32d48027204cd8181e6b3f
-
Phillip Kuznetsov authored
Summary: Call RemoveUserFromOrg() mutation when we hit the `Remove User` button. Simple wiring up Test Plan: Tested on dev backend and successfully updated the state of hte user. Also faked an error to make sure those errors show up as well. Reviewers: nlanam, vihang, michelle Reviewed By: vihang JIRA Issues: PC-1321 Signed-off-by:
Phillip Kuznetsov <pkuznetsov@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10371 GitOrigin-RevId: 1a971bab175851185cbbd1dfba0868535d1e19a6
-
Zain Asgar authored
Summary: This reverts commit 6dfbea3cc783360525043442e58047714950ed51. Fixes ASAN/TSAN tests. Test Plan: Jenkins Reviewers: vihang, michelle, oazizi, #third_party_approvers Reviewed By: vihang, oazizi, #third_party_approvers Signed-off-by:
Zain Asgar <zasgar@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10374 GitOrigin-RevId: 695662c7b626249147445caa5ad480b14b4fc5b3
-
Hannah Troisi authored
Summary: Previously, `ctrl+enter` to expand a truncated table cell would not work for the 1st row. Test Plan: Built & tested the CLI. Reviewers: michelle, zasgar Reviewed By: zasgar JIRA Issues: PX-139 Signed-off-by:
Hannah Troisi <htroisi@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10368 GitOrigin-RevId: ac9abf5d70311cab164cd46facaa7927335b6c86
-
- 14 Dec, 2021 19 commits
-
-
Dom Del Nano authored
Summary: A variety of fixes to the Mux protocol parser and stitcher. Test Plan: Tests updated Reviewers: #stirling, rcheng Reviewed By: #stirling, rcheng Subscribers: rcheng Signed-off-by:
Dom Del Nano <ddelnano@gmail.com> Differential Revision: https://phab.corp.pixielabs.ai/D10369 GitOrigin-RevId: 7983d8eb36c2dff6046557f0bf5fa1f5377cf9fe
-
Omid Azizi authored
Summary: Add Go 1.17 versions of our Go TLS tracing targets. Test Plan: Manual. Will be incorporated into a BPF test in a future diff. Reviewers: #stirling, jps Reviewed By: #stirling, jps Signed-off-by:
Omid Azizi <oazizi@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10361 GitOrigin-RevId: bba9c274d65eb27621580f1191cfe992b279acd2
-
Pete Stevenson authored
Adding a 3rd party dep. on jattach forked into pixie-io; jattach is a utility that injects JVMTI agents into Java processes. We will use it to inject the pixie symbolization agent. Summary: Adding a 3rd party dep. on jattach forked into pixie-io; jattach is a utility that injects JVMTI agents into Java processes. We will use it to inject the pixie symbolization agent. Test Plan: Will be tested by e2e test of java symbolization, i.e. because java symbolization *will not* work w/out this. Reviewers: #stirling, #third_party_approvers, oazizi Reviewed By: #stirling, #third_party_approvers, oazizi Subscribers: oazizi JIRA Issues: PP-2982 Signed-off-by:
Pete Stevenson <jps@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10301 GitOrigin-RevId: 3f217cf20726628f957ba988cf94079699592e0f
-
Natalie Serrino authored
Summary: We want to extend the pod phases we track in autocomplete and Vizier in order to include a terminated state. This is so that autocomplete and Vizier queries can report a "Terminated" state for pods whose last known kubernetes phase was "running" but are in fact terminated by the system. (Failed and Succeeded pods will continue to have the same phase as before). In this diff, we add the terminated pod phase to Vizier (both in the metadata service as well as in the metadata UDFs). This is an extension of the Kubernetes pod phases, see PC-1311 and PC-1305 for more context. Test Plan: Ran Vizier against prod cloud and deleted a pod, unit tests Reviewers: michelle, philkuz Reviewed By: philkuz JIRA Issues: PC-1311 Signed-off-by:
Natalie Serrino <nserrino@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10339 GitOrigin-RevId: 4ca6f1fdaf3e99f5f8a4dc6adbad5edc24f25744
-
Phillip Kuznetsov authored
Summary: This diff makes the invite_token id valid by grabbing it from the backend. 1. Had to grab the org id from the bakcned 2. Then pass org_id to token. I'm not super satisfied with my chaining of useMutation and useQuery by having a useEffect, but I can't think of a less ugly solution. Test Plan: Tested with a valid skaffold backend. Reviewers: nlanam, michelle Reviewed By: nlanam JIRA Issues: PC-1318 Signed-off-by:
Phillip Kuznetsov <pkuznetsov@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10370 GitOrigin-RevId: 947a6bab1427f4a8dce9dcf2e3a84f3368b7c528
-
Pete Stevenson authored
Summary: perf_profiler/java: move test cases to be peers to their underlying collateral. Test Plan: This is a test. Reviewers: #stirling, oazizi Reviewed By: #stirling, oazizi JIRA Issues: PP-2982 Signed-off-by:
Pete Stevenson <jps@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10282 GitOrigin-RevId: 331d2f025c5a1a411e31c2f02e712beb76aafa79
-
Phillip Kuznetsov authored
Summary: We don't want to allow users to login who don't have an org DomainName set - these are users who we need to migrate from the old org model into the new one. We updated this check because the old one started to fail when we started to allow google users to manually create orgs. Test Plan: Added tests for new conditions, updated old tests to be precise. Reviewers: vihang, michelle Reviewed By: vihang JIRA Issues: PC-1316 Signed-off-by:
Phillip Kuznetsov <pkuznetsov@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10350 GitOrigin-RevId: 58a9c3f0983968f3552bb5b3f4520b3fcbbff268
-
Phillip Kuznetsov authored
Summary: domain_name NULL is now useful information to track orgs that have not migrated to our new system. We want users who have a NULL domain name and use google_oauth to be alerted so that we can migrate them. We currently gate on those google users who don't have a hosted domain, their org_name is not equal to their email - but that's going to change when we allow those users to join orgs with manual names. Test Plan: Updated test and tried some end-to-end things as well. Reviewers: michelle, vihang Reviewed By: vihang JIRA Issues: PC-1316 Signed-off-by:
Phillip Kuznetsov <pkuznetsov@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10349 GitOrigin-RevId: 6b4dbcd4f05441102753e30596060455e99c215c
-
Vihang Mehta authored
Summary: Looks like this is why D10346 didn't run any builds. Test Plan: N/A Reviewers: zasgar, oazizi, michelle Reviewed By: zasgar Signed-off-by:
Vihang Mehta <vihang@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10367 GitOrigin-RevId: 5da93c47d05b5bec3ed76df6749763da515eb55d
-
Zain Asgar authored
Summary: This reverts commit f09a9e74ef9237f1f15d27c64de3110c28cbd67a. Looks like this breaks ASAN. Will fix an resubmit. Test Plan: jenkins Reviewers: oazizi, michelle, vihang, #third_party_approvers Reviewed By: michelle, #third_party_approvers Signed-off-by:
Zain Asgar <zasgar@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10366 GitOrigin-RevId: 6dfbea3cc783360525043442e58047714950ed51
-
Vihang Mehta authored
Summary: Add RemoveUserFromOrg to GraphQL Test Plan: Added unit tests Reviewers: michelle, philkuz Reviewed By: michelle JIRA Issues: PC-1323 Signed-off-by:
Vihang Mehta <vihang@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10358 GitOrigin-RevId: e719279f7777ff197e13141f01772074f5e05a50
-
Vihang Mehta authored
Summary: Adds support to let admins remove users from their org. Test Plan: Added unit tests Reviewers: michelle, philkuz Reviewed By: michelle, philkuz JIRA Issues: PC-1320 Signed-off-by:
Vihang Mehta <vihang@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10357 GitOrigin-RevId: b61ae4171c534a24579a3935f9d9f01e03eb90e2
-
Phillip Kuznetsov authored
Summary: user who were invited were not being added to the org they were invited to join, if they existed. Now we check if the user exists and add them to the org if they do. Test Plan: Tested on skaffold and add test Reviewers: vihang, michelle Reviewed By: vihang Signed-off-by:
Phillip Kuznetsov <pkuznetsov@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10354 GitOrigin-RevId: c9e56f3607514ab5350f091266fc199a27d403c9
-
Phillip Kuznetsov authored
Summary: We check if num users is 0 to add user to org, but we don't need to. This blocked the invite feature. Removing for now Test Plan: updated test Reviewers: vihang, michelle Reviewed By: vihang Signed-off-by:
Phillip Kuznetsov <pkuznetsov@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10353 GitOrigin-RevId: 1556aec02d9b7cc75bbc522291f6be47788e1d11
-
Omid Azizi authored
Summary: Using more than one go toolchain in bazel is difficult, so here, we build our go tracing target test binaries in containers. We plan to extend this model to all of our tracing target test binary builds. Test Plan: Existing tests Reviewers: #stirling, jps Reviewed By: #stirling, jps Subscribers: jps Signed-off-by:
Omid Azizi <oazizi@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10359 GitOrigin-RevId: 2b3091e0cbd54514d4be60d09b22059eeb7df8bf
-
Michelle Nguyen authored
Summary: We can now autocomplete nodes. This diff updates the APIs to include the node type Test Plan: deployed to testing cloud, deployed vizier off main, then tested e2e Reviewers: nserrino, vihang Reviewed By: vihang JIRA Issues: PC-1292 Signed-off-by:
Michelle Nguyen <michellenguyen@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10351 GitOrigin-RevId: 833db4ec666508ac66d3c1c8dab34693b4edeaec
-
Pete Stevenson authored
Summary: perf_profiler/java: added sub-trees for docker build of glibc & musl versions of symbolization agent. Test Plan: TBD / add a java app in an Alpine linux container and get symbol file out. Reviewers: #stirling, #third_party_approvers, yzhao, oazizi, zasgar Reviewed By: #stirling, #third_party_approvers, yzhao, oazizi, zasgar Subscribers: zasgar, oazizi, yzhao JIRA Issues: PP-2982 Signed-off-by:
Pete Stevenson <jps@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10259 GitOrigin-RevId: 317357d4cf7250617dcaf552d6de1d6570d52caa
-
Zain Asgar authored
Summary: We need this for the upcoming clang-13 update. Test Plan: Jenkins Reviewers: michelle, vihang, oazizi, #third_party_approvers Reviewed By: vihang, oazizi, #third_party_approvers Signed-off-by:
Zain Asgar <zasgar@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10346 GitOrigin-RevId: f09a9e74ef9237f1f15d27c64de3110c28cbd67a
-
Phillip Kuznetsov authored
Summary: We expected cloudpb::CreateOrg() to set the domain_name empty, but right now it leaves it null. We want to use the NULL domain_name to track users who have been migrated to our new org system so its important that new orgs do not have a NULL domain_name. Will need to migrate manual orgs created before this change so that they do not have a NULL domain_name in the db. Test Plan: Updated tests Reviewers: vihang, michelle Reviewed By: vihang JIRA Issues: PC-1319 Signed-off-by:
Phillip Kuznetsov <pkuznetsov@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10347 GitOrigin-RevId: c6c4c03aad540e72ccf3aeba98e3310cc80d09f8
-
- 13 Dec, 2021 3 commits
-
-
Ryan Cheng authored
Summary: Update to use the correct graph Test Plan: NA Reviewers: #stirling, oazizi Reviewed By: #stirling, oazizi Signed-off-by:
Ryan Cheng <rcheng@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10345 GitOrigin-RevId: a038e7ed930c4a0ff54d0c29baaae8c4b9aa84cd
-
Omid Azizi authored
Summary: Stylistic changes. Test Plan: Existing tests Reviewers: #stirling, yzhao Reviewed By: #stirling, yzhao Subscribers: yzhao Signed-off-by:
Omid Azizi <oazizi@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10344 GitOrigin-RevId: a5d00ea51bc65533bbe7bbf4eb02f79e13793d91
-
Pete Stevenson authored
Summary: stirling/bpf_tools/bcc_bpf_intf/upid.h: add #include <cstdint> Test Plan: None, this adds a missing #include. Reviewers: #stirling, yzhao Reviewed By: #stirling, yzhao Signed-off-by:
Pete Stevenson <jps@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10313 GitOrigin-RevId: 65b53924386b5cd57ee6dedeca36758fe00ecc3b
-
- 10 Dec, 2021 6 commits
-
-
Ryan Cheng authored
Summary: Mutating the container in a range-based for loop is unsafe. Test Plan: Existing tests Reviewers: #stirling, oazizi Reviewed By: #stirling, oazizi Subscribers: oazizi Signed-off-by:
Ryan Cheng <rcheng@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10289 GitOrigin-RevId: 99ec1855cfe6d97c4a411b3b8a6ce6c1f3255301
-
Omid Azizi authored
Summary: Use a map-in-map type to simplify the clean-up once a PID no longer exists. Test Plan: Existing tests. Reviewers: #stirling, yzhao Reviewed By: #stirling, yzhao Subscribers: yzhao Signed-off-by:
Omid Azizi <oazizi@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10280 GitOrigin-RevId: a4bb064d1963e18e963a1691d82b45aa4a464de1
-
Michelle Nguyen authored
Summary: this diff adds handling node updates to the indexer. Test Plan: add unit test Reviewers: nserrino, vihang Reviewed By: vihang JIRA Issues: PC-1291 Signed-off-by:
Michelle Nguyen <michellenguyen@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10340 GitOrigin-RevId: dadf5bcda13db0ddf92418f1d24f23ccf2b26449
-
Michelle Nguyen authored
Summary: We were already watching for node updates to get a mapping from node -> IP. However, since we didn't actually use the node updates in the PEMs or our cloud, we didn't send them over NATS. this diff updates our metadata handler to send our node updates to PEMs + cloud (currently ignored in both) Test Plan: update unit test Reviewers: nserrino, vihang Reviewed By: nserrino JIRA Issues: PC-1291 Signed-off-by:
Michelle Nguyen <michellenguyen@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10336 GitOrigin-RevId: e6152eeeac6b31577d39151bdffcbf21a2916ca9
-
Omid Azizi authored
Summary: For Go 1.17 tracing, we are going to have to rely on DWARF information to identify whether values are in registers or on the stack. Part of this requires that we use the DWARF AT_location field more consistently. To prepare for that, start converting probes to be entry/return style. DWARF info will give us the locaiton of the arguments on the entry probe. Test Plan: Existing tests - no functional changes expected yet. Reviewers: #stirling, yzhao Reviewed By: #stirling, yzhao Subscribers: yzhao, jps Signed-off-by:
Omid Azizi <oazizi@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D9905 GitOrigin-RevId: 12a944e2f60b92487f73ea9f9a69db407c835f7d
-
Ryan Cheng authored
Summary: This doc serves as a guide for the community to contribute new protocol parsers to Stirling. https://docs.google.com/document/d/15wAxeMjF1RHGrMKKs2ygc7XOhiP4E5-EEO59NRLH1GA/edit#heading=h.g5s5u51ibp4m Test Plan: NA Reviewers: #stirling, oazizi Reviewed By: #stirling, oazizi Signed-off-by:
Ryan Cheng <rcheng@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10171 GitOrigin-RevId: 7b36a653ebcea3c3ee76da92dd033def0813fed3
-
- 09 Dec, 2021 4 commits
-
-
Omid Azizi authored
Summary: Move some code into a thrift.bzl file to keep WORKSPACE less cluttered. Test Plan: Jenkins Reviewers: zasgar, vihang Reviewed By: vihang Signed-off-by:
Omid Azizi <oazizi@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10342 GitOrigin-RevId: 1dac182bc4281cf89130f15bbb9e7e8c7a2a01d0
-
Dom Del Nano authored
Summary: Code required to build scala client server images. To be used for end to end testing of Thriftmux protocol tracing. Test Plan: Coming separately Reviewers: zasgar, #third_party_approvers, #stirling, yzhao Reviewed By: zasgar, #third_party_approvers, #stirling, yzhao Subscribers: yzhao Signed-off-by:
Dom Del Nano <ddelnano@gmail.com> Differential Revision: https://phab.corp.pixielabs.ai/D10299 GitOrigin-RevId: 57f5957b1647f27d59c5a8c45d4054d8a61350a4
-
Zain Asgar authored
Summary: The behavior of entrypoint changed with new versions of packer. Test Plan: test `run_docker.sh` Reviewers: vihang, michelle Reviewed By: vihang Signed-off-by:
Zain Asgar <zasgar@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10341 GitOrigin-RevId: b7292cf1a4cb3b930b9d4fedd0ab404935568773
-
Zain Asgar authored
Summary: TSIA Test Plan: build docker images Reviewers: vihang, michelle Reviewed By: vihang Signed-off-by:
Zain Asgar <zasgar@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D10330 GitOrigin-RevId: 74771e516422e1001a03cd7e823e6281f639509e
-