Skip to content

feat: add docker image tests #51

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

Merged
merged 19 commits into from
Jul 23, 2024

Conversation

obs-gh-nikhildua
Copy link
Collaborator

@obs-gh-nikhildua obs-gh-nikhildua commented Jul 15, 2024

Description

Ticket: https://observe.atlassian.net/browse/OB-33428

Checklist

  • Created tests which fail without the change (if possible)
  • Extended the README / documentation, if necessary

@obs-gh-nikhildua obs-gh-nikhildua changed the title foundation for docker [DRAFT] feat: add docker image tests Jul 15, 2024
@obs-gh-nikhildua obs-gh-nikhildua changed the title [DRAFT] feat: add docker image tests feat: add docker image tests Jul 23, 2024
@obs-gh-nikhildua obs-gh-nikhildua marked this pull request as ready for review July 23, 2024 16:22
@obs-gh-nikhildua obs-gh-nikhildua merged commit e0db84f into main Jul 23, 2024
8 checks passed
@obs-gh-nikhildua obs-gh-nikhildua deleted the nikhil/add-docker-integration-tests branch July 23, 2024 20:19
obs-gh-alexlew pushed a commit that referenced this pull request Jul 24, 2024
### Description

Ticket: https://observe.atlassian.net/browse/OB-33428

- Adds docker image integration tests 
- Updates to support `cloud-init` for Ubuntu (debian) machine for docker
purposes
- Docker images are also now part of build step, for upload. They are
packaged as `.tar` for each distribution to "install" on the debian
machine
(See https://docs.docker.com/reference/cli/docker/image/save/ &
https://docs.docker.com/reference/cli/docker/image/load/)
- Minor wrapper helper function for common work between windows, linux,
docker

### Checklist
- [x] Created tests which fail without the change (if possible)
- [x] Extended the README / documentation, if necessary
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.

2 participants