Skip to content

upkeep/docker-image-version-and-py-dependencies#36

Closed
bartekus wants to merge 2 commits intobcgov:mainfrom
bartekus:upkeep/docker-image-version-and-py-dependencies
Closed

upkeep/docker-image-version-and-py-dependencies#36
bartekus wants to merge 2 commits intobcgov:mainfrom
bartekus:upkeep/docker-image-version-and-py-dependencies

Conversation

@bartekus
Copy link
Copy Markdown

  • Add & update deps, adjust integration test to handle python 3.7 to 3.6 downgrade

Signed-off-by: bartekus bartekus@gmail.com

@bartekus
Copy link
Copy Markdown
Author

Just wanted to escalate the version update as we are currently experiencing issues with revocation ranging from random PUT failures to not accepting revocation entries which have empty spaces in them (a schema with human readable name for example)

@swcurran swcurran requested a review from WadeBarnes June 28, 2022 23:22
@swcurran
Copy link
Copy Markdown
Contributor

@WadeBarnes -- please review. Thanks!

Thanks, @bartekus !

@bartekus
Copy link
Copy Markdown
Author

bartekus added 2 commits June 28, 2022 19:31
…6 downgrade

Signed-off-by: bartekus <bartekus@gmail.com>
Signed-off-by: Bartlomiej P Kus <bartekus@gmail.com>
Signed-off-by: bartekus <bartekus@gmail.com>
@bartekus bartekus force-pushed the upkeep/docker-image-version-and-py-dependencies branch from d88b105 to c7fc10b Compare June 29, 2022 01:31
Copy link
Copy Markdown
Member

@WadeBarnes WadeBarnes left a comment

Choose a reason for hiding this comment

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

We don't want to downgrade to Python 3.6 since it's EOL and no longer supported. The better option would be to update the base image to the next-2 image or even update the base image code and create a new release of the base image(s); https://github.com/bcgov/von-image

@bartekus
Copy link
Copy Markdown
Author

Is there anything I can do regarding the new base image being created then updating this to that revision?

@WadeBarnes
Copy link
Copy Markdown
Member

@bartekus, If you have time you could see if the next-2 image works for the purpose, and/or look into updating the next set of images to suit.

@WadeBarnes
Copy link
Copy Markdown
Member

Closing PR. The project was migrated to using a standard Python image and dependency updates have been performed.

@WadeBarnes WadeBarnes closed this Aug 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants