Skip to content

Docker: bump package-spec version to 3.2.2#11196

Merged
zmoog merged 7 commits intoelastic:mainfrom
zmoog:zmoog/docker-package-spec-v3
Sep 24, 2024
Merged

Docker: bump package-spec version to 3.2.2#11196
zmoog merged 7 commits intoelastic:mainfrom
zmoog:zmoog/docker-package-spec-v3

Conversation

@zmoog
Copy link
Copy Markdown
Contributor

@zmoog zmoog commented Sep 20, 2024

Proposed commit message

Bump package-spec version to 3.2.2.

Reasons:

  • Align with best practices and requirements in package-spec v3.
  • Serverless requires a package-spec version of 3.0.x.

Notes:

  • As of today, to test the Docker integration on non-serverless stack version 9.0, you must build the package with a conditions.kibana.version: >= 8.8.0.
  • Future elastic-package version Disable version check in 9.0 snapshots elastic-package#2116 will allow test the integration on stack version 9.0.0-SNAPSHOT without changing conditions.kibana.version.

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.

Related issues

@zmoog
Copy link
Copy Markdown
Contributor Author

zmoog commented Sep 20, 2024

I'm not 100% sure about object_type for the docker.memory.stats.* fields.

fields:
- name: stats.*
type: object
object_type: float
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.

I think the correct would be long or double. But looking at the stats, I would pick long

@andrewkroh andrewkroh added enhancement New feature or request Integration:docker Docker labels Sep 20, 2024
@zmoog zmoog self-assigned this Sep 20, 2024
The package-spec >= 3.2 does not work on 9.0 and Serverless
We picked package-spec 3.0.4 instead of 3.2.2. As of today, kibana
requires spec 3.0.x to run on Serverless.
@zmoog zmoog marked this pull request as ready for review September 20, 2024 13:54
@zmoog zmoog requested a review from a team as a code owner September 20, 2024 13:54
@zmoog zmoog changed the title Docker: bump package-spec version to 3.2.2 Docker: bump package-spec version to 3.0.4 Sep 20, 2024
@andrewkroh andrewkroh added the Team:Cloudnative-Monitoring Cloud Native Monitoring team [elastic/obs-cloudnative-monitoring] label Sep 20, 2024
@gizas
Copy link
Copy Markdown
Contributor

gizas commented Sep 23, 2024

LGTM, package build is successful without any errors

@zmoog
Copy link
Copy Markdown
Contributor Author

zmoog commented Sep 23, 2024

@gizas @MichaelKatsoulis I'm testing this one more time with package-spec 3.2.2; maybe I did something wrong last time. Stay tuned!

@zmoog
Copy link
Copy Markdown
Contributor Author

zmoog commented Sep 23, 2024

Tested on stack version 9.0.0-SNAPSHOT and package-spec 3.2.2:

$ docker images | grep 9.0.0-SNAPSHOT

docker.elastic.co/elastic-agent/elastic-agent-wolfi      9.0.0-SNAPSHOT   6a86b26fe188   3 days ago      1.58GB
docker.elastic.co/elasticsearch/elasticsearch            9.0.0-SNAPSHOT   535cb5782b83   3 days ago      1.38GB
docker.elastic.co/kibana/kibana                          9.0.0-SNAPSHOT   277a97ad4979   4 days ago      1.93GB

And I can successfully see and install the Docker integration:

CleanShot.2024-09-23.at.17.24.11.mp4

The latest test run shows it works on 3.2.2 too, I probably made some
mistakes when I tested it before.
@zmoog zmoog changed the title Docker: bump package-spec version to 3.0.4 Docker: bump package-spec version to 3.2.2 Sep 23, 2024
@zmoog
Copy link
Copy Markdown
Contributor Author

zmoog commented Sep 23, 2024

@gizas @MichaelKatsoulis, could you take a final look when you have time?

@elasticmachine
Copy link
Copy Markdown

💚 Build Succeeded

History

cc @zmoog

@elastic-sonarqube
Copy link
Copy Markdown

@MichaelKatsoulis
Copy link
Copy Markdown
Contributor

@gizas @MichaelKatsoulis, could you take a final look when you have time?

LGTM!

@gizas
Copy link
Copy Markdown
Contributor

gizas commented Sep 24, 2024

all good @zmoog from my side!

@zmoog zmoog merged commit 9f186c5 into elastic:main Sep 24, 2024
@zmoog zmoog deleted the zmoog/docker-package-spec-v3 branch September 24, 2024 09:58
@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

Package docker - 2.11.0 containing this change is available at https://epr.elastic.co/search?package=docker

harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 4, 2025
Bump package-spec version to 3.2.2.

Reasons:

- Align with best practices and requirements in package-spec v3.
- Serverless requires a package-spec version of 3.0.x.

Notes: 

- As of today, to test the Docker integration on non-serverless stack version 9.0, you must build the package with a `conditions.kibana.version: >= 8.8.0`.
- Future elastic-package version elastic/elastic-package#2116 will allow test the integration on stack version `9.0.0-SNAPSHOT` without changing `conditions.kibana.version`.
harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 5, 2025
Bump package-spec version to 3.2.2.

Reasons:

- Align with best practices and requirements in package-spec v3.
- Serverless requires a package-spec version of 3.0.x.

Notes: 

- As of today, to test the Docker integration on non-serverless stack version 9.0, you must build the package with a `conditions.kibana.version: >= 8.8.0`.
- Future elastic-package version elastic/elastic-package#2116 will allow test the integration on stack version `9.0.0-SNAPSHOT` without changing `conditions.kibana.version`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Integration:docker Docker Team:Cloudnative-Monitoring Cloud Native Monitoring team [elastic/obs-cloudnative-monitoring]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants