Vinicius D. Cerutti

Results 345 comments of Vinicius D. Cerutti

As an extra comment here, I think we should get the current default context from kubectl (we are doing this somewhere already), and apply as the default value. Though, I...

> @viniciusdc does it make sense to edit the issue body and title to reflect the fact that we don't need an extra flag as we already have one implemented?...

Thanks for raising the issue, @tylergraff . Indeed, this has come to our attention since last week. We are not entirely sure why this started happening now. My main guess...

Re-opening as the issue still persists, though the work done in the PR above, helped clear some related matters.

>I next wanted to investigate how the other tests were passing, and looking at tests/tests_unit/conftest.py I saw that there were pytest fixtures for other assets such as instances, kubernetes versions,...

Tests are failing, @aktech may I move this to draft?

The end result looks like this: https://github.com/nebari-dev/nebari/actions/runs/11192761281/attempts/1#summary-31117309003

@joneszc, it would be great to compare your script with the idea above, as I think some images are missing, such as the `/registry` ones in there. Even if this...

@marcelovilla I will add more details to this as an organization bord for future issues/tasks. The general discussion will be held as part of an RFD with the same name.

To fix this issue, look for the [validation function](https://github.com/nebari-dev/nebari/blob/develop/src/_nebari/utils.py#L343) and replace the `required_variables` to `missing_variables` in here https://github.com/nebari-dev/nebari/blob/9baab7e29e2ec9b172ef439ddc66a5e51a380066/src/_nebari/utils.py#L352-L355