Skip to content

Conversation

@prabhjyotsingh
Copy link
Contributor

@prabhjyotsingh prabhjyotsingh commented Nov 14, 2025

What changes were proposed in this pull request?

Add a way to encrypted configurations.json file in the ambari-agent's cache

How was this patch tested?

before enabling encryption:

cluster_cache]# cat configurations.json 
{
  "2": {
    "configurations": {
      "admin-logback": {
        "content": "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n<!--\n  Licensed to the Apache Software Foundation (ASF) under one or more\n  contributor license agreements.  See the NOTICE file distributed with\n  this work for additional information regarding copyright ownership.\n  The ASF licenses this file to You under the Apache License, Version 2.0\n  (the \"License\"); 

after enabling encryption:

cluster_cache]# cat configurations.json 
 b'${enc=aes128_hex, value=35343134333930393632613735326136333432373638353461333162313635613a3a373......

@JiaLiangC
Copy link
Contributor

@prabhjyotsingh plz add some unit test for your change

@prabhjyotsingh
Copy link
Contributor Author

@JiaLiangC sure I've added a few.

@prabhjyotsingh
Copy link
Contributor Author

@JiaLiangC can you help me review this PR?

@prabhjyotsingh
Copy link
Contributor Author

@JiaLiangC can you help me review this PR?

After spending more time on this, I realized there’s an internal implementation of ambari_pbkdf2, so I decided to use that instead.

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.

2 participants