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.
- 04 Oct, 2021 12 commits
-
-
Yaxiong Zhao authored
Summary: Right now the offsets are still hardcoded. Follow up diffs will populate them from dwarf symbols, or from pre-built ones for the corresponding version. Test Plan: Jenkins Reviewers: #stirling, oazizi Reviewed By: #stirling, oazizi Subscribers: oazizi Signed-off-by:
Yaxiong Zhao <yzhao@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D9877 GitOrigin-RevId: 0fde2f31e26b833b63670b22f1d7d381aa2064cd
-
Yaxiong Zhao authored
Test Plan: Jenkins Reviewers: #stirling, oazizi Reviewed By: #stirling, oazizi Signed-off-by:
-
Michelle Nguyen authored
Summary: this diff implements the gql endpoint, which uses the API grpc interface to make a call to the profile service. Test Plan: unit test Reviewers: vihang, zasgar Reviewed By: vihang JIRA Issues: PP-3063 Signed-off-by:
Michelle Nguyen <michellenguyen@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D9897 GitOrigin-RevId: f3ae25886bc5b4e94ea5cddf72847c5e8136fcc2
-
Vihang Mehta authored
Summary: TSIA Test Plan: N/A Reviewers: michelle Reviewed By: michelle JIRA Issues: PXC-61 Signed-off-by:
Vihang Mehta <vihang@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D9896 GitOrigin-RevId: dfd509c5a177fc9307eed3322f03eb14f0c94da6
-
Michelle Nguyen authored
Summary: this implements the IDEConfig functions (delete, add, get) for the API server. Test Plan: unit tests Reviewers: vihang, zasgar Reviewed By: vihang JIRA Issues: PP-3063 Signed-off-by:
-
Michelle Nguyen authored
Summary: we will allow others to use the grpc API to make configurations to their org (specifically, adding IDE configs) Test Plan: n/a Reviewers: vihang, zasgar Reviewed By: vihang JIRA Issues: PP-3063 Signed-off-by:
-
Omid Azizi authored
Summary: These scripts were move for organizational purposes in D9846, but restore them since there may be uses in the wild. Test Plan: Manual Reviewers: htroisi, michelle Reviewed By: michelle Signed-off-by:
Omid Azizi <oazizi@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D9891 GitOrigin-RevId: cdbac26a53951ef083393c62a59ed3be16cdf52c
-
Michelle Nguyen authored
Summary: previous diff implemented the profile datastore. this diff implements the profile server for IDEConfigs, which uses the datastore Test Plan: unit test Reviewers: vihang, zasgar Reviewed By: vihang JIRA Issues: PP-3063 Signed-off-by:
-
Michelle Nguyen authored
Summary: before, we were planning to use golang templates to fill in the IDEConfig paths. however, this part will actually be done in the UI. we should use mustache.js template styles here instead. updating the proto comments to reflect this new change Test Plan: n/a Reviewers: vihang, zasgar Reviewed By: vihang Signed-off-by:
-
Michelle Nguyen authored
Summary: this implements the functions for adding/getting IDEConfigs for an org in the datastore Test Plan: unit test Reviewers: vihang, zasgar Reviewed By: vihang JIRA Issues: PP-3063 Signed-off-by:
-
Michelle Nguyen authored
Summary: On right-click (or other interaction), the UI should show the user a dropdown of their configured IDEs and the path to the selected symbol. there were two possibilities i could think of for the gql endpoint: 1. (chosen in this diff) Upon right click, make a call to our backend with the symbol. the backend returns the IDE paths. 2. Preload the orgs configured IDE settings. Upon right click, fill in the IDE paths with the symbol in the UI. I figured that (1) provides more flexibility for configuring the IDE path/keeps unnecessary logic out of the UI. Our paths are also expected to be go templates, so it will be easier to fill out there. Test Plan: existing tests pass/build Reviewers: vihang, zasgar Reviewed By: vihang JIRA Issues: PP-3063 Signed-off-by:
-
Michelle Nguyen authored
Summary: was going to add the IDEConfig settings to the profile service/its datastore, but figured some cleanup needed to be done. we used to pair user/org very tightly together, however we have started adding more functions to org/user overtime. this list will likely continue to grow. for cleanup, I split up the Datastore interface into smaller ones. There are some functions (for example, CreateUserAndOrg and DeleteOrgAndUsers, etc) which cross the domain of org+users, so it isn't a perfect split. am also open to other ideas about where these could go. Test Plan: existing tests pass Reviewers: vihang, philkuz, zasgar Reviewed By: vihang JIRA Issues: PP-3063 Signed-off-by:
-
- 01 Oct, 2021 10 commits
-
-
Yaxiong Zhao authored
Summary: This helps improve the contextual info of the logs. Test Plan: Jenkins Reviewers: #stirling, rcheng Reviewed By: #stirling, rcheng Signed-off-by:
-
Michelle Nguyen authored
Summary: previous diff added the IDEconfig protos to the API service. however, that is just a proxy to our profile service. this diff adds the protos to our profile service. they are more-or-less the same. also separating some of the org functionality into a separate service, since our list of rpc methods are growing (and will probably grow even more in the coming future) Test Plan: existing tests pass + build Reviewers: vihang, zasgar Reviewed By: vihang JIRA Issues: PP-3063 Signed-off-by:
-
Yaxiong Zhao authored
Summary: Instead of the source file name. This affect pl_bpf_cc_resource() in that the symbol name now loses the `_bpf_src_` name component. But the user code does not need to change. It works by copying the input file to another with the rule's name. With this diff, the names of the intermediate rules of the wrapper rules (pl_cc_resource, pl_bpf_cc_resource) no longer affect the output. Noticed this while testing embedding a text protobuf file. Also removed the debug build rules, as they can be verified by building the embedded rule directly. Test Plan: Jenkins Reviewers: #stirling, oazizi Reviewed By: #stirling, oazizi Subscribers: oazizi Signed-off-by:
-
Natalie Serrino authored
Summary: Depends on D9864. Previously, we were doing an inner join between the process_stats table and http_events table to generate the list of services present. Now that PC-1174 is fixed, we can do a left join, and show non-HTTP services even when we have HTTP-specific columns. Also, HTTP-specific columns in Services list are now prepended with HTTP to make it clear that these are HTTP statistics, not overall statistics for the service. Test Plan: ran it in UI Reviewers: philkuz, oazizi, htroisi, zasgar Reviewed By: oazizi JIRA Issues: PC-1175 Signed-off-by:
Natalie Serrino <nserrino@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D9870 GitOrigin-RevId: edbc69049b6c23d031d27ea74b978d5f13d62aff
-
Natalie Serrino authored
Summary: We want requestor IP and responder IP to show up on the service graph widget for external traffic (a.k.a traffic into or out of the K8s cluster). This diff adds those fields to the vis spec, and updates the PxL scripts using the service graph widget to produce those values. Note that everything will still work even if those values are not produced, so it's not breaking either way from the perspective of the UI. Test Plan: ran the scripts, ran the CLI to make sure that proto stays happy Reviewers: michelle, oazizi, vihang, zasgar Reviewed By: oazizi JIRA Issues: PC-1180 Signed-off-by:
-
Michelle Nguyen authored
Summary: we will allow users to "open in IDE" from the flamegraph. they can configure their paths for their org, and we will store it in this table. since we will support multiple IDEs, we have a separate table where they can add as many configs as they want Test Plan: datastore tests pass Reviewers: vihang, zasgar Reviewed By: vihang JIRA Issues: PP-3063 Signed-off-by:
-
Michelle Nguyen authored
Summary: We need to be able to add ConfigIDEs to orgs. this adds the protos for adding/deleting/and getting them. considered rollign this into a general "GetOrgSettings" method, but it didnt seem as clean to add/delete IDEConfigs from there, since there can be multiple configs. Test Plan: everything builds Reviewers: vihang, zasgar Reviewed By: vihang JIRA Issues: PP-3063 Signed-off-by:
-
Vihang Mehta authored
Summary: Bazel reported that this could be updated Test Plan: Exsiting Reviewers: oazizi, zasgar, #third_party_approvers Reviewed By: oazizi, zasgar, #third_party_approvers Signed-off-by:
-
Zain Asgar authored
Summary: We need to update certs to match the nats discovery domains. Test Plan: N/A Reviewers: michelle, vihang Reviewed By: vihang Signed-off-by:
Zain Asgar <zasgar@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D9879 GitOrigin-RevId: 730470addb6baefc423f978f68ea6a84d3a36639
-
Zain Asgar authored
Summary: Switch from CLA to DCO. Test Plan: N/A Reviewers: michelle, vihang Reviewed By: vihang JIRA Issues: PP-3046 Signed-off-by:
-
- 30 Sep, 2021 15 commits
-
-
Ryan Cheng authored
Summary: PluckArrayUDF extends PxL's ability to handle json objects. It allows picking out a value in the ith position of an array in a json object. Test Plan: Added unit tests Reviewers: #stirling, nserrino, michelle, oazizi Reviewed By: #stirling, nserrino, oazizi Subscribers: oazizi Signed-off-by:
Ryan Cheng <rcheng@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D9871 GitOrigin-RevId: 29d709c47409697b818efc42f9d78a9f8782c873
-
Yaxiong Zhao authored
Test Plan: Jenkins Reviewers: #stirling, oazizi Reviewed By: #stirling, oazizi Subscribers: vihang, oazizi Signed-off-by:
-
Omid Azizi authored
Summary: Use more inclusive language. Test Plan: Existing tests Reviewers: #stirling, yzhao Reviewed By: #stirling, yzhao Signed-off-by:
-
Omid Azizi authored
Summary: Move bpftrace scripts from the BPFTrace community into a folder for organization purposes. Test Plan: Manual Reviewers: zasgar, htroisi, nserrino Reviewed By: htroisi Subscribers: michelle Signed-off-by:
-
Natalie Serrino authored
Summary: In these tables, we are displaying inbound traffic to (pod, service) to the user. However, we don't mention anywhere in the title that this is only HTTP traffic, so let's update the titles to make that clear. Test Plan: n/a Reviewers: oazizi, zasgar, michelle, htroisi Reviewed By: oazizi Signed-off-by:
-
Omid Azizi authored
Summary: More inclusive naming in DynamicTracer Test Plan: Existing tests Reviewers: #stirling, zasgar Reviewed By: zasgar Signed-off-by:
-
Phillip Kuznetsov authored
Summary: Our CRD version is only compatible for 1.16 and up. https://kubernetes.io/docs/reference/using-api/deprecation-guide/#v1-22 We've already updated our messaging in the docs, needed to update it here as well. https://github.com/pixie-io/pixie-docs/pull/86 Test Plan: n/a Reviewers: michelle, vihang, nserrino Reviewed By: vihang Signed-off-by:
Phillip Kuznetsov <pkuznetsov@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D9865 GitOrigin-RevId: 4e54c8a0febfab1c8595c8993add88c3ce7b52d4
-
Nick Lanam authored
Summary: TSIA. This is to set up for calling dataFromProto more frequently with smaller batches, and making those calls more efficient when they do happen. Test Plan: Run the UI locally. For complex streamed scripts (like `http_data` on a high-traffic cluster), it should take a little longer for the page to become unresponsive with tens of thousands of rows. dataFromProto is much faster, but it's still a slow method in a long critical path. Reviewers: michelle, vihang Reviewed By: michelle JIRA Issues: PP-3039 Signed-off-by:
Nick Lanam <nlanam@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D9867 GitOrigin-RevId: 8263be3c59ab7174146db7119f870975fb3a73cd
-
Nick Lanam authored
Summary: More prep work for PP-3039. Also fixes lint issues in touched files, which will likely see a lot more activity soon. Performance improvement is insignificant (measured in microseconds per script run); this is more to match style with other pending changes. Test Plan: Run the UI locally. Should not have any noticeable difference. Reviewers: michelle, vihang Reviewed By: michelle JIRA Issues: PP-3039 Signed-off-by:
-
Omid Azizi authored
Summary: More inclusive terminology. Test Plan: Existing tests Reviewers: #stirling, zasgar Reviewed By: zasgar Signed-off-by:
-
Omid Azizi authored
Summary: More inclusive naming style. Test Plan: Existing tests. Reviewers: #stirling, zasgar Reviewed By: zasgar Signed-off-by:
-
Vihang Mehta authored
Summary: TSIA Test Plan: Install hooks Reviewers: zasgar, michelle Reviewed By: zasgar Signed-off-by:
-
Vihang Mehta authored
Summary: These should be installed in the phabricator instance we use to add the new field. Test Plan: Try on a dockerized test instance Reviewers: zasgar, michelle Reviewed By: zasgar JIRA Issues: PP-3046 Signed-off-by:
-
Vihang Mehta authored
Summary: This gives us symbols in nats for flamegraphs. Test Plan: Run a dev cloud Reviewers: michelle, philkuz, zasgar Reviewed By: michelle Differential Revision: https://phab.corp.pixielabs.ai/D9862 GitOrigin-RevId: 978f5ddd5213abe1749505599cfc45179f5c1ffd
-
Yaxiong Zhao authored
Test Plan: bazel coverage ... passed Reviewers: #stirling, oazizi Reviewed By: #stirling, oazizi Differential Revision: https://phab.corp.pixielabs.ai/D9863 GitOrigin-RevId: 1c32636b28928562ae4f253dc7ae0ceb62200675
-
- 29 Sep, 2021 3 commits
-
-
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
-
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
-
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
-
