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

Removed deprecated GridFS column from repository #419

Merged
merged 5 commits into from Jan 19, 2017

Conversation

kaizimmerm
Copy link
Member

@kaizimmerm kaizimmerm commented Jan 16, 2017

Refactored repository ArtifactManagement by means of removing the grids column. This is something that was used in the past when MongoDb GridFS artifact binary repository was hardcoded into the system.

I refactored as well the wild mix of identifying the DbBArtifact by artfact ID or SHA1 (especially DMF got this mixed). Its now always SHA1. That is the way to identify the artifact.

Reviewers: @Jonkno @melanieretter

Signed-off-by: kaizimmerm <kai.zimmermann@bosch-si.com>
Signed-off-by: kaizimmerm <kai.zimmermann@bosch-si.com>
Signed-off-by: kaizimmerm <kai.zimmermann@bosch-si.com>
@@ -81,6 +86,42 @@ public void controllerAddsActionStatus() {
}

@Test
@Description("Verifies that assignement verfication works based on SHA1 hash. By design it is not important which artifact "
Copy link
Contributor

Choose a reason for hiding this comment

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

assignment verification

Signed-off-by: kaizimmerm <kai.zimmermann@bosch-si.com>
@kaizimmerm kaizimmerm merged commit 9d0a064 into eclipse:master Jan 19, 2017
@kaizimmerm kaizimmerm deleted the fix_gridfs_deprecated branch January 19, 2017 07:43
@kaizimmerm kaizimmerm modified the milestone: 0.2.0M3 Apr 3, 2017
AmmarBikic pushed a commit to bosch-io/hawkbit that referenced this pull request Oct 2, 2020
* Merged artifact sha1 hash and gridfs column.

Signed-off-by: kaizimmerm <kai.zimmermann@bosch-si.com>

* Renamed exception to get rid of old GridFS name.

Signed-off-by: kaizimmerm <kai.zimmermann@bosch-si.com>

* Added test description.

Signed-off-by: kaizimmerm <kai.zimmermann@bosch-si.com>

* Fix typo.

Signed-off-by: kaizimmerm <kai.zimmermann@bosch-si.com>
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.

None yet

3 participants