Added Basic externs for PNA: Meter, Counter, Hash, etc

[rupesh-chiluka-marvell]

Part of Issue: #1245

[qobilidop]

Is there a way we can test the code?

[antoninbas]

Is there a way we can test the code?

I think the priority should be to add some PNA-specific tests (see #1255 (comment)).

[rupesh-chiluka-marvell]

Is there a way we can test the code?

I think the priority should be to add some PNA-specific tests (see #1255 (comment)).

Yes. Ok.

[qobilidop]

Thanks for adding the tests! Now it looks good from my side.

[qobilidop]

Nit: format to align indentation?

[rupesh-chiluka-marvell]

Aligned it.
tabs are used for indentation in configuration files. Everywhere else, spaces are used.

[qobilidop]

It does feel there could be some common code factored out to test both psa_switch and pna_nic's hash and internet_checksum blocks. But that could probably be left as future work.

[rupesh-chiluka-marvell]

Yes. I was thinking about how to do it. Based on future tests, we can factor out common code.

[antoninbas]

@rupesh-chiluka-marvell please break up this PR. Everything related to the pna-demo-L2-one-table.json test should be in its own PR, which I'd like to review & merge first. Unless I am missing something, this basic end-to-end test of PNA has no dependencies on the externs you want to add as par of this PR.

[rupesh-chiluka-marvell]

@rupesh-chiluka-marvell please break up this PR. Everything related to the pna-demo-L2-one-table.json test should be in its own PR, which I'd like to review & merge first. Unless I am missing something, this basic end-to-end test of PNA has no dependencies on the externs you want to add as par of this PR.

Yes. Ok.

[rupesh-chiluka-marvell]

@rupesh-chiluka-marvell please break up this PR. Everything related to the pna-demo-L2-one-table.json test should be in its own PR, which I'd like to review & merge first. Unless I am missing something, this basic end-to-end test of PNA has no dependencies on the externs you want to add as par of this PR.

Hi @antoninbas, @qobilidop , I created a new PR for the basic test of PNA here: #1262. Please review it.