Jolokia2 skips over 404 read responses

plugins/inputs/jolokia2/gatherer.go

@@ -77,7 +77,12 @@ func (g *Gatherer) gatherPoints(metric Metric, responses []ReadResponse) []point
 	points := make([]point, 0)
 
 	for _, response := range responses {
-		if response.Status != 200 {
+		switch response.Status {
+		case 200:
+			break
+		case 404:
+			continue
+		default:
 			g.accumulator.AddError(fmt.Errorf("Unexpected status in response from target %s: %d",
 				response.RequestTarget, response.Status))
 			continue

plugins/inputs/jolokia2/jolokia_test.go

@@ -266,6 +266,61 @@ func TestJolokia2_ObjectValues(t *testing.T) {
 	})
 }
 
+func TestJolokia2_StatusCodes(t *testing.T) {
+	config := `
+	[[inputs.jolokia2]]
+		urls = ["%s"]
+
+		[[inputs.jolokia2.metric]]
+			name     = "ok"
+			mbean    = "ok"
+
+		[[inputs.jolokia2.metric]]
+			name       = "not_found"
+			mbean      = "not_found"
+
+		[[inputs.jolokia2.metric]]
+			name       = "unknown"
+			mbean      = "unknown"`
+
+	response := `[{
+		"request": {
+			"mbean": "ok",
+			"type": "read"
+		},
+		"value": 1,
+		"status": 200
+	}, {
+		"request": {
+			"mbean": "not_found",
+			"type": "read"
+		},
+		"status": 404
+	}, {
+		"request": {
+			"mbean": "unknown",
+			"type": "read"
+		},
+		"status": 500
+	}]`
+
+	server := setupServer(http.StatusOK, response)
+	defer server.Close()
+	jolokia := setupJolokia(t, fmt.Sprintf(config, server.URL))
+
+	var acc testutil.Accumulator
+	assert.NoError(t, jolokia.Gather(&acc))
+
+	acc.AssertContainsTaggedFields(t, "ok", map[string]interface{}{
+		"value": 1.0,
+	}, map[string]string{
+		"jolokia_agent_url": server.URL,
+	})
+
+	acc.AssertDoesNotContainMeasurement(t, "not_found")
+	acc.AssertDoesNotContainMeasurement(t, "unknown")
+}
+
 func TestJolokia2_TagRenaming(t *testing.T) {
 	config := `
 	[[inputs.jolokia2]]