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

Add Maven plugin extension API #2380

Merged
merged 3 commits into from
Apr 8, 2020
Merged

Conversation

chanseokoh
Copy link
Member

@chanseokoh chanseokoh commented Apr 2, 2020

The main plugin extension point is through the following method signature. The focus is on the capability to adjust a build plan prepared by Jib plugins.

ContainerBuildPlan extendContainerBuildPlan(
      ContainerBuildPlan buildPlan,
      MavenProject project,
      MavenSession session,
      ExtensionLogger logger)
      throws JibPluginExtensionException;

JibPluginExtensionException is defined to allow extensions to propagate any problems during extension execution.

ExtensionLogger is to allow extensions to log to the console. Note it is not dependent on any of the logging facilities in jib-core.

Copy link
Member

@loosebazooka loosebazooka left a comment

Choose a reason for hiding this comment

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

I like it, keeps it pretty simple for the extension developers. Just some comments.

@chanseokoh
Copy link
Member Author

Merging this to put up the next PR. If anyone finds an issue, we'll have plenty of time to fix it while things are being shaped up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants