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.
- 06 Oct, 2021 3 commits
-
-
Phillip Kuznetsov authored
Summary: We recently added some more fields to the cluster status change event. Unfortunately, the analytics implementation would split up maps into columns based on the keys in the map ratehr than serializing the map outright. this caused analytics to break on one implementation. We fixed this problem for now by removing those fields and addressing the issue in the analytics engine. Test Plan: Test on a running cloud. Reviewers: michelle, vihang, zasgar Reviewed By: michelle Signed-off-by:
Phillip Kuznetsov <pkuznetsov@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D9926 GitOrigin-RevId: c6a539c1d794859afcbe91a893a92e65c5998220
-
Vihang Mehta authored
Summary: When `updateQueue` is full, `AddToUpdateQueue` would block but also hold the mutex to the queue map. This would prevent `ProcessUpdateQueue` from progressing since it could never acquire the lock to the queue map, and thus prevent the `updateQueue` from being drained. Instead restructure this so that `AddToUpdateQueue` falls through if the queue is full. Also reduce queue size to slow down update rollout. Test Plan: Added test that locks without these changes Reviewers: zasgar, michelle Reviewed By: michelle Signed-off-by:
Vihang Mehta <vihang@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D9916 GitOrigin-RevId: 3da738fe84e7654c07d86a398f39bcd668c6840a
-
Vihang Mehta authored
Summary: The cluster path got doubly wrapped in an array of arrays D9900. Test Plan: staging Reviewers: michelle, zasgar Reviewed By: michelle, zasgar Signed-off-by:
-
- 05 Oct, 2021 8 commits
-
-
Pete Stevenson authored
jvm_stats_connector.cc: use DetectApplication to pre-filter attempts to find the 'hot spot perf data' file. Summary: jvm_stats_connector.cc: use DetectApplication to pre-filter attempts to find the 'hot spot perf data' file. Test Plan: Existing 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/D9911 GitOrigin-RevId: f6e2024458a7cd998c976820b979e9e77b43d3a4
-
Vihang Mehta authored
Summary: We have seen multiple reports of our operator monitoring failing on the k8s watcher and restarting the watcher also fails since the RV is too old. On further investigation, it seems like we should be using informers to handle resource watching, so switch to informers Test Plan: skaffold deploy the operator with some additional logging Reviewers: michelle, philkuz Reviewed By: philkuz JIRA Issues: PP-3066 Signed-off-by:
-
Vihang Mehta authored
Summary: TSIA, creating RSA keys is expensive, let's cache them. Test Plan: Run the go api examples Reviewers: zasgar, philkuz Reviewed By: zasgar Signed-off-by:
-
Michelle Nguyen authored
Summary: there were a lot of spammy `Failed to parse current Node Kernel version` messages for people. there is some additional cleanup of the version string that we do for our checks in `px deploy`, which should be done here as well. Test Plan: skaffold ran operator Reviewers: vihang, philkuz, zasgar Reviewed By: vihang Signed-off-by:
Michelle Nguyen <michellenguyen@pixielabs.ai> Differential Revision: https://phab.corp.pixielabs.ai/D9907 GitOrigin-RevId: ca3ee637c7c1a6d1f1f3099a6b430e401e84eaaf
-
Pete Stevenson authored
Summary: detect_application.h|detect_application.cc: add support for java. Test Plan: Existing tests. Reviewers: #stirling, oazizi Reviewed By: #stirling, oazizi JIRA Issues: PP-2982 Signed-off-by:
-
Pete Stevenson authored
Summary: Moved detect_application.h|cc from socket_tracer sub-tree into utils sub-tree. Next, we will add support to java to the same. Test Plan: Normal tests. Reviewers: #stirling, oazizi Reviewed By: #stirling, oazizi JIRA Issues: PP-2982 Signed-off-by:
-
Michelle Nguyen authored
Summary: Shift-click in a go symbol will now pop up a menu, if the user has IDEs configured for their org. Test Plan: yarn dev {F169831} Reviewers: vihang, nlanam, zasgar, #third_party_approvers Reviewed By: vihang, #third_party_approvers JIRA Issues: PP-3064 Signed-off-by: -
Michelle Nguyen authored
Summary: we will use this in the flamegraphs IDE menu Test Plan: yarn dev Reviewers: vihang, nlanam, zasgar Reviewed By: vihang JIRA Issues: PP-3064 Signed-off-by:
-
- 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:
-
Vihang Mehta authored
Summary: TSIA Test Plan: N/A Reviewers: michelle Reviewed By: michelle JIRA Issues: PXC-61 Signed-off-by:
-
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 7 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:
-
