Skip to content

Commit 513a728

Browse files
authored
Merge pull request #771 from snyk/feat/rename_workload_events_policy
[RUN-1559] Rename workload-auto-import policy to workload-events
2 parents 1843bc1 + cb24a47 commit 513a728

File tree

7 files changed

+20
-20
lines changed

7 files changed

+20
-20
lines changed

src/common/policy.ts

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3,31 +3,31 @@ import { resolve as resolvePath } from 'path';
33
import { promisify } from 'util';
44

55
import { logger } from './logger';
6-
import { constructWorkloadAutoImportPolicy } from '../transmitter/payload';
7-
import { sendWorkloadAutoImportPolicy } from '../transmitter';
6+
import { constructWorkloadEventsPolicy } from '../transmitter/payload';
7+
import { sendWorkloadEventsPolicy } from '../transmitter';
88
import { config } from './config';
99

1010
const readFileAsync = promisify(readFile);
1111

12-
export async function loadAndSendWorkloadAutoImportPolicy(): Promise<void> {
12+
export async function loadAndSendWorkloadEventsPolicy(): Promise<void> {
1313
try {
1414
/** This path is set in snyk-monitor during installation/deployment and is defined in the Helm chart. */
1515
const userProvidedRegoPolicyPath = resolvePath(
1616
config.POLICIES_STORAGE_ROOT,
17-
'workload-auto-import.rego',
17+
'workload-events.rego',
1818
);
1919
if (!existsSync(userProvidedRegoPolicyPath)) {
2020
logger.info({}, 'Rego policy file does not exist, skipping loading');
2121
return;
2222
}
2323

2424
const regoPolicy = await readFileAsync(userProvidedRegoPolicyPath, 'utf8');
25-
const payload = constructWorkloadAutoImportPolicy(regoPolicy);
26-
await sendWorkloadAutoImportPolicy(payload);
25+
const payload = constructWorkloadEventsPolicy(regoPolicy);
26+
await sendWorkloadEventsPolicy(payload);
2727
} catch (err) {
2828
logger.error(
2929
{ err },
30-
'Unexpected error occurred while loading workload auto-import policy',
30+
'Unexpected error occurred while loading workload events policy',
3131
);
3232
}
3333
}

src/index.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import { config } from './common/config';
77
import { logger } from './common/logger';
88
import { currentClusterName } from './supervisor/cluster';
99
import { beginWatchingWorkloads } from './supervisor/watchers';
10-
import { loadAndSendWorkloadAutoImportPolicy } from './common/policy';
10+
import { loadAndSendWorkloadEventsPolicy } from './common/policy';
1111

1212
process.on('uncaughtException', (err) => {
1313
if (state.shutdownInProgress) {
@@ -62,6 +62,6 @@ cleanUpTempStorage();
6262

6363
// Allow running in an async context
6464
setImmediate(async function setUpAndMonitor(): Promise<void> {
65-
await loadAndSendWorkloadAutoImportPolicy();
65+
await loadAndSendWorkloadEventsPolicy();
6666
await monitor();
6767
});

src/transmitter/index.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ import {
1515
IRequestError,
1616
ScanResultsPayload,
1717
IDependencyGraphPayload,
18-
WorkloadAutoImportPolicyPayload,
18+
WorkloadEventsPolicyPayload,
1919
} from './types';
2020
import { getProxyAgent } from './proxy';
2121

@@ -147,8 +147,8 @@ export async function sendWorkloadMetadata(
147147
}
148148
}
149149

150-
export async function sendWorkloadAutoImportPolicy(
151-
payload: WorkloadAutoImportPolicyPayload,
150+
export async function sendWorkloadEventsPolicy(
151+
payload: WorkloadEventsPolicyPayload,
152152
): Promise<void> {
153153
try {
154154
logger.info(

src/transmitter/payload.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ import {
1212
IKubernetesMonitorMetadata,
1313
ScanResultsPayload,
1414
IDependencyGraphPayload,
15-
WorkloadAutoImportPolicyPayload,
15+
WorkloadEventsPolicyPayload,
1616
} from './types';
1717

1818
export function constructDepGraph(
@@ -135,9 +135,9 @@ export function constructDeleteWorkload(
135135
};
136136
}
137137

138-
export function constructWorkloadAutoImportPolicy(
138+
export function constructWorkloadEventsPolicy(
139139
policy: string,
140-
): WorkloadAutoImportPolicyPayload {
140+
): WorkloadEventsPolicyPayload {
141141
return {
142142
policy,
143143
userLocator: config.INTEGRATION_ID,

src/transmitter/types.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ export interface IDeleteWorkloadPayload {
6363
agentId: string;
6464
}
6565

66-
export interface WorkloadAutoImportPolicyPayload {
66+
export interface WorkloadEventsPolicyPayload {
6767
userLocator: string;
6868
cluster: string;
6969
agentId: string;
File renamed without changes.

test/system/kind.spec.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -81,15 +81,15 @@ test('Kubernetes-Monitor with KinD', async (jestDoneCallback) => {
8181

8282
// Create a copy of the policy file fixture in the location that snyk-monitor is expecting to load it from.
8383
const regoPolicyFixturePath = resolvePath(
84-
'./test/fixtures/workload-auto-import.rego',
84+
'./test/fixtures/workload-events.rego',
8585
);
8686
const expectedPoliciesPath = resolvePath('/tmp/policies');
8787
if (!(await existsAsync(expectedPoliciesPath))) {
8888
await mkdirAsync(expectedPoliciesPath);
8989
}
9090
await copyFileAsync(
9191
regoPolicyFixturePath,
92-
resolvePath(expectedPoliciesPath, 'workload-auto-import.rego'),
92+
resolvePath(expectedPoliciesPath, 'workload-events.rego'),
9393
);
9494

9595
const regoPolicyContents = await readFileAsync(regoPolicyFixturePath, 'utf8');
@@ -98,11 +98,11 @@ test('Kubernetes-Monitor with KinD', async (jestDoneCallback) => {
9898
.times(1)
9999
.reply(
100100
200,
101-
(uri, requestBody: transmitterTypes.WorkloadAutoImportPolicyPayload) => {
101+
(uri, requestBody: transmitterTypes.WorkloadEventsPolicyPayload) => {
102102
try {
103103
expect(
104104
requestBody,
105-
).toEqual<transmitterTypes.WorkloadAutoImportPolicyPayload>({
105+
).toEqual<transmitterTypes.WorkloadEventsPolicyPayload>({
106106
agentId: expect.any(String),
107107
cluster: expect.any(String),
108108
userLocator: expect.any(String),

0 commit comments

Comments
 (0)