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

PXP-6245 PXP-9633 Poetry + Python 3.9 #387

Merged
merged 10 commits into from
Feb 2, 2023
Merged

PXP-6245 PXP-9633 Poetry + Python 3.9 #387

merged 10 commits into from
Feb 2, 2023

Conversation

paulineribeyre
Copy link
Contributor

@paulineribeyre paulineribeyre commented Jan 27, 2023

Jira Ticket: PXP-6245
Jira Ticket: PXP-9633

New Features

Breaking Changes

Bug Fixes

Improvements

  • The unit tests have been refactored so that some of the duplicated setup is now shared

Dependency updates

  • Python 3.9
  • indexclient to 2.1.1
  • Use Poetry for dependency management. Update and clean up dependencies. Set more flexible dependency versions.

Deployment changes

  • Sheepdog does not rely on configuration files in the cloud-automation repository anymore. The Sheepdog configuration files are in the Sheepdog repository.

@paulineribeyre paulineribeyre marked this pull request as ready for review February 1, 2023 23:11
@@ -1,31 +1,15 @@
dist: trusty
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we run these in GH actions? We can tackle that in a separate PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah there are already more changes than i would have liked in this PR 😅

# To run:
# - Create and fill out `creds.json`:
# {
# "fence_host": "",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't remember if I hacked the config in helm, but can we also read these from environment variables?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it doesn't look like you've updated sheepdog to read them from env vars, i don't see any code doing it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants