Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove deprecated code #2751

Closed
6 tasks done
squakez opened this issue Nov 16, 2021 · 4 comments · Fixed by #2932
Closed
6 tasks done

Remove deprecated code #2751

squakez opened this issue Nov 16, 2021 · 4 comments · Fixed by #2932
Assignees
Labels
kind/task General activity

Comments

@squakez
Copy link
Contributor

squakez commented Nov 16, 2021

We have a few deprecated things around the code we may fix in the next releases:

  • Remove older config flags (ie, secret, configmap and property-file flag)
  • Remove DeprecatedConfigMap in master addon trait
  • Remove Timeout in MavenSpec
  • Remove Kit from IntegrationSpec
  • Remove Flow from KameletSpec
  • Remove NewFilteredSharedInformerFactory

@astefanutti @nicolaferraro can you please confirm we should be fine removing all this stuff?

@astefanutti
Copy link
Member

That looks good to me. There is also the container trait configuration parameters related to health checks that are deprecated and that we may want to remove as well.

@astefanutti astefanutti added the kind/task General activity label Nov 16, 2021
@squakez
Copy link
Contributor Author

squakez commented Nov 16, 2021

That looks good to me. There is also the container trait configuration parameters related to health checks that are deprecated and that we may want to remove as well.

I think it was deprecated quite recently. I'd wait for at least one more major upgrade before removing that one.

@astefanutti
Copy link
Member

That looks good to me. There is also the container trait configuration parameters related to health checks that are deprecated and that we may want to remove as well.

I think it was deprecated quite recently. I'd wait for at least one more major upgrade before removing that one.

Right, that sounds good. The others you've listed are older enough.

@phantomjinx
Copy link
Contributor

Looking at e2e tests, property-file also appears deprecated too.

Warn: --configmap has been deprecated. You should use --config configmap:my-configmap instead.
Warn: --secret has been deprecated. You should use --config secret:my-secret instead.
Warn: --property-file has been deprecated. You should use --property file:/path

@squakez squakez self-assigned this Jan 24, 2022
squakez added a commit to squakez/camel-k that referenced this issue Jan 24, 2022
* --property-file
* --configmap
* -- secret

Ref apache#2751
squakez added a commit to squakez/camel-k that referenced this issue Jan 24, 2022
squakez added a commit to squakez/camel-k that referenced this issue Jan 24, 2022
squakez added a commit to squakez/camel-k that referenced this issue Jan 24, 2022
squakez added a commit to squakez/camel-k that referenced this issue Jan 24, 2022
squakez added a commit to squakez/camel-k that referenced this issue Jan 24, 2022
squakez added a commit to squakez/camel-k that referenced this issue Jan 24, 2022
squakez added a commit to squakez/camel-k that referenced this issue Jan 24, 2022
* --property-file
* --configmap
* -- secret

Ref apache#2751
squakez added a commit to squakez/camel-k that referenced this issue Jan 24, 2022
squakez added a commit to squakez/camel-k that referenced this issue Jan 24, 2022
squakez added a commit to squakez/camel-k that referenced this issue Jan 24, 2022
squakez added a commit to squakez/camel-k that referenced this issue Jan 24, 2022
squakez added a commit to squakez/camel-k that referenced this issue Jan 24, 2022
squakez added a commit to squakez/camel-k that referenced this issue Jan 24, 2022
squakez added a commit to squakez/camel-k that referenced this issue Jan 25, 2022
* --property-file
* --configmap
* -- secret

Ref apache#2751
squakez added a commit to squakez/camel-k that referenced this issue Jan 25, 2022
squakez added a commit to squakez/camel-k that referenced this issue Jan 25, 2022
squakez added a commit to squakez/camel-k that referenced this issue Jan 25, 2022
squakez added a commit to squakez/camel-k that referenced this issue Jan 25, 2022
squakez added a commit to squakez/camel-k that referenced this issue Jan 25, 2022
squakez added a commit to squakez/camel-k that referenced this issue Jan 25, 2022
squakez added a commit to squakez/camel-k that referenced this issue Jan 26, 2022
* --property-file
* --configmap
* -- secret

Ref apache#2751
squakez added a commit to squakez/camel-k that referenced this issue Feb 3, 2022
* --property-file
* --configmap
* -- secret

Ref apache#2751
squakez added a commit to squakez/camel-k that referenced this issue Feb 3, 2022
squakez added a commit to squakez/camel-k that referenced this issue Feb 3, 2022
squakez added a commit to squakez/camel-k that referenced this issue Feb 3, 2022
squakez added a commit to squakez/camel-k that referenced this issue Feb 3, 2022
squakez added a commit to squakez/camel-k that referenced this issue Feb 3, 2022
squakez added a commit to squakez/camel-k that referenced this issue Feb 3, 2022
squakez added a commit to squakez/camel-k that referenced this issue Feb 8, 2022
* --property-file
* --configmap
* -- secret

Ref apache#2751
squakez added a commit to squakez/camel-k that referenced this issue Feb 8, 2022
squakez added a commit to squakez/camel-k that referenced this issue Feb 8, 2022
squakez added a commit to squakez/camel-k that referenced this issue Feb 8, 2022
squakez added a commit to squakez/camel-k that referenced this issue Feb 8, 2022
squakez added a commit to squakez/camel-k that referenced this issue Feb 8, 2022
squakez added a commit to squakez/camel-k that referenced this issue Feb 8, 2022
squakez added a commit to squakez/camel-k that referenced this issue Mar 1, 2022
* --property-file
* --configmap
* -- secret

Ref apache#2751
squakez added a commit to squakez/camel-k that referenced this issue Mar 1, 2022
squakez added a commit to squakez/camel-k that referenced this issue Mar 1, 2022
squakez added a commit to squakez/camel-k that referenced this issue Mar 1, 2022
squakez added a commit to squakez/camel-k that referenced this issue Mar 1, 2022
squakez added a commit to squakez/camel-k that referenced this issue Mar 1, 2022
squakez added a commit to squakez/camel-k that referenced this issue Mar 1, 2022
squakez added a commit that referenced this issue Mar 1, 2022
* --property-file
* --configmap
* -- secret

Ref #2751
squakez added a commit that referenced this issue Mar 1, 2022
squakez added a commit that referenced this issue Mar 1, 2022
squakez added a commit that referenced this issue Mar 1, 2022
squakez added a commit that referenced this issue Mar 1, 2022
squakez added a commit that referenced this issue Mar 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/task General activity
Projects
None yet
3 participants