Add Multi-arch support to OrientDB

[odidev]

This PR is raised after an agreed discussion in the ticket orientechnologies/orientdb-docker#48
and the merged PR at orientechnologies/orientdb-docker#51

Please have a look and provide your guidance.

Regards,

[tianon]

cc @luigidellaquila

I'm looking over these, and I'm really not seeing a very compelling reason for these to be completely separate Dockerfiles -- can you elaborate on the reasoning behind that choice?

See https://github.com/docker-library/official-images#multiple-architectures for more information on our multi-architecture stance.

[tianon]

To clarify a bit, this is going to cause the same tag to be drastically different on different architectures, which no obvious indication of that fact for users. If there's a desire to support both ibmjava-based and alpine-based containers concurrently, I think it makes a lot more sense to make it obvious to users that's happening.

[odidev]

Hi @tianon

The changes are done w.r.t, to work done in ticket orientechnologies/orientdb-docker#48

However, it would be great if you can provide us with some solution to this problem, as I am aware of the fact that you are the expert in this domain 😃

Regards,

[lag-linaro]

@tianon although your points seem to be valid and reasonable, apart from the fact that it was this PR which bought the issue to your attention, I'm not sure I see the connection between them and @odidev's submission.

IMHO @odidev has kept loyal to the current project workflow. I believe it would be more helpful for you to raise a new issue against the parent project in order to discuss them with the associated maintainers instead of discussing here.

This PR LGTM.

[tianon]

It's related to this PR because this is the PR which introduced multi-arch to this official image, and the method by which it has been implemented is not something we're willing to accept, regardless of how long it has been accepted in that repository (until now, it has never been part of the bits we review).

As such, I'm closing this -- we're happy to discuss and brainstorm further to help implement this in an acceptable way, but it will require upstream cooperation (@luigidellaquila) and changes to that structure in the repository.

[lag-linaro]

The patch looked as though multi-arch was present already. I saw X86 and PPC.

However, I do understand your reasoning.

Thanks for your feedback. I'll turn my hand to it tomorrow.