apache / apache/camel-k

Review integration digest computation and projection to the integration container as env var

Open
#5,192 15 comments 0 reactions 0 assignees View on GitHub
kind/bug
Dominant language
Go
Stars
927
Forks
387
Avg merge
15h 5m
Merged PRs (30d)
36

Description

### What happened?

Since an integration digest ends up begin [projected as a container env var](https://github.com/apache/camel-k/blob/9c058daa180106fb5b770558f9b553b7bfb9d2ef/pkg/trait/container.go#L181), [including the traits when computing the digest](https://github.com/apache/camel-k/blob/9c058daa180106fb5b770558f9b553b7bfb9d2ef/pkg/util/digest/digest.go#L111) has a number of side effects, as example by enabling/disabling the creation of a route or service, would lead to the container to be restarted which is not expected.

### Steps to reproduce

_No response_

### Relevant log output

_No response_

### Camel K version

_No response_

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with pkg/trait/container.go at the projection of the integration digest as a container environment variable, then read pkg/util/digest/digest.go around digest computation and trait inclusion. Review how trait changes affect the digest and determine a behavior that avoids unexpected container restarts; done means the digest projection and computation no longer produce those side effects.

Written by the indexing model from the issue text.

Assessment

Tech stack
go, kubernetes
Domain
infrastructure
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.