Skip to content
This repository was archived by the owner on Mar 5, 2025. It is now read-only.

Remove discovery and reporting-client from sample & skeleton #981

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions CHANGES
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,11 @@ Platform 3.27
package private configuration attributes. The configuration classes
and their configuration getters and setters must be made public.

* Sample and skeleton servers

We removed discovery and switched from reporting-client to reporting-prometheus
in both the sample and skeleton servers.

* Library Upgrades

- ByteBuddy to 1.15.11 (was 1.12.9)
Expand Down
2 changes: 1 addition & 1 deletion jmx/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@

<dependency>
<groupId>com.proofpoint.platform</groupId>
<artifactId>discovery</artifactId>
<artifactId>node</artifactId>
</dependency>

<dependency>
Expand Down
2 changes: 0 additions & 2 deletions sample-server/etc/config.properties
Original file line number Diff line number Diff line change
@@ -1,3 +1 @@
node.environment=test
testing.discovery.static=true
service-balancer.reporting.uri=https://pulse-int-ingest.lab.ppops.net
7 changes: 1 addition & 6 deletions sample-server/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -79,11 +79,6 @@
<artifactId>bootstrap</artifactId>
</dependency>

<dependency>
<groupId>com.proofpoint.platform</groupId>
<artifactId>discovery</artifactId>
</dependency>

<dependency>
<groupId>com.proofpoint.platform</groupId>
<artifactId>log</artifactId>
Expand Down Expand Up @@ -116,7 +111,7 @@

<dependency>
<groupId>com.proofpoint.platform</groupId>
<artifactId>reporting-client</artifactId>
<artifactId>reporting-prometheus</artifactId>
</dependency>

<dependency>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,6 @@

import com.google.inject.Injector;
import com.proofpoint.audit.AuditLogModule;
import com.proofpoint.discovery.client.DiscoveryModule;
import com.proofpoint.discovery.client.announce.Announcer;
import com.proofpoint.http.server.HttpServerModule;
import com.proofpoint.jaxrs.JaxrsModule;
import com.proofpoint.jmx.JmxHttpModule;
Expand All @@ -27,8 +25,8 @@
import com.proofpoint.log.LogJmxModule;
import com.proofpoint.log.Logger;
import com.proofpoint.node.NodeModule;
import com.proofpoint.reporting.ReportingClientModule;
import com.proofpoint.reporting.ReportingModule;
import com.proofpoint.reporting.ReportingPrometheusModule;
import org.weakref.jmx.guice.MBeanModule;

import static com.proofpoint.bootstrap.Bootstrap.bootstrapApplication;
Expand All @@ -47,7 +45,6 @@ public static void main(String[] args)
Injector injector = bootstrapApplication("sample-server")
.withModules(
new NodeModule(),
new DiscoveryModule(),
new HttpServerModule(),
new JsonModule(),
new JaxrsModule(),
Expand All @@ -57,12 +54,10 @@ public static void main(String[] args)
new LogJmxModule(),
new AuditLogModule(),
new ReportingModule(),
new ReportingClientModule(),
new ReportingPrometheusModule(),
new MainModule()
)
.initialize();

injector.getInstance(Announcer.class).start();
}
catch (Throwable e) {
log.error(e);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@
import com.google.inject.Scopes;

import static com.proofpoint.configuration.ConfigBinder.bindConfig;
import static com.proofpoint.discovery.client.DiscoveryBinder.discoveryBinder;
import static com.proofpoint.jaxrs.JaxrsBinder.jaxrsBinder;
import static com.proofpoint.reporting.HealthBinder.healthBinder;
import static com.proofpoint.reporting.ReportBinder.reportBinder;
Expand All @@ -43,8 +42,6 @@ public void configure(Binder binder)
jaxrsBinder(binder).bind(PersonResource.class).withApplicationPrefix();

bindConfig(binder).bind(StoreConfig.class);

discoveryBinder(binder).bindHttpAnnouncement("person");
}

@Provides
Expand Down
2 changes: 0 additions & 2 deletions skeleton-server/etc/config.properties
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
node.environment=test
testing.discovery.static=true
service-balancer.reporting.uri=https://pulse-int-ingest.lab.ppops.net
http-server.http.enabled=true
http-server.https.enabled=false
7 changes: 1 addition & 6 deletions skeleton-server/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -49,11 +49,6 @@
<artifactId>bootstrap</artifactId>
</dependency>

<dependency>
<groupId>com.proofpoint.platform</groupId>
<artifactId>discovery</artifactId>
</dependency>

<dependency>
<groupId>com.proofpoint.platform</groupId>
<artifactId>log</artifactId>
Expand Down Expand Up @@ -86,7 +81,7 @@

<dependency>
<groupId>com.proofpoint.platform</groupId>
<artifactId>reporting-client</artifactId>
<artifactId>reporting-prometheus</artifactId>
</dependency>

<dependency>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@
import com.google.common.collect.ImmutableMap;
import com.google.inject.Injector;
import com.proofpoint.audit.AuditLogModule;
import com.proofpoint.discovery.client.DiscoveryModule;
import com.proofpoint.discovery.client.announce.Announcer;
import com.proofpoint.http.server.HttpServerModule;
import com.proofpoint.jaxrs.JaxrsModule;
Expand All @@ -28,8 +27,8 @@
import com.proofpoint.log.LogJmxModule;
import com.proofpoint.log.Logger;
import com.proofpoint.node.NodeModule;
import com.proofpoint.reporting.ReportingClientModule;
import com.proofpoint.reporting.ReportingModule;
import com.proofpoint.reporting.ReportingPrometheusModule;
import org.weakref.jmx.guice.MBeanModule;

import static com.proofpoint.bootstrap.Bootstrap.bootstrapApplication;
Expand All @@ -48,7 +47,6 @@ public static void main(String[] args)
Injector injector = bootstrapApplication("skeleton")
.withModules(
new NodeModule(),
new DiscoveryModule(),
new HttpServerModule(),
new JsonModule(),
new JaxrsModule(),
Expand All @@ -58,7 +56,7 @@ public static void main(String[] args)
new LogJmxModule(),
new AuditLogModule(),
new ReportingModule(),
new ReportingClientModule(),
new ReportingPrometheusModule(),
new MainModule()
)
.withApplicationDefaults(ImmutableMap.<String, String>builder()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,6 @@
import com.google.inject.Binder;
import com.google.inject.Module;

import static com.proofpoint.discovery.client.DiscoveryBinder.discoveryBinder;

public class MainModule
implements Module
{
Expand All @@ -28,7 +26,5 @@ public void configure(Binder binder)
{
binder.requireExplicitBindings();
binder.disableCircularProxies();

discoveryBinder(binder).bindHttpAnnouncement("skeleton");
}
}
Loading