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. 27 Apr, 2021 11 commits
    • Vihang Mehta's avatar
      Remove unused parts of the "internal" vizier proto · b3502275
      Vihang Mehta authored
      Summary: TSIA
      
      Test Plan: Existing tests still pass.
      
      Reviewers: zasgar, michelle, nserrino
      
      Reviewed By: michelle
      
      JIRA Issues: PP-2661
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8419
      
      GitOrigin-RevId: 61016b2b9ad2c088e57063bd0dff566445208fdf
      b3502275
    • Vihang Mehta's avatar
      Rename cloudapipb folder and targets to cloudpb · a6df0c4b
      Vihang Mehta authored
      Summary: TSIA. This is more in line with our other protos.
      
      Test Plan: Existing tests.
      
      Reviewers: michelle, zasgar
      
      Reviewed By: michelle
      
      JIRA Issues: PP-2671
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8421
      
      GitOrigin-RevId: c1c4d617633108dadff408bbfee4cb2118cdc969
      a6df0c4b
    • Vihang Mehta's avatar
      Remove unused build rules · 6c83f3a8
      Vihang Mehta authored
      Summary:
      These are left over from the old python api process and the old vizier
      proto locations.
      
      Test Plan: Existing build and tests.
      
      Reviewers: zasgar, michelle
      
      Reviewed By: michelle
      
      JIRA Issues: PP-2671
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8423
      
      GitOrigin-RevId: 73ab605a2c130f9f3780bfe4d3e07cc113632aad
      6c83f3a8
    • James Bartlett's avatar
      Fix chef, by dropping sourcegraph lsif in favor of unpatched microsoft one. · 54fb8159
      James Bartlett authored
      Summary: Chef is failing because of some clownery by the people at the @sourcegraph/lsif-node repo. Replacing their version with the default microsoft package.
      
      Test Plan: ci/collect_and_upload_lsif.sh works. Tbh not sure how this will affect the lsif output.
      
      Reviewers: vihang, nlanam
      
      Reviewed By: vihang
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8409
      
      GitOrigin-RevId: e7240f73e0b5ed77cc5fb2d87f113733997701b7
      54fb8159
    • Vihang Mehta's avatar
      Check to see if changed file exists when triggering targets · b4dd4ff9
      Vihang Mehta authored
      Summary:
      git tells us about all changed files including ones that were deleted.
      When triggering all targets due to a change in a `BUILD.bazel` file, check for
      existence of the `BUILD.bazel` file so that we don't try to build and test removed
      targets.
      
      Test Plan: Ran the script with some folder renames.
      
      Reviewers: zasgar, oazizi, yzhao
      
      Reviewed By: yzhao
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8424
      
      GitOrigin-RevId: 3183e5b1ab679a3ce134cc7d7261498b4d6dbf35
      b4dd4ff9
    • Pete Stevenson's avatar
      scripts/create_gke_cluster.sh: add optional argument -z for zone. · c52a7c1e
      Pete Stevenson authored
      Summary:
      scripts/create_gke_cluster.sh: add optional argument -z for zone.
      
      We are adding an argument "-z" to specifiy the zone for the cluster. This is useful for the stirling perf eval flow because we want to run multiple perf evals in parallel (as a single experiement where we have a baseline and multiple perf experiments to evaluate). For isolation, sandboxing, and reproducibility, we spin-up and tear-down a new cluster for the individual experiments. Zone us-west1-a is oversubscribed by our team. Allowing the stirling perf eval flow to choose a different zone (it will use us-west1-b for now) enables the desired parallelism.
      
      Test Plan: Use case by stirling perf eval.
      
      Reviewers: #stirling, zasgar, vihang, michelle, oazizi
      
      Reviewed By: #stirling, oazizi
      
      Subscribers: oazizi
      
      JIRA Issues: PP-2555
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8422
      
      GitOrigin-RevId: 67b329d88f30ccbc7a99ea9993faff5e6351e0ae
      c52a7c1e
    • Yaxiong Zhao's avatar
      Add names to SourceConnector subclasses · 1c11ce8f
      Yaxiong Zhao authored
      Test Plan: Jenkins
      
      Reviewers: #stirling, oazizi
      
      Reviewed By: #stirling, oazizi
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8404
      
      GitOrigin-RevId: 14ad3daed873724d558e1837e86bca0fb8413a62
      1c11ce8f
    • Yaxiong Zhao's avatar
      ProcStatConnector output multiple tables · 52de5760
      Yaxiong Zhao authored
      Test Plan: Jenkins
      
      Reviewers: #stirling, jps
      
      Reviewed By: #stirling, jps
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8417
      
      GitOrigin-RevId: 84ac27b3bba97fbb2264007ec5438cdea0ae9451
      52de5760
    • Zain Asgar's avatar
      PC-865 - Create deploy keys from the Go Client API · 856e37b2
      Zain Asgar authored
      Summary: Allow the Go client API to create deploy keys.
      
      Test Plan: Added an example program and ran tests manually. Will add unit tests to this soon.
      
      Reviewers: vihang, michelle
      
      Reviewed By: vihang
      
      JIRA Issues: PC-865
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8412
      
      GitOrigin-RevId: 9940c00958fdcaf96a9a6e184e461c06ec7679d8
      856e37b2
    • Omid Azizi's avatar
      [Cleanup] ConnTracker timestamps · 9e5bd668
      Omid Azizi authored
      Summary: Tidying up a bit. Simplifying some logic.
      
      Test Plan: Existing tests. Should only affect CONN_TRACE logs, so risk is low.
      
      Reviewers: #stirling, yzhao
      
      Reviewed By: #stirling, yzhao
      
      Subscribers: yzhao
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8351
      
      GitOrigin-RevId: 4509cd87365dfeb63ca2cbc7dc563a338ff3b628
      9e5bd668
    • Omid Azizi's avatar
      Fewer calls to now(): part 4 · 97e606f0
      Omid Azizi authored
      Summary:
      Removes all instances of now() in ConnTracker.
      
      Call to now() to be passed in and can be shared across all ConnTrackers.
      
      Test Plan: Existing tests
      
      Reviewers: #stirling, yzhao
      
      Reviewed By: #stirling, yzhao
      
      Subscribers: yzhao
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8349
      
      GitOrigin-RevId: 27eb2c7a789142b16e364ae1a0579ee990bd441d
      97e606f0
  2. 26 Apr, 2021 14 commits
  3. 24 Apr, 2021 3 commits
    • Michelle Nguyen's avatar
      PP-2653 Add skaffold file for operator · 53cf65c9
      Michelle Nguyen authored
      Summary:
      deploying the operator through skaffold will speed up the development process.
      this diff just adds the skaffold file.
      
      Test Plan: ran `skaffold run -f skaffold/skaffold_operator.yaml`, verified that the operator starts up and runs.
      
      Reviewers: vihang, zasgar
      
      Reviewed By: vihang
      
      JIRA Issues: PP-2653
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8391
      
      GitOrigin-RevId: 619dbb71ad282a696816f2d51f275d294f1c0f10
      53cf65c9
    • Zain Asgar's avatar
      Update badges in readme · 69454079
      Zain Asgar authored
      Summary: TSIA
      
      Test Plan: n/a
      
      Reviewers: vihang
      
      Reviewed By: vihang
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8390
      
      GitOrigin-RevId: 18c636633d10201452a29e91fb69bbd230d4f782
      69454079
    • Pete Stevenson's avatar
      perf_profile_connector.h|cc: prep. work to filter out non-k8s processes from... · b17dc5e9
      Pete Stevenson authored
      perf_profile_connector.h|cc: prep. work to filter out non-k8s processes from symbolization in the perf. profiler. Make md::UPID a member of SymbolicStackTrace, pass ctx into AggregateStackTraces()
      
      Summary: perf_profile_connector.h|cc: prep. work to filter out non-k8s processes from symbolization in the perf. profiler. Make md::UPID a member of SymbolicStackTrace, pass ctx into AggregateStackTraces()
      
      Test Plan: Existing tests.
      
      Reviewers: #stirling, oazizi
      
      Reviewed By: #stirling, oazizi
      
      JIRA Issues: PP-2555, PP-2621
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8384
      
      GitOrigin-RevId: 2d0f9bdb536203d098041b6d86eaa098842c8f05
      b17dc5e9
  4. 23 Apr, 2021 12 commits
    • Nick Lanam's avatar
      [PC-674] Implement deep linking in /admin. · a7312be8
      Nick Lanam authored
      Summary:
      `/admin` now redirects to `/admin/cluster` (`/admin/cluster/somename` still works). There is also a `/admin/deployment-keys` and `/admin/api-keys`. The tab headers correspond to these new links.
      This also gets rid of needless nesting of `<Router>` and cleans up lint/type warnings in touched files. Scouts' rule, and all that.
      
      Test Plan: Try directly browsing to `/admin`, `/admin/api-keys`, and going to the admin page from the Live view (which goes to `/admin`). The API Keys link should directly open that tab; clicking a different tab should update the URL accordingly.
      
      Reviewers: michelle, vihang
      
      Reviewed By: michelle
      
      JIRA Issues: PC-546, PC-674
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8372
      
      GitOrigin-RevId: d95a967ccdc87c47d7d49b0c483368bf613a8c0e
      a7312be8
    • Zain Asgar's avatar
      Move coverage token to be a secret · cd854b12
      Zain Asgar authored
      Summary: TSIA
      
      Test Plan: jenkins
      
      Reviewers: vihang
      
      Reviewed By: vihang
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8386
      
      GitOrigin-RevId: b6d92a3a83156052baec2f3f1f7de307eb90eb78
      cd854b12
    • Michelle Nguyen's avatar
      PP-2651 Add YAMLs for operator · 52bca213
      Michelle Nguyen authored
      Summary:
      a typical operator deployment has:
      - the operator deployment to launch the operator pod
      - the service account associated with the operator
      - clusterroles/clusterrolebindings that allow the operator to modify resources in the cluster.
      
      the clusterroles I included here are the same clusterrules we require for our updaterjob + our new Vizier CRD.
      
      Test Plan: deployed the operator with these YAMLs
      
      Reviewers: vihang, zasgar
      
      Reviewed By: vihang
      
      JIRA Issues: PP-2651
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8368
      
      GitOrigin-RevId: 9288579fb6d267de35e35d50b4699d6b1f48b1ea
      52bca213
    • Michelle Nguyen's avatar
      PP-2650 Add initial boilerplate for operator · dc17199f
      Michelle Nguyen authored
      Summary:
      Adding initial boilerplate for our Vizier operator, based off of the generated kubebuilder project.
      kubebuilder generated:
      - contents of operator directory
      - k8s/operator/crd
      I heavily modified the contents of the operator directory to remove the additional items we didnt need and made it work with our build system. removed most of the items in the Makefile except for the commands that help generate the k8s crd yaml and generated go files.
      
      Test Plan: deployed the crd, deployed the operator, and deployed an instance of the Vizier custom resource. verified that the operator detected the change and triggered the "Reconcile" function.
      
      Reviewers: zasgar, vihang, #third_party_approvers
      
      Reviewed By: vihang, #third_party_approvers
      
      JIRA Issues: PP-2650
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8367
      
      GitOrigin-RevId: 2c0ae0e45c97309258f083eea5d33acb2fc15578
      dc17199f
    • Zain Asgar's avatar
      PP-2612 - Ignore experimental folder if it does not exist · bfcdf754
      Zain Asgar authored
      Summary: Allow bazel_ci script to work even if experimental directory is missing.
      
      Test Plan: jenkins
      
      Reviewers: vihang
      
      Reviewed By: vihang
      
      JIRA Issues: PP-2612
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8385
      
      GitOrigin-RevId: 20fa1fd2debe44d15c7d9274e020133ce1e778ca
      bfcdf754
    • Vihang Mehta's avatar
      Update some readme pointers · 05a6483e
      Vihang Mehta authored
      Summary: These were broken. :)
      
      Test Plan: N/A
      
      Reviewers: zasgar, michelle
      
      Reviewed By: michelle
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8383
      
      GitOrigin-RevId: 1eb220288b0e25cd2009ba5c138dcf21f3e88cfa
      05a6483e
    • Omid Azizi's avatar
      Fewer calls to now(): Part 3 · e1f0c569
      Omid Azizi authored
      Summary: This completes the series of diffs. There should be one call to now() for all trackers in an iteration.
      
      Test Plan: Existing tests + manual check of flamegraph to see if performance overhead went down (still pending).
      
      Reviewers: #stirling, yzhao
      
      Reviewed By: #stirling, yzhao
      
      Subscribers: yzhao
      
      JIRA Issues: PP-2647
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8345
      
      GitOrigin-RevId: faeceeae994f7ca46374614074432dec73e518b4
      e1f0c569
    • Omid Azizi's avatar
      [Cleanup] include/using statements · 5ec41d55
      Omid Azizi authored
      Summary: Just cleanup
      
      Test Plan: None
      
      Reviewers: #stirling, yzhao
      
      Reviewed By: #stirling, yzhao
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8360
      
      GitOrigin-RevId: 9cec93adda87717d79989c5041df2da577adcd62
      5ec41d55
    • Omid Azizi's avatar
      Postgres tracing: Fix typo in record · 4b282085
      Omid Azizi authored
      Summary: formt -> format
      
      Test Plan: Existing tests
      
      Reviewers: #stirling, yzhao
      
      Reviewed By: #stirling, yzhao
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8364
      
      GitOrigin-RevId: 01246122a3e34e43d264c2d4c66864bcb54ab8ae
      4b282085
    • Vihang Mehta's avatar
      New py pxapi release process · b174f06a
      Vihang Mehta authored
      Summary:
      Cleanup the imports, cleanup the python directory structure.
      We leave the protos nested in the wheel which feels a bit strange but we
      rexport the stuff that's useful from `pxapi` anyway. This also saves us
      from having to rewrite the proto imports.
      
      Upgrade `rules_python` to use new requirements helpers.
      Use `py_wheel` from `rules_python` to buid the wheel.
      Write a small uploader so we can upload wheels from bazel targets.
      
      Note that the uploader doesn't work in interactive mode and expects you to have
      valid auth in `~/.pypirc`
      
      Test Plan: Uploaded wheel to testpypi
      
      Reviewers: zasgar, philkuz, #third_party_approvers
      
      Reviewed By: zasgar, philkuz, #third_party_approvers
      
      JIRA Issues: PP-2598
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8381
      
      GitOrigin-RevId: a6cc9915f8ac65a0b8a453acfa4ecaceb8b6aa80
      b174f06a
    • Zain Asgar's avatar
      PP-2612 - Add Jenkins config for OSS builds · b788b24c
      Zain Asgar authored
      Summary: This adds the jenkins config for gihub repo build. This is basically the same build as our regular repo, but runs only on OSS assets.
      
      Test Plan: tested on branch.
      
      Reviewers: michelle, vihang
      
      Reviewed By: vihang
      
      JIRA Issues: PP-2612
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8377
      
      GitOrigin-RevId: 6d51973ed6e39b4cb56456719e148ebd3ce9666b
      b788b24c
    • Vishal Jain's avatar
      Accept Script from Grafana UI & Execute script. · 3889681c
      Vishal Jain authored
      Summary: Accept Script from Grafana UI and Execute Script.
      
      Test Plan: Manual testing.
      
      Reviewers: nserrino, vihang, nlanam
      
      Reviewed By: vihang, nlanam
      
      Differential Revision: https://phab.corp.pixielabs.ai/D8355
      
      GitOrigin-RevId: 5265e8512c12a12d137f7b3febdc90c565f50310
      3889681c