Add ARMV6L to architectures that support database encryption

Narayanan Iyerrequested to merge
nars1/YDB:armv6l_encrypt into master

Background

  • The recently added r136/ydb575 subtest failed on ARMV6L with the following diff.

    --- ydb575/ydb575.diff ---
9c9
< %YDB-E-CRYPTINIT, Could not initialize encryption library while opening encrypted file r136_0/ydb575/mumps.dat. Environment variable ydb_passwd/gtm_passwd not set
---
> %YDB-E-GVUNDEF, Global variable undefined: ^x

  • The test creates an encrypted database and then does a get of ^x. It expects to see a CRYPTINIT error but in the actual test, we get a GVUNDEF error.

  • This is because the database was created with encryption turned OFF even though the test explicitly enables encryption.

Issue

  • While adding support for YottaDB on ARMV6L, 0023d7a1 forgot to add it to the list of architectures that support ENCRYPTION in sr_port/gdeinit.m. This meant that even if one runs a change -segment DEFAULT -encryption inside GDE, a show -segment would still show as ENCR= OFF. Not ENCR= ON.

  • This meant there is no way to create encrypted databases on ARMV6L.

Fix

  • sr_port/gdeinit.m is fixed in this commit to recognize armv6l as an architecture that supports database encryption.

  • While at this, armv7l is also added to the same list since it should also support encryption just like armv6l does.

Merge request reports