(ranger) branch ranger-2.8 updated: RANGER-3491: fix error logs while bootstraping service-defs - TAG service-def not found (#125)

[madhan]

This is an automated email from the ASF dual-hosted git repository.

madhan pushed a commit to branch ranger-2.8
in repository https://gitbox.apache.org/repos/asf/ranger.git


The following commit(s) were added to refs/heads/ranger-2.8 by this push:
     new 68c18f305 RANGER-3491: fix error logs while bootstraping service-defs 
- TAG service-def not found (#125)
68c18f305 is described below

commit 68c18f305401c95539602b948b66d4e8c07a8e41
Author: hervor <h...@tencent.com>
AuthorDate: Sun Jan 11 19:12:57 2026 -0800

    RANGER-3491: fix error logs while bootstraping service-defs - TAG 
service-def not found (#125)
    
    Signed-off-by: Madhan Neethiraj <mad...@apache.org>
    (cherry picked from commit 2921ffe5ddfb513d3fe8145e87f80dc4429c17cc)
---
 .../apache/ranger/plugin/store/EmbeddedServiceDefsUtil.java  | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git 
a/agents-common/src/main/java/org/apache/ranger/plugin/store/EmbeddedServiceDefsUtil.java
 
b/agents-common/src/main/java/org/apache/ranger/plugin/store/EmbeddedServiceDefsUtil.java
index 9accb45f9..b15f95059 100755
--- 
a/agents-common/src/main/java/org/apache/ranger/plugin/store/EmbeddedServiceDefsUtil.java
+++ 
b/agents-common/src/main/java/org/apache/ranger/plugin/store/EmbeddedServiceDefsUtil.java
@@ -142,10 +142,10 @@ public void init(ServiceStore store) {
                        createEmbeddedServiceDefs = 
config.getBoolean(PROPERTY_CREATE_EMBEDDED_SERVICE_DEFS, true);
 
                        supportedServiceDefs =getSupportedServiceDef();
-                       /*
-                        * Maintaining the following service-def create-order 
is critical for the
-                        * the legacy service-defs (HDFS/HBase/Hive/Knox/Storm) 
to be assigned IDs
-                        * that were used in earlier version (0.4) */
+
+                       /* TAG service-def must be created before any other 
service-def */
+                       tagServiceDef = getOrCreateServiceDef(store, 
EMBEDDED_SERVICEDEF_TAG_NAME);
+
                        hdfsServiceDef  = getOrCreateServiceDef(store, 
EMBEDDED_SERVICEDEF_HDFS_NAME);
                        hBaseServiceDef = getOrCreateServiceDef(store, 
EMBEDDED_SERVICEDEF_HBASE_NAME);
                        hiveServiceDef  = getOrCreateServiceDef(store, 
EMBEDDED_SERVICEDEF_HIVE_NAME);
@@ -160,7 +160,6 @@ public void init(ServiceStore store) {
                        nifiRegistryServiceDef = getOrCreateServiceDef(store, 
EMBEDDED_SERVICEDEF_NIFI_REGISTRY_NAME);
                        atlasServiceDef = getOrCreateServiceDef(store, 
EMBEDDED_SERVICEDEF_ATLAS_NAME);
 
-                       tagServiceDef = getOrCreateServiceDef(store, 
EMBEDDED_SERVICEDEF_TAG_NAME);
                        wasbServiceDef = getOrCreateServiceDef(store, 
EMBEDDED_SERVICEDEF_WASB_NAME);
                        sqoopServiceDef = getOrCreateServiceDef(store, 
EMBEDDED_SERVICEDEF_SQOOP_NAME);
                        kylinServiceDef = getOrCreateServiceDef(store, 
EMBEDDED_SERVICEDEF_KYLIN_NAME);
@@ -171,9 +170,6 @@ public void init(ServiceStore store) {
                        ozoneServiceDef = getOrCreateServiceDef(store, 
EMBEDDED_SERVICEDEF_OZONE_NAME);
                        kuduServiceDef = getOrCreateServiceDef(store, 
EMBEDDED_SERVICEDEF_KUDU_NAME);
                        nestedStructureServiveDef = 
getOrCreateServiceDef(store, EMBEDDED_SERVICEDEF_NESTEDSTRUCTURE_NAME);
-
-                       // Ensure that tag service def is updated with access 
types of all service defs
-                       store.updateTagServiceDefForAccessTypes();
                } catch(Throwable excp) {
                        LOG.error("EmbeddedServiceDefsUtil.init(): failed", 
excp);
                }