Skip to content

[Atlassian Confluence] Add agentless support - #18516

Merged
moxarth-rathod merged 4 commits into
elastic:mainfrom
moxarth-rathod:confluence-agentless
Jun 10, 2026
Merged

[Atlassian Confluence] Add agentless support#18516
moxarth-rathod merged 4 commits into
elastic:mainfrom
moxarth-rathod:confluence-agentless

Conversation

@moxarth-rathod

@moxarth-rathod moxarth-rathod commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

Proposed commit message

atlassian_confluence: add agentless support and update kibana version constraint

Adds support for agentless and updates the Kibana version constraint to avoid a known issue 
where selecting agentless deployment mode hides configuration fields and blocks 
deployment if inputs default to `enabled: false`.

Ref: https://github.com/elastic/kibana/pull/265106

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

How to test this PR locally

  • Clone integrations repo.
  • Install the elastic package locally.
  • Start the elastic stack using the elastic package.
  • Move to integrations/packages/atlassian_confluence directory.
  • Run the following command to run tests.

elastic-package test -v

Related issues

@moxarth-rathod moxarth-rathod self-assigned this Apr 20, 2026
@moxarth-rathod
moxarth-rathod requested a review from a team as a code owner April 20, 2026 05:44
@moxarth-rathod moxarth-rathod added enhancement New feature or request Integration:atlassian_confluence Atlassian Confluence Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] Team:SDE-Crest Crest developers on the Security Integrations team [elastic/sit-crest-contractors] labels Apr 20, 2026
@elasticmachine

Copy link
Copy Markdown

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

@moxarth-rathod moxarth-rathod changed the title [Atlassian Confluence] Add agentless deployment support Apr 20, 2026
@github-actions

github-actions Bot commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

✅ Vale Linting Results

No issues found on modified lines!


The Vale linter checks documentation changes against the Elastic Docs style guide.

To use Vale locally or report issues, refer to Elastic style guide for Vale.

Comment thread packages/atlassian_confluence/changelog.yml Outdated
Comment on lines 52 to 54
template_path: httpjson.yml.hbs
vars:
- name: api_url

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Medium audit/manifest.yml:52

Removing enabled: false from the httpjson input causes it to default to enabled alongside the logfile input. Since api_url is required with no default, users who only want logfile collection will have an enabled httpjson input lacking required configuration, causing agent errors. The sibling Atlassian integrations (atlassian_jira, atlassian_bitbucket) retain enabled: false on their httpjson streams, suggesting this removal is inconsistent with the established pattern. Consider restoring enabled: false to keep httpjson opt-in.

   - input: httpjson
     title: Confluence audit logs via Confluence audit API
     description: Collect Confluence audit logs via Confluence audit API
+    enabled: false
     template_path: httpjson.yml.hbs
🤖 Copy this AI Prompt to have your agent fix this:
In file packages/atlassian_confluence/data_stream/audit/manifest.yml around lines 52-54:

Removing `enabled: false` from the `httpjson` input causes it to default to enabled alongside the `logfile` input. Since `api_url` is required with no default, users who only want logfile collection will have an enabled httpjson input lacking required configuration, causing agent errors. The sibling Atlassian integrations (`atlassian_jira`, `atlassian_bitbucket`) retain `enabled: false` on their httpjson streams, suggesting this removal is inconsistent with the established pattern. Consider restoring `enabled: false` to keep httpjson opt-in.
Comment thread packages/atlassian_confluence/changelog.yml Outdated
@elastic-vault-github-plugin-prod

elastic-vault-github-plugin-prod Bot commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@andrewkroh andrewkroh added the documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. label Apr 20, 2026
Co-authored-by: Dan Kortschak <dan.kortschak@elastic.co>

@kcreddy kcreddy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Waiting on clarification for path forward: #18141 (comment) and elastic/kibana#265106 (comment)

@moxarth-rathod
moxarth-rathod requested a review from kcreddy May 6, 2026 08:54
Comment thread packages/atlassian_confluence/manifest.yml
@moxarth-rathod
moxarth-rathod marked this pull request as draft May 7, 2026 05:24
@moxarth-rathod
moxarth-rathod marked this pull request as ready for review June 1, 2026 11:00
@moxarth-rathod
moxarth-rathod requested a review from a team as a code owner June 1, 2026 11:00
@moxarth-rathod
moxarth-rathod requested a review from efd6 June 1, 2026 11:00
@efd6

efd6 commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

/test

description: Collect Confluence audit logs via Confluence audit API
enabled: false
template_path: httpjson.yml.hbs
enabled: false

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Revert this placement to the original location to avoid the diff noise.

@kcreddy kcreddy Jun 3, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@moxarth-rathod this is still not done - it shows up in the diff.

Comment thread packages/atlassian_confluence/manifest.yml
@moxarth-rathod
moxarth-rathod requested a review from efd6 June 3, 2026 06:39
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

✅ All changelog entries have the correct PR link.

@kcreddy kcreddy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, thanks!

@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @moxarth-rathod

@moxarth-rathod
moxarth-rathod merged commit 5723ae1 into elastic:main Jun 10, 2026
11 checks passed
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Package atlassian_confluence - 1.31.0 containing this change is available at https://epr.elastic.co/package/atlassian_confluence/1.31.0/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. enhancement New feature or request Integration:atlassian_confluence Atlassian Confluence Team:SDE-Crest Crest developers on the Security Integrations team [elastic/sit-crest-contractors] Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

5 participants