Utkarsh Srivastava

Results 29 comments of Utkarsh Srivastava

I think we need to swap `test` and `test-postgres` recipes in Makefile or should change `run_npm_test_on_test_container.sh` to spin up postgres DB instead of Mongo. WDYT @guymguym?

**Streamlining discussion (TLDR of above comments)** for reviewers _All of the "resolved" comments have been addressed_ There are 2 items still needs to be discussed in this PR: 1. `has_bucket_permission`...

After discussions over the items mentioned in the previous comment: 1. `has_bucket_permission` rigorous permission checks if can be avoided or reduced must be reduced. This may involve removing redundant permission...

After discussion with @jackyalbo, `load_bucket` and `find_bucket` in the bucket_server and object_server no longer rely on `has_bucket_permission` which essentially means, that if an RPC is performed and the user is...

Rebased the branch ✔️. Thank you @jackyalbo for reviewing this PR 😄! Merging the PR...

I think I can squash the commits after addressing the comments on this PR.

@romayalon, > @utkarsh-pro WOW! what an awesome PR, very clean and sharp :) I have some concerns I want to put here for discussion: > > 1. **Admission controller** -...

The CI tests are failing with error `Exiting due to GUEST_MISSING_CONNTRACK: Sorry, Kubernetes 1.20.2 requires conntrack to be installed in root's path`. Do we need to update our workflow?

### NooBaa Team Discussion Summary 8th Sept 1. Minimise any type of access control (namely, `allowedNamespaces` annotations) in this iteration (i.e iteration 1). - Rationale: May lead to regressions and...

@hershd23 right now, both "import" and "apply" happen in the same command. Before we understand how it is done through `mesheryctl` let's clarify what "importing" and "applying" means in the...