Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ensure main is accurately tracking release changes #1949

Merged
merged 129 commits into from
Sep 6, 2023
Merged

Ensure main is accurately tracking release changes #1949

merged 129 commits into from
Sep 6, 2023

Conversation

iameskild
Copy link
Member

@iameskild iameskild commented Aug 28, 2023

Reference Issues or PRs

Closes #1912
Closes #1885
Closes #1863

What does this implement/fix?

Put a x in the boxes that apply

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds a feature)
  • Breaking change (fix or feature that would cause existing features not to work as expected)
  • Documentation Update
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no API changes)
  • Build related changes
  • Other (please describe): Repo clean up

Testing

  • Did you test the pull request locally?
  • Did you add new tests?

Any other comments?

iameskild and others added 30 commits November 21, 2022 15:03
* _ --> -

* Fix CLI issues
Co-authored-by: Pavithra Eswaramoorthy <pavithraes@outlook.com>
Co-authored-by: Nebari-sensei <116370392+nebari-sensei@users.noreply.github.com>
Co-authored-by: Nebari-sensei <116370392+nebari-sensei@users.noreply.github.com>
Co-authored-by: eskild <42120229+iameskild@users.noreply.github.com>
Co-authored-by: Nebari-sensei <116370392+nebari-sensei@users.noreply.github.com>
Co-authored-by: eskild <42120229+iameskild@users.noreply.github.com>
…po (#1487)

Co-authored-by: iameskild <eskild@doublee.io>
Co-authored-by: Nebari-sensei <116370392+nebari-sensei@users.noreply.github.com>
Co-authored-by: eskild <42120229+iameskild@users.noreply.github.com>
Co-authored-by: Tania Allard <taniar.allard@gmail.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
…1621)

Co-authored-by: iameskild <eskild@doublee.io>
Co-authored-by: iameskild <eskild@doublee.io>
Co-authored-by: iameskild <eskild@doublee.io>
Co-authored-by: iameskild <eskild@doublee.io>
Co-authored-by: iameskild <eskild.eriksen122@gmail.com>
Co-authored-by: Christopher Ostrouchov <chris.ostrouchov@gmail.com>
@iameskild iameskild added needs: review 👀 This PR is complete and ready for reviewing type: release 🏷 Items related to Nebari releases area:developer-experience 👩🏻‍💻 labels Aug 28, 2023
@iameskild iameskild requested a review from pavithraes August 28, 2023 16:43
@pavithraes
Copy link
Member

@iameskild - Several tests are failing, I think we'll need to take a closer look?

@iameskild
Copy link
Member Author

@pavithraes I believe the kubernetes test is going to fail regardless because these changes don't account for the JupyterHub upgrade and the deploy uses NEBARI_GH_BRANCH=main which is using the upgrade JupyterHub python app. Checking the workflow logs, we see the JupyterHub API health check fail at the end of the deployment which is expected.

The other CI should all be passing.

@pavithraes
Copy link
Member

pavithraes commented Sep 6, 2023

Thanks, @iameskild!

While I didn't review all the files, I cross-checked the commits in this branch :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:developer-experience 👩🏻‍💻 needs: review 👀 This PR is complete and ready for reviewing type: release 🏷 Items related to Nebari releases
Projects
Development

Successfully merging this pull request may close these issues.

Resolve the divergence of main and develop branches [RELEASE] 2023.7.2 hot-fix