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

feat: Update latest images tags to 3.0.0 #230

Merged
merged 4 commits into from
Jun 5, 2023

Conversation

jim-wang-intel
Copy link
Contributor

Update docker-compose examples to use image tags 3.0.0 instead of latest

Closes: #228

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-examples/blob/main/.github/Contributing.md

PR Checklist

Please check if your PR fulfills the following requirements:

  • I have added unit tests for the new feature or bug fix (if not, why?)
  • I have fully tested (add details below) this the new feature or bug fix (if not, why?)

Testing Instructions

  • git clone this pr
  • follow the instructions from README.md

  Update docker-compose examples to use image tags 3.0.0 instead of latest

  Closes: edgexfoundry#228

Signed-off-by: Jim Wang <[email protected]>
@jim-wang-intel jim-wang-intel self-assigned this Jun 2, 2023
Copy link
Collaborator

@bnevis-i bnevis-i left a comment

Choose a reason for hiding this comment

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

The docker-compose.original is there intentionally in order to show the difference between the original and modified docker-composes

@jim-wang-intel
Copy link
Contributor Author

jim-wang-intel commented Jun 2, 2023

The docker-compose.original is there intentionally in order to show the difference between the original and modified docker-composes

ok, i can revert those back but then you have to keep track of version from this to the next release?
IMO, it is too much burden for developer

So for example, the previous one would be renamed to something like docker-compose.2.x or something like that

@bnevis-i bnevis-i dismissed their stale review June 2, 2023 18:48

Dismissing review... Lenny to take responsiblity.

@jim-wang-intel jim-wang-intel requested a review from bnevis-i June 2, 2023 18:50
bnevis-i
bnevis-i previously approved these changes Jun 2, 2023
Copy link
Member

@lenny-goodell lenny-goodell left a comment

Choose a reason for hiding this comment

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

LGTM, need 2nd review from Camera team since changing their example app
@presatish , @ajcasagrande , please review change from docker-compose to docker compose in Camera Mgmt app.

Copy link

@ajcasagrande ajcasagrande left a comment

Choose a reason for hiding this comment

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

Camera management changes LGTM

@lenny-goodell lenny-goodell merged commit d2ab50b into edgexfoundry:main Jun 5, 2023
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.

Clean-up use of latest tag for images in examples. Should be current release 3.0.0
4 participants